inkeep
diff --git a/‎.changeset/remove-launch-json-pane-arming.md‎
Lines changed: 11 additions & 0 deletions b/‎.changeset/remove-launch-json-pane-arming.md‎
Lines changed: 11 additions & 0 deletions
diff --git a/‎docs/content/reference/mcp.mdx‎
Lines changed: 2 additions & 2 deletions b/‎docs/content/reference/mcp.mdx‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎docs/content/reference/what-open-knowledge-writes.mdx‎
Lines changed: 2 additions & 4 deletions b/‎docs/content/reference/what-open-knowledge-writes.mdx‎
Lines changed: 2 additions & 4 deletions
diff --git a/‎packages/app/src/App.dom.test.tsx‎
Lines changed: 0 additions & 70 deletions b/‎packages/app/src/App.dom.test.tsx‎
Lines changed: 0 additions & 70 deletions
diff --git a/‎packages/app/src/App.tsx‎
Lines changed: 0 additions & 48 deletions b/‎packages/app/src/App.tsx‎
Lines changed: 0 additions & 48 deletions
diff --git a/‎packages/app/src/components/ConfigSharingInfoTooltip.tsx‎
Lines changed: 1 addition & 1 deletion b/‎packages/app/src/components/ConfigSharingInfoTooltip.tsx‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎packages/app/src/components/ConsentDialogBody.tsx‎
Lines changed: 1 addition & 4 deletions b/‎packages/app/src/components/ConsentDialogBody.tsx‎
Lines changed: 1 addition & 4 deletions
diff --git a/‎packages/app/src/lib/api-config.test.ts‎
Lines changed: 3 additions & 30 deletions b/‎packages/app/src/lib/api-config.test.ts‎
Lines changed: 3 additions & 30 deletions
@@ -0,0 +1,11 @@
+---
+"@inkeep/open-knowledge": minor
+"@inkeep/open-knowledge-core": minor
+"@inkeep/open-knowledge-server": minor
+"@inkeep/open-knowledge-app": minor
+"@inkeep/open-knowledge-desktop": minor
+---
+
+Remove the launch.json preview machinery and the pane-target arming subsystem. Claude Code Desktop's in-app Browser pane now opens the preview URL directly (`preview_start({url})` + `navigate({url})`), so OK no longer scaffolds `.claude/launch.json`.
+
+Breaking for consumers: the `@inkeep/open-knowledge` package drops the `scaffoldLaunchJson`, `LAUNCH_UI_CHAIN_*`, and `LaunchJsonResult` exports; the `preview_url` MCP tool drops the `armPaneTarget` param; and the `/api/config` response drops the `paneTarget` field. The `ok start` repair sweep now removes any pre-existing `open-knowledge-ui` launch.json entry instead of rewriting it.
@@ -21,7 +21,7 @@ The surface is **19 tools**. The four write verbs — `write`, `edit`, `delete`,
 | `skills` | Find and read agent skills. Omit `name` to list every skill across Project + Global (rows carry `installed` + `hosts`); pass `name` to read one (addressed by `name`+`scope`, never by path); pass `name` + `file` to read one bundle file's text — the read path for `references/` and `scripts/` content |
 | `config` | Read the effective merged config (`config({ key: "appearance.theme" })` for a sub-tree; omit `key` for the whole config) |
 | `palette` | Markdown-native authoring forms, themed `html preview` embed starters, and theme tokens. Pass `components` for the canonical components' full JSX prop schemas |
-| `preview_url` | Resolve the browser-reachable preview URL. Targets a `document`, `folder`, or `skill` in the current project, or — via `file`, an absolute path — a markdown file outside it (resolved through the session serving that file). Per-response `previewUrl` fields elsewhere are route-only (`/#/<doc>`); call this when you need the full openable URL. `armPaneTarget` briefly arms the target so a host preview pane opens on it |
+| `preview_url` | Resolve the browser-reachable preview URL. Targets a `document`, `folder`, or `skill` in the current project, or — via `file`, an absolute path — a markdown file outside it (resolved through the session serving that file). Per-response `previewUrl` fields elsewhere are route-only (`/#/<doc>`); call this when you need the full openable URL |
 | `share_link` | Build a GitHub-substrate share URL for a doc or folder to send to a teammate (`kind` disambiguates; `path: ""` with `kind: "folder"` shares the content root). Responses carry a `freshness` verdict — `current`, `stale` (unpushed changes), or `absent` (never pushed), omitted when the probe can't run — and prepend a matching warning to relay with the link. Read-only against `.git/`; never publishes |
 | `write` | Create or overwrite a `document`, `folder`, `template`, `skill`, or `asset`. For a document: `{ path, content }`, or `{ path, template }` to instantiate from a folder template. `position` selects `replace` (full rewrite; the only mode that touches frontmatter), `append`, or `prepend`. `extension` selects `.md` (default) or `.mdx` for a new document; an existing doc keeps its extension (the MCP cannot change it). An `asset` takes exactly one of `content` (base64, for small files) or `source` (a local path the server reads, for large files) |
 | `edit` | Modify a `document` (body find/replace **or** a frontmatter merge-patch), a `folder` (frontmatter merge-patch), a `template`, or a `skill`. Body edits take `{ path, find, replace, occurrence? }`; `occurrence` selects which match (1 = first) |
@@ -73,6 +73,6 @@ To detect conflict state proactively without waiting for a 409, call `conflicts(
 
 ## Preview links
 
-Read and write tools return a `previewUrl` for any doc they touched, but **those are route-only** (`/#/<doc>` with no host:port), meant to identify the doc, not to be opened as-is. Hosts that want to open the editor (Claude Code Desktop's `preview_start`, terminal `open <url>`) call `preview_url` to resolve the full URL once per session.
+Read and write tools return a `previewUrl` for any doc they touched, but **those are route-only** (`/#/<doc>` with no host:port), meant to identify the doc, not to be opened as-is. Hosts that want to open the editor (Claude Code Desktop's `preview_start({url})`, terminal `open <url>`) call `preview_url` to resolve the full URL once per session.
 
 `preview_url` always carries an `autoOpen` boolean (top-level on its response) reflecting the user's [`appearance.preview.autoOpen`](/docs/reference/configuration) preference. The write tools (`write`, `edit`) carry the same boolean on the `warning` object, which fires only when no browser is attached to the preview. In the common case (a browser is attached), the write response has no `warning` and no `autoOpen` field. Agents honor `autoOpen` before navigating: `true` (default) follows the host-capability routing in the bundled skill; `false` means the user is managing their own preview window, so the agent surfaces the URL on request but does not open or refresh anything. The value is resolved fresh on every tool call, so a mid-session toggle propagates within 0–1 calls, with no client restart.
@@ -87,7 +87,6 @@ When you initialize or open a project, the app scaffolds the same project files
 | `.pi/extensions/open-knowledge.ts` | Managed bridge extension connecting [Pi](/docs/integrations/pi) (which has no MCP support) to the project | In-project, committed |
 | `.claude/skills/`, `.cursor/skills/`, `.codex/skills/`, `.opencode/skills/`, `.pi/skills/` | Project-local "rich" OpenKnowledge skill (`SKILL.md`) for the same five editors | In-project, committed |
 | `.ok/skill-targets.json` | Committed list of which editors receive skills, written when you customize the targets | In-project, committed |
-| `.claude/launch.json` | Configures Claude Code's preview pane to open the editor (on a local UI port, `39848`) | In-project, committed |
 | `~/.claude.json`, `~/Library/Application Support/Claude/claude_desktop_config.json`, `~/.cursor/mcp.json`, `~/.codex/config.toml`, `~/.config/opencode/opencode.json`, `~/.openclaw/openclaw.json`, `~/.gemini/config/mcp_config.json`, `~/.lmstudio/mcp.json`, `~/.hermes/config.yaml` | User-level `open-knowledge` MCP registration, one per detected editor, so it's available in every project (`~/.openclaw/openclaw.json` only when `~/.openclaw/` exists; `~/.gemini/config/mcp_config.json` is Antigravity's shared IDE + `agy` config, only when `~/.gemini/` exists; `~/.lmstudio/mcp.json` and `~/.hermes/config.yaml` only when those apps are installed) | Outside-project (editor config) |
 
 ### While a project is open
@@ -109,7 +108,7 @@ The app runs a collaboration server for the open project. It writes runtime stat
 
 Diagnostic logs and telemetry are **on by default but local-only** — credential-bearing attributes are redacted (`[REDACTED]`) before anything is written, the files rotate at the size caps above, and nothing leaves the machine until you explicitly run `ok diagnose bundle`. Turn the local sink off with `telemetry.localSink.enabled: false`; see [Configuration](/docs/reference/configuration).
 
-On each launch and project open, the app also runs **repair sweeps**: it rewrites *existing* `open-knowledge` MCP entries (in the editor configs above), `.claude/launch.json`, and the OpenKnowledge skill bundles (`open-knowledge-discovery` and `open-knowledge-write-skill`, in `~/.agents/skills/` and each detected editor's `~/.claude/skills/`, `~/.cursor/skills/`, `~/.codex/skills/`, `~/.opencode/skills/`) to the current canonical form — a no-op when nothing has drifted. It **never adds** an MCP entry to an editor that doesn't already have one; and when you turn on skill management for a project, it also adopts your existing editor skills into `.ok/skills/`. Same `OK_RECLAIM_DISABLE=1` opt-out.
+On each launch and project open, the app also runs **repair sweeps**: it rewrites *existing* `open-knowledge` MCP entries (in the editor configs above) and the OpenKnowledge skill bundles (`open-knowledge-discovery` and `open-knowledge-write-skill`, in `~/.agents/skills/` and each detected editor's `~/.claude/skills/`, `~/.cursor/skills/`, `~/.codex/skills/`, `~/.opencode/skills/`) to the current canonical form — a no-op when nothing has drifted. It also removes any stale `open-knowledge-ui` entry left in `.claude/launch.json` by an older OK version. It **never adds** an MCP entry to an editor that doesn't already have one; and when you turn on skill management for a project, it also adopts your existing editor skills into `.ok/skills/`. Same `OK_RECLAIM_DISABLE=1` opt-out.
 
 ### Home directory and credentials (`~/.ok/`)
 
@@ -165,7 +164,6 @@ The `postinstall` step is **non-fatal** (install always succeeds even if it fail
 | `.pi/extensions/open-knowledge.ts` | Managed bridge extension connecting [Pi](/docs/integrations/pi) (which has no MCP support) to the project | In-project, committed |
 | `.claude/skills/`, `.cursor/skills/`, `.codex/skills/`, `.opencode/skills/`, `.pi/skills/` | Project-local "rich" OpenKnowledge skill (`SKILL.md`) for the same five editors | In-project, committed |
 | `.ok/skill-targets.json` | Committed list of which editors receive skills, written when you customize the targets | In-project, committed |
-| `.claude/launch.json` | Configures Claude Code's preview pane to open the editor (on a local UI port, `39848`) | In-project, committed |
 
 With `--scope user` or `both` (the default), `ok init` also registers the `open-knowledge` MCP server in each detected editor's **user-level** config, so it's available in every project:
 
@@ -217,8 +215,8 @@ On each boot, `ok start` also runs **repair sweeps**: it re-checks a few things
 | What it re-checks | Where | Scope |
 | --- | --- | --- |
 | Existing `open-knowledge` MCP entries | The same editor configs `ok init` writes — **both** user-level (`~/.claude.json`, `~/.cursor/mcp.json`, …) and project-level | Outside-project (editor config) **and** in-project |
-| `.claude/launch.json` | Your project | In-project, committed |
 | OpenKnowledge skill bundles (`open-knowledge-discovery`, `open-knowledge-write-skill`) | Project (`.claude/skills/…`) and user-global — `~/.agents/skills/` plus each detected editor's `~/.claude/skills/`, `~/.cursor/skills/`, `~/.codex/skills/`, `~/.opencode/skills/` | In-project **and** outside-project (home dir) |
+| Stale `open-knowledge-ui` entry in `.claude/launch.json` | Project | Removed if present — OK no longer writes launch.json |
 
 Disable all three with `OK_RECLAIM_DISABLE=1`.
 
 
@@ -29,7 +29,6 @@ let fetchApiConfigMock = mock(() =>
       collabUrl: null,
       previewUrl: null,
       port: 0,
-      paneTarget: null,
       singleFile: false,
     },
   }),
@@ -261,7 +260,6 @@ describe('App runtime wiring', () => {
           collabUrl: null,
           previewUrl: null,
           port: 0,
-          paneTarget: null,
           singleFile: false,
         },
       }),
@@ -365,74 +363,6 @@ describe('App runtime wiring', () => {
     expect(resolveNavigationTargetMock).not.toHaveBeenCalled();
   });
 
-  test('base-open pane target applies a well-formed config route once and consumes it', async () => {
-    fetchApiConfigMock = mock(() =>
-      Promise.resolve({
-        status: 'ok' as const,
-        config: {
-          collabUrl: null,
-          previewUrl: null,
-          port: 0,
-          paneTarget: '#/docs/pane-target',
-          singleFile: false,
-        },
-      }),
-    );
-
-    renderApp();
-
-    await waitFor(() => {
-      expect(window.location.hash).toBe('#/docs/pane-target');
-    });
-    expect(fetchApiConfigMock).toHaveBeenCalledTimes(1);
-    expect(globalThis.fetch).toHaveBeenCalledWith('/api/config', { method: 'DELETE' });
-  });
-
-  test('base-open pane target ignores malformed and direct-navigation routes', async () => {
-    fetchApiConfigMock = mock(() =>
-      Promise.resolve({
-        status: 'ok' as const,
-        config: {
-          collabUrl: null,
-          previewUrl: null,
-          port: 0,
-          paneTarget: 'https://example.invalid/docs/readme',
-          singleFile: false,
-        },
-      }),
-    );
-
-    renderApp();
-
-    await waitFor(() => {
-      expect(fetchApiConfigMock).toHaveBeenCalledTimes(1);
-    });
-    expect(window.location.hash).toBe('');
-    expect(globalThis.fetch).not.toHaveBeenCalled();
-
-    cleanup();
-    fetchApiConfigMock = mock(() =>
-      Promise.resolve({
-        status: 'ok' as const,
-        config: {
-          collabUrl: null,
-          previewUrl: null,
-          port: 0,
-          paneTarget: '#/docs/pane-target',
-          singleFile: false,
-        },
-      }),
-    );
-    setHash('#/docs/already-open');
-
-    renderApp();
-
-    await waitFor(() => {
-      expect(fetchApiConfigMock).not.toHaveBeenCalled();
-    });
-    expect(window.location.hash).toBe('#/docs/already-open');
-  });
-
   test('active doc and folder targets are pushed to the desktop bridge', async () => {
     const bridge = createBridge();
     activeTarget = { kind: 'doc', target: 'docs/readme', docName: 'docs/readme' };
 
@@ -31,7 +31,6 @@ import {
 } from '@/editor/DocumentContext';
 import { parseEditorTabId } from '@/editor/editor-tabs';
 import { useReconcileSkillTabs } from '@/hooks/use-reconcile-skill-tabs';
-import { fetchApiConfig } from '@/lib/api-config';
 import { ConfigProvider } from '@/lib/config-provider';
 import {
   assetPathFromHash,
@@ -248,52 +247,6 @@ function NavigationHandler() {
   return null;
 }
 
-/**
- * One-shot base-open deep-link: when the UI opens at its base (no doc / asset /
- * dialog in the hash — e.g. the Claude pane's `preview_start`), apply an armed
- * pane target from `/api/config` so the pane lands on the agent's intended
- * doc/folder instead of an empty splash. Fires once at mount; navigates only if
- * the page is still at base when the fetch resolves (so it never overrides a
- * deep link the user opened directly or navigated to meanwhile). When no target
- * is armed (or it expired), it does nothing and the normal root view shows.
- */
-function PaneTargetLanding() {
-  // No `didRun` ref: a ref set synchronously persists across React Strict
-  // Mode's dev double-invoke, which would make the first run's cleanup cancel
-  // the fetch and the second run early-return — so the effect would silently do
-  // nothing in dev. The `cancelled` flag + the `atBase` re-check already make
-  // re-running safe and idempotent (it only navigates from a base hash).
-  useEffect(() => {
-    const atBase = (hash: string) =>
-      !isAuxiliaryDialogHash(hash) &&
-      !assetPathFromHash(hash) &&
-      !docNameFromHash(hash) &&
-      (hash === '' || hash === '#' || hash === '#/');
-    if (!atBase(window.location.hash)) return;
-    const controller = new AbortController();
-    void fetchApiConfig(controller.signal)
-      .then((result) => {
-        if (controller.signal.aborted || result.status !== 'ok') return;
-        const target = result.config.paneTarget;
-        // Only apply a well-formed in-app route fragment, and only if we're still
-        // at base (the user hasn't navigated during the fetch).
-        if (!target?.startsWith('#/')) return;
-        if (!atBase(window.location.hash)) return;
-        window.location.hash = target;
-        // Consume the one-shot target so a reload within its TTL doesn't yank
-        // back here. Fire-and-forget — re-navigation is the only cost if it
-        // fails, and the server-side TTL caps that anyway.
-        void fetch('/api/config', { method: 'DELETE' }).catch(() => {});
-      })
-      // fetchApiConfig rethrows AbortError on unmount — expected, swallow it.
-      .catch(() => {});
-    // Aborting on unmount cancels the inflight request, not just its handler —
-    // symmetric with the rest of the app's /api/config consumers.
-    return () => controller.abort();
-  }, []);
-  return null;
-}
-
 /**
  * Mounts `InstallInClaudeDesktopDialog` at the App root and opens it when
  * `window.location.hash === '#install-claude-desktop'`. Docs and in-app CTAs
@@ -533,7 +486,6 @@ function AppBody() {
       <PageListProvider>
         <SystemDocSubscriber />
         <NavigationHandler />
-        <PaneTargetLanding />
         <ActiveTargetBridgePush />
         <NewItemShortcutHandler />
         {/* Settings is unavailable in single-file mode (config editing is
 
@@ -28,7 +28,7 @@ export function ConfigSharingInfoTooltip() {
           <p className="leading-relaxed wrap-break-word">
             <Trans>
               Setup files include: <code>.ok/</code>, AI-tool MCP configs (<code>.mcp.json</code>{' '}
-              and per-tool files), project skills, and <code>.claude/launch.json</code>.
+              and per-tool files), project skills.
               <br />
               <strong className="font-semibold">Shared</strong> commits them to git, so anyone who
               clones the repo gets the same setup. <br />
 
@@ -391,10 +391,7 @@ function ConsentDialogForm({ payload, store, toast }: ConsentDialogFormProps) {
                     <Trans>Connect to AI tools</Trans>
                   </legend>
                   <p className="text-1sm text-muted-foreground">
-                    <Trans>
-                      Writes a project-MCP config for each selected tool; Claude also gets{' '}
-                      <code>.claude/launch.json</code>.
-                    </Trans>
+                    <Trans>Writes a project-MCP config for each selected tool.</Trans>
                   </p>
                   {payload.editorOptions.map((editor) => {
                     const checkboxId = `consent-editor-${editor.id}-cb`;
 
@@ -36,7 +36,6 @@ describe('fetchApiConfig', () => {
         collabUrl: 'ws://localhost:52000/collab',
         previewUrl: 'http://localhost:3000/',
         port: 3000,
-        paneTarget: null,
         singleFile: false,
       },
     });
@@ -52,7 +51,7 @@ describe('fetchApiConfig', () => {
     const result = await fetchApiConfig();
     expect(result).toEqual({
       status: 'ok',
-      config: { collabUrl: null, previewUrl: null, port: 0, paneTarget: null, singleFile: false },
+      config: { collabUrl: null, previewUrl: null, port: 0, singleFile: false },
     });
   });
 
@@ -67,33 +66,7 @@ describe('fetchApiConfig', () => {
     const result = await fetchApiConfig();
     expect(result).toEqual({
       status: 'ok',
-      config: { collabUrl: null, previewUrl: null, port: 0, paneTarget: null, singleFile: true },
-    });
-  });
-
-  it('parses an armed paneTarget route fragment', async () => {
-    stubFetch(
-      async () =>
-        new Response(
-          JSON.stringify({
-            collabUrl: null,
-            previewUrl: null,
-            port: 0,
-            paneTarget: '#/specs/foo/',
-          }),
-          { status: 200 },
-        ),
-    );
-    const result = await fetchApiConfig();
-    expect(result).toEqual({
-      status: 'ok',
-      config: {
-        collabUrl: null,
-        previewUrl: null,
-        port: 0,
-        paneTarget: '#/specs/foo/',
-        singleFile: false,
-      },
+      config: { collabUrl: null, previewUrl: null, port: 0, singleFile: true },
     });
   });
 
@@ -139,7 +112,7 @@ describe('fetchApiConfig', () => {
     const result = await fetchApiConfig();
     expect(result).toEqual({
       status: 'ok',
-      config: { collabUrl: null, previewUrl: null, port: 0, paneTarget: null, singleFile: false },
+      config: { collabUrl: null, previewUrl: null, port: 0, singleFile: false },
     });
   });