feat(mcp): add slop-risk CLI for pre-push slop self-checks - #2655
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Caution 🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥 🛑 Gittensory review result - fixes requiredReview updated: 2026-07-03 05:38:58 UTC
🛑 Suggested Action - Fix Blockers
Review summary Nits — 1 non-blocking
CI checks failing
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2655 +/- ##
==========================================
- Coverage 96.13% 96.13% -0.01%
==========================================
Files 239 239
Lines 26739 26737 -2
Branches 9699 9697 -2
==========================================
- Hits 25705 25703 -2
Misses 424 424
Partials 610 610 🚀 New features to boost your workflow:
|
Summary
The
gittensory_check_slop_riskMCP tool andPOST /v1/lint/slop-riskAPI already assess deterministic slop risk from local diff metadata and a PR description, but contributors had no shell equivalent for CI hooks or pre-push checks without spinning up an MCP session.This change adds
gittensory-mcp slop-risk, a thin CLI wrapper that posts to/v1/lint/slop-riskwith--description,--description-file, repeatable--changed-file(path[:additions:deletions]),--test, and--test-file. Plain output prints the score, band, and findings;--jsonreturns the full report.No linked issue — discovered by comparing existing MCP lint tools against the CLI surface where
lint-pr-textalready has a shell mirror.Scope
type(scope): short summaryConventional Commit format, for examplefix(api): restore profile access checks.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlintnpm run typechecknpm run test:coveragelocally;codecov/patchrequires ≥99% coverage of the lines AND branches you changed (aim for 100% on your diff so CI variance does not fail near the threshold). Global coverage is a non-blocking trend with a loose 90% backstop, not the gate.npm run test:workersnpm run build:mcpnpm run test:mcp-packnpm run ui:openapi:checknpm run ui:lintnpm run ui:typechecknpm run ui:buildnpm audit --audit-level=moderateIf any required check was skipped, explain why:
npm run test:ci(includes all checks above).Safety
UI Evidencesection below with JPG/JPEG or PNG screenshots arranged as organized, captioned, clickable thumbnails. SVG screenshots are not used as review evidence. Review-only screenshots or recordings are not committed to the repository.UI Evidence
N/A — MCP CLI only; no visible UI changes.
Notes
gittensory-mcp slop-risk --changed-file src/widget.ts:80:2 --description "Adds retry handling." --test-file test/unit/widget.test.ts --json