@vibe-agent-toolkit/cli 0.2.0-rc.10 → 0.2.0-rc.12

package/dist/utils/command-error.js

@@ -1,230 +0,0 @@
-/**
- * Shared command error handling utilities.
- *
- * The failure endings a command may take, each publishing a document and
- * ending on one member of the `ExitCode` contract. `commands/skills/` used to
- * carry its own `handleCommandError` that logged to stderr and exited 2 having
- * written nothing to stdout — same name, same signature, same documented
- * contract, silently different behaviour. A command that ends some other way
- * still names its code from `ExitCode`; the lint rule
- * `no-literal-process-exit` refuses a bare number.
- */
-import { ExitCode, buildErrorReport, countBySeverity, errorDiagnostics } from '@vibe-agent-toolkit/schema';
-import { writeJsonOutput, writeYamlOutput } from './output.js';
-/**
- * Format duration for human readability
- * @param ms - Duration in milliseconds
- * @returns Formatted duration string (e.g., "500ms", "1.5s", "1.5m")
- */
-export function formatDuration(ms) {
-    if (ms < 1000) {
-        return `${ms}ms`;
-    }
-    if (ms < 60000) {
-        return `${(ms / 1000).toFixed(1)}s`;
-    }
-    return `${(ms / 60000).toFixed(1)}m`;
-}
-/**
- * Report an unexpected failure on stderr and return the document that describes
- * it, WITHOUT deciding how the caller ends.
- *
- * Split out of {@link handleCommandError} because the same failure now has two
- * endings and must not grow two policies. A command run from the command line
- * writes this document to stdout and exits 2; the same command run as a phase of
- * `vat validate` / `vat verify` / `vat build` runs in the orchestrator's own
- * process, where exiting would kill the whole run and skip every later phase —
- * so it hands the document back and the orchestrator nests it under that phase's
- * `report`, exactly as it used to nest the child's captured stdout.
- *
- * The stderr half is identical in both lanes: progress and diagnostics have
- * always streamed live, and that is unchanged by there no longer being a child.
- *
- * @param error - The value that was thrown
- * @param logger - Logger instance for error output
- * @param startTime - Command start time (from Date.now())
- * @param commandName - Name of the command (for the error message)
- */
-export function reportCommandError(error, logger, startTime, commandName) {
-    const duration = Date.now() - startTime;
-    const message = error instanceof Error ? error.message : 'Unknown error';
-    logger.error(`${commandName} failed: ${message}`);
-    logger.debug(errorDiagnostics(error));
-    return { status: 'error', error: message, duration: formatDuration(duration) };
-}
-/**
- * Handle command error with standard formatting and exit.
- *
- * The command-line ending for {@link reportCommandError}. `ExitCode.ERROR` is
- * the UNEXPECTED failure, per the exit-code contract every command's help documents.
- *
- * 🔑 **`format` is not decoration.** Every command offering `--format json`
- * honoured it on the success path and emitted YAML here, so the one document a
- * CI wrapper most needs to read — the one explaining why the command failed —
- * arrived in a format its parser rejects. A consumer running
- * `vat resources check --format json | jq .error` got a parse error on top of
- * whatever went wrong, and had to guess at the second failure to find the first.
- * A caller that has a `--format` option MUST pass it.
- *
- * @param error - The error that occurred
- * @param logger - Logger instance for error output
- * @param startTime - Command start time (from Date.now())
- * @param commandName - Name of the command (for error message)
- * @param format - What the operator asked for: `json`, or anything else (and
- *   omitted) for YAML — matching the success path's own two-branch switch
- */
-export function handleCommandError(error, logger, startTime, commandName, format) {
-    const document = reportCommandError(error, logger, startTime, commandName);
-    return publishFailure(document, format, ExitCode.ERROR);
-}
-/**
- * Publish an EXPECTED failure in the format the operator asked for, then exit
- * with the code the command's own contract assigns it.
- *
- * The third ending of one failure, beside {@link handleCommandError} (the
- * UNEXPECTED failure, always `ERROR`) and {@link handleValidationGateFailure}
- * (a validation gate, always `FINDINGS`, always YAML). This one covers the
- * failure that is neither: the command ran, understood the project, and has a
- * documented non-zero code for what it found — `vat ard emit` ending on
- * `FINDINGS` for "this project declares no `ard:` block" or `ERROR` for "there
- * is no config file here".
- *
- * 🔑 It exists because those endings were written inline, and an inline ending
- * publishes nothing: `ard emit` honoured `--format json` on its success path
- * and through `handleCommandError`, then wrote **zero bytes to stdout** on the
- * two exits a repository actually reaches — the commonest being the first run
- * of any repository that never opted into ARD. A CI wrapper reading stdout got
- * an empty document and a bare non-zero code, which is the exact failure the
- * `format` note on {@link handleCommandError} exists to forbid.
- *
- * The message goes to stderr as it always did — humans read it there — and the
- * document goes to stdout, matching what both neighbours already do.
- *
- * @param message - What went wrong, in the command's own words
- * @param exitCode - The code this command's `--help` assigns to this outcome
- * @param startTime - Command start time (from Date.now()), so the document
- *   reports the run's real duration rather than a fabricated zero
- * @param format - What the operator asked for: `json`, or anything else (and
- *   omitted) for YAML — the same two-branch switch the success path uses
- */
-export function handleExpectedFailure(message, exitCode, startTime, format) {
-    process.stderr.write(`${message}\n`);
-    const document = {
-        status: 'error',
-        error: message,
-        duration: formatDuration(Date.now() - startTime),
-    };
-    return publishFailure(document, format, exitCode);
-}
-/**
- * Publish a failure document in the operator's format, then end on `exitCode`.
- *
- * @param document - What to write to stdout
- * @param format - `json`, or anything else (and omitted) for YAML
- * @param exitCode - The member of the contract this ending is
- */
-function publishFailure(document, format, exitCode) {
-    if (format === 'json') {
-        writeJsonOutput(document);
-    }
-    else {
-        writeYamlOutput(document);
-    }
-    process.exit(exitCode);
-}
-/**
- * {@link handleCommandError} for a command whose document is the shared
- * `Report<T>` envelope: the failure document IS the envelope
- * (`buildErrorReport`), so the emitted `schemas/<command>.json` describes the
- * failed run too. Every command registered as `report` in `report-schemas.ts`
- * ends its unexpected failures here; `handleCommandError`'s
- * `CommandErrorDocument` is the shape the legacy documents keep.
- *
- * @param error - The value that was thrown
- * @param logger - Logger instance for error output
- * @param startTime - Command start time (from Date.now())
- * @param commandName - Name of the command (for the stderr line)
- * @param format - What the operator asked for: `json`, or anything else (and
- *   omitted) for YAML
- */
-export function handleReportCommandError(error, logger, startTime, commandName, format) {
-    const { error: message } = reportCommandError(error, logger, startTime, commandName);
-    return publishFailure(buildErrorReport(message, Date.now() - startTime), format, ExitCode.ERROR);
-}
-/**
- * {@link handleExpectedFailure} for a command whose document is the shared
- * `Report<T>` envelope — same document as {@link handleReportCommandError},
- * with the exit code the command's own contract assigns.
- *
- * @param message - What went wrong, in the command's own words
- * @param exitCode - The code this command's `--help` assigns to this outcome
- * @param startTime - Command start time (from Date.now())
- * @param format - What the operator asked for: `json`, or anything else (and
- *   omitted) for YAML
- */
-export function handleReportExpectedFailure(message, exitCode, startTime, format) {
-    process.stderr.write(`${message}\n`);
-    return publishFailure(buildErrorReport(message, Date.now() - startTime), format, exitCode);
-}
-/**
- * Pure: the document {@link handleValidationGateFailure} writes.
- *
- * The per-severity distribution rides beside the status because `status: error`
- * alone cannot say whether the run also carried warnings and info — and the
- * findings themselves went to stderr, where a piped consumer never sees them.
- */
-export function buildValidationGateFailure(subject, issues) {
-    return { status: 'error', issueCounts: countBySeverity(issues), skill: subject };
-}
-/**
- * Publish the documented failure payload for a command stopped by its OWN
- * validation gate, then end on `FINDINGS`.
- *
- * {@link handleCommandError} above covers the UNEXPECTED failure (`ERROR`). This
- * covers the expected one, and `vat skills build` / `vat skills package` both
- * exited it after writing zero bytes to stdout — while their `--help` texts
- * document a YAML summary on stdout and reserve exit 1 for exactly this case.
- * A consumer running `vat skills build | jq .status` got an empty document and
- * a bare non-zero code.
- */
-export function handleValidationGateFailure(subject, issues) {
-    writeYamlOutput(buildValidationGateFailure(subject, issues));
-    process.exit(ExitCode.FINDINGS);
-}
-/**
- * Map a commander termination to the exit code VAT's contract promises.
- *
- * A USAGE mistake — a flag this program does not declare, a verb it does not
- * know, a missing or invalid argument — ends on `ExitCode.ERROR`, and the
- * reason is not style. Commander's default for a usage mistake is
- * `process.exit(1)`, so `vat resources check --json` (the option is
- * `--format json`) exited **1** — which, read against the contract the command
- * itself prints, asserts that at least one check was violated. Nothing had
- * run. A CI wrapper spelled `if [ $? -ne 0 ]; then report_findings` therefore
- * reported findings that were never computed. The same hole swallowed a
- * mistyped verb (`vat resources chekc`) and every unknown option on every
- * command.
- *
- * Commander reports BOTH successful and failing terminations through the same
- * `CommanderError`: `--help` and `--version` carry `exitCode` 0, while an
- * unknown option, an unknown command and `help({ error: true })` carry non-zero.
- * Anything non-zero is a usage mistake by construction — commander raises these
- * only while parsing, before any action runs — so it maps to `ERROR`.
- *
- * ⚠️ **Help and `--version` are NOT usage mistakes.** They terminate through
- * the same `_exit` path with code 0 and must stay `OK`; only a non-zero
- * commander ending is remapped.
- *
- * 🪤 Do NOT switch on `error.code` (`commander.unknownOption`, `commander.help`,
- * …). The unknown-COMMAND path arrives as `commander.help` with exitCode 1,
- * because the `command:*` handler renders help with `{ error: true }` — so the
- * code string says "help" for a case that is emphatically not one. The exit
- * code commander already computed is the honest discriminator; the string is not.
- *
- * @param commanderExitCode - The `exitCode` commander put on its CommanderError
- * @returns `OK` for a successful ending, `ERROR` otherwise
- */
-export function exitCodeForCommanderEnding(commanderExitCode) {
-    return commanderExitCode === 0 ? ExitCode.OK : ExitCode.ERROR;
-}
-//# sourceMappingURL=command-error.js.map

package/dist/utils/command-error.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"command-error.js","sourceRoot":"","sources":["../../src/utils/command-error.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAGH,OAAO,EAAE,QAAQ,EAAE,gBAAgB,EAAE,eAAe,EAAE,gBAAgB,EAAiE,MAAM,4BAA4B,CAAC;AAG1K,OAAO,EAAE,eAAe,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAE/D;;;;GAIG;AACH,MAAM,UAAU,cAAc,CAAC,EAAU;IACvC,IAAI,EAAE,GAAG,IAAI,EAAE,CAAC;QACd,OAAO,GAAG,EAAE,IAAI,CAAC;IACnB,CAAC;IACD,IAAI,EAAE,GAAG,KAAK,EAAE,CAAC;QACf,OAAO,GAAG,CAAC,EAAE,GAAG,IAAI,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC;IACtC,CAAC;IACD,OAAO,GAAG,CAAC,EAAE,GAAG,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC;AACvC,CAAC;AASD;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,kBAAkB,CAChC,KAAc,EACd,MAAc,EACd,SAAiB,EACjB,WAAmB;IAEnB,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,SAAS,CAAC;IACxC,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,eAAe,CAAC;IAEzE,MAAM,CAAC,KAAK,CAAC,GAAG,WAAW,YAAY,OAAO,EAAE,CAAC,CAAC;IAClD,MAAM,CAAC,KAAK,CAAC,gBAAgB,CAAC,KAAK,CAAC,CAAC,CAAC;IAEtC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,cAAc,CAAC,QAAQ,CAAC,EAAE,CAAC;AACjF,CAAC;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,UAAU,kBAAkB,CAChC,KAAc,EACd,MAAc,EACd,SAAiB,EACjB,WAAmB,EACnB,MAA2B;IAE3B,MAAM,QAAQ,GAAG,kBAAkB,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,WAAW,CAAC,CAAC;IAC3E,OAAO,cAAc,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;AAC1D,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6BG;AACH,MAAM,UAAU,qBAAqB,CACnC,OAAe,EACf,QAAuB,EACvB,SAAiB,EACjB,MAA2B;IAE3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,OAAO,IAAI,CAAC,CAAC;IACrC,MAAM,QAAQ,GAAyB;QACrC,MAAM,EAAE,OAAO;QACf,KAAK,EAAE,OAAO;QACd,QAAQ,EAAE,cAAc,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,SAAS,CAAC;KACjD,CAAC;IACF,OAAO,cAAc,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;AACpD,CAAC;AAED;;;;;;GAMG;AACH,SAAS,cAAc,CAAC,QAAgB,EAAE,MAA0B,EAAE,QAAuB;IAC3F,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;QACtB,eAAe,CAAC,QAAQ,CAAC,CAAC;IAC5B,CAAC;SAAM,CAAC;QACN,eAAe,CAAC,QAAQ,CAAC,CAAC;IAC5B,CAAC;IACD,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;AACzB,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,UAAU,wBAAwB,CACtC,KAAc,EACd,MAAc,EACd,SAAiB,EACjB,WAAmB,EACnB,MAA2B;IAE3B,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,GAAG,kBAAkB,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,WAAW,CAAC,CAAC;IACrF,OAAO,cAAc,CAAC,gBAAgB,CAAC,OAAO,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,SAAS,CAAC,EAAE,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC;AACnG,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,UAAU,2BAA2B,CACzC,OAAe,EACf,QAAuB,EACvB,SAAiB,EACjB,MAA2B;IAE3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,OAAO,IAAI,CAAC,CAAC;IACrC,OAAO,cAAc,CAAC,gBAAgB,CAAC,OAAO,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,SAAS,CAAC,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;AAC7F,CAAC;AAUD;;;;;;GAMG;AACH,MAAM,UAAU,0BAA0B,CACxC,OAAe,EACf,MAAkC;IAElC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,WAAW,EAAE,eAAe,CAAC,MAAM,CAAC,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC;AACnF,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,UAAU,2BAA2B,CACzC,OAAe,EACf,MAAkC;IAElC,eAAe,CAAC,0BAA0B,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC;IAC7D,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;AAClC,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAgCG;AACH,MAAM,UAAU,0BAA0B,CAAC,iBAAyB;IAClE,OAAO,iBAAiB,KAAK,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC;AAChE,CAAC"}