fix: don't crash if GitHub API call for rulesetId option fails

package.json

@@ -79,7 +79,7 @@
 		"@vitest/eslint-plugin": "1.1.31",
 		"all-contributors-cli": "6.26.1",
 		"bingo-stratum-testers": "0.5.1",
-		"bingo-testers": "0.5.1",
+		"bingo-testers": "0.5.2",
 		"console-fail-test": "0.5.0",
 		"cspell": "8.17.3",
 		"eslint": "9.20.1",

src/base.ts

@@ -9,7 +9,7 @@ import lazyValue from "lazy-value";
 import npmUser from "npm-user";
 import { z } from "zod";
 
-import { inputFromOctokit } from "./inputs/inputFromGitHub.js";
+import { inputFromOctokit } from "./inputs/inputFromOctokit.js";
 import { parsePackageAuthor } from "./options/parsePackageAuthor.js";
 import { readDefaultsFromReadme } from "./options/readDefaultsFromReadme.js";
 import { readDescription } from "./options/readDescription.js";
@@ -175,9 +175,9 @@ export const base = createBase({
 					owner: await owner(),
 					repo: await repository(),
 				},
-			})) as { id: string; name: string }[];
+			})) as undefined | { id: string; name: string }[];
 
-			return rulesets.find(
+			return rulesets?.find(
 				(ruleset) => ruleset.name === "Branch protection for main",
 			)?.id;
 		});

src/inputs/inputFromOctokit.test.ts

@@ -0,0 +1,45 @@
+import { createMockFetchers, testInput } from "bingo-testers";
+import { describe, expect, it, vi } from "vitest";
+
+import { inputFromOctokit } from "./inputFromOctokit.js";
+
+describe("inputFromOctokit", () => {
+	it("returns data when the request resolves", async () => {
+		const data = JSON.stringify({ found: true });
+
+		const actual = await testInput(inputFromOctokit, {
+			args: {
+				endpoint: "GET /repos/{owner}/{repo}/rulesets",
+				options: {},
+			},
+			fetchers: createMockFetchers(
+				vi.fn().mockResolvedValueOnce(
+					// eslint-disable-next-line n/no-unsupported-features/node-builtins
+					new Response(data),
+				),
+			),
+		});
+
+		expect(actual).toEqual(data);
+	});
+
+	it("returns undefined when the request rejects", async () => {
+		const actual = await testInput(inputFromOctokit, {
+			args: {
+				endpoint: "GET /repos/{owner}/{repo}/rulesets",
+				options: {},
+			},
+			fetchers: createMockFetchers(
+				vi.fn().mockResolvedValueOnce(
+					// eslint-disable-next-line n/no-unsupported-features/node-builtins
+					new Response("", {
+						status: 404,
+						statusText: "Not found.",
+					}),
+				),
+			),
+		});
+
+		expect(actual).toBe(undefined);
+	});
+});

src/inputs/inputFromGitHub.ts → src/inputs/inputFromOctokit.ts

@@ -10,13 +10,17 @@ export const inputFromOctokit = createInput({
 	// This will require smart types around GitHub endpoints, similar to:
 	// https://github.com/JoshuaKGoldberg/bingo/issues/65
 	async produce({ args, fetchers }): Promise<unknown> {
-		const response = await fetchers.octokit.request(args.endpoint, {
-			headers: {
-				"X-GitHub-Api-Version": "2022-11-28",
-			},
-			...args.options,
-		});
+		try {
+			const response = await fetchers.octokit.request(args.endpoint, {
+				headers: {
+					"X-GitHub-Api-Version": "2022-11-28",
+				},
+				...args.options,
+			});
 
-		return response.data;
+			return response.data;
+		} catch {
+			return undefined;
+		}
 	},
 });