Repository navigation
feat: define agent tool access contracts - #3543
Merged
Merged
Conversation
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
ananthsub
marked this pull request as ready for review
September 18, 2026 22:12
ananthsub
requested review from
adil-a,
bxyu-nvidia,
cmunley1,
ffrujeri,
marta-sd and
yaoyu-33
September 18, 2026 22:12
This was referenced Sep 18, 2026
ananthsub
force-pushed
the
ananthsub/environment-server-tool-access
branch
from
September 18, 2026 22:22
e0c1da6 to
dd7b005
Compare
ananthsub
force-pushed
the
ananthsub/environment-server-tool-access
branch
from
September 18, 2026 22:27
dd7b005 to
5ca28f3
Compare
ananthsub
force-pushed
the
ananthsub/environment-server-tool-access
branch
from
September 21, 2026 13:38
ab65cdc to
1326bd1
Compare
ananthsub
force-pushed
the
ananthsub/environment-server-tool-access
branch
from
September 21, 2026 17:19
1326bd1 to
c2c0da1
Compare
ananthsub
force-pushed
the
ananthsub/environment-server-tool-access
branch
from
September 22, 2026 13:13
c2c0da1 to
9ecb0d7
Compare
ananthsub
force-pushed
the
ananthsub/environment-server-tool-access
branch
from
September 22, 2026 13:18
9ecb0d7 to
da4b903
Compare
Signed-off-by: Ananth Subramaniam <ansubramania@nvidia.com>
This branch was previously deployed
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.
What does this PR do?
Defines named direct-HTTP and streamable-HTTP MCP access independently of any session lifecycle. These contracts carry episode-scoped connectivity and authorization data; locally launched MCP processes remain Agent Server configuration.
Single-agent interaction flow
This diagram shows the built-in single-agent use case. Other Environment Server protocols may coordinate different participants and use different request, result, and lifecycle flows.
sequenceDiagram participant C as Rollout collector participant E as Environment Server participant R as Task / Resources Server participant A as Agent Server participant M as Model Server participant S as Task sandbox C->>E: POST /run E->>R: Seed task session R-->>E: Tool access and optional SandboxAccess E->>A: Seed agent session A->>S: Connect as borrower when supplied E->>A: Invoke /v1/responses A->>M: Model and capture requests A-->>E: Agent response E->>A: Close agent session E->>R: Verify response E->>R: Close task session E-->>C: Episode resultThe Environment Server coordinates ordering and the final result. Agent and Task/Resources Servers retain ownership of the state and resources they create.
This is part 4 of 14 in the Environment Server stack. Previous: #3484. Next: #3545.
Validation
uv run pytest tests/unit_tests/test_tool_access.py -qRollouts
See the final PR for end-to-end rollout evidence.
Compatibility and benchmark impact
Additive contract definitions only. Static stdio MCP configuration remains owned by individual Agent Server implementations.