sequant 2.5.0 → 2.6.0

package/.claude-plugin/marketplace.json

@@ -8,7 +8,7 @@
     {
       "name": "sequant",
       "description": "AI coding agent orchestrator for Claude Code — resolve GitHub issues end-to-end with isolated git worktrees, quality gates, and an MCP server. Includes 17 skills, workflow MCP tools, and pre/post-tool hooks.",
-      "version": "2.5.0",
+      "version": "2.6.0",
       "author": {
         "name": "sequant-io",
         "email": "hello@sequant.io"

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "sequant",
   "description": "AI coding agent orchestrator for Claude Code — resolve GitHub issues end-to-end with isolated git worktrees and quality gates, through spec → exec → qa phases.",
-  "version": "2.5.0",
+  "version": "2.6.0",
   "author": {
     "name": "sequant-io",
     "email": "hello@sequant.io"

package/README.md

@@ -19,7 +19,7 @@ See the [CHANGELOG](CHANGELOG.md) for release notes, or the [migration guide](CH
 ### What's new in 2.5
 
 - **`sequant ready <issue>`** — a post-resolve A+ QA gate that drives a resolved issue through a full-weight `qa → loop → qa` pass and **stops at the human merge gate — it never merges**.
-- **Live phase-matrix TUI** — `sequant ready` and `sequant run` render the active phase and quality-loop iteration in place (boxed Ink dashboard on a TTY), so a long run is never indistinguishable from a hang.
+- **Live phase-matrix TUI** — `sequant ready` and `sequant run` render the active phase and quality-loop iteration in place (boxed Ink dashboard on a TTY by default), so a long run is never indistinguishable from a hang. Opt out with `--no-tui` (line renderer) or `-s`/`--quiet` (heartbeat-only); non-TTY output auto-degrades.
 - **Per-issue concurrency locks** — a second session on the same issue is skipped with a clear message instead of clobbering the first; `sequant locks` inspects and clears them.
 
 ## Quick Start

package/dist/bin/cli.js

@@ -4,7 +4,7 @@
  *
  * Sequential AI phases with quality gates for any codebase.
  */
-import { Command, InvalidArgumentError } from "commander";
+import { Command, InvalidArgumentError, Option } from "commander";
 import chalk from "chalk";
 import { fileURLToPath } from "url";
 import { dirname, resolve } from "path";
@@ -173,13 +173,18 @@ program
     .option("--no-log", "Disable JSON logging for this run")
     .option("--log-path <path>", "Custom log directory path")
     .option("-Q, --quality-loop", "Enable quality loop with auto-retry")
+    // #705: `-q` is a hidden alias for the quality loop (Commander 14 allows only
+    // one short flag per Option, so it can't live on --quality-loop directly).
+    // `runCommand` ORs `qualityLoopAlias` into `qualityLoop`. `-q` no longer maps
+    // to --quiet, which moved to `-s` to end the `-q`/`-Q` collision.
+    .addOption(new Option("-q, --quality-loop-alias", "Alias for -Q/--quality-loop").hideHelp())
     .option("--max-iterations <n>", "Max iterations for quality loop (default: 3)", parseInt)
     .option("--batch <issues>", 'Group of issues to run together (e.g., --batch "1 2" --batch "3")', (value, prev) => prev.concat([value]), [])
     .option("--smart-tests", "Enable smart test detection (default)")
     .option("--no-smart-tests", "Disable smart test detection")
     .option("--testgen", "Run testgen phase after spec")
     .option("--security-review", "Run security-review phase after spec")
-    .option("-q, --quiet", "Suppress version warnings and non-essential output")
+    .option("-s, --quiet", "Suppress version warnings and non-essential output (heartbeat-only)")
     .option("--chain", "Chain issues: each branches from previous (implies --sequential)")
     .option("--stacked", "Stack PRs: middle PRs target predecessor branch instead of main; first/last target main (implies --chain)")
     .option("--qa-gate", "Wait for QA pass before starting next issue in chain (requires --chain)")
@@ -195,7 +200,12 @@ program
     .option("--isolate-parallel", "Isolate parallel agent groups in separate worktrees (prevents file conflicts)")
     .option("--reflect", "Analyze run results and suggest improvements")
     .option("--agent <name>", 'Agent driver for phase execution (default: "claude-code")')
-    .option("--experimental-tui", "Render live multi-issue dashboard (requires TTY; falls back to linear output when piped)")
+    // #705: the boxed Ink TUI is now the default on a TTY. `--no-tui` opts out to
+    // the line-based phase-matrix renderer; non-TTY / piped output auto-degrades.
+    .option("--no-tui", "Disable the boxed Ink dashboard; use the line-based phase-matrix renderer")
+    // #705: `--experimental-tui` is now a hidden no-op alias (the TUI is the
+    // default) so existing scripts and muscle-memory keep parsing.
+    .addOption(new Option("--experimental-tui").hideHelp())
     .option("--no-relay", "Disable interactive relay (#383); `sequant prompt` cannot reach this run")
     .action(runCommand);
 program

package/dist/marketplace/external_plugins/sequant/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "sequant",
   "description": "AI coding agent orchestrator for Claude Code — resolve GitHub issues end-to-end with isolated git worktrees and quality gates, through spec → exec → qa phases.",
-  "version": "2.5.0",
+  "version": "2.6.0",
   "author": {
     "name": "sequant-io",
     "email": "hello@sequant.io"

package/dist/src/commands/run-flags.d.ts

@@ -0,0 +1,31 @@
+/**
+ * Run flag normalization (#705) — keeps run.ts thin (#503 AC-2: <200 LOC).
+ *
+ * Two pure resolvers for the `run` command's flag surface:
+ *   - `normalizeQualityLoop`: ORs the hidden `-q` alias into `--quality-loop`.
+ *   - `resolveTuiEnabled`: decides whether the boxed Ink TUI mounts.
+ *
+ * Extracted as pure functions so the flag behavior is unit-testable without
+ * driving the full `runCommand` side effects.
+ */
+import type { RunOptions } from "../lib/workflow/types.js";
+/**
+ * #705: `-q` is a hidden alias for the quality loop (it no longer maps to
+ * `--quiet`, which moved to `-s`). Returns the effective quality-loop flag so
+ * `-q` and `-Q` produce identical behavior. Must run before any consumer reads
+ * `options.qualityLoop`.
+ */
+export declare function normalizeQualityLoop(options: RunOptions): boolean;
+/**
+ * #705: the boxed Ink TUI is the default on a TTY.
+ *
+ * - `--no-tui` (Commander surfaces `options.tui === false`) opts out to the
+ *   line-based phase-matrix renderer.
+ * - Non-TTY / piped output auto-degrades (`isTTY === false`), so no Ink writes
+ *   corrupt pipes.
+ * - `--quiet`/`-s` suppresses the renderer entirely (heartbeat-only),
+ *   regardless of the TUI default (AC-2).
+ * - `--experimental-tui` is a hidden no-op alias — the default already covers
+ *   it, so it is intentionally not consulted here.
+ */
+export declare function resolveTuiEnabled(options: RunOptions, isTTY: boolean): boolean;

package/dist/src/commands/run-flags.js

@@ -0,0 +1,34 @@
+/**
+ * Run flag normalization (#705) — keeps run.ts thin (#503 AC-2: <200 LOC).
+ *
+ * Two pure resolvers for the `run` command's flag surface:
+ *   - `normalizeQualityLoop`: ORs the hidden `-q` alias into `--quality-loop`.
+ *   - `resolveTuiEnabled`: decides whether the boxed Ink TUI mounts.
+ *
+ * Extracted as pure functions so the flag behavior is unit-testable without
+ * driving the full `runCommand` side effects.
+ */
+/**
+ * #705: `-q` is a hidden alias for the quality loop (it no longer maps to
+ * `--quiet`, which moved to `-s`). Returns the effective quality-loop flag so
+ * `-q` and `-Q` produce identical behavior. Must run before any consumer reads
+ * `options.qualityLoop`.
+ */
+export function normalizeQualityLoop(options) {
+    return Boolean(options.qualityLoop || options.qualityLoopAlias);
+}
+/**
+ * #705: the boxed Ink TUI is the default on a TTY.
+ *
+ * - `--no-tui` (Commander surfaces `options.tui === false`) opts out to the
+ *   line-based phase-matrix renderer.
+ * - Non-TTY / piped output auto-degrades (`isTTY === false`), so no Ink writes
+ *   corrupt pipes.
+ * - `--quiet`/`-s` suppresses the renderer entirely (heartbeat-only),
+ *   regardless of the TUI default (AC-2).
+ * - `--experimental-tui` is a hidden no-op alias — the default already covers
+ *   it, so it is intentionally not consulted here.
+ */
+export function resolveTuiEnabled(options, isTTY) {
+    return options.tui !== false && isTTY && !options.quiet;
+}

package/dist/src/commands/run.js

@@ -8,10 +8,14 @@ import { parseBatches } from "../lib/workflow/batch-executor.js";
 import { RunOrchestrator } from "../lib/workflow/run-orchestrator.js";
 import { displayConfig, displaySummary } from "./run-display.js";
 import { buildProgressWiring } from "./run-progress.js";
+import { normalizeQualityLoop, resolveTuiEnabled } from "./run-flags.js";
 // Re-export public API for backwards compatibility
 export * from "./run-compat.js";
 /** Parse CLI args → validate → delegate to RunOrchestrator.run() → display summary. */
 export async function runCommand(issues, options) {
+    // #705: fold the hidden `-q` alias into qualityLoop before any consumer reads
+    // it (`-q` no longer maps to --quiet, which moved to `-s`). See run-flags.ts.
+    options.qualityLoop = normalizeQualityLoop(options);
     console.log(ui.headerBox("SEQUANT WORKFLOW"));
     if (!options.quiet) {
         try {
@@ -69,7 +73,9 @@ export async function runCommand(issues, options) {
     };
     const resolved = RunOrchestrator.resolveConfig(init, issues, batches);
     displayConfig(resolved);
-    const tuiEnabled = Boolean(options.experimentalTui) && Boolean(process.stdout.isTTY);
+    // #705: boxed Ink TUI is the default on a TTY; resolveTuiEnabled owns the
+    // --no-tui / non-TTY / --quiet precedence (see run-flags.ts).
+    const tuiEnabled = resolveTuiEnabled(options, Boolean(process.stdout.isTTY));
     // RunRenderer (#618) + LivenessHeartbeat (#574) wiring lives in
     // run-progress.ts to keep this adapter under the 200-LOC cap (#503 AC-2).
     const { renderer, heartbeat, onProgress, onPhasePlan } = buildProgressWiring({

package/dist/src/lib/workflow/types.d.ts

@@ -214,6 +214,13 @@ export interface RunOptions {
     noLog?: boolean;
     logPath?: string;
     qualityLoop?: boolean;
+    /**
+     * #705: hidden `-q` alias for the quality loop. Commander 14 allows only one
+     * short flag per Option, so `-q` lives on its own `--quality-loop-alias`
+     * Option and `runCommand` ORs it into `qualityLoop` before any consumer reads
+     * it. `-q` no longer maps to `--quiet` (which moved to `-s`).
+     */
+    qualityLoopAlias?: boolean;
     maxIterations?: number;
     batch?: string[];
     smartTests?: boolean;
@@ -303,9 +310,17 @@ export interface RunOptions {
      */
     isolateParallel?: boolean;
     /**
-     * Render a live multi-issue dashboard during the run.
-     * Requires a TTY; auto-falls back to linear output when stdout is piped.
-     * Experimental — surface and behavior may change.
+     * #705: the boxed Ink dashboard is the default on a TTY. Set via `--no-tui`,
+     * which Commander surfaces as `options.tui === false` to opt out to the
+     * line-based phase-matrix renderer. Non-TTY / piped output auto-degrades, and
+     * `--quiet`/`-s` suppresses the renderer entirely regardless of this flag.
+     * Resolution: `tuiEnabled = options.tui !== false && isTTY && !quiet`.
+     */
+    tui?: boolean;
+    /**
+     * #705: now a hidden no-op alias — the boxed Ink TUI is the default, so
+     * `--experimental-tui` only parses for backward compatibility and no longer
+     * gates rendering. Kept so existing scripts/muscle-memory don't break.
      */
     experimentalTui?: boolean;
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sequant",
-  "version": "2.5.0",
+  "version": "2.6.0",
   "description": "AI coding agent orchestrator — resolve GitHub issues end-to-end with isolated git worktrees, quality gates, and an MCP server. Works with Claude Code or Aider.",
   "type": "module",
   "bin": {