@agent-plan/pi-adapter 0.2.7 → 0.2.9

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;AA00BtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAqxF9D"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,iCAAiC,CAAC;AA4yBtF,MAAM,CAAC,OAAO,UAAU,eAAe,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAmrF9D"}

package/dist/index.js

@@ -111,8 +111,8 @@ const PLANNER_COMMAND_COMPLETIONS = [
     { value: "export-full", label: "export-full", description: "Export full detailed plan as Markdown" },
     { value: "bypass", label: "bypass", description: "Authorize edit/write without a task in-progress (15 min)" },
     { value: "clear-bypass", label: "clear-bypass", description: "Revoke the guard bypass" },
-    { value: "load", label: "load", description: "Re-enable planner and start web UI" },
-    { value: "disable", label: "disable", description: "Reset planner preferences and disable for this session" },
+    { value: "load", label: "load", description: "Enable planner + start web UI (LAN) and show resume summary" },
+    { value: "stop", label: "stop", description: "Disable planner and shut down the web UI" },
 ];
 // ─── Helpers ────────────────────────────────────────────────────────────
 function nowISO() {
@@ -165,24 +165,6 @@ async function maybeHealStatuses(st) {
 function normalizeLanguagePref(value) {
     return value?.trim() ?? "";
 }
-// Safety net: ctx.ui.input() can hang forever when the extension is loaded from
-// an NPM-installed package (the prompt never renders, the await never resolves,
-// and Pi freezes). Race the prompt against a timeout; on timeout, fall back to the
-// empty answer (treated as "no / this session only" by the gating logic) and notify
-// the user how to enable manually. This preserves the interactive prompt for the
-// working case while guaranteeing session_start can never block Pi permanently.
-const PROMPT_TIMEOUT_MS = 20000;
-async function safeInput(ctx, prompt) {
-    try {
-        return await Promise.race([
-            ctx.ui.input(prompt),
-            new Promise((resolve) => setTimeout(() => resolve("__PLANNER_PROMPT_TIMEOUT__"), PROMPT_TIMEOUT_MS)),
-        ]);
-    }
-    catch {
-        return "";
-    }
-}
 async function ensureProjectLanguagePreferences(st) {
     const project = await st.loadProject();
     const contentLanguage = normalizeLanguagePref(project.contentLanguage);
@@ -661,16 +643,14 @@ async function maybeStartWeb(ctx) {
         plannerSessionEnabled = true;
         return;
     }
-    const ans = await ctx.ui.input("Planner active for this session. Start the web UI now? (y/N)");
-    if (ans && /^(y|yes|s|si|sì)$/i.test(ans.trim())) {
-        ctx.ui.notify("Starting web server…", "info");
-        try {
-            await startServer(ctx);
-            ctx.ui.notify(`Web UI ready. Open: ${server?.url ?? "?"}`, "info");
-        }
-        catch (err) {
-            ctx.ui.notify(`Failed to start web server: ${String(err)}`, "error");
-        }
+    ctx.ui.notify("Starting web server (LAN)…", "info");
+    try {
+        await startServer(ctx, undefined, "lan");
+        const srv = server;
+        ctx.ui.notify(srv?.lanUrl ? `Web UI ready. Local: ${srv.localUrl} — LAN: ${srv.lanUrl}` : `Web UI ready. Open: ${srv?.url ?? "?"}`, "info");
+    }
+    catch (err) {
+        ctx.ui.notify(`Failed to start web server: ${String(err)}`, "error");
     }
 }
 function normalizeVisibility(input) {
@@ -683,17 +663,7 @@ function normalizeVisibility(input) {
         return "local";
     return undefined;
 }
-async function promptWebVisibility(ctx) {
-    try {
-        const ans = await ctx.ui.input("Web UI visibility: (local) only this machine  /  (lan) visible on the local network? [local/lan]");
-        const v = normalizeVisibility(ans);
-        if (v)
-            return v;
-    }
-    catch { }
-    return "local";
-}
-async function startServer(ctx, requestedPort, visibility = "local") {
+async function startServer(ctx, requestedPort, visibility = "lan") {
     plannerSessionEnabled = true;
     if (server)
         return;
@@ -819,61 +789,14 @@ export default function planPiExtension(pi) {
             const exists = await st.exists().catch(() => false);
             let project = exists ? await st.loadProject().catch(() => null) : null;
             // ── Step 1: Enable gating ───────────────────────────────────────
+            // No blocking prompt in session_start: ctx.ui.input()/select() do not
+            // render for NPM-loaded ESM extensions during session_start (they hang
+            // until the safety-net timeout). The planner is always DISABLED at
+            // startup; the user enables it with '/planner load' (which also starts
+            // the web UI on LAN and triggers the resume summary showing the URL).
             let enablePlanner = false;
             if (exists) {
-                if (project?.plannerAutoEnable) {
-                    enablePlanner = true; // 'always' persisted — no prompt
-                }
-                else if (project?.plannerNeverAsk) {
-                    enablePlanner = false; // 'never' persisted — no prompt, no activation
-                }
-                else {
-                    try {
-                        const ans = await safeInput(ctx, "Planner detected in this project. Enable the planner extension? (y)es / (n)o / (a)lways / n(e)ver)");
-                        const normalized = ans?.trim().toLowerCase() ?? "";
-                        if (["a", "always", "sempre"].includes(normalized)) {
-                            enablePlanner = true;
-                            if (project) {
-                                project.plannerAutoEnable = true;
-                                project.plannerNeverAsk = false;
-                                await st.saveProject(project);
-                            }
-                            ctx.ui.notify("Saved: planner will auto-enable for this project.", "info");
-                        }
-                        else if (["e", "never", "mai"].includes(normalized)) {
-                            enablePlanner = false;
-                            if (project) {
-                                project.plannerNeverAsk = true;
-                                project.plannerAutoEnable = false;
-                                await st.saveProject(project);
-                            }
-                            try {
-                                capturedPi?.appendEntry("plan-web-state", { running: false });
-                            }
-                            catch { }
-                            ctx.ui.notify("Saved: planner will not ask again for this project. Use '/planner load' to re-enable manually.", "info");
-                        }
-                        else if (/^(y|yes|s|si|sì)$/i.test(normalized)) {
-                            enablePlanner = true; // this session only
-                        }
-                        else {
-                            // 'n' / 'no' / empty — this session only, NOT persisted
-                            enablePlanner = false;
-                            try {
-                                capturedPi?.appendEntry("plan-web-state", { running: false });
-                            }
-                            catch { }
-                            ctx.ui.notify("Planner disabled for this session. Use '/planner load' to enable manually.", "info");
-                        }
-                    }
-                    catch {
-                        enablePlanner = false;
-                        try {
-                            capturedPi?.appendEntry("plan-web-state", { running: false });
-                        }
-                        catch { }
-                    }
-                }
+                ctx.ui.notify("Planner detected in this project. Run '/planner load' to enable the planner and start the web UI (LAN).", "info");
             }
             plannerSessionEnabled = enablePlanner;
             // Proactive check for leftover handoff files.
@@ -889,95 +812,6 @@ export default function planPiExtension(pi) {
                 startupResumeSummaryPending = false;
                 return;
             }
-            // ── Step 2: Web UI gating (only if planner enabled) ─────────────
-            startupResumePromptPending = exists && plannerSessionEnabled;
-            startupResumeSummaryPending = exists && plannerSessionEnabled;
-            if (exists && plannerSessionEnabled) {
-                // Migration first.
-                try {
-                    await migrateToUuids(st);
-                }
-                catch (e) {
-                    ctx.ui.notify(`Migration failed: ${e instanceof Error ? e.message : String(e)}`, "error");
-                }
-                await st.ensureStructureOrdering().catch(() => { });
-                // Reconcile derived statuses once on startup: backfills any drift from
-                // manual edits, pre-fix data, or tools that bypassed the rollup.
-                // Uses maybeHealStatuses (idempotent via healedStatusRoots guard) so the
-                // later call below and before_agent_start don't re-run it (startup dedup).
-                await maybeHealStatuses(st).catch(() => { });
-                // Decide whether to start the web UI based on persisted prefs or prompt.
-                let startWeb = false;
-                if (project?.plannerAutoStartWeb) {
-                    startWeb = true; // 'always' persisted
-                }
-                else if (project?.plannerNeverStartWeb) {
-                    startWeb = false; // 'never' persisted
-                }
-                else if (server === null) {
-                    try {
-                        const ans = await safeInput(ctx, "Start the planner web UI? (y)es / (n)o / (a)lways / n(e)ver)");
-                        const normalized = ans?.trim().toLowerCase() ?? "";
-                        if (["a", "always", "sempre"].includes(normalized)) {
-                            startWeb = true;
-                            if (project) {
-                                project.plannerAutoStartWeb = true;
-                                project.plannerNeverStartWeb = false;
-                                await st.saveProject(project);
-                            }
-                        }
-                        else if (["e", "never", "mai"].includes(normalized)) {
-                            startWeb = false;
-                            if (project) {
-                                project.plannerNeverStartWeb = true;
-                                project.plannerAutoStartWeb = false;
-                                await st.saveProject(project);
-                            }
-                        }
-                        else if (/^(y|yes|s|si|sì)$/i.test(normalized)) {
-                            startWeb = true;
-                        }
-                        else {
-                            startWeb = false; // 'n' / 'no' / empty — this session only
-                        }
-                    }
-                    catch {
-                        startWeb = false;
-                    }
-                }
-                if (startWeb && server === null) {
-                    const visibility = await promptWebVisibility(ctx);
-                    await startServer(ctx, preferredPort, visibility).catch(() => { });
-                    const srv = server;
-                    if (srv) {
-                        const urls = srv.lanUrl ? `${srv.localUrl} (LAN: ${srv.lanUrl})` : srv.url;
-                        ctx.ui.notify(`Web UI started at ${urls}`, "info");
-                    }
-                }
-                const url = server?.url ?? "Starting server...";
-                ctx.ui.notify(`Planner enabled. Dashboard: ${url}\nAnalyzing project state and preparing resume summary...`, "info");
-                await ensureProjectLanguagePreferences(st).catch(() => null);
-                // (maybeHealStatuses already ran above — skipped here thanks to the guard.)
-                // Background cleanup of orphan .bak/.tmp.* files (non-blocking)
-                st.cleanupOrphanBackups().catch(() => { });
-                const handoff = await st.loadHandoff().catch(() => null);
-                if (handoff?.content) {
-                    ctx.ui.notify(`Handoff detected at .planner/HANDOFF.md (updated ${handoff.updatedAt}). It will be loaded automatically when the agent starts.`, "info");
-                }
-                // Mark heavy init done so before_agent_start doesn't re-run migration,
-                // status healing, and language prefs on the first turn (startup dedup:
-                // all of these already ran in this session_start block). refreshResume is
-                // safe to skip here because the per-turn context builder falls back to
-                // loadResume() ?? refreshResume() when building the system prompt.
-                plannerHeavyInitDone = true;
-                pi.sendMessage({
-                    customType: "planner-resume-trigger",
-                    content: "Emit the mandatory planner startup resume summary now.",
-                    display: false,
-                }, {
-                    triggerTurn: true,
-                });
-            }
         }
         catch (e) {
             // Defensive: never let session_start throw crash Pi silently.
@@ -1158,9 +992,9 @@ export default function planPiExtension(pi) {
             "bypass",
             "clear-bypass",
             "load",
-            "disable",
+            "stop",
         ];
-        const SUB_HELP = "Available: init, show, repair, project, feature, phase, task, discuss, handoff, web, export, export-full, bypass, clear-bypass, load, disable\n" +
+        const SUB_HELP = "Available: init, show, repair, project, feature, phase, task, discuss, handoff, web, export, export-full, bypass, clear-bypass, load, stop\n" +
             "Try: /planner <TAB>  |  /planner feature list  |  /planner task start  |  /planner export-full\n" +
             "Handoff actions: /planner handoff prepare | show | write | clear";
         if (!a) {
@@ -2345,7 +2179,7 @@ export default function planPiExtension(pi) {
                     }
                     const visibilityArg = normalizeVisibility(parts[2]);
                     const portArg = parts[2]?.trim() && Number.isFinite(parseInt(parts[2], 10)) ? parseInt(parts[2], 10) : undefined;
-                    const visibility = visibilityArg ?? await promptWebVisibility(ctx);
+                    const visibility = visibilityArg ?? "lan";
                     ctx.ui.notify(`Starting web server (${visibility})${portArg ? ` on port ${portArg}` : ""} …`, "info");
                     await startServer(ctx, portArg, visibility);
                     const srv = server;
@@ -2418,46 +2252,34 @@ export default function planPiExtension(pi) {
             return;
         }
         if (a === "load") {
-            const project = await st.loadProject().catch(() => null);
-            if (project) {
-                let changed = false;
-                if (project.plannerNeverAsk) {
-                    project.plannerNeverAsk = false;
-                    changed = true;
-                }
-                if (project.plannerNeverStartWeb) {
-                    project.plannerNeverStartWeb = false;
-                    changed = true;
-                }
-                if (changed) {
-                    await st.saveProject(project);
-                    ctx.ui.notify("Cleared 'never' flags. Planner enabled for this session.", "info");
-                }
-            }
             plannerSessionEnabled = true;
             if (!server) {
-                ctx.ui.notify("Starting web server …", "info");
-                await startServer(ctx);
-            }
-            ctx.ui.notify(`Planner loaded. Web UI: ${server?.url ?? "(not started)"}`, "info");
+                ctx.ui.notify("Starting web server (LAN) …", "info");
+                await startServer(ctx, undefined, "lan").catch(() => { });
+            }
+            const srv = server;
+            ctx.ui.notify(srv?.lanUrl ? `Planner loaded. Web UI: ${srv.localUrl} (LAN: ${srv.lanUrl})` : `Planner loaded. Web UI: ${srv?.url ?? "(not started)"}`, "info");
+            // Trigger the mandatory startup resume summary so the agent surfaces the
+            // web UI address (and project state) immediately after loading.
+            startupResumePromptPending = true;
+            startupResumeSummaryPending = true;
+            pi.sendMessage({
+                customType: "planner-resume-trigger",
+                content: "Emit the mandatory planner startup resume summary now (include the web UI address and port).",
+                display: false,
+            }, {
+                triggerTurn: true,
+            });
             return;
         }
-        if (a === "disable") {
-            const project = await st.loadProject().catch(() => null);
-            if (project) {
-                project.plannerAutoEnable = false;
-                project.plannerNeverAsk = false;
-                project.plannerAutoStartWeb = false;
-                project.plannerNeverStartWeb = false;
-                await st.saveProject(project);
-            }
+        if (a === "stop" || a === "disable") {
             plannerSessionEnabled = false;
             await stopServer().catch(() => { });
             try {
                 capturedPi?.appendEntry("plan-web-state", { running: false });
             }
             catch { }
-            ctx.ui.notify("Planner preferences reset to 'ask'. Disabled for this session. Next restart will prompt again.", "info");
+            ctx.ui.notify("Planner stopped. Web UI shut down. Run '/planner load' to re-enable.", "info");
             return;
         }
         ctx.ui.notify(`Unknown "${a}". ${SUB_HELP}`, "warning");
@@ -3649,7 +3471,18 @@ export default function planPiExtension(pi) {
                 : (handoff?.updatedAt
                     ? "Read the handoff, then validate current plan ordering and dependencies before picking the next task."
                     : "Review the current plan and choose the next task.");
-            const webUrl = server?.url ?? "";
+            const webServer = server;
+            const webLocalUrl = webServer?.localUrl ?? webServer?.url ?? "";
+            const webLanUrl = webServer?.lanUrl ?? "";
+            let webPort = "";
+            try {
+                const p = new URL(webLocalUrl).port;
+                if (p)
+                    webPort = p;
+            }
+            catch { }
+            const webUrl = webLocalUrl;
+            const webUrlFull = webLanUrl ? `${webLocalUrl} (LAN: ${webLanUrl})` : webLocalUrl;
             const openQuestions = plan.phases
                 .flatMap((p) => p.openQuestions.map((q) => `[${p.id}] ${q}`))
                 .slice(0, 12);
@@ -3679,7 +3512,7 @@ export default function planPiExtension(pi) {
                 currentPhase ? `- Mention current phase ONLY because it is actually active: ${currentPhase.id} — ${phaseLabel(currentPhase)} (${currentPhase.status}).` : "- Mention that no current phase is active.",
                 currentTask ? `- Mention current task ONLY because it is actually active: ${currentTask.id} — ${taskLabel(currentTask)} (${currentTask.status}).` : "- Mention that no current task is active.",
                 !hasActiveWork ? "- If a handoff exists, describe it only as a previous-session hint to validate against current planner state, ordering, and dependencies. Do NOT present it as the current focus." : "",
-                webUrl ? `- Mention the active web dashboard URL: ${webUrl}` : "- Explicitly say whether the web dashboard is active in this session.",
+                webUrl ? `- At the END of your summary, you MUST print a dedicated line with the web dashboard address and port, exactly: '🌐 Web UI: ${webLocalUrl}${webPort ? ` (port ${webPort})` : ""}${webLanUrl ? ` — LAN: ${webLanUrl}` : ""}'. This line is mandatory; do not omit it.` : "- Explicitly tell the user whether the web dashboard is active in this session (and how to start it with '/planner web start' if not).",
                 `- Mention the next suggested activity: ${nextActivity}`,
                 "- THEN ask the user explicitly whether they want to resume that activity now.",
                 "- Do NOT assume yes. Wait for the user's answer before continuing implementation work.",
@@ -3688,6 +3521,9 @@ export default function planPiExtension(pi) {
             // Build/refresh the context block (slow path: cache miss).
             const contextBlock = [
                 `[Plan Context — ${project.name}]`,
+                // ── Web UI status (prominent — agent must surface URL in its summary) ──
+                webUrl ? `🌐 WEB UI RUNNING: ${webUrlFull}${webPort ? ` (port ${webPort})` : ""}. Include this address+port at the end of your resume summary.` : `Web UI: not running. Start it with '/planner web start'.`,
+                "",
                 // ── MANDATORY OPERATIONAL PROTOCOL (top priority, non-negotiable) ──
                 "",
                 "═══════════════════════════════════════════════════════════════",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-plan/pi-adapter",
-  "version": "0.2.7",
+  "version": "0.2.9",
   "private": false,
   "description": "Pi agent extension for Agent Plan — structured project planning in .planner/",
   "keywords": [