dsh-codegraph-init 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -20,3 +20,23 @@ the project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html
   delegating to `codegraph install -t dsh` whenever the CLI supports it.
 - Resolution fallback so a `link:`-installed checkout can still load
   `@deepseek-ai/dsh-typert-protocol` from the running installation.
+
+## 0.1.1 — 2026-11-12
+
+### Added
+
+- The bundle now wires itself: activating the plugin writes the
+  `@deepseek-ai/dsh-mcp-client` row into `$DSH_HOME/cordis.patch.yml` and the
+  `<!-- CODEGRAPH_START -->` block into `$DSH_HOME/AGENTS.md`, so installing the
+  plugin also installs the MCP server and the agent instructions. Unloading it
+  removes exactly the parts it wrote (the patch file returns to `[]`).
+- Nothing is written when a `codegraph` server already exists in this home or in
+  any profile, or when `codegraph` is not on `PATH` — the sidebar dialog stays
+  the place that installs the CLI.
+- `DSH_CODEGRAPH_AUTO_WIRE=0` disables the install-time wiring and leaves the
+  dialog as the only writer.
+
+### Fixed
+
+- Removing the row from a patch file that only held `[]` and comments now
+  restores an empty-list literal instead of leaving a `null` document.

package/README.md

@@ -24,8 +24,21 @@
 
 两个入口共用同一个对话框；对话框第一次打开时会读一次宿主状态，把 DSH 目录、CLI 版本、当前是否已接线一并列出来，再决定按钮文案。
 
+## 装上插件就已经接好线
+
+装插件这一步本身就是安装：插件首次激活时会自己写接线，不用非得点菜单。
+
+1. 读 `$DSH_HOME` 与所有 profile 的 `cordis.patch.yml`，只要已经存在 `codegraph` 的 MCP 行就**什么都不写**——那条行可能是 CLI 或你自己写的，卸插件时也不会去删别人的东西。
+2. `command -v codegraph` 找不到 CLI 就跳过：接线的 `command:` 会指向空，装 CLI 是对话框的活。
+3. 都没有才动手：`cordis.patch.yml` 写 MCP 行（`command` 用 CLI 绝对路径），`AGENTS.md` 追加 `<!-- CODEGRAPH_START -->` 块。
+4. 卸载/禁用插件时只撤掉**它自己写的那两处**：patch 文件回到 `[]`，`AGENTS.md` 若被清空则整个删除。
+
+关掉这个行为：`DSH_CODEGRAPH_AUTO_WIRE=0`，此后只有侧栏对话框会写接线。
+
 ## 安装时发生了什么
 
+从侧栏菜单点「安装 codegraph」走的是完整流程（装 CLI、修复接线、校验）：
+
 1. **检查 CLI**：`command -v codegraph`，找不到就 `npm install -g @colbymchenry/codegraph@latest`。
 2. **判断 target**：`codegraph install --print-config dsh` 能通过就走 CLI，否则插件自己写。
 3. **写接线**：`cordis.patch.yml` 追加 MCP 客户端条目；`AGENTS.md` 追加 `<!-- CODEGRAPH_START -->` 块。

package/lib/index.js

@@ -124,6 +124,15 @@ const RUN_RETENTION_MS = 30 * 60 * 1000;
 /** The argument vector the MCP server is started with. */
 const MCP_ARGS = ["serve", "--mcp"];
 
+/**
+ * Whether activating the bundle wires the MCP server by itself. Installing the
+ * plugin is then the whole installation: the row and the instructions block
+ * land on first activation, and unloading the plugin takes away exactly the
+ * parts it wrote. `DSH_CODEGRAPH_AUTO_WIRE=0` turns this off for a profile that
+ * wants the dialog to be the only writer.
+ */
+const AUTO_WIRE = process.env.DSH_CODEGRAPH_AUTO_WIRE !== "0";
+
 /**
  * The instructions block, byte-identical to `codegraph`'s own installer so a
  * later `codegraph install -t dsh` rewrites it instead of appending a second.
@@ -545,6 +554,83 @@ function removeInstructions(text) {
 	return { text: next, action: "removed" };
 }
 
+/**
+ * Wire the MCP row and the instructions block on activation, so installing the
+ * bundle is also installing the wiring.
+ *
+ * Nothing is written when a `codegraph` server already exists anywhere (this
+ * home or any profile): that row belongs to the CLI or to the user, and taking
+ * it away on unload would delete wiring this plugin never added. A missing CLI
+ * is a skip as well — the row's `command` would point at nothing, and the
+ * sidebar dialog is the place that installs it.
+ *
+ * @param {string} home - Absolute DSH home.
+ * @returns {Promise<{row: boolean, block: boolean, patchPath: string, instructionsPath: string, skipped: "server-exists" | "cli-missing" | null}>} what was written.
+ */
+async function applyAutoWiring(home) {
+	const patchPath = path.join(home, PATCH_FILE);
+	const instructionsPath = path.join(home, INSTRUCTIONS_FILE);
+	const base = { row: false, block: false, patchPath, instructionsPath, skipped: null };
+	const state = await wiringState(home);
+	if (state.configured) return { ...base, skipped: "server-exists" };
+	const cli = await probeCli();
+	if (!cli.found) return { ...base, skipped: "cli-missing" };
+
+	const current = (await readText(patchPath)) ?? "";
+	const row = insertServerRow(current, cli.path ?? SERVER_NAME);
+	if (row.action !== "unchanged") {
+		await mkdir(path.dirname(patchPath), { recursive: true });
+		await writeFile(patchPath, row.text, "utf8");
+	}
+
+	let block = false;
+	const existing = await readText(instructionsPath);
+	if (existing === null) {
+		await mkdir(path.dirname(instructionsPath), { recursive: true });
+		await writeFile(instructionsPath, `${INSTRUCTIONS_BLOCK}\n`, "utf8");
+		block = true;
+	} else {
+		const edit = upsertInstructions(existing);
+		// Only a block this plugin appended is ours to delete later; updating one
+		// that the CLI wrote keeps the CLI as its owner.
+		if (edit.action === "created") {
+			await writeFile(instructionsPath, edit.text, "utf8");
+			block = true;
+		}
+	}
+
+	return { row: row.action !== "unchanged", block, patchPath, instructionsPath, skipped: null };
+}
+
+/**
+ * Undo exactly what `applyAutoWiring` added; a skipped run has nothing to undo.
+ *
+ * @param {object | null} applied - The record `applyAutoWiring` returned.
+ * @returns {Promise<void>} resolution after both files are restored.
+ */
+async function undoAutoWiring(applied) {
+	if (applied === null || applied.skipped !== null) return;
+	if (applied.row) {
+		const current = (await readText(applied.patchPath)) ?? "";
+		const edit = removeServerRows(current);
+		if (edit.action === "removed") {
+			// An empty patch file parses as `null`, which the loader rejects; the
+			// empty-list literal is how this file looked before the row went in.
+			const { lines, eol } = splitLines(edit.text);
+			const text = layoutOf(lines).kind === "empty" ? joinLines([...lines, "[]"], eol) : edit.text;
+			await writeFile(applied.patchPath, text, "utf8");
+		}
+	}
+	if (applied.block) {
+		const current = await readText(applied.instructionsPath);
+		if (current === null) return;
+		const edit = removeInstructions(current);
+		if (edit.action !== "removed") return;
+		if (edit.text.trim() === "") await rm(applied.instructionsPath, { force: true });
+		else await writeFile(applied.instructionsPath, edit.text, "utf8");
+	}
+}
+
 /**
  * Append one line to a run's bounded buffer.
  *
@@ -626,6 +712,44 @@ class CodegraphInstallService extends TypertRemoteService {
 		super(ctx, SERVICE);
 		this.runs = new Map();
 		this.activeRunId = null;
+		this.autoWiring = null;
+		if (AUTO_WIRE) this.armAutoWiring();
+	}
+
+	/**
+	 * Bind the install-time wiring to this plugin's lifetime: activation writes
+	 * the row, unload takes back the parts it wrote. The write itself is a pure
+	 * file edit, so no service has to be injected first.
+	 *
+	 * @returns {void}
+	 */
+	armAutoWiring() {
+		this.ctx.effect(() => {
+			let disposed = false;
+			void applyAutoWiring(dshHome())
+				.then(async (applied) => {
+					if (applied.skipped !== null) {
+						this.ctx.logger?.debug?.(`codegraph-init: auto wiring skipped (${applied.skipped})`);
+						return;
+					}
+					this.autoWiring = applied;
+					this.ctx.logger?.info?.(`codegraph-init: wired the codegraph MCP server into ${applied.patchPath}`);
+					if (disposed) await undoAutoWiring(applied);
+				})
+				.catch((error) => {
+					const message = error instanceof Error ? error.message : String(error);
+					this.ctx.logger?.warn?.(`codegraph-init: auto wiring failed: ${message}`);
+				});
+			return () => {
+				disposed = true;
+				const applied = this.autoWiring;
+				this.autoWiring = null;
+				void undoAutoWiring(applied).catch((error) => {
+					const message = error instanceof Error ? error.message : String(error);
+					this.ctx.logger?.warn?.(`codegraph-init: auto unwiring failed: ${message}`);
+				});
+			};
+		}, "codegraph-init: auto wiring");
 	}
 
 	/**
@@ -995,4 +1119,4 @@ declareRemote(CodegraphInstallService, "poll");
 declareRemote(CodegraphInstallService, "cancel");
 declareRemote(CodegraphInstallService, "forget");
 
-export { CodegraphInstallService, CodegraphInstallService as default };
+export { CodegraphInstallService, CodegraphInstallService as default, applyAutoWiring, undoAutoWiring };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-codegraph-init",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Add codegraph install / uninstall actions to the sidebar workspace context menu, streaming the npm-based CLI install and the DSH MCP wiring into one dialog.",
   "license": "MIT",
   "author": "scccy",