@bensandee/forgejo-cli 0.4.4 → 0.5.1

package/README.md

@@ -55,6 +55,7 @@ command takes `--json`, which prints exactly one JSON document, failures include
 | `pr:files`          | List the files a pull request changes                                                                                                                                                                                |
 | `pr:merge`          | Merge an open, non-draft, mergeable PR pinned to the head it read; refuses closed, draft, conflicted, unconfirmed or non-green checks (zero statuses included) unless `--ignore-checks`; `--delete-branch` is opt-in |
 | `pr:ready`          | Mark a draft pull request ready, dropping one `WIP:` prefix or taking `--title`, and confirm the forge no longer reports a draft                                                                                     |
+| `pr:close`          | Close an open pull request, optionally posting `--comment`/`--comment-file` first; reports an already-closed or merged PR without writing; refuses an issue                                                          |
 | `pr:retarget`       | Repoint an open pull request at a new base branch, confirm the forge took it, and report its mergeability as advice                                                                                                  |
 | `workflow:dispatch` | Dispatch a workflow with `--workflow`, `--ref` and repeatable `--input`; takes `--repo owner/repo`, not a URL                                                                                                        |
 | `ci:runs`           | List a repository's workflow runs with their jobs, filtered by workflow, ref, pull request, head SHA or status                                                                                                       |

package/dist/bin.mjs

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 import { existsSync, mkdirSync, readFileSync, statSync, writeFileSync, writeSync } from "node:fs";
-import * as path$1 from "node:path";
-import path from "node:path";
+import path, * as path$1 from "node:path";
 import { parseArgs, parseEnv } from "node:util";
 import { FatalError, TransientError, UnexpectedError, describeError, redact, redact as redact$1 } from "@bensandee/common";
 import { z } from "zod";
@@ -163,7 +162,7 @@ function loadEnvFileForBin(rawArgs, bin, io = processIo) {
 //#region src/version.ts
 const parsed = z.object({ version: z.string() }).safeParse({
 	name: "@bensandee/forgejo-cli",
-	version: "0.4.4",
+	version: "0.5.1",
 	description: "bst-fj: Forgejo API commands without the rest of bst",
 	license: "MIT",
 	repository: {
@@ -5064,6 +5063,9 @@ var EmptyCommentError = class extends FatalError {
 		super("The comment resolved to empty text.");
 	}
 };
+function rejectEmptyComment(comment) {
+	if (comment !== void 0 && comment.trim() === "") throw new EmptyCommentError();
+}
 var PullRequestStateError = class extends FatalError {
 	number;
 	action;
@@ -5073,20 +5075,35 @@ var PullRequestStateError = class extends FatalError {
 		this.action = action;
 	}
 };
+var NotAPullRequestError = class extends FatalError {
+	number;
+	constructor(number) {
+		super(`#${String(number)} is an issue, not a pull request.`);
+		this.number = number;
+	}
+};
+var StateNotTakenError = class extends FatalError {
+	number;
+	expected;
+	actual;
+	constructor(number, expected, actual) {
+		super(`The forge did not change #${String(number)}: it is still ${actual}, not ${expected}.`);
+		this.number = number;
+		this.expected = expected;
+		this.actual = actual;
+	}
+};
 const TARGET_STATE = {
 	close: "closed",
 	reopen: "open"
 };
-async function setIssueState(resolve, request) {
-	const ref = parseIssueRef(request.ref);
-	const number = requireIssueNumber(ref);
-	if (request.comment !== void 0 && request.comment.trim() === "") throw new EmptyCommentError();
-	const commentGiven = request.comment !== void 0;
-	const { api } = resolve(resolveIssueTarget(ref.target, request.repo));
+async function changeState(api, number, request, subject) {
 	const before = await api.get(number);
-	if (before.isPullRequest) throw new PullRequestStateError(number, request.action);
+	if (subject === "issue" && before.isPullRequest) throw new PullRequestStateError(number, request.action);
+	if (subject === "pull-request" && !before.isPullRequest) throw new NotAPullRequestError(number);
+	const commentGiven = request.comment !== void 0;
 	const targetState = TARGET_STATE[request.action];
-	const buildResult = (url, current, changed, posted) => ({
+	const buildResult = (url, current, changed, posted, merged) => ({
 		number,
 		url,
 		action: request.action,
@@ -5095,20 +5112,32 @@ async function setIssueState(resolve, request) {
 			current
 		},
 		changed,
-		...commentGiven ? { comment: { posted } } : {}
+		...commentGiven ? { comment: { posted } } : {},
+		...merged === void 0 ? {} : { merged }
 	});
-	if (before.state === targetState) return buildResult(before.url, before.state, false, false);
+	if (before.state === targetState) {
+		const merged = subject === "pull-request" ? (await api.pullRequest(number)).merged : void 0;
+		return buildResult(before.url, before.state, false, false, merged);
+	}
 	if (request.dryRun) return buildResult(before.url, targetState, true, true);
 	if (commentGiven) await api.addComment(number, request.comment ?? "");
 	let after;
 	try {
 		after = await api.setState(number, targetState);
+		if (after.state !== targetState) throw new StateNotTakenError(number, targetState, after.state);
 	} catch (error) {
 		if (commentGiven && error instanceof Error) error.message = `The comment on #${String(number)} was already posted; a retry will duplicate it. ` + error.message;
 		throw error;
 	}
 	return buildResult(after.url, after.state, true, true);
 }
+async function setIssueState(resolve, request) {
+	const ref = parseIssueRef(request.ref);
+	const number = requireIssueNumber(ref);
+	rejectEmptyComment(request.comment);
+	const { api } = resolve(resolveIssueTarget(ref.target, request.repo));
+	return changeState(api, number, request, "issue");
+}
 const PAST_TENSE = {
 	close: "Closed",
 	reopen: "Reopened"
@@ -5187,10 +5216,13 @@ function defineIssueStateCommand(action) {
 			} catch (error) {
 				if (error instanceof IssueNumberRequiredError) throw new FatalError(describeIssueNumberRequired(`bst-fj ${name} 273`));
 				if (error instanceof EmptyCommentError) throw new FatalError(describeEmptyComment(flags));
-				if (error instanceof PullRequestStateError) throw new FatalError(describePullRequestState(error, {
-					command: `bst-fj ${name}`,
-					advice: "Use the forge UI to change a pull request's state."
-				}));
+				if (error instanceof PullRequestStateError) {
+					const advice = action === "close" ? `Run \`bst-fj pr:close ${String(error.number)}\` to close it.` : "Use the forge UI to change a pull request's state.";
+					throw new FatalError(describePullRequestState(error, {
+						command: `bst-fj ${name}`,
+						advice
+					}));
+				}
 				throw error;
 			}
 		}
@@ -5660,6 +5692,96 @@ const prChecksCommand = defineCommand({
 	}
 });
 //#endregion
+//#region ../forgejo/src/pr-close.ts
+async function closePullRequest(api, request) {
+	rejectEmptyComment(request.comment);
+	return changeState(api, request.number, {
+		action: "close",
+		dryRun: request.dryRun,
+		...request.comment === void 0 ? {} : { comment: request.comment }
+	}, "pull-request");
+}
+function formatPrClose(result, dryRun) {
+	if (result.changed) return formatIssueState(result, dryRun);
+	const what = result.merged === true ? "merged" : "closed";
+	const lines = [`#${String(result.number)} is already ${what} -- nothing to close. (${result.url})`];
+	if (result.comment !== void 0) lines.push("  comment: not posted (pull request was already closed)");
+	return lines.join("\n");
+}
+const PR_CLOSE_JSON_HELP = "Emit one JSON document: number, url, action (close), state ({previous, current}), changed, merged? (present only when the pull request was already closed), comment? ({posted}; present only when a comment was given).";
+//#endregion
+//#region ../forgejo/src/pr-number.ts
+function parsePrNumber(raw) {
+	const value = Number(raw);
+	if (!/^\d+$/.test(raw) || !Number.isSafeInteger(value) || value < 1) throw new FatalError(`Invalid pull request number "${raw}" (expected a positive integer).`);
+	return value;
+}
+//#endregion
+//#region src/commands/pr-close.ts
+const prCloseCommand = defineCommand({
+	meta: {
+		name: "pr:close",
+		description: "Close an open pull request, optionally posting a comment first. A pull request already closed or merged is reported, not an error, and no comment is posted. An issue is refused (Forgejo-only; use gh pr close on GitHub)"
+	},
+	args: {
+		number: {
+			type: "positional",
+			required: true,
+			description: "Pull request number"
+		},
+		comment: {
+			type: "string",
+			description: "Post this comment before writing state"
+		},
+		"comment-file": {
+			type: "string",
+			description: "Read the comment from a file ('-' for stdin)"
+		},
+		repo: {
+			type: "string",
+			description: "Target repository URL (default: this repo's origin)"
+		},
+		"dry-run": {
+			type: "boolean",
+			description: "Report what would change, mutate nothing"
+		},
+		json: {
+			type: "boolean",
+			description: PR_CLOSE_JSON_HELP
+		},
+		verbose: {
+			type: "boolean",
+			description: "Emit detailed debug output"
+		}
+	},
+	async run({ args }) {
+		const out = createCliOutput(args.json === true);
+		const flags = {
+			text: "--comment",
+			file: "--comment-file"
+		};
+		const number = parsePrNumber(args.number);
+		const comment = readBodySource(args.comment ?? void 0, args["comment-file"] ?? void 0, { flags });
+		const dryRun = args["dry-run"] === true;
+		try {
+			rejectEmptyComment(comment);
+			const conn = requireForgejoTargetFromRepo(process.cwd(), args.repo ?? void 0, "pr:close is Forgejo-only -- use gh pr close <n> on GitHub.");
+			const result = await closePullRequest(forgejoIssueApi(cliFetcher(), conn), {
+				number,
+				dryRun,
+				...comment === void 0 ? {} : { comment }
+			});
+			out.json(result);
+			out.text(formatPrClose(result, dryRun));
+			if (dryRun && comment !== void 0 && result.comment?.posted === true) out.text(`[dry-run] Comment that would be posted:\n${comment}`);
+		} catch (error) {
+			if (error instanceof EmptyCommentError) throw new FatalError(describeEmptyComment(flags));
+			if (error instanceof NotAPullRequestError) throw new FatalError(`#${String(error.number)} is an issue, not a pull request -- pr:close refuses to write issue state. Use \`bst-fj issue:close ${String(error.number)}\` to close it.`);
+			throw error;
+		}
+	}
+});
+//#endregion
 //#region ../forgejo/src/closing-refs.ts
 const CLOSING_REF = /(?:^|[\s([])(clos(?:e|es|ed)|fix(?:es|ed)?|resolv(?:e|es|ed))(?::[ \t]*|[ \t]+)((?:[\w.-]+\/[\w.-]+)?#\d+)(?![\w-])/gim;
 function capitalize(word) {
@@ -6339,13 +6461,6 @@ async function readPrFilesPage(executor, conn, prNumber, page, limit) {
 	};
 }
 //#endregion
-//#region ../forgejo/src/pr-number.ts
-function parsePrNumber(raw) {
-	const value = Number(raw);
-	if (!/^\d+$/.test(raw) || !Number.isSafeInteger(value) || value < 1) throw new FatalError(`Invalid pull request number "${raw}" (expected a positive integer).`);
-	return value;
-}
-//#endregion
 //#region src/pr-files.ts
 function runPrFiles(fetcher, options) {
 	const number = parsePrNumber(options.number);
@@ -6877,6 +6992,7 @@ function renderPrReady(result) {
 //#region src/registry-pr.ts
 const prSubCommands = {
 	"pr:checks": prChecksCommand,
+	"pr:close": prCloseCommand,
 	"pr:create": prCreateCommand,
 	"pr:files": prFilesCommand,
 	"pr:list": prListCommand,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bensandee/forgejo-cli",
-  "version": "0.4.4",
+  "version": "0.5.1",
   "description": "bst-fj: Forgejo API commands without the rest of bst",
   "license": "MIT",
   "repository": {