docs(create-agent-tui): sync with openrouter-web changes - #162
docs(create-agent-tui): sync with openrouter-web changes#162devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Original prompt from Automation
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
There was a problem hiding this comment.
Perry's Review
Verdict: ✅ LGTM
Risk: 🟢 Low
Review details
Summary
Docs-only change syncing skills/create-agent-tui/SKILL.md with openrouter-web #32317, which adds a public mode field to web-search configuration. The PR adds mode to the Web Search server-tool config table and a one-paragraph note covering the engine-specific valid values, defaults, and the engine-selection caveat.
Diff analysis
- Config table (
engine, mode, max_results, domain filtering): correctly insertsmodebetweenengineandmax_results, matching the logical order (engine selection → mode/depth → result count → domain filter). - Mode paragraph: accurately documents the engine-native search depth values — Exa (
instant,fast,auto(default),deep-lite,deep,deep-reasoning) and Parallel (turbo(default),basic,advanced). The caveat ("Setengineexplicitly whenever settingmode— a mode the selected engine does not support is ignored and the engine's default applies") is clear and actionable. The cost note ("Deeper modes cost more per request") is a useful callout.
Blast radius
- The change affects only the Web Search server-tool row and its accompanying paragraph. No code is changed — this is purely documentation for a scaffolding skill.
- The related
create-headless-agentskill lists the tool without a Config column, so it correctly needs no sync (as noted in the PR body).
CI
No CI checks configured for this PR.
Risk assessment
Risk: 🟢 Low
| Dimension | Severity | Risk | Reasoning |
|---|---|---|---|
| Implementation risk | 🟩 | Low | Docs-only change; no code, no logic, no runtime behavior. |
| Premise risk | 🟩 | Low | The source PR (#32317) is explicitly cited and the mode values match its description. |
| Estimated impact | 🟩 | Low | Worst case: a scaffolded agent generates slightly inaccurate web-search config guidance; no runtime, data, or security impact. |
| Risk Factor | Severity | Risk | Reasoning |
|---|---|---|---|
| Reversibility | 🟩 | Low | A docs change is trivially reverted. |
| Detectability | 🟩 | Low | Any inaccuracy surfaces immediately when a user configures mode on a scaffolded agent. |
| Blast radius | 🟩 | Low | A single skill file; only affects agents scaffolded with this skill. |
| Data integrity | None | No persisted state is touched. | |
| Financial exposure | 🟩 | Low | The "deeper modes cost more" note is a general statement with no specific billing claim. |
| Security and privacy exposure | None | Nothing sensitive is reachable from this change. | |
| Propagation | None | No downstream code reads this skill file. | |
| Availability | None | The change cannot affect whether anything serves. | |
| Recovery cost | None | A rollback completes the recovery. | |
| Time to correct | None | A problem is corrected as soon as it is noticed. |
Summary
Source: openrouter-web #32317 —
feat(web-search): support selectable Exa and Parallel modes.That PR adds a public
modefield to web-search configuration (web plugin prefs, chat-completions web-search config, Responses web-search tool, and theopenrouter:web_searchserver tool), backed by a newWebSearchModeschema inprojects/docs/openapi/openapi.yaml. Valid values are engine-specific:instant,fast,auto(default),deep-lite,deep,deep-reasoningturbo(default),basic,advancedA mode the selected engine doesn't support is ignored and the engine default applies, so the engine should be set explicitly alongside the mode.
The skill's server-tool checklist enumerates the configurable knobs of
openrouter:web_searchand listed onlyengine, max_results, domain filtering; this addsmodeplus a one-paragraph note on values, defaults, and the engine-selection caveat.Other candidates from the same 24h window were reviewed and need no change: the Seedance reference-input commit (#31977) only widens an OpenAPI description and changes runtime capability detection —
openrouter-videonever claimed the narrower Seedance-2.0-only rule; the fusion default changes (#32429, #32853) touch documented defaults for a surface no skill teaches.create-headless-agentlists the tool but teaches no configuration, so it is unaffected.Link to Devin session: https://openrouter.devinenterprise.com/sessions/33d74573e39445308cc84b8158d6e273