Polish: theme auto-skill report, animate cloud graph sync - #284
Merged
Conversation
- Colorize the --auto-skill report (Detected label, error/check icons, installed summary) via theme.Tint, honoring NO_COLOR/TTY. - Wrap cloud graph sync's graph-build and upload steps in a CLIProgress spinner for animated progress feedback.
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 free
to 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.
Summary
Two focused polish improvements on the coding-agent CLI:
Theme the
--auto-skillreport (internal/plugin/auto_skill.go): the auto-skill analysis output was plain text with icons but no color. Now theDetected:label, per-skill error (✖) and success (✓) lines, and theInstalled N skill(s)summary are colorized viatheme.Tint, which honorsNO_COLOR/TTY so piped output stays plain.Animate
graycode cloud graph sync(cmd/cloud_graph.go): the sync does a slow execution-graph build plus a network upload with no progress indication. Wrapped both steps in aCLIProgressspinner (Building execution graph→Uploading to Graycode Cloud), matching the doctor/preflight/update pattern. Non-TTY renders clean lines.Verification
go build ./...,go vet ./cmd/ ./internal/plugin/,gofumpt -lall clean.go test ./...exit 0 (full suite).--auto-skillearly-return path (plain, correct); theme mechanism already proven.