Skip to content

feat(coder-labs/modules/codex): add boundary support via agentapi module#795

Merged
35C4n0r merged 6 commits into
coder:mainfrom
shanewhite97:feat/codex-boundary-support
Mar 11, 2026
Merged

feat(coder-labs/modules/codex): add boundary support via agentapi module#795
35C4n0r merged 6 commits into
coder:mainfrom
shanewhite97:feat/codex-boundary-support

Conversation

@shanewhite97
Copy link
Copy Markdown
Contributor

Description

Adds boundary support to the Codex module by passing boundary
variables through to the agentapi module and using
AGENTAPI_BOUNDARY_PREFIX in the start script.

Depends on #780

Type of Change

  • Feature/enhancement

Module Information

Path: registry/coder-labs/modules/codex
Breaking change: No

@shanewhite97
Copy link
Copy Markdown
Contributor Author

@35C4n0r Codex PR as requested, please let me know if you need any changes. I added a test which I think covers the flow you were looking to validate so let me know if this is okay!

@35C4n0r
Copy link
Copy Markdown
Collaborator

35C4n0r commented Mar 10, 2026

@shanewhite97 , the changes look fine and also tested, can you resolve the conflicts ? I think you are working on outdated codex code.

@shanewhite97
Copy link
Copy Markdown
Contributor Author

@35C4n0r There you go, merge conflicts should be resolved

@35C4n0r 35C4n0r self-requested a review March 11, 2026 15:18
@35C4n0r
Copy link
Copy Markdown
Collaborator

35C4n0r commented Mar 11, 2026

@shanewhite97, let's remove the temporary source and use the 2.3.0 agentapi https://registry.coder.com/modules/coder/agentapi

Also bump(minor) codex version in README.md

@shanewhite97
Copy link
Copy Markdown
Contributor Author

@35C4n0r Sorry for the delay. That should be done now if you want to take a look

Comment thread registry/coder-labs/modules/codex/README.md Outdated
@35C4n0r 35C4n0r added the version:minor Add to PRs requiring a minor version upgrade label Mar 11, 2026
Copy link
Copy Markdown
Collaborator

@35C4n0r 35C4n0r left a comment

Choose a reason for hiding this comment

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

Great work, Thanks for the contribution @shanewhite97
LGTM!

@35C4n0r 35C4n0r changed the title feat(coder-labs/codex): add boundary support via agentapi module feat(coder-labs/modules/codex): add boundary support via agentapi module Mar 11, 2026
@35C4n0r 35C4n0r merged commit a0430e6 into coder:main Mar 11, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:minor Add to PRs requiring a minor version upgrade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants