Skip to content

Add turn-scoped context contributions#28911

Merged
jif-oai merged 6 commits into
mainfrom
jif/context-contributor-turn-scope
Jun 18, 2026
Merged

Add turn-scoped context contributions#28911
jif-oai merged 6 commits into
mainfrom
jif/context-contributor-turn-scope

Conversation

@jif-oai

@jif-oai jif-oai commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • keep context injection on a single ContextContributor trait
  • split context injection into thread-scoped and turn-scoped contribution methods
  • wire turn-scoped fragments into initial context assembly so extensions can contribute context from turn-local state

@jif-oai jif-oai requested a review from a team as a code owner June 18, 2026 12:35
chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Jun 18, 2026
View reviewed changes

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

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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7e1012d6da

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

Comment thread codex-rs/core/src/session/mod.rs Outdated
Comment thread codex-rs/core/src/session/mod.rs Outdated
Comment thread codex-rs/ext/extension-api/src/contributors.rs
@jif-oai jif-oai force-pushed the jif/context-contributor-turn-scope branch from 7e1012d to 4913b38 Compare June 18, 2026 12:47
@jif-oai jif-oai force-pushed the jif/context-contributor-turn-scope branch from 4913b38 to 9fa63da Compare June 18, 2026 12:54
@jif-oai

jif-oai commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Jun 18, 2026
View reviewed changes

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

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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9fa63da602

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

Comment thread codex-rs/core/src/session/mod.rs
Comment thread codex-rs/core/src/session/mod.rs
Comment thread codex-rs/ext/extension-api/src/contributors.rs
@github-actions github-actions Bot mentioned this pull request Jun 18, 2026
Open
@jif-oai jif-oai merged commit 9684ec2 into main Jun 18, 2026
29 of 31 checks passed
@jif-oai jif-oai deleted the jif/context-contributor-turn-scope branch June 18, 2026 17:40
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jif-oai