Say who actually uses this, and which lane - #16
Merged
Merged
Conversation
Used by listed Vibe Bar and then the single word Auspex, and predated the Rust lane entirely. It now names all three consumers, what each one takes, and the fact that Auspex is the only user of AgentSessionLive -- which is the thing a reader most wants to know before touching that target. It also states the consequence of static linking that the version section explains but this list did not: a kit release reaches nobody until the app that consumes it ships a build. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Scoping CI (Swift) to Swift and contract paths made sense for pushes, but Build and test is a required status check: a filtered workflow never reports on a docs-only change, so the branch rule waits for a run that can never start and the pull request cannot be merged at all -- not even with admin. This one hit that on its own README change. Pushes to main keep the filter, where the run is informational rather than a gate. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
Used bylisted Vibe Bar, then the single word "Auspex.", and predated the Rust lane entirely.It now names all three consumers and what each takes:
AgentSessionKitfor the index, transcripts, deletion planning and the MCP transport.AgentSessionLive. That is the thing a reader most wants to know before touching that target, and the old entry did not say it.agent-session-core, so both Vibe Bar clients read sessions by one set of rules rather than two.Also states the consequence of static linking that the versioning section explains but this list did not: a kit release reaches nobody until the app consuming it ships a build.
Docs only.
🤖 Generated with Claude Code