From 57f6345428b2baf580cac8cda6acca863c7cd65d Mon Sep 17 00:00:00 2001 From: Codex Date: Sun, 31 May 2026 11:09:31 +0000 Subject: [PATCH] fix: align gh pr option aliases --- scripts/src/gh.ts | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/scripts/src/gh.ts b/scripts/src/gh.ts index 033b59d7..13490ac9 100644 --- a/scripts/src/gh.ts +++ b/scripts/src/gh.ts @@ -6052,7 +6052,7 @@ export async function runGhCommand(args: string[]): Promise value !== undefined).join(" ") || "gh"; return validationError(command, options.repo, "--stat is only supported by gh pr diff --stat.", { supportedCommands: [ @@ -6216,29 +6216,29 @@ export async function runGhCommand(args: string[]): Promise --stat or gh pr files for a bounded REST file/stat summary.", { rawDiffIncluded: false, supportedCommands: [ - `bun scripts/cli.ts gh pr files ${number} --repo ${options.repo} --limit 30`, - `bun scripts/cli.ts gh pr diff ${number} --stat --repo ${options.repo} --limit 30`, + `bun scripts/cli.ts gh pr files ${resolved.number} --repo ${resolved.repo} --limit 30`, + `bun scripts/cli.ts gh pr diff ${resolved.number} --stat --repo ${resolved.repo} --limit 30`, ], }); } const { token, probe } = resolveToken(true); - const missing = authRequired(options.repo, "pr diff --stat", probe); - if (missing !== null || token === null) return missing ?? authRequired(options.repo, "pr diff --stat", { present: false, source: null, ghFallbackAttempted: true }); - return prFiles(options.repo, token, number, options.limit, "pr diff --stat"); + const missing = authRequired(resolved.repo, "pr diff --stat", probe); + if (missing !== null || token === null) return missing ?? authRequired(resolved.repo, "pr diff --stat", { present: false, source: null, ghFallbackAttempted: true }); + return prFiles(resolved.repo, token, resolved.number, options.limit, "pr diff --stat"); } if (sub === "files") { - const number = parsePositionalNumberForCommand(options.repo, args, 2, "pr files"); - if (typeof number !== "number") return number; + const resolved = resolvePositionalPrReference(args, 2, "pr files", options); + if (isGitHubCommandResult(resolved)) return resolved; const { token, probe } = resolveToken(true); - const missing = authRequired(options.repo, "pr files", probe); - if (missing !== null || token === null) return missing ?? authRequired(options.repo, "pr files", { present: false, source: null, ghFallbackAttempted: true }); - return prFiles(options.repo, token, number, options.limit, "pr files"); + const missing = authRequired(resolved.repo, "pr files", probe); + if (missing !== null || token === null) return missing ?? authRequired(resolved.repo, "pr files", { present: false, source: null, ghFallbackAttempted: true }); + return prFiles(resolved.repo, token, resolved.number, options.limit, "pr files"); } if (sub === "delete") return unsupportedCommand("pr delete", options.repo, "GitHub REST does not support hard-deleting pull requests; use gh pr close for lifecycle deletion semantics."); if (sub === "preflight" || sub === "closeout") { @@ -6304,12 +6304,12 @@ export async function runGhCommand(args: string[]): Promise