Update octokit and escapegoat

domenic · domenic · commit eaed63e18c1e · 2025-06-02T14:37:55.000+09:00
Now that we are on standard JS modules, this is possible.
diff --git a/lib/server-infra/error-handler.mjs b/lib/server-infra/error-handler.mjs
@@ -42,5 +42,5 @@ export default async function errorHandler(ctx, next) {
 // But if one of these reaches us, it's an internal server error, and should not be communicated as
 // whatever status code the GitHub API returned.
 function isOctokitError(err) {
-  return err.documentation_url && err.documentation_url.startsWith("https://developer.github.com/");
+  return err.response?.documentation_url?.startsWith("https://docs.github.com/");
 }
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -16,8 +16,8 @@
   },
   "dependencies": {
     "@koa/router": "^13.1.0",
-    "@octokit/rest": "^20.1.2",
-    "escape-goat": "^3.0.0",
+    "@octokit/rest": "^22.0.0",
+    "escape-goat": "^4.0.0",
     "github-username-regex": "^1.0.0",
     "handlebars": "^4.7.8",
     "http-errors": "^2.0.0",

Original file line number	Diff line number	Diff line change
`@@ -42,5 +42,5 @@ export default async function errorHandler(ctx, next) {`
`42`	`42`	`// But if one of these reaches us, it's an internal server error, and should not be communicated as`
`43`	`43`	`// whatever status code the GitHub API returned.`
`44`	`44`	`function isOctokitError(err) {`
`45`		`- return err.documentation_url && err.documentation_url.startsWith("https://developer.github.com/");`
	`45`	`+ return err.response?.documentation_url?.startsWith("https://docs.github.com/");`
`46`	`46`	`}`