Skip to content

Fix SDK-288 tracing auth and diagnostics - #334

Draft
Stephen Belanger (Qard) wants to merge 9 commits into
mainfrom
t3code/sdk-288-bt
Draft

Fix SDK-288 tracing auth and diagnostics#334
Stephen Belanger (Qard) wants to merge 9 commits into
mainfrom
t3code/sdk-288-bt

Conversation

@Qard

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

Copy link
Copy Markdown
Contributor

Summary

Implements the bt-owned P0 and P1 work from SDK-288 in issue-level commits:

  • SDK-272: make trace enable / trace setup, trace run, and trace import complete from the credentials already available to the command. Environment auth remains invocation-local for run/import; persistent setup materializes the API key into a normal saved profile for future hooks; interactive commands collect a missing key, org, or project instead of requiring a separate login step.
  • SDK-274: require explicit consent before ordinary bt login saves BRAINTRUST_API_KEY; the persistent tracing setup command itself is the explicit request to create the durable login it needs.
  • SDK-270 / SDK-275: keep bare bt status concise; expand saved profile and credential-storage details only under --all, and show precedence only for an active CLI/environment override.
  • SDK-276: clarify that logout is local-only and add a one-confirmation logout --all flow with per-profile JSON results.
  • SDK-273: pin the landed daemon revision and provide side-effect-free, credential-redacted auth provenance to bt trace doctor.

Dependency

braintrustdata/braintrust-coding-agent-plugins#32 has landed. This PR pins its squash commit on main (face71d8d758363e881e15310ce79d5fcafa2912).

Validation

  • cargo fmt --check
  • cargo test --all-targets --locked (757 unit tests, 56 CLI tests, and all dataset, eval, functions, and setup integration suites passed)
  • cargo clippy --all-targets --locked -- -D warnings -A clippy::collapsible_match
  • git diff --check
  • focused regressions prove environment-key persistent setup and explicit-key managed run both complete with saved-profile routes, while environment-key managed run remains invocation-local

Plain cargo clippy --all-targets --locked -- -D warnings is currently blocked by pre-existing collapsible_match findings in unrelated dataset, SQL, and trace TUI code under Rust 1.95; allowing that lint leaves the branch clean.

@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit f4de4c6b962d:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • 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-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

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.

1 participant