@bensandee/forgejo-cli 0.5.3 → 0.5.4

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.3",
+	version: "0.5.4",
 	description: "bst-fj: Forgejo API commands without the rest of bst",
 	license: "MIT",
 	repository: {
@@ -192,13 +192,13 @@ const parsed = z.object({ version: z.string() }).safeParse({
 	dependencies: {
 		"@bensandee/common": "0.2.0",
 		"citty": "^0.2.2",
-		"zod": "^4.6.5"
+		"zod": "^4.6.7"
 	},
 	devDependencies: {
 		"@bensandee/cli-kit": "workspace:*",
 		"@bensandee/forgejo": "workspace:*",
-		"@types/node": "24.19.1",
-		"tsdown": "0.23.0"
+		"@types/node": "24.19.2",
+		"tsdown": "0.23.1"
 	}
 });
 if (!parsed.success) throw new UnexpectedError(`@bensandee/forgejo-cli's package.json has no string version.`);
@@ -896,7 +896,8 @@ function resolveRepoUrlFrom(readOrigin, readPackageUrl) {
 	const originUrl = normalizeGitUrl(readOrigin() ?? "");
 	if (originUrl !== null && parseGitUrl(originUrl) !== null) return originUrl;
 	const pkgUrl = readPackageUrl();
-	return pkgUrl !== null && parseGitUrl(pkgUrl) !== null ? pkgUrl : null;
+	if (pkgUrl === null || parseGitUrl(pkgUrl) === null) return null;
+	return normalizeGitUrl(pkgUrl);
 }
 function resolveForgejoConnection(env, repoUrl) {
 	const serverUrl = env["FORGEJO_SERVER_URL"];
@@ -987,10 +988,11 @@ function resolveIssueTarget(refTarget, repo) {
 	throw new RepoTargetMismatchError(`${repoTarget.serverUrl}/${repoTarget.repository}`, `${refTarget.serverUrl}/${refTarget.repository}`);
 }
 function normalizeGitUrl(raw) {
-	const sshMatch = raw.match(/^git@([^:]+):(.+?)(?:\.git)?$/);
+	const value = raw.startsWith("git+") ? raw.slice(4) : raw;
+	const sshMatch = value.match(/^git@([^:]+):(.+?)(?:\.git)?$/);
 	if (sshMatch) return `https://${sshMatch[1]}/${sshMatch[2]}`;
 	try {
-		const url = new URL(raw);
+		const url = new URL(value);
 		url.pathname = url.pathname.replace(/\.git$/, "");
 		url.username = "";
 		url.password = "";
@@ -2576,10 +2578,9 @@ const releaseCreateCommand = defineCommand({
 });
 //#endregion
 //#region ../forgejo/src/secrets.ts
-const ActionSecretListSchema = z.object({
-	secrets: z.array(z.object({ name: z.string() })),
-	total_count: z.number()
-});
+const ActionSecretListSchema = z.array(z.object({ name: z.string() }));
+const SECRETS_PAGE_SIZE = 50;
+const SECRETS_MAX_PAGES = 20;
 z.object({ sha1: z.string() });
 const NAME_PATTERN = /^[A-Za-z_][A-Za-z0-9_]*$/;
 const FORBIDDEN_PREFIX_PATTERN = /^(FORGEJO_|GITEA_|GITHUB_|[0-9])/i;
@@ -2605,10 +2606,8 @@ async function setActionSecret(executor, conn, secretName, secretValue) {
 async function listActionSecrets(executor, conn) {
 	const [owner, repo] = conn.repository.split("/");
 	const names = [];
-	let page = 1;
-	const limit = 50;
-	for (;;) {
-		const url = `${conn.serverUrl}/api/v1/repos/${owner}/${repo}/actions/secrets?page=${String(page)}&limit=${String(limit)}`;
+	for (let page = 1; page <= 21; page++) {
+		const url = `${conn.serverUrl}/api/v1/repos/${owner}/${repo}/actions/secrets?page=${String(page)}&limit=${String(SECRETS_PAGE_SIZE)}`;
 		const res = await executor.fetch(url, { headers: { Authorization: `token ${conn.token}` } });
 		if (!res.ok) throw new TransientError(await forgejoErrorMessage(res, {
 			verb: "list secrets",
@@ -2616,9 +2615,9 @@ async function listActionSecrets(executor, conn) {
 		}));
 		const parsed = ActionSecretListSchema.safeParse(await res.json());
 		if (!parsed.success) throw new UnexpectedError(`Unexpected secrets list response: ${parsed.error.message}`);
-		for (const secret of parsed.data.secrets) names.push(secret.name);
-		if (names.length >= parsed.data.total_count) break;
-		page++;
+		for (const secret of parsed.data) names.push(secret.name);
+		if (parsed.data.length === 0) break;
+		if (page === 21) throw new UnexpectedError(`${owner}/${repo} still lists secrets after ${String(SECRETS_MAX_PAGES)} pages; refusing to report a prefix of them`);
 	}
 	return names;
 }
@@ -2879,11 +2878,11 @@ async function hasAdminAccess(executor, conn) {
 const OrgSchema = z.object({ username: z.string() });
 const OrgListSchema = z.array(OrgSchema);
 const PAGE_LIMIT = 50;
+const MAX_PAGES = 100;
 async function paginate(executor, conn, endpoint, verb, extract) {
 	const all = [];
-	let page = 1;
 	const separator = endpoint.includes("?") ? "&" : "?";
-	for (;;) {
+	for (let page = 1; page <= 101; page++) {
 		const url = `${conn.serverUrl}/api/v1${endpoint}${separator}page=${String(page)}&limit=${String(PAGE_LIMIT)}`;
 		const res = await executor.fetch(url, { headers: { Authorization: `token ${conn.token}` } });
 		if (!res.ok) throw new TransientError(await forgejoErrorMessage(res, {
@@ -2893,7 +2892,7 @@ async function paginate(executor, conn, endpoint, verb, extract) {
 		const items = extract(await res.json());
 		all.push(...items);
 		if (items.length < PAGE_LIMIT) break;
-		page++;
+		if (page === 101) throw new UnexpectedError(`${endpoint} still returns full pages after ${String(MAX_PAGES)} pages; refusing to report a prefix of it`);
 	}
 	return all;
 }
@@ -3516,6 +3515,7 @@ function collectInputArgs(rawArgs) {
 	for (let i = 0; i < rawArgs.length; i += 1) {
 		const arg = rawArgs[i];
 		if (arg === void 0) continue;
+		if (arg === "--") break;
 		if (arg === INPUT_FLAG) {
 			const next = rawArgs[i + 1];
 			if (next === void 0) throw new FatalError("Missing value for --input: expected `--input key=value`");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bensandee/forgejo-cli",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "bst-fj: Forgejo API commands without the rest of bst",
   "license": "MIT",
   "repository": {
@@ -29,13 +29,13 @@
   "dependencies": {
     "@bensandee/common": "0.2.0",
     "citty": "^0.2.2",
-    "zod": "^4.6.5"
+    "zod": "^4.6.7"
   },
   "devDependencies": {
     "@bensandee/cli-kit": "0.0.0",
     "@bensandee/forgejo": "0.7.0",
-    "@types/node": "24.19.1",
-    "tsdown": "0.23.0"
+    "@types/node": "24.19.2",
+    "tsdown": "0.23.1"
   },
   "scripts": {
     "build": "tsdown",