@manny-est/node-red-flowpilot 0.5.0 → 0.5.1

package/lib/core/init.js

@@ -23,7 +23,8 @@
         "- `/feedback` — bug report / feature request info\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\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\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" +
@@ -62,6 +63,7 @@
         { 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: "/demo",     desc: "Type in a demo prompt" },
         { cmd: "/help",     desc: "Show all available commands" },
         { cmd: "/feedback", desc: "Show feedback info" }
@@ -231,6 +233,10 @@
                 addMessage("assistant", FEEDBACK_TEXT);
                 if ($promptBox.length) { $promptBox.val(""); }
                 break;
+            case "/refresh":
+                refreshView();
+                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-
@@ -549,105 +555,25 @@
     // selection-status strip is a relayed MIRROR (relayStatusStripToPopout,
     // called from the parent's updateSelectionStatus/updateDebugStatus).
 
-    // Slice 3: relayed HTML for a plain Generate/Document review panel
-    // (addGeneratedReview tags these with data-fp-apply-flow — see there
-    // for why the /build loop's panels are excluded) carries the flow data
-    // right in the markup, since outerHTML serializes every attribute.
-    // Re-validating/re-rendering in the pop-out isn't needed (and wouldn't
-    // work anyway — RED.nodes.getType has no installed types in this
-    // window) — the parent already validated everything before the
-    // snapshot was relayed. This just rebinds ONE button to ask the parent
-    // to do exactly what it would do if the same button were clicked in
-    // the sidebar.
-    function bindApplyButtons($scope) {
-        // appendMessage's scope IS the tagged panel itself (a single newly
-        // relayed top-level element); initialSync's scope is the container
-        // around many descendants — cover both with filter()+find().
-        $scope.filter("[data-fp-apply-flow]").add($scope.find("[data-fp-apply-flow]")).each(function () {
-            var $panel = $(this);
-            if ($panel.data("fp-apply-bound")) { return; }
-            $panel.data("fp-apply-bound", true);
-            var flow;
-            try { flow = JSON.parse($panel.attr("data-fp-apply-flow")); } catch (e) { return; }
-            $panel.find(".fp-review-actions button.red-ui-button-primary").on("click", function () {
-                var $btn = $(this);
-                $btn.prop("disabled", true).text("Click the canvas to place…");
-                if (!window.opener || window.opener.closed) { return; }
-                try {
-                    window.opener.postMessage({ event: "applyGenerated", flow: flow }, location.origin);
-                } catch (e) { /* ignore */ }
-            });
-        });
-    }
-
-    // Same idea as bindApplyButtons, but for a relayed PLAIN Modify review
-    // panel (addModifyReview tags these with data-fp-apply-modify; a
-    // /build loop fix gets a separate tag — see bindBuildFixApplyButtons).
-    // nodeDiffs/removeNodes/newNodes/newWires already reflect the diff the
-    // parent computed against live RED.nodes state at review time; the
-    // pop-out doesn't recompute anything, it just asks the parent to run
-    // applyInsertions/applyModifications with this exact data, same as a
-    // sidebar click would.
-    function bindModifyApplyButtons($scope) {
-        $scope.filter("[data-fp-apply-modify]").add($scope.find("[data-fp-apply-modify]")).each(function () {
+    // Phase 10 0B: all review panels carry data-fp-record-id instead of the
+    // old per-kind data-fp-apply-* attribute family. The pop-out posts a
+    // recordId to the parent, which looks up the live record and applies from
+    // stored payload — no giant JSON blob in the DOM attribute, no separate
+    // bind function per review kind.
+    function bindReviewApplyButtons($scope) {
+        $scope.filter("[data-fp-record-id]").add($scope.find("[data-fp-record-id]")).each(function () {
             var $panel = $(this);
-            if ($panel.data("fp-apply-modify-bound")) { return; }
-            $panel.data("fp-apply-modify-bound", true);
-            var applyData;
-            try { applyData = JSON.parse($panel.attr("data-fp-apply-modify")); } catch (e) { return; }
+            if ($panel.data("fp-review-apply-bound")) { return; }
+            $panel.data("fp-review-apply-bound", true);
+            var recordId = parseInt($panel.attr("data-fp-record-id"), 10);
+            if (isNaN(recordId)) { return; }
             $panel.find(".fp-review-actions button.red-ui-button-primary").on("click", function () {
                 var $btn = $(this);
+                if ($btn.prop("disabled")) { return; }
                 $btn.prop("disabled", true).text("Applying…");
                 if (!window.opener || window.opener.closed) { return; }
                 try {
-                    window.opener.postMessage({ event: "applyModify", data: applyData }, location.origin);
-                } catch (e) { /* ignore */ }
-            });
-        });
-    }
-
-    // /build loop, first proposal: addGeneratedReview tags this panel
-    // (onImported set AND a buildGoal — plain Generate/Document panels get
-    // data-fp-apply-flow instead, bound above) with the flow plus the
-    // original goal text. The parent's "applyBuild" handler runs
-    // importGeneratedFlow then startBuildLoop with it, exactly like
-    // handleBuildResult's own onImported closure would.
-    function bindBuildApplyButtons($scope) {
-        $scope.filter("[data-fp-apply-build]").add($scope.find("[data-fp-apply-build]")).each(function () {
-            var $panel = $(this);
-            if ($panel.data("fp-apply-build-bound")) { return; }
-            $panel.data("fp-apply-build-bound", true);
-            var applyData;
-            try { applyData = JSON.parse($panel.attr("data-fp-apply-build")); } catch (e) { return; }
-            $panel.find(".fp-review-actions button.red-ui-button-primary").on("click", function () {
-                var $btn = $(this);
-                $btn.prop("disabled", true).text("Click the canvas to place…");
-                if (!window.opener || window.opener.closed) { return; }
-                try {
-                    window.opener.postMessage({ event: "applyBuild", data: applyData }, location.origin);
-                } catch (e) { /* ignore */ }
-            });
-        });
-    }
-
-    // /build loop, fix iterations: addModifyReview tags this panel with
-    // data-fp-apply-build-fix (instead of data-fp-apply-modify) when
-    // buildFixInfo was passed — see applyBuildLoopFix. The parent's
-    // "applyBuildFix" handler runs applyInsertions/applyBuildLoopFix
-    // with the relayed data, same loop bookkeeping a local click would do.
-    function bindBuildFixApplyButtons($scope) {
-        $scope.filter("[data-fp-apply-build-fix]").add($scope.find("[data-fp-apply-build-fix]")).each(function () {
-            var $panel = $(this);
-            if ($panel.data("fp-apply-build-fix-bound")) { return; }
-            $panel.data("fp-apply-build-fix-bound", true);
-            var applyData;
-            try { applyData = JSON.parse($panel.attr("data-fp-apply-build-fix")); } catch (e) { return; }
-            $panel.find(".fp-review-actions button.red-ui-button-primary").on("click", function () {
-                var $btn = $(this);
-                $btn.prop("disabled", true).text("Applying…");
-                if (!window.opener || window.opener.closed) { return; }
-                try {
-                    window.opener.postMessage({ event: "applyBuildFix", data: applyData }, location.origin);
+                    window.opener.postMessage({ event: "applyByRecordId", recordId: recordId }, location.origin);
                 } catch (e) { /* ignore */ }
             });
         });
@@ -903,20 +829,14 @@
             var data = evt.data || {};
             if (data.event === "initialSync") {
                 el("#fp-messages").html(data.html);
-                bindApplyButtons(el("#fp-messages"));
-                bindModifyApplyButtons(el("#fp-messages"));
-                bindBuildApplyButtons(el("#fp-messages"));
-                bindBuildFixApplyButtons(el("#fp-messages"));
+                bindReviewApplyButtons(el("#fp-messages"));
                 bindStopLoopButton(el("#fp-messages"));
                 bindTabSwitching(el("#fp-messages"));
                 bindDebugAttachButtons(el("#fp-messages"));
                 scrollMessagesToBottom(true);
             } else if (data.event === "appendMessage") {
                 el("#fp-messages").append(data.html);
-                bindApplyButtons(el("#fp-messages").children().last());
-                bindModifyApplyButtons(el("#fp-messages").children().last());
-                bindBuildApplyButtons(el("#fp-messages").children().last());
-                bindBuildFixApplyButtons(el("#fp-messages").children().last());
+                bindReviewApplyButtons(el("#fp-messages").children().last());
                 bindStopLoopButton(el("#fp-messages").children().last());
                 bindTabSwitching(el("#fp-messages").children().last());
                 bindDebugAttachButtons(el("#fp-messages").children().last());
@@ -1305,7 +1225,13 @@
 
             // Re-enable Test provider live as the user types a model.
             content.find("#fp-model").on("input", function () {
-                el("#fp-test-provider").prop("disabled", !($(this).val() || "").trim());
+                var val = ($(this).val() || "").trim();
+                el("#fp-test-provider").prop("disabled", !val);
+                // Mirror live edits to the provider status so the user can see
+                // what model will be used before hitting Save.
+                var ap = activeProvider();
+                var name = (ap && ap.providerName) || "Provider";
+                el("#fp-provider-status").text("Provider: " + (val ? (name + " / " + val) : name + " (no model)"));
             });
 
             content.find("#fp-save-settings").on("click", function () {
@@ -1402,36 +1328,35 @@
                     else { send("chat"); }
                 } else if (data.event === "runSlashCommand" && data.command) {
                     handleSlashCommand(data.command);
-                } else if (data.event === "applyGenerated" && Array.isArray(data.flow)) {
-                    importGeneratedFlow(data.flow);
-                } else if (data.event === "applyModify" && data.data) {
-                    var ad = data.data;
-                    var idMap = {};
-                    if (Array.isArray(ad.newNodes) && ad.newNodes.length) {
-                        idMap = applyInsertions(ad.newNodes, ad.newWires || [], ad.existingNodeIds || []) || {};
-                    }
-                    if (ad.hasMutations) {
-                        applyModifications(ad.nodeDiffs || [], ad.removeNodes || [], null, idMap);
-                    }
-                    if (Array.isArray(ad.newGroups) && ad.newGroups.length) {
-                        applyGroupChanges(ad.newGroups, idMap);
+                } else if (data.event === "applyByRecordId" && typeof data.recordId === "number") {
+                    var rec = null;
+                    for (var ri = 0; ri < messageRecords.length; ri++) {
+                        if (messageRecords[ri].id === data.recordId) { rec = messageRecords[ri]; break; }
                     }
-                } else if (data.event === "applyBuild" && data.data && Array.isArray(data.data.flow)) {
-                    var bd = data.data;
-                    importGeneratedFlow(bd.flow, function (importResult) {
-                        startBuildLoop(bd.goal, bd.flow, importResult);
-                    });
-                } else if (data.event === "applyBuildFix" && data.data) {
-                    var bf = data.data;
-                    var fixIdMap = {};
-                    if (Array.isArray(bf.newNodes) && bf.newNodes.length) {
-                        fixIdMap = applyInsertions(bf.newNodes, bf.newWires || [], bf.existingNodeIds || []) || {};
-                    }
-                    if (bf.hasMutations) {
-                        applyBuildLoopFix(bf.nodeDiffs || [], bf.removeNodes || [], fixIdMap, !!bf.capReached);
-                    }
-                    if (Array.isArray(bf.newGroups) && bf.newGroups.length) {
-                        applyGroupChanges(bf.newGroups, fixIdMap);
+                    if (rec && rec.kind === "review" && rec.state !== "applied") {
+                        var d0 = rec.sharedApplyData || {};
+                        var nd0 = Array.isArray(d0.nodeDiffs) ? d0.nodeDiffs : [];
+                        var rn0 = Array.isArray(d0.removeNodes) ? d0.removeNodes : [];
+                        var nn0 = Array.isArray(d0.newNodes) ? d0.newNodes : [];
+                        var nw0 = Array.isArray(d0.newWires) ? d0.newWires : [];
+                        var ng0 = Array.isArray(d0.newGroups) ? d0.newGroups : [];
+                        var eids0 = Array.isArray(d0.existingNodeIds) ? d0.existingNodeIds : [];
+                        var idMap0 = {};
+                        if (rec.subkind === "generate") {
+                            importGeneratedFlow(rec.flow || [], null);
+                        } else if (rec.subkind === "build-generate") {
+                            importGeneratedFlow(rec.flow || [], function (importResult) {
+                                startBuildLoop(rec.buildGoal || "", rec.flow || [], importResult);
+                            });
+                        } else if (rec.subkind === "modify") {
+                            if (nn0.length) { idMap0 = applyInsertions(nn0, nw0, eids0) || {}; }
+                            if (d0.hasMutations) { applyModifications(nd0, rn0, null, idMap0); }
+                            if (ng0.length) { applyGroupChanges(ng0, idMap0); }
+                        } else if (rec.subkind === "build-fix") {
+                            if (nn0.length) { idMap0 = applyInsertions(nn0, nw0, eids0) || {}; }
+                            applyBuildLoopFix(nd0, rn0, idMap0, !!d0.capReached);
+                        }
+                        rec.state = "applied";
                     }
                 } else if (data.event === "stopBuildLoop") {
                     stopBuildLoop("Build loop stopped — applied nodes remain as-is.");

package/lib/core/main.js

@@ -1,5 +1,5 @@
 
-    var VERSION = "0.5.0";
+    var VERSION = "0.5.1";
 
     // Idempotency guard: Node-RED can invoke a plugin's onadd more than once
     // in a single editor load. Without this, each call builds another #fp-root
@@ -30,6 +30,70 @@
     // settings) is pure local state and needs no flag at all.
     var isPopoutContext = false;
 
+    // ---------------------------------------------------------------------
+    // Shadow record store (Phase 10, Workstream 0A).
+    // Every code path that appends DOM to #fp-messages also appends a record
+    // here. refreshView() clears the message container and re-renders from
+    // records, restoring interactive elements without losing conversation.
+    // Records are in-memory only — no sessionStorage, no persistence.
+    // ---------------------------------------------------------------------
+    var messageRecords = [];
+    var _nextRecordId = 0;
+
+    function addRecord(kind, payload) {
+        var rec = { id: _nextRecordId++, ts: Date.now(), kind: kind };
+        if (payload) {
+            Object.keys(payload).forEach(function (k) { rec[k] = payload[k]; });
+        }
+        messageRecords.push(rec);
+        return rec;
+    }
+
+    function refreshView() {
+        var records = messageRecords.slice();
+        var $box = el("#fp-messages");
+        if (!$box.length) { return; }
+        messageRecords = [];
+        $box.empty();
+        records.forEach(rerenderRecord);
+        scrollMessagesToBottom(true);
+        // Sync pop-out: cheapest approach is a full re-sync of the
+        // refreshed HTML (same as the initial pop-out open sync).
+        if (popoutWindow && !popoutWindow.closed) {
+            try {
+                popoutWindow.postMessage({
+                    event: "initialSync",
+                    html: el("#fp-messages").html()
+                }, location.origin);
+            } catch (e) { /* ignore */ }
+        }
+    }
+
+    function rerenderRecord(rec) {
+        if (!rec) { return; }
+        switch (rec.kind) {
+            case "chat":
+                addMessage(rec.role || "assistant", rec.text || "");
+                break;
+            case "chip":
+                if (rec.chipType === "suggestedAction") { renderActionChip(rec.suggestedAction); }
+                break;
+            case "question":
+                if (rec.loopCheckpoint) {
+                    renderLoopCheckpoint(activeBuildLoop);
+                } else {
+                    renderClarifyingQuestion(rec.options || []);
+                }
+                break;
+            case "review":
+                rerenderReviewRecord(rec);
+                break;
+            case "buildStep":
+                rerenderBuildStepRecord(rec);
+                break;
+        }
+    }
+
     // Holds the most recently loaded settings so warning logic can read the
     // user's thresholds and suppression preference without refetching.
     var currentSettings = {};
@@ -518,6 +582,7 @@
     // sees — "start a fresh conversation".
     function clearChat() {
         el("#fp-messages").empty();
+        messageRecords = [];
         relayClearMessagesToPopout();
         conversationHistory = [];
         attachedDebugMessages = [];
@@ -828,9 +893,11 @@
         $("<div>").addClass("fp-md").html(renderMarkdown(text || "")).appendTo($msg);
 
         $box.append($msg);
+        var _rec = addRecord("chat", { role: role, text: text || "" });
         // Sending a message always jumps to the bottom and resumes
         // auto-follow; an incoming message only follows if already snapped.
         scrollMessagesToBottom(role === "user");
+        return _rec;
     }
 
     // Pending "typing" indicator shown in the thread while awaiting a reply.