@manny-est/node-red-flowpilot 0.6.0-beta.1 → 0.6.0

package/lib/core/history.js

@@ -4,6 +4,85 @@
     // a page reload continues the same transcript; reset by clearChat()
     // ("start a fresh conversation" gets a fresh transcript file too).
     // ---------------------------------------------------------------------
+    var FP_CONVERSATION_ID_KEY = "fp-conversation-id";
+    var FP_RUN_MARKER_KEY = "fp-run-marker";
+
+    function flowpilotStorageLog(level, event, data) {
+        var logger = console[level] || console.log;
+        try {
+            logger.call(console, "[FlowPilot][storage] " + event, data || {});
+        } catch (e) { /* console unavailable */ }
+    }
+
+    function flowpilotSessionStorage() {
+        return window.sessionStorage;
+    }
+
+    function flowpilotStorageGet(key, reason) {
+        try {
+            var value = flowpilotSessionStorage().getItem(key);
+            flowpilotStorageLog("log", "get", {
+                key: key,
+                reason: reason || "",
+                value: value,
+                href: location.href
+            });
+            return value;
+        } catch (e) {
+            flowpilotStorageLog("warn", "get-failed", {
+                key: key,
+                reason: reason || "",
+                error: e && e.message ? e.message : String(e),
+                href: location.href
+            });
+            return null;
+        }
+    }
+
+    function flowpilotStorageSet(key, value, reason) {
+        try {
+            flowpilotSessionStorage().setItem(key, value);
+            var readBack = flowpilotSessionStorage().getItem(key);
+            flowpilotStorageLog("log", "set", {
+                key: key,
+                reason: reason || "",
+                value: value,
+                readBack: readBack,
+                href: location.href
+            });
+            return true;
+        } catch (e) {
+            flowpilotStorageLog("warn", "set-failed", {
+                key: key,
+                reason: reason || "",
+                value: value,
+                error: e && e.message ? e.message : String(e),
+                href: location.href
+            });
+            return false;
+        }
+    }
+
+    function flowpilotStorageRemove(key, reason) {
+        try {
+            flowpilotSessionStorage().removeItem(key);
+            flowpilotStorageLog("log", "remove", {
+                key: key,
+                reason: reason || "",
+                href: location.href
+            });
+            return true;
+        } catch (e) {
+            flowpilotStorageLog("warn", "remove-failed", {
+                key: key,
+                reason: reason || "",
+                error: e && e.message ? e.message : String(e),
+                href: location.href
+            });
+            return false;
+        }
+    }
+
     function makeConversationId() {
         if (window.crypto && typeof window.crypto.randomUUID === "function") {
             return window.crypto.randomUUID();
@@ -13,10 +92,68 @@
 
     function newConversationId() {
         var id = makeConversationId();
-        try { sessionStorage.setItem("fp-conversation-id", id); } catch (e) { /* storage unavailable */ }
+        flowpilotStorageSet(FP_CONVERSATION_ID_KEY, id, "newConversationId");
         return id;
     }
 
+    function persistConversationId(id, reason) {
+        if (!id) { return false; }
+        return flowpilotStorageSet(FP_CONVERSATION_ID_KEY, String(id), reason || "persistConversationId");
+    }
+
+    function clearConversationId(reason) {
+        return flowpilotStorageRemove(FP_CONVERSATION_ID_KEY, reason || "clearConversationId");
+    }
+
+    function readRunMarker() {
+        var raw = flowpilotStorageGet(FP_RUN_MARKER_KEY, "readRunMarker");
+        if (!raw) { return null; }
+        try {
+            return JSON.parse(raw);
+        } catch (e) {
+            flowpilotStorageLog("warn", "run-marker-parse-failed", {
+                raw: raw,
+                error: e && e.message ? e.message : String(e)
+            });
+            flowpilotStorageRemove(FP_RUN_MARKER_KEY, "invalid run marker json");
+            return null;
+        }
+    }
+
+    function writeRunMarker(marker, reason) {
+        if (!marker) { return false; }
+        return flowpilotStorageSet(FP_RUN_MARKER_KEY, JSON.stringify(marker), reason || "writeRunMarker");
+    }
+
+    function clearRunMarker(reason) {
+        return flowpilotStorageRemove(FP_RUN_MARKER_KEY, reason || "clearRunMarker");
+    }
+
+    function renderInterruptedRunMessage(appliedCount) {
+        addMessage("assistant",
+            "⚠ This run was interrupted after step " + appliedCount +
+            " — completed steps are applied (Ctrl+Z to undo). Re-send to continue from here.");
+    }
+
+    function restoreInterruptedRunMarker(expectedConversationId) {
+        var marker = readRunMarker();
+        if (!marker) { return; }
+        flowpilotStorageLog("log", "restore-run-marker", {
+            marker: marker,
+            expectedConversationId: expectedConversationId || null
+        });
+        if (expectedConversationId && marker.conversationId && marker.conversationId !== expectedConversationId) {
+            flowpilotStorageLog("warn", "run-marker-conversation-mismatch", {
+                markerConversationId: marker.conversationId,
+                expectedConversationId: expectedConversationId
+            });
+            clearRunMarker("run marker conversation mismatch");
+            return;
+        }
+        renderInterruptedRunMessage(Number(marker.appliedCount) || 0);
+        clearRunMarker("restored interrupted run banner");
+    }
+
     // CLAUDE-029: whether conversationId above came from an existing
     // sessionStorage entry (a page reload continuing a prior conversation)
     // rather than being freshly minted — drives whether page init rehydrates
@@ -24,14 +161,19 @@
     var conversationIdWasRestored = false;
 
     var conversationId = (function () {
-        try {
-            var existing = sessionStorage.getItem("fp-conversation-id");
-            if (existing) {
-                conversationIdWasRestored = true;
-                return existing;
-            }
-        } catch (e) { /* storage unavailable */ }
-        return newConversationId();
+        flowpilotStorageLog("log", "conversation-id-init-enter", {
+            href: location.href,
+            readyState: document.readyState
+        });
+        var existing = flowpilotStorageGet(FP_CONVERSATION_ID_KEY, "conversationId init");
+        if (existing) {
+            conversationIdWasRestored = true;
+            flowpilotStorageLog("log", "conversation-id-restored", { conversationId: existing });
+            return existing;
+        }
+        var fresh = newConversationId();
+        flowpilotStorageLog("log", "conversation-id-created", { conversationId: fresh });
+        return fresh;
     })();
 
     // ---------------------------------------------------------------------

package/lib/core/init.js

@@ -14,18 +14,20 @@
         "- `/generate` — arm Generate mode\n" +
         "- `/document` — arm Document mode\n" +
         "- `/modify` — arm Modify mode\n" +
-        "- `/query` — back to Query (disarm)\n" +
+        "- `/query` / `/chat` — back to Query (disarm)\n" +
         "- `/clear` — start a fresh conversation (clears chat and memory)\n" +
         "- `/history` — open the Flight log (past conversations)\n" +
         "- `/settings` — open the Hangar (providers, behavior, safety)\n\n" +
         "Typing a shortcut with extra text, e.g. `/modify add a debug node`, switches mode and leaves the rest in the box so you can review before sending.\n\n" +
         "- `/demo` — load a sample Generate request (a dad joke flow) into the compose box\n" +
         "- `/feedback` — bug report / feature request info\n" +
+        "- `/beta` — beta-channel install/revert instructions\n" +
         "- `/build` — describe a goal; I'll plan, propose, and walk an iterative build → deploy → debug → review → fix loop with you\n" +
         "- `/compact` — hide labels on the selected node(s) (icon-only); `/expand` restores them. Instant, no AI involved — one Ctrl+Z undoes it.\n" +
         "- `/disable` — disable the selected node(s) (skipped on Deploy); `/enable` re-enables them. Instant, no AI involved — one Ctrl+Z undoes it.\n" +
         "- `/refresh` — re-render all messages from the in-memory record store (restores interactive Apply buttons if they were lost).\n" +
         "- `/debug` — toggle debug logging (full prompts/replies/decisions to `flowpilot/debug.log`) on/off. Instant, no AI involved.\n\n" +
+        "- `/summarize` — summarize older chat history into one compact note, keeping the latest two exchanges intact.\n\n" +
         "### Also worth knowing\n\n" +
         "- Action chips (paper-plane buttons) offer a one-click follow-up — review and send, nothing fires automatically.\n" +
         "- When I ask a clarifying question, I'll often offer quick-reply buttons (plus \"Other\" for your own answer) — clicking one sends it right away.\n" +
@@ -43,11 +45,30 @@
         "human crew reads every report.\n\n" +
         "- **Report an issue**: https://github.com/manny-est/flowpilot/issues\n" +
         "- **Browse the repo**: https://github.com/manny-est/flowpilot\n\n" +
-        "A good report travels light but packs the essentials: your Node-RED " +
-        "version, the provider/model you're flying with, and the steps to " +
-        "reproduce. That's usually enough to get a fix off the ground.\n\n" +
+        "A good report travels light but packs the essentials: your FlowPilot " +
+        "version (top of Settings), your Node-RED version, the provider/model " +
+        "you're flying with, and the steps to reproduce. That's usually enough " +
+        "to get a fix off the ground.\n\n" +
         "Safe travels.";
 
+    var BETA_TEXT = "## Beta channel\n\n" +
+        "Thanks for testing FlowPilot on the beta track.\n\n" +
+        "**Install the beta:**\n" +
+        "```\n" +
+        "cd ~/.node-red        (or your mounted Node-RED data dir, e.g. /data in Docker)\n" +
+        "npm install @manny-est/node-red-flowpilot@beta\n" +
+        "```\n" +
+        "Then **restart Node-RED** (or the container) — a browser refresh alone " +
+        "won't pick it up; FlowPilot's editor UI is cached server-side.\n\n" +
+        "**Revert to stable:**\n" +
+        "```\n" +
+        "cd ~/.node-red        (or /data)\n" +
+        "npm install @manny-est/node-red-flowpilot@latest\n" +
+        "```\n" +
+        "Restart again afterward.\n\n" +
+        "**Found a bug?** Type `/feedback`, or go straight to " +
+        "https://github.com/manny-est/flowpilot/issues — every report helps.";
+
     var demoTypeTimer = null;
 
     // Registry of all slash commands — drives the autocomplete panel that
@@ -60,15 +81,20 @@
         { cmd: "/build",    desc: "Start a deploy-verify build loop" },
         { cmd: "/chat",     desc: "Switch to Chat mode" },
         { cmd: "/query",    desc: "Add or toggle a Query intent" },
+        { cmd: "/clear",    desc: "Start a fresh conversation" },
+        { cmd: "/history",  desc: "Open past conversations" },
+        { cmd: "/settings", desc: "Open provider/behavior settings" },
         { cmd: "/compact",  desc: "Compact labels on selected nodes" },
         { cmd: "/expand",   desc: "Expand labels on selected nodes" },
         { cmd: "/disable",  desc: "Disable selected nodes" },
         { cmd: "/enable",   desc: "Enable selected nodes" },
         { cmd: "/refresh",  desc: "Re-render all messages from shadow record store" },
         { cmd: "/debug",    desc: "Toggle debug logging on/off" },
+        { cmd: "/summarize", desc: "Summarize older history into a compact note" },
         { cmd: "/demo",     desc: "Type in a demo prompt" },
         { cmd: "/help",     desc: "Show all available commands" },
-        { cmd: "/feedback", desc: "Show feedback info" }
+        { cmd: "/feedback", desc: "Show feedback info" },
+        { cmd: "/beta",     desc: "Beta install/revert instructions" }
     ];
 
     function bindSlashAutocomplete($promptBox) {
@@ -235,6 +261,10 @@
                 addMessage("assistant", FEEDBACK_TEXT);
                 if ($promptBox.length) { $promptBox.val(""); }
                 break;
+            case "/beta":
+                addMessage("assistant", BETA_TEXT);
+                if ($promptBox.length) { $promptBox.val(""); }
+                break;
             case "/refresh":
                 refreshView();
                 if ($promptBox.length) { $promptBox.val(""); }
@@ -250,6 +280,10 @@
                 });
                 if ($promptBox.length) { $promptBox.val(""); }
                 break;
+            case "/summarize":
+                summarizeConversationHistory();
+                if ($promptBox.length) { $promptBox.val(""); }
+                break;
             // Deterministic, no LLM round-trip: just invokes Node-RED's own
             // native "show/hide selected node labels" action (RED.actions
             // "core:show-selected-node-labels" / "core:hide-selected-node-
@@ -800,6 +834,7 @@
             '      <div class="fp-status-strip">' +
             '        <span id="fp-selection-status" class="fp-selection-status">No nodes selected</span>' +
             '        <a href="#" id="fp-preview-nodes" class="fp-preview-link fp-hidden" title="Open this from the main window to see the exact sanitized node JSON">Preview JSON</a>' +
+            '        <span id="fp-dev-warning-status" class="fp-secrets-status" title="Development/test only. Anything you send may leave this Node-RED instance. Don\'t include credentials or proprietary data; local/private AI recommended.">⚠</span>' +
             '        <span id="fp-size-status" class="fp-size-status fp-hidden"></span>' +
             '        <span id="fp-secrets-status" class="fp-secrets-status fp-hidden" title="Context may include node config and code. Don\'t send credentials or proprietary data. Local/private AI recommended.">⚠</span>' +
             '        <span id="fp-debug-status" class="fp-debug-status fp-hidden"></span>' +
@@ -961,7 +996,7 @@
             } else if (data.event === "conversationList") {
                 renderHistoryList(data.conversations || []);
             } else if (data.event === "settingsLoaded") {
-                fillSettings(data.settings);
+                fillSettings(data.settings, "popout settingsLoaded relay");
                 renderIntents(el("#fp-intents"));
                 updateSelectionStatus();
             }
@@ -1004,11 +1039,7 @@
                 '  </div>' +
 
                 '  <div id="fp-chat-panel" class="fp-panel">' +
-                '    <div id="fp-dev-banner" class="fp-warning">' +
-                '      <strong>Development/test only.</strong> ' +
-                '      Anything you send may leave this Node-RED instance. ' +
-                '      Don\'t include credentials or proprietary data; local/private AI recommended.' +
-                '    </div>' +
+                '    <div id="fp-update-banner" class="fp-warning fp-hidden"></div>' +
                 '    <div id="fp-messages" class="fp-messages"></div>' +
                 '    <div class="fp-compose">' +
                 '      <div class="fp-action-bar">' +
@@ -1031,6 +1062,7 @@
                 '      <div class="fp-status-strip">' +
                 '        <span id="fp-selection-status" class="fp-selection-status">No nodes selected</span>' +
                 '        <a href="#" id="fp-preview-nodes" class="fp-preview-link fp-hidden" title="Show the exact sanitized node JSON that will be sent">Preview JSON</a>' +
+                '        <span id="fp-dev-warning-status" class="fp-secrets-status" title="Development/test only. Anything you send may leave this Node-RED instance. Don\'t include credentials or proprietary data; local/private AI recommended.">⚠</span>' +
                 '        <span id="fp-size-status" class="fp-size-status fp-hidden"></span>' +
                 '        <span id="fp-secrets-status" class="fp-secrets-status fp-hidden" title="Context may include node config and code. Don\'t send credentials or proprietary data. Local/private AI recommended.">⚠</span>' +
                 '        <span id="fp-debug-status" class="fp-debug-status fp-hidden"></span>' +
@@ -1044,6 +1076,7 @@
 
                 '  <div id="fp-settings-panel" class="fp-panel fp-hidden">' +
                 '    <div class="fp-form">' +
+                '      <div id="fp-flowpilot-version" class="fp-consent-hint"></div>' +
 
                 '      <details class="fp-settings-group" open>' +
                 '      <summary title="Hangar — where your AI providers are configured">Providers</summary>' +
@@ -1073,6 +1106,11 @@
                 '      <input id="fp-model" type="text" list="fp-model-options" placeholder="Model name for this provider">' +
                 '      <datalist id="fp-model-options"></datalist>' +
                 '      <div id="fp-models-hint" class="fp-consent-hint fp-hidden"></div>' +
+                '      <label>Context window (tokens, optional)</label>' +
+                '      <input id="fp-num-ctx" type="number" min="0" step="1" placeholder="0 = unknown">' +
+                '      <div class="fp-consent-hint">If this provider/model has a known context window ' +
+                '        (e.g. 128000, 250000 — check the provider’s own docs), set it here so FlowPilot ' +
+                '        can warn before a request would overflow it. Leave at 0 if unknown.</div>' +
                 '      <label>Temperature</label>' +
                 '      <input id="fp-temperature" type="number" min="0" max="2" step="0.1" placeholder="0.2">' +
                 '      <div class="fp-consent-hint">Controls randomness. Lower (e.g. 0.2) is more ' +
@@ -1125,10 +1163,26 @@
                 '      <div class="fp-consent-hint">How long to wait for a provider response before ' +
                 '        giving up. Raise this if you\'re running a large local model on slow hardware ' +
                 '        (e.g. Ollama without a GPU) and seeing timeout errors.</div>' +
-                '      <label>Max tokens per agent turn</label>' +
+                '      <div class="fp-settings-section">Updates</div>' +
+                '      <label class="fp-checkbox-row">' +
+                '        <input id="fp-check-for-updates" type="checkbox"> ' +
+                '        Check for FlowPilot updates' +
+                '      </label>' +
+                '      <div class="fp-consent-hint">On by default. Periodically checks npm for a newer ' +
+                '        FlowPilot version on your current track (beta or stable) and shows a small ' +
+                '        banner if one is available. Uncheck to disable this entirely; no network ' +
+                '        request is made when off.</div>' +
+                '      <label>Max tokens per model response (agent step)</label>' +
                 '      <input id="fp-agent-turn-max-tokens" type="number" min="1" max="65536" step="1" placeholder="4096">' +
-                '      <div class="fp-consent-hint">Hard output limit for each tool-calling agent step. ' +
-                '        Classic (no-tools) requests are not capped.</div>' +
+                '      <div class="fp-consent-hint">Hard output limit for EACH individual tool-calling ' +
+                '        step’s model response. Classic (no-tools) requests are not capped.</div>' +
+                '      <label>Max total tokens per agent turn</label>' +
+                '      <input id="fp-agent-loop-token-ceiling" type="number" min="1" step="1" placeholder="50000">' +
+                '      <div class="fp-consent-hint">Cumulative token budget (prompt + response, summed ' +
+                '        across every step) for one multi-step agent turn before FlowPilot stops with an ' +
+                '        error instead of continuing. This is the number that appears in the "FlowPilot ' +
+                '        stopped after using N tokens" error — raise it here if you’re hitting that message ' +
+                '        on a model with a large context window.</div>' +
 
                 '      <div class="fp-settings-section">Agentic build loop</div>' +
                 '      <label>Max build/fix attempts</label>' +
@@ -1168,12 +1222,10 @@
                 '      <div class="fp-settings-section">Risk warnings</div>' +
                 '      <label class="fp-checkbox-row">' +
                 '        <input id="fp-suppress-warnings" type="checkbox"> ' +
-                '        Hide the recurring credentials/size warning bar' +
+                '        Hide the recurring credentials/size warning' +
                 '      </label>' +
-                '      <div class="fp-consent-hint">To hide the warning, check the box and type ' +
-                '        <strong>I understand the risk</strong> below. ' +
-                '        Anything you send may leave this Node-RED instance.</div>' +
-                '      <input id="fp-suppress-confirm" type="text" placeholder="Type: I understand the risk">' +
+                '      <div class="fp-consent-hint">Anything you send may leave this Node-RED ' +
+                '        instance. Check this box to hide the warning indicator near Send.</div>' +
 
                 '      <div class="fp-settings-section">Redaction</div>' +
                 '      <label class="fp-checkbox-row">' +
@@ -1547,6 +1599,7 @@
             });
 
             loadSettings();
+            checkForFlowPilotUpdates();
             updateSelectionStatus();
             showChat();
             rehydrateConversationOnLoad();
@@ -1561,6 +1614,11 @@
     // render path the History panel's click handler uses) so there is no
     // second copy of the message-rendering logic to drift out of sync.
     function rehydrateConversationOnLoad() {
+        flowpilotStorageLog("log", "rehydrateConversationOnLoad-enter", {
+            conversationIdWasRestored: conversationIdWasRestored,
+            conversationId: conversationId,
+            href: location.href
+        });
         if (!conversationIdWasRestored) { return; }
         loadConversation(conversationId, function (msg, xhr) {
             // Conversation no longer exists server-side (404) — the
@@ -1569,10 +1627,21 @@
             // failure (network hiccup, etc.) leaves it in place in case it
             // was transient. Either way: stay quiet, Chat simply stays
             // empty exactly as it does today — no error bubble on load.
+            flowpilotStorageLog("warn", "rehydrateConversationOnLoad-error", {
+                conversationId: conversationId,
+                status: xhr && xhr.status,
+                message: msg
+            });
             if (xhr && xhr.status === 404) {
-                try { sessionStorage.removeItem("fp-conversation-id"); } catch (e) { /* storage unavailable */ }
+                clearConversationId("rehydrate 404");
+                clearRunMarker("rehydrate 404");
             }
-        });
+        }, function () {
+            flowpilotStorageLog("log", "rehydrateConversationOnLoad-success", {
+                conversationId: conversationId
+            });
+            restoreInterruptedRunMarker(conversationId);
+        }, { preserveRunMarker: true });
     }
 
     window.FlowPilotCore = { initMainWindow: initMainWindow, initPopout: initPopout };