Skip to content

Add api and token commands - #97

Open
Ankur Goyal (ankrgyl) wants to merge 5 commits into
mainfrom
http-commands
Open

Ankur Goyal (ankrgyl) wants to merge 5 commits into
mainfrom
http-commands

Conversation

@ankrgyl

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Apr 5, 2026 •

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit bc323464262e:

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a couple nits but otherwise looks good

Comment thread src/api.rs Outdated
Comment thread src/auth.rs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

command structure and such look good!

@viadezo1er

Copy link
Copy Markdown
Contributor

Stephen Belanger (Qard) added a commit that referenced this pull request Sep 23, 2026
## Summary

- Pin `bt-daemon` to the merged plugins [PR
#97](braintrustdata/braintrust-coding-agent-plugins#97)
revision in `Cargo.toml` and `Cargo.lock`.
- Return `HostRouteSelection` from bt's trace host, containing only the
auth selection and default project name.
- Let the daemon construct its own session route and trace-specific
settings.

## Validation

- `cargo test --locked --bin bt` (830 passed)
- `pre-commit run --all-files --show-diff-on-failure` (all hooks passed)

This branch has not been deployed

No deployments
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.

4 participants