@vellumai/assistant 0.4.43 → 0.4.45

package/src/__tests__/skills.test.ts

@@ -23,7 +23,6 @@ mock.module("../util/platform.js", () => ({
   getSandboxWorkingDir: () => TEST_DIR,
   getInterfacesDir: () => join(TEST_DIR, "interfaces"),
   ensureDataDir: () => {},
-  getSocketPath: () => join(TEST_DIR, "vellum.sock"),
   getPidPath: () => join(TEST_DIR, "vellum.pid"),
   getDbPath: () => join(TEST_DIR, "data", "assistant.db"),
   getLogPath: () => join(TEST_DIR, "logs", "vellum.log"),
@@ -621,56 +620,14 @@ describe("bundled browser skill", () => {
   });
 });
 
-describe("bundled public-ingress skill", () => {
-  beforeEach(() => {
-    mkdirSync(join(TEST_DIR, "skills"), { recursive: true });
-  });
-
-  afterEach(() => {
-    if (existsSync(TEST_DIR)) {
-      rmSync(TEST_DIR, { recursive: true, force: true });
-    }
-  });
-
-  test("public-ingress skill appears in full catalog (including bundled)", () => {
-    const catalog = loadSkillCatalog();
-    const skill = catalog.find((s) => s.id === "public-ingress");
-    expect(skill).toBeDefined();
-    expect(skill!.name).toBe("public-ingress");
-    expect(skill!.displayName).toBe("Public Ingress");
-    expect(skill!.bundled).toBe(true);
-  });
-
-  test("public-ingress skill has correct description", () => {
-    const catalog = loadSkillCatalog();
-    const skill = catalog.find((s) => s.id === "public-ingress");
-    expect(skill).toBeDefined();
-    expect(skill!.description).toContain("ngrok");
-    expect(skill!.description).toContain("ingress.publicBaseUrl");
-  });
-
-  test("public-ingress skill is user-invocable", () => {
-    const catalog = loadSkillCatalog();
-    const skill = catalog.find((s) => s.id === "public-ingress");
-    expect(skill).toBeDefined();
-    expect(skill!.userInvocable).toBe(true);
-  });
-
-  test("public-ingress skill has no tool manifest (instructions-only)", () => {
-    const catalog = loadSkillCatalog();
-    const skill = catalog.find((s) => s.id === "public-ingress");
-    expect(skill).toBeDefined();
-    expect(skill!.toolManifest).toBeUndefined();
-  });
-});
-
 describe("ingress-dependent setup skills declare public-ingress", () => {
   const FRONTMATTER_REGEX = /^---\r?\n([\s\S]*?)\r?\n---(?:\r?\n|$)/;
-  const BUNDLED_SKILLS_DIR = join(
+  const FIRST_PARTY_SKILLS_DIR = join(
     import.meta.dir,
     "..",
-    "config",
-    "bundled-skills",
+    "..",
+    "..",
+    "skills",
   );
 
   function readSkillIncludes(
@@ -710,14 +667,17 @@ describe("ingress-dependent setup skills declare public-ingress", () => {
   }
 
   test("telegram-setup includes public-ingress", () => {
-    const includes = readSkillIncludes(BUNDLED_SKILLS_DIR, "telegram-setup");
+    const includes = readSkillIncludes(
+      FIRST_PARTY_SKILLS_DIR,
+      "telegram-setup",
+    );
     expect(includes).toBeDefined();
     expect(includes).toContain("public-ingress");
   });
 
   test("google-oauth-setup includes public-ingress", () => {
     const includes = readSkillIncludes(
-      BUNDLED_SKILLS_DIR,
+      FIRST_PARTY_SKILLS_DIR,
       "google-oauth-setup",
     );
     expect(includes).toBeDefined();
@@ -725,7 +685,10 @@ describe("ingress-dependent setup skills declare public-ingress", () => {
   });
 
   test("slack-oauth-setup includes browser", () => {
-    const includes = readSkillIncludes(BUNDLED_SKILLS_DIR, "slack-oauth-setup");
+    const includes = readSkillIncludes(
+      FIRST_PARTY_SKILLS_DIR,
+      "slack-oauth-setup",
+    );
     expect(includes).toBeDefined();
     expect(includes).toContain("browser");
   });
@@ -772,11 +735,9 @@ describe("bundled computer-use skill", () => {
     expect(cuSkill!.toolManifest).toBeDefined();
     expect(cuSkill!.toolManifest!.present).toBe(true);
     expect(cuSkill!.toolManifest!.valid).toBe(true);
-    expect(cuSkill!.toolManifest!.toolCount).toBe(12);
+    expect(cuSkill!.toolManifest!.toolCount).toBe(10);
     expect(cuSkill!.toolManifest!.toolNames).toEqual([
       "computer_use_click",
-      "computer_use_double_click",
-      "computer_use_right_click",
       "computer_use_type_text",
       "computer_use_key",
       "computer_use_scroll",

package/src/__tests__/slack-channel-config.test.ts

@@ -61,7 +61,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/slack-inbound-verification.test.ts

@@ -25,7 +25,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/slack-skill.test.ts

@@ -129,7 +129,7 @@ describe("slack skill SKILL.md", () => {
   });
 
   test("is user-invocable", () => {
-    expect(skillMd).toContain('"user-invocable":true');
+    expect(skillMd).toContain("user-invocable: true");
   });
 
   test("mentions privacy rules", () => {

package/src/__tests__/starter-task-flow.test.ts

@@ -4,30 +4,20 @@ import type { ServerMessage, SurfaceType } from "../daemon/message-protocol.js";
 
 mock.module("../memory/app-store.js", () => ({
   getApp: (id: string) => {
-    if (id !== "home-base-app") return null;
+    if (id !== "test-app") return null;
     return {
       id,
-      name: "Home Base",
-      htmlDefinition:
-        '<main id="home-base-root" data-vellum-home-base="v1"></main>',
+      name: "Test App",
+      description: "A test app",
+      htmlDefinition: "<main>Test App</main>",
     };
   },
+  getAppPreview: () => null,
   updateApp: () => {
     throw new Error("updateApp should not be called in this test");
   },
 }));
 
-mock.module("../home-base/prebuilt/seed.js", () => ({
-  findSeededHomeBaseApp: () => ({ id: "home-base-app" }),
-  getPrebuiltHomeBasePreview: () => ({
-    title: "Home Base",
-    subtitle: "Dashboard",
-    description: "Preview",
-    icon: "🏠",
-    metrics: [{ label: "Starter tasks", value: "3" }],
-  }),
-}));
-
 import {
   createSurfaceMutex,
   handleSurfaceAction,
@@ -70,12 +60,12 @@ describe("starter task surface actions", () => {
     ctx.pendingSurfaceActions.set("surf-1", { surfaceType: "dynamic_page" });
 
     handleSurfaceAction(ctx, "surf-1", "relay_prompt", {
-      prompt: "Help me customize Home Base with a calmer palette.",
+      prompt: "Help me customize the app with a calmer palette.",
       task: "change_look_and_feel",
     });
 
     expect(forwarded).toEqual([
-      "Help me customize Home Base with a calmer palette.",
+      "Help me customize the app with a calmer palette.",
     ]);
     expect(ctx.pendingSurfaceActions.has("surf-1")).toBe(true);
   });
@@ -133,7 +123,7 @@ describe("starter task surface actions", () => {
     ctx.sendToClient = (msg) => sent.push(msg);
 
     const result = await surfaceProxyResolver(ctx, "app_open", {
-      app_id: "home-base-app",
+      app_id: "test-app",
     });
 
     expect(result.isError).toBe(false);
@@ -141,7 +131,7 @@ describe("starter task surface actions", () => {
       surfaceId: string;
       appId: string;
     };
-    expect(parsed.appId).toBe("home-base-app");
+    expect(parsed.appId).toBe("test-app");
     expect(ctx.pendingSurfaceActions.get(parsed.surfaceId)?.surfaceType).toBe(
       "dynamic_page",
     );

package/src/__tests__/subagent-tools.test.ts

@@ -91,7 +91,7 @@ function injectSubagent(
     messages: [],
     sendToClient: () => {},
     usageStats: { inputTokens: 0, outputTokens: 0, estimatedCost: 0 },
-    enqueueMessage: () => ({ queued: false, rejected: false }),
+    enqueueMessage: () => ({ queued: false }),
     persistUserMessage: () => "msg-1",
     runAgentLoop: async () => {},
   };
@@ -160,7 +160,7 @@ describe("Subagent tool execute validation", () => {
       makeContext("sess-1"),
     );
     expect(result.isError).toBe(true);
-    expect(result.content).toContain("No IPC client");
+    expect(result.content).toContain("No client connected");
   });
 
   test("spawn returns error when missing label", async () => {

package/src/__tests__/system-prompt.test.ts

@@ -26,7 +26,6 @@ mock.module("../util/platform.js", () => ({
   getWorkspaceHooksDir: () => join(TEST_DIR, "hooks"),
   getWorkspacePromptPath: (file: string) => join(TEST_DIR, file),
   ensureDataDir: () => {},
-  getSocketPath: () => join(TEST_DIR, "vellum.sock"),
   getPidPath: () => join(TEST_DIR, "vellum.pid"),
   getDbPath: () => join(TEST_DIR, "data", "assistant.db"),
   getLogPath: () => join(TEST_DIR, "logs", "vellum.log"),
@@ -192,7 +191,9 @@ describe("buildSystemPrompt", () => {
     expect(result).toContain('id="release-checklist"');
     expect(result).toContain('name="Release Checklist"');
     expect(result).toContain('description="Deployment checks."');
-    expect(result).toContain("call the `skill_load` tool with its `id`");
+    expect(result).toContain(
+      "call `skill_load` to load the full instructions, then use `skill_execute` to invoke the skill's tools.",
+    );
   });
 
   test("keeps SOUL.md and IDENTITY.md additive with skills", () => {
@@ -267,7 +268,7 @@ describe("buildSystemPrompt", () => {
   test("includes memory persistence section", () => {
     const result = buildSystemPrompt();
     expect(result).toContain("## Memory Persistence");
-    expect(result).toContain("memory_save");
+    expect(result).toContain("memory_manage");
     expect(result).toContain("Saved > unsaved. Always.");
   });
 
@@ -320,13 +321,12 @@ describe("buildSystemPrompt", () => {
       expect(result).not.toContain("use `app_update` to change the HTML");
     });
 
-    test("onboarding playbook uses app_file_edit for accent color, not app_update", () => {
+    test("onboarding playbook does not reference Home Base for accent color", () => {
       // Starter task playbooks only included during onboarding (BOOTSTRAP.md exists)
       writeFileSync(join(TEST_DIR, "BOOTSTRAP.md"), "# First run");
       const result = buildSystemPrompt();
-      expect(result).toContain(
-        "using `app_file_edit` to update the theme styles",
-      );
+      // The make_it_yours playbook should not reference Home Base anymore
+      expect(result).not.toContain("Home Base dashboard");
       expect(result).not.toContain(
         "using `app_update` to regenerate the Home Base HTML",
       );

package/src/__tests__/task-compiler.test.ts

@@ -11,7 +11,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/task-management-tools.test.ts

@@ -10,7 +10,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/task-memory-cleanup.test.ts

@@ -22,7 +22,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/task-runner.test.ts

@@ -10,7 +10,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/task-scheduler.test.ts

@@ -10,7 +10,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/terminal-tools.test.ts

@@ -25,7 +25,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testTmpDir, "test.sock"),
   getPidPath: () => join(testTmpDir, "test.pid"),
   getDbPath: () => join(testTmpDir, "test.db"),
   getLogPath: () => join(testTmpDir, "test.log"),

package/src/__tests__/test-support/computer-use-skill-harness.ts

@@ -2,11 +2,9 @@
  * Reusable constants and helpers for the computer-use skill migration test suite.
  */
 
-/** The 12 computer_use_* action tool names provided by the bundled computer-use skill. */
+/** The 10 computer_use_* action tool names provided by the bundled computer-use skill. */
 export const COMPUTER_USE_TOOL_NAMES = [
   "computer_use_click",
-  "computer_use_double_click",
-  "computer_use_right_click",
   "computer_use_type_text",
   "computer_use_key",
   "computer_use_scroll",
@@ -22,7 +20,7 @@ export const COMPUTER_USE_TOOL_NAMES = [
 export const COMPUTER_USE_SKILL_ID = "computer-use";
 
 /** Number of computer_use_* tools. */
-export const COMPUTER_USE_TOOL_COUNT = COMPUTER_USE_TOOL_NAMES.length; // 12
+export const COMPUTER_USE_TOOL_COUNT = COMPUTER_USE_TOOL_NAMES.length; // 10
 
 import { expect } from "bun:test";
 

package/src/__tests__/thread-seed-composer.test.ts

@@ -398,14 +398,14 @@ describe("composeThreadSeed", () => {
 
   describe("empty copy fallback", () => {
     test("falls back to event name when both title and body are empty", () => {
-      const signal = makeSignal({ sourceEventName: "reminder.fired" });
+      const signal = makeSignal({ sourceEventName: "schedule.notify" });
       const copy = makeCopy({ title: "", body: "" });
       const seed = composeThreadSeed(
         signal,
         "vellum" as NotificationChannel,
         copy,
       );
-      expect(seed).toBe("Reminder fired");
+      expect(seed).toBe("Schedule notify");
     });
 
     test('falls back to event name when title is "Notification" and body is empty', () => {
@@ -421,7 +421,7 @@ describe("composeThreadSeed", () => {
 
     test('uses context payload "message" field in fallback when available', () => {
       const signal = makeSignal({
-        sourceEventName: "reminder.fired",
+        sourceEventName: "schedule.notify",
         contextPayload: { message: "Take out the trash" },
       });
       const copy = makeCopy({ title: "", body: "" });
@@ -430,7 +430,7 @@ describe("composeThreadSeed", () => {
         "vellum" as NotificationChannel,
         copy,
       );
-      expect(seed).toBe("Reminder fired: Take out the trash");
+      expect(seed).toBe("Schedule notify: Take out the trash");
     });
 
     test('uses context payload "summary" field in fallback', () => {
@@ -463,7 +463,7 @@ describe("composeThreadSeed", () => {
 
     test("fallback is consistent across rich and compact channels", () => {
       const signal = makeSignal({
-        sourceEventName: "reminder.fired",
+        sourceEventName: "schedule.notify",
         contextPayload: { message: "Call the doctor" },
       });
       const copy = makeCopy({ title: "", body: "" });

package/src/__tests__/tool-approval-handler.test.ts

@@ -10,7 +10,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/tool-execution-abort-cleanup.test.ts

@@ -80,7 +80,6 @@ mock.module("../tools/network/script-proxy/index.js", () => ({
 mock.module("../util/platform.js", () => ({
   getRootDir: () => "/tmp",
   getDataDir: () => "/tmp",
-  getSocketPath: () => "/tmp/vellum.sock",
 }));
 
 mock.module("../tools/credentials/resolve.js", () => ({

package/src/__tests__/tool-execution-pipeline.benchmark.test.ts

@@ -32,7 +32,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/tool-executor.test.ts

@@ -1167,7 +1167,7 @@ describe("isSideEffectTool", () => {
     const readOnlyTools = [
       "file_read",
       "memory_recall",
-      "memory_save",
+      "memory_manage",
       "web_search",
       "browser_snapshot",
       "browser_screenshot",
@@ -1190,44 +1190,6 @@ describe("isSideEffectTool", () => {
   });
 
   describe("action-aware classification for mixed-action tools", () => {
-    test("account_manage create is a side-effect", () => {
-      expect(isSideEffectTool("account_manage", { action: "create" })).toBe(
-        true,
-      );
-    });
-
-    test("account_manage update is a side-effect", () => {
-      expect(isSideEffectTool("account_manage", { action: "update" })).toBe(
-        true,
-      );
-    });
-
-    test("account_manage list is NOT a side-effect", () => {
-      expect(isSideEffectTool("account_manage", { action: "list" })).toBe(
-        false,
-      );
-    });
-
-    test("account_manage get is NOT a side-effect", () => {
-      expect(isSideEffectTool("account_manage", { action: "get" })).toBe(false);
-    });
-
-    test("account_manage without input is NOT a side-effect", () => {
-      expect(isSideEffectTool("account_manage")).toBe(false);
-    });
-
-    test("reminder_create is a side-effect", () => {
-      expect(isSideEffectTool("reminder_create")).toBe(true);
-    });
-
-    test("reminder_cancel is a side-effect", () => {
-      expect(isSideEffectTool("reminder_cancel")).toBe(true);
-    });
-
-    test("reminder_list is NOT a side-effect", () => {
-      expect(isSideEffectTool("reminder_list")).toBe(false);
-    });
-
     test("credential_store store is a side-effect", () => {
       expect(isSideEffectTool("credential_store", { action: "store" })).toBe(
         true,
@@ -1524,15 +1486,6 @@ describe("ToolExecutor forcePromptSideEffects enforcement", () => {
       },
       { name: "document_create", input: { title: "doc", content: "body" } },
       { name: "document_update", input: { id: "doc-1", content: "updated" } },
-      { name: "account_manage", input: { action: "create", name: "acct" } },
-      {
-        name: "reminder_create",
-        input: {
-          fire_at: "2030-01-01T00:00:00Z",
-          label: "test",
-          message: "remind me",
-        },
-      },
       {
         name: "credential_store",
         input: { action: "store", name: "api-key", value: "secret" },
@@ -1828,47 +1781,16 @@ describe("ToolExecutor forcePromptSideEffects enforcement", () => {
     expect(promptCalled).toBe(true);
   });
 
-  // ── Action-aware mixed-action tools (PR fix5) ──────────
-
-  test("account_manage create forces prompt in private thread", async () => {
-    checkResultOverride = { decision: "allow", reason: "Matched trust rule" };
-
-    const executor = new ToolExecutor(makeTrackingPrompter());
-    const result = await executor.execute(
-      "account_manage",
-      { action: "create", name: "test-account" },
-      makeContext({ forcePromptSideEffects: true }),
-    );
-
-    expect(result.isError).toBe(false);
-    expect(promptCalled).toBe(true);
-  });
-
-  test("account_manage list does NOT force prompt in private thread", async () => {
-    checkResultOverride = { decision: "allow", reason: "Matched trust rule" };
-
-    const executor = new ToolExecutor(makeTrackingPrompter());
-    const result = await executor.execute(
-      "account_manage",
-      { action: "list" },
-      makeContext({ forcePromptSideEffects: true }),
-    );
-
-    expect(result.isError).toBe(false);
-    // list is read-only — must NOT trigger forced prompting
-    expect(promptCalled).toBe(false);
-  });
-
-  test("reminder_create forces prompt in private thread", async () => {
+  test("schedule_create forces prompt in private thread", async () => {
     checkResultOverride = { decision: "allow", reason: "Matched trust rule" };
 
     const executor = new ToolExecutor(makeTrackingPrompter());
     const result = await executor.execute(
-      "reminder_create",
+      "schedule_create",
       {
-        fire_at: "2030-01-01T00:00:00Z",
-        label: "test",
-        message: "test reminder",
+        name: "test schedule",
+        expression: "0 9 * * *",
+        message: "test",
       },
       makeContext({ forcePromptSideEffects: true }),
     );
@@ -1877,12 +1799,12 @@ describe("ToolExecutor forcePromptSideEffects enforcement", () => {
     expect(promptCalled).toBe(true);
   });
 
-  test("reminder_list does NOT force prompt in private thread", async () => {
+  test("schedule_list does NOT force prompt in private thread", async () => {
     checkResultOverride = { decision: "allow", reason: "Matched trust rule" };
 
     const executor = new ToolExecutor(makeTrackingPrompter());
     const result = await executor.execute(
-      "reminder_list",
+      "schedule_list",
       {},
       makeContext({ forcePromptSideEffects: true }),
     );

package/src/__tests__/tool-grant-request-escalation.test.ts

@@ -20,7 +20,6 @@ mock.module("../util/platform.js", () => ({
   isMacOS: () => process.platform === "darwin",
   isLinux: () => process.platform === "linux",
   isWindows: () => process.platform === "win32",
-  getSocketPath: () => join(testDir, "test.sock"),
   getPidPath: () => join(testDir, "test.pid"),
   getDbPath: () => join(testDir, "test.db"),
   getLogPath: () => join(testDir, "test.log"),

package/src/__tests__/tool-notification-listener.test.ts

@@ -6,7 +6,7 @@ import type { AssistantDomainEvents } from "../events/domain-events.js";
 import { registerToolNotificationListener } from "../events/tool-notification-listener.js";
 
 describe("registerToolNotificationListener", () => {
-  test("forwards tool.secret.detected events to IPC secret_detected messages", async () => {
+  test("forwards tool.secret.detected events to secret_detected messages", async () => {
     const bus = new EventBus<AssistantDomainEvents>();
     const messages: ServerMessage[] = [];
     registerToolNotificationListener(bus, (msg) => messages.push(msg));