Skip to content

Delegate trace commands to plugin runtime - #299

Merged
Stephen Belanger (Qard) merged 1 commit into
mainfrom
agent/refresh-codex-trace-plugin
Aug 12, 2026
Merged

Delegate trace commands to plugin runtime#299
Stephen Belanger (Qard) merged 1 commit into
mainfrom
agent/refresh-codex-trace-plugin

Conversation

@Qard

@Qard Stephen Belanger (Qard) commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • pin the plugin runtime from braintrust-coding-agent-plugins#20
  • mount the exported bt_daemon::TraceArgs tree under the top-level bt trace namespace
  • delete src/agents.rs and delegate the complete trace command to bt_daemon::run_trace
  • retain only a small host adapter for Braintrust credential leases and destination selection

Ownership boundary

bt no longer matches or interprets any coding-agent trace subcommand. It hands the parsed trace command to the plugin runtime with the current executable mount prefix and an implementation of the host-service interface.

The plugin repository owns setup, daemon lifecycle, hooks, status, stop, imports, managed runs, output rendering, agent package identities, marketplace behavior, and settings reconciliation. bt continues to own its profile store, OAuth/API-key handling, project picker, and top-level Clap mount.

This PR depends on braintrustdata/braintrust-coding-agent-plugins#20 and pins its exact commit.

Validation

  • cargo fmt --all --check
  • cargo check --locked
  • cargo test --locked --test cli trace_ (13 passed)
  • cargo test --locked --quiet -- --test-threads=1 (803 passed)
  • cargo clippy --locked --all-targets --all-features -- -D warnings -A clippy::collapsible_match
  • git diff --check

Strict Clippy without the targeted allowance remains blocked by 12 existing collapsible_match findings in unrelated files.

@Qard
Stephen Belanger (Qard) force-pushed the agent/refresh-codex-trace-plugin branch from c4b2175 to deaeb8e Compare August 12, 2026 20:14
@Qard Stephen Belanger (Qard) changed the title Refresh Codex tracing plugin during setup Mount trace commands from plugin runtime Aug 12, 2026
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit bf03da0a31e6:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@Qard
Stephen Belanger (Qard) force-pushed the agent/refresh-codex-trace-plugin branch from deaeb8e to 9f9ed79 Compare August 12, 2026 20:31
@Qard Stephen Belanger (Qard) changed the title Mount trace commands from plugin runtime Delegate trace commands to plugin runtime Aug 12, 2026
@Qard
Stephen Belanger (Qard) marked this pull request as ready for review August 12, 2026 20:47
@Qard
Stephen Belanger (Qard) force-pushed the agent/refresh-codex-trace-plugin branch from 9f9ed79 to bf03da0 Compare August 12, 2026 21:06
@Qard
Stephen Belanger (Qard) merged commit 8edb15a into main Aug 12, 2026
43 of 46 checks passed
@Qard
Stephen Belanger (Qard) deleted the agent/refresh-codex-trace-plugin branch August 12, 2026 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants