Skip to content

fix(cli): dedupe help environment entries#1366

Open
EfeDurmaz16 wants to merge 1 commit into
vercel-labs:mainfrom
EfeDurmaz16:efe/dedupe-session-name-help
Open

fix(cli): dedupe help environment entries#1366
EfeDurmaz16 wants to merge 1 commit into
vercel-labs:mainfrom
EfeDurmaz16:efe/dedupe-session-name-help

Conversation

@EfeDurmaz16
Copy link
Copy Markdown

Summary

  • remove duplicate top-level help entries for AGENT_BROWSER_SESSION_NAME, AGENT_BROWSER_STATE_EXPIRE_DAYS, and AGENT_BROWSER_ENCRYPTION_KEY
  • add an integration test that runs agent-browser --help and asserts AGENT_BROWSER_/AI_GATEWAY env entries are unique within the Environment section

Context

Issue #1335 reports duplicate AGENT_BROWSER_SESSION_NAME help output. While checking main, the functional env path is already wired through parse_flags -> DaemonOptions -> daemon env, so this PR keeps scope to the reproducible help-output duplication.

Duplicate check

Before implementation I checked open PRs and did not find an existing PR for this help/env dedupe. Nearby covered items include #1363 for viewport config, #1364 for proxy slash, #1362 for XDG paths, #1354 for network route body, and #1289/#1315 for comma args.

Tests

  • cargo fmt --manifest-path cli/Cargo.toml -- --check
  • cargo test --manifest-path cli/Cargo.toml --test help_cli
  • cargo test --manifest-path cli/Cargo.toml output::tests

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 15, 2026

@EfeDurmaz16 is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

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