glm-coding-router 1.1.2 → 2.1.0

package/dist/templates/agents-block.js

@@ -1,46 +1,55 @@
 /** 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.
+
+Worker exit codes 41 and 42 are NOT crashes:
+
+- stdout carries one JSON object: \`{"status":"handoff_required", ...}\`
+- read \`handoff_path\` — a handoff.md with what was done, what remains, files
+  changed, and untracked files that are NOT in diff.patch
+- continue the task yourself in the SAME worktree, starting from "Remaining"
+- do not re-run the worker until quota resets
+- 41 means nothing was spawned; 42 means work was done and is preserved
+
 <!-- glm-coding-router:end -->`;

package/dist/templates/claude-block.js

@@ -1,49 +1,58 @@
 /** 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
+
+Worker exit codes 41 and 42 are NOT crashes:
+
+- stdout carries one JSON object: \`{"status":"handoff_required", ...}\`
+- read \`handoff_path\` — a handoff.md with what was done, what remains, files
+  changed, and untracked files that are NOT in diff.patch
+- continue the task yourself in the SAME worktree, starting from "Remaining"
+- do not re-run the worker until quota resets
+- 41 means nothing was spawned; 42 means work was done and is preserved
+
 <!-- glm-coding-router:end -->`;

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

@@ -1,68 +1,79 @@
 /** 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
+
+## Exit codes 41 and 42 are not crashes
+
+A worker may stop because the GLM quota ran low rather than because it broke.
+
+- stdout carries one JSON object: \`{"status":"handoff_required", ...}\`
+- read \`handoff_path\` — a handoff.md with what was done, what remains, files
+  changed, and untracked files that are NOT in diff.patch
+- continue the task yourself in the SAME worktree, starting from "Remaining"
+- do not re-run the worker until quota resets
+- 41 means nothing was spawned; 42 means work was done and is preserved
 `;

package/dist/tui/command-ui.js

@@ -0,0 +1,158 @@
+/**
+ * Shared presentation layer for the management screens — `doctor`, `status`,
+ * `usage`, and the root landing page (specs/terminal-ui-doctor.md §A). This
+ * module never fetches data, reads credentials, or makes routing decisions;
+ * it only turns already-computed values into aligned, width-aware text.
+ *
+ * All ANSI stays in render.ts (`paint`); this file composes plain strings
+ * and colors them through that one choke point.
+ */
+import { displayWidth, padEndDisplay, paint, truncate } from "./render.js";
+const STATUS_TAG = {
+    ok: "[OK]",
+    warn: "[WARN]",
+    fail: "[FAIL]",
+    info: "[INFO]",
+};
+const STATUS_STYLE = {
+    ok: "green",
+    warn: "yellow",
+    fail: "red",
+    info: "cyan",
+};
+/** Widest tag ("[WARN]"/"[FAIL]"/"[INFO]") plus one separating space. */
+const TAG_COLUMN = 7;
+const BASE_INDENT = "  ";
+const LABEL_COLUMN = 24;
+const MIN_BOX_WIDTH = 44;
+const MAX_BOX_WIDTH = 78;
+const BAR_SLOTS = 20;
+function effectiveWidth(writer) {
+    return Number.isFinite(writer.columns) && writer.columns > 0 ? Math.floor(writer.columns) : 80;
+}
+/** Strip control characters from text that did not originate in this module (spec §A). */
+function sanitize(text, allowNewline) {
+    const pattern = allowNewline ? /[\x00-\x09\x0B\x0C\x0E-\x1F\x7F]/g : /[\x00-\x1F\x7F]/g;
+    return text.replace(pattern, "");
+}
+function border(width) {
+    return `+${"-".repeat(Math.max(width - 2, 0))}+`;
+}
+function boxLine(text, width) {
+    const inner = Math.max(width - 4, 0);
+    return `| ${padEndDisplay(truncate(text, inner), inner)} |`;
+}
+/**
+ * Wrap `text` to `width`-wide lines (measured in display columns, not code
+ * units — spec §A "Width uses display cells") without breaking a word
+ * mid-way when avoidable.
+ */
+function wrap(text, width) {
+    if (width <= 0)
+        return [text];
+    const words = text.split(/\s+/).filter((w) => w.length > 0);
+    const lines = [];
+    let current = "";
+    for (const word of words) {
+        const candidate = current.length === 0 ? word : `${current} ${word}`;
+        if (displayWidth(candidate) > width && current.length > 0) {
+            lines.push(current);
+            current = word;
+        }
+        else {
+            current = candidate;
+        }
+    }
+    if (current.length > 0)
+        lines.push(current);
+    return lines.length > 0 ? lines : [""];
+}
+export function createCommandUi(writer, options = {}) {
+    const quiet = options.quiet ?? false;
+    return {
+        header(title, subtitle) {
+            if (quiet)
+                return "";
+            const width = effectiveWidth(writer);
+            const cleanTitle = sanitize(title, false);
+            const cleanSubtitle = subtitle !== undefined ? sanitize(subtitle, false) : undefined;
+            if (width < MIN_BOX_WIDTH) {
+                const lines = [paint(writer, "cyan", truncate(cleanTitle, width))];
+                if (cleanSubtitle)
+                    lines.push(paint(writer, "dim", truncate(cleanSubtitle, width)));
+                return lines.join("\n");
+            }
+            const boxWidth = Math.min(width, MAX_BOX_WIDTH);
+            const lines = [border(boxWidth), boxLine(cleanTitle, boxWidth)];
+            if (cleanSubtitle)
+                lines.push(boxLine(cleanSubtitle, boxWidth));
+            lines.push(border(boxWidth));
+            return lines.join("\n");
+        },
+        section(title) {
+            return paint(writer, "bold", sanitize(title, false));
+        },
+        row(label, value, status) {
+            const width = effectiveWidth(writer);
+            const cleanLabel = sanitize(label, false);
+            const cleanValue = sanitize(value, false);
+            const tagField = status !== undefined ? STATUS_TAG[status].padEnd(TAG_COLUMN) : "";
+            const labelField = padEndDisplay(cleanLabel, LABEL_COLUMN);
+            const plainPrefix = `${BASE_INDENT}${tagField}${labelField}`;
+            const singleLine = `${plainPrefix}${cleanValue}`;
+            const coloredPrefix = () => {
+                if (status === undefined)
+                    return plainPrefix;
+                const tag = STATUS_TAG[status];
+                return `${BASE_INDENT}${paint(writer, STATUS_STYLE[status], tag)}${tagField.slice(tag.length)}${labelField}`;
+            };
+            if (cleanValue.length === 0 || displayWidth(singleLine.replace(/\s+$/, "")) <= width) {
+                return `${coloredPrefix()}${cleanValue}`.replace(/\s+$/, "");
+            }
+            // Stack: tag + label on one line, value wrapped and indented beneath.
+            const tagLine = status !== undefined
+                ? `${BASE_INDENT}${paint(writer, STATUS_STYLE[status], STATUS_TAG[status])} ${cleanLabel}`
+                : `${BASE_INDENT}${cleanLabel}`;
+            const valueIndent = `${BASE_INDENT}${" ".repeat(TAG_COLUMN)}`;
+            const valueWidth = Math.max(width - valueIndent.length, 8);
+            const valueLines = wrap(cleanValue, valueWidth).map((line) => `${valueIndent}${line}`);
+            return [tagLine, ...valueLines].join("\n");
+        },
+        detail(text) {
+            const width = effectiveWidth(writer);
+            const indent = `${BASE_INDENT}${" ".repeat(TAG_COLUMN - 2)}`;
+            const availableWidth = Math.max(width - indent.length, 8);
+            const cleanText = sanitize(text, true);
+            const lines = [];
+            for (const paragraph of cleanText.split("\n")) {
+                if (paragraph.length === 0) {
+                    lines.push("");
+                    continue;
+                }
+                for (const line of wrap(paragraph, availableWidth)) {
+                    lines.push(`${indent}${line}`);
+                }
+            }
+            return lines.join("\n");
+        },
+        footer(text) {
+            if (quiet)
+                return "";
+            const cleanText = sanitize(text, true);
+            return cleanText
+                .split("\n")
+                .map((line) => (line.length === 0 ? "" : `${BASE_INDENT}${line}`))
+                .join("\n");
+        },
+        bar(percent) {
+            if (percent === undefined || !Number.isFinite(percent)) {
+                return `[${"-".repeat(BAR_SLOTS)}] unknown`;
+            }
+            const clamped = Math.min(100, Math.max(0, percent));
+            const filled = Math.round((clamped / 100) * BAR_SLOTS);
+            const bar = `[${"#".repeat(filled)}${"-".repeat(BAR_SLOTS - filled)}]`;
+            const style = clamped >= 90 ? "red" : clamped >= 70 ? "yellow" : "green";
+            return `${paint(writer, style, bar)} ${Math.round(clamped)}% used`;
+        },
+    };
+}