@anchrd/intel-ui 0.1.1 → 0.2.1

package/README.md

@@ -1,7 +1,7 @@
 # @anchrd/intel-ui
 
 The Intel browser application: Knowledge, Flows and Tools for the
-[`@anchrd/intel`](https://www.npmjs.com/package/@anchrd/intel) server.
+[`@anchrd/intel-api`](https://www.npmjs.com/package/@anchrd/intel-api) server.
 
 This package ships **source, not a bundle**. You compile it yourself so your own theme, logo and
 languages are applied at build time and the output belongs to you — no customer build artifact ever
@@ -9,10 +9,10 @@ lives in `node_modules`.
 
 ## Install
 
-Install it next to `@anchrd/intel`, then let the CLI build it:
+Install it next to `@anchrd/intel-api`, then let the CLI build it:
 
 ```bash
-npm install @anchrd/intel @anchrd/intel-ui
+npm install @anchrd/intel-api @anchrd/intel-ui
 npx intel build
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@anchrd/intel-ui",
-  "version": "0.1.1",
+  "version": "0.2.1",
   "type": "module",
   "license": "UNLICENSED",
   "repository": {
@@ -28,7 +28,7 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@anchrd/intel-contract": "^0.1.1",
+    "@anchrd/intel-contract": "^0.2.0",
     "@blocknote/core": "^0.52.1",
     "@blocknote/react": "^0.52.1",
     "@blocknote/shadcn": "^0.52.1",

package/src/data/intel-data-provider/intel-data-provider.ts

@@ -4,9 +4,7 @@ import {
   CreateFlowInput,
   CreateKnowledgeLinkInput,
   CreateKnowledgeNodeInput,
-  CreateToolSourceInput,
   DeleteKnowledgeLinkInput,
-  DiscoverToolSourceInput,
   Flow,
   FlowDocument,
   FlowList,
@@ -36,8 +34,6 @@ import {
   StartFlowRunInput,
   TestToolInput,
   ToolCatalog,
-  ToolSource,
-  ToolSourceList,
   ToolTestResult,
   UpdateKnowledgeNodeInput,
 } from "@anchrd/intel-contract";
@@ -49,6 +45,10 @@ const DeleteResult = z.strictObject({ deleted: z.boolean() });
 // Attachment uploads and downloads share this budget, so it is generous rather than snappy.
 const RequestTimeoutMs = 60_000;
 
+// Deeper nesting than this is not a tree anyone navigates; the limit exists to bound the request
+// fan-out on first paint, not to express a product rule.
+const MaxTreeDepth = 12;
+
 // The Worker serves the login route, so the path belongs to the data layer rather than to any view.
 export function loginPath(returnTo?: string): string {
   return returnTo ? `/auth/login?returnTo=${encodeURIComponent(returnTo)}` : "/auth/login";
@@ -115,12 +115,23 @@ export function createIntelDataProvider(
     return await request(`/knowledge?${params}`, KnowledgeNodeList);
   }
 
-  async function loadChildren(parentId: string | null): Promise<KnowledgeTreeNode[]> {
+  // One request per folder, so a deep tree is an N+1 on the first paint. The depth limit bounds that
+  // and, more importantly, makes a cyclic parentId impossible to hang on: without it a cycle would
+  // recurse until the tab dies. `seen` stops a cycle at the first repeat rather than at the limit.
+  async function loadChildren(
+    parentId: string | null,
+    depth = 0,
+    seen: ReadonlySet<string> = new Set(),
+  ): Promise<KnowledgeTreeNode[]> {
+    if (depth >= MaxTreeDepth) return [];
     const { items } = await listKnowledge({ parentId });
     return await Promise.all(
       items.map(async (node) => ({
         ...node,
-        children: node.kind === "folder" ? await loadChildren(node.id) : [],
+        children:
+          node.kind === "folder" && !seen.has(node.id)
+            ? await loadChildren(node.id, depth + 1, new Set([...seen, node.id]))
+            : [],
       })),
     );
   }
@@ -294,26 +305,6 @@ export function createIntelDataProvider(
         body: JSON.stringify(parsed),
       });
     },
-    async listToolSources() {
-      return await request("/tool-sources", ToolSourceList);
-    },
-    async createToolSource(input) {
-      return await request("/tool-sources", ToolSource, {
-        method: "POST",
-        body: JSON.stringify(CreateToolSourceInput.parse(input)),
-      });
-    },
-    async discoverToolSource(input) {
-      const parsed = DiscoverToolSourceInput.parse(input);
-      return await request(
-        `/tool-sources/${encodeURIComponent(parsed.sourceId)}/discover`,
-        ToolCatalog,
-        {
-          method: "POST",
-          body: JSON.stringify(parsed),
-        },
-      );
-    },
     async listTools() {
       return await request("/tools", ToolCatalog);
     },
@@ -324,8 +315,8 @@ export function createIntelDataProvider(
       });
     },
 
-    toolSourceConnectUrl(sourceId, returnTo = "/tools") {
-      return `${baseUrl}/auth/connect/${encodeURIComponent(sourceId)}?returnTo=${encodeURIComponent(returnTo)}`;
+    portalConnectUrl(returnTo = "/tools") {
+      return `${baseUrl}/auth/connect?returnTo=${encodeURIComponent(returnTo)}`;
     },
     async logout() {
       const response = await doFetch(`${baseUrl}/auth/logout`, {

package/src/data/intel-data-provider/intel-data-provider.types.ts

@@ -4,9 +4,7 @@ import type {
   CreateFlowInput,
   CreateKnowledgeLinkInput,
   CreateKnowledgeNodeInput,
-  CreateToolSourceInput,
   DeleteKnowledgeLinkInput,
-  DiscoverToolSourceInput,
   Flow,
   FlowDocument,
   FlowList,
@@ -35,8 +33,6 @@ import type {
   StartFlowRunInput,
   TestToolInput,
   ToolCatalog,
-  ToolSource,
-  ToolSourceList,
   ToolTestResult,
   UpdateKnowledgeNodeInput,
 } from "@anchrd/intel-contract";
@@ -57,7 +53,7 @@ export interface IntelDataProvider {
   saveKnowledge(input: SaveKnowledgeVersionInput): Promise<KnowledgeDocument>;
   saveKnowledgeAttachment(input: SaveKnowledgeAttachmentInput): Promise<KnowledgeDocument>;
   getKnowledgeAttachment(nodeId: string): Promise<Blob>;
-  toolSourceConnectUrl(sourceId: string, returnTo?: string): string;
+  portalConnectUrl(returnTo?: string): string;
   listKnowledgeVersions(nodeId: string): Promise<KnowledgeVersionList>;
   updateKnowledge(input: UpdateKnowledgeNodeInput): Promise<KnowledgeNode>;
   archiveKnowledge(input: ArchiveKnowledgeNodeInput): Promise<KnowledgeNode>;
@@ -76,9 +72,6 @@ export interface IntelDataProvider {
   startFlow(input: StartFlowRunInput): Promise<FlowRunStep>;
   getFlowRun(runId: string): Promise<FlowRunStep>;
   completeFlowStep(input: CompleteFlowRunStepInput): Promise<FlowRunStep>;
-  listToolSources(): Promise<ToolSourceList>;
-  createToolSource(input: CreateToolSourceInput): Promise<ToolSource>;
-  discoverToolSource(input: DiscoverToolSourceInput): Promise<ToolCatalog>;
   listTools(): Promise<ToolCatalog>;
   testTool(input: TestToolInput): Promise<ToolTestResult>;
   logout(): Promise<void>;

package/src/flows/flows.tsx

@@ -198,7 +198,6 @@ function newNode(
         ...common,
         kind,
         configuration: {
-          sourceId: "select-a-source",
           toolName: "select-a-tool",
           fingerprint: null,
           arguments: {},
@@ -599,10 +598,7 @@ function NodeInspector({
 }: {
   node: CanvasNode | null;
   update(fn: (node: FlowNode) => FlowNode): void;
-  tools: Array<{
-    source: { id: string; name: string };
-    capability: { name: string; title: string | null; fingerprint: string };
-  }>;
+  tools: Array<{ name: string; title: string | null; fingerprint: string }>;
   knowledge: KnowledgeTreeNode[];
 }) {
   const { i18n } = useIntelRouterContext();
@@ -726,23 +722,18 @@ function NodeInspector({
           <label className="block text-sm font-medium">
             {i18n.t("flows.tool")}
             <select
-              value={`${contract.configuration.sourceId}:${contract.configuration.toolName}`}
+              value={contract.configuration.toolName}
               onChange={(event) => {
-                const [sourceId, ...name] = event.target.value.split(":");
-                const toolName = name.join(":");
+                const toolName = event.target.value;
                 update((value) =>
-                  value.kind === "tool" && sourceId
+                  value.kind === "tool"
                     ? {
                         ...value,
                         configuration: {
                           ...value.configuration,
-                          sourceId,
                           toolName,
                           fingerprint:
-                            tools.find(
-                              (entry) =>
-                                entry.source.id === sourceId && entry.capability.name === toolName,
-                            )?.capability.fingerprint ?? null,
+                            tools.find((entry) => entry.name === toolName)?.fingerprint ?? null,
                         },
                       }
                     : value,
@@ -750,13 +741,10 @@ function NodeInspector({
               }}
               className="mt-2 w-full rounded-md border bg-background px-3 py-2 outline-none focus-visible:ring-2 focus-visible:ring-ring"
             >
-              <option value="select-a-source:select-a-tool">{i18n.t("flows.selectTool")}</option>
+              <option value="select-a-tool">{i18n.t("flows.selectTool")}</option>
               {tools.map((entry) => (
-                <option
-                  key={`${entry.source.id}:${entry.capability.name}`}
-                  value={`${entry.source.id}:${entry.capability.name}`}
-                >
-                  {entry.source.name} · {entry.capability.title ?? entry.capability.name}
+                <option key={entry.name} value={entry.name}>
+                  {entry.title ?? entry.name}
                 </option>
               ))}
             </select>

package/src/i18n/en.json

@@ -57,30 +57,23 @@
   "knowledge.saveConflict": "This document changed elsewhere. Reload it before saving again.",
   "knowledge.saveError": "This document could not be saved. Reload and try again.",
   "tools.title": "Company tools",
-  "tools.description": "Governed MCP capabilities available to people and AI.",
-  "tools.empty": "No MCP source registered",
-  "tools.emptyHelp": "Register the aggregated Cloudflare Portal endpoint. Intel discovers its tools while Gate remains the credential and permission boundary.",
-  "tools.addSource": "Add MCP source",
-  "tools.endpoint": "Streamable HTTP endpoint",
-  "tools.connectionHandle": "Gate connection handle",
-  "tools.connect": "Connect with Gate",
-  "tools.sourceHelp": "Intel registers the handle/connect interface in Gate. Grant it to the right roles and keep the credential in Gate.",
-  "tools.discover": "Discover tools",
-  "tools.notDiscovered": "No tools cached yet. Connect the source in Gate, then run discovery.",
+  "tools.description": "The MCP tools you can reach through the company portal.",
+  "tools.connect": "Connect the portal",
+  "tools.reconnect": "Reconnect",
+  "tools.disconnected": "Portal not connected",
+  "tools.disconnectedHelp": "Sign in to the company MCP portal once. The portal decides which servers you may use and holds their credentials; Intel never sees them.",
+  "tools.empty": "No tools available to you",
+  "tools.emptyHelp": "The portal returned no tools for your account. An administrator configures which servers it offers.",
   "tools.testTitle": "Tool playground",
-  "tools.select": "Select a discovered tool to inspect and test it.",
+  "tools.select": "Select a tool to inspect and test it.",
   "tools.arguments": "Arguments (JSON object)",
   "tools.runTest": "Run controlled test",
   "tools.invalidJson": "Arguments must be a valid JSON object.",
-  "tools.destructive": "This source marks the tool as potentially destructive. Review the arguments before running it.",
-  "tools.connectionRequired": "Your Gate connection is missing, forbidden, or invalid.",
+  "tools.destructive": "The portal marks this tool as potentially destructive. Review the arguments before running it.",
+  "tools.destructiveShort": "Destructive",
   "tools.testUnsafe": "The playground only runs tools explicitly marked read-only and non-destructive.",
-  "tools.operationFailed": "The MCP operation failed. Check the source and try again.",
-  "tools.connectFailed": "The MCP source could not be connected with your current access.",
-  "tools.status.connected": "Connected",
-  "tools.status.missing": "Not connected",
-  "tools.status.forbidden": "No access",
-  "tools.status.invalid": "Invalid",
+  "tools.operationFailed": "The tool list could not be loaded. Reconnect the portal and try again.",
+  "tools.connectFailed": "The portal could not be connected with your current access.",
   "flows.title": "Company flows",
   "flows.description": "Model-agnostic processes that connect instructions, Knowledge, tools, and people.",
   "flows.new": "New flow",

package/src/tools/tools.tsx

@@ -1,56 +1,17 @@
-import type { ToolCatalogEntry } from "@anchrd/intel-contract";
-import { useMutation, useQuery, useQueryClient } from "@tanstack/react-query";
-import {
-  AlertTriangle,
-  CheckCircle2,
-  CircleOff,
-  KeyRound,
-  LogIn,
-  Play,
-  Plus,
-  RefreshCw,
-  ShieldAlert,
-  Wrench,
-} from "lucide-react";
-import { useMemo, useState } from "react";
-import { Modal } from "@/modal/modal.tsx";
+import type { ToolCapability } from "@anchrd/intel-contract";
+import { useMutation, useQuery } from "@tanstack/react-query";
+import { AlertTriangle, LogIn, Play, ShieldAlert, Wrench } from "lucide-react";
+import { useState } from "react";
 import { useIntelRouterContext } from "@/router/router-context.ts";
 
-const statusIcon = {
-  connected: CheckCircle2,
-  missing: KeyRound,
-  forbidden: ShieldAlert,
-  invalid: CircleOff,
-} as const;
-
 export function Tools() {
   const { data, i18n } = useIntelRouterContext();
-  const queryClient = useQueryClient();
-  const sources = useQuery({ queryKey: ["tool-sources"], queryFn: () => data.listToolSources() });
   const catalog = useQuery({ queryKey: ["tools"], queryFn: () => data.listTools() });
-  const [registering, setRegistering] = useState(false);
-  const [selected, setSelected] = useState<ToolCatalogEntry | null>(null);
+  const [selected, setSelected] = useState<ToolCapability | null>(null);
   const connectError =
     typeof window === "undefined"
       ? null
       : new URLSearchParams(window.location.search).get("connectError");
-  const grouped = useMemo(() => {
-    const entries = catalog.data?.items ?? [];
-    return (sources.data?.items ?? []).map((source) => ({
-      source,
-      entries: entries.filter((entry) => entry.source.id === source.id),
-    }));
-  }, [catalog.data, sources.data]);
-  const discover = useMutation({
-    mutationFn: (sourceId: string) =>
-      data.discoverToolSource({ sourceId, idempotencyKey: crypto.randomUUID() }),
-    onSuccess: async () => {
-      await Promise.all([
-        queryClient.invalidateQueries({ queryKey: ["tool-sources"] }),
-        queryClient.invalidateQueries({ queryKey: ["tools"] }),
-      ]);
-    },
-  });
 
   return (
     <div className="min-h-screen">
@@ -59,26 +20,19 @@ export function Tools() {
           <h1 className="text-xl font-semibold tracking-tight">{i18n.t("tools.title")}</h1>
           <p className="mt-1 text-sm text-muted-foreground">{i18n.t("tools.description")}</p>
         </div>
-        <button
-          type="button"
-          onClick={() => setRegistering(true)}
-          className="inline-flex items-center gap-2 rounded-md bg-primary px-3 py-2 text-sm font-medium text-primary-foreground outline-none hover:bg-primary/90 focus-visible:ring-2 focus-visible:ring-ring"
-        >
-          <Plus aria-hidden="true" className="size-4" />
-          {i18n.t("tools.addSource")}
-        </button>
+        {catalog.data?.portalConnected && (
+          <a
+            href={data.portalConnectUrl("/tools")}
+            className="inline-flex items-center gap-2 rounded-md border bg-background px-3 py-2 text-sm outline-none hover:bg-muted focus-visible:ring-2 focus-visible:ring-ring"
+          >
+            <LogIn aria-hidden="true" className="size-4" />
+            {i18n.t("tools.reconnect")}
+          </a>
+        )}
       </header>
 
       <div className="grid gap-5 p-8 xl:grid-cols-[minmax(0,1fr)_24rem]">
         <div className="space-y-5">
-          {discover.isError && (
-            <p
-              role="alert"
-              className="rounded-md border border-destructive/30 p-3 text-sm text-destructive"
-            >
-              {i18n.t("tools.operationFailed")}
-            </p>
-          )}
           {connectError && (
             <p
               role="alert"
@@ -87,9 +41,28 @@ export function Tools() {
               {i18n.t("tools.connectFailed")}
             </p>
           )}
-          {sources.isPending || catalog.isPending ? (
+          {catalog.isPending ? (
             <p className="text-sm text-muted-foreground">{i18n.t("common.loading")}</p>
-          ) : grouped.length === 0 ? (
+          ) : catalog.isError ? (
+            <p role="alert" className="text-sm text-destructive">
+              {i18n.t("tools.operationFailed")}
+            </p>
+          ) : !catalog.data?.portalConnected ? (
+            <section className="rounded-xl border border-dashed bg-card p-12 text-center">
+              <Wrench aria-hidden="true" className="mx-auto size-8 text-muted-foreground" />
+              <h2 className="mt-4 font-semibold">{i18n.t("tools.disconnected")}</h2>
+              <p className="mx-auto mt-2 max-w-lg text-sm text-muted-foreground">
+                {i18n.t("tools.disconnectedHelp")}
+              </p>
+              <a
+                href={data.portalConnectUrl("/tools")}
+                className="mt-5 inline-flex items-center gap-2 rounded-md bg-primary px-4 py-2 text-sm font-medium text-primary-foreground outline-none hover:bg-primary/90 focus-visible:ring-2 focus-visible:ring-ring"
+              >
+                <LogIn aria-hidden="true" className="size-4" />
+                {i18n.t("tools.connect")}
+              </a>
+            </section>
+          ) : catalog.data.items.length === 0 ? (
             <section className="rounded-xl border border-dashed bg-card p-12 text-center">
               <Wrench aria-hidden="true" className="mx-auto size-8 text-muted-foreground" />
               <h2 className="mt-4 font-semibold">{i18n.t("tools.empty")}</h2>
@@ -98,120 +71,81 @@ export function Tools() {
               </p>
             </section>
           ) : (
-            grouped.map(({ source, entries }) => (
-              <section
-                key={source.id}
-                className="overflow-hidden rounded-xl border bg-card shadow-sm"
-              >
-                <div className="flex items-start justify-between gap-5 border-b bg-muted/20 px-5 py-4">
-                  <div className="min-w-0">
-                    <h2 className="font-semibold">{source.name}</h2>
-                    <p className="mt-1 truncate text-xs text-muted-foreground">{source.url}</p>
-                    <p className="mt-2 text-xs text-muted-foreground">
-                      {i18n.t("tools.connectionHandle")}: {source.connectionHandle}
-                    </p>
-                  </div>
-                  <div className="flex shrink-0 items-center gap-2">
-                    <a
-                      href={data.toolSourceConnectUrl(source.id, "/tools")}
-                      className="inline-flex items-center gap-2 rounded-md border bg-background px-3 py-2 text-xs font-medium outline-none hover:bg-muted focus-visible:ring-2 focus-visible:ring-ring"
-                    >
-                      <LogIn aria-hidden="true" className="size-3.5" />
-                      {i18n.t("tools.connect")}
-                    </a>
+            <section className="overflow-hidden rounded-xl border bg-card shadow-sm">
+              <ul className="divide-y">
+                {catalog.data.items.map((capability) => (
+                  <li key={capability.name}>
                     <button
                       type="button"
-                      disabled={discover.isPending && discover.variables === source.id}
-                      onClick={() => discover.mutate(source.id)}
-                      className="inline-flex items-center gap-2 rounded-md border bg-background px-3 py-2 text-xs font-medium outline-none hover:bg-muted focus-visible:ring-2 focus-visible:ring-ring disabled:opacity-50"
+                      onClick={() => setSelected(capability)}
+                      className="flex w-full items-start justify-between gap-5 px-5 py-4 text-left outline-none hover:bg-muted/50 focus-visible:ring-2 focus-visible:ring-inset focus-visible:ring-ring"
                     >
-                      <RefreshCw aria-hidden="true" className="size-3.5" />
-                      {i18n.t("tools.discover")}
+                      <span className="min-w-0">
+                        <span className="block text-sm font-medium">
+                          {capability.title ?? capability.name}
+                        </span>
+                        <span className="mt-1 block truncate font-mono text-xs text-muted-foreground">
+                          {capability.name}
+                        </span>
+                        {capability.description && (
+                          <span className="mt-2 line-clamp-2 block text-xs text-muted-foreground">
+                            {capability.description}
+                          </span>
+                        )}
+                      </span>
+                      {capability.annotations.destructiveHint && (
+                        <span className="inline-flex shrink-0 items-center gap-1.5 rounded-full bg-destructive/10 px-2 py-1 text-xs text-destructive">
+                          <AlertTriangle aria-hidden="true" className="size-3.5" />
+                          {i18n.t("tools.destructiveShort")}
+                        </span>
+                      )}
                     </button>
-                  </div>
-                </div>
-                {entries.length === 0 ? (
-                  <p className="p-5 text-sm text-muted-foreground">
-                    {i18n.t("tools.notDiscovered")}
-                  </p>
-                ) : (
-                  <ul className="divide-y">
-                    {entries.map((entry) => {
-                      const Status = statusIcon[entry.connectionStatus];
-                      return (
-                        <li key={entry.capability.name}>
-                          <button
-                            type="button"
-                            onClick={() => setSelected(entry)}
-                            className="flex w-full items-start justify-between gap-5 px-5 py-4 text-left outline-none hover:bg-muted/50 focus-visible:ring-2 focus-visible:ring-inset focus-visible:ring-ring"
-                          >
-                            <span className="min-w-0">
-                              <span className="block text-sm font-medium">
-                                {entry.capability.title ?? entry.capability.name}
-                              </span>
-                              <span className="mt-1 block truncate font-mono text-xs text-muted-foreground">
-                                {entry.capability.name}
-                              </span>
-                              {entry.capability.description && (
-                                <span className="mt-2 line-clamp-2 block text-xs text-muted-foreground">
-                                  {entry.capability.description}
-                                </span>
-                              )}
-                            </span>
-                            <span className="inline-flex shrink-0 items-center gap-1.5 rounded-full bg-muted px-2 py-1 text-xs text-muted-foreground">
-                              <Status aria-hidden="true" className="size-3.5" />
-                              {i18n.t(`tools.status.${entry.connectionStatus}`)}
-                            </span>
-                          </button>
-                        </li>
-                      );
-                    })}
-                  </ul>
-                )}
-              </section>
-            ))
+                  </li>
+                ))}
+              </ul>
+            </section>
           )}
         </div>
-        <ToolTester entry={selected} />
+        <ToolTester capability={selected} />
       </div>
-      {registering && <RegisterSource close={() => setRegistering(false)} />}
     </div>
   );
 }
 
-function ToolTester({ entry }: { entry: ToolCatalogEntry | null }) {
+function ToolTester({ capability }: { capability: ToolCapability | null }) {
   const { data, i18n } = useIntelRouterContext();
   const [argumentsText, setArgumentsText] = useState("{}");
   const [parseError, setParseError] = useState(false);
   const safeToTest =
-    entry?.capability.annotations.readOnlyHint === true &&
-    entry.capability.annotations.destructiveHint !== true;
+    capability?.annotations.readOnlyHint === true &&
+    capability.annotations.destructiveHint !== true;
   const test = useMutation({
     mutationFn: async () => {
       const parsed: unknown = JSON.parse(argumentsText);
-      if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) throw new Error("object");
+      if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
+        throw new SyntaxError("Arguments must be a JSON object");
+      }
       return await data.testTool({
-        sourceId: entry?.source.id ?? "",
-        name: entry?.capability.name ?? "",
+        name: capability?.name ?? "",
         arguments: parsed as Record<string, unknown>,
       });
     },
     onMutate: () => setParseError(false),
-    onError: (error) => setParseError(error instanceof SyntaxError || error.message === "object"),
+    onError: (error) => setParseError(error instanceof SyntaxError),
   });
 
   return (
     <aside className="sticky top-5 h-fit rounded-xl border bg-card p-5 shadow-sm">
       <h2 className="font-semibold">{i18n.t("tools.testTitle")}</h2>
-      {!entry ? (
+      {!capability ? (
         <p className="mt-3 text-sm text-muted-foreground">{i18n.t("tools.select")}</p>
       ) : (
         <div className="mt-4 space-y-4">
           <div>
-            <p className="text-sm font-medium">{entry.capability.title ?? entry.capability.name}</p>
-            <p className="mt-1 font-mono text-xs text-muted-foreground">{entry.capability.name}</p>
+            <p className="text-sm font-medium">{capability.title ?? capability.name}</p>
+            <p className="mt-1 font-mono text-xs text-muted-foreground">{capability.name}</p>
           </div>
-          {entry.capability.annotations.destructiveHint && (
+          {capability.annotations.destructiveHint && (
             <p className="flex gap-2 rounded-md bg-destructive/10 p-3 text-xs text-destructive">
               <AlertTriangle aria-hidden="true" className="size-4 shrink-0" />
               {i18n.t("tools.destructive")}
@@ -223,11 +157,6 @@ function ToolTester({ entry }: { entry: ToolCatalogEntry | null }) {
               {i18n.t("tools.testUnsafe")}
             </p>
           )}
-          {entry.connectionStatus !== "connected" && (
-            <p className="rounded-md bg-muted p-3 text-xs text-muted-foreground">
-              {i18n.t("tools.connectionRequired")}
-            </p>
-          )}
           <label className="block text-xs font-medium">
             {i18n.t("tools.arguments")}
             <textarea
@@ -241,7 +170,7 @@ function ToolTester({ entry }: { entry: ToolCatalogEntry | null }) {
           {parseError && <p className="text-xs text-destructive">{i18n.t("tools.invalidJson")}</p>}
           <button
             type="button"
-            disabled={test.isPending || entry.connectionStatus !== "connected" || !safeToTest}
+            disabled={test.isPending || !safeToTest}
             onClick={() => test.mutate()}
             className="inline-flex w-full items-center justify-center gap-2 rounded-md bg-primary px-3 py-2 text-sm font-medium text-primary-foreground outline-none hover:bg-primary/90 focus-visible:ring-2 focus-visible:ring-ring disabled:opacity-50"
           >
@@ -261,86 +190,3 @@ function ToolTester({ entry }: { entry: ToolCatalogEntry | null }) {
     </aside>
   );
 }
-
-function RegisterSource({ close }: { close(): void }) {
-  const { data, i18n } = useIntelRouterContext();
-  const queryClient = useQueryClient();
-  const [name, setName] = useState("");
-  const [url, setUrl] = useState("");
-  const [connectionHandle, setConnectionHandle] = useState("intel-tools");
-  const create = useMutation({
-    mutationFn: () =>
-      data.createToolSource({
-        name,
-        url,
-        connectionHandle,
-        idempotencyKey: crypto.randomUUID(),
-      }),
-    onSuccess: async () => {
-      await queryClient.invalidateQueries({ queryKey: ["tool-sources"] });
-      close();
-    },
-  });
-  return (
-    <Modal title={i18n.t("tools.addSource")} close={close} className="max-w-lg">
-      <form
-        className="space-y-4"
-        onSubmit={(event) => {
-          event.preventDefault();
-          if (create.isPending) return;
-          create.mutate();
-        }}
-      >
-        <Field label={i18n.t("common.title")} value={name} setValue={setName} />
-        <Field label={i18n.t("tools.endpoint")} value={url} setValue={setUrl} type="url" />
-        <Field
-          label={i18n.t("tools.connectionHandle")}
-          value={connectionHandle}
-          setValue={setConnectionHandle}
-          pattern="[a-z0-9][a-z0-9-]{1,62}"
-        />
-        <p className="text-xs text-muted-foreground">{i18n.t("tools.sourceHelp")}</p>
-        {create.isError && (
-          <p role="alert" className="text-sm text-destructive">
-            {i18n.t("tools.operationFailed")}
-          </p>
-        )}
-        <button
-          type="submit"
-          disabled={create.isPending}
-          className="w-full rounded-md bg-primary px-4 py-2 text-sm font-medium text-primary-foreground outline-none hover:bg-primary/90 focus-visible:ring-2 focus-visible:ring-ring disabled:opacity-50"
-        >
-          {create.isPending ? i18n.t("common.saving") : i18n.t("common.create")}
-        </button>
-      </form>
-    </Modal>
-  );
-}
-
-function Field({
-  label,
-  value,
-  setValue,
-  type = "text",
-  pattern,
-}: {
-  label: string;
-  value: string;
-  setValue(value: string): void;
-  type?: string;
-  pattern?: string;
-}) {
-  return (
-    <label className="block text-sm font-medium">
-      {label}
-      <input
-        required
-        type={type}
-        pattern={pattern}
-        value={value}
-        onChange={(event) => setValue(event.target.value)}
-        className="mt-2 w-full rounded-md border bg-background px-3 py-2 outline-none focus-visible:ring-2 focus-visible:ring-ring"
-      />
-    </label>
-  );
-}