@agent-native/core 0.192.0 → 0.192.1-nightly-20260926144920

package/corpus/templates/design/app/components/design/MultiScreenCanvas.tsx

@@ -882,11 +882,6 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
     setPenPointer(null);
     setPenCloseHover(false);
   }, []);
-  const cancelActivePenPath = useCallback(() => {
-    clearActivePenPath();
-    penContinuesVectorEditRef.current = false;
-    penContinuationBaseCountRef.current = 0;
-  }, [clearActivePenPath]);
   // Last raw client point the pen ghost/close-hover preview was computed
   // from (P18). A wheel pan/zoom gesture mutates pan/zoom every animation
   // frame via applyViewToDom without the mouse itself moving, so the
@@ -5636,19 +5631,16 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
   useEffect(() => {
     const handleKeyDown = (event: KeyboardEvent) => {
       if (event.key !== "Escape") return;
-      const hasActivePenPath =
-        Boolean(activePenPathRef.current) &&
-        !isEditableHotkeyTarget(event.target);
       const cancelled = cancelActiveDrag(
         eventEpochMilliseconds(event.timeStamp),
       );
-      if (hasActivePenPath) cancelActivePenPath();
-      if (cancelled || hasActivePenPath) {
+      if (cancelled) {
         event.preventDefault();
         event.stopPropagation();
         event.stopImmediatePropagation();
         return;
       }
+      if (activePenPathRef.current) return;
       // No in-flight drag to cancel: Escape while in vector edit mode exits
       // the mode entirely (matches Figma), rather than being a no-op.
       if (vectorEdit) {
@@ -5663,7 +5655,7 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
     return () => {
       window.removeEventListener("keydown", handleKeyDown, true);
     };
-  }, [cancelActiveDrag, cancelActivePenPath, vectorEdit]);
+  }, [cancelActiveDrag, vectorEdit]);
 
   const beginPan = useCallback(
     (e: React.MouseEvent) => {
@@ -6359,7 +6351,9 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
     const lastPersisted = persistedEntries[persistedEntries.length - 1];
     // Do not call onPrimitiveCreated for board objects (sentinel frameId).
     if (lastPersisted && lastPersisted.frameId !== "__board__") {
-      onPrimitiveCreated?.(lastPersisted.frameId, lastPersisted.nodeId);
+      onPrimitiveCreated?.(lastPersisted.frameId, lastPersisted.nodeId, {
+        preserveActiveTool: true,
+      });
     }
   }, [
     onCreatePrimitive,
@@ -6382,7 +6376,7 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
         nextTool?: "move" | "pen";
       },
     ) => {
-      const nextTool = options?.nextTool ?? "pen";
+      let nextTool: "move" | "pen" = options?.nextTool ?? "pen";
       // Clear before committing: the commit flushes React synchronously, and
       // an effect it wakes can re-enter here and commit the same path twice.
       clearActivePenPath();
@@ -6392,31 +6386,47 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
       }
 
       if (penContinuesVectorEditRef.current) {
-        penContinuesVectorEditRef.current = false;
         const active = vectorEditRef.current;
         const baseCount = penContinuationBaseCountRef.current;
-        penContinuationBaseCountRef.current = 0;
-        if (
-          active &&
-          (path.nodes.length > baseCount || path.closed !== active.path.closed)
-        ) {
-          const toLocal = (point: Point): Point => ({
-            x: point.x - active.originCanvas.x,
-            y: point.y - active.originCanvas.y,
-          });
-          active.onChange(
-            {
-              closed: path.closed,
-              nodes: path.nodes.map((node) => ({
-                ...node,
-                point: toLocal(node.point),
-                handleIn: node.handleIn ? toLocal(node.handleIn) : undefined,
-                handleOut: node.handleOut ? toLocal(node.handleOut) : undefined,
-              })),
-            },
-            "commit",
+        const changed =
+          path.nodes.length > baseCount || path.closed !== active?.path.closed;
+        const accepted =
+          !changed ||
+          Boolean(
+            active &&
+            active.onChange(
+              {
+                closed: path.closed,
+                nodes: path.nodes.map((node) => ({
+                  ...node,
+                  point: {
+                    x: node.point.x - active.originCanvas.x,
+                    y: node.point.y - active.originCanvas.y,
+                  },
+                  handleIn: node.handleIn
+                    ? {
+                        x: node.handleIn.x - active.originCanvas.x,
+                        y: node.handleIn.y - active.originCanvas.y,
+                      }
+                    : undefined,
+                  handleOut: node.handleOut
+                    ? {
+                        x: node.handleOut.x - active.originCanvas.x,
+                        y: node.handleOut.y - active.originCanvas.y,
+                      }
+                    : undefined,
+                })),
+              },
+              "commit",
+            ),
           );
+        if (!accepted) {
+          activePenPathRef.current = path;
+          setActivePenPath(path);
+          return;
         }
+        penContinuesVectorEditRef.current = false;
+        penContinuationBaseCountRef.current = 0;
         if (nextTool === "move") active?.onExit();
         onActiveToolChange?.(nextTool);
         return;
@@ -6432,7 +6442,6 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
           updateNodeId: continuation.nodeId,
         });
         if (!persisted) {
-          continuationPenPathRef.current = null;
           activePenPathRef.current = path;
           setActivePenPath(path);
           return;
@@ -6445,6 +6454,7 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
         const persisted = commitDraftPrimitive(draft, undefined, {
           nextTool,
         });
+        if (!persisted) nextTool = "pen";
         continuationPenPathRef.current =
           persisted && !path.closed && options?.continueAfterCommit
             ? {
@@ -6455,9 +6465,7 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
             : null;
       }
       // The parent selection callback receives nextTool, but board primitives
-      // intentionally bypass that generic callback and
-      // asynchronous selection reconciliation can otherwise paint Move for a
-      // frame. Drive the controlled tool explicitly at the commit boundary.
+      // intentionally bypass it, so update the controlled tool here too.
       onActiveToolChange?.(nextTool);
     },
     [
@@ -10513,13 +10521,27 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
         event.preventDefault();
         event.stopPropagation();
         event.stopImmediatePropagation();
-        finishPenPath(path, {
-          continueAfterCommit: true,
-          nextTool: "move",
+        if (dragState.current?.type === "pen-node") cancelActiveDrag();
+        const pathToFinish = activePenPathRef.current;
+        if (!pathToFinish) return;
+        const continuesExistingPath =
+          continuationPenPathRef.current !== null ||
+          penContinuesVectorEditRef.current;
+        finishPenPath(pathToFinish, {
+          continueAfterCommit: continuationPenPathRef.current !== null,
+          nextTool: continuesExistingPath ? "pen" : "move",
         });
         return;
       }
 
+      if (event.key === "Escape") {
+        event.preventDefault();
+        event.stopPropagation();
+        event.stopImmediatePropagation();
+        finishPenPath(path);
+        return;
+      }
+
       if (event.key === "Delete" || event.key === "Backspace") {
         event.preventDefault();
         event.stopPropagation();
@@ -10532,7 +10554,7 @@ export const MultiScreenCanvas = memo(function MultiScreenCanvas({
     return () => {
       window.removeEventListener("keydown", handleKeyDown, true);
     };
-  }, [finishPenPath, undoActivePenPathSegment]);
+  }, [cancelActiveDrag, finishPenPath, undoActivePenPathSegment]);
 
   useEffect(() => {
     const tool = normalizeCanvasTool(activeTool ?? localActiveTool);

package/corpus/templates/design/app/components/design/design-canvas/creation.ts

@@ -18,6 +18,7 @@ export interface CreatePrimitiveSpec {
   fromClick: boolean;
   /** Commit an in-progress pen path without overriding a newly chosen tool. */
   preserveActiveTool?: boolean;
+  nextTool?: "move" | "pen";
 }
 
 /**

package/corpus/templates/design/app/components/design/multi-screen/types.ts

@@ -293,7 +293,10 @@ export interface MultiScreenCanvasProps {
   onPrimitiveCreated?: (
     screenId: string,
     nodeId: string,
-    options?: { nextTool?: "move" | "pen" },
+    options?: {
+      nextTool?: "move" | "pen";
+      preserveActiveTool?: boolean;
+    },
   ) => void;
   onUpdatePenPath?: (
     screenId: string,
@@ -855,7 +858,7 @@ export interface VectorEditOverlayState {
   originCanvas: Point;
   selectedAnchorIndex: number | null;
   onSelectedAnchorChange: (nodeIndex: number | null) => void;
-  onChange: (nextPath: PenPath, phase: "preview" | "commit") => void;
+  onChange: (nextPath: PenPath, phase: "preview" | "commit") => boolean;
   onExit: () => void;
 }
 

package/corpus/templates/design/app/pages/design-editor/commands/duplicate-screen.ts

@@ -58,50 +58,76 @@ export function getDuplicateScreenGeometry(
   sourceGeometry: FrameGeometry,
   occupiedGeometries: readonly FrameGeometry[],
 ): FrameGeometry {
-  const rowBottom = sourceGeometry.y + sourceGeometry.height;
-  const rowGeometries = occupiedGeometries.filter(
-    (geometry) =>
-      geometry.y < rowBottom && geometry.y + geometry.height > sourceGeometry.y,
-  );
-  const x = Math.max(
-    sourceGeometry.x + sourceGeometry.width + DUPLICATE_SCREEN_GAP,
-    ...rowGeometries.map(
-      (geometry) => geometry.x + geometry.width + DUPLICATE_SCREEN_GAP,
+  return {
+    ...getFirstFreeDuplicateGeometry(
+      {
+        ...sourceGeometry,
+        x: sourceGeometry.x + sourceGeometry.width + DUPLICATE_SCREEN_GAP,
+        y: sourceGeometry.y,
+      },
+      occupiedGeometries,
     ),
-  );
-  const z = Math.max(
-    sourceGeometry.z ?? 0,
-    ...occupiedGeometries.map((geometry) => geometry.z ?? 0),
-  );
-  return { ...sourceGeometry, x, y: sourceGeometry.y, z: z + 1 };
+    z: (sourceGeometry.z ?? 0) + 1,
+  };
+}
+
+function getFirstFreeDuplicateGeometry(
+  candidate: FrameGeometry,
+  occupiedGeometries: readonly FrameGeometry[],
+): FrameGeometry {
+  let free = { ...candidate };
+  while (true) {
+    const overlap = occupiedGeometries
+      .filter((geometry) => duplicateGeometriesOverlap(free, geometry))
+      .sort((left, right) => left.x - right.x)[0];
+    if (!overlap) return free;
+    free = {
+      ...free,
+      x: overlap.x + overlap.width + DUPLICATE_SCREEN_GAP,
+    };
+  }
 }
 
 function duplicateGeometriesOverlap(
   left: FrameGeometry,
   right: FrameGeometry,
+  gap = DUPLICATE_SCREEN_GAP,
 ): boolean {
   const sameRow =
     left.y < right.y + right.height && right.y < left.y + left.height;
   return (
     sameRow &&
-    left.x < right.x + right.width + DUPLICATE_SCREEN_GAP &&
-    right.x < left.x + left.width + DUPLICATE_SCREEN_GAP
+    left.x < right.x + right.width + gap &&
+    right.x < left.x + left.width + gap
   );
 }
 
 function reserveDuplicateGeometry(
   filename: string,
   candidate: FrameGeometry,
+  occupiedGeometries: readonly FrameGeometry[],
   pendingGeometries: ReadonlyMap<string, FrameGeometry>,
+  preserveRequestedPosition: boolean,
 ): FrameGeometry {
-  let reserved = { ...candidate };
-  for (const [pendingFilename, pendingGeometry] of pendingGeometries) {
-    if (
-      pendingFilename !== filename &&
-      duplicateGeometriesOverlap(reserved, pendingGeometry)
-    ) {
-      reserved.x =
-        pendingGeometry.x + pendingGeometry.width + DUPLICATE_SCREEN_GAP;
+  const otherPending = [...pendingGeometries.entries()]
+    .filter(([pendingFilename]) => pendingFilename !== filename)
+    .map(([, geometry]) => geometry);
+  const reserved =
+    preserveRequestedPosition &&
+    !otherPending.some((geometry) =>
+      duplicateGeometriesOverlap(candidate, geometry),
+    )
+      ? { ...candidate }
+      : getFirstFreeDuplicateGeometry(candidate, [
+          ...occupiedGeometries,
+          ...otherPending,
+        ]);
+  if (reserved.x === candidate.x && reserved.y === candidate.y) {
+    const overlappingZ = occupiedGeometries
+      .filter((geometry) => duplicateGeometriesOverlap(candidate, geometry, 0))
+      .map((geometry) => geometry.z ?? 0);
+    if (overlappingZ.length > 0) {
+      reserved.z = Math.max(reserved.z ?? 0, ...overlappingZ) + 1;
     }
   }
   const pendingZ = [...pendingGeometries.values()].map(
@@ -185,6 +211,7 @@ export function runDuplicateScreen(
   }: DuplicateScreenArgs,
   screenId: string,
   request?: {
+    mode?: "alt-click" | "alt-drag";
     canvasPosition?: { x: number; y: number };
     preserveCamera?: boolean;
     historyBatchId?: string;
@@ -269,12 +296,16 @@ export function runDuplicateScreen(
           z: (adjacentGeometry.z ?? 0) + (request.duplicateStackIndex ?? 0),
         }
       : adjacentGeometry);
+  const preserveRequestedPosition =
+    request?.canvasPosition !== undefined && request.mode !== "alt-click";
   const createdGeometry = recoveryState?.geometry
     ? recoveryState.geometry
     : reserveDuplicateGeometry(
         filename,
         requestedGeometry,
+        occupiedGeometries,
         pendingDuplicateGeometriesRef.current,
+        preserveRequestedPosition,
       );
   pendingDuplicateGeometriesRef.current.set(filename, createdGeometry);
   // Carry screen dimensions/height mode for every duplicate so the new frame

package/corpus/templates/design/app/routes/design-systems.tsx

@@ -5,10 +5,8 @@ import type { LoaderFunctionArgs, MetaFunction } from "react-router";
 import enUSMessages from "@/i18n/en-US";
 
 import { getDb, schema } from "../../server/db";
-import {
-  designResourceMeta,
-  publicDesignMetaLoaderData,
-} from "./public-design-meta";
+import { publicDesignMetaLoaderData } from "../../server/lib/public-design-meta.server";
+import { designResourceMeta } from "./public-design-meta";
 
 export { default } from "../pages/DesignSystems";
 

package/corpus/templates/design/app/routes/design.$id.tsx

@@ -2,8 +2,9 @@ import type { LoaderFunctionArgs, MetaFunction } from "react-router";
 
 import enUSMessages from "@/i18n/en-US";
 
+import { loadPublicDesignMeta } from "../../server/lib/public-design-meta.server";
 import DesignEditorRoute from "../pages/DesignEditor";
-import { designResourceMeta, loadPublicDesignMeta } from "./public-design-meta";
+import { designResourceMeta } from "./public-design-meta";
 
 /**
  * Keep the route module itself as a React Fast Refresh boundary. A bare

package/corpus/templates/design/app/routes/public-design-meta.ts

@@ -1,14 +1,8 @@
-import { getConfiguredAppBasePath } from "@agent-native/core/server";
 import {
-  SSR_QUERY_CACHE_KEY_HEADER,
   buildResourceSocialMeta,
   normalizeDocumentTitle,
   type SocialMetaDescriptor,
 } from "@agent-native/core/shared";
-import { and, eq } from "drizzle-orm";
-import { data } from "react-router";
-
-import { getDb, schema } from "../../server/db";
 
 export interface PublicDesignMetaData {
   resource: { title: string; description: string | null } | null;
@@ -16,39 +10,6 @@ export interface PublicDesignMetaData {
   basePath: string;
 }
 
-export function publicDesignMetaLoaderData(payload: PublicDesignMetaData) {
-  return data(payload, {
-    headers: { [SSR_QUERY_CACHE_KEY_HEADER]: "query" },
-  });
-}
-
-export async function loadPublicDesignMeta(
-  id: string | undefined,
-  requestUrl: string,
-): Promise<PublicDesignMetaData> {
-  const [resource] = id
-    ? await getDb()
-        .select({
-          title: schema.designs.title,
-          description: schema.designs.description,
-        })
-        .from(schema.designs)
-        .where(
-          and(
-            eq(schema.designs.id, id),
-            eq(schema.designs.visibility, "public"),
-          ),
-        )
-        .limit(1)
-    : [];
-
-  return {
-    resource: resource ?? null,
-    origin: new URL(requestUrl).origin,
-    basePath: getConfiguredAppBasePath(),
-  };
-}
-
 export function designResourceMeta(
   loaderData: PublicDesignMetaData | undefined,
   fallbackTitle: string,

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

@@ -5,10 +5,8 @@ import type { LoaderFunctionArgs, MetaFunction } from "react-router";
 import enUSMessages from "@/i18n/en-US";
 
 import { getDb, schema } from "../../server/db";
-import {
-  designResourceMeta,
-  publicDesignMetaLoaderData,
-} from "./public-design-meta";
+import { publicDesignMetaLoaderData } from "../../server/lib/public-design-meta.server";
+import { designResourceMeta } from "./public-design-meta";
 
 export { default } from "../pages/Templates";
 

package/corpus/templates/design/app/routes/visual-edit_.$id.tsx

@@ -2,8 +2,9 @@ import type { LoaderFunctionArgs, MetaFunction } from "react-router";
 
 import enUSMessages from "@/i18n/en-US";
 
+import { loadPublicDesignMeta } from "../../server/lib/public-design-meta.server";
 import DesignEditorRoute from "../pages/DesignEditor";
-import { designResourceMeta, loadPublicDesignMeta } from "./public-design-meta";
+import { designResourceMeta } from "./public-design-meta";
 
 /**
  * The skill's local editor has its own route so a capability can authorize this

package/corpus/templates/design/changelog/2026-09-25-escape-finishes-open-pen-paths.md

@@ -0,0 +1,7 @@
+---
+type: fixed
+date: 2026-09-25
+---
+
+Escape finishes a Pen path as an open vector; during a pointer gesture, it
+cancels that gesture and keeps previously placed anchors.

package/corpus/templates/design/changelog/2026-09-25-pen-paths-remain-available-after-save-failures-and-retries-k.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-25
+---
+
+Pen paths remain available after save failures, and retries keep the active tool

package/corpus/templates/design/changelog/2026-09-25-pen-retries-keep-extending-the-selected-vector-after-a-save-.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-25
+---
+
+Pen retries keep extending the selected vector after a save failure

package/corpus/templates/design/changelog/2026-09-25-pressing-enter-finishes-a-new-pen-path-and-selects-its-vector-on-move.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-25
+---
+
+Pressing Enter finishes a new Pen path and selects its vector on Move.

package/corpus/templates/design/changelog/2026-09-26-cmd-d-places-screen-duplicates-in-the-next-open-slot.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-26
+---
+
+Cmd+D places screen duplicates in the next open slot

package/corpus/templates/design/changelog/2026-09-26-dropping-a-layer-beside-a-sibling-keeps-that-insertion-point.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-26
+---
+
+Dropping a layer beside a sibling keeps that insertion point.

package/corpus/templates/design/changelog/2026-09-26-layers-dragged-out-of-frames-now-land-at-the-board-level-and.md

@@ -0,0 +1,6 @@
+---
+type: improved
+date: 2026-09-26
+---
+
+Layers dragged out of frames now land at the board level, and copies keep their intended order and selection.

package/corpus/templates/design/changelog/2026-09-26-opening-shared-designs-no-longer-fails-during-client-startup.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-26
+---
+
+Opening shared designs no longer fails during client startup

package/corpus/templates/design/package.json

@@ -5,7 +5,7 @@
   "type": "module",
   "scripts": {
     "dev": "agent-native dev --inspect",
-    "build": "agent-native build",
+    "build": "agent-native build && tsx scripts/check-client-server-boundary.ts",
     "start": "agent-native start",
     "test": "vitest --run",
     "typecheck": "agent-native typecheck && pnpm exec tsc --noEmit -p app/components/design/bridge/tsconfig.json",

package/corpus/templates/design/scripts/check-client-server-boundary.ts

@@ -0,0 +1,55 @@
+import { readdir, readFile } from "node:fs/promises";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+const clientRoot = path.resolve(
+  path.dirname(fileURLToPath(import.meta.url)),
+  "../build/client",
+);
+const textAsset = /\.(?:css|html|js|json|map|mjs)$/i;
+const forbidden = [
+  ["PGlite database client", "__agentNativePgliteClients"],
+  ["server database module", /server[\\/]db(?:[\\/.'"?]|$)/i],
+] as const;
+
+async function textAssets(directory: string): Promise<string[]> {
+  const entries = await readdir(directory, { withFileTypes: true });
+  const paths = await Promise.all(
+    entries.map((entry) => {
+      const filePath = path.join(directory, entry.name);
+      return entry.isDirectory()
+        ? textAssets(filePath)
+        : textAsset.test(entry.name)
+          ? [filePath]
+          : [];
+    }),
+  );
+  return paths.flat();
+}
+
+const assets = await textAssets(clientRoot);
+if (assets.length === 0) {
+  throw new Error(`No text assets found in client build: ${clientRoot}`);
+}
+
+const leaks: string[] = [];
+for (const asset of assets) {
+  const source = await readFile(asset, "utf8");
+  for (const [label, pattern] of forbidden) {
+    if (
+      typeof pattern === "string"
+        ? source.includes(pattern)
+        : pattern.test(source)
+    ) {
+      leaks.push(`${path.relative(clientRoot, asset)}: ${label}`);
+    }
+  }
+}
+
+if (leaks.length > 0) {
+  throw new Error(
+    `Server database code found in client assets:\n${leaks.join("\n")}`,
+  );
+}
+
+console.log(`Checked ${assets.length} client assets for server database code.`);

package/corpus/templates/design/server/lib/public-design-meta.server.ts

@@ -0,0 +1,40 @@
+import { getConfiguredAppBasePath } from "@agent-native/core/server";
+import { SSR_QUERY_CACHE_KEY_HEADER } from "@agent-native/core/shared";
+import { and, eq } from "drizzle-orm";
+import { data } from "react-router";
+
+import type { PublicDesignMetaData } from "../../app/routes/public-design-meta";
+import { getDb, schema } from "../db";
+
+export function publicDesignMetaLoaderData(payload: PublicDesignMetaData) {
+  return data(payload, {
+    headers: { [SSR_QUERY_CACHE_KEY_HEADER]: "query" },
+  });
+}
+
+export async function loadPublicDesignMeta(
+  id: string | undefined,
+  requestUrl: string,
+): Promise<PublicDesignMetaData> {
+  const [resource] = id
+    ? await getDb()
+        .select({
+          title: schema.designs.title,
+          description: schema.designs.description,
+        })
+        .from(schema.designs)
+        .where(
+          and(
+            eq(schema.designs.id, id),
+            eq(schema.designs.visibility, "public"),
+          ),
+        )
+        .limit(1)
+    : [];
+
+  return {
+    resource: resource ?? null,
+    origin: new URL(requestUrl).origin,
+    basePath: getConfiguredAppBasePath(),
+  };
+}