Uh oh!
There was an error while loading. Please reload this page.
feat(cli): scannable outdated report with command highlighting - #121
Merged
Conversation
The outdated report packed every name into one sentence, so a 13 Skill no-match line ran past 500 characters and nothing could be scanned at a glance. Rows are records now: a glyph, the Skill name, a dim status badge, then aligned detail lines. Name lists render as groups with one row per Skill. Lockfile parse failures name the problem instead of leaking the serde detail, and streamed progress dims so results stand out. Plain output keeps the exact sentence records.
harlan-zw
commented
Aug 21, 2026
CollaboratorAuthor
Checked by hand, since CI cannot cover it:
|
Commands in output are code the user types, so they highlight like code: brand binary, bold subcommand, dim flags, plain values. Record detail values carry a kind now, and paths inside the working directory display relative while machine records keep absolute paths. The update summary and error lines join the theme; plain text is unchanged.
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❓ Type of change
📚 Description
The
outdated --allreport packed every Skill into one sentence, so a 13 item no-match line ran past 500 characters:Rows are records now: a glyph, the Skill name, a dim status badge, then aligned detail lines. Name lists render as groups with one row per Skill and an aligned name column, the same shape as the update TUI.
Color goes deeper than whole-line roles, the way syntax highlighting works. Commands the user can type highlight as code: brand
skilld, bold subcommand, dim flags, plain values. Paths dim and display relative to the working directory, since that is how they were typed; machine records keep absolute paths.Three fixes fell out: lockfile parse failures said
expected value at line 1 column 1(the serde detail) and now say the lockfile is not valid JSON; the streamed• foundprogress dims so result rows carry the color; error lines and the update exit summary join the theme.Plain output keeps the exact sentence records, so agents and scripts see no change.