@agent-native/core 0.185.0-nightly-20260923150956 → 0.185.0

package/corpus/templates/slides/actions/apply-design-system.ts

@@ -5,14 +5,11 @@ import { z } from "zod";
 
 import { getDb, schema } from "../server/db/index.js";
 import { notifyClients } from "../server/handlers/decks.js";
-import { parseDesignSystemIndexingStatus } from "../shared/design-system-validation.js";
 
 export default defineAction({
   description:
     "Link a design system to a deck. The deck will use this design system's " +
-    "colors, typography, and styling. Requires editor access on the deck. " +
-    "The design system must have finished Builder indexing (list-design-systems " +
-    "reports indexingStatus 'ready'); a still-indexing or unavailable system is rejected.",
+    "colors, typography, and styling. Requires editor access on the deck.",
   schema: z.object({
     deckId: z.string().describe("Deck ID to apply the design system to"),
     designSystemId: z.string().describe("Design system ID to link to the deck"),
@@ -20,24 +17,7 @@ export default defineAction({
   run: async ({ deckId, designSystemId }) => {
     // Verify access to both the deck and the design system
     await assertAccess("deck", deckId, "editor");
-    const designSystemAccess = await assertAccess(
-      "design-system",
-      designSystemId,
-      "viewer",
-    );
-    const indexingStatus = parseDesignSystemIndexingStatus(
-      designSystemAccess.resource.data,
-    );
-    if (indexingStatus !== "ready") {
-      throw Object.assign(
-        new Error(
-          indexingStatus === "indexing"
-            ? `Design system ${designSystemId} is still indexing and has no usable tokens/components yet. Wait for indexing to finish, or use a different design system.`
-            : `Design system ${designSystemId} is unavailable (indexing failed or its data could not be read). Choose a different design system.`,
-        ),
-        { statusCode: 409 },
-      );
-    }
+    await assertAccess("design-system", designSystemId, "viewer");
 
     const db = getDb();
     const now = new Date().toISOString();

package/corpus/templates/slides/actions/create-deck.ts

@@ -30,7 +30,6 @@ import {
   assertHumanReadableDeckTitle,
   repairGeneratedDeckTitle,
 } from "../shared/deck-title.js";
-import { parseDesignSystemIndexingStatus } from "../shared/design-system-validation.js";
 import {
   ensureUniqueSlideIds,
   rebindCreativeContextSlideLabels,
@@ -117,25 +116,6 @@ function deckNavigationCommand(deckId: string): Record<string, string> {
   };
 }
 
-/**
- * `list-design-systems` tells agents not to pass a non-'ready' id here, but an
- * agent can still race a status change or ignore that guidance — this is the
- * boundary that actually owns linking a design system, so it re-checks rather
- * than trusting the caller.
- */
-function assertDesignSystemReady(designSystemId: string, data: string): void {
-  const status = parseDesignSystemIndexingStatus(data);
-  if (status === "ready") return;
-  throw Object.assign(
-    new Error(
-      status === "indexing"
-        ? `Design system ${designSystemId} is still indexing and has no usable tokens/components yet. Wait for indexing to finish, or use a different design system.`
-        : `Design system ${designSystemId} is unavailable (indexing failed or its data could not be read). Choose a different design system.`,
-    ),
-    { statusCode: 409 },
-  );
-}
-
 export default defineAction({
   title: "Create Slides deck",
   description:
@@ -310,15 +290,7 @@ export default defineAction({
 
     if (deckId) {
       if (designSystemId) {
-        const designSystemAccess = await assertAccess(
-          "design-system",
-          designSystemId,
-          "viewer",
-        );
-        assertDesignSystemReady(
-          designSystemId,
-          designSystemAccess.resource.data,
-        );
+        await assertAccess("design-system", designSystemId, "viewer");
       }
       // Update existing deck — requires editor access.
       await assertAccess("deck", deckId, "editor");
@@ -426,32 +398,10 @@ export default defineAction({
 
     let resolvedDesignSystemId = designSystemId;
     if (resolvedDesignSystemId) {
-      const designSystemAccess = await assertAccess(
-        "design-system",
-        resolvedDesignSystemId,
-        "viewer",
-      );
-      assertDesignSystemReady(
-        resolvedDesignSystemId,
-        designSystemAccess.resource.data,
-      );
+      await assertAccess("design-system", resolvedDesignSystemId, "viewer");
     } else {
-      const candidateDefaultId = await resolveDefaultDesignSystemId(ownerEmail);
-      if (candidateDefaultId) {
-        // An implicit default is a convenience, not an explicit request —
-        // fall back to no design system instead of failing deck creation
-        // outright when the caller's default happens to still be indexing.
-        const [defaultRow] = await db
-          .select({ data: schema.designSystems.data })
-          .from(schema.designSystems)
-          .where(eq(schema.designSystems.id, candidateDefaultId))
-          .limit(1);
-        resolvedDesignSystemId =
-          defaultRow &&
-          parseDesignSystemIndexingStatus(defaultRow.data) === "ready"
-            ? candidateDefaultId
-            : undefined;
-      }
+      resolvedDesignSystemId =
+        (await resolveDefaultDesignSystemId(ownerEmail)) ?? undefined;
     }
 
     const id = `deck-${Date.now()}-${Math.random().toString(36).slice(2, 7)}`;

package/corpus/templates/slides/actions/get-design-system.ts

@@ -4,9 +4,10 @@ import {
   parseBuilderDesignSystemProxyReference,
 } from "@agent-native/core/server";
 import { resolveAccess } from "@agent-native/core/sharing";
+import { and, eq } from "drizzle-orm";
 import { z } from "zod";
 
-import "../server/db/index.js"; // ensure registerShareableResource runs
+import { getDb, schema } from "../server/db/index.js";
 
 const MAX_AGENT_CONTEXT_CHARS = 14_000;
 const MAX_JSON_CONTEXT_CHARS = 2_500;
@@ -32,7 +33,8 @@ interface BuilderGenerationContext {
     tokenValues?: Record<string, string>;
   }>;
   tokenValues: Record<string, string>;
-  docCount: number;
+  /** null when Builder could not be read at all; 0 means still indexing. */
+  docCount: number | null;
   warning?: string;
 }
 
@@ -41,6 +43,37 @@ function truncate(value: string, maxChars: number): string {
   return `${value.slice(0, maxChars).trimEnd()}\n[truncated]`;
 }
 
+// list-design-systems only ever reads the docCount baked into row.data at
+// proxy creation time (always absent) - this is the one place that hydrates
+// a live count from Builder, so it is the one place that can refresh the
+// cache. Compare-and-set against the exact data snapshot this request read
+// avoids clobbering a concurrent sync/update; a lost race just leaves the
+// next call to persist it. Never bumps updatedAt, so a background count
+// refresh doesn't reorder the list.
+async function persistBuilderDocCount(
+  row: { id: string; data: string | null },
+  docCount: number,
+): Promise<void> {
+  if (!row.data) return;
+  let parsed: Record<string, unknown>;
+  try {
+    parsed = JSON.parse(row.data);
+  } catch {
+    return;
+  }
+  if (parsed.docCount === docCount) return;
+  const db = getDb();
+  await db
+    .update(schema.designSystems)
+    .set({ data: JSON.stringify({ ...parsed, docCount }) })
+    .where(
+      and(
+        eq(schema.designSystems.id, row.id),
+        eq(schema.designSystems.data, row.data),
+      ),
+    );
+}
+
 function parseJson(value: string | null | undefined): unknown {
   if (!value) return null;
   try {
@@ -282,7 +315,7 @@ export default defineAction({
             ...builderReference,
             docs: [],
             tokenValues: {},
-            docCount: 0,
+            docCount: null,
             warning:
               error instanceof Error
                 ? error.message
@@ -291,6 +324,10 @@ export default defineAction({
         )
       : null;
 
+    if (builder && typeof builder.docCount === "number") {
+      await persistBuilderDocCount(row, builder.docCount);
+    }
+
     return {
       id: row.id,
       title: row.title,

package/corpus/templates/slides/actions/list-design-systems.ts

@@ -13,7 +13,6 @@ import { z } from "zod";
 
 import { getDb, schema } from "../server/db/index.js";
 import { resolveDefaultDesignSystemId } from "../server/workspace-defaults.js";
-import { parseDesignSystemIndexingStatus } from "../shared/design-system-validation.js";
 
 type EffectiveRole = "owner" | ShareRole;
 
@@ -35,16 +34,32 @@ function strongerRole(current: ShareRole | null, next: ShareRole): ShareRole {
   return current;
 }
 
+/**
+ * Builder-reported indexed document count cached on the row. Undefined means
+ * "not measured yet", which is not the same as a system with zero documents.
+ */
+function cachedBuilderDocCount(data: string | null): number | undefined {
+  if (!data) return undefined;
+  let parsed: unknown;
+  try {
+    parsed = JSON.parse(data);
+  } catch {
+    // coercion-ok: unparseable row data leaves the count unknown, and
+    // undefined stays distinguishable from a measured zero.
+    return undefined;
+  }
+  if (!parsed || typeof parsed !== "object") return undefined;
+  const docCount = (parsed as Record<string, unknown>).docCount;
+  return typeof docCount === "number" ? docCount : undefined;
+}
+
 export default defineAction({
   description:
     "List all design systems accessible to the current user. Returns title, " +
-    "id, isDefault (true only for the caller's effective default), and " +
-    "indexingStatus ('ready' | 'indexing' | 'unavailable'). Do not pass a " +
-    "non-'ready' id to create-deck or apply-design-system — its tokens and " +
-    "components are not queryable yet; call get-design-system to confirm " +
-    "status if unsure. For a named system, match the exact title and pass " +
-    "its id as designSystemId — or pass the title as `designSystem` on " +
-    "create-deck — then call get-design-system once before authoring.",
+    "id, and isDefault (true only for the caller's effective default). For a " +
+    "named system, match the exact title and pass its id as designSystemId " +
+    "— or pass the title as `designSystem` on create-deck — then call " +
+    "get-design-system once before authoring.",
   schema: z.object({
     compact: z
       .enum(["true", "false"])
@@ -145,8 +160,8 @@ export default defineAction({
         role = "owner";
       }
       const canManage = canManageRole(role);
-      const indexingStatus = parseDesignSystemIndexingStatus(row.data);
 
+      const docCount = cachedBuilderDocCount(row.data);
       if (args.compact === "true") {
         return {
           id: row.id,
@@ -154,7 +169,7 @@ export default defineAction({
           isDefault: row.id === effectiveDefaultId,
           accessRole: role,
           canManage,
-          indexingStatus,
+          docCount,
         };
       }
       return {
@@ -162,13 +177,13 @@ export default defineAction({
         title: row.title,
         description: row.description,
         data: row.data,
+        docCount,
         isDefault: row.id === effectiveDefaultId,
         visibility: row.visibility,
         accessRole: role,
         canManage,
         createdAt: row.createdAt,
         updatedAt: row.updatedAt,
-        indexingStatus,
       };
     });
 

package/corpus/templates/slides/app/components/design-system/DesignSystemSetup.tsx

@@ -97,9 +97,9 @@ interface BuilderSourceDetails {
   builderUrl?: string;
   builderStatus?: string;
   sourceKind?: BuilderSourceKind;
-  docs?: Array<unknown>;
   tokenValues?: Record<string, string>;
-  docCount?: number;
+  /** null when Builder could not be read at all; 0 means still indexing. */
+  docCount?: number | null;
   warning?: string;
   githubSources?: Array<{
     repoUrl: string;
@@ -276,10 +276,11 @@ export function DesignSystemSetup({
     editingId ? { id: editingId } : undefined,
     {
       enabled: !!editingId && open,
+      // Builder's status string lags the real index state; a zero document
+      // count is the only reliable "still indexing" signal. A null count
+      // means Builder could not be read, so stop rather than spin.
       refetchInterval: (query) =>
-        query.state.data?.builder?.builderStatus === "in-progress"
-          ? 5_000
-          : false,
+        query.state.data?.builder?.docCount === 0 ? 5_000 : false,
     },
   );
 
@@ -1544,26 +1545,18 @@ function BuilderSourceStatus({
   syncing?: boolean;
 }) {
   const t = useT();
-  const docs = builder.docCount ?? builder.docs?.length ?? 0;
+  // Builder's status string drifts out of sync with the real index state, so
+  // the reported document count decides: absent means Builder could not be
+  // read at all, zero means indexing, positive means ready.
+  const docCount = builder.docCount;
+  const docs = docCount ?? 0;
   const tokens = Object.keys(builder.tokenValues ?? {}).length;
-  const normalizedStatus = builder.builderStatus?.toLowerCase();
   const hasIndexedResults = docs > 0 || tokens > 0;
-  const isIndexed =
-    hasIndexedResults ||
-    normalizedStatus === "ready" ||
-    normalizedStatus === "complete" ||
-    normalizedStatus === "completed";
-  const isIndexing = ["in-progress", "pending", "processing"].includes(
-    normalizedStatus ?? "",
-  );
-  const state =
-    isIndexing && !isIndexed
-      ? "indexing"
-      : builder.warning
-        ? "unavailable"
-        : isIndexed
-          ? "indexed"
-          : "indexing";
+  const state = hasIndexedResults
+    ? "indexed"
+    : builder.warning
+      ? "unavailable"
+      : "indexing";
   const sourceKind = builder.sourceKind;
   const SourceIcon =
     sourceKind === "figma"

package/corpus/templates/slides/app/components/editor/NewDeckReferenceStep.tsx

@@ -7,7 +7,6 @@ import {
   IconChevronDown,
   IconFileText,
   IconFileTypePdf,
-  IconLoader2,
   IconPresentation,
   IconWorld,
 } from "@tabler/icons-react";
@@ -39,13 +38,9 @@ import {
 } from "@/components/ui/select";
 import type { Deck } from "@/context/DeckContext";
 import { sortDecksByRecency } from "@/lib/deck-sorting";
-import {
-  isDesignSystemSelectable,
-  resolveSelectableDesignSystemId,
-} from "@/lib/design-system-selection";
+import { resolveSelectableDesignSystemId } from "@/lib/design-system-selection";
 import { cn } from "@/lib/utils";
 
-import type { DesignSystemIndexingStatus } from "../../../shared/design-system-validation";
 import { GoogleDriveConnectionCta } from "./GoogleDriveConnectionCta";
 export interface NewDeckReferenceSelection {
   designSystemId?: string | null;
@@ -82,7 +77,6 @@ interface DesignSystemOption {
   id: string;
   title: string;
   isDefault?: boolean;
-  indexingStatus?: DesignSystemIndexingStatus;
 }
 
 interface NewDeckReferenceStepProps {
@@ -137,9 +131,7 @@ export function NewDeckReferenceStep({
   const t = useT();
   const [selectedDesignSystemId, setSelectedDesignSystemId] = useState<
     string | null
-  >(() =>
-    resolveSelectableDesignSystemId(designSystems, defaultDesignSystemId),
-  );
+  >(resolveSelectableDesignSystemId(designSystems, defaultDesignSystemId));
   const [selectedReferenceDeckId, setSelectedReferenceDeckId] = useState<
     string | null
   >(defaultReferenceDeckId);
@@ -157,6 +149,9 @@ export function NewDeckReferenceStep({
   const [showDesignSystemSetup, setShowDesignSystemSetup] = useState(false);
   const busy = importing || continuing;
 
+  // True while the picker still reflects an auto-applied default rather than
+  // an explicit user choice, so a default that resolves after this step is
+  // already open can still land - see the hydration effects below.
   const designSystemAutoRef = useRef(true);
   const referenceDeckAutoRef = useRef(true);
 
@@ -165,14 +160,6 @@ export function NewDeckReferenceStep({
   const selectedReferenceDeck = selectedReferenceDeckId
     ? deckById.get(selectedReferenceDeckId)
     : undefined;
-  const selectedDesignSystem = selectedDesignSystemId
-    ? designSystems.find((ds) => ds.id === selectedDesignSystemId)
-    : undefined;
-  // Selecting one from the list below already disables non-ready rows; this
-  // also covers a system that starts re-indexing after it was selected.
-  const selectedDesignSystemUnavailable = Boolean(
-    selectedDesignSystem && !isDesignSystemSelectable(selectedDesignSystem),
-  );
   const hasSelection = Boolean(
     selectedDesignSystemId ||
     selectedReferenceDeckId ||
@@ -183,7 +170,9 @@ export function NewDeckReferenceStep({
     if (!open) return;
     designSystemAutoRef.current = true;
     referenceDeckAutoRef.current = true;
-    setSelectedDesignSystemId(null);
+    setSelectedDesignSystemId(
+      resolveSelectableDesignSystemId(designSystems, defaultDesignSystemId),
+    );
     setSelectedReferenceDeckId(defaultReferenceDeckId);
     setReferenceDeckTouched(defaultReferenceDeckId !== null);
     setImportedReference(null);
@@ -235,7 +224,7 @@ export function NewDeckReferenceStep({
   };
 
   const handleContinue = async () => {
-    if (busy || selectedDesignSystemUnavailable || !hasSelection) return;
+    if (busy || !hasSelection) return;
     const trimmedSource =
       selectedSource && selectedSource.value.trim()
         ? { ...selectedSource, value: selectedSource.value.trim() }
@@ -376,38 +365,13 @@ export function NewDeckReferenceStep({
                 </SelectTrigger>
                 <SelectContent>
                   <SelectItem value="none">{t("home.none")}</SelectItem>
-                  {designSystems.map((designSystem) => {
-                    const selectable = isDesignSystemSelectable(designSystem);
-                    return (
-                      <SelectItem
-                        key={designSystem.id}
-                        value={designSystem.id}
-                        disabled={!selectable}
-                      >
-                        <span className="flex items-center gap-1.5">
-                          {designSystem.title}
-                          {designSystem.indexingStatus === "indexing" && (
-                            <span className="inline-flex items-center gap-1 text-xs text-muted-foreground">
-                              <IconLoader2 className="size-3 animate-spin" />
-                              {t("home.designSystemIndexing")}
-                            </span>
-                          )}
-                          {designSystem.indexingStatus === "unavailable" && (
-                            <span className="text-xs text-muted-foreground">
-                              ({t("home.designSystemUnavailable")})
-                            </span>
-                          )}
-                        </span>
-                      </SelectItem>
-                    );
-                  })}
+                  {designSystems.map((designSystem) => (
+                    <SelectItem key={designSystem.id} value={designSystem.id}>
+                      {designSystem.title}
+                    </SelectItem>
+                  ))}
                 </SelectContent>
               </Select>
-              {selectedDesignSystemUnavailable && (
-                <p className="text-xs text-amber-500">
-                  {t("home.designSystemIndexingNotice")}
-                </p>
-              )}
             </div>
 
             <div className="grid gap-2">
@@ -644,7 +608,6 @@ export function NewDeckReferenceStep({
           aria-busy={busy}
           disabled={
             busy ||
-            selectedDesignSystemUnavailable ||
             !hasSelection ||
             Boolean(selectedSource && !selectedSource.value.trim())
           }