@workos/oagen-emitters 0.2.0 → 0.3.0

package/test/node/enums.test.ts

@@ -1,6 +1,7 @@
 import { describe, it, expect } from 'vitest';
 import { generateEnums } from '../../src/node/enums.js';
 import type { EmitterContext, ApiSpec, Enum, Service } from '@workos/oagen';
+import { defaultSdkBehavior } from '@workos/oagen';
 
 const emptySpec: ApiSpec = {
   name: 'Test',
@@ -9,6 +10,7 @@ const emptySpec: ApiSpec = {
   services: [],
   models: [],
   enums: [],
+  sdk: defaultSdkBehavior(),
 };
 
 const ctx: EmitterContext = {
@@ -30,7 +32,13 @@ describe('generateEnums', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: {
@@ -76,7 +84,13 @@ describe('generateEnums', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'enum', name: 'OrgStatus' },
@@ -103,13 +117,71 @@ describe('generateEnums', () => {
     expect(files[0].path).toBe('src/organizations/interfaces/org-status.interface.ts');
   });
 
+  it('derives PascalCase member names when merging new enum values into baseline', () => {
+    const enums: Enum[] = [
+      {
+        name: 'OrganizationDomainState',
+        values: [
+          { name: 'FAILED', value: 'failed' },
+          { name: 'PENDING', value: 'pending' },
+          { name: 'VERIFIED', value: 'verified' },
+          { name: 'LEGACY_VERIFIED', value: 'legacy_verified' },
+          { name: 'UNVERIFIED', value: 'unverified' },
+        ],
+      },
+    ];
+
+    const testCtx: EmitterContext = {
+      ...ctx,
+      apiSurface: {
+        language: 'node',
+        extractedFrom: 'test',
+        extractedAt: '2024-01-01',
+        classes: {},
+        interfaces: {},
+        typeAliases: {},
+        enums: {
+          OrganizationDomainState: {
+            name: 'OrganizationDomainState',
+            members: {
+              Failed: 'failed',
+              Pending: 'pending',
+              Verified: 'verified',
+            },
+          },
+        },
+        exports: {},
+      },
+    };
+
+    const files = generateEnums(enums, testCtx);
+    const content = files[0].content;
+
+    // Existing members should be preserved as-is
+    expect(content).toContain("Failed = 'failed',");
+    expect(content).toContain("Pending = 'pending',");
+    expect(content).toContain("Verified = 'verified',");
+
+    // New members should be PascalCase, not lowercased
+    expect(content).toContain("LegacyVerified = 'legacy_verified',");
+    expect(content).toContain("Unverified = 'unverified',");
+
+    // Should NOT produce lowercased member names
+    expect(content).not.toContain('legacyverified');
+  });
+
   it('renders @deprecated on enum values', () => {
     const enums: Enum[] = [
       {
         name: 'Status',
         values: [
           { name: 'ACTIVE', value: 'active' },
-          { name: 'LEGACY', value: 'legacy', description: 'No longer supported.', deprecated: true },
+          {
+            name: 'LEGACY',
+            value: 'legacy',
+            description: 'No longer supported.',
+            deprecated: true,
+          },
           { name: 'OLD', value: 'old', deprecated: true },
         ],
       },

package/test/node/errors.test.ts

@@ -2,47 +2,8 @@ import { describe, it, expect } from 'vitest';
 import { generateErrors } from '../../src/node/errors.js';
 
 describe('generateErrors', () => {
-  it('generates all exception classes', () => {
+  it('returns empty array without context (static exceptions now hand-maintained)', () => {
     const files = generateErrors();
-
-    const names = files.map((f) => f.path);
-    expect(names).toContain('src/common/exceptions/bad-request.exception.ts');
-    expect(names).toContain('src/common/exceptions/unauthorized.exception.ts');
-    expect(names).toContain('src/common/exceptions/not-found.exception.ts');
-    expect(names).toContain('src/common/exceptions/conflict.exception.ts');
-    expect(names).toContain('src/common/exceptions/unprocessable-entity.exception.ts');
-    expect(names).toContain('src/common/exceptions/rate-limit-exceeded.exception.ts');
-    expect(names).toContain('src/common/exceptions/generic-server.exception.ts');
-    expect(names).toContain('src/common/exceptions/no-api-key-provided.exception.ts');
-    expect(names).toContain('src/common/exceptions/index.ts');
-  });
-
-  it('generates NotFoundException with correct status', () => {
-    const files = generateErrors();
-    const notFoundFile = files.find((f) => f.path.includes('not-found.exception.ts'))!;
-
-    expect(notFoundFile.content).toContain('export class NotFoundException extends Error');
-    expect(notFoundFile.content).toContain('readonly status = 404;');
-    expect(notFoundFile.content).toContain('requestID: string');
-  });
-
-  it('generates RateLimitExceededException with retryAfter', () => {
-    const files = generateErrors();
-    const rateLimitFile = files.find((f) => f.path.includes('rate-limit-exceeded.exception.ts'))!;
-
-    expect(rateLimitFile.content).toContain('export class RateLimitExceededException extends Error');
-    expect(rateLimitFile.content).toContain('readonly status = 429;');
-    expect(rateLimitFile.content).toContain('retryAfter?: number');
-  });
-
-  it('generates exception barrel with all exports', () => {
-    const files = generateErrors();
-    const barrel = files.find((f) => f.path === 'src/common/exceptions/index.ts')!;
-
-    expect(barrel.content).toContain('export { BadRequestException }');
-    expect(barrel.content).toContain('export { UnauthorizedException }');
-    expect(barrel.content).toContain('export { NotFoundException }');
-    expect(barrel.content).toContain('export { RateLimitExceededException }');
-    expect(barrel.content).toContain('export { NoApiKeyProvidedException }');
+    expect(files).toEqual([]);
   });
 });

package/test/node/models.test.ts

@@ -1,6 +1,7 @@
 import { describe, it, expect } from 'vitest';
 import { generateModels } from '../../src/node/models.js';
 import type { EmitterContext, ApiSpec, Model, Service } from '@workos/oagen';
+import { defaultSdkBehavior } from '@workos/oagen';
 
 const emptySpec: ApiSpec = {
   name: 'Test',
@@ -9,6 +10,7 @@ const emptySpec: ApiSpec = {
   services: [],
   models: [],
   enums: [],
+  sdk: defaultSdkBehavior(),
 };
 
 const ctx: EmitterContext = {
@@ -30,7 +32,13 @@ describe('generateModels', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'model', name: 'Organization' },
@@ -101,7 +109,13 @@ describe('generateModels', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'model', name: 'Organization' },
@@ -247,7 +261,13 @@ describe('generateModels', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'model', name: 'Organization' },
@@ -306,7 +326,13 @@ describe('generateModels', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'model', name: 'Organization' },
@@ -386,7 +412,13 @@ describe('generateModels', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'model', name: 'Organization' },
@@ -470,12 +502,18 @@ describe('generateModels', () => {
         fields: [
           {
             name: 'before',
-            type: { kind: 'nullable', inner: { kind: 'primitive', type: 'string' } },
+            type: {
+              kind: 'nullable',
+              inner: { kind: 'primitive', type: 'string' },
+            },
             required: false,
           },
           {
             name: 'after',
-            type: { kind: 'nullable', inner: { kind: 'primitive', type: 'string' } },
+            type: {
+              kind: 'nullable',
+              inner: { kind: 'primitive', type: 'string' },
+            },
             required: false,
           },
         ],
@@ -542,7 +580,10 @@ describe('generateModels', () => {
           },
           {
             name: 'data',
-            type: { kind: 'array', items: { kind: 'model', name: 'Connection' } },
+            type: {
+              kind: 'array',
+              items: { kind: 'model', name: 'Connection' },
+            },
             required: true,
           },
           {
@@ -557,12 +598,18 @@ describe('generateModels', () => {
         fields: [
           {
             name: 'before',
-            type: { kind: 'nullable', inner: { kind: 'primitive', type: 'string' } },
+            type: {
+              kind: 'nullable',
+              inner: { kind: 'primitive', type: 'string' },
+            },
             required: false,
           },
           {
             name: 'after',
-            type: { kind: 'nullable', inner: { kind: 'primitive', type: 'string' } },
+            type: {
+              kind: 'nullable',
+              inner: { kind: 'primitive', type: 'string' },
+            },
             required: false,
           },
         ],
@@ -607,7 +654,13 @@ describe('generateModels', () => {
           name: 'getOrganization',
           httpMethod: 'get',
           path: '/organizations/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'model', name: 'Pagination' },
@@ -623,12 +676,18 @@ describe('generateModels', () => {
         fields: [
           {
             name: 'before',
-            type: { kind: 'nullable', inner: { kind: 'primitive', type: 'string' } },
+            type: {
+              kind: 'nullable',
+              inner: { kind: 'primitive', type: 'string' },
+            },
             required: false,
           },
           {
             name: 'after',
-            type: { kind: 'nullable', inner: { kind: 'primitive', type: 'string' } },
+            type: {
+              kind: 'nullable',
+              inner: { kind: 'primitive', type: 'string' },
+            },
             required: false,
           },
           {
@@ -661,7 +720,13 @@ describe('model deduplication', () => {
           name: 'getRole',
           httpMethod: 'get',
           path: '/roles/{id}',
-          pathParams: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+          pathParams: [
+            {
+              name: 'id',
+              type: { kind: 'primitive', type: 'string' },
+              required: true,
+            },
+          ],
           queryParams: [],
           headerParams: [],
           response: { kind: 'model', name: 'EnvironmentRole' },
@@ -675,17 +740,41 @@ describe('model deduplication', () => {
       {
         name: 'EnvironmentRole',
         fields: [
-          { name: 'id', type: { kind: 'primitive', type: 'string' }, required: true },
-          { name: 'name', type: { kind: 'primitive', type: 'string' }, required: true },
-          { name: 'type', type: { kind: 'literal', value: 'environment_role' }, required: true },
+          {
+            name: 'id',
+            type: { kind: 'primitive', type: 'string' },
+            required: true,
+          },
+          {
+            name: 'name',
+            type: { kind: 'primitive', type: 'string' },
+            required: true,
+          },
+          {
+            name: 'type',
+            type: { kind: 'literal', value: 'environment_role' },
+            required: true,
+          },
         ],
       },
       {
         name: 'OrganizationRole',
         fields: [
-          { name: 'id', type: { kind: 'primitive', type: 'string' }, required: true },
-          { name: 'name', type: { kind: 'primitive', type: 'string' }, required: true },
-          { name: 'type', type: { kind: 'literal', value: 'environment_role' }, required: true },
+          {
+            name: 'id',
+            type: { kind: 'primitive', type: 'string' },
+            required: true,
+          },
+          {
+            name: 'name',
+            type: { kind: 'primitive', type: 'string' },
+            required: true,
+          },
+          {
+            name: 'type',
+            type: { kind: 'literal', value: 'environment_role' },
+            required: true,
+          },
         ],
       },
     ];

package/test/node/naming.test.ts

@@ -10,10 +10,31 @@ import {
   servicePropertyName,
   resolveServiceName,
   buildServiceNameMap,
+  stripNoiseSuffixes,
 } from '../../src/node/naming.js';
 import type { EmitterContext, ApiSpec, Service } from '@workos/oagen';
+import { defaultSdkBehavior } from '@workos/oagen';
 
 describe('naming', () => {
+  describe('stripNoiseSuffixes', () => {
+    it('strips trailing Dto', () => {
+      expect(stripNoiseSuffixes('OrganizationDto')).toBe('Organization');
+      expect(stripNoiseSuffixes('UpdateOrganizationDto')).toBe('UpdateOrganization');
+    });
+
+    it('is case-insensitive for the Dto suffix', () => {
+      expect(stripNoiseSuffixes('OrganizationDTO')).toBe('Organization');
+    });
+
+    it('does not strip Dto from the middle of a name', () => {
+      expect(stripNoiseSuffixes('DtoFactory')).toBe('DtoFactory');
+    });
+
+    it('leaves names without Dto unchanged', () => {
+      expect(stripNoiseSuffixes('Organization')).toBe('Organization');
+    });
+  });
+
   describe('className', () => {
     it('converts to PascalCase', () => {
       expect(className('organizations')).toBe('Organizations');
@@ -88,6 +109,7 @@ describe('naming', () => {
       services: [],
       models: [],
       enums: [],
+      sdk: defaultSdkBehavior(),
     };
 
     it('returns overlay class name when available', () => {
@@ -116,7 +138,12 @@ describe('naming', () => {
           methodByOperation: new Map([
             [
               'POST /auth/factors/enroll',
-              { className: 'Mfa', methodName: 'enrollFactor', params: [], returnType: 'void' },
+              {
+                className: 'Mfa',
+                methodName: 'enrollFactor',
+                params: [],
+                returnType: 'void',
+              },
             ],
           ]),
           httpKeyByMethod: new Map(),
@@ -133,7 +160,7 @@ describe('naming', () => {
 
     it('falls back to PascalCase of service.name', () => {
       const service: Service = {
-        name: 'MultiFactorAuth',
+        name: 'SomeNewService',
         operations: [],
       };
 
@@ -143,7 +170,7 @@ describe('naming', () => {
         spec: emptySpec,
       };
 
-      expect(resolveServiceName(service, ctx)).toBe('MultiFactorAuth');
+      expect(resolveServiceName(service, ctx)).toBe('SomeNewService');
     });
   });
 
@@ -155,6 +182,7 @@ describe('naming', () => {
       services: [],
       models: [],
       enums: [],
+      sdk: defaultSdkBehavior(),
     };
 
     it('maps IR names to resolved names', () => {
@@ -189,7 +217,12 @@ describe('naming', () => {
           methodByOperation: new Map([
             [
               'POST /auth/factors/enroll',
-              { className: 'Mfa', methodName: 'enrollFactor', params: [], returnType: 'void' },
+              {
+                className: 'Mfa',
+                methodName: 'enrollFactor',
+                params: [],
+                returnType: 'void',
+              },
             ],
           ]),
           httpKeyByMethod: new Map(),