@goodfellow-cms/github 0.6.0 → 0.7.0

package/dist/index.js

@@ -1,4 +1,4 @@
-import { ConflictError, DomainError, DraftError, EditorsError, GitApiError, NIGHTLY_VARIABLE, PluginTaskError, REBUILD_JOB, SetupError, SignInError, TASK_BRANCH_PREFIX, TASK_EVENT, TASK_POLL_MS, TASK_RESULT_FILE, TASK_TIMEOUT_MS, TASK_WORKFLOW, UpdatesError, assertContentChanges, branchDrafts, buildStepFor, compareReleases, credentialStorage, decodeBase64, encodeBase64, encodeBase64Bytes, hostSource, nightlyMode, siteVersion, taskRequestProblem } from "@goodfellow-cms/core";
+import { ConflictError, DomainError, DraftError, EditorsError, GitApiError, NIGHTLY_VARIABLE, PluginTaskError, REBUILD_JOB, SetupError, SignInError, TASK_BRANCH_PREFIX, TASK_EVENT, TASK_POLL_MS, TASK_RESULT_FILE, TASK_TIMEOUT_MS, TASK_WORKFLOW, UpdatesError, assertContentChanges, branchDrafts, buildStepFor, compareReleases, credentialStorage, decodeBase64, encodeBase64, encodeBase64Bytes, hostSource, nightlyMode, siteVersion, taskRequestProblem, withRateLimits } from "@goodfellow-cms/core";
 //#region src/api.ts
 /** Calls the GitHub REST or GraphQL API. Treats 401 as an expired sign-in; other errors become `GitApiError`. */
 async function githubRequest({ apiUrl, token, fetch }, path, init = {}) {
@@ -703,6 +703,14 @@ var GitHubBackend = class GitHubBackend {
 	draftHost() {
 		const base = `/repos/${this.repo}`;
 		const ref = (branch) => `${base}/git/refs/heads/${encodeURIComponent(branch)}`;
+		const openPulls = (branch) => {
+			const owner = decodeURIComponent(this.repo).split("/")[0];
+			const query = new URLSearchParams({
+				head: `${owner}:${branch}`,
+				state: "open"
+			});
+			return githubJson(this.api, `${base}/pulls?${query}`);
+		};
 		return {
 			liveBranch: this.branch,
 			branchStore: (branch) => new GitHubBackend({
@@ -733,7 +741,7 @@ var GitHubBackend = class GitHubBackend {
 			listBranches: async (prefix) => {
 				return (await githubJson(this.api, `${base}/git/matching-refs/heads/${encodeURIComponent(prefix)}`)).map((found) => found.ref.replace(/^refs\/heads\//, ""));
 			},
-			openReview: async ({ branch, title, description }) => {
+			openReview: async ({ branch, title, description, reviewers = [] }) => {
 				const response = await githubRequest(this.api, `${base}/pulls`, {
 					method: "POST",
 					body: {
@@ -748,16 +756,46 @@ var GitHubBackend = class GitHubBackend {
 						422
 					]
 				});
-				if (response.ok) return { url: (await response.json()).html_url };
-				if (response.status !== 422) throw new DraftError("no-reviews", "This sign-in can't open pull requests on the site's repository.");
-				const owner = decodeURIComponent(this.repo).split("/")[0];
-				const query = new URLSearchParams({
-					head: `${owner}:${branch}`,
-					state: "open"
+				let review;
+				if (response.ok) review = await response.json();
+				else if (response.status !== 422) throw new DraftError("no-reviews", "This sign-in can't open pull requests on the site's repository.");
+				else {
+					[review] = await openPulls(branch);
+					if (!review) throw new GitApiError(422, "GitHub couldn't open the pull request.");
+				}
+				for (const reviewer of reviewers) await githubRequest(this.api, `${base}/pulls/${review.number}/requested_reviewers`, {
+					method: "POST",
+					body: { reviewers: [reviewer] },
+					allow: [
+						403,
+						404,
+						422
+					]
+				});
+				return { url: review.html_url };
+			},
+			listReviews: async (prefix) => {
+				const pulls = await githubJson(this.api, `${base}/pulls?${new URLSearchParams({
+					state: "open",
+					per_page: "100"
+				})}`);
+				const repo = decodeURIComponent(this.repo).toLowerCase();
+				return pulls.filter((pull) => pull.head.ref.startsWith(prefix) && (pull.head.repo?.full_name ?? repo).toLowerCase() === repo).map((pull) => ({
+					branch: pull.head.ref,
+					url: pull.html_url,
+					title: pull.title
+				}));
+			},
+			closeReview: async (branch) => {
+				for (const pull of await openPulls(branch)) await githubRequest(this.api, `${base}/pulls/${pull.number}`, {
+					method: "PATCH",
+					body: { state: "closed" },
+					allow: [
+						403,
+						404,
+						422
+					]
 				});
-				const [open] = await githubJson(this.api, `${base}/pulls?${query}`);
-				if (!open) throw new GitApiError(422, "GitHub couldn't open the pull request.");
-				return { url: open.html_url };
 			}
 		};
 	}
@@ -1820,7 +1858,7 @@ function github(options) {
 	const links = githubTokenLinks(options.repo, options.webUrl);
 	const storage = credentialStorage(`goodfellow:github:${options.repo}`, options.storage);
 	const pending = credentialStorage(`goodfellow:github:pending:${options.repo}`, options.storage);
-	const fetcher = options.fetch ?? ((input, init) => globalThis.fetch(input, init));
+	const fetcher = withRateLimits(options.fetch ?? ((input, init) => globalThis.fetch(input, init)), { host: "GitHub" });
 	const location = options.location ?? browserLocation();
 	const signInUrl = options.signIn;
 	const service = signInUrl && location && !isLocalAddress(location.href()) ? new SignInService(signInUrl, fetcher) : void 0;

package/dist/testing.js

@@ -870,9 +870,33 @@ function fakeGitHub(options) {
 			const head = url.searchParams.get("head")?.split(":")[1];
 			return json(repo.reviews.filter((review) => review.state === "open" && (!head || review.branch === head)).map((review) => ({
 				number: review.number,
-				html_url: `https://github.com/${options.repo}/pull/${review.number}`
+				title: review.title,
+				html_url: `https://github.com/${options.repo}/pull/${review.number}`,
+				head: {
+					ref: review.branch,
+					repo: { full_name: options.repo }
+				}
 			})));
 		}
+		const pull = rest.match(/^\/pulls\/(\d+)(\/requested_reviewers)?$/);
+		if (pull) {
+			const review = repo.reviews.find((found) => found.number === Number(pull[1]));
+			if (!review) return json({ message: "Not Found" }, 404);
+			if (user.bot && user.permissions?.pull_requests !== "write") return json({ message: "Resource not accessible by integration" }, 403);
+			if (pull[2] && request.method === "POST") {
+				const body = await request.json();
+				if ((body.reviewers ?? []).some((login) => login === user.login || login.startsWith("not-"))) return json({ message: "Reviews may only be requested from collaborators." }, 422);
+				review.reviewers.push(...body.reviewers ?? []);
+				return json({ number: review.number }, 201);
+			}
+			if (!pull[2] && request.method === "PATCH") {
+				if ((await request.json()).state === "closed") review.state = "closed";
+				return json({
+					number: review.number,
+					state: review.state
+				});
+			}
+		}
 		if (rest === "/pages" && request.method === "GET") {
 			const site = pages.current;
 			if (!site) return json({ message: "Not Found" }, 404);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodfellow-cms/github",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Edit a Goodfellow site stored on GitHub, straight from the browser.",
   "keywords": [
     "goodfellow",
@@ -35,7 +35,7 @@
     "dist"
   ],
   "dependencies": {
-    "@goodfellow-cms/core": "0.6.0",
+    "@goodfellow-cms/core": "0.7.0",
     "@noble/ciphers": "2.4.0",
     "@noble/hashes": "2.4.0"
   },