fix: minor improvements

Author: MathurAditya724

src/__tests__/zodv3.test.ts

@@ -228,4 +228,64 @@ describe("zod v3", () => {
 
     expect(specs).toMatchSnapshot();
   });
+
+  it("operation id for path with underscore", async () => {
+    const app = new Hono().get(
+      "/api_v1/users",
+      describeRoute({
+        tags: ["test"],
+        summary: "Test route",
+        description: "This is a test route",
+      }),
+      async (c) => {
+        return c.json({ message: "Hello" });
+      },
+    );
+
+    const specs = await generateSpecs(app);
+
+    expect(specs.paths?.["/api_v1/users"]?.get?.operationId).toBe(
+      "getApiV1Users",
+    );
+  });
+
+  it("operation id for path param with dash", async () => {
+    const app = new Hono().get(
+      "/users/:user-id",
+      describeRoute({
+        tags: ["test"],
+        summary: "Test route",
+        description: "This is a test route",
+      }),
+      async (c) => {
+        return c.json({ message: "Hello" });
+      },
+    );
+
+    const specs = await generateSpecs(app);
+
+    expect(specs.paths?.["/users/{user-id}"]?.get?.operationId).toBe(
+      "getUsersByUserId",
+    );
+  });
+
+  it("operation id for multiple dashed segments", async () => {
+    const app = new Hono().get(
+      "/api-v1/user-profile",
+      describeRoute({
+        tags: ["test"],
+        summary: "Test route",
+        description: "This is a test route",
+      }),
+      async (c) => {
+        return c.json({ message: "Hello" });
+      },
+    );
+
+    const specs = await generateSpecs(app);
+
+    expect(specs.paths?.["/api-v1/user-profile"]?.get?.operationId).toBe(
+      "getApiV1UserProfile",
+    );
+  });
 });

src/utils.ts

@@ -49,12 +49,10 @@ const toOpenAPIPath = (path: string) =>
 
 const toPascalCase = (text: string) =>
   text
-    // capitalize the first letter of each word
-    .replaceAll(/(\w)(\w*)/g, (_, firstChar: string, rest: string) => {
-      return `${firstChar.toUpperCase()}${rest}`;
-    })
-    // replace all non-word characters except the first character
-    .replaceAll(/(?<!^)\W/g, "");
+    .split(/[\W_]+/)
+    .filter(Boolean)
+    .map((word) => word.charAt(0).toUpperCase() + word.slice(1))
+    .join("");
 
 const generateOperationId = (route: RouterRoute) => {
   let operationId = route.method.toLowerCase();