Skip to content

feat: small improvements on readme and adding dependencies - #85

Merged
sergiofilhowz merged 1 commit into
mainfrom
feat/mcp-dependency
May 6, 2026
Merged

sergiofilhowz merged 1 commit into
mainfrom
feat/mcp-dependency

Conversation

@sergiofilhowz

@sergiofilhowz sergiofilhowz commented May 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Documentation

    • Updated Model Context Protocol configuration guide with clarified expose settings and custom API path examples.
    • Changed default port from 3000 to 3111 with updated quick-start examples.
  • Chores

    • Updated project dependencies.

@coderabbitai

coderabbitai Bot commented May 6, 2026 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f08eb4ce-ec14-4e2b-a4b6-c85b4ae0ec11

📥 Commits

Reviewing files that changed from the base of the PR and between a9c9cfb and c5a8862.

⛔ Files ignored due to path filters (1)
  • mcp/Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • mcp/README.md
  • mcp/iii.worker.yaml

📝 Walkthrough

Walkthrough

Documentation for the MCP bridge is updated to clarify its architecture as a Rust binary with HTTP/SSE endpoints, the Quickstart port is changed from 3000 to 3111 with corresponding examples, configuration sections are refined to explain require_expose and api_path binding, and out-of-scope features are expanded. A skills dependency (^0.2.1) is added to the worker manifest.

Changes

Documentation and Manifest Updates

Layer / File(s) Summary
Configuration Manifest
mcp/iii.worker.yaml
Added dependencies block declaring skills at version ^0.2.1.
Architecture & Integration Documentation
mcp/README.md (lines 3–9)
MCP bridge introduction reworded to describe a standalone Rust binary with HTTP/SSE endpoints and JSON-RPC mapping; external link and resource/prompts narrative updated.
Quickstart Guide
mcp/README.md (lines 19–38)
Port changed from 3000 to 3111; curl examples updated; SSE transport clarified with data-stream extraction details.
Configuration Reference
mcp/README.md (lines 61–62)
Clarified that require_expose: true advertises only functions with metadata.mcp.expose == true; added example for custom api_path binding (e.g., /mcp-custom).
Feature Scope Definition
mcp/README.md (lines 101–104)
Expanded out-of-scope list to denote resource subscriptions, completion, logging, notification fan-out, and stdio transport as not covered in this version.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • iii-hq/workers#54: Updates MCP specification coverage and feature documentation phases in mcp/README.md with overlapping scope clarifications.

Poem

🐰 A bridge of protocols, built clean and bright,
New paths to bind, and features in sight,
From port to docs, all carefully laid,
The worker skills dance in the upgrade parade! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/mcp-dependency

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

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

@sergiofilhowz
sergiofilhowz merged commit 3bfce47 into main May 6, 2026
6 of 7 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request May 12, 2026
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