Tidy _mcpc JSON output: stateless field, drop log size#261
Merged
Conversation
…ogSize → logSizeBytes The `--json` output of `mcpc @<session>`, `mcpc connect`, and the session list emitted `statefulness: "stateful"|"stateless"`. Replace it with a boolean `stateful` (omitted when not yet determined) — more idiomatic and jq-friendly. The internal three-state enum and the human-readable `(stateful)`/`(stateless)` suffix are unchanged. Also rename the `_mcpc.logSize` JSON field to `_mcpc.logSizeBytes` to make the unit explicit. Both fields are unreleased, so no breaking change. https://claude.ai/code/session_018sp5rqgbDLoXoSFF4VfYpq
The emitted log size was a point-in-time snapshot that goes stale as the bridge keeps writing, and was derivable by stat-ing the already-emitted `logPath` (or via `mcpc @<session> logs`). Stop emitting it; `logPath` stays. Nothing stored it internally, so the now-unused stat() call and its import are removed too. https://claude.ai/code/session_018sp5rqgbDLoXoSFF4VfYpq
jancurn
changed the title
_mcpc JSON output: boolean stateful, drop log size
…false/null Internal: rename the `SessionStatefulness` enum to `ConnectionMode` (and the `statefulness` fields / `deriveStatefulness()` accordingly). The enum values (`stateful`/`stateless`/`unknown`) and the human-readable `(stateful)`/ `(stateless)` suffix are unchanged. JSON: the `_mcpc.stateful` field is now always present as `true` (stateful), `false` (stateless), or `null` (not yet determined), rather than omitted when unknown — a stable schema for consumers. Both fields are unreleased, so no breaking change. https://claude.ai/code/session_018sp5rqgbDLoXoSFF4VfYpq
The `_mcpc.stateless` field is now `true` for stateless connections, `false` for stateful ones, and `null` when the mode isn't yet known. The internal `connectionMode` enum and the human-readable `(stateful)`/`(stateless)` suffix are unchanged. Unreleased, so no breaking change. https://claude.ai/code/session_018sp5rqgbDLoXoSFF4VfYpq
jancurn
changed the title
_mcpc JSON output: boolean stateful, drop log size_mcpc JSON output: stateless field, drop log size
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
3 participants
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.
Tidies the (unreleased)
_mcpcblock in the--jsonoutput — session info,mcpc connect, and the session list — after feedback that the shape was awkward.statefulnessstring (valuesstateful/stateless) with astatelessfield:truewhen stateless,falsewhen stateful,nullwhen not yet determined. The internal enum (nowconnectionMode) and the human-readable(stateful)/(stateless)suffix are unchanged.logSizefrom the output — it was a point-in-time snapshot that goes stale as the bridge keeps writing.logPathstays, and thelogscommand gives a fresh value.Both fields are unreleased, so there is no breaking change.
https://claude.ai/code/session_018sp5rqgbDLoXoSFF4VfYpq