feat(nemo-agents-plugin): add codex adapter to NAT plugin#472
Draft
mmogallapalli wants to merge 5 commits into
Draft
feat(nemo-agents-plugin): add codex adapter to NAT plugin#472mmogallapalli wants to merge 5 commits into
mmogallapalli wants to merge 5 commits into
Conversation
Contributor
|
91bd341 to
453296c
Compare
Signed-off-by: Manjesh Mogallapalli <mmogallapall@nvidia.com>
Signed-off-by: Manjesh Mogallapalli <mmogallapall@nvidia.com>
Signed-off-by: Manjesh Mogallapalli <mmogallapall@nvidia.com>
781d60e to
09c5e34
Compare
Signed-off-by: Manjesh Mogallapalli <mmogallapall@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Description
Summary
Adds experimental Codex agent support to
nemo-agentsby vendoring the NATCodex adapter and wiring it into the plugin package.
This PR:
plugins/nemo-agentsnat.componentsentry pointcodex_agentin nemo-agents config validationWhy vendor the Codex adapter?
The Codex adapter exists upstream in the NeMo Agent Toolkit repo under
examples/experimental/codex_agent_adapter, but it is not currently publishedas an installable package that end users can depend on directly.
That means a normal user installation like:
would not include the Codex adapter unless users also cloned the NAT repo and
manually installed the experimental example package. That is not a viable
product path for
nemo-agents.Vendoring gives us a shippable path now:
nemo-agentsinstalls with Codex adapter support includedThe vendored source records its upstream provenance in:
User Flow
Users install/use
nemo-agentsas usual, then provide a config with:The included example is:
The Codex adapter requires
codexandnemo-relayto be installed onPATH.Relay is used internally by the adapter to run Codex and bridge Codex events
into NAT telemetry.
Validation
Focused tests:
Result:
Manual smoke tests:
nemo agents package --no-buildvalidates the Codex example without unknown workflow warningsnemo agents create --name codex-agent --agent-config plugins/nemo-agents/examples/codex-agent/codex-agent.ymlnemo agents deploy --agent codex-agentnemo agents invoke --agent codex-agent --input "Read pyproject.toml and say only the project name. Do not edit files."Invoke returned: