fix(mcp): point README to live integration guide - #55
Conversation
|
Warning Review limit reachedNext included review available in 49 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
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. Comment |
Coverage Report for CI Build 34747002808Warning No base build found for commit Coverage: 79.481%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
The README linked to https://microlink.io/integration/mcp, which returns 404; the live page is https://microlink.io/integrations/mcp. Add a regression test asserting the canonical route. Signed-off-by: Jose Francisco 'Kiko' Verdú Gambín <2096101+Kikobeats@users.noreply.github.com>
2182c0c to
ef9e702
Compare
2 participants
Problem
The README's canonical link,
https://microlink.io/integration/mcp, returns 404 (verified live). The real guide lives athttps://microlink.io/integrations/mcp(200). An agent or user following the README lands on a dead page during setup - exactly the audience the MCP targets.Fix
One-word route fix (
/integration/mcp->/integrations/mcp), plus a regression test asserting the README references the live route and never the stale one.Tests
New
readme-links.test.js. MCP suite 104/104 (8 live opt-in tests skipped),standard+standard-markdownlint green.Note
Low Risk
Documentation URL correction and a static README assertion only; no runtime or auth behavior changes.
Overview
Fixes a broken setup link in
packages/mcp/README.md: the MCP integration walkthrough now points tohttps://microlink.io/integrations/mcpinstead of the 404/integration/mcppath.Adds
readme-links.test.js, which scans READMEmicrolink.ioURLs and fails if the stale route appears or the live/integrations/mcproute is missing.Reviewed by Cursor Bugbot for commit ef9e702. Bugbot is set up for automated code reviews on this repo. Configure here.