@agent-native/core 0.164.11 → 0.164.12

package/corpus/templates/factory/app/components/factory/FactoryInspector.tsx

@@ -1,6 +1,12 @@
 import { useActionQuery } from "@agent-native/core/client/hooks";
 import { useT } from "@agent-native/core/client/i18n";
-import { IconArrowRight, IconPlus, IconTrash } from "@tabler/icons-react";
+import {
+  IconAlertCircle,
+  IconArrowRight,
+  IconLoader2,
+  IconPlus,
+  IconTrash,
+} from "@tabler/icons-react";
 import { useMemo, useState } from "react";
 import { useSearchParams } from "react-router";
 
@@ -22,8 +28,13 @@ interface FactoryInspectorProps {
   factoryId?: string;
   dirty: boolean;
   saving: boolean;
+  saveError: string | null;
+  saveConflictNeedsResolution: boolean;
+  refreshing: boolean;
   onGraphChange: (graph: FactoryCanvasGraph) => void;
   onSave: () => void;
+  onRefresh: () => void;
+  onDiscardLocalChanges: () => void;
   onAddNode: () => void;
   onDeleteNode: (nodeId: string) => void;
   onConnect: (sourceId: string, targetId: string) => void;
@@ -54,8 +65,13 @@ export function FactoryInspector({
   factoryId,
   dirty,
   saving,
+  saveError,
+  saveConflictNeedsResolution,
+  refreshing,
   onGraphChange,
   onSave,
+  onRefresh,
+  onDiscardLocalChanges,
   onAddNode,
   onDeleteNode,
   onConnect,
@@ -383,12 +399,56 @@ export function FactoryInspector({
         </section>
       </div>
 
+      {saveError ? (
+        <div
+          className="flex items-start gap-2 border-t border-border/60 bg-destructive/5 p-4 text-sm text-destructive"
+          role="alert"
+        >
+          <IconAlertCircle className="mt-0.5 size-4 shrink-0" />
+          <div className="min-w-0 flex-1 space-y-2">
+            <p>{saveError}</p>
+            {saveConflictNeedsResolution ? (
+              <p className="text-xs leading-5 text-destructive/80">
+                {t("factoryInspector.saveConflictHint")}
+              </p>
+            ) : null}
+            <div className="flex flex-wrap gap-2">
+              {!saveConflictNeedsResolution ? (
+                <Button
+                  type="button"
+                  variant="link"
+                  size="sm"
+                  className="h-auto p-0 text-destructive"
+                  disabled={refreshing}
+                  onClick={onRefresh}
+                >
+                  {refreshing ? (
+                    <IconLoader2 className="size-3.5 animate-spin" />
+                  ) : null}
+                  {t("triage.refresh")}
+                </Button>
+              ) : (
+                <Button
+                  type="button"
+                  variant="link"
+                  size="sm"
+                  className="h-auto p-0 text-destructive"
+                  onClick={onDiscardLocalChanges}
+                >
+                  {t("factoryInspector.discardLocalChanges")}
+                </Button>
+              )}
+            </div>
+          </div>
+        </div>
+      ) : null}
+
       {(dirty || saving) && (
         <div className="bg-muted/15 p-4">
           <Button
             type="button"
             className="w-full"
-            disabled={saving}
+            disabled={saving || refreshing}
             onClick={onSave}
           >
             {saving

package/corpus/templates/factory/app/components/ui/alert-dialog.tsx

@@ -0,0 +1 @@
+export * from "@agent-native/toolkit/ui/alert-dialog";

package/corpus/templates/factory/app/i18n/en-US.ts

@@ -204,6 +204,46 @@ const messages = {
     auditUxImpact: "UX impact",
     auditOwnerArea: "Owner area",
     auditGuardsLabel: "Guards",
+    historyTab: "History",
+    historyTitle: "Saved versions",
+    historyLoading: "Loading saved versions",
+    historyLoadError: "Could not load Factory history.",
+    historyEmpty: "No saved versions yet",
+    historyEmptyHint:
+      "Save the visual graph once and Factory will keep each version here for review and rollback.",
+    historyVersion: "Version {{version}}",
+    historyCurrent: "Current",
+    historySource: "Source",
+    historySourceManual: "Visual editor",
+    historySourceAi: "Agent proposal",
+    historySourceSeed: "Factory seed",
+    historySourceRestore: "Rollback",
+    historySourceSaved: "Saved graph",
+    historyNodes: "Nodes",
+    historyConnections: "Connections",
+    historyChangeSummary: "Change summary",
+    historyNoSummary: "No change summary was recorded.",
+    historyCurrentHint: "This is the version currently used by the Factory.",
+    historyRestore: "Restore this version",
+    historyRestoreTitle: "Restore version {{version}}?",
+    historyRestoreDescription:
+      "Factory will create a new current version from this snapshot. Existing versions stay available in history.",
+    historyRestoreUnsaved:
+      "You also have unsaved visual changes. Restoring will discard them.",
+    historyCancel: "Cancel",
+    historyRestoreConfirm: "Restore version",
+    historyRestored: "Version restored. A new current version was created.",
+    historyRestoreFailed: "Could not restore this Factory version.",
+    historyRestoreConflict:
+      "Factory changed while restoring. Review the refreshed history before trying again.",
+    historyRefreshFailed:
+      "The version was restored, but the Factory view could not refresh.",
+    historyLoadOlder: "Load older versions",
+    historySnapshotLoadError: "Could not load this version preview.",
+    historyRestoreUnverified:
+      "Restore completed, but the result could not be verified. Refresh history before trying again.",
+    saveConflictFallback:
+      "Factory changed while saving. Refresh Factory and try again.",
     automationsTitle: "Automations",
     automationsDescription: "Schedule and run Factory prompts.",
     automationsLoading: "Loading automations...",
@@ -312,6 +352,9 @@ const messages = {
     addComment: "Add comment",
     savingGraph: "Saving graph...",
     saveGraph: "Save factory changes",
+    saveConflictHint:
+      "Your unsaved edits are preserved. Discard them to load the newer Factory version.",
+    discardLocalChanges: "Discard local edits and load current version",
     savedGraph: "Factory is saved",
     saveNote: "Saves a new version. Existing runs keep their original context.",
     factoryName: "Factory name",

package/corpus/templates/factory/app/routes/factory.tsx

@@ -31,6 +31,7 @@ import {
   type FactoryCanvasGraph,
   type FactoryCanvasNode,
 } from "@/components/factory/FactoryCanvas";
+import { FactoryHistoryView } from "@/components/factory/FactoryHistoryView";
 import { FactoryInspector } from "@/components/factory/FactoryInspector";
 import { TriageStatusPill } from "@/components/triage/triage-status-pill";
 import { Button } from "@/components/ui/button";
@@ -131,6 +132,7 @@ type WorkspaceTab =
   | "automations"
   | "agents"
   | "audit"
+  | "history"
   | "settings";
 
 type FactoryAutomationRun = {
@@ -184,9 +186,14 @@ export default function FactoryRoute() {
   const [creating, setCreating] = useState(false);
   const [draftGraph, setDraftGraph] = useState<FactoryCanvasGraph | null>(null);
   const [dirty, setDirty] = useState(false);
+  const [saveError, setSaveError] = useState<string | null>(null);
+  const [saveConflictRemoteGraph, setSaveConflictRemoteGraph] =
+    useState<FactoryCanvasGraph | null>(null);
+  const [refreshingFactory, setRefreshingFactory] = useState(false);
   const [selectedNodeId, setSelectedNodeId] = useState<string | null>(null);
   const [selectedEdgeId, setSelectedEdgeId] = useState<string | null>(null);
   const [auditRefreshToken, setAuditRefreshToken] = useState(0);
+  const draftRevisionRef = useRef(0);
 
   function setActiveTab(tab: WorkspaceTab) {
     setSearchParams(
@@ -194,6 +201,10 @@ export default function FactoryRoute() {
         const next = new URLSearchParams(current);
         if (tab === "overview") next.delete("tab");
         else next.set("tab", tab);
+        if (tab === "history") {
+          next.delete("node");
+          next.delete("edge");
+        }
         return next;
       },
       { replace: true },
@@ -255,17 +266,17 @@ export default function FactoryRoute() {
   const graphData =
     rawGraphData?.factory.id === factoryId ? rawGraphData : undefined;
   const graph = draftGraph ?? graphData?.graph ?? null;
-  const graphVersion = graphData?.factory.graphVersion ?? graph?.version ?? 1;
+  const graphVersion = graph?.version ?? graphData?.factory.graphVersion ?? 1;
   const saveGraphMutation = useActionMutation("save-factory-graph");
   const factoryList = (factoryListQuery.data ?? []) as FactorySummary[];
 
   useEffect(() => {
-    if (!graphData || creating) return;
+    if (!graphData || creating || dirty) return;
     setDraftGraph(graphData.graph);
     setDirty(false);
     setSelectedNodeId(null);
     setSelectedEdgeId(null);
-  }, [creating, graphData]);
+  }, [creating, dirty, graphData]);
 
   useEffect(() => {
     setSelectedNodeId(searchParams.get("node"));
@@ -313,6 +324,9 @@ export default function FactoryRoute() {
   }
 
   function updateGraph(next: FactoryCanvasGraph) {
+    draftRevisionRef.current += 1;
+    setSaveError(null);
+    setSaveConflictRemoteGraph(null);
     setDraftGraph(next);
     setDirty(
       !graphData?.graph ||
@@ -396,20 +410,87 @@ export default function FactoryRoute() {
 
   async function saveGraph() {
     if (!graph || !selectedFactoryId) return;
-    await saveGraphMutation.mutateAsync({
-      factoryId: selectedFactoryId,
-      name: graph.name,
-      description: graph.description,
-      prompt: creating ? "" : (graphData?.factory.prompt ?? ""),
-      source: "manual",
-      changeSummary: creating
-        ? "Created from the Factory visual editor."
-        : "Updated in the Factory visual editor.",
-      graph,
-    });
+    const submittedDraftRevision = draftRevisionRef.current;
+    setSaveError(null);
+    try {
+      await saveGraphMutation.mutateAsync({
+        factoryId: selectedFactoryId,
+        name: graph.name,
+        description: graph.description,
+        prompt: creating ? "" : (graphData?.factory.prompt ?? ""),
+        source: "manual",
+        changeSummary: creating
+          ? "Created from the Factory visual editor."
+          : "Updated in the Factory visual editor.",
+        expectedGraphVersion: creating ? 0 : graph.version,
+        graph,
+      });
+      setCreating(false);
+      setDirty(draftRevisionRef.current !== submittedDraftRevision);
+      setSaveConflictRemoteGraph(null);
+      await Promise.all([graphQuery.refetch(), factoryListQuery.refetch()]);
+    } catch (error) {
+      setSaveError(
+        error instanceof Error
+          ? error.message
+          : t("factoryRoute.saveConflictFallback"),
+      );
+    }
+  }
+
+  async function refreshFactoryAfterSaveConflict() {
+    setRefreshingFactory(true);
+    try {
+      const results = await Promise.all([
+        graphQuery.refetch(),
+        factoryListQuery.refetch(),
+      ]);
+      if (results.some((result) => result.isError)) {
+        throw new Error("Factory refresh failed.");
+      }
+      const remoteGraph = (results[0].data as FactoryGraphResponse | undefined)
+        ?.graph;
+      if (dirty) {
+        if (!remoteGraph) throw new Error("Factory graph refresh failed.");
+        setSaveConflictRemoteGraph(remoteGraph);
+        setSaveError(t("factoryRoute.saveConflictFallback"));
+        return;
+      }
+      setSaveError(null);
+    } catch {
+      setSaveError(t("factoryRoute.saveConflictFallback"));
+    } finally {
+      setRefreshingFactory(false);
+    }
+  }
+
+  function discardLocalFactoryChanges() {
+    if (!saveConflictRemoteGraph) return;
+    setDraftGraph(saveConflictRemoteGraph);
+    setSaveConflictRemoteGraph(null);
+    setSaveError(null);
+    setDirty(false);
+    setCreating(false);
+    setSelectedNodeId(null);
+    setSelectedEdgeId(null);
+  }
+
+  async function handleFactoryRestored(result?: { graph: FactoryCanvasGraph }) {
     setCreating(false);
+    if (result?.graph) setDraftGraph(result.graph);
+    else setDraftGraph(null);
     setDirty(false);
-    await Promise.all([graphQuery.refetch(), factoryListQuery.refetch()]);
+    setSaveError(null);
+    setSaveConflictRemoteGraph(null);
+    setSelectedNodeId(null);
+    setSelectedEdgeId(null);
+    const results = await Promise.all([
+      graphQuery.refetch(),
+      factoryListQuery.refetch(),
+    ]);
+    if (results.some((result) => result.isError)) {
+      throw new Error("Factory view refresh failed.");
+    }
   }
 
   if (!selectedFactoryId && activeTab === "agents") {
@@ -659,6 +740,12 @@ export default function FactoryRoute() {
             >
               Activity
             </TabButton>
+            <TabButton
+              active={activeTab === "history"}
+              onClick={() => setActiveTab("history")}
+            >
+              {t("factoryRoute.historyTab")}
+            </TabButton>
             <TabButton
               active={activeTab === "settings"}
               onClick={() => setActiveTab("settings")}
@@ -727,8 +814,13 @@ export default function FactoryRoute() {
               factoryId={factoryId}
               dirty={dirty}
               saving={saveGraphMutation.isPending}
+              saveError={saveError}
+              saveConflictNeedsResolution={Boolean(saveConflictRemoteGraph)}
+              refreshing={refreshingFactory}
               onGraphChange={updateGraph}
               onSave={() => void saveGraph()}
+              onRefresh={refreshFactoryAfterSaveConflict}
+              onDiscardLocalChanges={discardLocalFactoryChanges}
               onAddNode={addNode}
               onDeleteNode={deleteNode}
               onConnect={connectNodes}
@@ -747,6 +839,13 @@ export default function FactoryRoute() {
             factoryId={factoryId}
             refreshToken={auditRefreshToken}
           />
+        ) : activeTab === "history" ? (
+          <FactoryHistoryView
+            key={factoryId}
+            factoryId={factoryId}
+            hasUnsavedChanges={dirty}
+            onRestored={handleFactoryRestored}
+          />
         ) : (
           <SettingsView t={t} />
         )}
@@ -783,6 +882,7 @@ function parseWorkspaceTab(value: string | null): WorkspaceTab {
     value === "automations" ||
     value === "agents" ||
     value === "audit" ||
+    value === "history" ||
     value === "settings"
     ? value
     : "overview";

package/corpus/templates/factory/server/plugins/agent-chat.ts

@@ -57,7 +57,9 @@ current factory graph, and executes only the explicit automation prompts that
 are stored in the organization.
 Use the Factory actions as the source of truth. When a user asks to
 create or change a factory, first inspect the current graph, then propose a complete
-versioned graph through save-factory-graph with source=ai and a concise changeSummary.
+versioned graph through save-factory-graph with source=ai, the inspected graphVersion
+as expectedGraphVersion, and a concise changeSummary. Never save a graph from a
+stale read: refresh and re-propose when the action reports a version conflict.
 Never hide a graph change in prose: the visual map and the saved graph must agree.
 The graph is currently a reviewable blueprint, not the runtime router: automation
 markdown resources are the runtime prompts, while enabled triage rules are evaluated

package/dist/collab/awareness.d.ts

@@ -62,11 +62,11 @@ export declare const postAwareness: import("h3").EventHandlerWithFetch<import("h
     error: string;
     states?: undefined;
 } | {
+    error?: undefined;
     states: {
         clientId: number;
         state: string;
     }[];
-    error?: undefined;
 }>>;
 /**
  * GET /_agent-native/collab/:docId/users
@@ -77,9 +77,9 @@ export declare const getActiveUsers: import("h3").EventHandlerWithFetch<import("
     error: string;
     users?: undefined;
 } | {
+    error?: undefined;
     users: {
         clientId: number;
         lastSeen: number;
     }[];
-    error?: undefined;
 }>>;

package/dist/collab/struct-routes.d.ts

@@ -13,8 +13,8 @@
  * Body: { json: any, fieldName?: string, type?: "map"|"array", requestSource?: string }
  */
 export declare const postCollabJson: import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
-    ok?: undefined;
     error: string;
+    ok?: undefined;
 } | {
     error?: undefined;
     ok: boolean;

package/dist/notifications/routes.d.ts

@@ -11,23 +11,23 @@
  *   DELETE /_agent-native/notifications/:id         — delete
  */
 export declare function createNotificationsHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<"" | import("./types.js").Notification[] | {
+    error?: undefined;
     count: number;
     updated?: undefined;
     ok?: undefined;
-    error?: undefined;
 } | {
+    error?: undefined;
     count?: undefined;
     updated: number;
     ok?: undefined;
-    error?: undefined;
 } | {
     count?: undefined;
     updated?: undefined;
     error: string;
     ok?: undefined;
 } | {
+    error?: undefined;
     count?: undefined;
     updated?: undefined;
     ok: boolean;
-    error?: undefined;
 }>>;

package/dist/observability/routes.d.ts

@@ -42,22 +42,22 @@ export declare function createObservabilityHandler(): import("h3").EventHandlerW
     avgEvalScore: number;
 } | {
     error?: undefined;
+    ok?: undefined;
     summary: import("./types.js").TraceSummary;
     spans: import("./types.js").TraceSpan[];
     id?: undefined;
-    ok?: undefined;
 } | {
     error?: undefined;
+    ok?: undefined;
     summary?: undefined;
     spans?: undefined;
     id: string;
-    ok?: undefined;
 } | {
+    ok?: undefined;
     summary?: undefined;
     spans?: undefined;
     id?: undefined;
     error: any;
-    ok?: undefined;
 } | {
     error?: undefined;
     summary?: undefined;

package/dist/progress/routes.d.ts

@@ -15,6 +15,6 @@ export declare function createProgressHandler(): import("h3").EventHandlerWithFe
     error: string;
     ok?: undefined;
 } | {
-    ok: boolean;
     error?: undefined;
+    ok: boolean;
 }>>;

package/dist/secrets/routes.d.ts

@@ -34,37 +34,37 @@ export declare function createListSecretsHandler(): import("h3").EventHandlerWit
 /** POST /_agent-native/secrets/:key — write a secret. */
 export declare function createWriteSecretHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
     error: string;
-    ok?: undefined;
     status?: undefined;
+    ok?: undefined;
 } | {
-    error?: undefined;
     ok: boolean;
     status: string;
+    error?: undefined;
 } | {
-    ok?: undefined;
     error: string;
     removed?: undefined;
+    ok?: undefined;
 } | {
-    error?: undefined;
     ok: boolean;
     removed: boolean;
+    error?: undefined;
 }>>;
 /**
  * POST /_agent-native/secrets/:key/test — validate an optional candidate value
  * or the current stored value without changing anything.
  */
 export declare function createTestSecretHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
-    ok?: undefined;
     error: string;
     note?: undefined;
+    ok?: undefined;
 } | {
-    error?: undefined;
     ok: boolean;
     note?: undefined;
-} | {
     error?: undefined;
+} | {
     ok: boolean;
     note: string;
+    error?: undefined;
 } | {
     note?: undefined;
     ok: boolean;
@@ -95,11 +95,11 @@ export interface AdHocSecretPayload {
 export declare function createAdHocSecretHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<AdHocSecretPayload[] | {
     error: string;
 } | {
-    error?: undefined;
     ok: boolean;
     key: string;
-} | {
     error?: undefined;
+} | {
     ok: boolean;
     removed: boolean;
+    error?: undefined;
 }>>;

package/dist/server/agent-engine-api-key-route.d.ts

@@ -27,10 +27,10 @@ export declare function resolveAgentEngineApiKeyWriteTarget(event: H3Event, scop
 export declare function createAgentEngineApiKeyHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
     error: any;
 } | {
-    error?: undefined;
     ok: boolean;
     key: string;
     baseUrlKey?: string;
     scope: AgentEngineApiKeyScope;
+    error?: undefined;
 }>>;
 export {};

package/dist/server/realtime-token.d.ts

@@ -26,8 +26,8 @@ export declare function createRealtimeTokenHandler(): import("h3").EventHandlerW
     expiresAt?: undefined;
     ttlSeconds?: undefined;
 } | {
+    error?: undefined;
     token: string;
     expiresAt: string;
     ttlSeconds: number;
-    error?: undefined;
 }>>;