docs: update configuration, sandbox, tools, Go SDK, and MCP catalog docs#3068
Conversation
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
Reviewed 5 documentation files (98 diff lines). Two LIKELY findings in newly added content:
| Severity | File | Issue |
|---|---|---|
| 🟡 MEDIUM | docs/configuration/models/index.md:68 |
New example uses claude-opus-4-5; all other Opus refs in same file use claude-opus-4-7 |
| 🔵 LOW | docs/configuration/tools/index.md:170 |
"Checksum-verified" bold claim overstates guarantee when weak/unsupported checksums are warn-skipped |
| title-generation call to a smaller, faster model: | ||
|
|
||
| ```yaml | ||
| model: anthropic/claude-opus-4-5 |
There was a problem hiding this comment.
[MEDIUM] Likely incorrect Anthropic model identifier: claude-opus-4-5
The newly added YAML example uses anthropic/claude-opus-4-5, but every other Opus reference in this same file uses claude-opus-4-7 (lines 214, 227, 264). The Haiku model in this same example (claude-haiku-4-5) is consistent with line 382, suggesting the Opus suffix was inadvertently set to -4-5 instead of -4-7.
Users who copy this example verbatim may end up with an invalid model identifier. Consider changing to:
model: anthropic/claude-opus-4-7
title_model: anthropic/claude-haiku-4-5|
|
||
| ### Checksum Verification | ||
|
|
||
| Downloaded binaries are **checksum-verified** before installation. Verification fails closed: if a checksum is advertised in the aqua registry manifest but does not match the downloaded archive (or the manifest is missing entirely), the install is aborted and an error is returned. Checksum types that are unsupported or too weak are skipped with a warning rather than causing a hard failure. |
There was a problem hiding this comment.
[LOW] "Checksum-verified" framing overstates the security guarantee
The bold checksum-verified claim in the opening sentence could mislead security-conscious readers: the same paragraph immediately documents that downloads with unsupported or weak checksum types are silently skipped with only a warning (no hard failure). A user relying on the "verified" label for security assurance would not realise that some downloads bypass hard verification entirely.
Consider qualifying the opening sentence, for example:
Downloaded binaries are verified against the aqua registry checksum manifest where supported. Verification fails closed on mismatches...
Documentation updates
This PR updates the docs to reflect five code changes merged into
mainin the last 36 hours.52f5b9d0title_modelconfig field for delegated session-title generation6e2154afmodels.devis always allowlisted in the sandbox network proxy767067daversion_overridesresolution in tool auto-installe613b784toolconfirm,StaticSessionState,Stopper)6fc01903Details
docs/configuration/models/index.md—title_modelfield (#3051)Added a row to the Properties Reference table and a new "Delegating Session-Title Generation" section with a YAML example showing how to delegate title generation to a cheaper model (e.g. Opus → Haiku). Includes a warning callout that
title_modelcannot be combined withfirst_available.docs/configuration/sandbox/index.md—models.devallowlist (#3041)The sandbox network proxy now always opens
models.devso the in-sandbox agent can resolve model metadata (context limits, pricing, capabilities). Without this, the first catalog lookup fails with403 Blocked by network policy. Addedmodels.devto the auto-opened hosts in the network allowlist prose.docs/configuration/tools/index.md— auto-install integrity and compatibility (#3046)Added two subsections to the Auto-Installing Tools section:
version_overridesResolution: aqua registryversion_overridesentries are now resolved correctly, fixing silent install failures for many common tools (e.g.fzf).docs/guides/go-sdk/index.md— TUI embedding packages (#3059)Added three packages to the Core Packages table (
pkg/tui/components/toolconfirm,pkg/tui/service,pkg/tui/animation) and a new "Embedding TUI Components" section explaining the embedding contract: usetoolconfirmfor the shared permission policy,StaticSessionStateas a stubSessionStateReader, andanimation.Stopperto avoid tick leaks.docs/tools/mcp-catalog/index.md— catalog membership note (#3048)Added a bullet to Notes and Limitations: the catalog list changes between releases as integrations are added or removed (Supabase and Tally were removed in this cycle as their OAuth flows were unreliable).
PRs reviewed and found up to date