glm-coding-router 0.1.0 → 0.1.1

package/dist/commands/init.js

@@ -3,6 +3,7 @@ import fs from "node:fs";
 import os from "node:os";
 import { loadConfig, saveConfig } from "../core/config.js";
 import { configPath } from "../core/paths.js";
+import { ExitCode } from "../core/errors.js";
 import { runDoctorChecks } from "./doctor.js";
 import { setWindowsUserEnv, ZAI_API_KEY_ENV } from "../core/zai-key.js";
 import { isWindows } from "../core/platform.js";
@@ -16,15 +17,14 @@ function renderEnvironment(results) {
     }
     return lines.join("\n");
 }
-async function askChoices(existingKey, options) {
+async function askChoices(existingKey, options, prompt) {
     if (options.yes) {
-        return { configureKey: !existingKey, claude: true, codex: true, codexSkill: true };
+        return { kind: "ok", choices: { configureKey: !existingKey, claude: true, codex: true, codexSkill: true } };
     }
     if (!process.stdin.isTTY) {
-        process.stdout.write("Non-interactive terminal detected. Re-run with --yes to accept defaults.\n");
-        process.exit(2);
+        return { kind: "non-interactive" };
     }
-    const response = await prompts([
+    const response = await prompt([
         {
             type: existingKey ? null : "confirm",
             name: "configureKey",
@@ -36,27 +36,41 @@ async function askChoices(existingKey, options) {
         { type: "confirm", name: "codexSkill", message: "Install Codex delegation skill?", initial: true },
     ]);
     if (response.claude === undefined) {
-        process.stdout.write("Cancelled.\n");
-        process.exit(1);
+        return { kind: "cancelled" };
     }
     return {
-        configureKey: existingKey ? true : Boolean(response.configureKey),
-        claude: Boolean(response.claude),
-        codex: Boolean(response.codex),
-        codexSkill: Boolean(response.codexSkill),
+        kind: "ok",
+        choices: {
+            configureKey: existingKey ? true : Boolean(response.configureKey),
+            claude: Boolean(response.claude),
+            codex: Boolean(response.codex),
+            codexSkill: Boolean(response.codexSkill),
+        },
     };
 }
 /** glm-router init (spec §7): environment report, key setup, config, skill. Idempotent. */
-export async function initCommand(options) {
+export async function initCommand(options, deps = {}) {
+    const prompt = deps.prompt ?? prompts;
+    const home = deps.home ?? os.homedir();
+    const env = deps.env ?? process.env;
+    const setEnv = deps.setEnv ?? setWindowsUserEnv;
     process.stdout.write(`GLM Coding Router v${version}\n\n`);
-    const report = runDoctorChecks();
+    const report = runDoctorChecks({ home, env, readUserEnv: deps.readUserEnv });
     process.stdout.write("Environment\n\n");
     process.stdout.write(renderEnvironment(report.results) + "\n\n");
     const existingKey = report.keySource !== undefined;
-    const choices = await askChoices(existingKey, options);
+    const askResult = await askChoices(existingKey, options, prompt);
+    if (askResult.kind === "non-interactive") {
+        process.stdout.write("Non-interactive terminal detected. Re-run with --yes to accept defaults.\n");
+        return ExitCode.InvalidArgs;
+    }
+    if (askResult.kind === "cancelled") {
+        process.stdout.write("Cancelled.\n");
+        return ExitCode.GenericFailure;
+    }
+    const choices = askResult.choices;
     process.stdout.write("\nInstalling...\n\n");
-    const home = os.homedir();
-    const config = loadConfig();
+    const config = loadConfig(home);
     const nextConfig = {
         ...config,
         integrations: {
@@ -74,7 +88,7 @@ export async function initCommand(options) {
             process.stdout.write("⚠ Key storage requires Windows in v0.1 — skipped\n");
         }
         else {
-            const keyResponse = await prompts({
+            const keyResponse = await prompt({
                 type: "password",
                 name: "key",
                 message: "Enter Z.ai Coding Plan API key:",
@@ -84,7 +98,7 @@ export async function initCommand(options) {
                 process.stderr.write("Cancelled.\n");
                 return 1;
             }
-            setWindowsUserEnv(ZAI_API_KEY_ENV, String(keyResponse.key).trim());
+            setEnv(ZAI_API_KEY_ENV, String(keyResponse.key).trim());
             process.stdout.write(`✓ ${ZAI_API_KEY_ENV} configured\n`);
         }
     }

package/dist/commands/key.js

@@ -5,9 +5,11 @@ import { logger } from "../core/logging.js";
 import { ZAI_API_KEY_ENV, deleteWindowsUserEnv, resolveZaiApiKey, setWindowsUserEnv, } from "../core/zai-key.js";
 import { emitJson } from "./context.js";
 /** glm-router key set (spec §11): prompt, save to Windows User Environment. */
-export async function keySetCommand(_options) {
+export async function keySetCommand(_options, deps = {}) {
     assertWindows();
-    const response = await prompts({
+    const prompt = deps.prompt ?? prompts;
+    const setEnv = deps.setEnv ?? setWindowsUserEnv;
+    const response = await prompt({
         type: "password",
         name: "key",
         message: "Enter Z.ai Coding Plan API key:",
@@ -19,7 +21,7 @@ export async function keySetCommand(_options) {
     }
     const key = String(response.key).trim();
     try {
-        setWindowsUserEnv(ZAI_API_KEY_ENV, key);
+        setEnv(ZAI_API_KEY_ENV, key);
     }
     catch (error) {
         logger.error(`Failed to write the Windows User Environment: ${error instanceof Error ? error.message : String(error)}`);
@@ -30,8 +32,8 @@ export async function keySetCommand(_options) {
     return 0;
 }
 /** glm-router key check (spec §11): report presence + source; never the value. */
-export function keyCheckCommand(options) {
-    const resolved = resolveZaiApiKey();
+export function keyCheckCommand(options, deps = {}) {
+    const resolved = resolveZaiApiKey({ env: deps.env, readUserEnv: deps.readUserEnv });
     if (options.json) {
         emitJson({
             configured: Boolean(resolved),
@@ -50,7 +52,8 @@ export function keyCheckCommand(options) {
     return 0;
 }
 /** Used by uninstall; keeps the key by default (spec §44). */
-export async function keyRemoveCommand() {
+export async function keyRemoveCommand(deps = {}) {
     assertWindows();
-    deleteWindowsUserEnv(ZAI_API_KEY_ENV);
+    const deleteEnv = deps.deleteEnv ?? deleteWindowsUserEnv;
+    deleteEnv(ZAI_API_KEY_ENV);
 }

package/dist/commands/project-init.js

@@ -29,15 +29,15 @@ export function renderChangeDiff(change) {
  * CLAUDE.md and AGENTS.md at the project root. Idempotent; never overwrites
  * user content; supports --dry-run.
  */
-export function projectInitCommand(options) {
-    const root = findProjectRoot();
-    const config = loadConfig();
+export function projectInitCommand(options, deps = {}) {
+    const root = deps.root ?? findProjectRoot();
+    const config = loadConfig(deps.home);
     const changes = [];
     if (config.integrations.claude) {
-        changes.push(installClaudeIntegration(root, { dryRun: options.dryRun }));
+        changes.push(installClaudeIntegration(root, { home: deps.home, dryRun: options.dryRun }));
     }
     if (config.integrations.codex) {
-        changes.push(installCodexIntegration(root, { dryRun: options.dryRun }));
+        changes.push(installCodexIntegration(root, { home: deps.home, dryRun: options.dryRun }));
     }
     if (changes.length === 0) {
         process.stdout.write("All integrations are disabled in config — nothing to do.\n");
@@ -45,7 +45,12 @@ export function projectInitCommand(options) {
     }
     for (const change of changes) {
         if (options.dryRun) {
-            process.stdout.write(`[dry-run] would update ${change.file}:\n${renderChangeDiff(change)}\n`);
+            if (change.changed) {
+                process.stdout.write(`[dry-run] would update ${change.file}:\n${renderChangeDiff(change)}\n`);
+            }
+            else {
+                process.stdout.write(`[dry-run] ${change.file} — already up to date\n`);
+            }
         }
         else if (!change.changed) {
             process.stdout.write(`✓ ${change.file} — already up to date\n`);

package/dist/commands/status.js

@@ -7,20 +7,21 @@ import { CodexSkillInstaller } from "../integrations/skill.js";
 import { GLM_DELEGATION_SKILL_NAME } from "../templates/glm-delegation-skill.js";
 import { emitJson } from "./context.js";
 /** Fast, fully offline summary (spec §41) — no API requests, no key values. */
-export function statusCommand(options) {
-    const config = loadConfig();
-    const home = os.homedir();
-    const resolved = resolveZaiApiKey();
+export function statusCommand(options, deps = {}) {
+    const home = deps.home ?? os.homedir();
+    const env = deps.env ?? process.env;
+    const config = loadConfig(home);
+    const resolved = resolveZaiApiKey({ env, readUserEnv: deps.readUserEnv });
     const claudeInstalled = (() => {
         try {
-            locateClaude(config);
+            locateClaude(config, env);
             return true;
         }
         catch {
             return false;
         }
     })();
-    const codexInstalled = Boolean(locateCodex(config));
+    const codexInstalled = Boolean(locateCodex(config, env));
     const skillInstaller = new CodexSkillInstaller(home);
     const skillInstalled = skillInstaller.detect() !== null && skillInstaller.isInstalled(GLM_DELEGATION_SKILL_NAME);
     if (options.json) {

package/dist/commands/uninstall.js

@@ -2,10 +2,11 @@ import prompts from "prompts";
 import fs from "node:fs";
 import os from "node:os";
 import { configDir } from "../core/paths.js";
+import { ExitCode } from "../core/errors.js";
 import { CodexSkillInstaller, glmDelegationSkill } from "../integrations/skill.js";
 import { keyRemoveCommand } from "./key.js";
 import { removeClaudeIntegration, removeCodexIntegration } from "../integrations/index.js";
-async function askChoices(options) {
+async function askChoices(options, prompt) {
     // Defaults per spec §44: keep ZAI_API_KEY; destructive credential removal
     // requires explicit consent, so --yes never flips it.
     const defaults = {
@@ -15,33 +16,46 @@ async function askChoices(options) {
         removeKey: false,
     };
     if (options.yes || options.force) {
-        return options.force ? { ...defaults, removeProjectIntegration: true } : defaults;
+        return {
+            kind: "ok",
+            choices: options.force ? { ...defaults, removeProjectIntegration: true } : defaults,
+        };
     }
     if (!process.stdin.isTTY) {
-        process.stdout.write("Non-interactive terminal detected. Re-run with --yes for safe defaults.\n");
-        process.exit(2);
+        return { kind: "non-interactive" };
     }
-    const response = await prompts([
+    const response = await prompt([
         { type: "confirm", name: "removeConfig", message: "Remove global configuration?", initial: true },
         { type: "confirm", name: "removeSkill", message: "Remove Codex skill?", initial: true },
         { type: "confirm", name: "removeProject", message: "Remove current project integration?", initial: false },
         { type: "confirm", name: "removeKey", message: "Remove ZAI_API_KEY?", initial: false },
     ]);
     if (response.removeConfig === undefined) {
-        process.stdout.write("Cancelled.\n");
-        process.exit(1);
+        return { kind: "cancelled" };
     }
     return {
-        removeConfig: Boolean(response.removeConfig),
-        removeSkill: Boolean(response.removeSkill),
-        removeProjectIntegration: Boolean(response.removeProject),
-        removeKey: Boolean(response.removeKey),
+        kind: "ok",
+        choices: {
+            removeConfig: Boolean(response.removeConfig),
+            removeSkill: Boolean(response.removeSkill),
+            removeProjectIntegration: Boolean(response.removeProject),
+            removeKey: Boolean(response.removeKey),
+        },
     };
 }
 /** glm-router uninstall (spec §44): wizard with safe defaults. */
-export async function uninstallCommand(options) {
-    const choices = await askChoices(options);
-    const home = os.homedir();
+export async function uninstallCommand(options, deps = {}) {
+    const askResult = await askChoices(options, deps.prompt ?? prompts);
+    if (askResult.kind === "non-interactive") {
+        process.stdout.write("Non-interactive terminal detected. Re-run with --yes for safe defaults.\n");
+        return ExitCode.InvalidArgs;
+    }
+    if (askResult.kind === "cancelled") {
+        process.stdout.write("Cancelled.\n");
+        return ExitCode.GenericFailure;
+    }
+    const choices = askResult.choices;
+    const home = deps.home ?? os.homedir();
     if (choices.removeSkill) {
         const skillInstaller = new CodexSkillInstaller(home);
         const skill = glmDelegationSkill();
@@ -54,7 +68,7 @@ export async function uninstallCommand(options) {
         }
     }
     if (choices.removeProjectIntegration) {
-        const root = process.cwd();
+        const root = deps.root ?? process.cwd();
         removeClaudeIntegration(root);
         removeCodexIntegration(root);
         process.stdout.write("✓ Project integration removed (CLAUDE.md / AGENTS.md managed blocks)\n");
@@ -70,7 +84,7 @@ export async function uninstallCommand(options) {
         }
     }
     if (choices.removeKey) {
-        await keyRemoveCommand();
+        await keyRemoveCommand({ deleteEnv: deps.deleteEnv });
         process.stdout.write("✓ ZAI_API_KEY removed from Windows User Environment\n");
     }
     else {

package/dist/core/claude.js

@@ -3,13 +3,14 @@ import fs from "node:fs";
 import path from "node:path";
 import { Errors } from "./errors.js";
 import { isWindows } from "./platform.js";
-function runWhere(name) {
+function runWhere(name, env = process.env) {
     const finder = isWindows() ? "where.exe" : "which";
     try {
         const output = execFileSync(finder, [name], {
             encoding: "utf8",
             windowsHide: true,
             stdio: ["ignore", "pipe", "ignore"],
+            env,
         });
         return output
             .split(/\r?\n/)
@@ -55,14 +56,17 @@ function preferNative(matches) {
  *   2. PATH search through Node
  *   3. config override
  *   4. error
+ *
+ * `env` defaults to `process.env` and is threaded through for testability —
+ * production callers never pass it.
  */
-export function locateClaude(config) {
-    const matches = runWhere("claude");
+export function locateClaude(config, env = process.env) {
+    const matches = runWhere("claude", env);
     const fromWhere = preferNative(matches);
     if (fromWhere) {
         return fromWhere;
     }
-    const fromPath = searchPathFor("claude");
+    const fromPath = searchPathFor("claude", env);
     if (fromPath) {
         return fromPath;
     }
@@ -80,12 +84,12 @@ export function locateClaude(config) {
  * the tool must work with a Claude-only setup. `required` callers
  * (glm commands that need codex) get a proper error.
  */
-export function locateCodex(config) {
-    const matches = runWhere("codex");
+export function locateCodex(config, env = process.env) {
+    const matches = runWhere("codex", env);
     if (matches.length > 0) {
         return preferNative(matches);
     }
-    const fromPath = searchPathFor("codex");
+    const fromPath = searchPathFor("codex", env);
     if (fromPath) {
         return fromPath;
     }
@@ -95,8 +99,8 @@ export function locateCodex(config) {
     }
     return undefined;
 }
-export function codexRequired(config) {
-    const found = locateCodex(config);
+export function codexRequired(config, env = process.env) {
+    const found = locateCodex(config, env);
     if (!found) {
         throw Errors.codexNotFound();
     }

package/dist/templates/agents-block.js

@@ -1,46 +1,46 @@
 /** Managed block content for AGENTS.md (spec §24). Keep in sync with the spec. */
-export const AGENTS_MANAGED_BLOCK = `<!-- glm-coding-router:start -->
-
-## GLM Worker Delegation
-
-Available commands:
-
-- \`glm-worker "<task>"\`
-- \`glm-review "<task>"\`
-
-Codex is the primary orchestrator.
-
-Delegate:
-- CRUD
-- boilerplate
-- tests
-- documentation
-- mechanical refactoring
-- repository exploration
-- straightforward implementation
-
-Keep in Codex:
-- requirements
-- planning
-- architecture
-- ambiguous business logic
-- complex debugging
-- security decisions
-- integration
-- final review
-
-Before delegation create a task packet:
-
-TASK
-SCOPE
-FILES ALLOWED TO MODIFY
-FILES NOT TO MODIFY
-REQUIREMENTS
-CONSTRAINTS
-ACCEPTANCE CRITERIA
-VALIDATION
-EXPECTED OUTPUT
-
-Never trust a worker's success report without inspecting the resulting changes.
-
+export const AGENTS_MANAGED_BLOCK = `<!-- glm-coding-router:start -->
+
+## GLM Worker Delegation
+
+Available commands:
+
+- \`glm-worker "<task>"\`
+- \`glm-review "<task>"\`
+
+Codex is the primary orchestrator.
+
+Delegate:
+- CRUD
+- boilerplate
+- tests
+- documentation
+- mechanical refactoring
+- repository exploration
+- straightforward implementation
+
+Keep in Codex:
+- requirements
+- planning
+- architecture
+- ambiguous business logic
+- complex debugging
+- security decisions
+- integration
+- final review
+
+Before delegation create a task packet:
+
+TASK
+SCOPE
+FILES ALLOWED TO MODIFY
+FILES NOT TO MODIFY
+REQUIREMENTS
+CONSTRAINTS
+ACCEPTANCE CRITERIA
+VALIDATION
+EXPECTED OUTPUT
+
+Never trust a worker's success report without inspecting the resulting changes.
+
 <!-- glm-coding-router:end -->`;

package/dist/templates/claude-block.js

@@ -1,49 +1,49 @@
 /** Managed block content for CLAUDE.md (spec §20). Keep in sync with the spec. */
-export const CLAUDE_MANAGED_BLOCK = `<!-- glm-coding-router:start -->
-
-## GLM Worker Delegation
-
-GLM workers available:
-
-- \`glm-worker "<task>"\`
-- \`glm-review "<task>"\`
-
-Delegate well-scoped, implementation-heavy work to GLM.
-
-Use GLM for:
-- repository exploration
-- CRUD
-- boilerplate
-- tests
-- documentation
-- mechanical refactoring
-- straightforward implementation
-
-Claude remains responsible for:
-- requirements
-- architecture
-- ambiguous business rules
-- security-sensitive decisions
-- complex debugging
-- integration
-- final review
-
-Before delegation, define:
-- task
-- scope
-- allowed files
-- forbidden files
-- requirements
-- constraints
-- acceptance criteria
-- validation command
-- expected output
-
-After worker completion:
-1. inspect the actual diff
-2. validate against requirements
-3. run relevant tests
-4. resolve integration problems
-5. accept only after verification
-
+export const CLAUDE_MANAGED_BLOCK = `<!-- glm-coding-router:start -->
+
+## GLM Worker Delegation
+
+GLM workers available:
+
+- \`glm-worker "<task>"\`
+- \`glm-review "<task>"\`
+
+Delegate well-scoped, implementation-heavy work to GLM.
+
+Use GLM for:
+- repository exploration
+- CRUD
+- boilerplate
+- tests
+- documentation
+- mechanical refactoring
+- straightforward implementation
+
+Claude remains responsible for:
+- requirements
+- architecture
+- ambiguous business rules
+- security-sensitive decisions
+- complex debugging
+- integration
+- final review
+
+Before delegation, define:
+- task
+- scope
+- allowed files
+- forbidden files
+- requirements
+- constraints
+- acceptance criteria
+- validation command
+- expected output
+
+After worker completion:
+1. inspect the actual diff
+2. validate against requirements
+3. run relevant tests
+4. resolve integration problems
+5. accept only after verification
+
 <!-- glm-coding-router:end -->`;