@agent-native/core 0.178.0 → 0.178.1-nightly-20260909221943

package/corpus/templates/design/actions/take-design-screenshot.ts

@@ -48,6 +48,7 @@ import { and, eq } from "drizzle-orm";
 import { z } from "zod";
 
 import { getDb, schema } from "../server/db/index.js";
+import { isAllowedFigmaSvgRenderRequest } from "../server/lib/design-to-figma-svg.js";
 import "../server/db/index.js"; // ensure registerShareableResource runs
 
 // ---------------------------------------------------------------------------
@@ -95,12 +96,22 @@ export interface ScreenshotDiagnostics {
   zeroSizeOrOffscreen: DiagnosticsZeroSizeEntry[];
 }
 
+export type ScreenshotUploadError =
+  | {
+      code: "file_storage_not_configured";
+      message: string;
+    }
+  | {
+      code: "file_upload_failed";
+      message: string;
+    };
+
 export interface ScreenshotResult {
   viewport: ScreenshotViewport;
   url: string;
   /** True when persisted via a durable upload provider. */
   persisted: boolean;
-  uploadError?: string;
+  uploadError?: ScreenshotUploadError;
   bytes: number;
   diagnostics: ScreenshotDiagnostics;
 }
@@ -586,6 +597,17 @@ export default defineAction({
         await context.addInitScript(
           "globalThis.__name = globalThis.__name || function (value) { return value; };",
         );
+        await context.route("**/*", async (route) => {
+          if (await isAllowedFigmaSvgRenderRequest(route.request().url())) {
+            await route.continue();
+          } else {
+            await route.abort("blockedbyclient");
+          }
+        });
+        // A static PNG never needs a live socket. Leaving a routed WebSocket
+        // unconnected blocks the separate WebSocket transport that
+        // BrowserContext.route("**/*") does not see.
+        await context.routeWebSocket("**/*", () => {});
         const page = await context.newPage();
         const consoleErrors: string[] = [];
         const fontLoadFailures: string[] = [];
@@ -677,23 +699,35 @@ export default defineAction({
           // for tall complex screens, reproduced on a 1440x3200 fixture.
           const png = await page.screenshot({ type: "png", fullPage: true });
 
-          const uploaded = await uploadFile({
-            data: png,
-            mimeType: "image/png",
-            filename: `design-${file.designId}-${file.filename}-${viewport.label}.png`,
-            ownerEmail,
-          }).catch(() => null);
+          let uploaded: Awaited<ReturnType<typeof uploadFile>> | null = null;
+          let uploadError: ScreenshotUploadError | undefined;
+          try {
+            uploaded = await uploadFile({
+              data: png,
+              mimeType: "image/png",
+              filename: `design-${file.designId}-${file.filename}-${viewport.label}.png`,
+              ownerEmail,
+            });
+          } catch {
+            uploadError = {
+              code: "file_upload_failed",
+              message:
+                "Configured file storage rejected the screenshot upload.",
+            };
+          }
+          if (!uploaded && !uploadError) {
+            uploadError = {
+              code: "file_storage_not_configured",
+              message:
+                "Screenshot was rendered but not returned because file storage is not configured.",
+            };
+          }
 
           screenshots.push({
             viewport,
             url: uploaded?.url ?? "",
             persisted: !!uploaded,
-            ...(uploaded?.url
-              ? {}
-              : {
-                  uploadError:
-                    "Screenshot was rendered but not returned because file storage is not configured.",
-                }),
+            ...(uploadError ? { uploadError } : {}),
             bytes: png.byteLength,
             diagnostics: {
               viewport,

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

@@ -39,6 +39,7 @@ const EXTERNAL_CONNECTOR_TOOL_NAMES = [
   "create-file",
   "update-file",
   "rename-screen",
+  "export-png",
 ];
 
 const INITIAL_TOOL_NAMES = [
@@ -77,6 +78,7 @@ const INITIAL_TOOL_NAMES = [
   "create-file",
   "update-file",
   "rename-screen",
+  "export-png",
   "navigate",
   "provider-api-catalog",
   "provider-api-docs",
@@ -235,7 +237,7 @@ export default createAgentChatPlugin({
   mcp: {
     connectorCatalog: EXTERNAL_CONNECTOR_TOOL_NAMES,
     instructions:
-      "Resolve a named template or prior design first with list-design-templates / list-designs; copy with create-design-from-template, then adapt with edit-design — never regenerate a copied screen with generate-design. For new-design exploration use create-design then present-design-variants (2-5 variants) and surface the returned open link; do not navigate. Hand-off goes through export-html / export-zip / export-coding-handoff / export-design-as-figma-svg. Persist early: create or update the design and its files as soon as a coherent candidate exists. " +
+      "Resolve a named template or prior design first with list-design-templates / list-designs; copy with create-design-from-template, then adapt with edit-design — never regenerate a copied screen with generate-design. For new-design exploration use create-design then present-design-variants (2-5 variants) and surface the returned open link; do not navigate. Hand-off goes through export-png for one screen, or export-html / export-zip / export-coding-handoff / export-design-as-figma-svg for other formats. Persist early: create or update the design and its files as soon as a coherent candidate exists. " +
       'Design system: get-design, get-design-snapshot, and view-screen return `designSystem` (a bounded summary with scope "summary" and a `next` line); call get-design-system { id } once before the first screen you author for the full context (create-design returns it in full), then reuse it. Apply designSystem.agentContext, plus index-design-tokens for an existing design, before authoring or restyling; never invent a generic palette. For a new design, pass the exact title as `designSystem` or a designSystemId; omit both to link the caller\'s default. Preserve existing screen composition as well as linked system tokens, fonts, assets, and custom instructions. Read back the saved file after every visual mutation.',
   },
   externalAgents: { writes: "allowlisted" },

package/corpus/templates/factory/actions/dispatch-factory-item.ts

@@ -47,6 +47,7 @@ import {
   createSlackReader,
   isAgentNativeSlackUserName,
 } from "../server/triage/slack-client.js";
+import { dispatchSkipStatusWrite } from "../server/triage/slack-review-window.js";
 
 /** Slack notifies only with `<@USERID>`. Plaintext @handles do not ping anyone. */
 const REPLY_INSTRUCTION =
@@ -280,7 +281,7 @@ export async function recordAutomaticBuilderDecision(input: {
   userEmail: string;
   orgId: string;
   factoryId: string;
-  outcome: "propose_fix" | "needs_manual";
+  outcome: "propose_fix" | "needs_manual" | "observe";
   reason: string;
   guardResults: Array<{ code: string; passed: boolean; reason: string }>;
 }) {
@@ -323,13 +324,20 @@ export async function recordAutomaticBuilderDecision(input: {
 
 export default defineAction({
   description:
-    "Tag Builder for a Factory item, or record a skip when clearBug is false. Slack items stay in-thread: this action pings Builder with the configured Slack member id; do not post Slack messages or @handles yourself. Pass optional reaction (an emoji name such as robot_face) to mark the item source when that provider can; omit it to add no reaction. Grouped Slack repeats share one Builder thread. GitHub issues and Sentry errors tag @builderio-bot on a GitHub issue in the factory repository. Owner-managed Clips, Design, and Content items are always left for their owner.",
+    "Tag Builder for a Factory item, or record a skip when clearBug is false or alreadyClaimed is true. Slack items stay in-thread: this action pings Builder with the configured Slack member id; do not post Slack messages or @handles yourself. Pass optional reaction (an emoji name such as robot_face) to mark the item source when that provider can; omit it to add no reaction. Grouped Slack repeats share one Builder thread. GitHub issues and Sentry errors tag @builderio-bot on a GitHub issue in the factory repository. Owner-managed Clips, Design, and Content items are always left for their owner.",
   schema: z.object({
     itemId: z.string().min(1),
+    alreadyClaimed: z
+      .boolean()
+      .default(false)
+      .describe(
+        "True when the Slack parent already has eyes or robot_face. Records the skip without starting Builder work: received items become needs_manual so they leave needsReview; items that already started keep their status. clearBug may be omitted or false.",
+      ),
     clearBug: z
       .boolean()
+      .default(false)
       .describe(
-        "True when the item is a concrete, reproducible defect with enough evidence to investigate (including visual/UI defects such as a duplicate control, broken layout, or incorrect state). False for feature requests, vague questions, and incomplete threads.",
+        "True when the item is a concrete, reproducible defect with enough evidence to investigate (including visual/UI defects such as a duplicate control, broken layout, or incorrect state). False for feature requests, vague questions, incomplete threads, and claimed Slack parents. May be omitted when alreadyClaimed is true.",
       ),
     reason: z.string().trim().min(1).max(4_000),
     productUxImplications: z
@@ -359,6 +367,7 @@ export default defineAction({
   run: async (
     {
       itemId,
+      alreadyClaimed,
       clearBug,
       reason,
       productUxImplications,
@@ -494,13 +503,19 @@ export default defineAction({
           ]
         : []),
     ];
-    const blocked = guardResults.some((guard) => !guard.passed);
+    const blocked =
+      alreadyClaimed || guardResults.some((guard) => !guard.passed);
+    const skipStatus = dispatchSkipStatusWrite(item.status);
     const decisionId = await recordAutomaticBuilderDecision({
       itemId,
       userEmail,
       orgId,
       factoryId,
-      outcome: blocked ? "needs_manual" : "propose_fix",
+      outcome: blocked
+        ? skipStatus.statusPreserved
+          ? "observe"
+          : "needs_manual"
+        : "propose_fix",
       reason,
       guardResults,
     });
@@ -518,35 +533,49 @@ export default defineAction({
         summary: reason,
         details: {
           decisionId,
+          alreadyClaimed,
           clearBug,
           productUxImplications,
           ownerOwnedArea: ownerManagedArea,
+          statusPreserved: blocked ? skipStatus.statusPreserved : false,
           guardResults,
         },
       },
     );
     if (blocked) {
-      await db.transaction(async (tx) => {
-        await tx
-          .update(triageItems)
-          .set({ status: "needs_manual", updatedAt: new Date().toISOString() })
-          .where(
-            and(
-              eq(triageItems.id, itemId),
-              eq(triageItems.orgId, orgId),
-              factoryStillPresent(tx as unknown as typeof db, orgId, factoryId),
-            ),
+      const nextStatus = skipStatus.nextStatus;
+      if (nextStatus) {
+        await db.transaction(async (tx) => {
+          await tx
+            .update(triageItems)
+            .set({
+              status: nextStatus,
+              updatedAt: new Date().toISOString(),
+            })
+            .where(
+              and(
+                eq(triageItems.id, itemId),
+                eq(triageItems.orgId, orgId),
+                factoryStillPresent(
+                  tx as unknown as typeof db,
+                  orgId,
+                  factoryId,
+                ),
+              ),
+            );
+          await requireExistingFactory(
+            tx as unknown as typeof db,
+            orgId,
+            factoryId,
           );
-        await requireExistingFactory(
-          tx as unknown as typeof db,
-          orgId,
-          factoryId,
-        );
-      });
+        });
+      }
       return {
         ok: true,
         started: false,
-        needsManual: true,
+        needsManual: skipStatus.needsManual,
+        alreadyClaimed,
+        statusPreserved: skipStatus.statusPreserved,
         decisionId,
         reason,
       };

package/corpus/templates/factory/actions/list-triage-items.ts

@@ -33,6 +33,7 @@ import {
   triageItemAuthorId,
 } from "../server/triage/metadata.js";
 import { babysitLeavesReviewWindow } from "../server/triage/pr-babysit.js";
+import { slackFeedbackLeavesReviewWindow } from "../server/triage/slack-review-window.js";
 import { readStoredUserLabels } from "../server/triage/slack-user-labels.js";
 
 export default defineAction({
@@ -79,24 +80,20 @@ export default defineAction({
     const fetchLimit =
       context?.caller === "automation" &&
       calling &&
-      calling.config.source === "github"
+      (calling.config.source === "github" || calling.config.source === "slack")
         ? Math.min(100, Math.max(effectiveLimit * 10, effectiveLimit))
         : effectiveLimit;
     const parsedCursor = cursor ? decodeInboxCursor(cursor) : null;
     const updatedAfterBound = parseUpdatedAfter(updatedAfter);
     const db = getDb();
-    const reviewStatuses =
-      source === "github"
-        ? ["pr_observed"]
-        : source === "slack"
-          ? ["received", "automation_started", "evidence_ready"]
-          : ["received"];
-    const filterGithubReviewPage =
+    const reviewStatuses = source === "github" ? ["pr_observed"] : ["received"];
+    const filterReviewPage =
       (context?.caller === "automation" &&
         calling &&
-        calling.config.source === "github") ||
-      (needsReview && source === "github");
-    const maxScanPages = filterGithubReviewPage ? 10 : 1;
+        (calling.config.source === "github" ||
+          calling.config.source === "slack")) ||
+      (needsReview && (source === "github" || source === "slack"));
+    const maxScanPages = filterReviewPage ? 10 : 1;
     const eligible: Array<(typeof triageItems)["$inferSelect"]> = [];
     let scanCursor = parsedCursor;
     let lastExamined: (typeof triageItems)["$inferSelect"] | undefined;
@@ -165,6 +162,19 @@ export default defineAction({
         ) {
           continue;
         }
+        if (
+          needsReview &&
+          source === "slack" &&
+          slackFeedbackLeavesReviewWindow({
+            status: item.status,
+            slackReactionName: metadataString(
+              parseTriageMetadata(item.metadataJson),
+              "slackReactionName",
+            ),
+          })
+        ) {
+          continue;
+        }
         eligible.push(item);
         lastKept = item;
         if (eligible.length === effectiveLimit) {

package/corpus/templates/factory/actions/poll-slack-channel.ts

@@ -26,6 +26,7 @@ import {
 import { recordFactoryAudit } from "../server/triage/audit.js";
 import type { IngestionEnvelope } from "../server/triage/contracts.js";
 import { itemDedupeKey } from "../server/triage/ids.js";
+import { mergeTriageMetadata } from "../server/triage/metadata.js";
 import {
   hasTriageSourceChanged,
   statusAfterTriageSourceUpdate,
@@ -178,6 +179,10 @@ export default defineAction({
         const lastSeenAt = sourceChanged
           ? sourceLastSeenAt
           : (existing?.lastSeenAt ?? now);
+        const metadataJson = mergeTriageMetadata(
+          existing?.metadataJson ?? "{}",
+          envelope.metadata ?? {},
+        );
         await tx
           .insert(triageItems)
           .values({
@@ -196,7 +201,7 @@ export default defineAction({
             headSha: envelope.headSha ?? null,
             coverage: envelope.coverage,
             dedupeKey: id,
-            metadataJson: JSON.stringify(envelope.metadata ?? {}),
+            metadataJson,
             lastSeenAt,
             createdAt: now,
             updatedAt,
@@ -213,7 +218,7 @@ export default defineAction({
               channelId: envelope.channelId ?? null,
               threadTs: envelope.threadTs ?? null,
               coverage: envelope.coverage,
-              metadataJson: JSON.stringify(envelope.metadata ?? {}),
+              metadataJson,
               status,
               lastSeenAt,
               updatedAt,

package/corpus/templates/factory/changelog/2026-09-09-slack-feedback-keeps-claimed-markers-after-poll.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-09
+---
+
+Slack feedback keeps a claimed 👀/🤖 marker after polling, so those threads stay out of the review queue.

package/corpus/templates/factory/changelog/2026-09-09-slack-feedback-preserves-started-claimed-threads.md

@@ -0,0 +1,6 @@
+---
+type: fixed
+date: 2026-09-09
+---
+
+Slack feedback no longer moves an already-started thread to needs-manual just because the parent already has 👀 or 🤖.

package/corpus/templates/factory/changelog/2026-09-09-slack-feedback-skips-claimed-parents.md

@@ -0,0 +1,6 @@
+---
+type: improved
+date: 2026-09-09
+---
+
+Slack feedback jobs skip threads whose parent already has 👀 or 🤖, instead of tagging Builder again.

package/corpus/templates/factory/server/lib/factory-automation-config.ts

@@ -479,7 +479,7 @@ export function buildGuardrailsText(
   }
   if (config.source === "slack") {
     lines.push(
-      "Never post Slack messages, reactions, or plaintext @handles. If the prompt names a reaction, pass it as reaction on dispatch-factory-item; that action adds it on the source when possible.",
+      "Never post Slack messages, reactions, or plaintext @handles. Pass reaction on dispatch-factory-item only when the prompt says to mark that item; omit it on skips. That action adds it on the source when possible.",
     );
   }
   const extraText = extra?.trim();

package/corpus/templates/factory/server/lib/slack-feedback-prompt.ts

@@ -17,6 +17,44 @@ non-bug report.`;
 const OBSOLETE_SLACK_TAG_PARAGRAPH =
   /The Builder reply must tag @builder\.io[\s\S]*?non-bug\s+report\.\s*/;
 
+const RETIRED_UNCONDITIONAL_ROBOT_FACE = `For each item, call dispatch-factory-item with clearBug true or false,
+productUxImplications false unless it is a pure product or design decision
+with no single correct fix, a short reason, and reaction robot_face 🤖.`;
+
+const RETIRED_CLAIMED_SKIP_VIA_CLEAR_BUG_FALSE = `Look at the parent message reactions from get-slack-feedback-context. If the
+parent already has eyes 👀 or robot_face 🤖, it has already been looked at:
+call dispatch-factory-item with clearBug false, omit reaction, and a short
+reason that names the existing marker. Do not start Builder work on it.
+
+For every other item, call dispatch-factory-item with clearBug true or false,
+productUxImplications false unless it is a pure product or design decision
+with no single correct fix, and a short reason. Pass reaction robot_face 🤖
+only when clearBug is true and the parent has neither eyes nor robot_face.
+Omit reaction on skips.`;
+
+const RETIRED_CLAIMED_SKIP_OMITS_CLEAR_BUG = `Look at the parent message reactions from get-slack-feedback-context. If the
+parent already has eyes 👀 or robot_face 🤖, it has already been looked at:
+call dispatch-factory-item with alreadyClaimed true, omit reaction, and a short
+reason that names the existing marker. Do not start Builder work on it.
+
+For every other item, call dispatch-factory-item with clearBug true or false,
+productUxImplications false unless it is a pure product or design decision
+with no single correct fix, and a short reason. Pass reaction robot_face 🤖
+only when clearBug is true and the parent has neither eyes nor robot_face.
+Omit reaction on skips.`;
+
+export const SLACK_FEEDBACK_DISPATCH_INSTRUCTIONS = `Look at the parent message reactions from get-slack-feedback-context. If the
+parent already has eyes 👀 or robot_face 🤖, it has already been looked at:
+call dispatch-factory-item with alreadyClaimed true (clearBug may be omitted
+or false), omit reaction, and a short reason that names the existing marker.
+Do not start Builder work on it.
+
+For every other item, call dispatch-factory-item with clearBug true or false,
+productUxImplications false unless it is a pure product or design decision
+with no single correct fix, and a short reason. Pass reaction robot_face 🤖
+only when clearBug is true and the parent has neither eyes nor robot_face.
+Omit reaction on skips.`;
+
 function stripPastedGuard(content: string, pasted: string): string {
   return content.split(pasted).join("");
 }
@@ -35,5 +73,14 @@ export function repairSlackFeedbackPrompt(content: string): string {
   next = stripPastedGuard(next, PASTED_HANDOFF_INSTRUCTION);
   next = stripPastedGuard(next, PASTED_MENTION_GUARD);
   next = stripPastedGuard(next, PASTED_SKIP_GUARD);
+  next = next
+    .split(RETIRED_UNCONDITIONAL_ROBOT_FACE)
+    .join(SLACK_FEEDBACK_DISPATCH_INSTRUCTIONS);
+  next = next
+    .split(RETIRED_CLAIMED_SKIP_VIA_CLEAR_BUG_FALSE)
+    .join(SLACK_FEEDBACK_DISPATCH_INSTRUCTIONS);
+  next = next
+    .split(RETIRED_CLAIMED_SKIP_OMITS_CLEAR_BUG)
+    .join(SLACK_FEEDBACK_DISPATCH_INSTRUCTIONS);
   return `${next.replace(/\n{3,}/g, "\n\n").trimEnd()}\n`;
 }

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

@@ -99,8 +99,11 @@ manual. Clips, Design, and Content are fully owner-managed: never react, tag
 Builder, auto-approve, or auto-merge those items. Slack clear bugs use the
 thread-preserving dispatch-factory-item flow; GitHub issues and Sentry clear
 bugs tag @builderio-bot on a GitHub issue. Slack repeat reports must be clustered
-by underlying symptom, with one Builder thread for the cluster. After classifying an item, call dispatch-factory-item with
-clearBug true or false and a short reason so a skip is recorded. Pass reaction when the prompt names one. Do not post
+by underlying symptom, with one Builder thread for the cluster. After classifying an item, call dispatch-factory-item so the skip or
+dispatch is recorded. If a Slack parent already has eyes or robot_face, pass
+alreadyClaimed true (clearBug may be omitted or false) so started work is not
+rewritten to needs_manual. Otherwise pass clearBug true or false and a short reason.
+Pass reaction only when the prompt says to mark that item; omit it on skips. Do not post
 Slack messages, reactions, or @mentions yourself; dispatch-factory-item owns
 the Builder ping. Use /address-feedback for the repository feedback workflow.
 For pull requests, follow review-prs: read the complete diff and review

package/corpus/templates/factory/server/plugins/factory-scheduler-job.ts

@@ -56,7 +56,10 @@ import {
   BABYSIT_SCOPE_INSTRUCTION,
   repairPrBabysitPrompt,
 } from "../lib/pr-babysit-prompt.js";
-import { repairSlackFeedbackPrompt } from "../lib/slack-feedback-prompt.js";
+import {
+  repairSlackFeedbackPrompt,
+  SLACK_FEEDBACK_DISPATCH_INSTRUCTIONS,
+} from "../lib/slack-feedback-prompt.js";
 import {
   syncManagedReviewSkillAlignment,
   type FactoryAutomationName,
@@ -225,9 +228,7 @@ enough evidence to investigate — including visual/UI defects such as a
 duplicate control or broken layout. Feature requests, vague questions, and
 incomplete threads are not.
 
-For each item, call dispatch-factory-item with clearBug true or false,
-productUxImplications false unless it is a pure product or design decision
-with no single correct fix, a short reason, and reaction robot_face 🤖.
+${SLACK_FEEDBACK_DISPATCH_INSTRUCTIONS}
 Cluster only items listed in this run: one dispatch with relatedItemIds. Do
 not dispatch needs_manual items or items that already started.
 

package/corpus/templates/factory/server/triage/review-skill-alignment.ts

@@ -21,10 +21,14 @@ contract is evidence-first and reply-producing:
   state.
 - Check the existing Slack reaction marker and owner before any write.
   Preserve an existing marker; if reactions cannot be read, do not guess or
-  add one. Do not react to or dispatch Design UX/interaction work (Sid) or
-  any Content work (Alice); record the owner instead.
-- For an actionable repo-owned Slack item with no existing marker, pass
-  \`reaction: robot_face\` 🤖 on \`dispatch-factory-item\`. Every parent this run
+  add one. If the parent already has eyes 👀 or robot_face 🤖, call
+  \`dispatch-factory-item\` with \`alreadyClaimed: true\` (\`clearBug\` may be
+  omitted or \`false\`), omit reaction, and do not start Builder work. Do not
+  react to or dispatch Design UX/interaction
+  work (Sid) or any Content work (Alice); record the owner instead.
+- For an actionable repo-owned Slack item with no existing eyes 👀 or
+  robot_face 🤖 marker, pass \`reaction: robot_face\` 🤖 on
+  \`dispatch-factory-item\`. Do not pass reaction eyes. Every parent this run
   marks must later receive a verified @agent-native Fixed, In progress, or
   Clarification needed reply; a reaction, forward, generic acknowledgement,
   or another person's reply is not a disposition. Group only genuinely
@@ -37,9 +41,10 @@ contract is evidence-first and reply-producing:
   separate. Do not claim a fix, PR, reply, or deployment without confirmation
   from the relevant action or runtime evidence.
 
-After classifying every processed item, call \`dispatch-factory-item\` with
-\`clearBug: true\` or \`false\` and a concise evidence-grounded reason so every
-skip or dispatch is recorded.`;
+After classifying every processed item, call \`dispatch-factory-item\` so every
+skip or dispatch is recorded: \`alreadyClaimed: true\` (\`clearBug\` may be
+omitted or \`false\`) when the parent already has eyes or robot_face, otherwise
+\`clearBug: true\` or \`false\` with a concise evidence-grounded reason.`;
 
 const PR_ALIGNMENT = `## Current review-prs contract
 

package/corpus/templates/factory/server/triage/slack-review-window.ts

@@ -0,0 +1,39 @@
+export function isClaimedSlackReactionName(
+  name: string | null | undefined,
+): boolean {
+  const normalized = name?.trim().toLowerCase();
+  return normalized === "eyes" || normalized === "robot_face";
+}
+
+export function dispatchSkipPreservesItemStatus(status: string): boolean {
+  return status === "automation_started" || status === "evidence_ready";
+}
+
+export function slackFeedbackLeavesReviewWindow(input: {
+  status: string;
+  slackReactionName?: string | null;
+}): boolean {
+  return (
+    dispatchSkipPreservesItemStatus(input.status) ||
+    isClaimedSlackReactionName(input.slackReactionName)
+  );
+}
+
+export function dispatchSkipStatusWrite(status: string): {
+  nextStatus: "needs_manual" | null;
+  needsManual: boolean;
+  statusPreserved: boolean;
+} {
+  if (dispatchSkipPreservesItemStatus(status)) {
+    return {
+      nextStatus: null,
+      needsManual: false,
+      statusPreserved: true,
+    };
+  }
+  return {
+    nextStatus: "needs_manual",
+    needsManual: true,
+    statusPreserved: false,
+  };
+}

package/dist/client/CommandMenu.js

@@ -224,6 +224,15 @@ export function CommandMenu({ open, onOpenChange, children, renderResults, place
             if (!React.isValidElement(child))
                 return child;
             const props = child.props;
+            if (child.type === React.Fragment) {
+                const fragmentChildren = filterChildren(props.children);
+                if (React.Children.count(fragmentChildren) === 0)
+                    return null;
+                return React.cloneElement(child, {
+                    ...props,
+                    children: fragmentChildren,
+                });
+            }
             // If it's a CommandGroup, filter its children
             if (child.type === CommandGroup) {
                 const groupChildren = filterChildren(props.children);
@@ -324,6 +333,10 @@ export function useCommandMenuShortcut(onOpen, options = {}) {
     useEffect(() => {
         const handleKeyDown = (e) => {
             if ((e.metaKey || e.ctrlKey) && e.key.toLowerCase() === "k") {
+                // Claim the shortcut before checking the focused element so an outer
+                // host cannot open its own command menu while this one is focused.
+                e.preventDefault();
+                e.stopPropagation();
                 // Don't trigger if user is typing in a native form control.
                 const target = e.target instanceof HTMLElement ? e.target : null;
                 const isContentEditable = target?.isContentEditable;
@@ -333,7 +346,6 @@ export function useCommandMenuShortcut(onOpen, options = {}) {
                     (!options.allowContentEditable && isContentEditable)) {
                     return;
                 }
-                e.preventDefault();
                 onOpen();
             }
         };

package/dist/client/analytics.js

@@ -7,6 +7,7 @@ import { toPostHogExceptionProperties } from "../tracking/posthog-exception.js";
 import { getAnalyticsClientPlatform } from "./analytics-platform.js";
 import { getOrCreateAnalyticsAnonymousId, getOrCreateAnalyticsSessionId, } from "./analytics-session.js";
 import { injectedAgentNativeConfig } from "./app-config.js";
+import { clientBuildId } from "./build-compatibility.js";
 export { clearAnalyticsSessionId, setAnalyticsSessionId, } from "./analytics-session.js";
 import { fetchAgentEngineStatus, fetchAuthSessionStatus, } from "./client-status-requests.js";
 import { installErrorCapture, } from "./error-capture.js";
@@ -721,6 +722,14 @@ function resolveClientDeploymentEnvironment() {
         env.MODE ||
         "production");
 }
+/**
+ * Must match `resolveSentryClientRelease()` in `vite/sentry-source-maps.ts`
+ * exactly — that's the release name uploaded source maps are attached to, so
+ * a mismatch here means captured events never resolve against them.
+ */
+function resolveClientRelease() {
+    return `agent-native-client@${clientBuildId() || "development"}`;
+}
 function captureWithSentry(module, error, context) {
     return module.withScope((scope) => {
         if (context.tags) {
@@ -764,6 +773,7 @@ function ensureSentry(loadWithoutDsn = false) {
         module.init({
             dsn,
             environment: resolveClientDeploymentEnvironment(),
+            release: resolveClientRelease(),
             beforeSend(event) {
                 if (isSyntheticBrowserTraffic())
                     return null;

package/dist/collab/routes.d.ts

@@ -26,8 +26,8 @@ export declare const getCollabState: import("h3").EventHandlerWithFetch<import("
  * Body: { update: string (base64), requestSource?: string }
  */
 export declare const postCollabUpdate: import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
-    error: string;
     ok?: undefined;
+    error: string;
 } | {
     error?: undefined;
     ok: boolean;

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;