Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@sebslight
Copy link
Member

Summary

  • add browser.snapshotDefaults.mode (currently supports "efficient") to the config schema/types
  • apply the config default in the browser tool + browser CLI snapshot command when no mode is supplied
  • add tests for the new default behavior and ARIA guard
  • document the new config knob in gateway config and browser tool docs

Why

We want a configuration switch that makes efficient snapshots the default only when explicitly configured, so operators can reduce snapshot size/noise without forcing all callers to pass mode=efficient in every tool call or CLI invocation.

Why this approach

  • Keep the setting under browser.* because it controls browser snapshot behavior, not agent policy or tool gating.
  • Apply the default at the tool/CLI layer so it works for LLM tool calls and CLI usage without altering server semantics for other HTTP clients.
  • Limit the schema to mode: "efficient" for now to minimize scope while matching existing behavior; the config does nothing unless explicitly set.

Tests

  • pnpm test -- src/agents/tools/browser-tool.test.ts

steipete added a commit that referenced this pull request Jan 21, 2026
@steipete
Copy link
Contributor

Landed via squash on main.

Commit:

Tests:

  • pnpm lint
  • pnpm build
  • pnpm test

@steipete
Copy link
Contributor

Closing since changes landed on main via squash (commit a0cd295).

@steipete steipete closed this Jan 21, 2026
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.

2 participants