Skip to content

Normalize fatal-error coloring across commands - #38

Merged
jayminwest merged 4 commits into
mainfrom
warren/run_p49eww0bw46n
May 28, 2026
Merged

Normalize fatal-error coloring across commands#38
jayminwest merged 4 commits into
mainfrom
warren/run_p49eww0bw46n

Conversation

@jayminwest

Copy link
Copy Markdown
Owner

Summary

Normalize fatal-error coloring across CLI commands

Run

  • Warren run: run_p49eww0bw46n
  • Agent: pi
  • Cost: $0.804 (47 in / 11.0k out / 796.3k cache-r)

Seeds

  • mulch-2a17 — Normalize fatal-error coloring across commands

Commits (3)

  • 74c247b Normalize fatal-error coloring across CLI commands
  • e8043b7 sd: close mulch-2a17
  • 976aab9 chore(warren): seeds state

Files changed

.seeds/issues.jsonl     |  2 +-
 src/commands/archive.ts |  4 ++--
 src/commands/delete.ts  |  4 ++--
 src/commands/edit.ts    |  4 ++--
 src/commands/learn.ts   |  2 +-
 src/commands/move.ts    |  4 ++--
 src/commands/onboard.ts |  2 +-
 src/commands/prime.ts   | 17 +++++++++--------
 src/commands/query.ts   |  7 ++++---
 src/commands/rank.ts    |  4 ++--
 src/commands/ready.ts   |  4 ++--
 src/commands/restore.ts |  4 ++--
 src/commands/search.ts  |  5 +++--
 13 files changed, 33 insertions(+), 30 deletions(-)

Prompt

Show prompt
work on sd mulch-2a17

🤖 Opened by warren run run_p49eww0bw46n

jayminwest and others added 3 commits May 28, 2026 10:24
Wrap all remaining plain console.error("Error: ...") and
console.error(`Error: ...`) fatal-error sites in chalk.red(...) to
match the convention already used by record/sync/validate/etc.
Added chalk imports to query.ts, prime.ts, and search.ts.

No message wording changes; JSON-mode error envelopes are unchanged
(chalk wrapping only applies to the human-readable stderr branch).

Closes mulch-2a17.
@jayminwest
jayminwest enabled auto-merge (squash) May 28, 2026 10:24
CI lint failed because the normalized fatal-error console.error calls
exceeded the line width; biome wants them wrapped.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jayminwest
jayminwest merged commit 0d62abd into main May 28, 2026
2 checks passed
@jayminwest
jayminwest deleted the warren/run_p49eww0bw46n branch May 28, 2026 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant