@agent-native/core 0.172.3 → 0.172.6

package/dist/client/settings/SettingsPanel.js

@@ -12,6 +12,7 @@ import { AgentWorkspaceContent } from "../agent-page/AgentWorkspaceContent.js";
 import { AGENT_PROVIDER_CATALOG, getAgentProviderOption, providerIdForEngine, } from "../agent-provider-catalog.js";
 import { agentNativePath } from "../api-path.js";
 import { BuilderBMark } from "../builder-mark.js";
+import { fetchAgentEngineStatus, fetchEnvironmentStatus, } from "../client-status-requests.js";
 import { Popover, PopoverContent, PopoverTrigger, } from "../components/ui/popover.js";
 import { Tooltip, TooltipContent, TooltipTrigger, } from "../components/ui/tooltip.js";
 import { useT } from "../i18n.js";
@@ -332,7 +333,7 @@ const PROVIDER_DOCS = {
     "ai-sdk:mistral": "https://console.mistral.ai/api-keys/",
     "ai-sdk:cohere": "https://dashboard.cohere.com/api-keys",
 };
-function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, orgName, envManaged, credentialSource, grouped = false, open, onToggle, }) {
+function LLMSectionInner({ builderFlow, builderLoading, builderStatusAvailable, connectUrl, connected, orgName, envManaged, credentialSource, grouped = false, open, onToggle, }) {
     const isPage = useSettingsSurface() === "page";
     const t = useT();
     const [envKeys, setEnvKeys] = useState([]);
@@ -354,24 +355,40 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
     const [testResult, setTestResult] = useState(null);
     const [settingsStatus, setSettingsStatus] = useState(null);
     const [disconnectError, setDisconnectError] = useState(null);
-    const [envLoaded, setEnvLoaded] = useState(false);
+    const [envProbeAvailable, setEnvProbeAvailable] = useState(false);
     const [enginesLoaded, setEnginesLoaded] = useState(false);
-    const [statusLoaded, setStatusLoaded] = useState(false);
-    const initialLoading = !envLoaded || !enginesLoaded || !statusLoaded || !!builderLoading;
-    useEffect(() => {
-        fetch(agentNativePath("/_agent-native/env-status"))
-            .then((r) => (r.ok ? r.json() : []))
-            .then(setEnvKeys)
-            .catch(() => { })
-            .finally(() => setEnvLoaded(true));
-    }, [saved]);
+    const [statusProbeAvailable, setStatusProbeAvailable] = useState(false);
+    const probeGenerationRef = useRef({ env: 0, status: 0 });
+    const initialLoading = !enginesLoaded || !!builderLoading;
+    const refreshEnvKeys = useCallback(() => {
+        const generation = ++probeGenerationRef.current.env;
+        setEnvProbeAvailable(false);
+        setEnvKeys([]);
+        void fetchEnvironmentStatus().then((result) => {
+            if (generation !== probeGenerationRef.current.env)
+                return;
+            if (result.state !== "available" || !Array.isArray(result.value)) {
+                return;
+            }
+            setEnvKeys(result.value);
+            setEnvProbeAvailable(true);
+        });
+    }, []);
     const notifyConfigChanged = useCallback(() => {
         window.dispatchEvent(new CustomEvent("agent-engine:configured-changed"));
     }, []);
     const refreshSettingsStatus = useCallback(() => {
-        fetch(agentNativePath("/_agent-native/agent-engine/status"))
-            .then((r) => (r.ok ? r.json() : null))
-            .then((data) => {
+        const generation = ++probeGenerationRef.current.status;
+        setStatusProbeAvailable(false);
+        setSettingsStatus(null);
+        setBaseUrlConfigured(false);
+        void fetchAgentEngineStatus()
+            .then((result) => {
+            if (generation !== probeGenerationRef.current.status)
+                return;
+            if (result.state !== "available")
+                return;
+            const data = result.value;
             setBaseUrlConfigured(Boolean(data?.openAiBaseUrlConfigured));
             if (data?.configured &&
                 typeof data.engine === "string" &&
@@ -387,13 +404,34 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
             else {
                 setSettingsStatus(null);
             }
+            setStatusProbeAvailable(true);
         })
-            .catch(() => { })
-            .finally(() => setStatusLoaded(true));
+            .catch(() => { });
     }, []);
+    useEffect(() => {
+        refreshEnvKeys();
+    }, [refreshEnvKeys]);
     useEffect(() => {
         refreshSettingsStatus();
     }, [refreshSettingsStatus]);
+    useEffect(() => {
+        const refresh = () => {
+            refreshEnvKeys();
+            refreshSettingsStatus();
+        };
+        window.addEventListener("agent-engine:configured-changed", refresh);
+        window.addEventListener("focus", refresh);
+        const onVisibilityChange = () => {
+            if (document.visibilityState === "visible")
+                refresh();
+        };
+        document.addEventListener("visibilitychange", onVisibilityChange);
+        return () => {
+            window.removeEventListener("agent-engine:configured-changed", refresh);
+            window.removeEventListener("focus", refresh);
+            document.removeEventListener("visibilitychange", onVisibilityChange);
+        };
+    }, [refreshEnvKeys, refreshSettingsStatus]);
     useEffect(() => {
         callAction("manage-agent-engine", { action: "list" })
             .then((data) => {
@@ -441,14 +479,21 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
         }
         return configured;
     }, [envKeys, settingsStatus]);
-    const builderConnected = connected || builderFlow.configured;
-    const anyKeyConfigured = builderConnected ||
-        (selectedEnginePackageInstalled &&
-            (envConfigured || settingsConfigured || configuredProviderIds.size > 0));
+    const builderConnected = builderStatusAvailable && (connected || builderFlow.configured);
+    const configurationKnown = envProbeAvailable && statusProbeAvailable;
+    const builderEngineSelected = selectedEngine === "builder";
+    const selectedConfigurationKnown = builderEngineSelected
+        ? builderStatusAvailable
+        : configurationKnown;
+    const anyKeyConfigured = (builderEngineSelected && builderConnected) ||
+        (!builderEngineSelected &&
+            configurationKnown &&
+            selectedEnginePackageInstalled &&
+            (envConfigured || settingsConfigured));
     const sourceBadge = computeSourceBadge({
-        settingsConfigured,
-        settingsStatus,
-        envConfigured,
+        settingsConfigured: configurationKnown && settingsConfigured,
+        settingsStatus: configurationKnown ? settingsStatus : null,
+        envConfigured: configurationKnown && envConfigured,
         envVar,
         builderConnected,
     });
@@ -479,7 +524,6 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
                 setBaseUrlConfigured(true);
             if (clearBaseUrl)
                 setBaseUrlConfigured(false);
-            refreshSettingsStatus();
             notifyConfigChanged();
             setTimeout(() => setSaved(false), 2000);
         }
@@ -496,7 +540,6 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
             if (res.ok) {
                 setTestResult(null);
                 setApplyNote(false);
-                refreshSettingsStatus();
                 notifyConfigChanged();
                 return;
             }
@@ -560,7 +603,6 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
             setCurrentEngine(selectedEngine);
             setCurrentModel(selectedModel);
             setApplyNote(true);
-            refreshSettingsStatus();
             notifyConfigChanged();
             setTimeout(() => setApplyNote(false), 4000);
         }
@@ -568,7 +610,9 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
             setApplyError(err instanceof Error ? err.message : String(err));
         }
     };
-    return (_jsx(SettingsSection, { id: settingsSectionDomId("llm"), icon: _jsx(IconBrain, { size: 14 }), title: "LLM", required: true, connected: initialLoading ? undefined : anyKeyConfigured, subtitle: isPage
+    return (_jsx(SettingsSection, { id: settingsSectionDomId("llm"), icon: _jsx(IconBrain, { size: 14 }), title: "LLM", required: true, connected: initialLoading || !selectedConfigurationKnown
+            ? undefined
+            : anyKeyConfigured, subtitle: isPage
             ? undefined
             : t("agentPanel.builderOrOwnKeys", {
                 defaultValue: "Choose Builder.io or custom keys.",
@@ -576,11 +620,11 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
                 ? "flex items-center justify-between gap-4 px-5 py-4 sm:px-6"
                 : "flex items-center justify-between gap-2"), children: [isPage && (_jsxs("div", { className: "min-w-0", children: [_jsx("p", { className: "text-sm font-medium text-foreground", children: "AI provider" }), _jsx("p", { className: "mt-1 text-sm leading-6 text-muted-foreground", children: t("agentPanel.builderOrOwnKeys", {
                                 defaultValue: "Choose Builder.io or custom keys.",
-                            }) })] })), _jsxs("div", { className: cn("flex flex-wrap items-center justify-end gap-2"), children: [!anyKeyConfigured && (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "llm_settings", trackingFlow: "connect_llm", label: "Connect Builder.io", compact: true })), _jsx(ManualSetupCard, { id: "llm-manual-setup", title: "Custom keys", hint: manualSetupHint, sourceBadge: builderConnected ? undefined : sourceBadge, bare: isPage, popover: true, popoverLabel: settingsConfigured
+                            }) })] })), _jsxs("div", { className: cn("flex flex-wrap items-center justify-end gap-2"), children: [selectedConfigurationKnown && !anyKeyConfigured && (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: builderConnected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "llm_settings", trackingFlow: "connect_llm", label: "Connect Builder.io", compact: true })), _jsx(ManualSetupCard, { id: "llm-manual-setup", title: "Custom keys", hint: manualSetupHint, sourceBadge: builderConnected ? undefined : sourceBadge, bare: isPage, popover: true, popoverLabel: settingsConfigured
                                 ? "Manage"
                                 : t("agentPanel.addOwnKeys", {
                                     defaultValue: "Custom keys",
-                                }), summaryContent: anyKeyConfigured ? (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "llm_settings", trackingFlow: "connect_llm", label: "Connect Builder.io" })) : undefined, children: _jsxs("div", { className: "space-y-2 mb-1", children: [_jsx(AgentProviderPicker, { value: selectedProvider, configuredProviders: configuredProviderIds, layout: isPage ? "page" : "compact", onChange: (provider) => {
+                                }), summaryContent: selectedConfigurationKnown && anyKeyConfigured ? (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: builderConnected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "llm_settings", trackingFlow: "connect_llm", label: "Connect Builder.io" })) : undefined, children: _jsxs("div", { className: "space-y-2 mb-1", children: [_jsx(AgentProviderPicker, { value: selectedProvider, configuredProviders: configurationKnown ? configuredProviderIds : undefined, layout: isPage ? "page" : "compact", onChange: (provider) => {
                                             const option = getAgentProviderOption(provider);
                                             setSelectedEngine(option.engine);
                                             setSelectedModel(option.defaultModel);
@@ -611,7 +655,9 @@ function LLMSectionInner({ builderFlow, builderLoading, connectUrl, connected, o
                                                 : `${envVar} configured`] })) : envVar ? (_jsxs("div", { className: "flex gap-1.5", children: [_jsx("input", { type: "password", value: apiKey, onChange: (e) => setApiKey(e.target.value), onKeyDown: (e) => {
                                                     if (e.key === "Enter")
                                                         handleSave();
-                                                }, placeholder: PROVIDER_ENV_PLACEHOLDERS[envVar] ?? "...", className: cn(textInputClass(isPage), "flex-1"), style: isPage ? CONTROL_STYLE_PAGE : undefined }), _jsx(Button, { intent: "primary", emphasis: "solid", onClick: handleSave, disabled: !providerSettingsChanged || saving, className: pillButtonClass(isPage, "solid"), children: saving ? (_jsx(IconLoader2, { size: isPage ? 14 : 10, className: "animate-spin" })) : saved ? (_jsx(IconCheck, { size: isPage ? 14 : 10 })) : ("Save") })] })) : null, _jsxs("div", { className: "flex items-center gap-2", children: [_jsx(Button, { intent: "neutral", emphasis: "outline", onClick: handleTest, disabled: testing, className: pillButtonClass(isPage, "outline"), children: testing ? (_jsxs("span", { className: "flex items-center gap-1", children: [_jsx(IconLoader2, { size: isPage ? 14 : 10, className: "animate-spin" }), "Testing\u2026"] })) : ("Test") }), PROVIDER_DOCS[selectedEngine] ? (_jsxs("a", { href: PROVIDER_DOCS[selectedEngine], target: "_blank", rel: "noopener noreferrer", className: cn(pillButtonClass(isPage, "outline"), "no-underline"), children: ["Get an API key", _jsx(IconExternalLink, { size: isPage ? 14 : 10 })] })) : null, engineChanged && (_jsx(Button, { intent: "primary", emphasis: "solid", onClick: handleApply, className: pillButtonClass(isPage, "solid"), children: "Apply" })), settingsStatus != null && (_jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx(Button, { intent: "danger", emphasis: "outline", onClick: handleDisconnect, className: cn(pillButtonClass(isPage, "outline"), "ms-auto text-muted-foreground hover:bg-destructive/10 hover:text-destructive hover:border-destructive/40"), children: "Disconnect" }) }), _jsx(TooltipContent, { children: "Clear the saved engine \u2014 the app will fall back to the default until you re-apply." })] }))] }), testResult && testResult.ok && (_jsxs("p", { className: cn("flex items-center gap-1 text-primary", isPage ? "text-xs" : "text-[10px]"), children: [_jsx(IconCheck, { size: isPage ? 14 : 10 }), "Test passed \u2014 ", testResult.latencyMs, "ms"] })), testResult && testResult.ok === false && (_jsxs("p", { className: cn("text-destructive", isPage ? "text-xs" : "text-[10px]"), children: ["Test failed: ", testResult.error] })), disconnectError && (_jsxs("p", { className: cn("text-destructive", isPage ? "text-xs" : "text-[10px]"), children: ["Disconnect failed: ", disconnectError] })), applyError && (_jsxs("p", { className: cn("text-destructive", isPage ? "text-xs" : "text-[10px]"), children: ["Apply failed: ", applyError] })), applyNote && (_jsx("p", { className: cn("text-muted-foreground", isPage ? "text-xs" : "text-[10px]"), children: "Changes take effect on next conversation" }))] }) })] })] })) }));
+                                                }, placeholder: PROVIDER_ENV_PLACEHOLDERS[envVar] ?? "...", className: cn(textInputClass(isPage), "flex-1"), style: isPage ? CONTROL_STYLE_PAGE : undefined }), _jsx(Button, { intent: "primary", emphasis: "solid", onClick: handleSave, disabled: !providerSettingsChanged || saving, className: pillButtonClass(isPage, "solid"), children: saving ? (_jsx(IconLoader2, { size: isPage ? 14 : 10, className: "animate-spin" })) : saved ? (_jsx(IconCheck, { size: isPage ? 14 : 10 })) : ("Save") })] })) : null, _jsxs("div", { className: "flex items-center gap-2", children: [_jsx(Button, { intent: "neutral", emphasis: "outline", onClick: handleTest, disabled: testing ||
+                                                    !selectedConfigurationKnown ||
+                                                    !anyKeyConfigured, className: pillButtonClass(isPage, "outline"), children: testing ? (_jsxs("span", { className: "flex items-center gap-1", children: [_jsx(IconLoader2, { size: isPage ? 14 : 10, className: "animate-spin" }), "Testing\u2026"] })) : ("Test") }), PROVIDER_DOCS[selectedEngine] ? (_jsxs("a", { href: PROVIDER_DOCS[selectedEngine], target: "_blank", rel: "noopener noreferrer", className: cn(pillButtonClass(isPage, "outline"), "no-underline"), children: ["Get an API key", _jsx(IconExternalLink, { size: isPage ? 14 : 10 })] })) : null, engineChanged && (_jsx(Button, { intent: "primary", emphasis: "solid", onClick: handleApply, className: pillButtonClass(isPage, "solid"), children: "Apply" })), settingsStatus != null && (_jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx(Button, { intent: "danger", emphasis: "outline", onClick: handleDisconnect, className: cn(pillButtonClass(isPage, "outline"), "ms-auto text-muted-foreground hover:bg-destructive/10 hover:text-destructive hover:border-destructive/40"), children: "Disconnect" }) }), _jsx(TooltipContent, { children: "Clear the saved engine \u2014 the app will fall back to the default until you re-apply." })] }))] }), testResult && testResult.ok && (_jsxs("p", { className: cn("flex items-center gap-1 text-primary", isPage ? "text-xs" : "text-[10px]"), children: [_jsx(IconCheck, { size: isPage ? 14 : 10 }), "Test passed \u2014 ", testResult.latencyMs, "ms"] })), testResult && testResult.ok === false && (_jsxs("p", { className: cn("text-destructive", isPage ? "text-xs" : "text-[10px]"), children: ["Test failed: ", testResult.error] })), disconnectError && (_jsxs("p", { className: cn("text-destructive", isPage ? "text-xs" : "text-[10px]"), children: ["Disconnect failed: ", disconnectError] })), applyError && (_jsxs("p", { className: cn("text-destructive", isPage ? "text-xs" : "text-[10px]"), children: ["Apply failed: ", applyError] })), applyNote && (_jsx("p", { className: cn("text-muted-foreground", isPage ? "text-xs" : "text-[10px]"), children: "Changes take effect on next conversation" }))] }) })] })] })) }));
 }
 function friendlyAppName(appId) {
     return appId
@@ -1056,10 +1102,11 @@ function CapabilityStatusStrip({ isDevMode, builderConnected, builderLoading, bu
     return (_jsxs("div", { className: "rounded-md border border-border bg-muted/20 px-2.5 py-2", children: [_jsx("div", { className: "mb-1.5 text-[10px] font-medium text-muted-foreground", children: "Available now" }), _jsxs("div", { className: "space-y-1.5", children: [_jsx(CapabilityStatusRow, { label: "App", value: "Chat + actions", active: true }), _jsx(CapabilityStatusRow, { label: "Code", value: codeLabel, active: codeAvailable }), _jsx(CapabilityStatusRow, { label: "Builder", active: builderConnected, value: builderLoading ? (_jsx(Skeleton, { className: "h-3 w-16" })) : builderConnected ? ("Connected") : (_jsx(Button, { type: "button", intent: "neutral", emphasis: "outline", onClick: onOpenLlm, className: "rounded border border-border px-1.5 py-0.5 text-[10px] text-muted-foreground transition-colors hover:bg-accent/40 hover:text-foreground", children: "Connect" })) })] })] }));
 }
 function SettingsPanelContent({ isDevMode, initialSection, sectionRequestKey, sections = ALL_SETTINGS_SECTIONS, showCapabilityStrip = true, className, surface = "sidebar", builderConnectionOwnedExternally = false, agentAdditionalContent, }) {
-    const { status: builder, loading: builderLoading } = useBuilderStatus({
+    const { status: builder, loading: builderLoading, stale: builderStatusStale, } = useBuilderStatus({
         enabled: !builderConnectionOwnedExternally,
     });
     const connected = builder?.configured ?? false;
+    const builderStatusAvailable = !builderLoading && !builderStatusStale && builder != null;
     const connectUrl = builder?.connectUrl;
     const orgName = builder?.orgName;
     const envManaged = !!builder?.envManaged;
@@ -1088,7 +1135,7 @@ function SettingsPanelContent({ isDevMode, initialSection, sectionRequestKey, se
     const isPage = surface === "page";
     const isWorkspacePage = isPage && sections.includes("hosting");
     return (_jsx(SettingsSurfaceProvider, { surface: surface, children: _jsxs("div", { className: cn(isPage ? "space-y-8" : "flex-1 min-h-0 overflow-y-auto p-3 space-y-2", className), style: isPage ? undefined : { overflowY: "auto" }, children: [showCapabilityStrip && (_jsx(CapabilityStatusStrip, { isDevMode: isDevMode, builderConnected: connected, builderLoading: builderLoading, builderBranchesAvailable: builderBranchesAvailable, onOpenLlm: () => openSettingsSection("llm", true) })), isPage &&
-                    ["llm", "app-models", "limits", "voice"].some((section) => shouldShowSection(section)) && (_jsxs(SettingsGroup, { title: "Agent", children: [shouldShowSection("llm") && (_jsx(LLMSectionInner, { builderFlow: builderFlow, builderLoading: builderLoading, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, grouped: true, open: openSection === "llm", onToggle: () => toggle("llm") })), shouldShowSection("app-models") && (_jsx(AppModelDefaultsSectionInner, { grouped: true, open: openSection === "app-models", onToggle: () => toggle("app-models") })), shouldShowSection("limits") && (_jsx(AgentLimitsSectionInner, { grouped: true, open: openSection === "limits", onToggle: () => toggle("limits") })), shouldShowSection("voice") && (_jsx(SettingsSection, { id: settingsSectionDomId("voice"), icon: _jsx(IconMicrophone, { size: 14 }), title: "Voice Transcription", subtitle: "Choose how voice input is transcribed.", grouped: true, flat: true, open: openSection === "voice", onToggle: () => toggle("voice"), children: _jsx(VoiceTranscriptionSection, { compact: true }) }))] })), isPage && agentAdditionalContent ? (_jsx(SettingsGroup, { title: "Notifications", children: agentAdditionalContent })) : null, isPage &&
+                    ["llm", "app-models", "limits", "voice"].some((section) => shouldShowSection(section)) && (_jsxs(SettingsGroup, { title: "Agent", children: [shouldShowSection("llm") && (_jsx(LLMSectionInner, { builderFlow: builderFlow, builderLoading: builderLoading, builderStatusAvailable: builderStatusAvailable, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, grouped: true, open: openSection === "llm", onToggle: () => toggle("llm") })), shouldShowSection("app-models") && (_jsx(AppModelDefaultsSectionInner, { grouped: true, open: openSection === "app-models", onToggle: () => toggle("app-models") })), shouldShowSection("limits") && (_jsx(AgentLimitsSectionInner, { grouped: true, open: openSection === "limits", onToggle: () => toggle("limits") })), shouldShowSection("voice") && (_jsx(SettingsSection, { id: settingsSectionDomId("voice"), icon: _jsx(IconMicrophone, { size: 14 }), title: "Voice Transcription", subtitle: "Choose how voice input is transcribed.", grouped: true, flat: true, open: openSection === "voice", onToggle: () => toggle("voice"), children: _jsx(VoiceTranscriptionSection, { compact: true }) }))] })), isPage && agentAdditionalContent ? (_jsx(SettingsGroup, { title: "Notifications", children: agentAdditionalContent })) : null, isPage &&
                     ["automations", "background"].some((section) => shouldShowSection(section)) && (_jsxs(SettingsGroup, { title: "Agent workflows", children: [shouldShowSection("automations") && (_jsx(SettingsSection, { id: settingsSectionDomId("automations"), icon: _jsx(IconBolt, { size: 14 }), title: "Automations", subtitle: "Scheduled and event-triggered agent tasks.", grouped: true, flat: true, open: openSection === "automations", onToggle: () => toggle("automations"), children: _jsx(SettingsRow, { label: "Automations", description: "Schedule agent tasks or run them from events.", control: _jsxs("a", { href: "/settings/agent/automations", className: "inline-flex items-center gap-1 rounded-md border border-border px-3 py-1.5 text-sm font-medium text-foreground no-underline transition-colors hover:bg-accent/40", children: ["Open automations", _jsx(IconExternalLink, { size: 14 })] }) }) })), shouldShowSection("background") && (_jsx(SettingsSection, { id: settingsSectionDomId("background"), icon: _jsx(IconGitBranch, { size: 14 }), title: "Background Agent", subtitle: "Make code changes from production mode via Builder.", grouped: true, flat: true, connected: connected, open: openSection === "background", onToggle: () => toggle("background"), children: _jsx(SettingsRow, { label: "Background agent", description: "Make code changes from production mode via Builder.", control: _jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "background_agent_settings", trackingFlow: "background_agent", compact: true }) }) }))] })), isWorkspacePage && (_jsxs(SettingsGroup, { title: "Workspace", children: [shouldShowSection("demo-mode") && (_jsx(SettingsRow, { id: settingsSectionDomId("demo-mode"), label: "Demo mode", description: "Use sample data in this browser for presentations.", control: _jsx(DemoModeSection, { compact: true }) })), shouldShowSection("hosting") && (_jsx(SettingsRow, { id: settingsSectionDomId("hosting"), label: "Hosting", description: "Deploy the app to the cloud.", control: _jsxs("div", { className: "flex flex-wrap items-center justify-end gap-2", children: [!connected && (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "hosting_settings", trackingFlow: "hosting", compact: true })), _jsx(ManualSetupCard, { title: "Set up manually", hint: "Deploy manually to Netlify, Vercel, Cloudflare, or any Nitro-supported target.", docsUrl: docsUrl("deployment", {
                                             campaign: "onboarding",
                                             content: "deployment_settings",
@@ -1101,7 +1148,7 @@ function SettingsPanelContent({ isDevMode, initialSection, sectionRequestKey, se
                                         }), dim: connected, bare: true, popover: true, popoverLabel: "Manage", summaryContent: connected ? (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "file_upload_settings", trackingFlow: "file_upload" })) : undefined, children: _jsx(FileStorageSettingsForm, {}) })] }) })), shouldShowSection("auth") && (_jsx(SettingsRow, { id: settingsSectionDomId("auth"), label: "Authentication", description: "Set up sign-in and access control.", control: _jsxs("div", { className: "flex flex-wrap items-center justify-end gap-2", children: [!connected && (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "auth_settings", trackingFlow: "auth", compact: true })), _jsx(ManualSetupCard, { title: "Set up manually", hint: "Configure Better Auth and optional Google or GitHub providers.", docsUrl: docsUrl("authentication", {
                                             campaign: "onboarding",
                                             content: "authentication_settings",
-                                        }), dim: connected, bare: true, popover: true, popoverLabel: "Manage", summaryContent: connected ? (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "auth_settings", trackingFlow: "auth" })) : undefined })] }) }))] })), !isPage && shouldShowSection("llm") && (_jsx(LLMSectionInner, { builderFlow: builderFlow, builderLoading: builderLoading, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, open: openSection === "llm", onToggle: () => toggle("llm") })), !isPage && shouldShowSection("app-models") && (_jsx(AppModelDefaultsSectionInner, { open: openSection === "app-models", onToggle: () => toggle("app-models") })), !isPage && shouldShowSection("limits") && (_jsx(AgentLimitsSectionInner, { open: openSection === "limits", onToggle: () => toggle("limits") })), !isPage && shouldShowSection("voice") && (_jsx(SettingsSection, { id: settingsSectionDomId("voice"), icon: _jsx(IconMicrophone, { size: 14 }), title: "Voice Transcription", subtitle: "How the composer microphone turns your voice into text.", flat: true, open: openSection === "voice", onToggle: () => toggle("voice"), children: _jsx(VoiceTranscriptionSection, {}) })), !isPage && shouldShowSection("demo-mode") && (_jsx(SettingsSection, { id: settingsSectionDomId("demo-mode"), icon: _jsx(IconEyeOff, { size: 14 }), title: "Demo mode", subtitle: "Replace displayed emails with realistic fake data in this browser and reshape supported charts for presentations. Backend, agent integrations, and agent results stay real and access-scoped.", flat: true, open: openSection === "demo-mode", onToggle: () => toggle("demo-mode"), children: _jsx(DemoModeSection, {}) })), !isPage && shouldShowSection("automations") && (_jsx(SettingsSection, { id: settingsSectionDomId("automations"), icon: _jsx(IconBolt, { size: 14 }), title: "Automations", subtitle: "Scheduled and event-triggered agent tasks.", flat: true, open: openSection === "automations", onToggle: () => toggle("automations"), children: _jsx(AutomationsSection, {}) })), shouldShowSection("secrets") && (_jsx(SettingsSection, { id: settingsSectionDomId("secrets"), icon: _jsx(IconKey, { size: 14 }), title: "API keys", subtitle: "Service credentials and automation keys.", flat: true, open: openSection === "secrets", onToggle: () => toggle("secrets"), children: _jsx(SecretsSection, { focusKey: focusSecretKey }) })), !isPage && shouldShowSection("hosting") && (_jsx(SettingsSection, { id: settingsSectionDomId("hosting"), icon: _jsx(IconCloud, { size: 14 }), title: "Hosting", subtitle: "Deploy your app to the cloud.", flat: true, connected: connected, open: openSection === "hosting", onToggle: () => toggle("hosting"), children: _jsxs("div", { className: "space-y-2", children: [_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "hosting_settings", trackingFlow: "hosting" }), _jsx(ManualSetupCard, { hint: "Deploy manually to Netlify, Vercel, Cloudflare, or any Nitro-supported target.", docsUrl: docsUrl("deployment", {
+                                        }), dim: connected, bare: true, popover: true, popoverLabel: "Manage", summaryContent: connected ? (_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "auth_settings", trackingFlow: "auth" })) : undefined })] }) }))] })), !isPage && shouldShowSection("llm") && (_jsx(LLMSectionInner, { builderFlow: builderFlow, builderLoading: builderLoading, builderStatusAvailable: builderStatusAvailable, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, open: openSection === "llm", onToggle: () => toggle("llm") })), !isPage && shouldShowSection("app-models") && (_jsx(AppModelDefaultsSectionInner, { open: openSection === "app-models", onToggle: () => toggle("app-models") })), !isPage && shouldShowSection("limits") && (_jsx(AgentLimitsSectionInner, { open: openSection === "limits", onToggle: () => toggle("limits") })), !isPage && shouldShowSection("voice") && (_jsx(SettingsSection, { id: settingsSectionDomId("voice"), icon: _jsx(IconMicrophone, { size: 14 }), title: "Voice Transcription", subtitle: "How the composer microphone turns your voice into text.", flat: true, open: openSection === "voice", onToggle: () => toggle("voice"), children: _jsx(VoiceTranscriptionSection, {}) })), !isPage && shouldShowSection("demo-mode") && (_jsx(SettingsSection, { id: settingsSectionDomId("demo-mode"), icon: _jsx(IconEyeOff, { size: 14 }), title: "Demo mode", subtitle: "Replace displayed emails with realistic fake data in this browser and reshape supported charts for presentations. Backend, agent integrations, and agent results stay real and access-scoped.", flat: true, open: openSection === "demo-mode", onToggle: () => toggle("demo-mode"), children: _jsx(DemoModeSection, {}) })), !isPage && shouldShowSection("automations") && (_jsx(SettingsSection, { id: settingsSectionDomId("automations"), icon: _jsx(IconBolt, { size: 14 }), title: "Automations", subtitle: "Scheduled and event-triggered agent tasks.", flat: true, open: openSection === "automations", onToggle: () => toggle("automations"), children: _jsx(AutomationsSection, {}) })), shouldShowSection("secrets") && (_jsx(SettingsSection, { id: settingsSectionDomId("secrets"), icon: _jsx(IconKey, { size: 14 }), title: "API keys", subtitle: "Service credentials and automation keys.", flat: true, open: openSection === "secrets", onToggle: () => toggle("secrets"), children: _jsx(SecretsSection, { focusKey: focusSecretKey }) })), !isPage && shouldShowSection("hosting") && (_jsx(SettingsSection, { id: settingsSectionDomId("hosting"), icon: _jsx(IconCloud, { size: 14 }), title: "Hosting", subtitle: "Deploy your app to the cloud.", flat: true, connected: connected, open: openSection === "hosting", onToggle: () => toggle("hosting"), children: _jsxs("div", { className: "space-y-2", children: [_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "hosting_settings", trackingFlow: "hosting" }), _jsx(ManualSetupCard, { hint: "Deploy manually to Netlify, Vercel, Cloudflare, or any Nitro-supported target.", docsUrl: docsUrl("deployment", {
                                     campaign: "onboarding",
                                     content: "deployment_settings",
                                 }), dim: connected })] }) })), !isPage && shouldShowSection("database") && (_jsx(SettingsSection, { id: settingsSectionDomId("database"), icon: _jsx(IconDatabase, { size: 14 }), title: "Database", subtitle: "Connect a cloud database for persistent storage.", flat: true, connected: connected, open: openSection === "database", onToggle: () => toggle("database"), children: _jsxs("div", { className: "space-y-2", children: [_jsx(UseBuilderCard, { builderFlow: builderFlow, connectUrl: connectUrl, connected: connected, orgName: orgName, envManaged: envManaged, credentialSource: credentialSource, trackingSource: "database_settings", trackingFlow: "database" }), _jsx(ManualSetupCard, { hint: "Set DATABASE_URL in your .env to connect Neon, Supabase, Turso, any Postgres/SQLite database, or local PGlite with pglite:./data/pglite.", docsUrl: docsUrl("database", {

package/dist/file-upload/actions/upload-image.d.ts

@@ -17,11 +17,11 @@ declare const _default: import("../../action.js").ActionDefinition<{
     id?: undefined;
     provider?: undefined;
 } | {
+    error?: undefined;
     configured?: undefined;
     connectPath?: undefined;
     url: string;
     id: string;
     provider: string;
-    error?: undefined;
 }>;
 export default _default;

package/dist/jobs/run-now.js

@@ -1,5 +1,5 @@
 import { AGENT_CHAT_BACKGROUND_RUN_FIELD, dispatchPathTargetsNetlifyBackgroundFunction, resolveAgentChatProcessRunDispatchPath, } from "../agent/durable-background.js";
-import { canUpdateAutomationResource, } from "../automations/service.js";
+import { canQueueAutomationRunNow, } from "../automations/service.js";
 import { isLocalDatabase } from "../db/client.js";
 import { organizationResourceOwner, resourceGetByPath, } from "../resources/store.js";
 import { fireInternalDispatch } from "../server/self-dispatch.js";
@@ -77,7 +77,7 @@ export async function queueAutomationRunNow(input) {
             statusCode: 404,
         });
     }
-    if (!(await canUpdateAutomationResource(input, resource))) {
+    if (!(await canQueueAutomationRunNow(input, resource, input.scope))) {
         throw Object.assign(new Error("Only the automation's creator or an organization admin can run it."), { statusCode: 403 });
     }
     const { body } = parseJobResource(resource.content);

package/dist/observability/posthog-ai.d.ts

@@ -24,10 +24,10 @@
  * schema, and a name it does not define today it may define tomorrow with
  * different meaning — `input_truncated` / `spans_dropped` cannot collide.
  *
- * Content (`$ai_input` / `$ai_output_choices` / `$ai_input_state` /
- * `$ai_output_state`) is gated on config and always OMITTED when disabled.
- * Sending `[]` instead would be indistinguishable from a run that genuinely had
- * no messages.
+ * Content (a generation's `$ai_input` / `$ai_output_choices`, a span's
+ * `$ai_input_state` / `$ai_output_state`) is gated on config and always OMITTED
+ * when disabled. Sending `[]` instead would be indistinguishable from a run
+ * that genuinely had no messages.
  *
  * @see https://posthog.com/docs/ai-observability/traces
  * @see https://posthog.com/docs/ai-observability/spans
@@ -42,12 +42,18 @@ export interface AiErrorDetail {
     terminal_state?: string;
     retryable?: boolean;
 }
+export declare function resolveAiError(isError: boolean, error: AiErrorDetail | undefined): AiErrorDetail | undefined;
 /**
  * Serialize a content value under a byte ceiling.
  *
- * Returns `{ truncated: true }` with a placeholder rather than a silently
- * shortened payload — a trace that shows half a conversation as if it were the
- * whole one is worse than one that says it was cut.
+ * A message list keeps the last user message behind a marker entry naming what
+ * was dropped — what was asked is what a trace is opened for, and it stays
+ * small. Replacing the whole conversation with a placeholder, which is what
+ * this did, left nothing at all.
+ *
+ * Anything else (a string, an object) still becomes a placeholder: there is no
+ * message to keep. Either way `truncated` is true, so a cut payload can never
+ * be read as a complete one.
  */
 export declare function boundAiContent(value: unknown): {
     value: unknown;
@@ -66,6 +72,10 @@ export interface AiTraceEventInput {
     durationMs: number;
     isError: boolean;
     error?: AiErrorDetail;
+    /** What KIND of failure this was — a terminal code, a cut-off reason, an
+     *  engine error code. A classification, never content, so it ships even when
+     *  the message is withheld. */
+    errorType?: string;
     inputTokens?: number;
     outputTokens?: number;
     costUsd?: number;
@@ -73,11 +83,15 @@ export interface AiTraceEventInput {
     /** Browser session id, when the run originated from a page. Links the trace
      *  to PostHog session replay. */
     browserSessionId?: string;
-    /** Omitted unless `capturePrompts` is on. */
-    inputState?: unknown;
-    outputState?: unknown;
     extraProperties?: Record<string, unknown>;
 }
+/**
+ * The run's prompt and answer are NOT carried here. Every round-trip emits a
+ * generation with its own `$ai_input` / `$ai_output_choices`, so a trace-level
+ * copy repeated the first call's prompt and the last call's answer on a second
+ * event. The trace owns what only it knows: the run's name, failure, metadata
+ * and totals.
+ */
 export declare function emitAiTraceEvent(input: AiTraceEventInput): void;
 export interface AiSpanEventInput {
     runId: string;
@@ -90,6 +104,9 @@ export interface AiSpanEventInput {
     latencySeconds: number;
     isError: boolean;
     error?: AiErrorDetail;
+    /** The tool failure's class — `tool_error`, `interrupted`. A classification,
+     *  never the tool's output, so it ships with `captureToolResults` off. */
+    errorType?: string;
     createdAt: number;
     browserSessionId?: string;
     /** Omitted unless `captureToolArgs` / `captureToolResults` are on. */
@@ -103,8 +120,12 @@ export interface AiFeedbackSurveyInput {
     threadId: string | null;
     userId: string | null;
     feedbackType: "thumbs_up" | "thumbs_down" | "category" | "text";
-    /** The submitted value: sentiment label, chosen category, or free text. */
+    /** The submitted value: chosen category or free text. Ignored for thumbs,
+     *  which answer a rating question and travel as PostHog's choice index. */
     value: string;
+    /** Groups the events of one response. A thumbs-down and the follow-up text
+     *  it opens are two `survey sent` events answering two questions of the SAME
+     *  response, so they must share this id. */
     submissionId: string;
     model?: string;
     browserSessionId?: string;
@@ -117,6 +138,10 @@ export interface AiFeedbackSurveyInput {
  * nowhere. Returns `false` and emits nothing when no survey id is configured:
  * inventing one would produce events attached to a survey that does not exist.
  *
+ * A thumbs vote answers the survey's first question; a category or free-text
+ * answer answers the follow-up question after it. Both share the caller's
+ * `submissionId`, which is how PostHog joins them into one response.
+ *
  * Sent to PostHog ONLY, not through `track()`. The survey response carries the
  * user's free-text feedback verbatim, and configuring a PostHog survey id must
  * not silently start shipping that text to Mixpanel, Amplitude, webhooks, or

package/dist/observability/posthog-ai.js

@@ -24,10 +24,10 @@
  * schema, and a name it does not define today it may define tomorrow with
  * different meaning — `input_truncated` / `spans_dropped` cannot collide.
  *
- * Content (`$ai_input` / `$ai_output_choices` / `$ai_input_state` /
- * `$ai_output_state`) is gated on config and always OMITTED when disabled.
- * Sending `[]` instead would be indistinguishable from a run that genuinely had
- * no messages.
+ * Content (a generation's `$ai_input` / `$ai_output_choices`, a span's
+ * `$ai_input_state` / `$ai_output_state`) is gated on config and always OMITTED
+ * when disabled. Sending `[]` instead would be indistinguishable from a run
+ * that genuinely had no messages.
  *
  * @see https://posthog.com/docs/ai-observability/traces
  * @see https://posthog.com/docs/ai-observability/spans
@@ -38,6 +38,21 @@ import { boundedText } from "../tracking/redaction.js";
 export const MAX_AI_CONTENT_BYTES = 128 * 1024;
 /** Hard ceiling on emitted `$ai_span` events per run. */
 export const MAX_AI_SPANS_PER_RUN = 100;
+/**
+ * `$ai_is_error: true` with no `$ai_error` is a dead end: the reader learns
+ * something broke and nothing else, which is how a failed tool span reached
+ * PostHog saying only "error". Emitters fall back to this rather than shipping
+ * the flag alone — a layer that failed without reporting a message is itself
+ * worth seeing, and is not the same as a message we chose to withhold.
+ */
+const UNREPORTED_ERROR = {
+    message: "failed without a reported error message",
+};
+export function resolveAiError(isError, error) {
+    if (!isError)
+        return undefined;
+    return error ?? UNREPORTED_ERROR;
+}
 function trackAiEvent(name, properties, userId, occurredAt) {
     for (const key of Object.keys(properties)) {
         if (properties[key] === undefined)
@@ -55,12 +70,24 @@ function trackAiEvent(name, properties, userId, occurredAt) {
         // Tracking must never affect the agent run or trace persistence.
     }
 }
+function utf8Bytes(text) {
+    return typeof Buffer !== "undefined"
+        ? Buffer.byteLength(text, "utf8")
+        : new TextEncoder().encode(text).length;
+}
+/** Room reserved for the marker entry that reports what was dropped. */
+const OMISSION_MARKER_BYTES = 512;
 /**
  * Serialize a content value under a byte ceiling.
  *
- * Returns `{ truncated: true }` with a placeholder rather than a silently
- * shortened payload — a trace that shows half a conversation as if it were the
- * whole one is worse than one that says it was cut.
+ * A message list keeps the last user message behind a marker entry naming what
+ * was dropped — what was asked is what a trace is opened for, and it stays
+ * small. Replacing the whole conversation with a placeholder, which is what
+ * this did, left nothing at all.
+ *
+ * Anything else (a string, an object) still becomes a placeholder: there is no
+ * message to keep. Either way `truncated` is true, so a cut payload can never
+ * be read as a complete one.
  */
 export function boundAiContent(value) {
     let serialized;
@@ -72,23 +99,45 @@ export function boundAiContent(value) {
     }
     if (serialized === undefined)
         return { value: undefined, truncated: false };
-    const bytes = typeof Buffer !== "undefined"
-        ? Buffer.byteLength(serialized, "utf8")
-        : new TextEncoder().encode(serialized).length;
+    const bytes = utf8Bytes(serialized);
     if (bytes <= MAX_AI_CONTENT_BYTES)
         return { value, truncated: false };
+    if (Array.isArray(value)) {
+        // What was ASKED is the one thing worth rescuing from an oversized
+        // transcript: the rest is context the thread itself still holds, and
+        // keeping as much of it as fits just ships the ceiling on every event.
+        const lastUser = value.findLast((entry) => !!entry &&
+            typeof entry === "object" &&
+            entry.role === "user");
+        const kept = lastUser !== undefined &&
+            utf8Bytes(JSON.stringify(lastUser) ?? "null") <=
+                MAX_AI_CONTENT_BYTES - OMISSION_MARKER_BYTES
+            ? [lastUser]
+            : [];
+        return {
+            value: [
+                {
+                    role: "system",
+                    content: `[${value.length - kept.length} message(s) omitted: ${bytes} bytes exceeded the ${MAX_AI_CONTENT_BYTES}-byte trace content limit]`,
+                },
+                ...kept,
+            ],
+            truncated: true,
+        };
+    }
     return {
         value: `[truncated: ${bytes} bytes exceeded the ${MAX_AI_CONTENT_BYTES}-byte trace content limit]`,
         truncated: true,
     };
 }
+/**
+ * The run's prompt and answer are NOT carried here. Every round-trip emits a
+ * generation with its own `$ai_input` / `$ai_output_choices`, so a trace-level
+ * copy repeated the first call's prompt and the last call's answer on a second
+ * event. The trace owns what only it knows: the run's name, failure, metadata
+ * and totals.
+ */
 export function emitAiTraceEvent(input) {
-    const inputContent = input.inputState === undefined
-        ? undefined
-        : boundAiContent(input.inputState);
-    const outputContent = input.outputState === undefined
-        ? undefined
-        : boundAiContent(input.outputState);
     trackAiEvent("$ai_trace", {
         ...input.extraProperties,
         $ai_trace_id: input.runId,
@@ -97,15 +146,14 @@ export function emitAiTraceEvent(input) {
         $ai_model: input.model,
         $ai_provider: input.provider,
         $ai_is_error: input.isError,
-        $ai_error: input.error,
+        $ai_error: resolveAiError(input.isError, input.error),
+        $ai_error_type: input.isError
+            ? (input.errorType ?? "run_error")
+            : undefined,
         duration_ms: Math.round(input.durationMs),
         input_tokens: input.inputTokens,
         output_tokens: input.outputTokens,
         cost_usd: input.costUsd,
-        $ai_input_state: inputContent?.value,
-        $ai_output_state: outputContent?.value,
-        input_truncated: inputContent?.truncated || undefined,
-        output_truncated: outputContent?.truncated || undefined,
         $session_id: input.browserSessionId,
         created_at: new Date(input.createdAt).toISOString(),
     }, input.userId, input.createdAt);
@@ -126,7 +174,10 @@ export function emitAiSpanEvent(input) {
         $ai_span_name: input.spanName,
         $ai_latency: input.latencySeconds,
         $ai_is_error: input.isError,
-        $ai_error: input.error,
+        $ai_error: resolveAiError(input.isError, input.error),
+        $ai_error_type: input.isError
+            ? (input.errorType ?? "tool_error")
+            : undefined,
         $ai_input_state: inputContent?.value,
         $ai_output_state: outputContent?.value,
         input_truncated: inputContent?.truncated || undefined,
@@ -135,6 +186,14 @@ export function emitAiSpanEvent(input) {
         created_at: new Date(input.createdAt).toISOString(),
     }, input.userId, input.createdAt);
 }
+/**
+ * PostHog's thumbs survey is a choice question, and the response is the
+ * choice's 1-based index — not a label. Sending "thumbs_up" put a string where
+ * the rating belongs, so the vote counted as an unrecognized answer.
+ *
+ * @see https://posthog.com/docs/ai-observability/user-feedback/manual-event-capture
+ */
+const THUMB_RESPONSE_INDEX = { thumbs_up: 1, thumbs_down: 2 };
 /**
  * Emit PostHog's documented manual feedback event for an LLM trace.
  *
@@ -143,6 +202,10 @@ export function emitAiSpanEvent(input) {
  * nowhere. Returns `false` and emits nothing when no survey id is configured:
  * inventing one would produce events attached to a survey that does not exist.
  *
+ * A thumbs vote answers the survey's first question; a category or free-text
+ * answer answers the follow-up question after it. Both share the caller's
+ * `submissionId`, which is how PostHog joins them into one response.
+ *
  * Sent to PostHog ONLY, not through `track()`. The survey response carries the
  * user's free-text feedback verbatim, and configuring a PostHog survey id must
  * not silently start shipping that text to Mixpanel, Amplitude, webhooks, or
@@ -155,17 +218,23 @@ export function emitAiFeedbackSurveyEvent(input) {
     const surveyId = process.env.POSTHOG_AI_FEEDBACK_SURVEY_ID?.trim();
     if (!surveyId)
         return false;
-    const questionId = process.env.POSTHOG_AI_FEEDBACK_SURVEY_QUESTION_ID?.trim();
-    // PostHog accepts `$survey_response` for a single-question survey and
-    // `$survey_response_<questionId>` when the survey has named questions.
-    const responseKey = questionId
-        ? `$survey_response_${questionId}`
-        : "$survey_response";
+    const thumbResponse = input.feedbackType === "thumbs_up" || input.feedbackType === "thumbs_down"
+        ? THUMB_RESPONSE_INDEX[input.feedbackType]
+        : undefined;
+    // `$survey_response` is the survey's first question — the thumbs — and
+    // `$survey_response_1` the follow-up asked after a thumbs-down. Putting the
+    // free text on the first key instead would file prose where the rating
+    // belongs and skew every thumbs metric in the survey.
+    const responses = thumbResponse !== undefined
+        ? { $survey_response: thumbResponse }
+        : { $survey_response_1: input.value };
     const properties = {
         $survey_id: surveyId,
-        [responseKey]: input.value,
+        ...responses,
         $survey_submission_id: input.submissionId,
-        $survey_completed: true,
+        // A thumbs-down opens the follow-up question, so the response is not
+        // complete until that answer arrives (or the user closes it without one).
+        $survey_completed: input.feedbackType !== "thumbs_down",
         $ai_trace_id: input.runId ?? undefined,
         $ai_session_id: input.threadId ?? undefined,
         $ai_model: input.model,

package/dist/observability/routes.js

@@ -216,8 +216,15 @@ export function createObservabilityHandler() {
                     threadId,
                     userId: owner,
                     feedbackType,
-                    value: isThumb ? feedbackType : value,
-                    submissionId: id,
+                    value,
+                    // One PostHog response per rated message, not per row: a thumbs-down
+                    // and the free text it opens are two answers to the same survey, and
+                    // a fresh id per row would file them as two unrelated responses.
+                    // Falls back to the row id when there is no message to key on, which
+                    // groups nothing — the honest outcome when nothing can be grouped.
+                    submissionId: runId && typeof body.messageSeq === "number"
+                        ? `${runId}:${body.messageSeq}`
+                        : id,
                     model,
                     browserSessionId: getRequestContext()?.browserSessionId,
                 });