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

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 () {
-            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; }
-            $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: "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 () {
+    // 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-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; }
+            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: "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());
@@ -1080,8 +1000,14 @@
                 '      </div>' +
                 '      <label>Provider Name</label>' +
                 '      <input id="fp-provider-name" type="text" placeholder="LocalAI">' +
+                '      <label>Provider type</label>' +
+                '      <select id="fp-provider-type">' +
+                '        <option value="openai-compatible">OpenAI-compatible (LocalAI, Ollama, SGLang, OpenAI, etc.)</option>' +
+                '        <option value="anthropic">Anthropic</option>' +
+                '      </select>' +
                 '      <label>Base URL</label>' +
                 '      <input id="fp-base-url" type="text" placeholder="http://localhost:8080">' +
+                '      <div id="fp-base-url-hint" class="fp-consent-hint fp-hidden">Leave blank to use api.anthropic.com.</div>' +
                 '      <label>API Key</label>' +
                 '      <input id="fp-api-key" type="password" placeholder="Optional">' +
                 '      <label>Model</label>' +
@@ -1253,6 +1179,7 @@
             // Nothing is sent to the backend until the user explicitly
             // attaches a message and sends a request.
             try { RED.comms.subscribe("debug", onDebugMessage); } catch (e) { /* comms unavailable */ }
+            try { RED.comms.subscribe("status/#", onNodeStatus); } catch (e) { /* comms unavailable */ }
 
             // Track whether the user is scrolled to the bottom of the chat,
             // so "Cruising…"/streaming updates only auto-follow when they
@@ -1296,6 +1223,9 @@
             content.find("#fp-provider-select").on("change", function () {
                 switchProvider($(this).val());
             });
+            content.find("#fp-provider-type").on("change", function () {
+                toggleAnthropicHint($(this).val());
+            });
             content.find("#fp-add-provider").on("click", function () { addProvider(); });
             content.find("#fp-remove-provider").on("click", function () { removeProvider(); });
             content.find("#fp-test-provider").on("click", function () { testProvider(); });
@@ -1305,7 +1235,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 +1338,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);
+                } 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; }
                     }
-                    if (Array.isArray(ad.newGroups) && ad.newGroups.length) {
-                        applyGroupChanges(ad.newGroups, idMap);
-                    }
-                } 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,75 @@
     // 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 if (rec.buildConsentGate) {
+                    renderBuildConsentGate(rec);
+                } else {
+                    renderClarifyingQuestion(rec.options || []);
+                }
+                break;
+            case "review":
+                rerenderReviewRecord(rec);
+                break;
+            case "buildStep":
+                rerenderBuildStepRecord(rec);
+                break;
+            case "todo":
+                rerenderTodoRecord(rec);
+                break;
+        }
+    }
+
     // Holds the most recently loaded settings so warning logic can read the
     // user's thresholds and suppression preference without refetching.
     var currentSettings = {};
@@ -77,6 +146,7 @@
         var entry = {
             id: nextDebugMessageId++,
             timestamp: Date.now(),
+            sourceKind: "debug",
             name: msg.name || msg.id || "(unnamed node)",
             topic: redactedTopic,
             // previewValue: short, for the scannable debug-log list only.
@@ -109,8 +179,14 @@
         // message from one trigger accumulate into attachedDebugMessages
         // before review actually runs, so the model sees the full picture
         // instead of whichever message happened to arrive first.
+        // WS4: a Skip decision at the build consent gate excludes this tap's
+        // real id from skipCheckpointTapIds — its messages still arrive here
+        // (still shown in the debug-log list/updateDebugStatus above) but
+        // don't drive the auto-attach/auto-review transition, so the loop
+        // falls back to manual confirmation or the honest-timeout instead.
         if (activeBuildLoop && activeBuildLoop.waypoint === "attach" &&
-            activeBuildLoop.nodeIds.indexOf(msg.id) !== -1) {
+            activeBuildLoop.nodeIds.indexOf(msg.id) !== -1 &&
+            (activeBuildLoop.skipCheckpointTapIds || []).indexOf(msg.id) === -1) {
             attachedDebugMessages.push(entry);
             updateDebugStatus();
             if (buildLoopNoDebugTimer) { clearTimeout(buildLoopNoDebugTimer); buildLoopNoDebugTimer = null; }
@@ -129,11 +205,76 @@
         }
     }
 
+    // W3: node-status stream evidence path.
+    // Fires for every deployed node that sets a status (fill/text via
+    // node.status() in user function code, or core nodes like http-request
+    // reporting "200 OK" / error text). Topic is "status/<nodeId>".
+    // Used as fallback evidence when the loop is waiting at "attach" but no
+    // debug node exists (HTTP endpoints, background workers, etc.) — gives
+    // the model something real to review instead of the loop hanging until
+    // the 20-second honest-timeout fires.
+    // Only used when no debug payload has arrived yet; if debug output is
+    // coming the richer payload data wins — let that path take over.
+    function onNodeStatus(topic, msg) {
+        if (!activeBuildLoop || activeBuildLoop.waypoint !== "attach") { return; }
+        var nodeId = topic.split("/")[1];
+        if (!nodeId || activeBuildLoop.nodeIds.indexOf(nodeId) === -1) { return; }
+        // WS4: same Skip-decision gate as onDebugMessage, keyed on the
+        // side-effecting node's own real id here (status events report the
+        // node itself, not a debug tap wired to it).
+        if ((activeBuildLoop.skipCheckpointNodeIds || []).indexOf(nodeId) !== -1) { return; }
+        if (attachedDebugMessages.length > 0) { return; }
+
+        // Skip in-progress ("blue") statuses — e.g. http-request emits
+        // fill:"blue" text:"requesting" before any response arrives. Locking
+        // in this early status starts the debounce before the actual error or
+        // response has a chance to land, so the model reviews a placeholder
+        // rather than the real result. Only terminal states (red=error,
+        // green=success, or no fill) are useful evidence.
+        if (msg && msg.fill === "blue") { return; }
+
+        var statusText = [msg && msg.fill, msg && msg.text].filter(Boolean).join(" — ");
+        if (!statusText) { return; }
+
+        var entry = {
+            id: nextDebugMessageId++,
+            timestamp: Date.now(),
+            sourceKind: "status",
+            name: nodeId + " (node status)",
+            topic: "node-status",
+            previewValue: statusText,
+            value: "Node " + nodeId + " reported status: " + statusText
+        };
+        attachedDebugMessages.push(entry);
+        updateDebugStatus();
+
+        if (buildLoopNoDebugTimer) { clearTimeout(buildLoopNoDebugTimer); buildLoopNoDebugTimer = null; }
+        if (buildLoopAttachTimer) { clearTimeout(buildLoopAttachTimer); }
+        buildLoopAttachTimer = setTimeout(function () {
+            buildLoopAttachTimer = null;
+            if (!activeBuildLoop || activeBuildLoop.waypoint !== "attach") { return; }
+            activeBuildLoop.waypoint = "review";
+            renderLoopStepper(activeBuildLoop);
+            if (currentSettings.loopHoldStep) {
+                renderLoopCheckpoint(activeBuildLoop);
+            } else {
+                runBuildReview(activeBuildLoop);
+            }
+        }, BUILD_LOOP_ATTACH_DEBOUNCE_MS);
+    }
+
     // The exact shape sent to the backend (and shown by "Preview debug") —
     // excludes previewValue, which exists only for the debug-log list.
     function buildDebugMessagesForSend() {
         return attachedDebugMessages.map(function (m) {
-            return { id: m.id, timestamp: m.timestamp, name: m.name, topic: m.topic, value: m.value };
+            return {
+                id: m.id,
+                timestamp: m.timestamp,
+                sourceKind: m.sourceKind,
+                name: m.name,
+                topic: m.topic,
+                value: m.value
+            };
         });
     }
 
@@ -518,6 +659,7 @@
     // sees — "start a fresh conversation".
     function clearChat() {
         el("#fp-messages").empty();
+        messageRecords = [];
         relayClearMessagesToPopout();
         conversationHistory = [];
         attachedDebugMessages = [];
@@ -828,9 +970,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.
@@ -975,10 +1119,19 @@
         if (active) { $sel.val(active.id); }
     }
 
+    function toggleAnthropicHint(type) {
+        var isAnthropic = type === "anthropic";
+        el("#fp-base-url-hint").toggleClass("fp-hidden", !isAnthropic);
+        el("#fp-base-url").attr("placeholder", isAnthropic ? "Leave blank for api.anthropic.com" : "http://localhost:8080");
+    }
+
     // Write the form fields from a given provider profile.
     function fillProviderFields(p) {
         p = p || {};
         el("#fp-provider-name").val(p.providerName || "");
+        var type = p.type || "openai-compatible";
+        el("#fp-provider-type").val(type);
+        toggleAnthropicHint(type);
         el("#fp-base-url").val(p.baseUrl || "");
         el("#fp-api-key").val(p.apiKey || "");
         el("#fp-model").val(p.model || "");
@@ -1067,6 +1220,7 @@
         var ap = activeProvider();
         if (!ap) { return; }
         ap.providerName = el("#fp-provider-name").val() || "Provider";
+        ap.type = el("#fp-provider-type").val() || "openai-compatible";
         ap.baseUrl = el("#fp-base-url").val() || "";
         ap.apiKey = el("#fp-api-key").val() || "";
         ap.model = el("#fp-model").val() || "";
@@ -1263,10 +1417,10 @@
         var payload = collectSettings();
         var list = payload.providers || [];
 
-        // Validation 1: every provider needs a base URL (the one field a
-        // provider cannot function without).
+        // Validation 1: every non-Anthropic provider needs a base URL.
+        // Anthropic providers default to api.anthropic.com when baseUrl is blank.
         var noUrl = list.filter(function (p) {
-            return !p.baseUrl || !String(p.baseUrl).trim();
+            return p.type !== "anthropic" && (!p.baseUrl || !String(p.baseUrl).trim());
         });
         if (noUrl.length) {
             var urlNames = noUrl.map(function (p) { return p.providerName || "(unnamed)"; }).join(", ");
@@ -1707,4 +1861,3 @@
 
         renderChip("Open Settings", "fa fa-cog", showSettings);
     }
-