feat(cli): polish terminal branding#2434
Conversation
|
@vyctorbrzezowski is attempting to deploy a commit to the Amantus Machina Team on Vercel. A member of the Team first needs to authorize it. |
|
Codex review: needs maintainer review before merge. Reviewed May 29, 2026, 12:56 PM ET / 16:56 UTC. Summary Reproducibility: not applicable. this is a CLI polish PR, not a bug report. The inspected before/after screenshots provide behavior proof for the changed terminal output instead of a failing current-main reproduction. Review metrics: 2 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Next step before merge
Security Review detailsBest possible solution: Land the polish if maintainers want this branded terminal direction, keeping the optional downloads field backward-compatible and covered by the included CLI, schema, API, artifact, build, and static checks. Do we have a high-confidence way to reproduce the issue? Not applicable; this is a CLI polish PR, not a bug report. The inspected before/after screenshots provide behavior proof for the changed terminal output instead of a failing current-main reproduction. Is this the best way to solve the issue? Yes; the patch keeps command behavior and API response compatibility intact while focusing changes on CLI formatting, schema, generated schema output, and targeted tests. The remaining decision is whether maintainers want this terminal branding as the default. AGENTS.md: found and applied where relevant. Codex review notes: model gpt-5.5, reasoning high; reviewed against 309723f7a863. Label changesLabel justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
c67122a to
252ba47
Compare
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
bbea66f to
4a83cfe
Compare
Summary
This PR is a focused CLI polish pass for ClawHub's terminal surface. It keeps command behavior intact while making the output feel branded, intentional, and easier to scan.
inspectinto a structured terminal card with readable UTC dates.syncwith the large red ClawHub banner, centeredClawHub // synctagline, section rails, selected/upload sections, and calmer spacing.downloadsso CLI search can show downloads instead of score when the registry supports it; older registries still fall back to score.Scope note
This is cosmetic/branding polish plus the optional search downloads field needed for the CLI display. The old
clawhub help/ unknown-root-command behavior is handled separately in #2433 and is not part of this PR's claim.Visual proof
Full gallery: https://stellar-yarrow-9yhz.here.now/
clawhub --helpclawhub inspect weatherclawhub search weather --limit 5clawhub explore --limit 5clawhub package explore --limit 5clawhub install weather --forceclawhub update --all --no-input --forceclawhub uninstall weather --yesclawhub sync --dry-run --allValidation
git diff --checkbunx vitest run src/cli/ui.test.ts src/cli/commands/skills.test.ts src/cli/commands/packages.test.ts src/cli/commands/sync.test.ts src/cli/commands/inspect.test.ts src/schema/schemas.test.tsfrompackages/clawhubbunx vitest run packages/schema/src/schemas.test.ts convex/httpApiV1.handlers.test.tsbunx tsc --noEmitbunx tsc -p packages/schema/tsconfig.json --noEmitbunx tsc -p packages/clawhub/tsconfig.json --noEmitbunx vitest run -c vitest.artifact.config.ts test-artifact/cli.artifact.test.tsfrompackages/clawhubbun run --cwd packages/clawhub buildbun run ci:packagesbun run ci:staticcodex review --uncommittedonce; completed with exit code 0