Add openrouter:advisor server tool to skills#47
Add openrouter:advisor server tool to skills#47devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Add the new advisor tool across three skills: - create-headless-agent: server tools table, README, sample tools/index.ts - create-agent-tui: server tools table, README, sample tools/index.ts - openrouter-typescript-sdk: new Server Tools + Advisor Tool section with parameters reference, usage examples (basic, instructions+sub-agent tools, named advisor profiles), and tool-call arguments table Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
🤖 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
Adds documentation and commented-out sample code for the openrouter:advisor server tool across the create-agent-tui, create-headless-agent, and openrouter-typescript-sdk skills.
Verdict: ✅ LGTM
Details
CI: no CI checks configured ✅
Findings (see inline comments for full context):
- 🟡
skills/openrouter-typescript-sdk/SKILL.md:556— unverified model alias in named-advisor example (see inline comment)
Codex: skipped (small tier)
Research: skipped (small tier)
Security: no concerns — documentation only, no runtime code paths touched
Test coverage: N/A — documentation PR
Unresolved threads: none
Scope: first review (full)
Tier: small (109 LoC)
| }, | ||
| { | ||
| name: 'architect', | ||
| model: '~openai/gpt-latest', |
There was a problem hiding this comment.
[suggestion] ~openai/gpt-latest — unverified OpenRouter alias; this slug does not appear elsewhere in the codebase
Details
The ~ prefix is correct for family aliases per the repo's conventions, but gpt-latest is not used anywhere else in this repo. Existing examples use pinned slugs like openai/gpt-5.2 or openai/gpt-4.1-mini. If ~openai/gpt-latest is a real OpenRouter alias, this is fine — but consider verifying against the models catalogue or using a known alias like ~openai/gpt-4o-latest so developers copy-pasting this example don't hit a routing error.
Prompt for agents
In `skills/openrouter-typescript-sdk/SKILL.md` at line 556, the named-advisor example uses `model: '~openai/gpt-latest'`. Verify whether this is a valid OpenRouter model alias at https://openrouter.ai/models. If it is not valid, replace it with a confirmed alias (e.g. `~openai/gpt-4o-latest`) or a pinned slug that matches the repo's existing examples (e.g. `openai/gpt-4.1-mini`).
Reviewed at fe4e7d6
Summary
Adds the new
openrouter:advisorserver tool across three skills:create-headless-agent & create-agent-tui — Added as an OFF-by-default entry in server tools tables (SKILL.md + README.md), with a brief description of capabilities (model pinning, instructions,
forward_transcript, sub-agent tools, named advisor profiles). Sampletools/index.tsgets a commented-outserverTool({ type: 'openrouter:advisor' })line showing how to wire it.openrouter-typescript-sdk — New "Server Tools" subsection under the Tool System section, listing all available server tools with a
serverTool()usage example. Followed by a dedicated "Advisor Tool" subsection with three usage patterns (basic, instructions + sub-agent tools, named profiles), a parameters reference table, and tool-call arguments table.Link to Devin session: https://openrouter.devinenterprise.com/sessions/d2c9ad508c2341f3bd69982b87ee345c