@pi-archimedes/notify 2.3.0 → 2.4.0

package/README.md

@@ -33,11 +33,12 @@ When the agent finishes a task (`agent_end`) or a question is asked (`ASK_REQUES
 
 | Setting | Type | Default | Description |
 |---------|------|---------|-------------|
-| `enabled` | bool | `true` | Enable desktop notifications |
 | `notifyOnAgentEnd` | bool | `true` | Notify when agent finishes a task |
 | `notifyOnQuestion` | bool | `true` | Notify when a question needs your answer |
 | `delayMs` | number | `30000` | Milliseconds to wait before sending notification (default 30 seconds) |
 
+On/off is managed by the suite: toggle via `/plugins` (`archimedes.notify.enabled`, default on).
+
 Settings are stored in `~/.pi/agent/settings.json` under the `archimedes.notify` namespace.
 
 ## Terminal compatibility

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-archimedes/notify",
-  "version": "2.3.0",
+  "version": "2.4.0",
   "type": "module",
   "keywords": [
     "pi-package"
@@ -14,7 +14,7 @@
     ".": "./src/index.ts"
   },
   "dependencies": {
-    "@pi-archimedes/core": "2.3.0"
+    "@pi-archimedes/core": "2.4.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": ">=0.1.0",

package/src/default-export.test.ts

@@ -0,0 +1,25 @@
+import { describe, expect, it, vi } from "vitest";
+import defaultExport from "./index.js";
+
+describe("notify default export (standalone factory)", () => {
+	it("is a function that registers the notify handlers (registerNotify subscribes its own session events)", () => {
+		expect(typeof defaultExport).toBe("function");
+
+		const on = vi.fn();
+		const pi = { on } as any;
+
+		defaultExport(pi);
+
+		const events = on.mock.calls.map((c: Array<unknown>) => c[0]);
+		expect(events).toEqual(
+			expect.arrayContaining([
+				"agent_end",
+				"input",
+				"before_agent_start",
+				"agent_start",
+				"session_start",
+				"session_shutdown",
+			]),
+		);
+	});
+});

package/src/index.ts

@@ -15,14 +15,14 @@ type TriggerType = (typeof TRIGGER)[keyof typeof TRIGGER];
 // ── Config ──────────────────────────────────────────────────────────────────
 
 export interface NotifyConfig {
-  enabled: boolean;
+  // suite-managed by meta's plugin gate (archimedes.notify.enabled — see ADR 0012); notify never reads this
+  enabled?: boolean;
   notifyOnAgentEnd: boolean;
   notifyOnQuestion: boolean;
   delayMs: number;
 }
 
 export const DEFAULT_NOTIFY_CONFIG: NotifyConfig = {
-  enabled: true,
   notifyOnAgentEnd: true,
   notifyOnQuestion: true,
   delayMs: 30_000,
@@ -153,10 +153,6 @@ export function scheduleNotify(trigger: TriggerType): void {
   // Load config fresh on each trigger
   const config = loadNotifyConfig();
 
-  if (!config.enabled) {
-    return;
-  }
-
   if (trigger === TRIGGER.AGENT_END && !config.notifyOnAgentEnd) {
     return;
   }
@@ -213,18 +209,19 @@ export function registerNotify(pi: ExtensionAPI): void {
   });
 }
 
+// ── Default export (for standalone pi.extensions loading) ─────────────────
+
+// registerNotify subscribes its own session_start/session_shutdown (and
+// agent/input) handlers internally, so the default factory just registers it.
+export default function (pi: ExtensionAPI): void {
+  registerNotify(pi);
+}
+
 // ── Settings UI ─────────────────────────────────────────────────────────────
 
 /** Build settings UI items for the notify package. */
 export function getNotifySettingsItems(config: NotifyConfig): SettingItem[] {
   return [
-    {
-      id: "enabled",
-      label: "Notifications",
-      description: "Enable desktop notifications",
-      currentValue: config.enabled ? "On" : "Off",
-      values: ["On", "Off"],
-    },
     {
       id: "notifyOnAgentEnd",
       label: "Notify on task complete",