Skip to content

Add openrouter:advisor server tool to skills#47

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1781031885-add-advisor-tool
Open

Add openrouter:advisor server tool to skills#47
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1781031885-add-advisor-tool

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Adds the new openrouter:advisor server 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). Sample tools/index.ts gets a commented-out serverTool({ 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

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-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@perry-the-pr-maintainer perry-the-pr-maintainer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[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

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