@agent-native/core 0.187.0 → 0.187.1-nightly-20260925003003

package/corpus/templates/design/app/pages/design-editor/commands/styles-change.ts

@@ -16,7 +16,7 @@ export interface StylesChangeArgs {
     styles: Record<string, string>,
   ) => boolean;
   commitRelativeStyleDeltaToSelectedLayers: (
-    property: string,
+    property: string | string[],
     operation: number | ScrubRelativeExpression,
     phase?: StyleChangeMeta["phase"],
   ) => boolean;
@@ -78,7 +78,7 @@ export function runStylesChange(
   if (meta?.capturedStyleTargets && meta.phase !== "preview") {
     commitCapturedStyleTargets(
       Object.fromEntries(
-        Object.entries(styles).filter(([, value]) => Boolean(value)),
+        Object.entries(styles).filter(([, value]) => value !== undefined),
       ),
       meta.capturedStyleTargets,
       meta.interactionState,
@@ -115,16 +115,29 @@ export function runStylesChange(
     return;
   }
   const selector = selectedElement?.selector ?? "body";
-  const entries = Object.entries(styles).filter(([, value]) => Boolean(value));
+  const entries = Object.entries(styles).filter(
+    ([, value]) => value !== undefined,
+  );
   if (entries.length === 0) return;
-  if (meta?.relativeExpression && entries.length === 1) {
-    const [property] = entries[0]!;
-    commitRelativeStyleDeltaToSelectedLayers(
-      property,
-      meta.relativeExpression,
-      meta.phase,
+  const relativeProperties =
+    meta?.relativeDeltaProperties ??
+    (entries.length === 1 ? [entries[0]![0]] : []);
+  const relativeOperation = meta?.relativeExpression ?? meta?.relativeDelta;
+  if (
+    relativeOperation !== undefined &&
+    relativeProperties.length > 0 &&
+    relativeProperties.every((property) =>
+      entries.some(([entryProperty]) => entryProperty === property),
+    )
+  ) {
+    const applied = commitRelativeStyleDeltaToSelectedLayers(
+      relativeProperties.length === 1
+        ? relativeProperties[0]!
+        : relativeProperties,
+      relativeOperation,
+      meta?.phase,
     );
-    return;
+    if (meta?.relativeExpression || applied) return;
   }
   const target = styleWriteTarget({ selector, selectedElement });
   // T10: mirror handleStyleChange's text-range routing here. Without
@@ -169,26 +182,6 @@ export function runStylesChange(
     }
     return;
   }
-  // Mixed-value arrow-step parity (item 7): see handleStyleChange's
-  // matching comment for the full defensive-read rationale. A relative
-  // delta is inherently single-valued (one scrub gesture on one field),
-  // so this only applies when the batched patch has exactly one entry —
-  // a multi-property patch (e.g. a shadow popover's X+Y+blur+spread all
-  // at once) has no single delta to apply per-node and falls through to
-  // the existing absolute-value paths unchanged.
-  const relativeDelta = (meta as { relativeDelta?: number } | undefined)
-    ?.relativeDelta;
-  if (typeof relativeDelta === "number" && entries.length === 1) {
-    const [singleProperty] = entries[0]!;
-    if (
-      commitRelativeStyleDeltaToSelectedLayers(
-        singleProperty,
-        relativeDelta,
-        meta?.phase,
-      )
-    )
-      return;
-  }
   if (
     selectedElement &&
     commitStylesToSelectedLayers(Object.fromEntries(entries), meta?.phase)

package/corpus/templates/design/app/pages/design-editor/commands/swap-fill-stroke.ts

@@ -0,0 +1,105 @@
+import { parseCssColor, rgbaToCss } from "@shared/color-utils";
+
+import { isVectorShapeElement } from "@/components/design/edit-panel/element-classification";
+import type { ElementInfo } from "@/components/design/types";
+
+export interface SwapFillStrokeArgs {
+  canEditDesign: boolean;
+  selectedElement: ElementInfo | null;
+  handleStylesChange: (styles: Record<string, string>) => void;
+}
+
+function hasPaint(value: string): boolean {
+  const normalized = value.trim().toLowerCase();
+  if (!normalized || normalized === "none" || normalized === "transparent") {
+    return false;
+  }
+  const color = parseCssColor(value);
+  return color ? color.a > 0 : true;
+}
+
+function swappedPaint(
+  paint: string,
+  opacity: string | undefined,
+): { paint: string; opacity: string } {
+  const alpha = Number.parseFloat(opacity ?? "1");
+  if (!Number.isFinite(alpha) || alpha >= 1) return { paint, opacity: "" };
+  const color = parseCssColor(paint);
+  if (color) {
+    return { paint: rgbaToCss({ ...color, a: color.a * alpha }), opacity: "" };
+  }
+  // A gradient value has nowhere to carry alpha, so it keeps its opacity.
+  return { paint, opacity: String(alpha) };
+}
+
+/**
+ * Figma's Shift+X. Matches Figma even when one side is empty: an element with
+ * a fill and no stroke ends up with a stroke and no fill (not a no-op). Both
+ * sides commit together so the swap is a single undo step.
+ */
+export function runSwapFillStroke({
+  canEditDesign,
+  selectedElement,
+  handleStylesChange,
+}: SwapFillStrokeArgs): void {
+  if (!canEditDesign || !selectedElement) return;
+  if (isVectorShapeElement(selectedElement)) {
+    const styles = selectedElement.computedStyles;
+    // SVG gradient styles render as url(#id), but the editor's authored
+    // custom properties hold the portable gradient values. Capture both
+    // before committing either side so Shift+X can rebuild each definition
+    // without leaving the other paint pointing at a removed id.
+    const fill =
+      selectedElement.inlineStyles?.["--an-vector-fill-gradient"] ||
+      styles["--an-vector-fill-gradient"] ||
+      styles.fill ||
+      "";
+    const stroke =
+      selectedElement.inlineStyles?.["--an-vector-stroke-gradient"] ||
+      styles["--an-vector-stroke-gradient"] ||
+      styles.stroke ||
+      "";
+    const fillHasPaint = hasPaint(fill);
+    const strokeWidth = Number.parseFloat(styles.strokeWidth ?? "0");
+    const strokeOpacity = Number.parseFloat(styles.strokeOpacity ?? "1");
+    const strokeHasPaint =
+      hasPaint(stroke) && strokeWidth > 0 && strokeOpacity > 0;
+    if (!fillHasPaint && !strokeHasPaint) return;
+    // Opacity rides in the colours where it can: an inline fill-opacity
+    // would override the fill-opacity="0" that keeps an open path's chord
+    // unpainted. An empty value removes the old declaration.
+    const nextFill = strokeHasPaint
+      ? swappedPaint(stroke, styles.strokeOpacity)
+      : { paint: "none", opacity: "" };
+    const nextStroke = fillHasPaint
+      ? swappedPaint(fill, styles.fillOpacity)
+      : { paint: "none", opacity: "" };
+    // Removing only the inline value would leave an imported SVG's
+    // presentation opacity multiplying the swapped colour.
+    const opacityFor = (
+      next: { paint: string; opacity: string },
+      current: string | undefined,
+    ) =>
+      next.opacity === "" &&
+      next.paint !== "none" &&
+      Number.parseFloat(current ?? "1") < 1
+        ? "1"
+        : next.opacity;
+    const fillOpacity = opacityFor(nextFill, styles.fillOpacity);
+    const strokeOpacityValue = opacityFor(nextStroke, styles.strokeOpacity);
+    handleStylesChange({
+      fill: nextFill.paint,
+      fillOpacity,
+      stroke: nextStroke.paint,
+      strokeOpacity: strokeOpacityValue,
+      ...(fillHasPaint && !strokeHasPaint ? { strokeWidth: "1px" } : {}),
+    });
+    return;
+  }
+  const currentFill = selectedElement.computedStyles.backgroundColor ?? "";
+  const currentStroke = selectedElement.computedStyles.borderColor ?? "";
+  handleStylesChange({
+    backgroundColor: currentStroke || "transparent",
+    borderColor: currentFill || "transparent",
+  });
+}

package/corpus/templates/design/app/pages/design-editor/pending-edits.ts

@@ -1764,6 +1764,27 @@ export function applyScopedVisualStyleEdit(args: {
     source,
   } = args;
   const normalizedProperty = normalizeCssPropertyName(property);
+  // An empty value clears the declaration at the edit's own scope.
+  if (value === "") {
+    return applyVisualEdit(
+      content,
+      upperBoundPx == null
+        ? {
+            kind: "style",
+            operation: "remove",
+            target,
+            property: normalizedProperty,
+          }
+        : {
+            kind: "breakpoint-style",
+            operation: "remove",
+            target,
+            maxWidthPx: upperBoundPx,
+            property: normalizedProperty,
+          },
+      { source },
+    );
+  }
   if (upperBoundPx != null && isVectorEndpointProperty(normalizedProperty)) {
     // Endpoint values require marker definitions and shape attributes, which
     // cannot be represented by a media-scoped custom property. Let the shared

package/corpus/templates/design/app/pages/present-review-state.ts

@@ -5,9 +5,15 @@ export interface PresentReviewState {
 
 export function resolvePresentEscapeAction(
   state: PresentReviewState,
-): "close-comments" | "defer-to-comment-mode" | "exit-presentation" {
+  reviewEmbed = false,
+):
+  | "close-comments"
+  | "defer-to-comment-mode"
+  | "stay-in-review-embed"
+  | "exit-presentation" {
   if (state.commentsOpen) return "close-comments";
   if (state.commentMode) return "defer-to-comment-mode";
+  if (reviewEmbed) return "stay-in-review-embed";
   return "exit-presentation";
 }
 

package/corpus/templates/design/changelog/2026-09-24-a-vector-s-stroke-row-shows-position-beside-weight-instead-o.md

@@ -0,0 +1,6 @@
+---
+type: improved
+date: 2026-09-24
+---
+
+A vector's stroke row shows Position beside Weight instead of a lone Weight field.

package/corpus/templates/design/changelog/2026-09-24-css-margins-can-be-adjusted-on-the-canvas-and-in-the-layout-.md

@@ -0,0 +1,6 @@
+---
+type: added
+date: 2026-09-24
+---
+
+CSS margins can be adjusted on the canvas and in the layout inspector.

package/corpus/templates/design/changelog/2026-09-24-design-reviews-show-the-real-canvas-beside-its-conversation.md

@@ -0,0 +1,6 @@
+---
+type: improved
+date: 2026-09-24
+---
+
+Design reviews show the real canvas beside its conversation.

package/corpus/templates/design/changelog/2026-09-24-double-clicking-text-or-a-vector-on-the-board-starts-editing.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-24
+---
+
+Double-clicking text or a vector on the board starts editing it, and clicking empty canvas deselects it.

package/corpus/templates/design/changelog/2026-09-24-interact-opens-a-focused-responsive-preview-without-reloadin.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-24
+---
+
+Interact opens a focused responsive preview without reloading the running design.

package/corpus/templates/design/changelog/2026-09-24-keep-gradient-values-visible-and-limit-canvas-paints-to-supp.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-24
+---
+
+Keep gradient values visible and limit canvas paints to supported backgrounds

package/corpus/templates/design/changelog/2026-09-24-open-pen-paths-no-longer-paint-a-filled-triangle-after-shift.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-24
+---
+
+Open pen paths no longer paint a filled triangle after Shift+X, matching Figma.

package/corpus/templates/design/changelog/2026-09-24-re-clicking-the-active-responsive-preview-returns-to-the-can.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-24
+---
+
+Re-clicking the active responsive preview returns to the canvas even with pending edits

package/corpus/templates/design/changelog/2026-09-24-resized-vectors-can-be-point-edited-and-the-k-scale-tool-sho.md

@@ -0,0 +1,6 @@
+---
+type: added
+date: 2026-09-24
+---
+
+Resized vectors can be point-edited, and the K scale tool shows a Scale section with a live factor.

package/corpus/templates/design/changelog/2026-09-24-while-editing-text-click-to-place-the-caret-and-double-click.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-24
+---
+
+While editing text, click to place the caret and double-click a word to restyle just that word.

package/corpus/templates/design/e2e/helpers.ts

@@ -457,6 +457,12 @@ export async function enterInteractView(
     "data-screen-interact-mode",
     "false",
   );
+  const previewIframe = screenShell
+    .locator(DESIGN_PREVIEW_IFRAME_SELECTOR)
+    .first();
+  await expect(previewIframe).toBeVisible();
+  const previewIframeHandle = await previewIframe.elementHandle();
+  if (!previewIframeHandle) throw new Error("screen preview iframe is missing");
   // The screen card can extend beneath the fixed inspector at narrow canvas
   // widths; invoke the button without relying on the panel's overlapping
   // physical hit area.
@@ -467,8 +473,19 @@ export async function enterInteractView(
     "data-screen-interact-mode",
     "true",
   );
-  // Interact is a responsive view of the same editor. Rails and the screen
-  // shell stay mounted; assert the view's own device preview below instead.
+  await expect(
+    page.getByRole("button", { name: "Exit responsive preview" }),
+  ).toBeVisible();
+  expect(
+    await previewIframeHandle.evaluate((before) =>
+      Boolean(
+        before
+          .closest("[data-screen-shell]")
+          ?.querySelector("iframe[data-design-preview-iframe]") === before,
+      ),
+    ),
+  ).toBe(true);
+  // The focused responsive preview keeps its original iframe in the DOM.
   await expect
     .poll(
       async () =>

package/corpus/templates/mail/app/components/email/ComposeModal.tsx

@@ -843,7 +843,7 @@ export function ComposeModal({
           ? "bottom-0 h-11 rounded-t-xl sm:w-[540px]"
           : isExpanded
             ? "top-0 bottom-0 h-auto rounded-none sm:top-4 sm:bottom-4 sm:w-[min(960px,calc(100vw-var(--compose-right)-1rem))] sm:rounded-xl"
-            : "bottom-0 h-[100dvh] sm:h-[min(540px,_calc(100dvh_-_2rem))] sm:w-[min(800px,_calc(100vw_-_var(--compose-right)_-_1rem))] sm:rounded-xl",
+            : "bottom-0 h-[100dvh] sm:h-[min(540px,_calc(100dvh_-_2rem))] md:w-[min(490px,_calc(100vw_-_var(--compose-right)_-_1rem))] sm:rounded-xl",
       )}
       data-mail-compose
       style={composeStyle}

package/corpus/templates/mail/changelog/2026-09-24-gmail-push-ack.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-24
+---
+
+Gmail inboxes refresh reliably when push notifications are delayed.

package/corpus/templates/mail/changelog/2026-09-24-the-composer-keeps-its-taller-layout-in-a-narrower-window.md

@@ -0,0 +1,6 @@
+---
+type: improved
+date: 2026-09-24
+---
+
+The composer keeps its taller layout in a narrower window.

package/corpus/templates/mail/server/db/schema.ts

@@ -1,4 +1,10 @@
-import { table, text, integer, index } from "@agent-native/core/db/schema";
+import {
+  bigint,
+  index,
+  integer,
+  table,
+  text,
+} from "@agent-native/core/db/schema";
 
 /**
  * Short-lived, owner-scoped continuation state for the external Mail
@@ -111,6 +117,9 @@ export const mailSyncAccounts = table(
       .default("idle"),
     lastError: text("last_error"),
     lastSyncedAt: integer("last_synced_at"),
+    lastPushGeneration: bigint("last_push_generation", { mode: "number" })
+      .notNull()
+      .default(0),
     syncClaimId: text("sync_claim_id"),
     syncClaimedAt: integer("sync_claimed_at"),
     // Compact cached labels.list result: [{id,name,type,color?,messagesTotal?,
@@ -123,6 +132,22 @@ export const mailSyncAccounts = table(
   (t) => [index("mail_sync_accounts_owner_idx").on(t.ownerEmail)],
 );
 
+export const mailInboxPushInvalidations = table(
+  "mail_inbox_push_invalidations",
+  {
+    id: text("id").primaryKey(),
+    ownerEmail: text("owner_email").notNull(),
+    accountEmail: text("account_email").notNull(),
+    generation: bigint("generation", { mode: "number" }).notNull().default(1),
+  },
+  (t) => [
+    index("mail_inbox_push_invalidations_owner_account_idx").on(
+      t.ownerEmail,
+      t.accountEmail,
+    ),
+  ],
+);
+
 /**
  * SQL mirror of each connected account's INBOX threads, kept fresh by
  * `server/lib/inbox-sync.ts`. Metadata only — no bodies, no HTML.

package/corpus/templates/mail/server/lib/inbox-store.ts

@@ -69,6 +69,7 @@ export type SyncAccountRow = {
   status: "idle" | "syncing" | "error" | "needs_reauth";
   lastError: string | null;
   lastSyncedAt: number | null;
+  lastPushGeneration: number;
   syncClaimId: string | null;
   syncClaimedAt: number | null;
   labels: CachedGmailLabel[] | null;
@@ -144,6 +145,7 @@ function toSyncAccountRow(
     status: row.status as SyncAccountRow["status"],
     lastError: row.lastError,
     lastSyncedAt: row.lastSyncedAt,
+    lastPushGeneration: row.lastPushGeneration,
     syncClaimId: row.syncClaimId,
     syncClaimedAt: row.syncClaimedAt,
     labels: row.labelsJson
@@ -631,6 +633,7 @@ export type ThreadUpsertInput = {
 
 export async function upsertInboxThreadRows(
   rows: ThreadUpsertInput[],
+  db: any = getDb(),
 ): Promise<void> {
   if (rows.length === 0) return;
   const now = Date.now();
@@ -670,7 +673,7 @@ export async function upsertInboxThreadRows(
       localMutationFields: null,
     }));
 
-  await getDb()
+  await db
     .insert(schema.mailInboxThreads)
     .values(values)
     .onConflictDoUpdate({
@@ -747,6 +750,7 @@ export async function deleteInboxThreadRow(
   accountEmail: string,
   threadId: string,
   readStartedAt?: number,
+  db: any = getDb(),
 ): Promise<void> {
   const conditions = [
     eq(
@@ -757,9 +761,7 @@ export async function deleteInboxThreadRow(
   if (readStartedAt !== undefined) {
     conditions.push(lt(schema.mailInboxThreads.updatedAt, readStartedAt));
   }
-  await getDb()
-    .delete(schema.mailInboxThreads)
-    .where(and(...conditions));
+  await db.delete(schema.mailInboxThreads).where(and(...conditions));
 }
 
 /**
@@ -770,8 +772,9 @@ export async function markThreadsOutOfInboxBeforeSync(
   ownerEmail: string,
   accountEmail: string,
   cutoffSyncedAt: number,
+  db: any = getDb(),
 ): Promise<void> {
-  await getDb()
+  await db
     .update(schema.mailInboxThreads)
     .set({ inInbox: 0, updatedAt: Date.now() })
     .where(
@@ -827,26 +830,25 @@ export class SyncClaimLostError extends Error {
   }
 }
 
-/**
- * Guards a batch of row writes (`upsertInboxThreadRows`,
- * `markThreadsOutOfInboxBeforeSync`, `deleteInboxThreadRow`) that have no
- * `sync_claim_id` column of their own to fence against, unlike
- * {@link patchSyncAccount}'s `opts.claimId`. One SELECT immediately before
- * the write; throws {@link SyncClaimLostError} when the claim no longer
- * matches instead of letting a lapsed worker overwrite a newer worker's rows.
- */
-export async function assertSyncClaimHeld(
+export async function withSyncClaim<T>(
   ownerEmail: string,
   accountEmail: string,
   claimId: string,
-): Promise<void> {
-  const rows = await getDb()
-    .select({ syncClaimId: schema.mailSyncAccounts.syncClaimId })
-    .from(schema.mailSyncAccounts)
-    .where(eq(schema.mailSyncAccounts.id, rowId(ownerEmail, accountEmail)))
-    .limit(1);
-  if (rows[0]?.syncClaimId !== claimId)
-    throw new SyncClaimLostError(accountEmail);
+  write: (tx: any) => Promise<T>,
+): Promise<T> {
+  return getDb().transaction(async (tx: any) => {
+    const rows = await tx
+      .select({ syncClaimId: schema.mailSyncAccounts.syncClaimId })
+      .from(schema.mailSyncAccounts)
+      .where(eq(schema.mailSyncAccounts.id, rowId(ownerEmail, accountEmail)))
+      .for("update")
+      .limit(1);
+    if (rows[0]?.syncClaimId !== claimId)
+      throw new SyncClaimLostError(accountEmail);
+
+    // Keep row mutations inside the lock so replacement claims cannot interleave.
+    return write(tx);
+  });
 }
 
 /**
@@ -917,12 +919,55 @@ export type SyncAccountPatch = Partial<{
   status: SyncAccountRow["status"];
   lastError: string | null;
   lastSyncedAt: number | null;
+  lastPushGeneration: number;
   syncClaimId: string | null;
   syncClaimedAt: number | null;
   labels: CachedGmailLabel[];
   labelsUpdatedAt: number;
 }>;
 
+export async function recordInboxPushInvalidation(
+  ownerEmail: string,
+  accountEmail: string,
+): Promise<void> {
+  await getDb()
+    .insert(schema.mailInboxPushInvalidations)
+    .values({
+      id: rowId(ownerEmail, accountEmail),
+      ownerEmail: ownerEmail.toLowerCase(),
+      accountEmail: accountEmail.toLowerCase(),
+      generation: 1,
+    })
+    .onConflictDoUpdate({
+      target: schema.mailInboxPushInvalidations.id,
+      set: {
+        generation: sql`${schema.mailInboxPushInvalidations.generation} + 1`,
+      },
+    });
+}
+
+export async function readInboxPushGeneration(
+  ownerEmail: string,
+  accountEmail: string,
+): Promise<number> {
+  const rows = await getDb()
+    .select({ generation: schema.mailInboxPushInvalidations.generation })
+    .from(schema.mailInboxPushInvalidations)
+    .where(
+      and(
+        eq(
+          schema.mailInboxPushInvalidations.ownerEmail,
+          ownerEmail.toLowerCase(),
+        ),
+        eq(
+          schema.mailInboxPushInvalidations.accountEmail,
+          accountEmail.toLowerCase(),
+        ),
+      ),
+    );
+  return rows.reduce((total, row) => total + row.generation, 0);
+}
+
 /**
  * Updates one sync-account row. When `opts.claimId` is given, the write is
  * fenced with `AND sync_claim_id = ?` and the return value says whether a row
@@ -960,7 +1005,7 @@ export async function patchSyncAccount(
  * running against the reset row keeps its claim, and a later fenced write
  * from that same stale run would succeed and could restore the old
  * watermark. Clearing the claim here means that worker's next fenced write
- * (via `patchSyncAccount`'s `opts.claimId` or `assertSyncClaimHeld`) raises
+ * (via `patchSyncAccount`'s `opts.claimId` or `withSyncClaim`) raises
  * {@link SyncClaimLostError} and stops it instead.
  */
 export async function resetSyncAccountProgress(