Record what is left of the connected-sessions work - #7
Merged
Merged
Conversation
The reasoning for three open items lived only in a conversation. Sessions can now read each other's work and have it reviewed by a spawned agent, both scoped to the project, and what remains is a set of decisions rather than a set of tasks. Connections were deferred rather than dropped, and the note says why: a connection gates the shared log and the analysis, so building it before the analysis existed would have stored a type nothing read. Now that both exist the question is answerable from use — whether project scope is actually too coarse — and the one case project scope cannot express is a connection across projects, which inverts what a connection is for. Live token counts need a parser rather than a field read, and are worth it only if a review runs long enough that a moving number says more than a spinner does. Pushing work to an existing session goes under "Not built yet" instead, since it is deliberately absent rather than pending. The case for it evaporated when the reviewer became a spawned agent: nothing has to be pushed into a session that is already there.
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.
The reasoning for three open items lived only in a conversation. This puts it in the two files that already hold that kind of thing.
docs/backlog.mdgains three entries, and its review date moves to today:stream-jsonwith--include-partial-messagesrather than reading the final envelope.CLAUDE.mdcannot provide because it is not committed.docs/architecture.mdgains one entry under Not built yet: pushing work to a session that already exists. It belongs there rather than in the backlog because it is deliberately absent, not pending — the case for it evaporated when the reviewer became a spawned agent.Documentation only; no code changes.