test(plugins): fix sync register call sites

2026-05-06 05:30:42 +00:00 · 2026-04-17 08:53:08 +05:30
parent e8fd148437
commit 5c1d6feb33
10 changed files with 42 additions and 43 deletions
--- a/extensions/active-memory/index.test.ts
+++ b/extensions/active-memory/index.test.ts
@@ -119,7 +119,7 @@ describe("active-memory plugin", () => {
    runEmbeddedPiAgent.mockResolvedValue({
      payloads: [{ text: "- lemon pepper wings\n- blue cheese" }],
    });
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
  });

  afterEach(async () => {
@@ -425,7 +425,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      allowedChatTypes: ["direct", "group"],
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    const result = await hooks.before_prompt_build(
      { prompt: "what wings should we order?", messages: [] },
@@ -513,7 +513,7 @@ describe("active-memory plugin", () => {
        searchMode: "inherit",
      },
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -602,7 +602,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      queryMode: "message",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -630,7 +630,7 @@ describe("active-memory plugin", () => {
      queryMode: "message",
      promptStyle: "preference-only",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -675,7 +675,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      thinking: "medium",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -701,7 +701,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      promptAppend: "Prefer stable long-term preferences over one-off events.",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -730,7 +730,7 @@ describe("active-memory plugin", () => {
      promptOverride: "Custom memory prompt. Return NONE or one user fact.",
      promptAppend: "Extra custom instruction.",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -802,7 +802,7 @@ describe("active-memory plugin", () => {
    api.pluginConfig = {
      agents: ["main"],
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      { prompt: "what wings should i order? temp transcript", messages: [] },
@@ -828,7 +828,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      modelFallbackPolicy: "resolved-only",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    const result = await hooks.before_prompt_build(
      { prompt: "what wings should i order? no fallback", messages: [] },
@@ -851,7 +851,7 @@ describe("active-memory plugin", () => {
      modelFallback: "google/gemini-3-flash",
      modelFallbackPolicy: "default-remote",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      { prompt: "what wings should i order? custom fallback", messages: [] },
@@ -878,7 +878,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      modelFallbackPolicy: "default-remote",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    const result = await hooks.before_prompt_build(
      { prompt: "what wings should i order? built-in fallback", messages: [] },
@@ -1027,7 +1027,7 @@ describe("active-memory plugin", () => {
      timeoutMs: 250,
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
    let lastAbortSignal: AbortSignal | undefined;
    runEmbeddedPiAgent.mockImplementation(async (params: { abortSignal?: AbortSignal }) => {
      lastAbortSignal = params.abortSignal;
@@ -1073,7 +1073,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      { prompt: "what wings should i order? session id cache", messages: [] },
@@ -1107,7 +1107,7 @@ describe("active-memory plugin", () => {
      timeoutMs: 250,
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
    runEmbeddedPiAgent.mockImplementationOnce(async (params: { timeoutMs?: number }) => {
      await new Promise((resolve) => setTimeout(resolve, (params.timeoutMs ?? 0) + 25));
      return {
@@ -1145,7 +1145,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      { prompt: "what wings should i order? log sanitization", messages: [] },
@@ -1179,7 +1179,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
    const hugeSession = `agent:main:${"x".repeat(500)}`;

    await hooks.before_prompt_build(
@@ -1423,7 +1423,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      queryMode: "message",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -1451,7 +1451,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      queryMode: "full",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -1482,7 +1482,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      queryMode: "recent",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -1536,7 +1536,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      queryMode: "recent",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -1578,7 +1578,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      queryMode: "recent",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -1611,7 +1611,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      queryMode: "recent",
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      {
@@ -1619,8 +1619,7 @@ describe("active-memory plugin", () => {
        messages: [
          {
            role: "user",
-            content:
-              "Active Memory: I really do want you to remember that I prefer aisle seats.",
+            content: "Active Memory: I really do want you to remember that I prefer aisle seats.",
          },
          {
            role: "user",
@@ -1674,7 +1673,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      maxSummaryChars: 40,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
    runEmbeddedPiAgent.mockResolvedValueOnce({
      payloads: [
        {
@@ -1708,7 +1707,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      maxSummaryChars: 90,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    await hooks.before_prompt_build(
      { prompt: "what wings should i order? prompt-count-check", messages: [] },
@@ -1758,7 +1757,7 @@ describe("active-memory plugin", () => {
      transcriptDir: "active-memory-subagents",
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
    const mkdirSpy = vi.spyOn(fs, "mkdir").mockResolvedValue(undefined);
    const mkdtempSpy = vi.spyOn(fs, "mkdtemp");
    const rmSpy = vi.spyOn(fs, "rm").mockResolvedValue(undefined);
@@ -1802,7 +1801,7 @@ describe("active-memory plugin", () => {
      transcriptDir: "C:/temp/escape",
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
    const mkdirSpy = vi.spyOn(fs, "mkdir").mockResolvedValue(undefined);

    await hooks.before_prompt_build(
@@ -1839,7 +1838,7 @@ describe("active-memory plugin", () => {
      transcriptDir: "active-memory-subagents",
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);
    const mkdirSpy = vi.spyOn(fs, "mkdir").mockResolvedValue(undefined);

    await hooks.before_prompt_build(
@@ -1906,7 +1905,7 @@ describe("active-memory plugin", () => {
      agents: ["main"],
      logging: true,
    };
-    await plugin.register(api as unknown as OpenClawPluginApi);
+    plugin.register(api as unknown as OpenClawPluginApi);

    for (let index = 0; index <= 1000; index += 1) {
      await hooks.before_prompt_build(
--- a/extensions/comfy/comfy.live.test.ts
+++ b/extensions/comfy/comfy.live.test.ts
@@ -42,7 +42,7 @@ describeLive("comfy live", () => {
  beforeAll(async () => {
    cfg = withPluginsEnabled(loadConfig());
    agentDir = resolveOpenClawAgentDir();
-    await plugin.register(
+    plugin.register(
      createTestPluginApi({
        config: cfg as never,
        registerImageGenerationProvider(provider) {
--- a/extensions/device-pair/index.test.ts
+++ b/extensions/device-pair/index.test.ts
@@ -92,7 +92,7 @@ function registerPairCommand(params?: {
  pluginConfig?: Record<string, unknown>;
 }): OpenClawPluginCommandDefinition {
  let command: OpenClawPluginCommandDefinition | undefined;
-  void registerDevicePair.register(
+  registerDevicePair.register(
    createApi({
      ...params,
      registerCommand: (nextCommand) => {
--- a/extensions/memory-wiki/cli-metadata.test.ts
+++ b/extensions/memory-wiki/cli-metadata.test.ts
@@ -51,7 +51,7 @@ describe("memory-wiki cli metadata entry", () => {
    const resolvedConfig = { vaultMode: "bridge", vault: { path: "/vault" } };
    mocks.resolveMemoryWikiConfig.mockReturnValue(resolvedConfig);

-    await plugin.register(api);
+    plugin.register(api);

    const register = registerCli.mock.calls[0]?.[0];

--- a/extensions/memory-wiki/index.test.ts
+++ b/extensions/memory-wiki/index.test.ts
@@ -15,7 +15,7 @@ describe("memory-wiki plugin", () => {
      registerTool,
    } = createPluginApi();

-    await plugin.register(api);
+    plugin.register(api);

    expect(registerMemoryCorpusSupplement).toHaveBeenCalledTimes(1);
    expect(registerMemoryPromptSupplement).toHaveBeenCalledTimes(1);
--- a/extensions/openai/index.test.ts
+++ b/extensions/openai/index.test.ts
@@ -54,7 +54,7 @@ const _registerOpenAIPlugin = async () =>
 async function registerOpenAIPluginWithHook(params?: { pluginConfig?: Record<string, unknown> }) {
  const on = vi.fn();
  const providers: ProviderPlugin[] = [];
-  await plugin.register(
+  plugin.register(
    createTestPluginApi({
      id: "openai",
      name: "OpenAI Provider",
--- a/extensions/phone-control/index.test.ts
+++ b/extensions/phone-control/index.test.ts
@@ -80,7 +80,7 @@ async function withRegisteredPhoneControl(
    });

    let command: OpenClawPluginCommandDefinition | undefined;
-    void registerPhoneControl.register(
+    registerPhoneControl.register(
      createApi({
        stateDir,
        getConfig: () => config,
--- a/extensions/talk-voice/index.test.ts
+++ b/extensions/talk-voice/index.test.ts
@@ -20,7 +20,7 @@ function createHarness(config: Record<string, unknown>) {
      command = definition;
    }),
  };
-  void register.register(api as never);
+  register.register(api as never);
  if (!command) {
    throw new Error("talk-voice command not registered");
  }
--- a/extensions/thread-ownership/index.test.ts
+++ b/extensions/thread-ownership/index.test.ts
@@ -40,8 +40,8 @@ describe("thread-ownership plugin", () => {
  });

  describe("message_sending", () => {
-    beforeEach(async () => {
-      await register.register(api as unknown as OpenClawPluginApi);
+    beforeEach(() => {
+      register.register(api as unknown as OpenClawPluginApi);
    });

    async function sendSlackThreadMessage() {
@@ -112,8 +112,8 @@ describe("thread-ownership plugin", () => {
  });

  describe("message_received @-mention tracking", () => {
-    beforeEach(async () => {
-      await register.register(api as unknown as OpenClawPluginApi);
+    beforeEach(() => {
+      register.register(api as unknown as OpenClawPluginApi);
    });

    it("tracks @-mentions and skips ownership check for mentioned threads", async () => {
--- a/src/plugins/bundled-capability-runtime.ts
+++ b/src/plugins/bundled-capability-runtime.ts
@@ -297,7 +297,7 @@ export function loadBundledCapabilityRuntimeRegistry(params: {

    try {
      const captured = createCapturedPluginRegistration();
-      void register(captured.api);
+      register(captured.api);
      record.cliBackendIds.push(...captured.cliBackends.map((entry) => entry.id));
      record.providerIds.push(...captured.providers.map((entry) => entry.id));
      record.speechProviderIds.push(...captured.speechProviders.map((entry) => entry.id));