glm-coding-router 0.2.0 → 0.4.0

package/dist/core/worktree.js

@@ -0,0 +1,118 @@
+import fs from "node:fs";
+import path from "node:path";
+import { runGit } from "./git.js";
+import { Errors } from "./errors.js";
+import { logger } from "./logging.js";
+/** Branch prefix for delegate worktrees (specs/delegate-worktrees.md). */
+export const DELEGATE_BRANCH_PREFIX = "glm/delegate/";
+/** Valid delegate slugs: no path separators, spaces, or leading dash (specs/delegate-worktrees.md). */
+export function validateDelegateName(name) {
+    if (!/^[A-Za-z0-9][A-Za-z0-9._-]*$/.test(name) || name === "." || name === ".." || name.toLowerCase() === ".git") {
+        throw Errors.invalidDelegateName(name);
+    }
+}
+export function delegateBranch(name) {
+    return `${DELEGATE_BRANCH_PREFIX}${name}`;
+}
+/**
+ * Worktrees live outside the repo so the main checkout's status stays clean
+ * and no .gitignore edit is ever needed: <parent-of-root>/<repo>.glm-worktrees/<name>
+ */
+export function worktreeBaseDir(repoRoot) {
+    return path.join(path.dirname(repoRoot), `${path.basename(repoRoot)}.glm-worktrees`);
+}
+export function delegateWorktreePath(repoRoot, name) {
+    return path.join(worktreeBaseDir(repoRoot), name);
+}
+function ok(result) {
+    return result.code === 0;
+}
+/** True when a local branch already exists (rev-parse --verify --quiet). */
+export async function branchExists(repoRoot, branch, deps = {}) {
+    const run = deps.runGit ?? runGit;
+    const result = await run(["rev-parse", "--verify", "--quiet", `refs/heads/${branch}`], repoRoot);
+    return ok(result);
+}
+/** True when HEAD resolves (the repo has at least one commit). */
+export async function headIsBorn(repoRoot, deps = {}) {
+    const run = deps.runGit ?? runGit;
+    const result = await run(["rev-parse", "--verify", "--quiet", "HEAD"], repoRoot);
+    return ok(result);
+}
+/**
+ * Pre-flight collision checks shared by the real run and --dry-run
+ * (specs/delegate-worktrees.md): fail before creating anything.
+ */
+export async function assertWorktreeAvailable(repoRoot, name, deps = {}) {
+    const existsSync = deps.existsSync ?? fs.existsSync;
+    const branch = delegateBranch(name);
+    const worktreePath = delegateWorktreePath(repoRoot, name);
+    if (await branchExists(repoRoot, branch, deps)) {
+        throw Errors.worktreeFailed("branch", `branch ${branch} already exists`, [
+            "Inspect or merge it first:",
+            "",
+            `  git log ${branch}`,
+            `  git merge ${branch}`,
+            "",
+            `or delete it if it is unwanted:`,
+            "",
+            `  git branch -D ${branch}`,
+        ]);
+    }
+    if (existsSync(worktreePath)) {
+        throw Errors.worktreeFailed("worktree add", `path already exists: ${worktreePath}`, [
+            "A previous worktree directory is in the way:",
+            "",
+            `  ${worktreePath}`,
+            "",
+            "Remove it (or run `git worktree prune`) and re-run the delegate command.",
+        ]);
+    }
+    if (!(await headIsBorn(repoRoot, deps))) {
+        throw Errors.worktreeFailed("worktree add", "HEAD does not point at a commit (repository has no commits yet)", [
+            "delegate creates the worktree from HEAD, so the repository needs at least one commit first:",
+            "",
+            "  git add -A && git commit -m \"initial commit\"",
+        ]);
+    }
+    return worktreePath;
+}
+/** Create the delegate worktree + branch from HEAD. Returns the worktree path. */
+export async function createDelegateWorktree(repoRoot, name, deps = {}) {
+    const run = deps.runGit ?? runGit;
+    const worktreePath = await assertWorktreeAvailable(repoRoot, name, deps);
+    const branch = delegateBranch(name);
+    logger.debug(`git worktree add -b ${branch} ${worktreePath}`);
+    const result = await run(["worktree", "add", "-b", branch, worktreePath], repoRoot);
+    if (!ok(result)) {
+        throw Errors.worktreeFailed("worktree add", result.stderr || result.stdout, ["The worktree was not created; the repository was left as it was."]);
+    }
+    return worktreePath;
+}
+/**
+ * Remove a delegate worktree with plain `git worktree remove` — git refuses
+ * on dirty/untracked trees, which is exactly the safety we want
+ * (specs/delegate-worktrees.md, `--remove`). Never touches the branch.
+ * Returns undefined when removal succeeded, or git's complaint when not.
+ */
+export async function removeDelegateWorktree(repoRoot, worktreePath, deps = {}) {
+    const run = deps.runGit ?? runGit;
+    const result = await run(["worktree", "remove", worktreePath], repoRoot);
+    if (ok(result)) {
+        return undefined;
+    }
+    return result.stderr || result.stdout || "git worktree remove failed";
+}
+/**
+ * Delete a branch this invocation created and never advanced (worker never
+ * started). Only ever called on the spawn-failure rollback path, where the
+ * branch provably sits at HEAD — never on branches a worker may have moved.
+ */
+export async function rollbackDelegateBranch(repoRoot, branch, deps = {}) {
+    const run = deps.runGit ?? runGit;
+    const result = await run(["branch", "-D", branch], repoRoot);
+    if (ok(result)) {
+        return undefined;
+    }
+    return result.stderr || result.stdout || "git branch -D failed";
+}

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/benchmark-tasks.js

@@ -0,0 +1,70 @@
+export const BENCHMARK_TASKS = [
+    {
+        id: "fn-reverse",
+        description: "implement reverseWords from a stub until node test.js passes",
+        files: {
+            "src/util.js": [
+                "// Implement reverseWords(str): reverse the ORDER of the words in str.",
+                '// Example: reverseWords("hello world") === "world hello".',
+                "// Collapse extra whitespace between words and trim the ends.",
+                "function reverseWords(str) {",
+                "  // TODO: implement",
+                "}",
+                "",
+                "module.exports = { reverseWords };",
+                "",
+            ].join("\n"),
+            "test.js": [
+                'const assert = require("node:assert");',
+                'const { reverseWords } = require("./src/util.js");',
+                'assert.strictEqual(reverseWords("hello world"), "world hello");',
+                'assert.strictEqual(reverseWords("a"), "a");',
+                'assert.strictEqual(reverseWords("  spaced   out  "), "out spaced");',
+                'assert.strictEqual(reverseWords(""), "");',
+                'console.log("PASS");',
+                "",
+            ].join("\n"),
+        },
+        prompt: [
+            "Implement reverseWords in src/util.js so that `node test.js` passes.",
+            "Words are separated by whitespace; collapse repeats and trim the ends.",
+            "Do not modify test.js. Verify by running `node test.js` with Bash.",
+        ].join(" "),
+        validate: ["node", "test.js"],
+    },
+    {
+        id: "fix-bug",
+        description: "repair an even-length median bug until node test.js passes",
+        files: {
+            "stats.js": [
+                "// median(values) returns the median of a non-empty list of numbers.",
+                "function median(values) {",
+                "  const sorted = [...values].sort((a, b) => a - b);",
+                "  const mid = Math.floor(sorted.length / 2);",
+                "  return sorted[mid]; // BUG: wrong for even-length lists",
+                "}",
+                "",
+                "module.exports = { median };",
+                "",
+            ].join("\n"),
+            "test.js": [
+                'const assert = require("node:assert");',
+                'const { median } = require("./stats.js");',
+                "assert.strictEqual(median([3, 1, 2]), 2);",
+                "assert.strictEqual(median([4, 1, 3, 2]), 2.5);",
+                "assert.strictEqual(median([5]), 5);",
+                'console.log("PASS");',
+                "",
+            ].join("\n"),
+        },
+        prompt: [
+            "stats.js has a bug: median() returns the wrong result for even-length lists.",
+            "Fix it so that `node test.js` passes (even lists return the average of the two middle values).",
+            "Do not modify test.js. Verify by running `node test.js` with Bash.",
+        ].join(" "),
+        validate: ["node", "test.js"],
+    },
+];
+export function benchmarkTaskById(id) {
+    return BENCHMARK_TASKS.find((task) => task.id === id);
+}

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 -->`;

package/dist/templates/glm-delegation-skill.js

@@ -1,68 +1,68 @@
 /** Codex skill definition (spec §26). Keep in sync with the spec. */
 export const GLM_DELEGATION_SKILL_NAME = "glm-delegation";
-export const GLM_DELEGATION_SKILL_MD = `---
-name: glm-delegation
-description: >
-  Delegate well-scoped implementation, testing,
-  repository exploration, boilerplate, CRUD,
-  documentation, and mechanical refactoring to
-  GLM Coding Plan workers.
----
-
-# GLM Delegation
-
-Available commands:
-
-glm-worker "<task>"
-glm-review "<task>"
-
-## Use glm-review for
-
-- repository exploration
-- dependency analysis
-- locating implementations
-- call-chain discovery
-- code review
-
-## Use glm-worker for
-
-- CRUD
-- unit tests
-- implementation
-- documentation
-- repetitive changes
-- mechanical refactoring
-
-## Keep in primary Codex agent
-
-- requirements
-- architecture
-- ambiguous rules
-- security-sensitive design
-- difficult debugging
-- integration
-- final acceptance
-
-## Delegation packet
-
-Always provide:
-
-TASK
-SCOPE
-ALLOWED FILES
-FORBIDDEN FILES
-REQUIREMENTS
-CONSTRAINTS
-ACCEPTANCE CRITERIA
-VALIDATION
-EXPECTED OUTPUT
-
-## Verification
-
-After GLM finishes:
-
-- inspect the actual diff
-- independently run relevant validation
-- compare implementation with requirements
-- reject or correct worker output when needed
+export const GLM_DELEGATION_SKILL_MD = `---
+name: glm-delegation
+description: >
+  Delegate well-scoped implementation, testing,
+  repository exploration, boilerplate, CRUD,
+  documentation, and mechanical refactoring to
+  GLM Coding Plan workers.
+---
+
+# GLM Delegation
+
+Available commands:
+
+glm-worker "<task>"
+glm-review "<task>"
+
+## Use glm-review for
+
+- repository exploration
+- dependency analysis
+- locating implementations
+- call-chain discovery
+- code review
+
+## Use glm-worker for
+
+- CRUD
+- unit tests
+- implementation
+- documentation
+- repetitive changes
+- mechanical refactoring
+
+## Keep in primary Codex agent
+
+- requirements
+- architecture
+- ambiguous rules
+- security-sensitive design
+- difficult debugging
+- integration
+- final acceptance
+
+## Delegation packet
+
+Always provide:
+
+TASK
+SCOPE
+ALLOWED FILES
+FORBIDDEN FILES
+REQUIREMENTS
+CONSTRAINTS
+ACCEPTANCE CRITERIA
+VALIDATION
+EXPECTED OUTPUT
+
+## Verification
+
+After GLM finishes:
+
+- inspect the actual diff
+- independently run relevant validation
+- compare implementation with requirements
+- reject or correct worker output when needed
 `;

package/package.json

@@ -1,47 +1,47 @@
-{
-  "name": "glm-coding-router",
-  "version": "0.2.0",
-  "description": "GLM Coding Plan workers for Claude Code and Codex",
-  "type": "module",
-  "license": "MIT",
-  "author": "hieu9721",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/hieu9721/GLM-coding-router.git"
-  },
-  "bin": {
-    "glm-router": "./dist/cli.js",
-    "glm-chat": "./dist/bin/glm-chat.js",
-    "glm-worker": "./dist/bin/glm-worker.js",
-    "glm-review": "./dist/bin/glm-review.js",
-    "glm-fast": "./dist/bin/glm-fast.js"
-  },
-  "files": [
-    "dist"
-  ],
-  "scripts": {
-    "dev": "tsx src/cli.ts",
-    "build": "tsc",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "lint": "eslint src tests",
-    "prepublishOnly": "npm run build && npm test"
-  },
-  "engines": {
-    "node": ">=20"
-  },
-  "dependencies": {
-    "commander": "^15.0.0",
-    "prompts": "^2.4.2",
-    "zod": "^4.6.5"
-  },
-  "devDependencies": {
-    "@types/node": "^22.20.3",
-    "@types/prompts": "^2.4.9",
-    "eslint": "^9.39.5",
-    "tsx": "^4.23.13",
-    "typescript": "^5.9.3",
-    "typescript-eslint": "^8.70.0",
-    "vitest": "^5.0.1"
-  }
-}
+{
+  "name": "glm-coding-router",
+  "version": "0.4.0",
+  "description": "GLM Coding Plan workers for Claude Code and Codex",
+  "type": "module",
+  "license": "MIT",
+  "author": "hieu9721",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/hieu9721/GLM-coding-router.git"
+  },
+  "bin": {
+    "glm-router": "./dist/cli.js",
+    "glm-chat": "./dist/bin/glm-chat.js",
+    "glm-worker": "./dist/bin/glm-worker.js",
+    "glm-review": "./dist/bin/glm-review.js",
+    "glm-fast": "./dist/bin/glm-fast.js"
+  },
+  "files": [
+    "dist"
+  ],
+  "scripts": {
+    "dev": "tsx src/cli.ts",
+    "build": "tsc",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "lint": "eslint src tests",
+    "prepublishOnly": "npm run build && npm test"
+  },
+  "engines": {
+    "node": ">=20"
+  },
+  "dependencies": {
+    "commander": "^15.0.0",
+    "prompts": "^2.4.2",
+    "zod": "^4.6.5"
+  },
+  "devDependencies": {
+    "@types/node": "^22.20.3",
+    "@types/prompts": "^2.4.9",
+    "eslint": "^9.39.5",
+    "tsx": "^4.23.13",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.70.0",
+    "vitest": "^5.0.1"
+  }
+}