Skip to content

Support additional metadata across trace commands - #25

Merged
Stephen Belanger (Qard) merged 1 commit into
mainfrom
t3code/plan-sdk-107
Aug 18, 2026
Merged

Support additional metadata across trace commands#25
Stephen Belanger (Qard) merged 1 commit into
mainfrom
t3code/plan-sdk-107

Conversation

@Qard

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

Copy link
Copy Markdown
Collaborator

Summary

  • accept BRAINTRUST_ADDITIONAL_METADATA and --additional-metadata for setup, hooks, managed runs, and import/attach
  • persist setup metadata, propagate invocation routes to the daemon, and keep hook shims thin
  • use only the resolved SessionConfig route for root metadata and attachment across Claude, Codex, OpenCode, and Pi

Validation

  • cargo test --manifest-path bt-daemon/Cargo.toml --all-features --locked
  • cargo clippy --manifest-path bt-daemon/Cargo.toml --all-targets --all-features --locked -- -D warnings
  • make test

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we just call it BRAINTRUST_METADATA?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

actually I saw the linear, let's do this!

@Qard
Stephen Belanger (Qard) merged commit 38eb332 into main Aug 18, 2026
20 checks passed
@Qard
Stephen Belanger (Qard) deleted the t3code/plan-sdk-107 branch August 18, 2026 15:41
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