server instructions

[angiejones]

nudge agent to use accessibility tree over screenshots

Summary by CodeRabbit

• Chores

  • Version updated to 0.2.2

• New Features

  • Server configuration now accepts an optional instructions field to allow custom guidance for instances

• Documentation

  • Tool usage guidance clarified with recommendations on when to rely on accessibility metadata versus screenshots and best-practice notes

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

This PR bumps package version to 0.2.2 and adds an optional instructions field to MCP server initialization while expanding the take_screenshot tool description with usage guidance.

Changes

Cohort / File(s)	Summary
Version Update package.json	Version changed from 0.2.1 to 0.2.2.
Server options & tool metadata src/lib/server.js	Added an instructions option to McpServer instantiation signature and expanded the take_screenshot tool description to prefer accessibility snapshots and advise screenshot usage only when visual layout matters.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A small bump, a gentle nudge to say,
Add guidance, tidy words along the way,
Screenshots hush, accessibility speaks first,
Instructions planted — clarity rehearsed,
A rabbit's cheer for changes light and spry.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'server instructions' is vague and does not clearly convey the main objective. While it relates to the PR's addition of instructions functionality, it fails to communicate the actual intent: nudging agents to prefer accessibility trees over screenshots.	Consider a more descriptive title like 'Add instructions to nudge agent toward accessibility tree' or 'Prefer accessibility tree in take_screenshot guidance' that better captures the PR's core objective.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch accessibility-tip

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/lib/server.js`:
- Around line 20-24: The McpServer instantiation passes the instructions string
in the first argument (serverInfo) where only name and version belong; move the
instructions property into the second argument (ServerOptions) so the agent
nudge is honored—update the McpServer call that constructs server (new
McpServer({...}, {...})) to pass { name, version } as the first arg and put
instructions into the second arg object.