@opengsd/gsd-core 1.7.0 → 1.9.0

package/skills/gsd-plan-phase/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: gsd-plan-phase
 description: "Create detailed phase plan (PLAN.md) with verification loop"
-argument-hint: "[phase] [--auto] [--research] [--skip-research] [--research-phase <N>] [--view] [--gaps] [--skip-verify] [--prd <file>] [--ingest <path-or-glob>] [--ingest-format <auto|nygard|madr|narrative>] [--reviews] [--text] [--tdd] [--mvp]"
+argument-hint: "[phase] [--auto] [--research] [--skip-research] [--research-phase <N>] [--view] [--gaps] [--skip-verify] [--prd <file>] [--ingest <path-or-glob>] [--ingest-format <auto|nygard|madr|narrative>] [--reviews] [--text] [--tdd] [--mvp] [--no-tracer] [--no-reversibility-gates]"
 effort: max
 allowed-tools:
   - Read
@@ -40,7 +40,7 @@ Create executable phase prompts (PLAN.md files) for a roadmap phase with integra
 </runtime_note>
 
 <context>
-Phase number: $ARGUMENTS (optional — auto-detects next unplanned phase if omitted)
+Phase number: $ARGUMENTS (optional — when omitted, the orchestrating workflow reads ROADMAP.md and selects the next unplanned phase; `gsd-tools.cjs` itself has no auto-detect feature and requires an explicit phase number)
 
 **Flags:**
 - `--research` — Force re-research even if RESEARCH.md exists
@@ -52,7 +52,9 @@ Phase number: $ARGUMENTS (optional — auto-detects next unplanned phase if omit
 - `--ingest-format <auto|nygard|madr|narrative>` — Optional ADR parser format override (`auto` default).
 - `--reviews` — Replan incorporating cross-AI review feedback from REVIEWS.md (produced by `/gsd-review`)
 - `--text` — Use plain-text numbered lists instead of TUI menus (required for `/rc` remote sessions)
-- `--mvp` — Vertical MVP mode. Planner organizes tasks as feature slices (UI→API→DB) instead of horizontal layers. On Phase 1 of a new project, also emits `SKELETON.md` (Walking Skeleton). Can be persisted on a phase via `**Mode:** mvp` in ROADMAP.md.
+- `--mvp` — MVP enrichment on top of the default tracer-first ordering: frames the phase goal as a user story and, on Phase 1 of a new project, also emits `SKELETON.md` (Walking Skeleton). Vertical slicing itself is now the default (see `--no-tracer`); `--mvp` no longer *turns it on*. Can be persisted on a phase via `**Mode:** mvp` in ROADMAP.md.
+- `--no-tracer` — Opt out of the default **tracer-first** decomposition and plan horizontal layers (the legacy default). By default every plan LEADS with one production-quality end-to-end `tracer` slice that is verified before any expansion task.
+- `--no-reversibility-gates` — Suppress the human checkpoint that a **one-way-door** decision normally earns, for runs you intend to leave unattended. By default a decision rated `one-way` (undo needs a migration, breaks a published contract, or is impossible) gets a `checkpoint:decision` before the task implementing it. Ratings are still recorded on tasks and `costly` items still flagged — the flag changes what stops the run, not what the plan remembers.
 
 Normalize phase input in step 2 before any directory lookups.
 </context>

package/skills/gsd-plan-review-convergence/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: gsd-plan-review-convergence
 description: "Cross-AI plan convergence - replan until review concerns are resolved."
-argument-hint: "<phase> [--codex] [--gemini] [--claude] [--opencode] [--ollama] [--lm-studio] [--llama-cpp] [--text] [--ws <name>] [--all] [--max-cycles N]"
+argument-hint: "<phase> [--gemini] [--claude] [--codex] [--coderabbit] [--opencode] [--qwen] [--cursor] [--antigravity] [--agy] [--ollama] [--lm-studio] [--llama-cpp] [--kimi-code] [--text] [--ws <name>] [--all] [--max-cycles N]"
 allowed-tools:
   - Read
   - Write
@@ -40,13 +40,18 @@ Replaces gsd-plan-phase's internal gsd-plan-checker with external AI reviewers (
 Phase number: extracted from $ARGUMENTS (required)
 
 **Flags:**
-- `--codex` — Use Codex CLI as reviewer (default if no reviewer specified)
+- `--codex` — Use Codex CLI as reviewer (default if no reviewer flag given AND `review.default_reviewers` is unset; otherwise `review.default_reviewers` wins per ADR-0011 — #2315)
 - `--gemini` — Use Gemini CLI as reviewer
+- `--agy` / `--antigravity` — Use Antigravity CLI as reviewer (successor to the discontinued Gemini CLI)
 - `--claude` — Use Claude CLI as reviewer (separate session)
+- `--coderabbit` — Use CodeRabbit as reviewer (reviews the working-tree diff, not the source tree)
 - `--opencode` — Use OpenCode as reviewer
+- `--qwen` — Use Qwen Code CLI as reviewer (Alibaba Qwen models)
+- `--cursor` — Use Cursor agent as reviewer
 - `--ollama` — Use local Ollama server as reviewer (OpenAI-compatible, default host `http://localhost:11434`; configure model via `review.models.ollama`)
 - `--lm-studio` — Use local LM Studio server as reviewer (OpenAI-compatible, default host `http://localhost:1234`; configure model via `review.models.lm_studio`)
 - `--llama-cpp` — Use local llama.cpp server as reviewer (OpenAI-compatible, default host `http://localhost:8080`; configure model via `review.models.llama_cpp`)
+- `--kimi-code` — Use Kimi Code CLI as reviewer (Moonshot AI)
 - `--all` — Use all available CLIs and running local model servers
 - `--max-cycles N` — Maximum replan→review cycles (default: 3)
 

package/vscode/package.json

@@ -2,7 +2,7 @@
   "name": "gsd-core-vscode",
   "displayName": "GSD Core",
   "description": "GSD orchestration engine embedded in VS Code (ADR-1239 IDE profile).",
-  "version": "1.7.0",
+  "version": "1.9.0",
   "publisher": "opengsd",
   "engines": {
     "vscode": "^1.105.0"

package/scripts/gen-golden-install-parity-zcode.cjs

@@ -1,77 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-/**
- * Standalone golden-fixture generator for tests/golden-install-parity.
- *
- * This is a BUILD-TIME generation script — NOT a test run. It imports the
- * canonical buildParityManifest builder from tests/helpers/install-shared.cjs
- * (issue #2266 — single source of truth shared with
- * tests/golden-install-parity.test.cjs) and captures the zcode fixture so the
- * parity test (which the gsd-test gate runs) has a committed artifact to
- * compare against. The authoritative test gate remains `gsd-test run`, never
- * a local `node --test`.
- *
- * Usage: node scripts/gen-golden-install-parity-zcode.cjs
- */
-const fs = require('node:fs');
-const path = require('node:path');
-const { execFileSync } = require('node:child_process');
-
-const ROOT = path.resolve(__dirname, '..');
-// buildParityManifest (and its exclusion constants) is the canonical single
-// source of truth in tests/helpers/install-shared.cjs (issue #2266) — the
-// generator no longer keeps its own inline copy, which had drifted from the
-// test harness's copy (missing the realpath/`<HOME>` normalization) and
-// mis-generated the claude-local fixture (#2100).
-const { runMinimalInstall, RUNTIME_META, buildParityManifest, BUILD_SCRIPT } = require(path.join(ROOT, 'tests', 'helpers', 'install-shared.cjs'));
-const FIXTURE_DIR = path.join(ROOT, 'tests', 'fixtures', 'golden-install-parity');
-
-function cleanup(root) {
-  try { fs.rmSync(root, { recursive: true, force: true }); } catch { /* best effort */ }
-}
-
-// Regenerate the fixture for every runtime in RUNTIME_META. Needed when a
-// SHARED gsd-core payload file (e.g. model-catalog.json, capability-registry)
-// changes content — its hash appears in every runtime's manifest, so all
-// fixtures must be recaptured together. Usage:
-//   node scripts/gen-golden-install-parity-zcode.cjs [runtime ...]
-// With no args, regenerates ALL runtimes. With args, only the named runtimes.
-const targets = process.argv.slice(2).length > 0 ? process.argv.slice(2) : Object.keys(RUNTIME_META);
-fs.mkdirSync(FIXTURE_DIR, { recursive: true });
-
-// Build hooks/dist before capturing so fixtures are complete even in a clean
-// checkout (hooks/dist is gitignored + built; DEFECT.HOOKS-DIST-SCOPED-CI). The
-// test harness's before() hook does the same; without it a fresh checkout omits
-// every hooks/* path and this generator silently writes short fixtures.
-execFileSync(process.execPath, [BUILD_SCRIPT], { stdio: 'pipe' });
-
-for (const runtime of targets) {
-  if (!Object.prototype.hasOwnProperty.call(RUNTIME_META, runtime)) {
-    process.stderr.write(`[gen] unknown runtime '${runtime}' (not in RUNTIME_META) — skipping\n`);
-    continue;
-  }
-  const { configDir, root } = runMinimalInstall({ runtime, scope: 'global' });
-  let actual;
-  try {
-    actual = buildParityManifest(configDir, root);
-  } finally {
-    cleanup(root);
-  }
-  const fixturePath = path.join(FIXTURE_DIR, `${runtime}.json`);
-  fs.writeFileSync(fixturePath, JSON.stringify(actual, null, 2) + '\n', 'utf8');
-  process.stdout.write(`[gen] ${runtime}: wrote ${Object.keys(actual).length} file hashes -> ${fixturePath}\n`);
-}
-
-// Also regenerate the claude LOCAL legacy-layout fixture (claude-local.json).
-// This layout is distinct from the global install (commands/gsd-*.md +
-// agents/gsd-*.md) and has its own parity assertion in the test harness.
-const { configDir: localConfigDir, root: localRoot } = runMinimalInstall({ runtime: 'claude', scope: 'local' });
-let localActual;
-try {
-  localActual = buildParityManifest(localConfigDir, localRoot);
-} finally {
-  cleanup(localRoot);
-}
-const localFixturePath = path.join(FIXTURE_DIR, 'claude-local.json');
-fs.writeFileSync(localFixturePath, JSON.stringify(localActual, null, 2) + '\n', 'utf8');
-process.stdout.write(`[gen] claude-local: wrote ${Object.keys(localActual).length} file hashes -> ${localFixturePath}\n`);

package/scripts/update-size-baseline.cjs

@@ -1,68 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * @file update-size-baseline.cjs
- *
- * Regenerates the committed per-file workflow size baseline
- * (`tests/workflow-size-baseline.json`) from the current workflow files.
- *
- * Run via `npm run size:baseline` whenever a workflow file legitimately grows
- * or shrinks.  Growth must still be justified in the PR; this script only
- * records the new reality so the CI guard (issue #1074) can diff against it.
- *
- * Idempotent: running it twice with no file changes produces no diff.
- */
-
-const fs = require('fs');
-const path = require('path');
-const { measureMdFiles, WORKFLOWS_DIR } = require('./workflow-size.cjs');
-
-const BASELINE_PATH = path.join(__dirname, '..', 'tests', 'workflow-size-baseline.json');
-const AGENTS_DIR = path.join(__dirname, '..', 'agents');
-const AGENT_BASELINE_PATH = path.join(__dirname, '..', 'tests', 'agent-size-baseline.json');
-const isGsdAgent = (f) => f.startsWith('gsd-');
-
-/**
- * Serialize a size map to the on-disk baseline format: keys sorted, 2-space
- * indent, trailing newline (so the file is a stable, minimal-diff artifact).
- *
- * @param {Object<string, number>} sizes
- * @returns {string}
- */
-function serializeBaseline(sizes) {
-  const sorted = {};
-  for (const key of Object.keys(sizes).sort()) sorted[key] = sizes[key];
-  return JSON.stringify(sorted, null, 2) + '\n';
-}
-
-/**
- * Write a baseline file from the measured `.md` sizes in a directory.
- *
- * @param {object} [opts]
- * @param {string} [opts.dir]       - Directory to measure (default: workflows).
- * @param {string} [opts.outPath]   - Baseline file to write (default: workflow).
- * @param {function(string): boolean} [opts.predicate] - Filename filter.
- * @returns {{ outPath: string, count: number, content: string }}
- */
-function generateBaseline({ dir = WORKFLOWS_DIR, outPath = BASELINE_PATH, predicate } = {}) {
-  const sizes = measureMdFiles(dir, predicate);
-  const content = serializeBaseline(sizes);
-  fs.writeFileSync(outPath, content);
-  return { outPath, count: Object.keys(sizes).length, content };
-}
-
-if (require.main === module) {
-  const targets = [
-    { label: 'workflow', dir: WORKFLOWS_DIR, outPath: BASELINE_PATH },
-    { label: 'agent', dir: AGENTS_DIR, outPath: AGENT_BASELINE_PATH, predicate: isGsdAgent },
-  ];
-  for (const t of targets) {
-    const { outPath, count } = generateBaseline(t);
-    process.stdout.write(
-      `Wrote ${count} ${t.label} sizes to ${path.relative(process.cwd(), outPath)}\n`
-    );
-  }
-}
-
-module.exports = { generateBaseline, serializeBaseline, BASELINE_PATH, AGENT_BASELINE_PATH };