Repository navigation
Introduce pipeline messages and checked routing acceptance - #174
Conversation
8706b90 to
cf9768c
Compare
0727848 to
b6119ce
Compare
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: b6119ce0c3
ℹ️ 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".
c6d0f76 to
026a97f
Compare
026a97f to
45ca37f
Compare
45ca37f to
3514528
Compare
|
we should make sure we bump rust sdk dep before merging in all these prs |
Stage extraction and completion-preserving messages establish the contracts for the recoverable pipeline.
Delivery<T>retains its completion obligation through mapping and forwarding; terminal extraction confirms event checkpoint persistence. Plugin outcomes describe results without naming successor stages.Braintrust sinks enable checked delivery and own a queue per routing branch, preventing a rejected sibling batch from poisoning another route's flush. Rejected flushes invalidate the actor's contiguous checkpoint boundary so consumed failed batches remain eligible for journal replay. Deferred delivery and checkpoint-write failures can still complete later. This layer preserves event replay; automatic recovery comes in subsequent layers.
The SDK pin uses the merged braintrust-sdk-rust#106 commit
f2216fa0ceee132e02f1392c0e5af89ae762e9c8, with the matching Cargo.lock revision. Unused channel-port traits are deferred to the next layer; dead-code allowances cover only individual methods and a field.Stacked on #194, which isolates doctor tests from installed settings, and main at
98051a5, including production unwrap/expect bans, security dependency updates, and all-span provenance. Main's expanded provenance coverage replaces the former root-only sink test.Pipeline layer 1 of 9; base: #194.
Validation at
3514528: all 513 tests pass in the locked all-feature Rust suite without exclusions or environment overrides; five live-agent tests remain normally ignored. Strict all-target/all-feature Clippy, formatting, and diff checks pass. Restacked on #194 atb2c24be; fresh remote CI is pending.SDK dependency update at
2c4e7be: strict Clippy, formatting, and diff checks pass. The local full-suite attempt stopped atstalled_initialize_keeps_cursor_tool_observation_fail_open; further validation is delegated to CI as requested.