@agent-native/core 0.152.0 → 0.153.0

package/corpus/templates/brain/server/lib/connectors.ts

@@ -1020,14 +1020,18 @@ async function resolveSlackChannel(
       token,
       "conversations.info",
       { channel: channelRef },
+      { toleratedErrors: ["channel_not_found", "not_in_channel"] },
     );
     return data.channel ?? null;
   }
   const byName = await resolveSlackChannelByName(token, channelRef);
   if (!byName) return null;
-  const data = await slackApi<SlackInfoResponse>(token, "conversations.info", {
-    channel: byName.id,
-  });
+  const data = await slackApi<SlackInfoResponse>(
+    token,
+    "conversations.info",
+    { channel: byName.id },
+    { toleratedErrors: ["channel_not_found", "not_in_channel"] },
+  );
   return data.channel ?? byName;
 }
 
@@ -1252,7 +1256,7 @@ export async function testSlackConnection(
   }
 
   return {
-    ok: true,
+    ok: channels.every((channel) => channel.status === "ok"),
     team: auth.team ?? null,
     teamId: auth.team_id ?? null,
     workspaceUrl: auth.url ?? null,
@@ -1306,6 +1310,16 @@ function slackPilotNextSteps(report: {
       "Run the pilot again before attempting any history sync.",
     ];
   }
+  if (
+    report.channelValidation.excluded > 0 ||
+    report.channelValidation.missing > 0 ||
+    report.channelValidation.skipped > 0
+  ) {
+    return [
+      "Fix every invalid Slack channel reference before syncing this source.",
+      "Use channel IDs or enable name resolution, and confirm private-channel membership.",
+    ];
+  }
   if (report.channelValidation.ok === 0) {
     return [
       "Fix the channel allow-list or enable name resolution for channel names.",
@@ -1414,8 +1428,8 @@ export async function runSlackPilot(
     channels,
   };
 
-  if (requestedRefs.length === 0 || !options.readHistory) {
-    const blocked = requestedRefs.length === 0;
+  if (requestedRefs.length === 0 || !credential.ok || !options.readHistory) {
+    const blocked = requestedRefs.length === 0 || !credential.ok;
     const partial = {
       status: blocked ? ("blocked" as const) : ("validated" as const),
       historyRead: false,

package/corpus/templates/brain/server/lib/search.ts

@@ -303,6 +303,9 @@ export function scoreSearchText(
     title?: string | null;
     summary?: string | null;
     body?: string | null;
+    topic?: string | null;
+    tags?: string | null;
+    entities?: string | null;
     provider?: string | null;
     status?: string | null;
   },
@@ -313,7 +316,7 @@ export function scoreSearchText(
       title: fields.title,
       summary: fields.summary,
       body: fields.body,
-      metadata: `${fields.provider ?? ""} ${fields.status ?? ""}`,
+      metadata: `${fields.topic ?? ""} ${fields.tags ?? ""} ${fields.entities ?? ""} ${fields.provider ?? ""} ${fields.status ?? ""}`,
     },
     terms,
   );
@@ -398,6 +401,8 @@ async function searchKnowledgeResults(
             schema.brainKnowledge.summary,
             schema.brainKnowledge.body,
             schema.brainKnowledge.topic,
+            schema.brainKnowledge.tagsJson,
+            schema.brainKnowledge.entitiesJson,
           ],
           terms,
         ),
@@ -417,6 +422,9 @@ async function searchKnowledgeResults(
           title: row.title,
           summary: row.summary,
           body: row.body,
+          topic: row.topic,
+          tags: row.tagsJson,
+          entities: row.entitiesJson,
           status: row.status,
         },
         terms,

package/corpus/templates/brain/server/lib/source-credentials.ts

@@ -2,20 +2,24 @@ import { resolveCredential } from "@agent-native/core/credentials";
 import type { CredentialContext } from "@agent-native/core/credentials";
 import { readAppSecret, type SecretRef } from "@agent-native/core/secrets";
 import {
-  getWorkspaceConnectionAppAccess,
-  listWorkspaceConnectionGrants,
   listWorkspaceConnections,
   resolveWorkspaceConnectionForApp,
   type SerializedWorkspaceConnection,
-  type SerializedWorkspaceConnectionGrant,
   type WorkspaceConnectionAppAccessMode,
   type WorkspaceConnectionCredentialRef,
+  type WorkspaceConnectionForApp,
 } from "@agent-native/core/workspace-connections";
 
 import type { BrainSourceProvider } from "../../shared/types.js";
 
 const APP_ID = "brain";
 
+const SOURCE_CREDENTIAL_KEYS: Record<string, string> = {
+  slack: "SLACK_BOT_TOKEN",
+  granola: "GRANOLA_API_KEY",
+  github: "GITHUB_TOKEN",
+};
+
 interface ResolveSourceCredentialOptions {
   provider: BrainSourceProvider | string;
   key: string;
@@ -76,12 +80,11 @@ function normalizeCredentialKey(key: string) {
   return key.trim().toUpperCase();
 }
 
-function credentialRefsForConnection(
-  connection: SerializedWorkspaceConnection,
-  grants: SerializedWorkspaceConnectionGrant[],
-) {
-  const grant = grants.find((entry) => entry.connectionId === connection.id);
-  return [...(grant?.credentialRefs ?? []), ...connection.credentialRefs];
+function credentialRefsForConnection(connection: WorkspaceConnectionForApp) {
+  return [
+    ...(connection.explicitGrant?.credentialRefs ?? []),
+    ...connection.credentialRefs,
+  ];
 }
 
 function refMatchesKey(ref: WorkspaceConnectionCredentialRef, key: string) {
@@ -166,22 +169,6 @@ async function readFirstSecretCandidate(
   return {};
 }
 
-function connectionStatusMessage(connection: SerializedWorkspaceConnection) {
-  switch (connection.status) {
-    case "checking":
-      return `${connection.label} is still checking health.`;
-    case "needs_reauth":
-      return `${connection.label} needs to be reauthorized before Brain can use it.`;
-    case "error":
-      return `${connection.label} is in an error state before Brain can use it.`;
-    case "disabled":
-      return `${connection.label} is disabled.`;
-    case "connected":
-    default:
-      return `${connection.label} is connected.`;
-  }
-}
-
 function missingCredentialMessage(
   provider: string,
   key: string,
@@ -260,12 +247,12 @@ async function resolveWorkspaceConnectionCredential({
   provenance: SourceCredentialProvenance;
 } | null> {
   let connections: SerializedWorkspaceConnection[] = [];
-  let grants: SerializedWorkspaceConnectionGrant[] = [];
   try {
-    [connections, grants] = await Promise.all([
-      listWorkspaceConnections({ provider, includeDisabled: true }),
-      listWorkspaceConnectionGrants({ provider, appId: APP_ID }),
-    ]);
+    connections = await listWorkspaceConnections({
+      provider,
+      appId: APP_ID,
+      includeDisabled: true,
+    });
   } catch (err) {
     checked.push({
       source: "workspace_connection",
@@ -296,36 +283,50 @@ async function resolveWorkspaceConnectionCredential({
   }
 
   for (const connection of providerConnections) {
-    const access = getWorkspaceConnectionAppAccess(connection, APP_ID, grants);
-    if (!access.available) {
+    let resolved;
+    try {
+      resolved = await resolveWorkspaceConnectionForApp({
+        appId: APP_ID,
+        provider,
+        connectionId: connection.id,
+        includeDisabled: true,
+        requireConnected: true,
+      });
+    } catch (err) {
       checked.push({
         source: "workspace_connection",
         key,
-        status: "not_granted",
-        message: access.reason,
+        status: "error",
+        message: `Workspace connection lookup failed: ${
+          err instanceof Error ? err.message : String(err)
+        }`,
         connectionId: connection.id,
         connectionLabel: connection.label,
-        grantId: access.grantId,
-        appAccessMode: access.mode,
       });
       continue;
     }
 
-    if (connection.status !== "connected") {
+    if (!resolved.available || !resolved.connection || !resolved.appAccess) {
+      const status =
+        resolved.appAccess?.available && resolved.connection
+          ? "unhealthy"
+          : "not_granted";
       checked.push({
         source: "workspace_connection",
         key,
-        status: "unhealthy",
-        message: connectionStatusMessage(connection),
+        status,
+        message: resolved.reason,
         connectionId: connection.id,
         connectionLabel: connection.label,
-        grantId: access.grantId,
-        appAccessMode: access.mode,
+        grantId: resolved.appAccess?.grantId ?? null,
+        appAccessMode: resolved.appAccess?.mode,
       });
       continue;
     }
 
-    const matchingRefs = credentialRefsForConnection(connection, grants).filter(
+    const access = resolved.appAccess;
+    const connectionResult = resolved.connection;
+    const matchingRefs = credentialRefsForConnection(connectionResult).filter(
       (ref) => refMatchesKey(ref, key),
     );
     if (matchingRefs.length === 0) {
@@ -333,12 +334,13 @@ async function resolveWorkspaceConnectionCredential({
         source: "workspace_connection",
         key,
         status: "missing",
-        message: `${connection.label} is granted to Brain but does not reference ${key}.`,
-        connectionId: connection.id,
-        connectionLabel: connection.label,
+        message: `${connectionResult.label} is granted to Brain but does not reference ${key}.`,
+        connectionId: connectionResult.id,
+        connectionLabel: connectionResult.label,
         grantId: access.grantId,
         appAccessMode: access.mode,
       });
+      continue;
     }
 
     for (const ref of matchingRefs) {
@@ -348,9 +350,9 @@ async function resolveWorkspaceConnectionCredential({
           source: "workspace_connection",
           key,
           status: "missing",
-          message: `${connection.label} references ${ref.key}, but its scope is unavailable in this request.`,
-          connectionId: connection.id,
-          connectionLabel: connection.label,
+          message: `${connectionResult.label} references ${ref.key}, but its scope is unavailable in this request.`,
+          connectionId: connectionResult.id,
+          connectionLabel: connectionResult.label,
           grantId: access.grantId,
           appAccessMode: access.mode,
         });
@@ -363,10 +365,10 @@ async function resolveWorkspaceConnectionCredential({
           source: "workspace_connection",
           key,
           status: "available",
-          message: `${key} is available through ${connection.label}.`,
+          message: `${key} is available through ${connectionResult.label}.`,
           scope: found.ref.scope,
-          connectionId: connection.id,
-          connectionLabel: connection.label,
+          connectionId: connectionResult.id,
+          connectionLabel: connectionResult.label,
           grantId: access.grantId,
           appAccessMode: access.mode,
         });
@@ -377,8 +379,8 @@ async function resolveWorkspaceConnectionCredential({
             key,
             provider,
             scope: found.ref.scope,
-            connectionId: connection.id,
-            connectionLabel: connection.label,
+            connectionId: connectionResult.id,
+            connectionLabel: connectionResult.label,
             grantId: access.grantId,
             appAccessMode: access.mode,
             credentialRefLabel: ref.label,
@@ -391,10 +393,10 @@ async function resolveWorkspaceConnectionCredential({
         key,
         status: found.error ? "error" : "missing",
         message: found.error
-          ? `${connection.label} credential lookup failed: ${found.error}`
-          : `${connection.label} references ${ref.key}, but no vault value was found.`,
-        connectionId: connection.id,
-        connectionLabel: connection.label,
+          ? `${connectionResult.label} credential lookup failed: ${found.error}`
+          : `${connectionResult.label} references ${ref.key}, but no vault value was found.`,
+        connectionId: connectionResult.id,
+        connectionLabel: connectionResult.label,
         grantId: access.grantId,
         appAccessMode: access.mode,
       });
@@ -588,3 +590,39 @@ export async function assertSourceWorkspaceConnectionAvailable({
     access: result.appAccess,
   };
 }
+
+/**
+ * Source setup must prove the selected connection can actually supply the
+ * provider credential. Connection access alone is only metadata and can leave
+ * a source saved in a state every later sync will fail to use.
+ */
+export async function assertSourceCredentialAvailable({
+  provider,
+  workspaceConnectionId,
+  ctx,
+}: {
+  provider: BrainSourceProvider | string;
+  workspaceConnectionId: string;
+  ctx: CredentialContext | null;
+}) {
+  const key = SOURCE_CREDENTIAL_KEYS[provider.trim().toLowerCase()];
+  if (!key) return;
+  if (!ctx) {
+    throw new Error(
+      "Source workspace connection setup requires an authenticated credential context.",
+    );
+  }
+
+  const availability = await inspectSourceCredentialAvailability({
+    provider,
+    key,
+    ctx,
+    workspaceConnectionId,
+  });
+  if (!availability.available) {
+    throw new Error(
+      availability.missingMessage ??
+        `The selected ${provider} workspace connection cannot provide ${key}.`,
+    );
+  }
+}

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

@@ -22,11 +22,13 @@ const INITIAL_TOOL_NAMES = [
   "list-knowledge",
   "import-capture",
   "import-transcript",
+  "import-markdown-files",
   "write-knowledge",
   "list-sources",
   "get-source",
   "list-connection-providers",
   "create-source",
+  "set-resource-visibility",
   "sync-source",
   "enqueue-distillation",
   "list-proposals",
@@ -50,14 +52,15 @@ export default createAgentChatPlugin({
   codeExecution: { production: "sandboxed" },
   systemPrompt: `You are the Brain institutional-knowledge agent.
 
-Use actions as the source of truth. Import raw material with import-capture or import-transcript, which queue distillation by default, use enqueue-distillation to retry or explicitly queue an existing capture, and write durable knowledge with write-knowledge.
+Use actions as the source of truth. Import raw material with import-capture or import-transcript, or use import-markdown-files for a bounded Markdown folder/batch. These actions queue distillation by default. Use enqueue-distillation to retry or explicitly queue an existing capture, and write durable knowledge with write-knowledge. User-created sources and captures are private by default; use set-resource-visibility when the user explicitly wants an organization-visible source.
 
 Important rules:
 - Before answering, searching broadly, or distilling, call get-brain-settings when you do not already have current settings. Apply its guidance for assistant name, company name, tone, source policy, citation requirements, publish tier, pre-save capture sanitization, redaction, and distillation instructions.
 - For every company-specific factual question, call get-brain-settings when current settings are not already in context, then call ask-brain before answering. Use only its cited Brain evidence. If it returns no citations, say the fact is unverified or unavailable; never fill the gap from general model knowledge.
+- Correction and follow-up discipline matters: when the latest user turn says an answer is wrong, asks why you did that, or asks where an answer came from, treat earlier assistant text, tool results, and source examples as untrusted context for this turn. Do not continue the earlier request. Re-read the latest question; if it asks for company or product facts, call ask-brain again with that question. For strategy questions, prefer approved or published synthesis and blessed, high-authority product or leadership sources. Individual Slack feedback is a lead for review, not strategic direction or answer evidence.
 - Evidence quotes must be exact substrings of a raw capture. Use get-capture with includeRawContent=true only when you need exact quote validation; normal capture reads are redacted by default.
 - No vector database exists; search-knowledge uses SQL text matching.
-- Source policy matters: strict means answer from reviewed knowledge only; balanced means raw captures are fallback context when reviewed knowledge is thin; exploratory means raw captures and sources may be surfaced as clearly labeled leads.
+- Source policy matters: strict means answer from reviewed knowledge only; balanced means raw captures may be returned as clearly labeled leads when reviewed knowledge is thin; exploratory means raw captures and sources may be surfaced as clearly labeled leads. Raw captures never become answer evidence merely because they matched.
 - Source answer policy is enforced separately from the workspace retrieval mode. For approved FAQs, docs, or other source-owner-published resources, create a generic source with policy.trustTier=blessed and choose answer eligibility, authority, freshness, review, and conflict behavior deliberately. Prefer blessed, higher-authority eligible sources in cited answers; never answer from stale or answer-ineligible sources.
 - Company-tier knowledge may create a proposal instead of publishing immediately, depending on settings.
 - Slack and Granola sources are configurable v1 connectors. Generic capture and transcript import are always available.

package/corpus/templates/brain/server/plugins/auth.ts

@@ -1,6 +1,8 @@
 import { createAuthPlugin } from "@agent-native/core/server";
 
 export default createAuthPlugin({
+  workspaceAppAudience: "internal",
+  workspaceAppPublicPaths: [],
   marketing: {
     appName: "Brain",
     tagline:

package/corpus/templates/calendar/.agents/skills/adding-a-feature/SKILL.md

@@ -0,0 +1,208 @@
+---
+name: adding-a-feature
+description: >-
+  The four-area checklist every new feature must complete. Use when adding any
+  feature, integration, or capability to ensure the agent and UI stay in parity.
+scope: dev
+metadata:
+  internal: true
+---
+
+# Adding a Feature — The Four-Area Checklist
+
+## Rule
+
+Every new feature MUST update all four areas. Skipping any one breaks the agent-native contract — the agent and UI must always be equal partners.
+
+Scope check first: this is the checklist for a feature that adds app data or a new capability. A restyle, a copy change, or a screen with no new data model and no new agent-visible operation only needs the areas it actually touches — do not manufacture actions, state keys, or a skill for it.
+
+## Why
+
+Agent-native apps are defined by parity: everything the UI can do, the agent can do, and vice versa. A feature that only has UI is invisible to the agent. A feature that only has scripts is invisible to the user. A feature without app-state sync means the agent is blind to what the user is doing.
+
+## The Checklist
+
+When you add a new feature, work through these four areas in order:
+
+### 1. UI Component
+
+Build the user-facing interface — a page, component, dialog, or route. Use `useActionQuery` and `useActionMutation` from `@agent-native/core/client` to call actions for data fetching and mutations. Do not create a custom REST endpoint just so React can call action-backed data; the action endpoint already exists.
+
+**Auto-refresh on agent writes is non-negotiable** — when the agent mutates data, the UI must reflect the change without a manual refresh. There are two paths, and you must pick the right one:
+
+- **`useActionQuery` / `useActionMutation`** — covered automatically. The framework's `useDbSync` invalidates `["action"]` on every change event, so every `useActionQuery` hook refetches on agent activity. No extra wiring required. **Prefer this path.**
+- **Raw `useQuery` with custom keys** — needs explicit wiring. Fold `useChangeVersions([<source>, "action"])` from `@agent-native/core/client` into the `queryKey` and set `placeholderData: (prev) => prev`. The `action` source is the reliable signal (the agent runner emits it after every successful tool call); the resource-specific source (`"dashboards"`, `"analyses"`, `"settings"`, etc.) is bonus when emitted. Without this wiring, agent writes will be invisible until manual refresh — that breaks the framework's #1 promise.
+
+  ```tsx
+  import { useChangeVersions } from "@agent-native/core/client/hooks";
+  import { useQuery } from "@tanstack/react-query";
+
+  const v = useChangeVersions(["dashboards", "action"]);
+  useQuery({
+    queryKey: ["dashboard", id, v],
+    queryFn: () => fetchDashboard(id),
+    placeholderData: (prev) => prev, // no flicker on refetch
+  });
+  ```
+
+  See the `real-time-sync` skill for the full pattern and source catalog.
+
+### 2. Action
+
+Create an action in `actions/` using `defineAction`. This serves double duty: the agent calls it as a tool, and the UI calls it through `useActionQuery` / `useActionMutation` while the framework owns the HTTP transport. Set `http: { method: "GET" }` for read actions, leave default for writes, or set `http: false` for agent-only actions like `navigate` and `view-screen`.
+
+Before adding a new route or endpoint, inspect the existing actions. Reuse an
+action if it already covers the business operation, extend it if the shared
+contract is incomplete, or create a new `defineAction` if the agent and UI both
+need the capability. Do not add pass-through `/api/*` routes that re-export
+actions. If client code needs a new framework/app route, expose a named helper
+or hook first and use that helper from components and docs.
+
+For provider-backed analysis/query/reporting integrations, do not turn every
+provider endpoint or filter into a rigid action. Prefer the shared
+`provider-api-catalog` / `provider-api-docs` / `provider-api-request` pattern
+from `@agent-native/core/provider-api`, then add narrow convenience actions only
+for workflows that truly deserve a first-class shortcut. Treat this as a
+capability requirement, not a nice-to-have: convenience actions must not become
+the ceiling of what the agent can ask the provider to do. Pair broad provider
+access with staging or sandboxed code execution when responses may be too large
+for chat context. If provider credentials live on resource/share rows, add the
+scoped resolver first so broad access preserves the same ownership boundary as
+the app UI.
+
+If the feature needs credentials, design the credential path in the same change.
+Never hardcode API keys, tokens, webhook URLs, signing secrets, private
+Builder/internal data, or customer data in the action, UI, seed data, fixtures,
+docs, prompts, or generated extension/app content. Register required secrets,
+use OAuth helpers, or read scoped values from the vault/credential store.
+
+If the feature involves attachments, images, recordings, screenshots, exports,
+or other file-like payloads, design the upload path in the same change:
+provider upload first, then URL/id/blob handle in SQL. Do not add base64/binary
+columns or stuff files into `application_state`.
+
+If the feature adds a server-side dependency carrying a native binary or heavy
+runtime — headless browser, ffmpeg, image/video processing, ML runtime — read
+the `performance` skill §9 before adding it. A deployed app has one `/*` page
+function that every visitor's cache miss wakes, and it ships whatever the server
+bundle depends on. Run that work from a background function or a job so the page
+function never carries the weight.
+
+**If the action produces or lists a navigable resource**, add a `link` builder that returns `{ url: buildDeepLink({ app, view, params }), label }`. External coding agents and MCP hosts (Claude / ChatGPT / Claude Code / Cowork / Codex, over MCP/A2A) then surface an "Open in … →" deep link that drops the user back into the running UI focused on the record — for free. If a compatible MCP host should render an inline review/edit surface, also add `mcpApp` with `embedApp()` so the action embeds the real React app route instead of a one-off HTML UI. The `link` builder and `mcpApp` metadata must be pure and synchronous (no I/O). Any external-agent read/ingest action must be `http: { method: "GET" }` + `readOnly: true` + `publicAgent: { expose: true, readOnly: true, requiresAuth: true }`. See the `external-agents` skill.
+
+### 3. Skills / Instructions
+
+Update `AGENTS.md` and/or create a skill in `.agents/skills/` if the feature introduces patterns the agent needs to know. At minimum, add the new actions to the action table in the template's `AGENTS.md`.
+
+Reusable actions are part of the app contract, not just implementation detail. When an action is useful outside one screen, update agent instructions in the same change so app agents know when to call it, which arguments matter, and what output to preserve. If the capability is workflow-heavy, cross-app, provider-backed, or has a non-obvious sequence of actions, add or update a skill instead of burying the behavior in one long `AGENTS.md` paragraph.
+
+Instruction examples may name secret keys like `SLACK_WEBHOOK`, but must use
+placeholders such as `${keys.SLACK_WEBHOOK}` or `<SLACK_WEBHOOK>`. Do not paste
+real keys, internal data, or customer data into instructions as examples.
+
+If the feature adds or changes visible UI copy, prompts, toasts, labels, empty
+states, or formatting, update the English source copy. Read the optional
+`internationalization` skill and update additional catalogs only when
+`translations.locales` in `agent-native.config.ts` includes them.
+
+For app-backed skills, declare skill visibility in the app-skill manifest:
+
+- `internal` — only the app's own agents should use it.
+- `exported` — marketplace installs receive it, but the app does not need it loaded internally.
+- `both` — shared between the app's internal agents and exported marketplace bundles.
+
+### 4. Application State Sync
+
+Expose navigation and selection state so the agent knows what the user is looking at. Write to the `navigation` app-state key on route changes. Update the `view-screen` action to fetch relevant data for the new feature. Add a `navigate` command if the agent needs to open the new view.
+
+## Examples
+
+### Adding "compose email" to a mail app
+
+| Area            | What to build                                                                            |
+| --------------- | ---------------------------------------------------------------------------------------- |
+| UI              | Compose panel with tabs, to/cc/bcc fields, body editor. Use `useActionQuery`/`useActionMutation` for data. |
+| Action          | `manage-draft` action (create/update/delete drafts), `send-email` action                 |
+| Skills/AGENTS   | Document compose state shape, draft lifecycle, action args in AGENTS.md                  |
+| App-state sync  | `compose-{id}` keys for each draft tab, `navigation` includes compose state              |
+
+### Adding "create form" to a forms app
+
+| Area            | What to build                                                                            |
+| --------------- | ---------------------------------------------------------------------------------------- |
+| UI              | Form builder page with drag-and-drop fields, preview, settings. Use `useActionQuery` for lists. |
+| Action          | `create-form` action, `update-form` action, `list-forms` action (GET)                    |
+| Skills/AGENTS   | Document form schema shape, field types, validation rules in AGENTS.md                   |
+| App-state sync  | `navigation` includes `{ view: "form-builder", formId: "..." }`, `view-screen` fetches form data |
+
+### Adding "chart type" to an analytics app
+
+| Area            | What to build                                                                            |
+| --------------- | ---------------------------------------------------------------------------------------- |
+| UI              | New chart component, chart type selector in dashboard                                    |
+| Action          | `create-chart` or `update-dashboard` action that sets chart type and config              |
+| Skills/AGENTS   | Document supported chart types, config options, data requirements                        |
+| App-state sync  | `navigation` includes selected chart/dashboard, `view-screen` returns chart config       |
+
+## Adding a new route
+
+Templates are single-page apps with client-side routing. The app shell (AgentSidebar + top-level nav) MUST persist across navigation — it is mounted once, either in `root.tsx` around `<Outlet />` or via a pathless `_app.tsx` layout route that all authed routes nest under.
+
+**Never wrap each new route in its own `<AppLayout>` / `<Layout>`.** That causes React to unmount the entire app shell on every navigation, reloading the agent sidebar and destroying in-progress work.
+
+- If the template has `<AppLayout>` in `root.tsx` — just render page content in your new route file, nothing else.
+- If the template has `app/routes/_app.tsx` (pathless layout) — name your new route `_app.<segment>.tsx` to inherit the shell, or bare `<segment>.tsx` for public routes that should NOT have the shell.
+- If a page needs per-route data (e.g. highlighting the active item in the sidebar), read it in the layout from `useParams()` / `useLocation()`. Don't pass it as a prop through every route file.
+
+See the "Client-Side Routing" section in the root `CLAUDE.md` for full details.
+
+## Anti-Patterns
+
+- **Per-route `<AppLayout>` wrappers** — Every route file wraps its content in `<AppLayout>` or `<Layout>`. React sees a different component at the outlet on each nav and unmounts the whole shell, causing the agent sidebar to reload on every click. Mount the shell once above `<Outlet />` (root.tsx or `_app.tsx` pathless layout).
+- **UI without actions** — The user can create forms but the agent cannot. The agent says "I don't have access to that" when it should be able to do it.
+- **Actions without AGENTS.md** — The actions exist but the agent doesn't know about them because they're not documented. The agent reinvents solutions instead of using the actions.
+- **Duplicate API routes** — Creating `/api/` routes for operations that actions already handle, including pass-through routes that just call or repackage an action. Use `useActionQuery`/`useActionMutation` instead.
+- **Raw client route calls** — Teaching or adding `fetch("/_agent-native/...")`,
+  `fetch(agentNativePath(...))`, or template `/api/*` calls in components for
+  normal app work. Add a named client helper/hook and call that instead.
+- **Features without app-state** — The agent cannot see that the user is looking at a specific form, email, or chart. It asks "which one?" instead of acting on the current selection.
+- **Actions without UI** — The agent can do something the user cannot. This is less common but still breaks parity.
+
+## Verification
+
+After completing all four areas, verify:
+
+1. Can the user perform the operation from the UI?
+2. Can the agent perform the same operation via actions?
+3. Does `pnpm action view-screen` show the relevant state when the user is using the feature?
+4. Can the agent navigate to the feature view via the `navigate` action?
+5. Is the feature documented in AGENTS.md with action names and args?
+6. Are credentials and sensitive data supplied only through approved runtime
+   channels, with no hardcoded real keys, tokens, webhook URLs, Builder/internal
+   data, or customer data?
+
+## One more area — sharing
+
+If the feature stores **user-authored resources** (documents, dashboards, forms, decks, etc.), make them ownable so they get private-by-default semantics and a share dialog for free. See the `sharing` skill.
+
+TL;DR: spread `ownableColumns()` into the resource table, pair it with `createSharesTable(...)`, call `registerShareableResource(...)`, wrap list/read queries with `accessFilter`, guard writes with `assertAccess`, and drop `<ShareButton>` in the resource header. The `share-resource`, `unshare-resource`, `list-resource-shares`, and `set-resource-visibility` actions are auto-mounted framework-wide.
+
+## One more area — who inside the app may do it
+
+If the feature is one **only some teammates should be able to perform inside this app** (an admin-only import, a settings reset), that is a per-app role — not a new organization, and not a share grant. See the `authentication` skill.
+
+TL;DR: declare the vocabulary once with `defineAppRoles({ appId, roles, defaultRole })`, guard the action with `authorize: appAccess.requireAny("...")`, and render the picker with `<TeamPage appRoles={descriptor} />`. `defaultRole` is display only — it never satisfies a guard — and `authorize` gates the operation while `accessFilter` / `assertAccess` still scope the rows.
+
+## Related Skills
+
+- **authentication** — Per-app member roles (`defineAppRoles`) when only some teammates may use a feature
+- **sharing** — How to make a new resource ownable (private by default, share with users/orgs/public)
+- **context-awareness** — How to expose UI state to the agent (area 4 in detail)
+- **actions** — How to create actions with `defineAction` and the `http` option (area 2 in detail)
+- **external-agents** — Add a `link` builder so external agents (MCP/A2A) get an "Open in … →" deep link
+- **create-skill** — How to create skills for new patterns (area 3 in detail)
+- **storing-data** — Where to store the feature's data
+- **real-time-sync** — How the UI stays in sync when the agent writes data
+- **performance** — Query/load cost, and (§9) cold-start artifact size when a
+  feature adds a server-side dependency
+- **internationalization** — How to update localized UI copy and catalogs