Repository navigation
feat: declare idempotency, describe tool outputs, answer the root with 200 - #48
Merged
Merged
Conversation
…h 200 Manufact's MCP catalog scored 93 and deducted for: no idempotentHint on the billed tools, crawl/sitemap/trending descriptions that did not say what they return, field descriptions that restated schema limits, and the declared widget domain answering with a 301 instead of responding. - Billed tools declare idempotentHint: false (each call costs credits). - crawl, sitemap and trending descriptions and output fields say what comes back; input descriptions explain usage instead of limits. - GET / answers 200 with a canonical link and an instant meta refresh to s1.dev/mcp-server, keeping query parameters. - Regenerated the LobeHub manifest. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SJGhGT4LDJjiEpCrQX6PW7
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Manufact's MCP catalog listed us at 93/100 (https://manufact.com/leaderboard/search1api). This PR addresses the deductions it reported:
idempotentHint. The billed tools now declareidempotentHint: false, because every call costs credits.render_resultskeepstrue.GET /onmcp.search1api.comused to 301 to s1.dev, which the scorer counted as "domain not responding". It now returns 200 with:lhm.plugin.json(npm run sync:manifest).Tests:
npm testpasses 46/46, including a new root-page test and an idempotency assertion.Generated by Claude Code