Skip to content

fix connection#353

Merged
jdchawla29 merged 1 commit intomainfrom
j/fix-connection
Feb 26, 2026
Merged

fix connection#353
jdchawla29 merged 1 commit intomainfrom
j/fix-connection

Conversation

@jdchawla29
Copy link
Collaborator

@jdchawla29 jdchawla29 commented Feb 26, 2026

Note

Medium Risk
Changes how connector copies are created and rewrites HUD-specific request headers, which could affect remote MCP session/environment routing if assumptions differ across servers. Scope is limited and covered by new unit tests for HUD vs non-HUD behavior.

Overview
Ensures Connector.copy() produces fully isolated state for parallel eval traces by deep-copying the transport and cloning ConnectionConfig list fields, rather than reusing shared mutable objects.

For HUD MCP servers only, copy() now normalizes Environment-Name casing and generates a fresh Environment-Id (UUID) per copied connector to prevent environment/session state leaking across concurrent contexts. Adds tests validating transport/config isolation and that HUD connectors get unique environment IDs while non-HUD connectors keep existing env/session headers.

Written by Cursor Bugbot for commit 903421e. This will update automatically on new commits. Configure here.

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@jdchawla29 jdchawla29 merged commit e8e9964 into main Feb 26, 2026
9 checks passed
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.

1 participant