@pi-unipi/kanboard 3.0.0-alpha.30 → 3.0.0-alpha.31

package/index.ts

@@ -20,19 +20,17 @@ import { Text } from "@earendil-works/pi-tui";
 import {
   MODULES,
   UNIPI_EVENTS,
-  emitEvent,
+  bus,
   getPackageVersion,
   getSettings,
-  getSharedKanboardStatus,
-  getSharedOwnerStatus,
   initUnipiDirs,
   isChildProcess,
   registerCommandRunner,
+  registerCompactionContext,
   registerEvidenceContributor,
   registerMoveHandler,
   registerNudgeProvider,
   registerProgressRenderer,
-  setSharedKanboardStatus,
   appendProgress,
 } from "@pi-unipi/core";
 
@@ -51,6 +49,7 @@ import {
 } from "./src/commands.js";
 import { createWriteGuard } from "./src/guard.js";
 import { NOTICE_ENTRY, flushNotices, NoticeBuffer } from "./src/notice-buffer.js";
+import { kanboardCompactionContext, rearmOnOwnerResume } from "./src/bus-hooks.js";
 import { createKanboardMonitor } from "./src/monitor.js";
 import { createDebugLog } from "./src/debug.js";
 import { createProgressTracker, registerProgressReminders, ANTI_POISONING_SUFFIX } from "./src/reminders.js";
@@ -67,9 +66,11 @@ import {
 const VERSION = getPackageVersion(dirname(fileURLToPath(import.meta.url)));
 
 /** Utility owns the frozen judged set, so kanboard asks it to reveal the skill. */
-export const SKILL_REVEAL_EVENT = "unipi:skills:reveal";
 export const KANBOARD_SKILL = "kanboard";
 
+/** Last non-null CLI prefix (`<bin> --actor agent --project <slug>`); survives CLI re-attach gaps (UNI-123d). */
+let lastCliPrefix: string | null = null;
+
 export default function (pi: ExtensionAPI) {
   registerMoveHandler(kanboardMoveHandler);
   // One session id shared by the agent's bash calls (refined at session_start
@@ -104,6 +105,12 @@ export default function (pi: ExtensionAPI) {
     }
   };
 
+  /** Full CLI prefix for board writes, or null while the CLI/slug is unknown. */
+  const currentCliPrefix = (): string | null => {
+    const slug = projectSlug();
+    return cli && slug ? `${cli.binary.path} --actor agent --project ${slug}` : null;
+  };
+
   // ── board reads (lead monitor/evidence/holder/notice) ────────────────
   const listTasks = async (args: string[] = []): Promise<KanboardTask[]> => {
     const client = cli;
@@ -133,7 +140,14 @@ export default function (pi: ExtensionAPI) {
     list: () => listTasks(),
     listReady: () => listTasks(["--ready"]),
     session: sessionId,
-    ownerStatus: () => getSharedOwnerStatus(),
+    // The long-horizon owner lifecycle rides the bus's sticky LH_STATE now;
+    // the monitor's deps shape is unchanged.
+    ownerStatus: () => {
+      const s = bus.get(UNIPI_EVENTS.LH_STATE);
+      return s
+        ? { ...(s.owner ? { owner: s.owner } : {}), ...(s.lastStop ? { lastStop: s.lastStop } : {}) }
+        : undefined;
+    },
     now: () => Date.now(),
     // User-only: only QUEUE here — a sendMessage custom message would be
     // converted to a user-role LLM message (never what a notice wants).
@@ -148,15 +162,17 @@ export default function (pi: ExtensionAPI) {
       void refreshHolder();
     },
     cliPrefix: () => {
-      const slug = projectSlug();
-      return cli && slug ? `${cli.binary.path} --actor agent --project ${slug}` : null;
+      // UNI-123d: before the CLI attaches, fall back to the last known prefix
+      // so nudge text carries the real --actor/--project invocation.
+      const current = currentCliPrefix();
+      return current ?? lastCliPrefix;
     },
     debug,
   });
 
   const refreshHolder = async (): Promise<void> => {
     try {
-      setSharedKanboardStatus({
+      bus.emit(UNIPI_EVENTS.KANBOARD_STATUS, {
         claims: (await ownClaims()).map((task) => task.id),
         autowork: guard.remaining().autowork,
       });
@@ -171,6 +187,19 @@ export default function (pi: ExtensionAPI) {
     } catch {
       // Registration must never block module load.
     }
+    // UNI-123a: the board contract survives compaction — the reminder rides
+    // every summary while claims are open.
+    try {
+      registerCompactionContext("kanboard", kanboardCompactionContext);
+    } catch {
+      // Registration must never block module load.
+    }
+    // UNI-123c: a resumed owner re-arms the settle monitor (pause disarmed it).
+    try {
+      rearmOnOwnerResume(pi, monitor);
+    } catch {
+      // Registration must never block module load.
+    }
     try {
       registerEvidenceContributor("kanboard", async () => ({
         blocking: (await ownClaims()).map(
@@ -208,7 +237,7 @@ export default function (pi: ExtensionAPI) {
 
   const revealSkill = (ctx: ExtensionContext | { cwd?: string }): void => {
     // Append-only reveal (never the system prompt), so the prefix cache holds.
-    emitEvent(pi, SKILL_REVEAL_EVENT, { names: [KANBOARD_SKILL], ctx });
+    bus.emit(UNIPI_EVENTS.SKILLS_REVEAL, { names: [KANBOARD_SKILL], ctx });
     debug(`reveal requested for ${KANBOARD_SKILL}`);
   };
 
@@ -348,7 +377,7 @@ export default function (pi: ExtensionAPI) {
       const deps = buildDeps();
       if (deps.cli) void syncPiRuntime(deps, ctx as unknown as ExtensionContext);
     }
-    emitEvent(pi, UNIPI_EVENTS.MODULE_READY, {
+    bus.emit(UNIPI_EVENTS.MODULE_READY, {
       name: MODULES.KANBOARD,
       version: VERSION,
       commands: ["kanboard"],
@@ -449,7 +478,12 @@ export default function (pi: ExtensionAPI) {
           .then(() => debug(`released ${task.id} at shutdown`))
           .catch((error) => debug(`shutdown release ${task.id} failed: ${error instanceof Error ? error.message : String(error)}`));
       }
-      setSharedKanboardStatus({ claims: [], autowork: guard.remaining().autowork });
+      const shutdownPrefix = currentCliPrefix() ?? lastCliPrefix;
+      bus.emit(UNIPI_EVENTS.KANBOARD_STATUS, {
+        claims: [],
+        autowork: guard.remaining().autowork,
+        ...(shutdownPrefix ? { cli: shutdownPrefix } : {}),
+      });
     } catch {
       // Shutdown must never hang the session teardown.
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/kanboard",
-  "version": "3.0.0-alpha.30",
+  "version": "3.0.0-alpha.31",
   "description": "Kanboard v3 — deferred-work board: CLI/daemon bridge, task runner and skill",
   "type": "module",
   "main": "index.ts",
@@ -31,7 +31,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@pi-unipi/core": "3.0.0-alpha.30"
+    "@pi-unipi/core": "3.0.0-alpha.31"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.87.1",
@@ -48,10 +48,10 @@
     "themes": []
   },
   "optionalDependencies": {
-    "@pi-unipi/kanboard-linux-x64": "3.0.0-alpha.30",
-    "@pi-unipi/kanboard-linux-arm64": "3.0.0-alpha.30",
-    "@pi-unipi/kanboard-darwin-x64": "3.0.0-alpha.30",
-    "@pi-unipi/kanboard-darwin-arm64": "3.0.0-alpha.30",
-    "@pi-unipi/kanboard-win32-x64": "3.0.0-alpha.30"
+    "@pi-unipi/kanboard-linux-x64": "3.0.0-alpha.31",
+    "@pi-unipi/kanboard-linux-arm64": "3.0.0-alpha.31",
+    "@pi-unipi/kanboard-darwin-x64": "3.0.0-alpha.31",
+    "@pi-unipi/kanboard-darwin-arm64": "3.0.0-alpha.31",
+    "@pi-unipi/kanboard-win32-x64": "3.0.0-alpha.31"
   }
 }

package/src/bus-hooks.ts

@@ -0,0 +1,28 @@
+/**
+ * kanboard ⇄ bus glue for the LEAD process (UNI-123).
+ *
+ * - kanboardCompactionContext: the board contract (finish / block / show for
+ *   this session's claims) rides every compaction summary, so the agent keeps
+ *   its duties after compaction summarizes the /unipi:kanboard-do message away.
+ * - rearmOnOwnerResume: a goal pause disarms the settle monitor; when the
+ *   parked long-horizon owner resumes, the monitor must come back.
+ */
+
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import { bus, kanboardClaimsReminder, UNIPI_EVENTS } from "@pi-unipi/core";
+
+/** Compaction-context provider: the board contract for this session's claims. */
+export function kanboardCompactionContext(): string | null {
+	return kanboardClaimsReminder(bus.get(UNIPI_EVENTS.KANBOARD_STATUS));
+}
+
+interface Armable {
+	arm(reason?: string): void;
+}
+
+/** Re-arm the settle monitor when a parked owner resumes. Returns unsubscribe. */
+export function rearmOnOwnerResume(pi: Pick<ExtensionAPI, "on">, monitor: Armable): () => void {
+	return bus.on(pi, UNIPI_EVENTS.LONG_HORIZON_OWNER_CHANGED, (e) => {
+		if (e.event === "resumed") monitor.arm("owner resumed");
+	});
+}