Fix: Slugify operation id (#307)

Author: jlalmes

src/generator/paths.ts

@@ -15,7 +15,7 @@ export const getOpenApiPathsObject = (
   const procedures = appRouter._def.procedures as OpenApiProcedureRecord;
 
   forEachOpenApiProcedure(procedures, ({ path: procedurePath, type, procedure, openapi }) => {
-    const operationId = `${type}.${procedurePath}`;
+    const procedureName = `${type}.${procedurePath}`;
 
     try {
       if (type === 'subscription') {
@@ -59,7 +59,7 @@ export const getOpenApiPathsObject = (
       pathsObject[path] = {
         ...pathsObject[path],
         [httpMethod]: {
-          operationId,
+          operationId: procedurePath.replace(/\./g, '-'),
           summary,
           description,
           tags: tags,
@@ -85,7 +85,7 @@ export const getOpenApiPathsObject = (
       };
     } catch (error: any) {
       // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
-      error.message = `[${operationId}] - ${error.message}`;
+      error.message = `[${procedureName}] - ${error.message}`;
       throw error;
     }
   });

test/generator.test.ts

@@ -557,7 +557,7 @@ describe('generator', () => {
           "/users": Object {
             "get": Object {
               "description": undefined,
-              "operationId": "query.readUsers",
+              "operationId": "readUsers",
               "parameters": Array [],
               "requestBody": undefined,
               "responses": Object {
@@ -597,7 +597,7 @@ describe('generator', () => {
             },
             "post": Object {
               "description": undefined,
-              "operationId": "mutation.createUser",
+              "operationId": "createUser",
               "parameters": Array [],
               "requestBody": Object {
                 "content": Object {
@@ -654,7 +654,7 @@ describe('generator', () => {
           "/users/{id}": Object {
             "delete": Object {
               "description": undefined,
-              "operationId": "mutation.deleteUser",
+              "operationId": "deleteUser",
               "parameters": Array [
                 Object {
                   "description": undefined,
@@ -686,7 +686,7 @@ describe('generator', () => {
             },
             "get": Object {
               "description": undefined,
-              "operationId": "query.readUser",
+              "operationId": "readUser",
               "parameters": Array [
                 Object {
                   "description": undefined,
@@ -733,7 +733,7 @@ describe('generator', () => {
             },
             "patch": Object {
               "description": undefined,
-              "operationId": "mutation.updateUser",
+              "operationId": "updateUser",
               "parameters": Array [
                 Object {
                   "description": undefined,
@@ -905,7 +905,7 @@ describe('generator', () => {
     expect(openApiDocument.paths['/user']!.post!).toMatchInlineSnapshot(`
       Object {
         "description": undefined,
-        "operationId": "mutation.createUser",
+        "operationId": "createUser",
         "parameters": Array [],
         "requestBody": Object {
           "content": Object {
@@ -974,7 +974,7 @@ describe('generator', () => {
     expect(openApiDocument.paths['/user']!.get!).toMatchInlineSnapshot(`
       Object {
         "description": undefined,
-        "operationId": "query.getUser",
+        "operationId": "getUser",
         "parameters": Array [
           Object {
             "description": "User ID",
@@ -2296,7 +2296,7 @@ describe('generator', () => {
         "/procedure": Object {
           "get": Object {
             "description": undefined,
-            "operationId": "query.procedure",
+            "operationId": "procedure",
             "parameters": Array [
               Object {
                 "description": undefined,
@@ -2341,7 +2341,7 @@ describe('generator', () => {
         "/router/procedure": Object {
           "get": Object {
             "description": undefined,
-            "operationId": "query.router.procedure",
+            "operationId": "router-procedure",
             "parameters": Array [
               Object {
                 "description": undefined,
@@ -2386,7 +2386,7 @@ describe('generator', () => {
         "/router/router/procedure": Object {
           "get": Object {
             "description": undefined,
-            "operationId": "query.router.router.procedure",
+            "operationId": "router-router-procedure",
             "parameters": Array [
               Object {
                 "description": undefined,