fix: move env to types, update packages, remove unused functions

package.json

@@ -35,7 +35,7 @@
     "smee-client": "^2.0.0"
   },
   "devDependencies": {
-    "@cloudflare/workers-types": "^4.20231121.0",
+    "@cloudflare/workers-types": "^4.20240117.0",
     "@commitlint/cli": "^18.4.3",
     "@commitlint/config-conventional": "^18.4.3",
     "@types/node": "^20.11.1",
@@ -53,7 +53,7 @@
     "prettier": "^3.1.0",
     "tsx": "^4.6.2",
     "typescript": "^5.0.4",
-    "wrangler": "^3.22.3"
+    "wrangler": "^3.23.0"
   },
   "lint-staged": {
     "*.ts": [

src/types/env.ts

@@ -0,0 +1,4 @@
+import { Type as T, type Static } from "@sinclair/typebox";
+
+export const envSchema = T.Object({ WEBHOOK_SECRET: T.String() });
+export type Env = Static<typeof envSchema>;

src/worker.ts

@@ -1,11 +1,9 @@
 import { EmitterWebhookEventName as WebhookEventName, emitterEventNames } from "@octokit/webhooks";
-import { Type as T, type Static } from "@sinclair/typebox";
 import { Value } from "@sinclair/typebox/value";
-
 import { EventHandler } from "./event-handler";
 import { bindHandlers } from "./handlers";
-const envSchema = T.Object({ WEBHOOK_SECRET: T.String() });
-type Env = Static<typeof envSchema>;
+import { envSchema, Env } from "./types/env";
+
 export default {
   async fetch(request: Request, env: Env): Promise<Response> {
     if (!Value.Check(envSchema, env)) {
@@ -16,6 +14,7 @@ export default {
         headers: { "content-type": "application/json" },
       });
     }
+
     let pathname;
     try {
       pathname = new URL(request.url, "http://localhost").pathname;
@@ -25,39 +24,45 @@ export default {
         headers: { "content-type": "application/json" },
       });
     }
+
     if (pathname !== "/events" || request.method !== "POST") {
       return new Response(JSON.stringify({ error: `Unknown route: ${request.method} ${request.url}` }), {
         status: 400,
         headers: { "content-type": "application/json" },
       });
     }
+
     if (!request.headers.get("content-type") || !request.headers.get("content-type")?.startsWith("application/json")) {
       return new Response(JSON.stringify({ error: `Unsupported "Content-Type" header value. Must be "application/json"` }), {
         status: 415,
         headers: { "content-type": "application/json" },
       });
     }
+
     const eventName = request.headers.get("x-github-event");
     if (!eventName) {
       return new Response(JSON.stringify({ error: `Missing "x-github-event" header` }), {
         status: 400,
         headers: { "content-type": "application/json" },
       });
     }
+
     const signatureSHA256 = request.headers.get("x-hub-signature-256");
     if (!signatureSHA256) {
       return new Response(JSON.stringify({ error: `Missing "x-hub-signature-256" header` }), {
         status: 400,
         headers: { "content-type": "application/json" },
       });
     }
+
     const id = request.headers.get("x-github-delivery");
     if (!id) {
       return new Response(JSON.stringify({ error: `Missing "x-github-delivery" header` }), {
         status: 400,
         headers: { "content-type": "application/json" },
       });
     }
+
     if (!emitterEventNames.includes(eventName as WebhookEventName)) {
       return new Response(JSON.stringify({ error: `Unsupported "x-github-event" header value: ${eventName}` }), {
         status: 400,