dsh-office-kit 0.4.0 → 0.4.2

package/assets/skills/excel-live-control/SKILL.md

@@ -4,6 +4,11 @@ description: Drive a workbook that is already open in Excel or WPS Spreadsheets
 
 # Live spreadsheets (Excel / WPS)
 
+> Windows only. This skill drives an already-open Excel or WPS Spreadsheets window
+> through COM using Windows PowerShell 5.1 (`Marshal.GetActiveObject`, which
+> PowerShell 7 dropped). On macOS and Linux every command exits `3` with
+> `stage: "powershell"` and a remedy; use the file-based `xlsx` verbs there instead.
+
 ## What "live" means here
 
 The workbook is open in a running Excel or WPS Spreadsheets window, and the user is

package/assets/skills/excel-live-control/container_tools/live_excel.py

@@ -605,6 +605,15 @@ def main() -> int:
     except ToolError as error:
         return emit(failure_payload(op, error), 2)
 
+    # macOS and Linux ship pwsh but no COM, and `shutil.which('powershell')` can find
+    # that binary; fail on the platform instead of surfacing a cryptic attach error.
+    if sys.platform != "win32":
+        return emit(failure_payload(op, ToolError(
+            "Excel/WPS live control drives the application through Windows PowerShell 5.1 "
+            "COM, which only exists on Windows",
+            "powershell",
+        )), 3)
+
     request = {
         "op": op,
         "launch_hidden": bool(getattr(args, "launch_hidden", False)),

package/assets/skills/office-documents/tasks/verify_render.md

@@ -32,7 +32,7 @@ The runtime reports `renderers.can_render.docx: false` because the DSH LibreOffi
 In that state:
 
 - Verify structure with `docx read`, `docx_kit.py parts|text|xml` and the audit scripts.
-- Tell the user plainly that layout was not visually verified, and that the fix is installing the plugin inside the harness tree (or setting `DSH_LIBREOFFICE_KIT_CLI`), not installing Word/LibreOffice.
+- Tell the user plainly that layout was not visually verified, and that the fix is a harness install that ships the kit (the plugin records its path in `config.json` when it loads), not installing Word/LibreOffice.
 - Do not claim a file "looks good" from XML evidence alone.
 
 ## After any script edit

package/assets/skills/office-documents/troubleshooting/libreoffice_headless.md

@@ -12,17 +12,19 @@ Look at `renderers.kit` and `renderers.can_render`. `can_render.docx: false` mea
 
 Rendering goes through the LibreOffice kit that DeepSeek Harness ships (`@deepseek-ai/libreoffice-kit`). On macOS and Windows it uses the kit's native engine; on Linux it uses the WASM engine. A separately installed Microsoft Office, WPS Office, or system LibreOffice is **not** used and does not make rendering work on its own.
 
-The plugin finds the kit by walking up from its own directory through `node_modules/@deepseek-ai/libreoffice-kit/lib/cli.js`. So:
+The plugin records the kit's location when the harness loads it, and the helper reads that record before anything else:
 
-- Installed inside the harness tree → the kit is found automatically.
-- Installed anywhere else → install the plugin where the harness can see its dependencies, or set `DSH_LIBREOFFICE_KIT_CLI` to the kit's `lib/cli.js`.
-- `DSH_NODE` overrides the node binary if `node` is not on `PATH`.
+1. `DSH_LIBREOFFICE_KIT_CLI` — an explicit override, also handy for a hand-run helper.
+2. `libreofficeKit` in `~/.dsh/office/config.json`. The host half writes this at load, resolving the kit from the harness entry point (`process.argv[1]`, e.g. `<dsh>/node_modules/@deepseek-ai/dsh/lib/bin.js`). This is what makes a profile install work, where the kit lives in the harness tree and not next to the plugin. Configs that pre-0.5 versions wrote in the OS config directory are still read as fallbacks.
+3. Walking up from the plugin directory through `node_modules/@deepseek-ai/libreoffice-kit/lib/cli.js` — covers a checkout or a plugin-local dependency.
+
+`DSH_OFFICE_CONFIG` points the helper at a different config file; `DSH_NODE` overrides the node binary if `node` is not on `PATH`.
 
 ## Common failures
 
 | Message | Cause | Fix |
 |---|---|---|
-| `the LibreOffice renderer is not available` | Kit not found from the plugin directory | Put the plugin inside the harness tree, or set `DSH_LIBREOFFICE_KIT_CLI` |
+| `the LibreOffice renderer is not available` | Kit not found | Load the plugin through the harness (it records the path), or set `DSH_LIBREOFFICE_KIT_CLI` to the kit's `lib/cli.js` |
 | `cannot start the LibreOffice kit` / `EEXIST` | Interrupted install, or a stale staging directory | Re-run the plugin postinstall; delete leftover `.kit-render-*` directories |
 | `Unsupported LibreOfficeKit host` | Unsupported platform/arch (the kit ships Windows x64/arm64, macOS x64/arm64, Linux via WASM) | Render on a supported host |
 | `warning: N font(s) used by this file are not installed` | Font substitution — the render succeeded but that text may reflow or show boxes | Install the font or substitute it deliberately, then re-render |

package/assets/skills/office-presentations/SKILL.md

@@ -150,9 +150,9 @@ Guidance: [authoring workflow](references/authoring-workflow.md) ·
 ## Genuine gaps
 
 - The DSH LibreOffice kit missing (`renderers.can_render.pptx: false`) → no true
-  render; use `--engine draft` and say the check was geometric. Installing the
-  plugin inside the harness tree (or setting `DSH_LIBREOFFICE_KIT_CLI`) fixes it;
-  a standalone Office/LibreOffice install does not.
+  render; use `--engine draft` and say the check was geometric. A harness install
+  that ships the kit fixes it (the plugin records its path in `config.json` when it
+  loads); a standalone Office/LibreOffice install does not.
 - Fonts the user's machine lacks are substituted by any renderer; the pack
   cannot install fonts.
 - Google Slides import fidelity is out of our control; see

package/assets/skills/office-spreadsheets/SKILL.md

@@ -10,7 +10,7 @@ A workbook is cells: each one holds a value, a format, and possibly a formula *s
 | --- | --- | --- |
 | Build a new workbook | Cells written top-left at A1; `=`-prefixed strings become formulas; styles are per-cell objects; sheets get names, widths, freeze panes, merges | `xlsx create` spec, `tasks/create-workbook.md` |
 | Change cells in an existing file | Only the cells you name change; a re-save can drop objects openpyxl does not model (pivots, foreign charts) | `container_tools/set_cells.py` and friends, `tasks/edit-workbook.md` |
-| The workbook is open in Excel/WPS and the user wants it changed live | The target is the window, not the file on disk: unsaved user edits are in play and a COM write is not transactional | `../excel-live-control/SKILL.md` |
+| The workbook is open in Excel/WPS and the user wants it changed live | The target is the window, not the file on disk: unsaved user edits are in play and a COM write is not transactional. **Windows only** — on macOS/Linux the live tool exits 3 with a remedy; edit the file instead | `../excel-live-control/SKILL.md` |
 | Inventory a file you did not write | Iterate sheets, used range, formulas vs values, merged ranges, conditional formats, defined names, charts | `container_tools/inspect_workbook.py` |
 | Compare two versions | Cell-by-cell walk with formula-aware, format-aware, structure-aware levels | `container_tools/diff_workbooks.py`, `tasks/diff-workbooks.md` |
 | Prove the numbers are real | Recalculate in Excel, then check every formula cell has a value, no error, no dangling sheet | `xlsx recalc` + `container_tools/recalc_gate.py`, `tasks/recalc-and-check.md` |

package/assets/skills/shared/common-workflow.md

@@ -125,10 +125,10 @@ a blank, clipped or miscoloured raster is invisible inside the host file.
 | --- | --- | --- |
 | No `## Office runtime` JSON; skill says *No working Python runtime is available* | The plugin cannot install or find a compatible interpreter | Do not run helper commands, do not write a document by hand. Give the user the remedy command from the skill body, ask them to run it, then call `office_status` again. |
 | `office_status` shows `cli.exists: false`, or `python`/`cli` file not found | The bundled helper or its install is incomplete | Report the missing path. Do not substitute another tool that produces a different file. |
-| `render` fails: `the LibreOffice renderer is not available` | The kit that DSH ships with was not found; create/read/edit still work but docx/pptx/xlsx cannot be rasterised | Do not claim you inspected it. Say the file was built and validated structurally (read verb), but the renderer is unavailable on this host so you could not check its appearance. The kit ships with DeepSeek Harness — installing this plugin inside the harness tree (or setting `DSH_LIBREOFFICE_KIT_CLI`) fixes it; a standalone LibreOffice/Office/WPS install does not. `pdf render` still works. If the user needs visual confidence now, offer to rebuild from a spec or to present the unverified file explicitly labelled as such. |
+| `render` fails: `the LibreOffice renderer is not available` | The kit that DSH ships with was not found; create/read/edit still work but docx/pptx/xlsx cannot be rasterised | Do not claim you inspected it. Say the file was built and validated structurally (read verb), but the renderer is unavailable on this host so you could not check its appearance. The kit ships with DeepSeek Harness and the plugin records its path in `config.json` when it loads, so this means the harness install lacks the kit or `DSH_LIBREOFFICE_KIT_CLI` points nowhere; a standalone LibreOffice/Office/WPS install does not help. `pdf render` still works. If the user needs visual confidence now, offer to rebuild from a spec or to present the unverified file explicitly labelled as such. |
 | Annotation region does not land on anything, or the payload says `unmapped: true` | The drawn region cannot be tied to an object in the file | Do not silently shift the region or guess a cell/shape. Say what you could identify (preview page, text, resolved cells) and which item you could not locate, then ask the user for the text or cell. |
 | `{"ok": false, "error": "..."}` and non-zero exit | The spec was rejected (unknown key, bad type, missing file) | Read the error text, fix the spec key it names, re-run the same command. Never strip a failing block to make the command pass — that silently deletes user content. |
-| `xlsx recalc` fails because the LibreOffice kit is missing | openpyxl cannot evaluate formulas, and the kit is the only calculation engine this plugin uses | Report formula cells as uncalculated; do not invent computed values. Fix the renderer first (install the plugin inside the harness tree) and re-run. |
+| `xlsx recalc` fails because the LibreOffice kit is missing | openpyxl cannot evaluate formulas, and the kit is the only calculation engine this plugin uses | Report formula cells as uncalculated; do not invent computed values. Fix the renderer first (see the row above) and re-run. |
 | The helper has no verb for what the requirement needs | The common path is short; the capability is not limited to it | Take the pack script route, or write a short snippet against `python-docx` / `openpyxl` / `python-pptx` / `pypdf` / `PyMuPDF` / `reportlab`, run it with `<python>`, then `read` the file back and render. Never refuse a job because a verb is missing. |
 | `read` returns an empty structure for a file the user supplied | Unsupported or corrupt input | Say what failed and stop; do not overwrite the user's file with an empty one. |
 

package/dist/index.js

@@ -1,12 +1,13 @@
+import { createRequire } from "node:module";
 import { execFile } from "node:child_process";
-import { existsSync, readFileSync, statSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, statSync, writeFileSync } from "node:fs";
 import { readFile } from "node:fs/promises";
 import { dirname, join } from "node:path";
 import process from "node:process";
-import { fileURLToPath } from "node:url";
 import { promisify } from "node:util";
 import { BUNDLED_SKILL_RANK } from "@deepseek-ai/dsh-skill";
 import { defineTool } from "@deepseek-ai/dsh-tools";
+import { fileURLToPath } from "node:url";
 import { homedir } from "node:os";
 //#region src/features.ts
 /** The four entries, in menu order. */
@@ -48,6 +49,33 @@ const OFFICE_FEATURES = [
 		extensions: [".xlsx"]
 	}
 ];
+//#endregion
+//#region src/kit.ts
+/** Package root of the installed plugin (dist/ during a build, src/ in a checkout). */
+const PACKAGE_ROOT$1 = fileURLToPath(new URL("..", import.meta.url));
+const KIT_CLI = "@deepseek-ai/libreoffice-kit/cli";
+/**
+* Locate the LibreOffice kit CLI that DeepSeek Harness ships.
+*
+* DSH installs third-party plugins into a profile (`~/.dsh/profiles/<name>/node_modules`)
+* while the kit lives in the harness dependency tree, so resolving relative to this
+* package cannot find it. Resolving from the harness entry point (`process.argv[1]`,
+* e.g. `<dsh>/node_modules/@deepseek-ai/dsh/lib/bin.js`) walks the tree the harness
+* itself loaded from, and resolving from this package covers a checkout or a
+* plugin-local install. `DSH_LIBREOFFICE_KIT_CLI` overrides both.
+*/
+function resolveKitCli(entry = process.argv[1]) {
+	const override = process.env.DSH_LIBREOFFICE_KIT_CLI;
+	if (override != null && existsSync(override)) return override;
+	for (const anchor of [entry, join(PACKAGE_ROOT$1, "package.json")]) {
+		if (anchor == null) continue;
+		try {
+			const cli = createRequire(anchor).resolve(KIT_CLI);
+			if (existsSync(cli)) return cli;
+		} catch {}
+	}
+	return null;
+}
 /** Python packages the helper CLI needs. */
 const PYTHON_PACKAGES = [
 	{
@@ -91,34 +119,41 @@ const PYTHON_PACKAGES = [
 		package: "mathml2omml"
 	}
 ];
-const PACKAGE_ROOT$1 = fileURLToPath(new URL("..", import.meta.url));
-const POSTINSTALL_SCRIPT = join(PACKAGE_ROOT$1, "postinstall.mjs");
+const PACKAGE_ROOT = fileURLToPath(new URL("..", import.meta.url));
+const POSTINSTALL_SCRIPT = join(PACKAGE_ROOT, "postinstall.mjs");
 /** Upper bound for one self-heal run; the remedy command remains available afterwards. */
 const POSTINSTALL_TIMEOUT = 540 * 1e3;
 /** Failed probes are retried after this delay instead of re-running pip on every status call. */
 const FAILURE_RETRY_DELAY = 300 * 1e3;
-function configDir() {
-	const home = homedir();
-	return process.platform === "win32" ? join(process.env.APPDATA ?? join(home, "AppData", "Roaming"), "dsh-office-kit") : process.platform === "darwin" ? join(home, "Library", "Application Support", "dsh-office-kit") : join(process.env.XDG_CONFIG_HOME ?? join(home, ".config"), "dsh-office-kit");
+/** Where every version of this plugin keeps its runtime, next to DSH's own data. */
+function officeHome() {
+	return join(homedir(), ".dsh", "office");
 }
 function legacyDataDir() {
 	const home = homedir();
 	return process.platform === "win32" ? join(process.env.LOCALAPPDATA ?? join(home, "AppData", "Local"), "dsh-office-kit") : process.platform === "darwin" ? join(home, "Library", "Application Support", "dsh-office-kit") : join(process.env.XDG_DATA_HOME ?? join(home, ".local", "share"), "dsh-office-kit");
 }
-/** Config location written by versions that ignored the platform branch. */
-function legacyConfigFile() {
-	return process.platform === "win32" ? join(configDir(), "config.json") : join(process.env.APPDATA ?? join(homedir(), "AppData", "Roaming"), "dsh-office-kit", "config.json");
+/**
+* Config files older versions wrote, newest first.
+*
+* The pre-0.5 config stayed in the OS config directory (`%APPDATA%\dsh-office-kit`,
+* `$XDG_CONFIG_HOME/dsh-office-kit`, `~/Library/Application Support/dsh-office-kit`).
+* Reading them keeps an upgraded install from rebuilding or mis-reporting its venv.
+*/
+function legacyConfigFiles() {
+	const home = homedir();
+	return [.../* @__PURE__ */ new Set([process.platform === "win32" ? join(process.env.APPDATA ?? join(home, "AppData", "Roaming"), "dsh-office-kit", "config.json") : process.platform === "darwin" ? join(home, "Library", "Application Support", "dsh-office-kit", "config.json") : join(process.env.XDG_CONFIG_HOME ?? join(home, ".config"), "dsh-office-kit", "config.json"), join(process.env.APPDATA ?? join(home, "AppData", "Roaming"), "dsh-office-kit", "config.json")])];
 }
 /** Resolve per-OS config/data directories and bundled resources. */
 function officePaths() {
-	const runtimeDir = join(homedir(), ".dsh", "office", "runtime");
+	const runtimeDir = join(officeHome(), "runtime");
 	return {
-		configFile: join(configDir(), "config.json"),
-		legacyConfigFile: legacyConfigFile(),
+		configFile: join(officeHome(), "config.json"),
+		legacyConfigFiles: legacyConfigFiles(),
 		venv: join(runtimeDir, "venv"),
 		legacyVenv: join(legacyDataDir(), "venv"),
-		cli: join(PACKAGE_ROOT$1, "python", "office_cli.py"),
-		skillsDir: join(PACKAGE_ROOT$1, "assets", "skills")
+		cli: join(PACKAGE_ROOT, "python", "office_cli.py"),
+		skillsDir: join(PACKAGE_ROOT, "assets", "skills")
 	};
 }
 /** One-liner the user can run to repair the runtime without pnpm. */
@@ -216,10 +251,11 @@ function failedInspection(path, error) {
 }
 /**
 * Read the interpreter path postinstall recorded; malformed files are ignored.
-* The legacy location is read second so an upgrade does not orphan a ready venv.
+* The legacy locations are read after the canonical one so an upgrade does not
+* orphan a ready venv.
 */
-function savedPython(configFile, legacyConfigFile) {
-	const files = legacyConfigFile == null ? [configFile] : [configFile, legacyConfigFile];
+function savedPython(configFile, legacyConfigFiles = []) {
+	const files = [configFile, ...legacyConfigFiles];
 	for (const file of files) {
 		if (!isFile(file)) continue;
 		try {
@@ -228,6 +264,61 @@ function savedPython(configFile, legacyConfigFile) {
 		} catch {}
 	}
 }
+/**
+* Read the plugin config, falling back to files older versions wrote.
+*
+* The canonical file wins key by key, so a legacy record contributes only what
+* the new file has not written yet (an upgrade usually still needs its venv).
+*/
+function loadConfig(configFile) {
+	const legacy = officePaths().legacyConfigFiles;
+	const merged = {};
+	for (const file of [...legacy].reverse()) try {
+		const parsed = JSON.parse(readFileSync(file, "utf8"));
+		if (parsed != null && typeof parsed === "object" && !Array.isArray(parsed)) Object.assign(merged, parsed);
+	} catch {}
+	try {
+		const parsed = JSON.parse(readFileSync(configFile, "utf8"));
+		if (parsed != null && typeof parsed === "object" && !Array.isArray(parsed)) Object.assign(merged, parsed);
+	} catch {}
+	return merged;
+}
+/**
+* Merge one key into the plugin's config without disturbing the rest.
+*
+* Versions that predate the current layout left stale absolute paths behind, so
+* the record is refreshed whenever the host resolves something different.
+*/
+function saveRecord(configFile, key, value) {
+	const config = loadConfig(configFile);
+	if (JSON.stringify(config[key]) === JSON.stringify(value) && isFile(configFile)) return;
+	const temporary = `${configFile}.${process.pid}.tmp`;
+	try {
+		mkdirSync(dirname(configFile), { recursive: true });
+		writeFileSync(temporary, `${JSON.stringify({
+			...config,
+			[key]: value
+		}, null, 2)}\n`, "utf8");
+		renameSync(temporary, configFile);
+	} catch {}
+}
+/**
+* Record the renderer the harness loaded so every later helper call finds it.
+*
+* A profile install keeps the plugin in `~/.dsh/profiles/<name>/node_modules`
+* and the kit in the harness tree, so no walk-up from the plugin can reach it.
+* The helper reads this record first; a read-only config must not stop loading.
+*/
+function saveKitPath(configFile, node, cli) {
+	saveRecord(configFile, "libreofficeKit", {
+		node,
+		cli
+	});
+}
+/** Record the interpreter this plugin settled on so the next probe can trust it. */
+function savePythonPath(configFile, python) {
+	saveRecord(configFile, "python", python);
+}
 /** The interpreter inside a virtual environment, which differs per platform. */
 function venvPython(venv) {
 	return join(venv, process.platform === "win32" ? "Scripts/python.exe" : "bin/python");
@@ -238,14 +329,10 @@ function candidates(configFile) {
 	const found = [];
 	const env = process.env.DSH_OFFICE_PYTHON;
 	if (env != null && env.length > 0) found.push(env);
-	const saved = savedPython(configFile, paths.legacyConfigFile);
+	found.push(venvPython(paths.venv));
+	const saved = savedPython(configFile, paths.legacyConfigFiles);
 	if (saved != null) found.push(saved);
-	found.push(venvPython(paths.venv), venvPython(paths.legacyVenv));
-	const marker = join(PACKAGE_ROOT$1, ".python-path");
-	if (isFile(marker)) {
-		const text = readFileSync(marker, "utf8").trim();
-		if (text.length > 0) found.push(text);
-	}
+	found.push(venvPython(paths.legacyVenv));
 	for (const name of ["python3", "python"]) found.push(name);
 	if (process.platform === "win32") {
 		for (const version of [
@@ -316,9 +403,20 @@ async function inspectRuntime(configFile = officePaths().configFile) {
 		if (expanded !== candidate && !isFile(expanded)) continue;
 		const inspection = await inspectPython(expanded);
 		seen.push(inspection);
-		if (inspection.compatible && inspection.missing.length === 0) return selectRuntime(seen);
+		if (inspection.compatible && inspection.missing.length === 0) return settledRuntime(seen, configFile);
 	}
-	return selectRuntime(seen);
+	return settledRuntime(seen, configFile);
+}
+/**
+* Freeze the probe result and keep config.json honest about which interpreter won.
+*
+* A venv migrated out of the legacy data directory leaves the old absolute path
+* recorded; rewriting it stops the next launch from probing a dead path first.
+*/
+function settledRuntime(seen, configFile) {
+	const status = selectRuntime(seen);
+	if (status.state === "ok" && status.python != null && status.python.path !== savedPython(configFile)) savePythonPath(configFile, status.python.path);
+	return status;
 }
 /** Run postinstall.mjs as a child process; resolve with an error summary or undefined. */
 function runPostinstall() {
@@ -405,24 +503,10 @@ function parseSkill(raw, path) {
 		content: raw.slice(match[0].length).trim()
 	};
 }
-const KIT_MISSING = "the DSH LibreOffice kit (@deepseek-ai/libreoffice-kit) was not found: install this plugin inside the harness tree, or point DSH_LIBREOFFICE_KIT_CLI at <kit>/lib/cli.js";
-const PACKAGE_ROOT = fileURLToPath(new URL("..", import.meta.url));
+const KIT_MISSING = "the DSH LibreOffice kit (@deepseek-ai/libreoffice-kit) was not found: the plugin records where the harness loaded it from in dsh-office-kit's config.json, and DSH_LIBREOFFICE_KIT_CLI overrides that for a plugin running outside the harness";
 const runKit = promisify(execFile);
-/** Locate the kit the same way python/office_cli.py does, so both halves agree. */
-function findKitCli() {
-	const override = process.env.DSH_LIBREOFFICE_KIT_CLI;
-	if (override != null && existsSync(override)) return override;
-	let directory = PACKAGE_ROOT;
-	for (;;) {
-		const candidate = join(directory, "node_modules", "@deepseek-ai", "libreoffice-kit", "lib", "cli.js");
-		if (existsSync(candidate)) return candidate;
-		const parent = dirname(directory);
-		if (parent === directory) return null;
-		directory = parent;
-	}
-}
 async function kitFacts() {
-	const cli = findKitCli();
+	const cli = resolveKitCli();
 	if (cli == null) return {
 		available: false,
 		node: null,
@@ -462,7 +546,7 @@ async function kitFacts() {
 }
 function pluginVersion() {
 	try {
-		const parsed = JSON.parse(readFileSync(join(PACKAGE_ROOT, "package.json"), "utf8"));
+		const parsed = JSON.parse(readFileSync(join(PACKAGE_ROOT$1, "package.json"), "utf8"));
 		return typeof parsed.version === "string" ? parsed.version : "unknown";
 	} catch {
 		return "unknown";
@@ -480,6 +564,8 @@ function runtimeReason(runtime) {
 function apply(ctx) {
 	const paths = officePaths();
 	const candidates = [];
+	const kitCli = resolveKitCli();
+	if (kitCli != null) saveKitPath(paths.configFile, process.execPath, kitCli);
 	for (const feature of OFFICE_FEATURES) {
 		const directory = join(paths.skillsDir, feature.skill);
 		const path = join(directory, "SKILL.md");
@@ -523,12 +609,17 @@ function apply(ctx) {
 			}
 			const parsed = parseSkill(raw, path);
 			const runtime = await ensureRuntime(paths.configFile);
-			const pythonLine = runtime.state === "ok" && runtime.python != null ? `\n\n## Office runtime\n\nUse these verified absolute paths for every helper command. Quote paths that contain spaces. \`skill\` is this skill's own directory: every script it names lives at \`<skill>/scripts/...\`; \`shared\` holds the cross-format workflow notes.\n\n\`\`\`json\n${JSON.stringify({
+			const kitCli = resolveKitCli();
+			const pythonLine = runtime.state === "ok" && runtime.python != null ? `\n\n## Office runtime\n\nUse these verified absolute paths for every helper command. Quote paths that contain spaces. \`skill\` is this skill's own directory: every script it names lives at \`<skill>/scripts/...\`; \`shared\` holds the cross-format workflow notes; the helper finds \`renderer\` on its own, so never pass it.\n\n\`\`\`json\n${JSON.stringify({
 				python: runtime.python.path,
 				cli: paths.cli,
 				skill: dirname(path).split("\\").join("/"),
 				shared: join(paths.skillsDir, "shared").split("\\").join("/"),
-				version: runtime.python.version
+				version: runtime.python.version,
+				...kitCli == null ? {} : { renderer: {
+					kind: "libreoffice-kit",
+					cli: kitCli
+				} }
 			}, null, 2)}\n\`\`\`` : `\n\n## Office runtime\n\nNo working Python runtime is available: ${runtimeReason(runtime)}. Do not run helper commands or claim success. Tell the user to run \`${runtime.remedy}\`, then call \`office_status\` again.`;
 			const { rank: _rank, locator: _locator, ...summary } = candidate;
 			return {
@@ -579,6 +670,10 @@ function apply(ctx) {
 					path: paths.cli,
 					exists: cliExists()
 				},
+				config: {
+					path: paths.configFile,
+					exists: existsSync(paths.configFile)
+				},
 				packages: runtime.packages,
 				renderers: await kitFacts(),
 				skills: candidates.map((candidate) => candidate.name)

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "dsh-office-kit",
   "type": "module",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "description": "Office documents, PDF, presentations, and spreadsheets for DeepSeek Harness",
   "author": "Hairy",
   "license": "MIT",

package/postinstall.mjs

@@ -16,7 +16,7 @@
  */
 
 import { spawnSync } from 'node:child_process'
-import { existsSync, mkdirSync, readFileSync, realpathSync, rmSync, writeFileSync } from 'node:fs'
+import { existsSync, mkdirSync, readFileSync, realpathSync, renameSync, rmSync, writeFileSync } from 'node:fs'
 import { homedir, tmpdir } from 'node:os'
 import { dirname, join } from 'node:path'
 import process from 'node:process'
@@ -35,21 +35,22 @@ const LEGACY_DATA_DIR = IS_WINDOWS
     : join(process.env.XDG_DATA_HOME || join(homedir(), '.local', 'share'), 'dsh-office-kit')
 export const LEGACY_VENV_DIR = join(LEGACY_DATA_DIR, 'venv')
 const LEGACY_VENV_PYTHON = join(LEGACY_VENV_DIR, IS_WINDOWS ? 'Scripts/python.exe' : 'bin/python')
-// Must mirror `configDir()` in src/python.ts: the host half reads this file on
-// every runtime probe, so a platform mismatch here makes a ready venv look missing.
-export const CONFIG_DIR = IS_WINDOWS
-  ? join(process.env.APPDATA || join(homedir(), 'AppData', 'Roaming'), 'dsh-office-kit')
-  : process.platform === 'darwin'
-    ? join(homedir(), 'Library', 'Application Support', 'dsh-office-kit')
-    : join(process.env.XDG_CONFIG_HOME || join(homedir(), '.config'), 'dsh-office-kit')
+// Must mirror `officePaths()` in src/python.ts: the host half reads this file on
+// every runtime probe, so a mismatch here makes a ready venv look missing.
+export const CONFIG_DIR = join(homedir(), '.dsh', 'office')
 export const CONFIG_FILE = join(CONFIG_DIR, 'config.json')
 /**
- * Legacy config location written by versions without the platform branch.
- * Read as a fallback so an existing runtime is not rebuilt on upgrade.
+ * Config files older versions wrote in the OS config directory. Read as
+ * fallbacks so an upgraded install is not rebuilt or mis-reported.
  */
-const LEGACY_CONFIG_FILE = IS_WINDOWS
-  ? CONFIG_FILE
-  : join(process.env.APPDATA || join(homedir(), 'AppData', 'Roaming'), 'dsh-office-kit', 'config.json')
+const LEGACY_CONFIG_FILES = [...new Set([
+  IS_WINDOWS
+    ? join(process.env.APPDATA || join(homedir(), 'AppData', 'Roaming'), 'dsh-office-kit', 'config.json')
+    : process.platform === 'darwin'
+      ? join(homedir(), 'Library', 'Application Support', 'dsh-office-kit', 'config.json')
+      : join(process.env.XDG_CONFIG_HOME || join(homedir(), '.config'), 'dsh-office-kit', 'config.json'),
+  join(process.env.APPDATA || join(homedir(), 'AppData', 'Roaming'), 'dsh-office-kit', 'config.json'),
+])]
 const REQUIREMENTS = join(PACKAGE_ROOT, 'python', 'requirements.txt')
 const LOG_PREFIX = '[dsh-office-kit]'
 
@@ -230,7 +231,37 @@ function installPython() {
   return true
 }
 
+/**
+ * Move an upgraded install's venv out of the pre-0.4 data directory.
+ *
+ * Reusing the legacy venv in place kept the old layout alive forever, so the
+ * canonical path was only ever reached by fresh installs. A rename is instant
+ * and offline; if it fails (another volume, a locked interpreter) the legacy
+ * venv stays usable and the next install retries.
+ */
+export function migrateLegacyVenv(legacy = LEGACY_VENV_DIR, target = VENV_DIR) {
+  const interpreter = venvPythonIn(legacy)
+  if (existsSync(target) || !existsSync(interpreter)) {
+    return false
+  }
+  try {
+    mkdirSync(dirname(target), { recursive: true })
+    renameSync(legacy, target)
+    log(`moved the virtual environment from ${legacy} to ${target}`)
+    return true
+  }
+  catch (error) {
+    warn(`could not move the legacy virtual environment: ${error instanceof Error ? error.message : String(error)}`)
+    return false
+  }
+}
+
+function venvPythonIn(directory) {
+  return join(directory, IS_WINDOWS ? 'Scripts/python.exe' : 'bin/python')
+}
+
 function ensureVenv(base) {
+  migrateLegacyVenv()
   const existing = workingVenv()
   if (existing) {
     if (existing.directory === LEGACY_VENV_DIR) {
@@ -284,18 +315,20 @@ function verifyImports(python) {
 }
 
 function readConfig() {
-  for (const file of [CONFIG_FILE, LEGACY_CONFIG_FILE]) {
+  // Oldest first so the canonical file and the newest legacy record win key by key.
+  const merged = {}
+  for (const file of [...LEGACY_CONFIG_FILES].reverse().concat(CONFIG_FILE)) {
     if (!existsSync(file)) {
       continue
     }
     try {
-      return JSON.parse(readFileSync(file, 'utf8'))
+      Object.assign(merged, JSON.parse(readFileSync(file, 'utf8')))
     }
     catch {
-      // A malformed file must not hide a usable legacy one.
+      // A malformed file must not hide a usable one.
     }
   }
-  return {}
+  return merged
 }
 
 function writeConfig(config) {
@@ -314,6 +347,14 @@ function writeReadyConfig(existingConfig, python, basePython, report) {
     updatedAt: new Date().toISOString(),
     imports: report,
   })
+  // The canonical file now holds everything the legacy files had; a pre-0.5 file
+  // that stays behind only makes "where is the config" ambiguous.
+  for (const file of LEGACY_CONFIG_FILES) {
+    try {
+      rmSync(file, { force: true })
+    }
+    catch {}
+  }
 }
 
 function main() {

package/python/office_cli.py

@@ -93,22 +93,86 @@ KIT_RELATIVE = Path("node_modules") / "@deepseek-ai" / "libreoffice-kit" / "lib"
 #: Actionable explanation used whenever the kit cannot be located.
 KIT_MISSING = (
     "the LibreOffice renderer is not available: the DSH LibreOffice kit was not found. "
-    "It ships with DeepSeek Harness as @deepseek-ai/libreoffice-kit; install the plugin "
-    "inside the harness tree, or point DSH_LIBREOFFICE_KIT_CLI at <kit>/lib/cli.js"
+    "The plugin records the path in dsh-office-kit's config.json when the harness loads it; "
+    "for a hand-run helper, set DSH_LIBREOFFICE_KIT_CLI to <kit>/lib/cli.js of "
+    "@deepseek-ai/libreoffice-kit"
 )
 
 
+def office_home() -> Path:
+    """Everything this plugin owns lives under `~/.dsh/office` (mirrors src/python.ts)."""
+    return Path.home() / ".dsh" / "office"
+
+
+def config_file() -> Path:
+    """The config.json the host half and postinstall write."""
+    override = os.environ.get("DSH_OFFICE_CONFIG")
+    return Path(override).expanduser() if override else office_home() / "config.json"
+
+
+def legacy_config_files() -> list[Path]:
+    """Config files older versions wrote in the OS config directory, newest first."""
+    if sys.platform == "darwin":
+        primary = Path.home() / "Library" / "Application Support" / "dsh-office-kit" / "config.json"
+    elif sys.platform == "win32":
+        primary = Path(os.environ.get("APPDATA") or Path.home() / "AppData" / "Roaming") / "dsh-office-kit" / "config.json"
+    else:
+        primary = Path(os.environ.get("XDG_CONFIG_HOME") or Path.home() / ".config") / "dsh-office-kit" / "config.json"
+    roaming = Path(os.environ.get("APPDATA") or Path.home() / "AppData" / "Roaming") / "dsh-office-kit" / "config.json"
+    return list(dict.fromkeys([primary, roaming]))
+
+
+def read_config() -> dict[str, Any]:
+    """Merged config: the canonical file wins key by key over older locations."""
+    merged: dict[str, Any] = {}
+    for file in [*reversed(legacy_config_files()), config_file()]:
+        try:
+            parsed = json.loads(file.read_text("utf-8"))
+        except (OSError, ValueError):
+            continue
+        if isinstance(parsed, dict):
+            merged.update(parsed)
+    return merged
+
+
+def saved_kit() -> tuple[str, str] | None:
+    """Return (node, kit CLI) as recorded by the host half, or None.
+
+    This is how a profile install works: the kit lives in the harness tree, which
+    no walk-up from the plugin directory can reach, so the host half looks it up
+    while it loads and stores the result here for every later helper call.
+    """
+    kit = read_config().get("libreofficeKit")
+    if not isinstance(kit, dict):
+        return None
+    cli = kit.get("cli")
+    if not isinstance(cli, str) or not Path(cli).is_file():
+        return None
+    node = kit.get("node")
+    if not isinstance(node, str) or not node:
+        node = os.environ.get("DSH_NODE") or shutil.which("node")
+    if not node:
+        return None
+    return node, cli
+
+
 def find_kit() -> tuple[str, str] | None:
     """Return (node, kit CLI) for the bundled renderer, or None when it is absent.
 
-    Resolution walks up from this file, which covers both layouts that matter:
-    a plugin-local dependency (`<plugin>/node_modules/...`) and the harness
-    install (`<dsh>/node_modules/dsh-office-kit/../../@deepseek-ai/...`).
+    Order: an explicit `DSH_LIBREOFFICE_KIT_CLI`, then the path the host half
+    recorded in config.json (the profile-install case), then a walk up from this
+    file, which covers a checkout or a plugin-local dependency.
     """
-    candidates: list[Path] = []
     override = os.environ.get("DSH_LIBREOFFICE_KIT_CLI")
     if override:
-        candidates.append(Path(override).expanduser())
+        cli = Path(override).expanduser()
+        if cli.is_file():
+            node = os.environ.get("DSH_NODE") or shutil.which("node")
+            return (node, str(cli)) if node else None
+    recorded = saved_kit()
+    if recorded is not None:
+        return recorded
+    candidates: list[Path] = []
     for root in (PACKAGE_ROOT, *PACKAGE_ROOT.parents):
         candidates.append(root / KIT_RELATIVE)
     cli = next((candidate for candidate in candidates if candidate.is_file()), None)