sequant 2.1.2 → 2.3.0

package/dist/src/commands/locks.js

@@ -0,0 +1,290 @@
+/**
+ * `sequant locks` — inspect and clear per-issue concurrency locks (#625).
+ */
+import chalk from "chalk";
+import { LockManager, formatLockedMessage, } from "../lib/locks/index.js";
+/** Human-readable line for the `--signal-other` log output (#637). */
+function formatSignalLine(issue, pid, result) {
+    switch (result.reason) {
+        case "sent":
+            return `Signaled PID ${pid} (SIGTERM) for #${issue}`;
+        case "cross-host":
+            return `Could not signal PID ${pid} for #${issue} (cross-host holder)`;
+        case "self-or-parent":
+            return `Refused to signal PID ${pid} for #${issue} (matches this process or its parent)`;
+        case "pid-dead":
+            return `Could not signal PID ${pid} for #${issue} (already exited)`;
+        case "kill-failed":
+            return `Could not signal PID ${pid} for #${issue} (kill syscall failed)`;
+        case "orchestrator":
+            return `Skipped signal for #${issue} (orchestrator mode)`;
+    }
+}
+function parseIssue(arg) {
+    const issue = Number.parseInt(arg, 10);
+    if (!Number.isInteger(issue) || issue <= 0) {
+        console.error(chalk.red(`Invalid issue number: ${arg}`));
+        process.exitCode = 1;
+        return null;
+    }
+    return issue;
+}
+/** `sequant locks list` — print every active lock with staleness metadata. */
+export async function locksListCommand(options = {}) {
+    const manager = new LockManager();
+    if (manager.isNoop) {
+        if (options.json) {
+            console.log(JSON.stringify({ locks: [], orchestratorMode: true }));
+        }
+        else {
+            console.log(chalk.gray("Lock operations are disabled (SEQUANT_ORCHESTRATOR set)."));
+        }
+        return;
+    }
+    const listings = manager.list();
+    if (options.json) {
+        console.log(JSON.stringify({ locks: listings }, null, 2));
+        return;
+    }
+    if (listings.length === 0) {
+        console.log(chalk.gray("No active locks."));
+        return;
+    }
+    console.log(chalk.bold(`Active locks (${listings.length}):`));
+    console.log("");
+    for (const l of listings) {
+        const ageMinutes = Math.floor(l.ageMs / 60_000);
+        const staleTag = l.stale ? chalk.yellow(`  (stale: ${l.staleReason})`) : "";
+        console.log(`  #${l.issue}  pid=${l.holder.pid}  host=${l.holder.hostname}  ` +
+            `age=${ageMinutes}m  started=${l.holder.startedAt}${staleTag}`);
+        console.log(`    command: ${l.holder.command}`);
+    }
+}
+/**
+ * `sequant locks clear <issue>` — remove a lock manually.
+ * By default refuses to clear a fresh same-host lock whose PID is alive;
+ * pass `--force` to override.
+ */
+export async function locksClearCommand(issueArg, options = {}) {
+    const issue = Number.parseInt(issueArg, 10);
+    if (!Number.isInteger(issue) || issue <= 0) {
+        console.error(chalk.red(`Invalid issue number: ${issueArg}`));
+        process.exitCode = 1;
+        return;
+    }
+    const manager = new LockManager();
+    if (manager.isNoop) {
+        console.log(chalk.gray("Lock operations are disabled (SEQUANT_ORCHESTRATOR set)."));
+        return;
+    }
+    const result = manager.clearLock(issue, { safetyCheck: !options.force });
+    if (options.json) {
+        console.log(JSON.stringify({ issue, ...result }));
+        return;
+    }
+    if (result.cleared) {
+        console.log(chalk.green(`✓ Cleared lock for #${issue}`));
+        return;
+    }
+    switch (result.reason) {
+        case "no-lock":
+            console.log(chalk.gray(`No lock found for #${issue}`));
+            return;
+        case "fresh-same-host-alive": {
+            const holder = manager.check(issue);
+            console.log(chalk.yellow(`Refusing to clear fresh lock on #${issue}` +
+                (holder ? ` (PID ${holder.pid} appears alive on this host)` : "") +
+                `. Re-run with --force if you really want to clear it.`));
+            process.exitCode = 1;
+            return;
+        }
+        default:
+            console.log(chalk.gray(`No-op (${result.reason})`));
+    }
+}
+/**
+ * `sequant locks acquire <issue>` — claim the lock from a shell context
+ * (e.g. a skill SKILL.md). Use `--skip-pid-check` for skill shells whose
+ * Node PID dies between acquire and release; stale recovery then falls back
+ * to age-only (2h).
+ *
+ * Exit codes:
+ *   0 — acquired
+ *   1 — locked by another holder (printed to stderr unless --json)
+ *   2 — invalid arguments
+ */
+export async function locksAcquireCommand(issueArg, options = {}) {
+    const issue = parseIssue(issueArg);
+    if (issue === null)
+        return;
+    const command = options.command ?? "unknown";
+    const manager = new LockManager();
+    if (manager.isNoop) {
+        if (options.json) {
+            console.log(JSON.stringify({ acquired: true, orchestratorMode: true }));
+        }
+        else {
+            console.log(chalk.gray("Lock operations are disabled (SEQUANT_ORCHESTRATOR set)."));
+        }
+        return;
+    }
+    if (options.force) {
+        const { previous } = manager.forceAcquire(issue, command, {
+            skipPidCheck: options.skipPidCheck,
+        });
+        if (previous && options.signalOther) {
+            const result = manager.signalOther(previous);
+            if (!options.json) {
+                console.log(chalk.gray(formatSignalLine(issue, previous.pid, result)));
+            }
+        }
+        if (options.json) {
+            console.log(JSON.stringify({
+                acquired: true,
+                forced: true,
+                previousHolder: previous,
+            }));
+        }
+        else {
+            console.log(chalk.green(`✓ Acquired lock for #${issue} (forced)`));
+        }
+        return;
+    }
+    const result = manager.acquire(issue, command, {
+        skipPidCheck: options.skipPidCheck,
+    });
+    if (result.acquired) {
+        if (options.json) {
+            console.log(JSON.stringify({ acquired: true, lockPath: result.lockPath }));
+        }
+        else {
+            console.log(chalk.green(`✓ Acquired lock for #${issue}`));
+        }
+        return;
+    }
+    // Blocked.
+    process.exitCode = 1;
+    if (options.json) {
+        console.log(JSON.stringify({
+            acquired: false,
+            holder: result.holder,
+            lockPath: result.lockPath,
+        }));
+    }
+    else {
+        console.error(chalk.yellow(formatLockedMessage(issue, result.holder)));
+    }
+}
+/**
+ * `sequant locks release <issue>` — release a lock previously acquired by
+ * a skill shell on this host. Refuses to release locks held by a foreign
+ * host or by a different process (use `locks clear --force` for that).
+ */
+export async function locksReleaseCommand(issueArg, options = {}) {
+    const issue = parseIssue(issueArg);
+    if (issue === null)
+        return;
+    const manager = new LockManager();
+    if (manager.isNoop) {
+        if (options.json) {
+            console.log(JSON.stringify({ issue, released: false, orchestratorMode: true }));
+        }
+        else {
+            console.log(chalk.gray("Lock operations are disabled (SEQUANT_ORCHESTRATOR set)."));
+        }
+        return;
+    }
+    const released = manager.releaseExternal(issue);
+    if (options.json) {
+        console.log(JSON.stringify({ issue, released }));
+        return;
+    }
+    if (released) {
+        console.log(chalk.green(`✓ Released lock for #${issue}`));
+    }
+    else {
+        console.log(chalk.gray(`No releasable lock for #${issue}`));
+    }
+}
+/**
+ * `sequant locks check <issue>` — read-only lock probe for `/assess`-style
+ * skills. Prints holder info if any, exit code 0 when free, 1 when held.
+ */
+export async function locksCheckCommand(issueArg, options = {}) {
+    const issue = parseIssue(issueArg);
+    if (issue === null)
+        return;
+    const manager = new LockManager();
+    if (manager.isNoop) {
+        if (options.json) {
+            console.log(JSON.stringify({ locked: false, orchestratorMode: true }));
+        }
+        else {
+            console.log(chalk.gray("Lock operations are disabled (SEQUANT_ORCHESTRATOR set)."));
+        }
+        return;
+    }
+    const holder = manager.check(issue);
+    if (!holder) {
+        if (options.json) {
+            console.log(JSON.stringify({ issue, locked: false }));
+        }
+        else {
+            console.log(chalk.gray(`#${issue} is not locked`));
+        }
+        return;
+    }
+    process.exitCode = 1;
+    if (options.json) {
+        console.log(JSON.stringify({ issue, locked: true, holder }));
+    }
+    else {
+        console.log(chalk.yellow(formatLockedMessage(issue, holder)));
+    }
+}
+/**
+ * `sequant locks check-batch <issue1> <issue2> ...` — read-only batch probe
+ * used by `/assess`. Text mode emits one canonical warning line per held
+ * issue (nothing for unheld), so the skill can paste the output directly
+ * above its dashboard. JSON mode emits a structured `{ warnings: [...] }`.
+ *
+ * Exit code is always 0 — `/assess` is read-only and should never abort
+ * on locked issues; the warning is informational.
+ */
+export async function locksCheckBatchCommand(issueArgs, options = {}) {
+    const issues = [];
+    for (const arg of issueArgs) {
+        const issue = Number.parseInt(arg, 10);
+        if (!Number.isInteger(issue) || issue <= 0) {
+            console.error(chalk.red(`Invalid issue number: ${arg}`));
+            process.exitCode = 2;
+            return;
+        }
+        issues.push(issue);
+    }
+    const manager = new LockManager();
+    if (manager.isNoop) {
+        if (options.json) {
+            console.log(JSON.stringify({ warnings: [], orchestratorMode: true, checked: 0 }));
+        }
+        // Non-JSON: silent in orchestrator mode (matches `acquire`/`release`
+        // semantics — no spurious output for /assess in MCP-driven runs).
+        return;
+    }
+    const warnings = [];
+    for (const issue of issues) {
+        const holder = manager.check(issue);
+        if (holder)
+            warnings.push({ issue, holder });
+    }
+    if (options.json) {
+        console.log(JSON.stringify({ warnings, checked: issues.length }, null, 2));
+        return;
+    }
+    // Text mode: one line per held issue (canonical `⚠` format that /assess
+    // pastes verbatim into its dashboard). Empty output when nothing is held.
+    for (const { issue, holder } of warnings) {
+        console.log(`⚠ #${issue} held by PID ${holder.pid} on ${holder.hostname} ` +
+            `since ${holder.startedAt} (${holder.command})`);
+    }
+}

package/dist/src/commands/merge.js

@@ -14,6 +14,7 @@
 import { spawnSync } from "child_process";
 import { ui, colors } from "../lib/cli-ui.js";
 import { runMergeChecks, formatReportMarkdown, } from "../lib/merge-check/index.js";
+import { LockManager, formatLockedMessage } from "../lib/locks/index.js";
 /**
  * Determine exit code from batch verdict
  */
@@ -68,6 +69,16 @@ export async function mergeCommand(issues, options) {
         console.log(colors.muted(issueNumbers.length > 0
             ? `Checking issues: ${issueNumbers.map((i) => `#${i}`).join(", ")} (mode: ${mode})`
             : `Auto-detecting issues from most recent run (mode: ${mode})`));
+        // #625: read-only lock awareness — warn but proceed.
+        const lockManager = new LockManager();
+        if (!lockManager.isNoop) {
+            for (const issue of issueNumbers) {
+                const holder = lockManager.check(issue);
+                if (holder) {
+                    console.log(colors.muted(`  !  ${formatLockedMessage(issue, holder)}`));
+                }
+            }
+        }
         console.log("");
     }
     try {

package/dist/src/commands/prompt.d.ts

@@ -0,0 +1,39 @@
+/**
+ * `sequant prompt <issue> "<message>"` — send a message into a running
+ * headless `sequant run` session via the interactive relay (#383).
+ */
+import { type RelayMessageType } from "../lib/relay/types.js";
+import type { IssueState } from "../lib/workflow/state-schema.js";
+export interface PromptCommandOptions {
+    type?: string;
+    json?: boolean;
+}
+export interface ParsedPromptArgs {
+    issue: number | null;
+    message: string;
+    type: RelayMessageType;
+}
+/** Validate raw CLI args. Throws on invalid type or empty message. */
+export declare function parseRelayPromptArgs(args: string[], options?: {
+    type?: string;
+}): ParsedPromptArgs;
+/** Identify candidate active runs for auto-resolution (AC-17b). */
+export declare function findActiveIssues(issues: IssueState[], isAlive: (pid: number) => boolean, cwd?: string): number[];
+/**
+ * Resolve which issue to target.
+ * - Explicit `issue` arg: use as-is.
+ * - Single active run: auto-resolve.
+ * - Zero active runs: error.
+ * - Multiple active runs: error with usage hint.
+ */
+export declare function resolveTargetIssue(args: {
+    explicit: number | null;
+    activeIssues: number[];
+}): {
+    issue: number;
+    reason: "explicit" | "single-active";
+};
+export declare function promptCommand(argsAndOptions: {
+    args: string[];
+    options: PromptCommandOptions;
+}): Promise<void>;

package/dist/src/commands/prompt.js

@@ -0,0 +1,179 @@
+/**
+ * `sequant prompt <issue> "<message>"` — send a message into a running
+ * headless `sequant run` session via the interactive relay (#383).
+ */
+import chalk from "chalk";
+import { RelayMessageTypeSchema, } from "../lib/relay/types.js";
+import { appendInboxMessage } from "../lib/relay/writer.js";
+import { cleanupStalePid, readPidFile } from "../lib/relay/pid.js";
+import { StateManager } from "../lib/workflow/state-manager.js";
+/** Validate raw CLI args. Throws on invalid type or empty message. */
+export function parseRelayPromptArgs(args, options = {}) {
+    // Allowed shapes:
+    //   ["<message>"]           — single arg, auto-resolve issue
+    //   ["<issue>", "<message>"] — both positional
+    let issueArg;
+    let messageArg;
+    if (args.length === 1) {
+        messageArg = args[0];
+    }
+    else if (args.length >= 2) {
+        issueArg = args[0];
+        messageArg = args.slice(1).join(" ");
+    }
+    else {
+        throw new Error('Usage: sequant prompt [issue] "<message>" [--type TYPE]');
+    }
+    if (!messageArg || messageArg.trim() === "") {
+        throw new Error("Message cannot be empty");
+    }
+    let issue = null;
+    if (issueArg !== undefined) {
+        const n = Number.parseInt(issueArg, 10);
+        if (!Number.isInteger(n) || n <= 0) {
+            throw new Error(`Invalid issue number: '${issueArg}'`);
+        }
+        issue = n;
+    }
+    const rawType = options.type ?? "query";
+    const parsedType = RelayMessageTypeSchema.safeParse(rawType);
+    if (!parsedType.success) {
+        throw new Error(`Invalid --type '${rawType}'. Valid values: query, directive, abort.`);
+    }
+    return { issue, message: messageArg, type: parsedType.data };
+}
+/** Identify candidate active runs for auto-resolution (AC-17b). */
+export function findActiveIssues(issues, isAlive, cwd = process.cwd()) {
+    const active = [];
+    for (const issue of issues) {
+        if (issue.status !== "in_progress")
+            continue;
+        const pid = readPidFile(issue.number, cwd);
+        if (pid !== null && isAlive(pid)) {
+            active.push(issue.number);
+        }
+    }
+    return active;
+}
+/**
+ * Resolve which issue to target.
+ * - Explicit `issue` arg: use as-is.
+ * - Single active run: auto-resolve.
+ * - Zero active runs: error.
+ * - Multiple active runs: error with usage hint.
+ */
+export function resolveTargetIssue(args) {
+    if (args.explicit !== null) {
+        return { issue: args.explicit, reason: "explicit" };
+    }
+    if (args.activeIssues.length === 0) {
+        throw new Error("No active sequant runs found. Start one with `sequant run <issue>` first.");
+    }
+    if (args.activeIssues.length > 1) {
+        const list = args.activeIssues.map((n) => `#${n}`).join(", ");
+        throw new Error(`Multiple active runs: ${list}. Specify an issue number.\n` +
+            `Usage: sequant prompt ${args.activeIssues[0]} "your message"`);
+    }
+    return { issue: args.activeIssues[0], reason: "single-active" };
+}
+export async function promptCommand(argsAndOptions) {
+    const { args, options } = argsAndOptions;
+    const parsed = parseRelayPromptArgs(args, { type: options.type });
+    // Resolve target issue.
+    const stateManager = new StateManager();
+    let issueState;
+    let issueNumber;
+    if (parsed.issue !== null) {
+        issueNumber = parsed.issue;
+        issueState = await stateManager.getIssueState(issueNumber);
+    }
+    else {
+        const all = stateManager.stateExists()
+            ? Object.values(await stateManager.getAllIssueStates())
+            : [];
+        const { defaultIsPidAlive } = await import("../lib/locks/lock-manager.js");
+        const active = findActiveIssues(all, defaultIsPidAlive);
+        const target = resolveTargetIssue({
+            explicit: null,
+            activeIssues: active,
+        });
+        issueNumber = target.issue;
+        issueState = await stateManager.getIssueState(issueNumber);
+    }
+    // Liveness check — refuse to write to a dead session.
+    const cleanup = cleanupStalePid(issueNumber);
+    if (cleanup.warning) {
+        // Also reflect deactivation in state.
+        if (issueState?.relay) {
+            try {
+                await stateManager.setRelayState(issueNumber, null);
+            }
+            catch {
+                /* swallow */
+            }
+        }
+        if (options.json) {
+            console.log(JSON.stringify({
+                ok: false,
+                issue: issueNumber,
+                error: cleanup.warning,
+            }));
+        }
+        else {
+            console.error(chalk.yellow(cleanup.warning));
+        }
+        process.exitCode = 1;
+        return;
+    }
+    if (!cleanup.alive) {
+        const msg = `No relay PID found for #${issueNumber}. Is the run active?`;
+        if (options.json) {
+            console.log(JSON.stringify({ ok: false, issue: issueNumber, error: msg }));
+        }
+        else {
+            console.error(chalk.yellow(msg));
+        }
+        process.exitCode = 1;
+        return;
+    }
+    // Write to inbox (the worktree path is recorded in IssueState).
+    const message = appendInboxMessage(issueNumber, {
+        type: parsed.type,
+        ...(parsed.type === "abort" && parsed.message.trim() === ""
+            ? {}
+            : { message: parsed.message }),
+    }, { worktreePath: issueState?.worktree });
+    // Increment relay messageCount in state.
+    try {
+        await stateManager.incrementRelayMessageCount(issueNumber, 1);
+    }
+    catch {
+        /* swallow */
+    }
+    // Build confirmation with current phase + elapsed time.
+    const phase = issueState?.currentPhase ?? "unknown";
+    let elapsedSegment = "";
+    if (issueState?.relay?.startedAt) {
+        const ms = Date.now() - new Date(issueState.relay.startedAt).getTime();
+        elapsedSegment = `, ${formatElapsed(ms)} elapsed`;
+    }
+    const confirmation = `Message sent to #${issueNumber} (${phase} phase${elapsedSegment})`;
+    if (options.json) {
+        console.log(JSON.stringify({
+            ok: true,
+            issue: issueNumber,
+            messageId: message.id,
+            type: parsed.type,
+            phase,
+        }));
+    }
+    else {
+        console.log(chalk.green(confirmation));
+    }
+}
+function formatElapsed(ms) {
+    const totalSec = Math.max(0, Math.floor(ms / 1000));
+    const minutes = Math.floor(totalSec / 60);
+    const seconds = totalSec % 60;
+    return `${minutes}m ${seconds}s`;
+}

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

@@ -0,0 +1,26 @@
+/**
+ * Display helpers for `sequant run` — pre-run config block + post-run summary.
+ *
+ * Kept separate from run.ts so the adapter stays thin (see AC-2 of #503).
+ *
+ * As of #618, the post-run summary delegates to the unified RunRenderer when
+ * one is provided. The renderless path (used by --experimental-tui and tests)
+ * falls back to `renderRunSummary` so output stays consistent across modes.
+ */
+import type { RunRenderer } from "../lib/cli-ui/run-renderer-types.js";
+import type { ResolvedRun, RunResult } from "../lib/workflow/run-orchestrator.js";
+/**
+ * Print pre-run config block.
+ *
+ * Columnar alignment via 15-char label padding. Conditional rows only
+ * appear when non-default, matching the pre-#503 format.
+ */
+export declare function displayConfig(r: ResolvedRun): void;
+/**
+ * Print post-run summary: per-issue grid, log path, reflection, tips.
+ *
+ * If a renderer is provided (default path), delegate to its `renderSummary`
+ * so the live zone is torn down cleanly first. Otherwise, fall back to the
+ * shared `renderRunSummary` helper (used by tests and TUI mode).
+ */
+export declare function displaySummary(result: RunResult, renderer?: RunRenderer | null): void;