Publish MCP 2026-07-28 Skill release - #26
Conversation
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
📝 WalkthroughWalkthroughThe release updates version metadata to 2.6.0, adds MCP discovery and cached tool-list responses, expands API field and reply guidance, refreshes public listings and task guides, and strengthens release-content validation. ChangesMCP v2.6.0 release
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant MCPClient
participant StubServer
participant DiscoveryResponse
participant ToolCatalog
MCPClient->>StubServer: server/discover
StubServer->>DiscoveryResponse: build supported versions and cache metadata
DiscoveryResponse-->>MCPClient: discovery capabilities
MCPClient->>StubServer: initialize
StubServer-->>MCPClient: legacy protocol and server info
MCPClient->>StubServer: tools/list
StubServer->>ToolCatalog: return cached tools and TTL
ToolCatalog-->>MCPClient: structured tool list
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@mcpize/SUBMISSION-STEPS.md`:
- Line 14: Align the submission description with the canonical description in
server.json, or update the acceptance check to validate only the required claims
instead of exact equality. Update the relevant description/check consistently so
the documented acceptance requirement is satisfied without maintaining
conflicting text.
In `@scripts/release-guard/repository-checks.mjs`:
- Line 41: Update the “127-operation REST count” pattern in the repository
checks to match the full “127 OpenAPI-documented REST operations” wording,
consistent with the corresponding 128-operation requirement in
content-policy.mjs, so stale old-phrase files are rejected.
In `@skills/x-twitter-scraper/references/types-x-api.md`:
- Around line 115-120: Update the privacy note near the fetching-account
guidance to state that follow-relationship lookup is available through GET
/api/v1/x/followers/check but requires explicit approval. Remove the absolute
wording that relationship state stays private, while preserving the existing
privacy guidance for fetching-account actions and permissions.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 15b760e1-0550-45f0-b899-bd90a77ad23c
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (63)
.claude-plugin/marketplace.json.claude-plugin/plugin.json.codex-plugin/plugin.jsonCHANGELOG.mdREADME.mddocker-mcp-registry/cyrus-submission-steps.mddocker-mcp-registry/xquik-remote/server.yamldocker-mcp-registry/xquik-remote/tools.jsonmcpize/SUBMISSION-STEPS.mdopenclaw.plugin.jsonpackage.jsonscripts/release-guard/content-policy.mjsscripts/release-guard/repository-checks.mjsserver.jsonskills/x-twitter-scraper/SKILL.mdskills/x-twitter-scraper/metadata.jsonskills/x-twitter-scraper/references/mcp-setup.mdskills/x-twitter-scraper/references/mcp-tools.mdskills/x-twitter-scraper/references/types-x-api.mdskills/x-twitter-scraper/skill-card.mdstub-server.mjstask-guides/check-mutuals.mdtask-guides/clean-followers.mdtask-guides/export-tweets-csv.mdtask-guides/extract-followers.mdtask-guides/find-bangers.mdtask-guides/find-influencers.mdtask-guides/find-viral-tweets.mdtask-guides/follow-unfollow.mdtask-guides/for-you-feed.mdtask-guides/going-viral.mdtask-guides/grow-followers.mdtask-guides/monitor-accounts.mdtask-guides/optimize-tweets.mdtask-guides/post-tweets.mdtask-guides/run-giveaway.mdtask-guides/search-tweets.mdtask-guides/send-dms.mdtask-guides/top-replies.mdtask-guides/track-competitors.mdtask-guides/track-hashtags.mdtask-guides/track-mentions.mdtask-guides/trending-news.mdtask-guides/tweet-analytics.mdtask-guides/tweet-ideas.mdtask-guides/tweet-replies.mdtask-guides/tweet-style.mdtask-guides/tweet-webhooks.mdtask-guides/update-x-profile.mdtask-guides/user-tweets.mdtask-guides/who-liked.mdtask-guides/who-quoted.mdtask-guides/who-retweeted.mdtask-guides/write-threads.mdtask-guides/write-tweets.mdtask-guides/x-articles.mdtask-guides/x-bookmarks.mdtask-guides/x-communities.mdtask-guides/x-lists.mdtask-guides/x-spaces.mdtask-guides/x-trends.mdtests/mcp-quickstart.test.mjstests/stub-server.test.mjs
There was a problem hiding this comment.
All reported issues were addressed across 64 files
Tip: instead of fixing issues one by one fix them all with cubic
Re-trigger cubic
adea5eb to
ac32e02
Compare
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Signed-off-by: kriptoburak <kriptoburak@users.noreply.github.com>
ac32e02 to
79461de
Compare
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
@furkanerday The signed head is |
Signed-off-by: kriptoburak <kriptoburak@users.noreply.github.com>
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@task-guides/top-replies.md`:
- Around line 66-69: Add the json language identifier to the opening fence for
both request examples: task-guides/top-replies.md lines 66-69 and
task-guides/tweet-replies.md lines 62-65. Leave the request bodies unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 6ecd5f6c-b775-40d3-92cd-ebfba5fc2060
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (64)
.claude-plugin/marketplace.json.claude-plugin/plugin.json.codex-plugin/plugin.jsonCHANGELOG.mdREADME.mddocker-mcp-registry/cyrus-submission-steps.mddocker-mcp-registry/xquik-remote/server.yamldocker-mcp-registry/xquik-remote/tools.jsonmcpize/SUBMISSION-STEPS.mdopenclaw.plugin.jsonpackage.jsonscripts/release-guard/content-policy.mjsscripts/release-guard/repository-checks.mjsserver.jsonskills/x-twitter-scraper/SKILL.mdskills/x-twitter-scraper/metadata.jsonskills/x-twitter-scraper/references/mcp-setup.mdskills/x-twitter-scraper/references/mcp-tools.mdskills/x-twitter-scraper/references/types-x-api.mdskills/x-twitter-scraper/skill-card.mdskills/x-twitter-scraper/skillspector-report.mdstub-server.mjstask-guides/check-mutuals.mdtask-guides/clean-followers.mdtask-guides/export-tweets-csv.mdtask-guides/extract-followers.mdtask-guides/find-bangers.mdtask-guides/find-influencers.mdtask-guides/find-viral-tweets.mdtask-guides/follow-unfollow.mdtask-guides/for-you-feed.mdtask-guides/going-viral.mdtask-guides/grow-followers.mdtask-guides/monitor-accounts.mdtask-guides/optimize-tweets.mdtask-guides/post-tweets.mdtask-guides/run-giveaway.mdtask-guides/search-tweets.mdtask-guides/send-dms.mdtask-guides/top-replies.mdtask-guides/track-competitors.mdtask-guides/track-hashtags.mdtask-guides/track-mentions.mdtask-guides/trending-news.mdtask-guides/tweet-analytics.mdtask-guides/tweet-ideas.mdtask-guides/tweet-replies.mdtask-guides/tweet-style.mdtask-guides/tweet-webhooks.mdtask-guides/update-x-profile.mdtask-guides/user-tweets.mdtask-guides/who-liked.mdtask-guides/who-quoted.mdtask-guides/who-retweeted.mdtask-guides/write-threads.mdtask-guides/write-tweets.mdtask-guides/x-articles.mdtask-guides/x-bookmarks.mdtask-guides/x-communities.mdtask-guides/x-lists.mdtask-guides/x-spaces.mdtask-guides/x-trends.mdtests/mcp-quickstart.test.mjstests/stub-server.test.mjs
🚧 Files skipped from review as they are similar to previous changes (50)
- task-guides/track-competitors.md
- task-guides/search-tweets.md
- task-guides/tweet-webhooks.md
- task-guides/track-mentions.md
- task-guides/x-spaces.md
- task-guides/extract-followers.md
- task-guides/x-lists.md
- task-guides/optimize-tweets.md
- task-guides/monitor-accounts.md
- task-guides/who-quoted.md
- task-guides/tweet-analytics.md
- task-guides/who-liked.md
- task-guides/find-viral-tweets.md
- task-guides/find-influencers.md
- task-guides/follow-unfollow.md
- task-guides/who-retweeted.md
- task-guides/x-communities.md
- task-guides/find-bangers.md
- task-guides/for-you-feed.md
- task-guides/write-tweets.md
- task-guides/tweet-style.md
- docker-mcp-registry/xquik-remote/server.yaml
- server.json
- task-guides/user-tweets.md
- task-guides/post-tweets.md
- task-guides/x-bookmarks.md
- package.json
- openclaw.plugin.json
- task-guides/run-giveaway.md
- task-guides/send-dms.md
- docker-mcp-registry/cyrus-submission-steps.md
- mcpize/SUBMISSION-STEPS.md
- task-guides/update-x-profile.md
- task-guides/track-hashtags.md
- task-guides/write-threads.md
- task-guides/clean-followers.md
- docker-mcp-registry/xquik-remote/tools.json
- skills/x-twitter-scraper/metadata.json
- skills/x-twitter-scraper/skill-card.md
- task-guides/x-trends.md
- .claude-plugin/plugin.json
- task-guides/grow-followers.md
- skills/x-twitter-scraper/references/mcp-tools.md
- task-guides/trending-news.md
- task-guides/export-tweets-csv.md
- .codex-plugin/plugin.json
- .claude-plugin/marketplace.json
- README.md
- skills/x-twitter-scraper/references/mcp-setup.md
- skills/x-twitter-scraper/SKILL.md
There was a problem hiding this comment.
All reported issues were addressed across 7 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
furkanerday
left a comment
There was a problem hiding this comment.
Two current-head issues remain. First, task-guides/top-replies.md:66 and :74 and task-guides/tweet-replies.md:62 and :70 add fenced JSON request examples without a language identifier, violating the repository's markdown policy (MD040). Second, scripts/release-guard/content-policy.mjs:29 checks only the first wrapped fragment of the security-sensitive README claim; removing or contradicting 'operations remain outside MCP' would still pass. Require that second fragment explicitly, then rerun the package/reproducibility checks.
Signed-off-by: kriptoburak <kriptoburak@users.noreply.github.com>
|
Addressed both current-head review findings in b06d02a. The four new request examples now declare JSON fences, and the release guard separately requires the security-sensitive 'operations remain outside MCP.' fragment. Verification: 14 tests passed, policy coverage remained above all thresholds, package archives are reproducible, every version surface matches 2.6.0, and the diff is clean. |
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Dismissed after verifying the requested changes are implemented on the current head and the relevant local or hosted checks pass. Independent re-review remains requested.
furkanerday
left a comment
There was a problem hiding this comment.
Reviewed the complete current diff, description, commit history, checks, discussion, repository policy, relevant contracts and tests, dependency and release integrity, licensing, DCO, and public-safety implications. Required checks are green and no actionable review threads remain.
furkanerday
left a comment
There was a problem hiding this comment.
Reviewed the complete current diff, description, commit history, checks, discussion, repository policy, relevant contracts and tests, dependency and release integrity, licensing, DCO, and public-safety implications. Required checks are green and no actionable review threads remain.
Summary
Verification
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by cubic
Release v2.6.0 of the Skill and MCP server with
server/discovernegotiation for MCP2026-07-28, updated counts (128 REST, 120 MCP, 119 JSON/text), richer read-data guidance, 5‑minute private cache hints, and stricter docs on MCP boundaries and request fencing.New Features
2.6.0across manifests and skill metadata; add publicCHANGELOG.mdand include it in published files.server/discoverreturningsupportedVersions: ["2026-07-28"],cacheScope: private, andttlMs: 300000; keep legacyinitialize(2025-11-25) for 2025-era clients.tools/listnow returnscacheScopeandttlMs; catalog updated to 120 routes (119 JSON/text).424 replies_incompletefallback viaconversation_id:<tweet_id>search; default top replies to 10.CHANGELOG.md); refresh SkillSpector v2.3.7 scan dated 2026‑07‑30 with 0 findings.Migration
server/discover; modern calls need noinitializeor session ID. Legacy clients remain compatible.@xquikx-developerv2.6.0 and thex-twitter-scraperSDK for automatic headers and request metadata.Written for commit b06d02a. Summary will update on new commits.
Note
Release MCP skill v2.6.0 with
2026-07-28protocol negotiation viaserver/discoverserver/discoverhandler to stub-server.mjs returning supported protocol versions (2026-07-28,2025-11-25), server info, and tool cache hints (cacheScope: 'private',ttlMs: 300000);initializecontinues to return the legacy protocol version2025-11-25for stateless compatibility.package.jsonpublished files; bumps version to 2.6.0 everywhere.Changes since #26 opened
contentChecksarray [b06d02a]Macroscope summarized 1c4c9e9.
Summary by CodeRabbit
New Features
server/discoverfor modern protocol negotiation, including discovery/tool cache metadata (scope + TTL).Documentation
Security