@bensandee/forgejo-cli 0.5.0 → 0.5.2

package/dist/bin.mjs

@@ -162,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.5.0",
+	version: "0.5.2",
 	description: "bst-fj: Forgejo API commands without the rest of bst",
 	license: "MIT",
 	repository: {
@@ -5762,9 +5762,10 @@ const prCloseCommand = defineCommand({
 		};
 		const number = parsePrNumber(args.number);
 		const comment = readBodySource(args.comment ?? void 0, args["comment-file"] ?? void 0, { flags });
-		const conn = requireForgejoTargetFromRepo(process.cwd(), args.repo ?? void 0, "pr:close is Forgejo-only -- use gh pr close <n> on GitHub.");
 		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,
@@ -6437,8 +6438,21 @@ const PrFileEntrySchema = z.object({
 	additions: z.number().default(0),
 	deletions: z.number().default(0)
 });
-async function listPrFiles(executor, conn, prNumber, options) {
-	return readPrFilesPage(executor, conn, prNumber, 1, options?.limit ?? DEFAULT_LIMIT);
+const ALL_FILES_MAX_PAGES = 20;
+async function listAllPrFiles(executor, conn, prNumber) {
+	const files = [];
+	for (let page = 1; page <= ALL_FILES_MAX_PAGES; page++) {
+		const listing = await readPrFilesPage(executor, conn, prNumber, page, DEFAULT_LIMIT);
+		files.push(...listing.files);
+		if (!listing.truncated) return {
+			files,
+			truncated: false
+		};
+	}
+	return {
+		files,
+		truncated: true
+	};
 }
 async function readPrFilesPage(executor, conn, prNumber, page, limit) {
 	const url = `${conn.serverUrl}/api/v1/repos/${conn.repository}/pulls/${String(prNumber)}/files?page=${String(page)}&limit=${String(limit)}`;
@@ -6463,7 +6477,7 @@ async function readPrFilesPage(executor, conn, prNumber, page, limit) {
 //#region src/pr-files.ts
 function runPrFiles(fetcher, options) {
 	const number = parsePrNumber(options.number);
-	return listPrFiles(fetcher, requireForgejoTarget(options.cwd, parseRepoTarget(options.repo), "pr:files is Forgejo-only -- gh pr diff --name-only already serves GitHub."), number);
+	return listAllPrFiles(fetcher, requireForgejoTarget(options.cwd, parseRepoTarget(options.repo), "pr:files is Forgejo-only -- gh pr diff --name-only already serves GitHub."), number);
 }
 function renderPrFiles(listing) {
 	const lines = listing.files.map((file) => `${file.status ?? "unknown"} ${file.filename}`);
@@ -6476,7 +6490,7 @@ function renderPrFiles(listing) {
 const prFilesCommand = defineCommand({
 	meta: {
 		name: "pr:files",
-		description: "List the files a pull request changes (Forgejo-only)"
+		description: "List every file a pull request changes, reading every page (Forgejo-only)"
 	},
 	args: {
 		number: {
@@ -6490,7 +6504,7 @@ const prFilesCommand = defineCommand({
 		},
 		json: {
 			type: "boolean",
-			description: "Emit one JSON document: files ([{filename, status (or null), additions, deletions}]), truncated."
+			description: "Emit one JSON document: files ([{filename, status (or null), additions, deletions}]), truncated (true only when the pull request changes more files than the command reads, at least 1000)."
 		},
 		verbose: {
 			type: "boolean",

package/package.json

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