@agent-plan/pi-adapter 0.4.1 → 0.4.2

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,iCAAiC,CAAC;AAm0BtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAy0K9D"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,iCAAiC,CAAC;AAw0BtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CA20K9D"}

package/dist/index.js

@@ -52,6 +52,10 @@ let store = null;
 let server = null;
 let lastKnownWebPort = null;
 let plannerSessionId = "pi:uninitialized";
+/** The planner session id of a Pi host context. One place, so every handler derives it the same way. */
+function piSessionIdOf(ctx) {
+    return `pi:${ctx.sessionManager.getSessionId()}`;
+}
 // True while the agent is mutating .planner/ files. The web server returns 503-busy
 // during this window so the UI doesn't render inconsistent data.
 let planBusy = false;
@@ -314,7 +318,7 @@ async function ensureProjectLanguagePreferences(st, persist = true) {
 async function buildHandoffMarkdown(st, reason, overrides) {
     const [plan, resume, activity] = await Promise.all([
         st.loadAll(),
-        st.refreshResume(),
+        st.refreshResume(undefined, undefined, plannerSessionId),
         st.loadActivityLog(),
     ]);
     const createdAt = nowISO();
@@ -773,7 +777,7 @@ export default function planPiExtension(pi) {
     capturedPi = pi;
     // ── Restore on session start/reload ─────────────────────────────────
     pi.on("session_start", async (_event, ctx) => {
-        plannerSessionId = `pi:${ctx.sessionManager.getSessionId()}`;
+        plannerSessionId = piSessionIdOf(ctx);
         invalidateReads(plannerSessionId);
         startReadSession(plannerSessionId);
         try {
@@ -2248,15 +2252,14 @@ export default function planPiExtension(pi) {
                 // task_recommend tool, so a person at the command line and an agent
                 // calling the tool see identical reasoning, just a compact few-line
                 // notify instead of a structured payload.
-                const [features, phases, project, resume, handoffs, archivedHandoffs] = await Promise.all([
+                const [features, phases, resume, handoffs, archivedHandoffs] = await Promise.all([
                     st.loadFeatures(),
                     st.loadAllPhases(),
-                    st.loadProject(),
-                    st.loadResume(),
+                    st.loadResume(plannerSessionId),
                     st.listHandoffs(),
                     st.listArchivedHandoffs(),
                 ]);
-                const result = recommendNextWork(features.features, phases, project.workDeviations, resume?.currentPhaseId, "", {
+                const result = recommendNextWork(features.features, phases, await st.listAllWorkDeviations(), resume?.currentPhaseId, plannerSessionId, {
                     handoffs: handoffs.map((handoff) => ({
                         compositeRef: handoff.compositeRef,
                         firstLine: handoff.firstLine,
@@ -2738,7 +2741,7 @@ export default function planPiExtension(pi) {
                 const plannerSkill = await st.syncPlannerSkill();
                 await st.syncGrillMeSkill();
                 const srv = server;
-                let recapText = await buildRecap(st, { localUrl: srv?.localUrl, lanUrl: srv?.lanUrl, port: lastKnownWebPort ?? undefined }, { harness: "pi" });
+                let recapText = await buildRecap(st, { localUrl: srv?.localUrl, lanUrl: srv?.lanUrl, port: lastKnownWebPort ?? undefined }, { harness: "pi", sessionId: plannerSessionId });
                 if (preparation.changed)
                     recapText = `${preparation.legacyProjectContext.summary}\n\n${recapText}`;
                 // Explicit planner load must deliver the complete project-level
@@ -3101,7 +3104,7 @@ export default function planPiExtension(pi) {
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
             const st = await requirePlan(ctx);
-            const plannerSessionId = `pi:${ctx.sessionManager.getSessionId()}`;
+            const plannerSessionId = piSessionIdOf(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
             const requirements = await st.loadRequirements();
@@ -3965,7 +3968,7 @@ export default function planPiExtension(pi) {
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
             const st = await requirePlan(ctx);
-            const plannerSessionId = `pi:${ctx.sessionManager.getSessionId()}`;
+            const plannerSessionId = piSessionIdOf(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
             const features = (await st.loadFeatures()).features;
@@ -4324,7 +4327,7 @@ export default function planPiExtension(pi) {
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
             const st = await requirePlan(ctx);
-            const plannerSessionId = `pi:${ctx.sessionManager.getSessionId()}`;
+            const plannerSessionId = piSessionIdOf(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
             const features = (await st.loadFeatures()).features;
@@ -4917,7 +4920,7 @@ export default function planPiExtension(pi) {
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
             const st = await requirePlan(ctx);
-            const plannerSessionId = `pi:${ctx.sessionManager.getSessionId()}`;
+            const plannerSessionId = piSessionIdOf(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
             const features = (await st.loadFeatures()).features;
@@ -4928,7 +4931,7 @@ export default function planPiExtension(pi) {
             const summary = `${found.task.title} — ${formatPhaseRef(found.phase.number, featureNumberOfPhase(found.phase, features))}/T${pad(found.task.number)}${found.task.shortId ? ` · ${found.task.shortId}` : ""} (${found.task.status})`;
             if (!params.full)
                 return { content: [{ type: "text", text: summary }], details: {} };
-            const project = await st.loadProject();
+            const project = await st.loadProject(plannerSessionId);
             const pendingDeviation = found.task.status === "done" || found.task.status === "canceled" || found.task.status === "rejected"
                 ? undefined
                 : project.workDeviations
@@ -5415,18 +5418,18 @@ export default function planPiExtension(pi) {
         description: "Return the next-work chain: active task if present, otherwise the lowest-priority ready feature → phase → task, plus bounded typed claims. Only persisted resume-ready handoffs are actionable; Markdown prose and terminal archives are not authority. Never blocks an explicit user choice.",
         parameters: Type.Object({}),
         async execute(_id, _params, _signal, _onUpdate, ctx) {
+            const plannerSessionId = piSessionIdOf(ctx);
             const st = await requirePlan(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
-            const [features, phases, project, resume, handoffs, archivedHandoffs] = await Promise.all([
+            const [features, phases, resume, handoffs, archivedHandoffs] = await Promise.all([
                 st.loadFeatures(),
                 st.loadAllPhases(),
-                st.loadProject(),
-                st.loadResume(),
+                st.loadResume(plannerSessionId),
                 st.listHandoffs(),
                 st.listArchivedHandoffs(),
             ]);
-            const result = recommendNextWork(features.features, phases, project.workDeviations, resume?.currentPhaseId, "", {
+            const result = recommendNextWork(features.features, phases, await st.listAllWorkDeviations(), resume?.currentPhaseId, plannerSessionId, {
                 handoffs: handoffs.map((handoff) => ({
                     compositeRef: handoff.compositeRef,
                     firstLine: handoff.firstLine,
@@ -5451,14 +5454,15 @@ export default function planPiExtension(pi) {
             reason: Type.String({ description: "Why this approved temporary deviation is needed" }),
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
+            const plannerSessionId = piSessionIdOf(ctx);
             const st = await requirePlan(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
-            const [features, phases, project, focus] = await Promise.all([st.loadFeatures(), st.loadAllPhases(), st.loadProject(), st.loadResume()]);
+            const [features, phases, focus] = await Promise.all([st.loadFeatures(), st.loadAllPhases(), st.loadResume(plannerSessionId)]);
             const temporary = findTaskByRef(phases, features.features, params.temporary_task.trim());
             if (!temporary)
                 return { content: [{ type: "text", text: `Task not found: ${params.temporary_task}` }], details: {} };
-            const selected = recommendNextTask(features.features, phases, project.workDeviations, focus?.currentPhaseId);
+            const selected = recommendNextTask(features.features, phases, await st.listAllWorkDeviations(), focus?.currentPhaseId, plannerSessionId);
             const resume = params.resume_task ? findTaskByRef(phases, features.features, params.resume_task.trim()) : selected.candidate;
             if (!resume)
                 return { content: [{ type: "text", text: `No resume task is available. Provide resume_task explicitly. ${selected.reason}` }], details: selected };
@@ -5473,7 +5477,7 @@ export default function planPiExtension(pi) {
                 requestedBy: "user", approvedBy: "user", state: "approved",
                 createdAt: timestamp, activatedAt: "", resumeRequiredAt: "", resolvedAt: "", resumedAt: "",
             };
-            await st.addWorkDeviation(record);
+            await st.addWorkDeviation(record, plannerSessionId);
             await st.writeGenerated();
             const tempRef = `${formatPhaseRef(temporary.phase.number, featureNumberOfPhase(temporary.phase, features.features))}/T${String(temporary.task.number).padStart(3, "0")}`;
             const resumeRef = `${formatPhaseRef(resume.phase.number, featureNumberOfPhase(resume.phase, features.features))}/T${String(resume.task.number).padStart(3, "0")}`;
@@ -5493,6 +5497,7 @@ export default function planPiExtension(pi) {
             paused_by: Type.Optional(Type.String({ description: "Agent/session identifier" })),
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
+            const plannerSessionId = piSessionIdOf(ctx);
             const st = await requirePlan(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
@@ -5510,7 +5515,7 @@ export default function planPiExtension(pi) {
             invalidateReads(plannerSessionId);
             const checkpointed = await st.pauseTask(found.phase.id, found.task.id, snapshot);
             await st.syncTaskStatusRollup(found.phase.id);
-            const activeDeviation = (await st.loadProject()).workDeviations
+            const activeDeviation = (await st.loadProject(plannerSessionId)).workDeviations
                 .filter((deviation) => (deviation.state === "approved" || deviation.state === "active")
                 && deviation.temporaryTaskId === found.task.id)
                 .sort((left, right) => right.createdAt.localeCompare(left.createdAt))[0];
@@ -5552,10 +5557,10 @@ export default function planPiExtension(pi) {
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
             const st = await requirePlan(ctx);
-            const plannerSessionId = `pi:${ctx.sessionManager.getSessionId()}`;
+            const plannerSessionId = piSessionIdOf(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
-            const [featuresDoc, phases, project, focus] = await Promise.all([st.loadFeatures(), st.loadAllPhases(), st.loadProject(), st.loadResume()]);
+            const [featuresDoc, phases, project, focus] = await Promise.all([st.loadFeatures(), st.loadAllPhases(), st.loadProject(plannerSessionId), st.loadResume(plannerSessionId)]);
             const features = featuresDoc.features;
             const source = findTaskByRef(phases, features, params.from_task.trim());
             const target = findTaskByRef(phases, features, params.to_task.trim());
@@ -5622,7 +5627,7 @@ export default function planPiExtension(pi) {
                 resumeLocation: params.resume_location.trim(), howToResume: params.how_to_resume.trim(),
                 relatedTaskId: target.task.id, pausedAt: timestamp, pausedBy: params.switched_by?.trim() ?? "",
             };
-            const selection = recommendNextTask(features, phases, project.workDeviations, focus?.currentPhaseId, plannerSessionId);
+            const selection = recommendNextTask(features, phases, await st.listAllWorkDeviations(), focus?.currentPhaseId, plannerSessionId);
             const record = {
                 id: crypto.randomUUID(), recommendedTaskId: selection.candidate?.task.id ?? source.task.id,
                 temporaryTaskId: target.task.id, resumeTaskId: source.task.id, reason: snapshot.reason, snapshot,
@@ -5672,7 +5677,7 @@ export default function planPiExtension(pi) {
                         });
                     }
                     targetStarted = true;
-                    await st.addWorkDeviation(record);
+                    await st.addWorkDeviation(record, plannerSessionId);
                     if (target.task.pauseSnapshot) {
                         const resumedDeviation = project.workDeviations
                             .filter((deviation) => deviation.resumeTaskId === target.task.id
@@ -5733,7 +5738,7 @@ export default function planPiExtension(pi) {
             if (!found)
                 return { content: [{ type: "text", text: `Task not found: ${params.taskId}` }], details: { reopened: false, errorCode: "TASK_NOT_FOUND" }, isError: true };
             try {
-                const reopened = await st.reopenTask(found.phase.id, found.task.id, { confirmed: params.confirmed, ownerSessionId: `pi:${ctx.sessionManager.getSessionId()}` });
+                const reopened = await st.reopenTask(found.phase.id, found.task.id, { confirmed: params.confirmed, ownerSessionId: piSessionIdOf(ctx) });
                 await st.syncTaskStatusRollup(found.phase.id);
                 await st.writeGenerated();
                 return { content: [{ type: "text", text: `✅ Task reopened: ${formatPhaseRef(found.phase.number, featureNumberOfPhase(found.phase, features))}/T${pad(reopened.number)} — ${reopened.title} (in-progress)` }], details: { reopened: true, task: reopened } };
@@ -5753,7 +5758,7 @@ export default function planPiExtension(pi) {
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
             const st = await requirePlan(ctx);
-            const plannerSessionId = `pi:${ctx.sessionManager.getSessionId()}`;
+            const plannerSessionId = piSessionIdOf(ctx);
             if (!st)
                 return taskStartFailure(taskStartDenied("PLAN_NOT_FOUND", "No .planner/ found.", ["Initialize or load the planner, then retry task_start."]));
             return st.runBatch(async () => {
@@ -5769,7 +5774,7 @@ export default function planPiExtension(pi) {
                 const featureRef = feature ? formatFeatureRef(feature.number) : "";
                 if (task.status === "done")
                     return taskStartFailure(taskStartDenied("TASK_DONE", `Task ${taskRef} is done and was not reopened.`, [`Use task_reopen with taskId=${taskRef} and confirmed=true after explicit user confirmation.`, `Retry task_start ${taskRef} only if reopening is not the intended operation.`], { taskId: task.id }));
-                const project = await st.loadProject();
+                const project = await st.loadProject(plannerSessionId);
                 // Targeted stale-context refresh (P102(F005)/T404): task_start delivery
                 // stays scoped to feature/phase/task context, but a session that
                 // already did a full planner-load gets a non-blocking advisory when
@@ -5822,11 +5827,11 @@ export default function planPiExtension(pi) {
                     const phaseOrderAdvisory = higherPriorityOpenPhaseAdvisory(higherPriorityPhase);
                     return { content: [{ type: "text", text: `✅ Task already started: ${taskRef} — ${task.title} (in-progress)\nstarted: true${projectContextAdvisory}${phaseOrderAdvisory}` }], details: { ...task, ...outcome, task, projectContextStale: projectContextReadState === "stale", higherPriorityOpenPhase: higherPriorityPhase } };
                 }
-                const [phases, focus] = await Promise.all([st.loadAllPhases(), st.loadResume()]);
+                const [phases, focus] = await Promise.all([st.loadAllPhases(), st.loadResume(plannerSessionId)]);
                 const eligibility = checkExplicitTaskStart(features, phases, task.id, project.workDeviations);
                 if (!eligibility.eligible)
                     return taskStartFailure(taskStartDenied("START_NOT_ALLOWED", eligibility.reason, ["Resolve the reported task readiness condition, then retry task_start."], { taskId: task.id }), { eligibility });
-                const selection = recommendNextTask(features, phases, project.workDeviations, focus?.currentPhaseId, plannerSessionId);
+                const selection = recommendNextTask(features, phases, await st.listAllWorkDeviations(), focus?.currentPhaseId, plannerSessionId);
                 if (selection.kind === "conflict") {
                     return taskStartFailure(taskStartDenied("ACTIVE_TASK_CONFLICT", selection.reason, ["Pause or reconcile the active tasks.", `Retry task_start ${taskRef}.`], { taskId: task.id }), { selection });
                 }
@@ -5930,6 +5935,7 @@ export default function planPiExtension(pi) {
             description_update: Type.String({ minLength: 10, description: "Required evidence: shipped work, verification level including partial verification, remaining/unverified work, files, decisions, and updated code references." }),
         }),
         async execute(_id, params, _signal, _onUpdate, ctx) {
+            const plannerSessionId = piSessionIdOf(ctx);
             const st = await requirePlan(ctx);
             if (!st)
                 return { content: [{ type: "text", text: "No .planner/ found." }], details: {} };
@@ -5979,7 +5985,7 @@ export default function planPiExtension(pi) {
                 return phase;
             });
             const clearedRef = await st.syncTaskStatusRollup(found.phase.id);
-            const completedDeviation = (await st.loadProject()).workDeviations
+            const completedDeviation = (await st.loadProject(plannerSessionId)).workDeviations
                 .filter((deviation) => (deviation.state === "approved" || deviation.state === "active") && deviation.temporaryTaskId === task.id)
                 .sort((left, right) => right.createdAt.localeCompare(left.createdAt))[0];
             if (completedDeviation)
@@ -6207,7 +6213,7 @@ export default function planPiExtension(pi) {
             const srv = server;
             let recap = "";
             try {
-                recap = await buildRecap(st, { localUrl: srv?.localUrl, lanUrl: srv?.lanUrl, port: lastKnownWebPort ?? undefined }, { harness: "pi" });
+                recap = await buildRecap(st, { localUrl: srv?.localUrl, lanUrl: srv?.lanUrl, port: lastKnownWebPort ?? undefined }, { harness: "pi", sessionId: plannerSessionId });
             }
             catch (e) {
                 recap = `(recap unavailable: ${e instanceof Error ? e.message : String(e)})`;
@@ -6264,7 +6270,7 @@ export default function planPiExtension(pi) {
             // Session initialization must not mutate canonical planner entities.
             // Migrations/backfills/repairs are explicit tools; resume is `.local`.
             if (!plannerHeavyInitDone) {
-                await st.refreshResume();
+                await st.refreshResume(undefined, undefined, plannerSessionId);
                 plannerHeavyInitDone = true;
             }
             // Resume-flag lifecycle — MUST run every turn, including cache hits.
@@ -6291,7 +6297,7 @@ export default function planPiExtension(pi) {
             const plan = await st.loadAll();
             const project = plan.project;
             const profile = await st.loadCodebaseProfile();
-            const resume = await st.loadResume().catch(() => null) ?? await st.refreshResume();
+            const resume = await st.loadResume(plannerSessionId).catch(() => null) ?? await st.refreshResume(undefined, undefined, plannerSessionId);
             const activity = await st.loadActivityLog();
             const recentActivity = activity.entries.slice(-3).reverse();
             const handoffs = await st.listHandoffs();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/pi-adapter",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "private": false,
   "description": "Pi agent extension for Agent Plan — structured project planning in .planner/",
   "keywords": [
@@ -28,8 +28,8 @@
   "dependencies": {
     "typebox": "^1.1.24",
     "zod": "^3.24.0",
-    "@agent-plan/core": "0.4.1",
-    "@agent-plan/server": "0.4.1"
+    "@agent-plan/server": "0.4.2",
+    "@agent-plan/core": "0.4.2"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "^0.74.0",