feat(status): reorganize output into sections + report daemon identity#117
Draft
rgao-coreweave wants to merge 2 commits into
Draft
feat(status): reorganize output into sections + report daemon identity#117rgao-coreweave wants to merge 2 commits into
rgao-coreweave wants to merge 2 commits into
Conversation
`status` previously showed only the CLI symlink path and whether the
socket was alive, so you couldn't tell which build the running daemon was
actually executing (e.g. a linked local dev build vs the published
install). The daemon now includes its pid, version, and resolved entry
script in the config-hash control reply, and `status` surfaces them:
Daemon socket: ... (alive)
Running: pid 44690 (v0.2.12)
From: /.../dist/cli.js
Also added to `status --json` as daemon: {pid, version, path}, all null
when no daemon is alive or it predates identity reporting.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Regroup the flat status list into Config / Plugin / Daemon sections with aligned key/value rows, a leading state headline (ready to trace / configuration incomplete / daemon stale) plus its next step, a per-row ✓/✗/⚠ glyph (doctor-command convention), and ~-abbreviated home paths. `--json` output is unchanged (stable contract); only the pretty printer is reorganized. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
rgao-coreweave
changed the title
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
weave-claude-code statusinto grouped Config / Plugin / Daemon sections: aligned rows, a leading state line, per-row ✓/✗/⚠ glyphs, and~-abbreviated paths.config-hashcontrol reply), sostatusshows which build is actually running. Exposed in--jsonas an additivedaemon: {pid, version, path}(null on older daemons); rest of the schema unchanged.Test plan
npm run check(build + 75 tests). Updated pretty-output assertions; added daemon-identity and null-when-not-running tests.🤖 Generated with Claude Code