Skip to content

Separate route preparation from native translation - #176

Merged
Stephen Belanger (Qard) merged 2 commits into
mainfrom
pipeline/03-route-neutral-spans
Oct 19, 2026
Merged

Stephen Belanger (Qard) merged 2 commits into
mainfrom
pipeline/03-route-neutral-spans

Conversation

@Qard

@Qard Stephen Belanger (Qard) commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Native translators produce route-neutral operations: attachment, additional metadata, and root tags are applied by routing preparation. The public translator/import API retains its behavior through a compatibility adapter; daemon translation produces canonical values directly. Correlation receives the route overlay before publishing parent candidates, independently of sink delivery.

Route tags remain on nested Codex and OpenCode agent roots, and OpenCode nested roots retain additional metadata. Native span identity and metadata take precedence over route customization; private metadata is excluded.

Based on main at 977101b, including merged #175 and the landed Rust SDK pin. Preserves main's session outcomes, Codex multimodal input and image tool results, Cursor subagent correlation, and production panic bans. Correlation preparation accepts the configuration already validated by its caller.

Pipeline layer 3; base: main (includes #175).

Validation at b77f18f: all 516 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.

Rebased at d1910be: both commits retain identical patches (git range-diff); diff checks pass. Tests are left to fresh CI as requested.

@Qard
Stephen Belanger (Qard) force-pushed the pipeline/03-route-neutral-spans branch from b369ef4 to 8852b3f Compare October 13, 2026 15:47
@Qard
Stephen Belanger (Qard) force-pushed the pipeline/03-route-neutral-spans branch from 8852b3f to aa9d98c Compare October 13, 2026 16:03
@Qard
Stephen Belanger (Qard) force-pushed the pipeline/03-route-neutral-spans branch from aa9d98c to 07920d1 Compare October 13, 2026 16:04
@Qard
Stephen Belanger (Qard) force-pushed the pipeline/03-route-neutral-spans branch from 07920d1 to 82a1ea5 Compare October 13, 2026 16:12
@Qard
Stephen Belanger (Qard) force-pushed the pipeline/03-route-neutral-spans branch from 82a1ea5 to ba1fcf8 Compare October 13, 2026 16:23
@Qard
Stephen Belanger (Qard) force-pushed the pipeline/03-route-neutral-spans branch from ba1fcf8 to 7f37168 Compare October 13, 2026 16:48
@Qard
Stephen Belanger (Qard) marked this pull request as ready for review October 13, 2026 17:04
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T17:08:29.401956Z 7f37168 Draft marked ready
ℹ️ About Codex in GitHub

Your 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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7f37168a86

ℹ️ 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".

Comment thread bt-daemon/src/pipeline/prepare.rs
Base automatically changed from pipeline/02-routing-actor to main October 19, 2026 18:08
@Qard
Stephen Belanger (Qard) force-pushed the pipeline/03-route-neutral-spans branch from b77f18f to d1910be Compare October 19, 2026 18:09
@Qard
Stephen Belanger (Qard) merged commit ece990c into main Oct 19, 2026
22 checks passed
@Qard
Stephen Belanger (Qard) deleted the pipeline/03-route-neutral-spans branch October 19, 2026 18:20
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