fix(mcp): list maintain and contributor-profile in --help#7036
Conversation
printHelp() listed ~30 real top-level commands but omitted two dispatched ones: maintain (13-plus subcommands, its own printMaintainHelp()) and contributor-profile (added by JSONbored#6737). A user running loopover-mcp --help had no way to discover either command exists. Added a usage line for each, following the existing entries' formatting; maintain's line points to loopover-mcp maintain --help for the full subcommand list rather than duplicating it.
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7036 +/- ##
=======================================
Coverage 93.74% 93.74%
=======================================
Files 692 692
Lines 68721 68721
Branches 18760 18760
=======================================
Hits 64424 64424
Misses 3302 3302
Partials 995 995
Flags with carried forward coverage won't be shown. Click here to find out more. |
|
Tip ✅ LoopOver review result - approve/merge recommendedReview updated: 2026-07-17 19:18:40 UTC
Review summary Nits — 3 non-blocking
Decision drivers
Context & advisory signals — never blocks the verdict
Review context
Contributor next steps
Signal definitions
🧪 Chat with LoopOverAsk LoopOver a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.
Full command reference: https://loopover.ai/docs/loopover-commands 🧪 Experimental — new and may change. 🟩 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 LoopOver, a quiet PR intelligence layer for OSS maintainers.
|
Summary
loopover-mcp --help(printHelp()) listed ~30 real, dispatched top-level commands but omitted two:maintain(dispatched with its ownprintMaintainHelp()covering 14 subcommands) andcontributor-profile(contributorProfileCli, added by CLI mirror for loopover_get_contributor_profile #6737). A user running--helphad no way to discover either command exists.maintainusage line following the existing entries' format, listing its subcommands and pointing toloopover-mcp maintain --helpfor the full detail (matching the issue's request not to duplicateprintMaintainHelp()'s own listing).contributor-profileusage line in the same style as the neighboringdecision-pack/repo-decisionentries.test/unit/mcp-cli-help.test.ts) asserting both commands appear in--helpoutput, and that the barehelpcommand (an alias) shows the same banner.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Closes #6991
Validation
git diff --checknpm run actionlint(no workflow changes in this PR)npm run typechecknpm run test:coverage(not applicable:packages/loopover-mcp/**is outside the vitest coverage collection scope percodecov.yml's documented src/**-scoped collection, same as every other CLI-only change in this file; the newmcp-cli-help.test.ts(3 tests) and the existingmcp-cli-completion-spec.test.ts/mcp-cli-basics.test.ts(34 tests) all pass locally)npm run test:workers(not applicable to this change)npm run build:mcp(not applicable to this change)npm run test:mcp-pack(not applicable to this change)npm run ui:openapi:check(no API/schema changes in this PR)npm run ui:lint(no UI changes in this PR)npm run ui:typecheck(no UI changes in this PR)npm run ui:build(no UI changes in this PR)npm audit --audit-level=moderate(no dependency changes in this PR)If any required check was skipped, explain why:
packages/loopover-mcp/bin/loopover-mcp.js's static--helpusage text plus a new test file -- a plain-text banner change with no branches, API surface, or dependency impact, so the checks above have no surface to exercise.npm run docs:drift-checkandnpm run command-reference:checkwere also run locally and pass.Safety
UI Evidencesection below with JPG/JPEG or PNG screenshots. (N/A -- this PR has no UI/visual surface; it's a CLI--helptext change.)Notes
--helpusage banner, not a UI surface -- the new automated test (mcp-cli-help.test.ts) is the evidence that both commands are now listed.