Skip to content

Update auth method terminal#863

Open
anaslimem wants to merge 2 commits intoagentclientprotocol:mainfrom
anaslimem:update-auth-method-terminal
Open

Update auth method terminal#863
anaslimem wants to merge 2 commits intoagentclientprotocol:mainfrom
anaslimem:update-auth-method-terminal

Conversation

@anaslimem
Copy link
Copy Markdown
Contributor

Summary

Change AuthMethodTerminal.env from HashMap<String, String> to Vec<EnvVariable> for consistency with McpServerStdio.env.

Changes

Code Changes (src/agent.rs)

  • Changed AuthMethodTerminal.env field type from HashMap<String, String> to Vec<EnvVariable>
  • Updated builder method to accept Vec<EnvVariable>
  • Removed unused HashMap import
  • Updated test to use new EnvVariable struct

Documentation Updates

  • docs/rfds/auth-methods.mdx: Updated JSON example and field description
  • docs/protocol/draft/schema.mdx: Updated env type from object to EnvVariable[]

Before vs After

Code

// Before
pub env: HashMap<String, String>

// After  
pub env: Vec<EnvVariable>

@anaslimem anaslimem requested a review from a team as a code owner March 27, 2026 15:19
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