@agent-native/core 0.100.2 → 0.101.2

package/corpus/core/src/cli/skills-content/visual-plan-skill.ts

@@ -201,10 +201,12 @@ forward only the code-research and plan-composition guidance here.
 
 ## Self-Review Before Handoff
 
-For high-stakes plans — architecture, backend, data-model, migration, multi-file,
-or otherwise risky work — run one adversarial self-review pass before treating the
-plan as final. Skip it for small, UI-only, or single-decision plans where the cost
-outweighs the value. Keep the pass cheap and non-blocking:
+This adversarial self-review pass is opt-in, not default: run it only for
+high-stakes plans — irreversible migrations, security-sensitive work, or when
+the user explicitly asks for extra rigor — and skip it otherwise. It roughly
+doubles the cost of plan generation, so the default for small, UI-only,
+single-decision, or ordinary plans is to skip it, not to run it. Keep the pass
+cheap and non-blocking when it does run:
 
 - **Surface the plan first, review concurrently.** Post the link and let the user
   start reading, then run the review in parallel — never make the user wait on it.

package/corpus/core/src/cli/skills-content/visual-recap-skill.ts

@@ -133,6 +133,10 @@ Budgets that keep the recap reviewable:
   tab; summarize or link the rest of a long file instead of dumping it.
 - Title at most ~70 characters; brief 1-3 sentences.
 
+These budgets are also the cost ceiling: do not exceed them in the name of
+thoroughness, and do not re-read the full diff after the initial sequential
+pass — work from the notes taken during that pass.
+
 **GOOD.** A 25-file auth change: Before/After wireframes of the login surface,
 a two-paragraph narrative, a diff-aware \`data-model\` of the sessions table, an
 \`api-endpoint\` for the new refresh route, a \`file-tree\` with change flags, and
@@ -213,7 +217,9 @@ sketchy rough overlay.
 When a browser tool is available, render a UI-impact recap in the Plan viewer
 and visually inspect it at the current theme before sharing. If any label,
 annotation, toolbar, or wireframe content overlaps another element, fix the MDX
-and re-import before reporting the link. A text-match screenshot is not enough;
+and re-import before reporting the link. Limit this to one render-and-inspect
+pass plus at most one fix-and-re-render; do not keep iterating beyond that
+unless the user explicitly asks. A text-match screenshot is not enough;
 visually inspect the captured image. When no browser is available (for example
 a headless CI agent), state that in the recap handoff instead.
 

package/corpus/core/src/client/AgentPanel.tsx

@@ -350,28 +350,30 @@ function ChatLoadingSkeleton({
             </div>
           </div>
           {composerSlot}
-          <div
-            className={cn(
-              "agent-composer-area shrink-0 px-3 py-2",
-              composerLayoutVariant !== "default" &&
-                `agent-composer-area--${composerLayoutVariant}`,
-              composerAreaClassName,
-            )}
-          >
+          <div className="agent-composer-stack">
             <div
               className={cn(
-                "agent-composer-root flex flex-col rounded-lg border border-input bg-muted/45 transition-colors",
+                "agent-composer-area shrink-0 px-3 py-2",
                 composerLayoutVariant !== "default" &&
-                  `agent-composer-root--${composerLayoutVariant}`,
+                  `agent-composer-area--${composerLayoutVariant}`,
+                composerAreaClassName,
               )}
             >
-              <div className="px-3 pt-3">
-                <div className="h-5 w-3/5 rounded bg-muted animate-pulse motion-reduce:animate-none" />
-              </div>
-              <div className="mt-auto flex items-center gap-2 px-3 py-2">
-                <div className="h-5 w-5 rounded bg-muted animate-pulse motion-reduce:animate-none" />
-                <div className="ml-auto h-4 w-28 rounded bg-muted animate-pulse motion-reduce:animate-none" />
-                <div className="h-7 w-7 rounded-md bg-muted animate-pulse motion-reduce:animate-none" />
+              <div
+                className={cn(
+                  "agent-composer-root flex flex-col rounded-lg border border-input bg-muted/45 transition-colors",
+                  composerLayoutVariant !== "default" &&
+                    `agent-composer-root--${composerLayoutVariant}`,
+                )}
+              >
+                <div className="px-3 pt-3">
+                  <div className="h-5 w-3/5 rounded bg-muted animate-pulse motion-reduce:animate-none" />
+                </div>
+                <div className="mt-auto flex items-center gap-2 px-3 py-2">
+                  <div className="h-5 w-5 rounded bg-muted animate-pulse motion-reduce:animate-none" />
+                  <div className="ml-auto h-4 w-28 rounded bg-muted animate-pulse motion-reduce:animate-none" />
+                  <div className="h-7 w-7 rounded-md bg-muted animate-pulse motion-reduce:animate-none" />
+                </div>
               </div>
             </div>
           </div>

package/corpus/core/src/client/analytics.ts

@@ -96,6 +96,11 @@ export type ConfigureTrackingOptions = {
   contentCapture?: boolean;
   /** Resolve content capture synchronously for each browser pathname. */
   contentCaptureForPath?: (pathname: string) => boolean;
+  /**
+   * Whether tracking may read the authenticated agent-engine status endpoint.
+   * Disable this on anonymous/public routes to avoid an expected 401 request.
+   */
+  llmConnectionStatus?: boolean;
   sessionReplay?: boolean | SessionReplayOptions;
   /**
    * First-party, Sentry-style error capture. Auto-captures uncaught errors
@@ -1042,7 +1047,9 @@ export function configureTracking(options: ConfigureTrackingOptions): void {
     ensureSentry();
     ensureAmplitude();
     captureFirstTouchAttribution();
-    installLlmConnectionRefresh();
+    if (options.llmConnectionStatus !== false) {
+      installLlmConnectionRefresh();
+    }
     installTrackingAuthSessionRefresh();
     installPageviewTracking();
     maybeInstallSessionReplay(

package/corpus/core/src/client/chat/markdown-renderer.tsx

@@ -47,20 +47,30 @@ type ReactMarkdownModule = {
   defaultUrlTransform: typeof DefaultUrlTransformType;
 };
 
+type RenderToStaticMarkupFn = (node: React.ReactElement) => string;
+
 export let markdownModule: ReactMarkdownModule | null = null;
 export let remarkGfmFn: typeof remarkGfmType | null = null;
+let renderToStaticMarkupFn: RenderToStaticMarkupFn | null = null;
 const markdownListeners = new Set<() => void>();
 
 export function loadMarkdown(): void {
   if (markdownModule !== null) return; // already loaded
-  Promise.all([import("react-markdown"), import("remark-gfm")]).then(
-    ([md, gfm]) => {
-      markdownModule = md as ReactMarkdownModule;
-      remarkGfmFn = gfm.default;
-      markdownListeners.forEach((fn) => fn());
-      markdownListeners.clear();
-    },
-  );
+  Promise.all([
+    import("react-markdown"),
+    import("remark-gfm"),
+    // react-dom/server powers the synchronous markdown→HTML string used for
+    // rich clipboard copy; loaded alongside so readiness stays a single gate.
+    import("react-dom/server"),
+  ]).then(([md, gfm, server]) => {
+    markdownModule = md as ReactMarkdownModule;
+    remarkGfmFn = gfm.default;
+    renderToStaticMarkupFn = (
+      server as { renderToStaticMarkup: RenderToStaticMarkupFn }
+    ).renderToStaticMarkup;
+    markdownListeners.forEach((fn) => fn());
+    markdownListeners.clear();
+  });
 }
 
 export function onMarkdownReady(fn: () => void): () => void {
@@ -286,6 +296,42 @@ export const markdownComponents = {
   },
 };
 
+// ─── Clipboard HTML rendering ─────────────────────────────────────────────────
+// A stripped component set for the `text/html` clipboard flavor: plain <a> and
+// <pre>/<code> with no in-app buttons, iframes, or syntax-highlight markup, so
+// pasted output is portable structure (bold, lists, links, code) rather than
+// app-specific chrome that receiving apps (Slack, Notion) discard anyway.
+
+const clipboardMarkdownComponents = {
+  a(props: React.AnchorHTMLAttributes<HTMLAnchorElement>) {
+    const { href, children } = props;
+    if (href === NEW_CHAT_ACTION_HREF || !href) return <span>{children}</span>;
+    return <a href={href}>{children}</a>;
+  },
+  pre(props: React.HTMLAttributes<HTMLPreElement>) {
+    return <pre>{props.children}</pre>;
+  },
+};
+
+// Renders joined message markdown to an HTML string for rich clipboard copy.
+// Returns null when the lazy markdown/react-dom-server modules haven't landed
+// yet; callers fall back to plain-text copy in that case.
+export function renderMarkdownToClipboardHtml(markdown: string): string | null {
+  const ReactMarkdown = markdownModule?.default;
+  const gfm = remarkGfmFn;
+  const renderToStaticMarkup = renderToStaticMarkupFn;
+  if (!ReactMarkdown || !gfm || !renderToStaticMarkup) return null;
+  return renderToStaticMarkup(
+    <ReactMarkdown
+      remarkPlugins={[gfm]}
+      components={clipboardMarkdownComponents}
+      urlTransform={markdownUrlTransform}
+    >
+      {markdown}
+    </ReactMarkdown>,
+  );
+}
+
 // ─── Smooth streaming ─────────────────────────────────────────────────────────
 
 function usePrefersReducedMotion(): boolean {

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

@@ -64,7 +64,10 @@ import { PastedTextChip } from "../composer/PastedTextChip.js";
 import { ThumbsFeedback } from "../observability/ThumbsFeedback.js";
 import type { ContentPart } from "../sse-event-processor.js";
 import { cn } from "../utils.js";
-import { MarkdownText } from "./markdown-renderer.js";
+import {
+  MarkdownText,
+  renderMarkdownToClipboardHtml,
+} from "./markdown-renderer.js";
 import {
   ToolCallFallback,
   FilesChangedSummary,
@@ -526,7 +529,10 @@ export function MessageActionsMenu({
       .filter((p) => p.type === "text")
       .map((p) => (p as { text: string }).text)
       .join("\n");
-    void writeClipboardText(text).then((ok) => {
+    // Rich flavor keeps formatting in targets that read text/html (e.g. Slack);
+    // null when the markdown renderer isn't ready yet, so we copy plain markdown.
+    const html = renderMarkdownToClipboardHtml(text);
+    void writeClipboardText(text, html ? { html } : undefined).then((ok) => {
       if (!ok) return;
       setCopied("message");
       setTimeout(() => {

package/corpus/core/src/client/clipboard.ts

@@ -39,7 +39,15 @@ function writeWithExecCommand(text: string): boolean {
   }
 }
 
-export async function writeClipboardText(text: string): Promise<boolean> {
+export async function writeClipboardText(
+  text: string,
+  // Optional rich flavor. When provided and the browser supports ClipboardItem,
+  // both text/plain (this text) and text/html are written so pasted output keeps
+  // formatting in rich targets (Slack) while staying markdown in editors
+  // (Notion). The desktop bridge and execCommand fallbacks are plain-text only,
+  // so html is ignored there.
+  options?: { html?: string },
+): Promise<boolean> {
   for (const desktopClipboard of getDesktopClipboards()) {
     try {
       const result = await desktopClipboard.writeText?.(text);
@@ -50,6 +58,25 @@ export async function writeClipboardText(text: string): Promise<boolean> {
   }
 
   if (typeof navigator !== "undefined" && navigator.clipboard?.writeText) {
+    const html = options?.html;
+    if (
+      html !== undefined &&
+      typeof ClipboardItem !== "undefined" &&
+      navigator.clipboard.write
+    ) {
+      try {
+        await navigator.clipboard.write([
+          new ClipboardItem({
+            "text/plain": new Blob([text], { type: "text/plain" }),
+            "text/html": new Blob([html], { type: "text/html" }),
+          }),
+        ]);
+        return true;
+      } catch {
+        // Fall through to the plain-text paths below.
+      }
+    }
+
     try {
       await navigator.clipboard.writeText(text);
       return true;

package/corpus/core/src/client/guided-questions.tsx

@@ -959,6 +959,8 @@ function normalizeBrowserTabId(browserTabId?: string): string | undefined {
 }
 
 export interface UseGuidedQuestionFlowOptions {
+  /** Disable application-state reads for signed-out or otherwise inactive surfaces. */
+  enabled?: boolean;
   stateKey?: string;
   /**
    * The current browser tab id. Agent actions that write the guided-questions
@@ -980,6 +982,7 @@ export interface UseGuidedQuestionFlowOptions {
 }
 
 export function useGuidedQuestionFlow({
+  enabled = true,
   stateKey = "show-questions",
   browserTabId,
   queryKey = ["show-questions"],
@@ -1030,6 +1033,7 @@ export function useGuidedQuestionFlow({
 
   const { data } = useQuery({
     queryKey: resolvedQueryKey,
+    enabled,
     queryFn: async () => {
       const read = async (key: string) => {
         const res = await fetch(endpointFor(key));

package/corpus/core/src/client/i18n.tsx

@@ -413,6 +413,7 @@ export function AgentNativeI18nProvider({
   }, []);
 
   useEffect(() => {
+    if (!persistPreference) return;
     void setClientAppState(
       "localization",
       {
@@ -424,7 +425,7 @@ export function AgentNativeI18nProvider({
     ).catch(() => {
       // Public/anonymous pages cannot write app-state; localization still works.
     });
-  }, [locale, preference]);
+  }, [locale, persistPreference, preference]);
 
   const setPreference = useCallback(
     async (next: LocalePreference) => {

package/corpus/core/src/client/index.ts

@@ -864,6 +864,7 @@ export {
   type VersionHistoryPanelProps,
 } from "./history/index.js";
 export {
+  ReviewCommentComposer,
   ReviewStatusBadge,
   ReviewThreadPanel,
   buildReviewThreads,
@@ -874,6 +875,7 @@ export {
   useResolveReviewThread,
   useReviewComments,
   useReviewFeedback,
+  useSendReviewThreadToAgent,
   useSetReviewStatus,
   type ConsumeReviewFeedbackInput,
   type CreateReviewCommentInput,
@@ -885,9 +887,11 @@ export {
   type ReplyReviewCommentInput,
   type ResolveReviewThreadInput,
   type ReviewStatusBadgeProps,
+  type ReviewCommentComposerProps,
   type ReviewThread,
   type ReviewThreadPanelProps,
   type SetReviewStatusInput,
+  type SendReviewThreadToAgentInput,
 } from "./review/index.js";
 export type {
   AppToFrameMessage,

package/corpus/core/src/client/review/ReviewCommentComposer.tsx

@@ -0,0 +1,108 @@
+import { Button } from "@agent-native/toolkit/ui/button";
+import { Spinner } from "@agent-native/toolkit/ui/spinner";
+import { Textarea } from "@agent-native/toolkit/ui/textarea";
+import { IconMessageCircle, IconSend } from "@tabler/icons-react";
+
+import type { ReviewResolutionTarget } from "../../review/types.js";
+import { cn } from "../utils.js";
+
+export interface ReviewCommentComposerProps {
+  value: string;
+  onChange: (value: string) => void;
+  onSubmit: (resolutionTarget: ReviewResolutionTarget) => void;
+  submittingTarget?: ReviewResolutionTarget | null;
+  disabled?: boolean;
+  showAgentAction?: boolean;
+  placeholder?: string;
+  commentLabel?: string;
+  agentLabel?: string;
+  autoFocus?: boolean;
+  submitOnEnter?: boolean;
+  onEscape?: () => void;
+  className?: string;
+}
+
+export function ReviewCommentComposer({
+  value,
+  onChange,
+  onSubmit,
+  submittingTarget = null,
+  disabled = false,
+  showAgentAction = false,
+  placeholder = "Add a comment...",
+  commentLabel = "Comment",
+  agentLabel = "Send to agent",
+  autoFocus = false,
+  submitOnEnter = false,
+  onEscape,
+  className,
+}: ReviewCommentComposerProps) {
+  const canSubmit = Boolean(value.trim()) && !disabled;
+  const submit = (resolutionTarget: ReviewResolutionTarget) => {
+    if (!canSubmit) return;
+    onSubmit(resolutionTarget);
+  };
+
+  return (
+    <form
+      className={cn("@container/review", className)}
+      onSubmit={(event) => {
+        event.preventDefault();
+        submit("human");
+      }}
+    >
+      <Textarea
+        autoFocus={autoFocus}
+        value={value}
+        disabled={disabled}
+        onChange={(event) => onChange(event.currentTarget.value)}
+        placeholder={placeholder}
+        className="min-h-16 resize-none text-sm"
+        onKeyDown={(event) => {
+          if (event.key === "Escape" && onEscape) {
+            event.stopPropagation();
+            event.preventDefault();
+            onEscape();
+            return;
+          }
+          if (submitOnEnter && event.key === "Enter" && !event.shiftKey) {
+            event.preventDefault();
+            submit("human");
+          }
+        }}
+      />
+      <div className="mt-2 flex flex-col items-stretch justify-end gap-2 @2xs/review:flex-row @2xs/review:items-center">
+        <Button
+          type="submit"
+          size="sm"
+          disabled={!canSubmit}
+          className="h-8 w-full min-w-0 gap-1.5 @2xs/review:w-auto"
+        >
+          {submittingTarget === "human" ? (
+            <Spinner className="size-3.5" />
+          ) : (
+            <IconMessageCircle className="size-3.5" />
+          )}
+          <span className="truncate">{commentLabel}</span>
+        </Button>
+        {showAgentAction ? (
+          <Button
+            type="button"
+            size="sm"
+            variant="outline"
+            disabled={!canSubmit}
+            className="h-8 w-full min-w-0 gap-1.5 @2xs/review:w-auto"
+            onClick={() => submit("agent")}
+          >
+            {submittingTarget === "agent" ? (
+              <Spinner className="size-3.5" />
+            ) : (
+              <IconSend className="size-3.5" />
+            )}
+            <span className="truncate">{agentLabel}</span>
+          </Button>
+        ) : null}
+      </div>
+    </form>
+  );
+}