refactor: use chained route (#17)

Author: Yusuke4869

src/main.ts

@@ -4,10 +4,9 @@ import { logger } from "hono/logger";
 import app from "./route/index.ts";
 import api from "./route/api.ts";
 
-const server = new Hono();
-
-server.use(logger());
-server.route("/", app);
-server.route("/api", api);
+const server = new Hono()
+  .use(logger())
+  .route("/", app)
+  .route("/api", api);
 
 Deno.serve(server.fetch);

src/route/api.ts

@@ -14,59 +14,58 @@ import { Logger } from "../service/logger.ts";
 
 const apiKeys = Deno.env.get("API_KEYS")?.split(",").filter(Boolean) ?? [];
 
-const api = new Hono();
 const logger = new Logger();
 
-api.use(async (c, next) => {
-  const apiKey = c.req.header("X-API-Key");
+const api = new Hono()
+  .use(async (c, next) => {
+    const apiKey = c.req.header("X-API-Key");
 
-  if (!apiKey) {
-    await logger.apiAccess(c, "Unauthorized", true, apiKey);
-    throw new HTTPException(401, { message: "Unauthorized" });
-  }
+    if (!apiKey) {
+      await logger.apiAccess(c, "Unauthorized", true, apiKey);
+      throw new HTTPException(401, { message: "Unauthorized" });
+    }
 
-  if (!apiKeys.includes(apiKey)) {
-    await logger.apiAccess(c, "Forbidden", true, apiKey);
-    throw new HTTPException(403, { message: "Forbidden" });
-  }
+    if (!apiKeys.includes(apiKey)) {
+      await logger.apiAccess(c, "Forbidden", true, apiKey);
+      throw new HTTPException(403, { message: "Forbidden" });
+    }
 
-  await logger.apiAccess(c, "Accessed", false, apiKey);
-  await next();
-});
-
-api.get("/all", getAllItemsController);
-api.get("/items/:param", getItemController);
-api.put(
-  "/items/:param",
-  zValidator(
-    "json",
-    z.object({
-      description: z.string().optional(),
-      url: z.string().url(),
-      count: z.number().optional(),
-    }),
-  ),
-  (c) => {
-    const { description, url, count } = c.req.valid("json");
-    return putItemController(c, description, url, count);
-  },
-);
-api.patch(
-  "/items/:param",
-  zValidator(
-    "json",
-    z.object({
-      description: z.string().optional(),
-      url: z.string().url().optional(),
-      count: z.number().optional(),
-      unavailable: z.boolean().optional(),
-    }),
-  ),
-  (c) => {
-    const { description, url, count, unavailable } = c.req.valid("json");
-    return patchItemController(c, description, url, count, unavailable);
-  },
-);
-api.delete("/items/:param", deleteItemController);
+    await logger.apiAccess(c, "Accessed", false, apiKey);
+    await next();
+  })
+  .get("/all", getAllItemsController)
+  .get("/items/:param", getItemController)
+  .put(
+    "/items/:param",
+    zValidator(
+      "json",
+      z.object({
+        description: z.string().optional(),
+        url: z.string().url(),
+        count: z.number().optional(),
+      }),
+    ),
+    (c) => {
+      const { description, url, count } = c.req.valid("json");
+      return putItemController(c, description, url, count);
+    },
+  )
+  .patch(
+    "/items/:param",
+    zValidator(
+      "json",
+      z.object({
+        description: z.string().optional(),
+        url: z.string().url().optional(),
+        count: z.number().optional(),
+        unavailable: z.boolean().optional(),
+      }),
+    ),
+    (c) => {
+      const { description, url, count, unavailable } = c.req.valid("json");
+      return patchItemController(c, description, url, count, unavailable);
+    },
+  )
+  .delete("/items/:param", deleteItemController);
 
 export default api;

src/route/index.ts

@@ -4,24 +4,23 @@ import { serveStatic } from "hono/deno";
 import { Logger } from "../service/logger.ts";
 import { home, redirect } from "../controller/index.ts";
 
-const app = new Hono();
 const logger = new Logger();
 
-app.use(async (c, next) => {
-  await next();
+const app = new Hono()
+  .use(async (c, next) => {
+    await next();
 
-  if (!c.req.path.startsWith("/api")) {
-    // 200-299 or 300-399
-    if (c.res.ok || (c.res.status >= 300 && c.res.status < 400)) {
-      await logger.access(c, false);
-    } else {
-      await logger.access(c, true);
+    if (!c.req.path.startsWith("/api")) {
+      // 200-299 or 300-399
+      if (c.res.ok || (c.res.status >= 300 && c.res.status < 400)) {
+        await logger.access(c, false);
+      } else {
+        await logger.access(c, true);
+      }
     }
-  }
-});
-app.use("/robots.txt", serveStatic({ path: "public/robots.txt" }));
-
-app.get("/", home);
-app.get("/:param", redirect);
+  })
+  .use("/robots.txt", serveStatic({ path: "public/robots.txt" }))
+  .get("/", home)
+  .get("/:param", redirect);
 
 export default app;