Repository navigation
Add Muse Code tracing, import, and attach - #70
Merged
Stephen Belanger (Qard) merged 21 commits intoOct 27, 2026
Merged
Conversation
Stephen Belanger (Qard)
force-pushed
the
t3code/plan-sdk-312-implementation
branch
2 times, most recently
from
September 11, 2026 13:21
a86284a to
345b900
Compare
Stephen Belanger (Qard)
force-pushed
the
t3code/plan-sdk-312-implementation
branch
3 times, most recently
from
September 24, 2026 15:47
1a0c8e0 to
6d624cd
Compare
Stephen Belanger (Qard)
force-pushed
the
t3code/plan-sdk-312-implementation
branch
from
October 16, 2026 18:34
7b8ecd6 to
45ce4f7
Compare
Stephen Belanger (Qard)
force-pushed
the
t3code/plan-sdk-312-implementation
branch
from
October 20, 2026 17:49
45ce4f7 to
6dc2a78
Compare
Stephen Belanger (Qard)
marked this pull request as ready for review
October 23, 2026 14:50
Stephen Belanger (Qard)
requested review from
Abhijeet Prasad (AbhiPrasad) and
Andrew Kent (realark)
October 23, 2026 14:50
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. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6be646487f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "Codex (@codex) review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "Codex (@codex) address that feedback".
Abhijeet Prasad (AbhiPrasad)
approved these changes
Oct 26, 2026
Stephen Belanger (Qard)
force-pushed
the
t3code/plan-sdk-312-implementation
branch
from
October 26, 2026 18:22
56c085b to
87559fc
Compare
Stephen Belanger (Qard)
deleted the
t3code/plan-sdk-312-implementation
branch
October 27, 2026 16:40
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
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.
Muse Code supports native tracing across live capture, historical import, managed runs, and attach. Live hooks and exports share deterministic model/tool IDs; detaching leaves native work open, and late completion enriches the same spans without allowing older root snapshots to replace newer answers.
muse-code-sdk = 1.4.12for MSP initialization, stdio framing, request correlation, and concrete session-list parameters/results. The client lives inbt-daemon/src/muse/msp.rs; subprocess creation retains the repository's Windows-safe helper, bounded diagnostics, timeouts, and cancellation cleanup.bt-daemon/src/muse. Decode hooks once into a typed event enum; stream export records through that same translator. Reducer state stays intranslate/muse*.rs.muse/config/settings.rs,hooks.rs, andownership.rstype existing settings, hook collections, command handlers, and strict owned-file recognition. Setup orchestration contains no dynamic JSON navigation. Missing/null distinctions and unknown extension data survive edits; only exact owned files/handlers are eligible for migration/removal. Typed settings use the shared atomic writer under Muse’s native lock.Validation:
cargo test --manifest-path bt-daemon/Cargo.toml --all-features --locked: 622 passed, 9 opt-in tests ignored.import muse --all; and both managed-hook coexistence/sanitized-environment tests.Native fidelity limits remain explicit: hook messages/output are previews; permission hooks do not report decisions; compaction lacks operation IDs. Observed built-in reminder child hook/export metadata omits parent identity, so ancestry is not guessed across concurrent sessions. Legacy exports without model_step cannot guarantee live/export model deduplication. Muse's official installer supports Linux/macOS; Windows runs Rust fixture tests only.
Distribution: hooks ship through bt. After merge, its existing
update-bt-daemon.ymlworkflow prepares the consumer pin/lockfile PR; publication follows the bt release process.Review follow-up: consistently resolve
BRAINTRUST_MUSE_BINthrough the environment helper; prepare fresh explicit daemon endpoints before sanitized hooks run; supply host information from the standalone entry point; and reject empty one-shot imports while allowing empty attach polls and preserving nonempty active sessions. Regressions cover standalone/custom-socket startup and empty imports; native managed-run verification also checks real hook delivery at a fresh explicit endpoint.