@agent-native/core 0.90.2 → 0.90.4

package/corpus/core/src/client/review/use-review.ts

@@ -0,0 +1,152 @@
+import type {
+  ReviewComment,
+  ReviewCommentKind,
+  ReviewMention,
+  ReviewResolutionTarget,
+  ReviewStatus,
+  ReviewStatusEntry,
+} from "../../review/types.js";
+import { useActionMutation, useActionQuery } from "../use-action.js";
+
+export interface ListReviewCommentsParams {
+  resourceType: string;
+  resourceId: string;
+  includeResolved?: boolean;
+  includeDeleted?: boolean;
+  targetId?: string | null;
+  limit?: number;
+}
+
+export interface ListReviewCommentsResult {
+  comments: ReviewComment[];
+  reviewStatus: ReviewStatusEntry | null;
+}
+
+export interface GetReviewFeedbackParams {
+  resourceType: string;
+  resourceId: string;
+  includeHumanTargeted?: boolean;
+  limit?: number;
+}
+
+export interface GetReviewFeedbackResult {
+  comments: ReviewComment[];
+}
+
+export interface CreateReviewCommentInput {
+  resourceType: string;
+  resourceId: string;
+  targetId?: string | null;
+  kind?: ReviewCommentKind;
+  anchor?: unknown | null;
+  body: string;
+  authorName?: string | null;
+  resolutionTarget?: ReviewResolutionTarget | null;
+  mentions?: ReviewMention[];
+  metadata?: Record<string, unknown>;
+}
+
+export interface ReplyReviewCommentInput {
+  resourceType: string;
+  resourceId: string;
+  commentId: string;
+  body: string;
+  authorName?: string | null;
+  resolutionTarget?: ReviewResolutionTarget | null;
+  mentions?: ReviewMention[];
+  metadata?: Record<string, unknown>;
+}
+
+export interface ResolveReviewThreadInput {
+  resourceType: string;
+  resourceId: string;
+  threadId?: string;
+  commentId?: string;
+}
+
+export interface DeleteReviewCommentInput {
+  resourceType: string;
+  resourceId: string;
+  commentId: string;
+}
+
+export interface ConsumeReviewFeedbackInput {
+  resourceType: string;
+  resourceId: string;
+  commentIds: string[];
+}
+
+export interface SetReviewStatusInput {
+  resourceType: string;
+  resourceId: string;
+  status: ReviewStatus;
+  note?: string | null;
+  metadata?: Record<string, unknown>;
+}
+
+export function useReviewComments(
+  params: ListReviewCommentsParams,
+  options?: { enabled?: boolean },
+) {
+  return useActionQuery<ListReviewCommentsResult>(
+    "list-review-comments",
+    params,
+    {
+      enabled:
+        options?.enabled ?? Boolean(params.resourceType && params.resourceId),
+    },
+  );
+}
+
+export function useReviewFeedback(
+  params: GetReviewFeedbackParams,
+  options?: { enabled?: boolean },
+) {
+  return useActionQuery<GetReviewFeedbackResult>(
+    "get-review-feedback",
+    params,
+    {
+      enabled:
+        options?.enabled ?? Boolean(params.resourceType && params.resourceId),
+    },
+  );
+}
+
+export function useCreateReviewComment() {
+  return useActionMutation<ReviewComment, CreateReviewCommentInput>(
+    "create-review-comment",
+  );
+}
+
+export function useReplyReviewComment() {
+  return useActionMutation<ReviewComment, ReplyReviewCommentInput>(
+    "reply-review-comment",
+  );
+}
+
+export function useResolveReviewThread() {
+  return useActionMutation<
+    { threadId: string; resolved: true; updatedCount: number },
+    ResolveReviewThreadInput
+  >("resolve-review-thread");
+}
+
+export function useDeleteReviewComment() {
+  return useActionMutation<
+    { commentId: string; deleted: true; updatedCount: number },
+    DeleteReviewCommentInput
+  >("delete-review-comment");
+}
+
+export function useConsumeReviewFeedback() {
+  return useActionMutation<
+    { consumedCommentIds: string[]; updatedCount: number },
+    ConsumeReviewFeedbackInput
+  >("consume-review-feedback");
+}
+
+export function useSetReviewStatus() {
+  return useActionMutation<ReviewStatusEntry, SetReviewStatusInput>(
+    "set-review-status",
+  );
+}

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

@@ -8,5 +8,15 @@ export {
   type SettingsTabItem,
   type SettingsTabsPageProps,
 } from "./SettingsTabsPage.js";
-export { useBuilderStatus } from "./useBuilderStatus.js";
+export {
+  openBuilderConnectPopup,
+  useBuilderConnectFlow,
+  useBuilderStatus,
+  withBuilderConnectTrackingParams,
+  type BuilderConnectFlow,
+  type BuilderConnectFlowOptions,
+  type BuilderConnectStartOptions,
+  type BuilderStatus,
+  type OpenBuilderConnectPopupOptions,
+} from "./useBuilderStatus.js";
 export { SecretsSection, type SecretsSectionProps } from "./SecretsSection.js";

package/corpus/core/src/client/setup-connections/BuilderConnectCard.tsx

@@ -0,0 +1,94 @@
+import { IconCheck, IconLoader2, IconPlugConnected } from "@tabler/icons-react";
+
+import { useBuilderConnectFlow } from "../settings/useBuilderStatus.js";
+import { cn } from "../utils.js";
+
+export interface BuilderConnectCardProps {
+  title?: string;
+  description?: string;
+  trackingSource?: string;
+  className?: string;
+  onConnected?: (orgName: string | null) => void;
+}
+
+export function BuilderConnectCard({
+  title = "Builder connect",
+  description = "Connect Builder for managed model access, browser automation, and workspace identity.",
+  trackingSource = "setup_connections_page",
+  className,
+  onConnected,
+}: BuilderConnectCardProps) {
+  const flow = useBuilderConnectFlow({
+    trackingSource,
+    onConnected: ({ orgName }) => onConnected?.(orgName),
+  });
+
+  const connectedLabel = flow.orgName
+    ? `Connected to ${flow.orgName}`
+    : "Connected";
+  const statusLabel = !flow.hasFetchedStatus
+    ? "Checking"
+    : flow.configured
+      ? connectedLabel
+      : flow.builderEnabled
+        ? "Ready to connect"
+        : "Project not configured";
+
+  return (
+    <section
+      className={cn(
+        "rounded-lg border border-border bg-background p-4 shadow-sm",
+        className,
+      )}
+    >
+      <div className="flex items-start gap-3">
+        <span
+          className={cn(
+            "mt-0.5 flex size-8 shrink-0 items-center justify-center rounded-md border",
+            flow.configured
+              ? "border-emerald-500/30 bg-emerald-500/10 text-emerald-600"
+              : "border-border bg-muted text-muted-foreground",
+          )}
+        >
+          {flow.configured ? (
+            <IconCheck className="size-4" />
+          ) : (
+            <IconPlugConnected className="size-4" />
+          )}
+        </span>
+        <div className="min-w-0 flex-1">
+          <div className="flex flex-wrap items-center gap-2">
+            <h2 className="text-sm font-semibold text-foreground">{title}</h2>
+            <span className="rounded-md border border-border bg-muted px-1.5 py-0.5 text-[11px] font-medium text-muted-foreground">
+              {statusLabel}
+            </span>
+          </div>
+          <p className="mt-1 text-sm leading-5 text-muted-foreground">
+            {description}
+          </p>
+          {flow.error ? (
+            <p className="mt-2 text-xs text-destructive">{flow.error}</p>
+          ) : null}
+          {!flow.configured ? (
+            <button
+              type="button"
+              onClick={() => flow.start()}
+              disabled={
+                flow.connecting ||
+                (!flow.builderEnabled && flow.hasFetchedStatus)
+              }
+              className="mt-3 inline-flex h-8 items-center gap-2 rounded-md border border-border bg-foreground px-3 text-xs font-medium text-background transition-colors hover:bg-foreground/90 disabled:cursor-not-allowed disabled:opacity-60"
+            >
+              {flow.connecting ? (
+                <IconLoader2 className="size-3.5 animate-spin" />
+              ) : (
+                <IconPlugConnected className="size-3.5" />
+              )}
+              Connect Builder
+            </button>
+          ) : null}
+        </div>
+      </div>
+    </section>
+  );
+}

package/corpus/core/src/client/setup-connections/ProviderReadinessBadge.tsx

@@ -0,0 +1,73 @@
+import {
+  IconAlertCircle,
+  IconCheck,
+  IconClock,
+  IconKey,
+} from "@tabler/icons-react";
+
+import type { WorkspaceConnectionProviderReadinessStatus } from "../../workspace-connections/index.js";
+import { cn } from "../utils.js";
+
+const STATUS_LABELS: Record<
+  WorkspaceConnectionProviderReadinessStatus,
+  string
+> = {
+  ready: "Ready",
+  checking: "Checking",
+  needs_credentials: "Needs credentials",
+  needs_attention: "Needs attention",
+  disabled: "Disabled",
+  not_configured: "Not configured",
+};
+
+const STATUS_STYLES: Record<
+  WorkspaceConnectionProviderReadinessStatus,
+  string
+> = {
+  ready:
+    "border-emerald-500/30 bg-emerald-500/10 text-emerald-700 dark:text-emerald-300",
+  checking:
+    "border-blue-500/30 bg-blue-500/10 text-blue-700 dark:text-blue-300",
+  needs_credentials:
+    "border-amber-500/30 bg-amber-500/10 text-amber-700 dark:text-amber-300",
+  needs_attention:
+    "border-red-500/30 bg-red-500/10 text-red-700 dark:text-red-300",
+  disabled: "border-border bg-muted text-muted-foreground",
+  not_configured: "border-border bg-muted text-muted-foreground",
+};
+
+function StatusIcon({
+  status,
+}: {
+  status: WorkspaceConnectionProviderReadinessStatus;
+}) {
+  if (status === "ready") return <IconCheck className="size-3.5" />;
+  if (status === "checking") return <IconClock className="size-3.5" />;
+  if (status === "needs_credentials") return <IconKey className="size-3.5" />;
+  return <IconAlertCircle className="size-3.5" />;
+}
+
+export interface ProviderReadinessBadgeProps {
+  status: WorkspaceConnectionProviderReadinessStatus;
+  label?: string;
+  className?: string;
+}
+
+export function ProviderReadinessBadge({
+  status,
+  label,
+  className,
+}: ProviderReadinessBadgeProps) {
+  return (
+    <span
+      className={cn(
+        "inline-flex min-h-6 items-center gap-1.5 rounded-md border px-2 text-xs font-medium",
+        STATUS_STYLES[status],
+        className,
+      )}
+    >
+      <StatusIcon status={status} />
+      {label ?? STATUS_LABELS[status]}
+    </span>
+  );
+}

package/corpus/core/src/client/setup-connections/SetupConnectionsPage.tsx

@@ -0,0 +1,94 @@
+import type { ReactNode } from "react";
+
+import { IntegrationsPanel } from "../integrations/IntegrationsPanel.js";
+import { OnboardingPanel } from "../onboarding/OnboardingPanel.js";
+import { SecretsSection } from "../settings/SecretsSection.js";
+import { cn } from "../utils.js";
+import { BuilderConnectCard } from "./BuilderConnectCard.js";
+
+export interface SetupConnectionsPageProps {
+  title?: string;
+  description?: string;
+  onboardingTitle?: string;
+  focusSecretKey?: string;
+  providerReadiness?: ReactNode;
+  workspaceConnections?: ReactNode;
+  children?: ReactNode;
+  showOnboarding?: boolean;
+  showBuilderConnect?: boolean;
+  showSecrets?: boolean;
+  showIntegrations?: boolean;
+  className?: string;
+}
+
+export function SetupConnectionsPage({
+  title = "Setup & connections",
+  description = "Manage setup status, Builder connect, app secrets, and workspace connections from one standard surface.",
+  onboardingTitle = "Setup checklist",
+  focusSecretKey,
+  providerReadiness,
+  workspaceConnections,
+  children,
+  showOnboarding = true,
+  showBuilderConnect = true,
+  showSecrets = true,
+  showIntegrations = false,
+  className,
+}: SetupConnectionsPageProps) {
+  return (
+    <div className={cn("min-h-full bg-background p-4 sm:p-6", className)}>
+      <div className="mx-auto flex w-full max-w-6xl flex-col gap-5">
+        <header className="space-y-1">
+          <h1 className="text-2xl font-semibold tracking-normal text-foreground">
+            {title}
+          </h1>
+          <p className="max-w-3xl text-sm leading-6 text-muted-foreground">
+            {description}
+          </p>
+        </header>
+
+        <div className="grid gap-4 lg:grid-cols-[minmax(0,1fr)_360px]">
+          <main className="flex min-w-0 flex-col gap-4">
+            {showOnboarding ? (
+              <OnboardingPanel title={onboardingTitle} />
+            ) : null}
+            {workspaceConnections ? (
+              <section className="rounded-lg border border-border bg-background p-4 shadow-sm">
+                {workspaceConnections}
+              </section>
+            ) : null}
+            {providerReadiness ? (
+              <section className="rounded-lg border border-border bg-background p-4 shadow-sm">
+                {providerReadiness}
+              </section>
+            ) : null}
+            {children}
+          </main>
+
+          <aside className="flex min-w-0 flex-col gap-4">
+            {showBuilderConnect ? <BuilderConnectCard /> : null}
+            {showSecrets ? (
+              <section className="rounded-lg border border-border bg-background p-4 shadow-sm">
+                <div className="mb-3">
+                  <h2 className="text-sm font-semibold text-foreground">
+                    Secrets
+                  </h2>
+                  <p className="mt-1 text-xs leading-5 text-muted-foreground">
+                    Registered keys and OAuth credentials are stored in the
+                    scoped secrets store.
+                  </p>
+                </div>
+                <SecretsSection focusKey={focusSecretKey} />
+              </section>
+            ) : null}
+            {showIntegrations ? (
+              <section className="rounded-lg border border-border bg-background p-4 shadow-sm">
+                <IntegrationsPanel />
+              </section>
+            ) : null}
+          </aside>
+        </div>
+      </div>
+    </div>
+  );
+}

package/corpus/core/src/client/setup-connections/index.ts

@@ -0,0 +1,46 @@
+export {
+  BuilderConnectCard,
+  type BuilderConnectCardProps,
+} from "./BuilderConnectCard.js";
+export {
+  ProviderReadinessBadge,
+  type ProviderReadinessBadgeProps,
+} from "./ProviderReadinessBadge.js";
+export {
+  SetupConnectionsPage,
+  type SetupConnectionsPageProps,
+} from "./SetupConnectionsPage.js";
+export {
+  OnboardingBanner,
+  OnboardingPanel,
+  SetupButton,
+  useOnboarding,
+  useOnboardingPreviewMode,
+  type OnboardingFormField,
+  type OnboardingMethod,
+  type OnboardingMethodBadge,
+  type OnboardingStep,
+  type OnboardingStepStatus,
+  type UseOnboardingResult,
+} from "../onboarding/index.js";
+export {
+  SecretsSection,
+  SettingsPanel,
+  SettingsTabsPage,
+  openBuilderConnectPopup,
+  useBuilderConnectFlow,
+  useBuilderStatus,
+  type BuilderConnectFlow,
+  type BuilderConnectFlowOptions,
+  type BuilderStatus,
+  type OpenBuilderConnectPopupOptions,
+  type SecretsSectionProps,
+  type SettingsPanelProps,
+  type SettingsTabItem,
+  type SettingsTabsPageProps,
+} from "../settings/index.js";
+export {
+  IntegrationsPanel,
+  useIntegrationStatus,
+  type IntegrationStatus,
+} from "../integrations/index.js";

package/corpus/core/src/client/sharing/ShareButton.tsx

@@ -1,7 +1,6 @@
 import * as Select from "@radix-ui/react-select";
 import {
   IconLock,
-  IconBuilding,
   IconWorld,
   IconTrash,
   IconCheck,
@@ -11,6 +10,7 @@ import {
   IconSearch,
   IconSearchOff,
   IconShare3,
+  IconUsersGroup,
 } from "@tabler/icons-react";
 import { useQueryClient } from "@tanstack/react-query";
 import {
@@ -46,11 +46,11 @@ export interface ShareButtonProps {
   /** @deprecated No longer affects rendering — trigger always says
    *  "Share". Kept for callsite compatibility. */
   variant?: "compact" | "label";
-  /** Optional trigger style. Defaults to the Google-Docs-style "Share" label.
-   *  Use "label-icon" to render a leading share glyph alongside the label so
-   *  the trigger matches adjacent icon+label buttons; "icon" is icon-only. */
+  /** Optional trigger style. Defaults to the Google-Docs-style "Share" label
+   *  with a leading visibility icon. "label-icon" is kept for callsites that
+   *  want to explicitly opt into icon+label parity; "icon" is icon-only. */
   trigger?: "label" | "icon" | "label-icon";
-  /** @deprecated Label triggers no longer render a visibility/share glyph. */
+  /** @deprecated No longer affects rendering — triggers show visibility state. */
   hideTriggerIcon?: boolean;
   /** Optional className applied to the trigger button. */
   triggerClassName?: string;
@@ -197,7 +197,7 @@ const VIS_META: Record<
   org: {
     label: "Organization",
     description: "Anyone in your organization can view",
-    Icon: IconBuilding,
+    Icon: IconUsersGroup,
   },
   public: {
     label: "Public",
@@ -356,10 +356,17 @@ export function ShareButton(props: ShareButtonProps) {
     });
   };
 
-  // The default trigger stays text-only; the icon trigger keeps the share glyph.
-  // "label-icon" renders the glyph alongside the label for icon+label parity.
+  const triggerVisibility =
+    pendingVisibility ??
+    (sharesQuery.data
+      ? ((sharesQuery.data.visibility as Visibility | null) ?? "private")
+      : null);
+  const triggerMeta = triggerVisibility
+    ? visibilityMeta(triggerVisibility, props.visibilityCopy)
+    : null;
+  const TriggerIcon = triggerMeta?.Icon ?? IconShare3;
+  const triggerLabel = triggerMeta ? `Share (${triggerMeta.label})` : "Share";
   const iconOnly = props.trigger === "icon";
-  const showLabelIcon = props.trigger === "label-icon";
 
   return (
     <Popover open={open} onOpenChange={handleOpenChange}>
@@ -370,11 +377,10 @@ export function ShareButton(props: ShareButtonProps) {
             iconOnly ? BUTTON_GHOST_ICON : BUTTON_OUTLINE_SM,
             props.triggerClassName,
           )}
-          aria-label={iconOnly ? "Share" : undefined}
+          aria-label={triggerLabel}
+          title={triggerLabel}
         >
-          {iconOnly || showLabelIcon ? (
-            <IconShare3 size={16} strokeWidth={1.75} />
-          ) : null}
+          <TriggerIcon size={16} strokeWidth={1.75} />
           {!iconOnly && <span>Share</span>}
         </button>
       </PopoverTrigger>
@@ -1663,7 +1669,7 @@ function Avatar({ label, org }: { label: string; org?: boolean }) {
       aria-hidden
       className="inline-flex h-7 w-7 shrink-0 items-center justify-center rounded-full bg-muted text-[11px] font-semibold text-muted-foreground"
     >
-      {org ? <IconBuilding size={14} strokeWidth={1.75} /> : initials(label)}
+      {org ? <IconUsersGroup size={14} strokeWidth={1.75} /> : initials(label)}
     </span>
   );
 }

package/corpus/core/src/client/sharing/ShareDialog.tsx

@@ -3,7 +3,6 @@ import {
   IconX,
   IconTrash,
   IconLock,
-  IconBuilding,
   IconWorld,
   IconCheck,
   IconCopy,
@@ -11,6 +10,7 @@ import {
   IconMail,
   IconCode,
   IconChevronDown,
+  IconUsersGroup,
 } from "@tabler/icons-react";
 import { useEffect, useState, type ReactNode } from "react";
 import { createPortal } from "react-dom";
@@ -117,7 +117,7 @@ const BUTTON_GHOST_ICON = cn(
 
 const VIS_ICONS: Record<Visibility, typeof IconLock> = {
   private: IconLock,
-  org: IconBuilding,
+  org: IconUsersGroup,
   public: IconWorld,
 };
 
@@ -824,7 +824,7 @@ function Avatar({ label, org }: { label: string; org?: boolean }) {
       className="inline-flex h-7 w-7 shrink-0 items-center justify-center rounded-full bg-muted text-[11px] font-semibold text-muted-foreground"
     >
       {org ? (
-        <IconBuilding size={14} strokeWidth={1.75} />
+        <IconUsersGroup size={14} strokeWidth={1.75} />
       ) : (
         (label.split("@")[0]?.[0] ?? label[0] ?? "?").toUpperCase()
       )}

package/corpus/core/src/command-navigation/actions.ts

@@ -0,0 +1,100 @@
+import { z } from "zod";
+
+import { defineAction } from "../action.js";
+import { readAppState, writeAppState } from "../application-state/index.js";
+
+const navigateSchema = z
+  .object({
+    view: z.string().optional().describe("View name to navigate to"),
+    path: z.string().optional().describe("URL path to navigate to"),
+  })
+  .passthrough();
+
+export interface CreateNavigateActionOptions {
+  description?: string;
+  stateKey?: string;
+  normalize?: (
+    args: z.infer<typeof navigateSchema>,
+  ) => Record<string, unknown> | Promise<Record<string, unknown>>;
+  describeResult?: (
+    navigation: Record<string, unknown>,
+    args: z.infer<typeof navigateSchema>,
+  ) => string;
+}
+
+export interface CreateViewScreenActionOptions {
+  description?: string;
+  navigationStateKey?: string;
+  extraStateKeys?: string[];
+  describeResult?: (
+    screen: Record<string, unknown>,
+  ) =>
+    | Record<string, unknown>
+    | string
+    | Promise<Record<string, unknown> | string>;
+}
+
+function writeId(): string {
+  return `${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+}
+
+export function createNavigateAction(
+  options: CreateNavigateActionOptions = {},
+) {
+  const stateKey = options.stateKey ?? "navigate";
+  return defineAction({
+    description:
+      options.description ??
+      "Navigate the UI to a specific view or path. Writes a navigate command to application state which the UI reads and auto-deletes.",
+    schema: navigateSchema,
+    http: false,
+    run: async (args) => {
+      if (!args.view && !args.path) {
+        throw new Error("At least --view or --path is required.");
+      }
+      const navigation = options.normalize
+        ? await options.normalize(args)
+        : ({
+            ...(args.view ? { view: args.view } : null),
+            ...(args.path ? { path: args.path } : null),
+          } as Record<string, unknown>);
+      navigation._writeId = writeId();
+      await writeAppState(stateKey, navigation);
+      return (
+        options.describeResult?.(navigation, args) ??
+        `Navigating to ${String(args.view || args.path)}`
+      );
+    },
+  });
+}
+
+export function createViewScreenAction(
+  options: CreateViewScreenActionOptions = {},
+) {
+  const navigationStateKey = options.navigationStateKey ?? "navigation";
+  return defineAction({
+    description:
+      options.description ??
+      "See what the user is currently looking at on screen. Returns the current navigation state.",
+    schema: z.object({}),
+    http: false,
+    readOnly: true,
+    run: async () => {
+      const screen: Record<string, unknown> = {};
+      const navigation = await readAppState(navigationStateKey);
+      if (navigation) screen.navigation = navigation;
+
+      for (const key of options.extraStateKeys ?? []) {
+        const value = await readAppState(key);
+        if (value !== null && value !== undefined) screen[key] = value;
+      }
+
+      if (Object.keys(screen).length === 0) {
+        return "No application state found. Is the app running?";
+      }
+      return options.describeResult
+        ? await options.describeResult(screen)
+        : screen;
+    },
+  });
+}