Skip to content

fix(mcp): make decision-pack and repo-decision CLI commands honor --help #5995

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
loopover-orb merged 1 commit into from
Jul 15, 2026
Merged

fix(mcp): make decision-pack and repo-decision CLI commands honor --help #5995

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions packages/loopover-mcp/bin/loopover-mcp.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2160,7 +2160,21 @@ async function issueSlopCli(args) {
for (const finding of payload.findings ?? []) process.stdout.write(`- ${finding.title}: ${finding.detail}\n`);
}

function printDecisionPackHelp() {
process.stdout.write(
[
"Usage: loopover-mcp decision-pack --login <github-login> [--json]",
"",
"Fetch the cached (or freshly built) contributor decision pack for a GitHub login.",
"Mirrors the loopover_get_decision_pack MCP tool and GET /v1/contributors/{login}/decision-pack. No source upload.",
"",
"Pass --json for machine-readable output.",
].join("\n") + "\n",
);
}

async function decisionPackCli(options) {
if (options.help === true) return printDecisionPackHelp();
const login = options.login ?? process.env.LOOPOVER_LOGIN ?? process.env.GITHUB_LOGIN;
if (!login) throw new Error("Pass --login <github-login> or set LOOPOVER_LOGIN.");
const payload = await getDecisionPackWithCache(login);
Expand All @@ -2173,7 +2187,21 @@ async function decisionPackCli(options) {
if (payload.cache?.rerunGuidance) process.stdout.write(`Rerun when: ${payload.cache.rerunGuidance}\n`);
}

function printRepoDecisionHelp() {
process.stdout.write(
[
"Usage: loopover-mcp repo-decision --login <github-login> --repo owner/repo [--json]",
"",
"Fetch the cached (or freshly built) repo decision for a GitHub login and repo.",
"Mirrors the loopover_explain_repo_decision MCP tool. No source upload.",
"",
"Pass --json for machine-readable output.",
].join("\n") + "\n",
);
}

async function repoDecisionCli(options) {
if (options.help === true) return printRepoDecisionHelp();
const login = options.login ?? process.env.LOOPOVER_LOGIN ?? process.env.GITHUB_LOGIN;
if (!login) throw new Error("Pass --login <github-login> or set LOOPOVER_LOGIN.");
const repoFullName = options.repo;
Expand Down
14 changes: 14 additions & 0 deletions test/unit/mcp-cli-packets.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,20 @@ describe("loopover-mcp CLI — packets", () => {
});
});

it("prints decision-pack help without requiring --login or making a network call", () => {
const help = run(["decision-pack", "--help"]);
expect(help).toMatch(/Usage: loopover-mcp decision-pack/);
expect(help).toMatch(/loopover_get_decision_pack/);
expect(help).toMatch(/contributor decision pack/);
});

it("prints repo-decision help without requiring --login/--repo or making a network call", () => {
const help = run(["repo-decision", "--help"]);
expect(help).toMatch(/Usage: loopover-mcp repo-decision/);
expect(help).toMatch(/loopover_explain_repo_decision/);
expect(help).toMatch(/repo decision/);
});

it("ignores incompatible decision-pack cache entries and clears cache entries on request", async () => {
tempDir = mkdtempSync(join(tmpdir(), "loopover-cli-"));
const url = await startFixtureServer();
Expand Down