@agent-native/core 0.102.1 → 0.103.0

package/corpus/core/src/client/chat/message-components.tsx

@@ -909,10 +909,24 @@ function ReasoningMessagePart() {
   );
 }
 
+const ALWAYS_VISIBLE_ASSISTANT_TOOLS = new Set(["connect-builder"]);
+
+export function isCollapsibleAssistantWorkPart(part: {
+  type?: string;
+  toolName?: string;
+}): boolean {
+  if (part.type === "reasoning") return true;
+  return (
+    part.type === "tool-call" &&
+    !ALWAYS_VISIBLE_ASSISTANT_TOOLS.has(part.toolName ?? "")
+  );
+}
+
 function groupAssistantWorkParts(part: {
   type?: string;
+  toolName?: string;
 }): ["group-work"] | null {
-  if (part.type === "reasoning" || part.type === "tool-call") {
+  if (isCollapsibleAssistantWorkPart(part)) {
     return ["group-work"];
   }
   return null;
@@ -1044,8 +1058,8 @@ export function AssistantMessage() {
     Array.isArray(msgContent) &&
     msgContent.some(
       (p) =>
-        p.type === "reasoning" ||
-        (p.type === "tool-call" && p.activity !== true),
+        (p.type !== "tool-call" || p.activity !== true) &&
+        isCollapsibleAssistantWorkPart(p),
     );
 
   if (!hasRenderableContent) return null;

package/corpus/core/src/client/composer/ComposerPlusMenu.tsx

@@ -42,6 +42,11 @@ import {
   type McpServerScope,
 } from "../resources/use-mcp-servers.js";
 import { cn } from "../utils.js";
+import {
+  createAssetPickerHandoffId,
+  isExternalAssetPickerUrl,
+  standaloneAssetPickerUrl,
+} from "./asset-picker-url.js";
 import type { ComposerMode } from "./types.js";
 
 interface ComposerPlusMenuProps {
@@ -73,6 +78,7 @@ interface EmbedEnvelope<TPayload = unknown> {
 
 interface AssetPickerPayload {
   assetId?: unknown;
+  handoffId?: unknown;
   url?: unknown;
   previewUrl?: unknown;
   downloadUrl?: unknown;
@@ -742,8 +748,31 @@ function AssetsPickerModal({
   onOpenChange: (open: boolean) => void;
 }) {
   const iframeRef = useRef<HTMLIFrameElement | null>(null);
+  const standaloneWindowRef = useRef<Window | null>(null);
   const [pickerReady, setPickerReady] = useState(false);
+  const [standaloneHandoffId, setStandaloneHandoffId] = useState<string | null>(
+    null,
+  );
   const sourceUrl = useMemo(() => assetPickerUrl(), []);
+  const externalPicker = useMemo(
+    () =>
+      typeof window !== "undefined" &&
+      isExternalAssetPickerUrl(sourceUrl, window.location.origin),
+    [sourceUrl],
+  );
+  const standaloneUrl = useMemo(
+    () =>
+      standaloneAssetPickerUrl(
+        sourceUrl,
+        typeof window !== "undefined" ? window.location.href : undefined,
+        {
+          handoffId: standaloneHandoffId ?? undefined,
+          returnOrigin:
+            typeof window !== "undefined" ? window.location.origin : undefined,
+        },
+      ),
+    [sourceUrl, standaloneHandoffId],
+  );
   const iframeUrl = useMemo(() => withEmbeddedParams(sourceUrl), [sourceUrl]);
   const targetOrigin = useMemo(() => assetPickerOrigin(iframeUrl), [iframeUrl]);
   const configurePicker = useCallback(() => {
@@ -758,14 +787,32 @@ function AssetsPickerModal({
   }, [targetOrigin]);
 
   useEffect(() => {
-    if (open) setPickerReady(false);
-  }, [iframeUrl, open]);
+    if (open) {
+      setPickerReady(false);
+      if (externalPicker) {
+        standaloneWindowRef.current = null;
+        setStandaloneHandoffId(createAssetPickerHandoffId());
+      } else {
+        setStandaloneHandoffId(null);
+      }
+      return;
+    }
+    if (!standaloneWindowRef.current) setStandaloneHandoffId(null);
+  }, [externalPicker, iframeUrl, open]);
 
   useEffect(() => {
-    if (!open || !targetOrigin) return;
+    if (
+      !targetOrigin ||
+      (!open && !standaloneWindowRef.current) ||
+      (externalPicker && !standaloneHandoffId)
+    )
+      return;
 
     const handleMessage = (event: MessageEvent) => {
-      if (event.source !== iframeRef.current?.contentWindow) return;
+      const expectedSource = externalPicker
+        ? standaloneWindowRef.current
+        : iframeRef.current?.contentWindow;
+      if (!expectedSource || event.source !== expectedSource) return;
       if (event.origin !== targetOrigin) return;
       if (!isEmbedEnvelope(event.data)) return;
 
@@ -776,6 +823,14 @@ function AssetsPickerModal({
       }
 
       if (event.data.type !== "message") return;
+      if (externalPicker) {
+        const payload = event.data.payload;
+        const handoffId =
+          payload && typeof payload === "object"
+            ? assetString((payload as AssetPickerPayload).handoffId)
+            : null;
+        if (handoffId !== standaloneHandoffId) return;
+      }
       if (event.data.name === "close") {
         onOpenChange(false);
         return;
@@ -798,6 +853,8 @@ function AssetsPickerModal({
         title: `Image: ${title}`,
         context: assetContext(event.data.payload, url),
       });
+      standaloneWindowRef.current = null;
+      setStandaloneHandoffId(null);
       onOpenChange(false);
     };
 
@@ -811,7 +868,26 @@ function AssetsPickerModal({
       window.removeEventListener("message", handleMessage);
       window.removeEventListener("keydown", handleKey);
     };
-  }, [configurePicker, onOpenChange, open, targetOrigin]);
+  }, [
+    configurePicker,
+    externalPicker,
+    onOpenChange,
+    open,
+    standaloneHandoffId,
+    targetOrigin,
+  ]);
+
+  const openStandalonePicker = useCallback(
+    (event: React.MouseEvent<HTMLAnchorElement>) => {
+      if (!standaloneHandoffId) return;
+      event.preventDefault();
+      const pickerWindow = window.open(standaloneUrl, "_blank");
+      if (!pickerWindow) return;
+      standaloneWindowRef.current = pickerWindow;
+      onOpenChange(false);
+    },
+    [onOpenChange, standaloneHandoffId, standaloneUrl],
+  );
 
   if (!open || typeof document === "undefined") return null;
 
@@ -842,7 +918,21 @@ function AssetsPickerModal({
             <IconX className="h-4 w-4" />
           </button>
         </div>
-        {targetOrigin ? (
+        {externalPicker ? (
+          <div className="flex min-h-0 flex-1 flex-col items-center justify-center gap-3 p-8 text-center">
+            <div className="max-w-md text-sm text-muted-foreground">
+              Open Assets in a new tab to sign in and choose an image securely.
+            </div>
+            <a
+              href={standaloneUrl}
+              target="_blank"
+              onClick={openStandalonePicker}
+              className="inline-flex h-9 items-center justify-center rounded-md bg-primary px-4 text-sm font-medium text-primary-foreground hover:bg-primary/90"
+            >
+              Open Assets image picker
+            </a>
+          </div>
+        ) : targetOrigin ? (
           <div className="relative min-h-0 flex-1 overflow-hidden bg-background">
             {!pickerReady && <AssetsPickerLoadingSkeleton />}
             <iframe

package/corpus/core/src/client/composer/asset-picker-url.ts

@@ -0,0 +1,80 @@
+const FALLBACK_BASE_URL = "http://agent-native.invalid";
+
+export const ASSET_PICKER_HANDOFF_PARAM = "__an_asset_picker_handoff";
+export const ASSET_PICKER_RETURN_ORIGIN_PARAM =
+  "__an_asset_picker_return_origin";
+
+export interface StandaloneAssetPickerOptions {
+  handoffId?: string;
+  returnOrigin?: string;
+}
+
+export function createAssetPickerHandoffId(): string {
+  if (typeof crypto !== "undefined" && crypto.randomUUID) {
+    return crypto.randomUUID();
+  }
+  if (typeof crypto !== "undefined" && crypto.getRandomValues) {
+    const bytes = crypto.getRandomValues(new Uint8Array(16));
+    return Array.from(bytes, (byte) => byte.toString(16).padStart(2, "0")).join(
+      "",
+    );
+  }
+  return `${Date.now().toString(36)}-${Math.random().toString(36).slice(2)}`;
+}
+
+/**
+ * Cross-origin Assets pages cannot inherit the host app's authenticated
+ * session inside an iframe. Treat those URLs as link-out targets so provider
+ * sign-in runs in a normal top-level browser context.
+ */
+export function isExternalAssetPickerUrl(
+  value: string,
+  currentOrigin: string,
+): boolean {
+  try {
+    return new URL(value, currentOrigin).origin !== currentOrigin;
+  } catch {
+    // A malformed configured URL should fail closed instead of being loaded in
+    // an iframe with an unknown auth boundary.
+    return true;
+  }
+}
+
+/** Build a top-level picker URL without iframe-only auth flags. */
+export function standaloneAssetPickerUrl(
+  value: string,
+  baseUrl = FALLBACK_BASE_URL,
+  options: StandaloneAssetPickerOptions = {},
+): string {
+  try {
+    const parsed = new URL(value, baseUrl);
+    parsed.searchParams.delete("embedded");
+    parsed.searchParams.delete("__an_embed_token");
+    parsed.searchParams.set("mediaType", "image");
+    let returnOrigin: string | null = null;
+    if (options.returnOrigin) {
+      try {
+        const parsedOrigin = new URL(options.returnOrigin);
+        if (
+          parsedOrigin.protocol === "http:" ||
+          parsedOrigin.protocol === "https:"
+        ) {
+          returnOrigin = parsedOrigin.origin;
+        }
+      } catch {
+        // Omit an invalid return target rather than emitting an unverified
+        // cross-origin callback URL.
+      }
+    }
+    if (options.handoffId && returnOrigin) {
+      parsed.searchParams.set(ASSET_PICKER_HANDOFF_PARAM, options.handoffId);
+      parsed.searchParams.set(ASSET_PICKER_RETURN_ORIGIN_PARAM, returnOrigin);
+    } else {
+      parsed.searchParams.delete(ASSET_PICKER_HANDOFF_PARAM);
+      parsed.searchParams.delete(ASSET_PICKER_RETURN_ORIGIN_PARAM);
+    }
+    return parsed.toString();
+  } catch {
+    return value;
+  }
+}

package/corpus/core/src/client/error-format.ts

@@ -102,6 +102,18 @@ function isProviderAuthenticationError(
   );
 }
 
+function isConnectionError(text: string, errorCode?: string): boolean {
+  const code = normalizeErrorCode(errorCode);
+  return (
+    code === "provider_network_error" ||
+    code === "connection_error" ||
+    code === "network_error" ||
+    /^(?:provider_network_error|connection_error|network_error)$/i.test(
+      text.trim(),
+    )
+  );
+}
+
 export function normalizeChatError(
   errorMessage: string,
   errorCode?: string,
@@ -126,6 +138,18 @@ export function normalizeChatError(
     };
   }
 
+  if (isConnectionError(text, errorCode)) {
+    const providerNetworkError =
+      normalizeErrorCode(errorCode) === "provider_network_error" ||
+      /provider_network_error/i.test(text);
+    return {
+      message: providerNetworkError
+        ? "The model provider could not be reached. Check your connection and retry."
+        : "The agent connection was interrupted. Check your connection and retry.",
+      details: text,
+    };
+  }
+
   if (/^Gateway error \(no detail; raw event:/i.test(text)) {
     // The previous copy promised auto-recovery and suggested switching models,
     // but neither helps for this code: the server already retried once and

package/corpus/core/src/client/resources/McpIntegrationDialog.tsx

@@ -115,7 +115,14 @@ export function McpIntegrationDialog({
       ...(mcpServersQuery.data?.user ?? []),
       ...(mcpServersQuery.data?.org ?? []),
     ];
-    return new Set(servers.map((server) => compareUrl(server.url)));
+    // A saved server is not necessarily a working connection. The settings
+    // page reports failed and unknown health states separately, so only mark
+    // catalog entries as connected after the health probe succeeds.
+    return new Set(
+      servers
+        .filter((server) => server.status.state === "connected")
+        .map((server) => compareUrl(server.url)),
+    );
   }, [mcpServersQuery.data]);
 
   const filteredIntegrations = useMemo(

package/corpus/core/src/local-artifacts/index.ts

@@ -16,6 +16,14 @@ export interface AgentNativeManifestRoot {
   extensions?: string[];
   include?: string[];
   hide?: string[];
+  source?: {
+    type: "local-folder";
+    connectionId: string;
+    truthPolicy?:
+      | "database_primary"
+      | "source_primary"
+      | "reviewed_bidirectional";
+  };
 }
 
 export interface AgentNativeManifestApp {
@@ -73,6 +81,7 @@ export interface LoadedLocalArtifactRoot {
   extensions: string[];
   hide: string[];
   include: string[];
+  source?: AgentNativeManifestRoot["source"];
 }
 
 export interface LoadedLocalArtifactApp {
@@ -267,6 +276,26 @@ function rootNameFromPath(rootPath: string): string {
 function normalizeManifestRoot(value: unknown): AgentNativeManifestRoot | null {
   if (typeof value === "string") return { path: value };
   if (!isRecord(value) || typeof value.path !== "string") return null;
+  const truthPolicy:
+    | NonNullable<AgentNativeManifestRoot["source"]>["truthPolicy"]
+    | undefined =
+    value.source &&
+    isRecord(value.source) &&
+    (value.source.truthPolicy === "database_primary" ||
+      value.source.truthPolicy === "source_primary" ||
+      value.source.truthPolicy === "reviewed_bidirectional")
+      ? value.source.truthPolicy
+      : undefined;
+  const source =
+    isRecord(value.source) &&
+    value.source.type === "local-folder" &&
+    typeof value.source.connectionId === "string"
+      ? {
+          type: "local-folder" as const,
+          connectionId: value.source.connectionId,
+          ...(truthPolicy ? { truthPolicy } : {}),
+        }
+      : undefined;
   return {
     name: typeof value.name === "string" ? value.name : undefined,
     path: value.path,
@@ -275,6 +304,7 @@ function normalizeManifestRoot(value: unknown): AgentNativeManifestRoot | null {
     extensions: normalizeExtensions(value.extensions),
     include: asStringArray(value.include),
     hide: asStringArray(value.hide),
+    ...(source ? { source } : {}),
   };
 }
 
@@ -483,6 +513,7 @@ export async function getLocalArtifactApp(
       extensions,
       hide: [...DEFAULT_HIDE_PATTERNS, ...asStringArray(root.hide)],
       include: asStringArray(root.include),
+      source: root.source,
     };
   });
 
@@ -683,6 +714,16 @@ export async function listLocalArtifactFiles(
   return files.sort((a, b) => a.path.localeCompare(b.path));
 }
 
+export async function listConfiguredLocalArtifactFiles(
+  options: LocalArtifactOptions,
+): Promise<LocalArtifactFileMeta[]> {
+  const app = await getLocalArtifactApp(options);
+  const files = (await Promise.all(app.roots.map((root) => walkRoot(root))))
+    .flat()
+    .filter((file) => !matchesPatterns(file.path, app.hide));
+  return files.sort((a, b) => a.path.localeCompare(b.path));
+}
+
 function rootForArtifactPath(
   app: LoadedLocalArtifactApp,
   artifactPath: string,
@@ -758,6 +799,30 @@ export async function readLocalArtifactFile(
   }
 }
 
+export async function readConfiguredLocalArtifactFile(
+  options: LocalArtifactOptions & { path: string },
+): Promise<LocalArtifactFile | null> {
+  const app = await getLocalArtifactApp(options);
+  const { root, safePath, absolutePath } = await resolveArtifactPath(
+    app,
+    options.path,
+  );
+  try {
+    const { content, stat } = readTextFileWithoutSymlink(root, absolutePath);
+    const meta = await fileMetaForPath(
+      root,
+      safePath,
+      absolutePath,
+      content,
+      stat,
+    );
+    return { ...meta, content };
+  } catch (error) {
+    if (errorCode(error) === "ENOENT") return null;
+    throw error;
+  }
+}
+
 export async function writeLocalArtifactFile(
   options: WriteLocalArtifactFileOptions & { path: string },
 ): Promise<LocalArtifactFileMeta> {