@workos/oagen-emitters 0.2.0 → 0.3.0

package/test/node/serializers.test.ts

@@ -1,6 +1,7 @@
 import { describe, it, expect } from 'vitest';
-import { generateSerializers } from '../../src/node/serializers.js';
+import { generateSerializers } 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('generateSerializers', () => {
           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' },
@@ -90,7 +98,13 @@ describe('generateSerializers', () => {
           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' },
@@ -151,7 +165,13 @@ describe('generateSerializers', () => {
           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' },
@@ -303,12 +323,18 @@ describe('generateSerializers', () => {
         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,
           },
         ],
@@ -375,7 +401,10 @@ describe('generateSerializers', () => {
           },
           {
             name: 'data',
-            type: { kind: 'array', items: { kind: 'model', name: 'Connection' } },
+            type: {
+              kind: 'array',
+              items: { kind: 'model', name: 'Connection' },
+            },
             required: true,
           },
           {

package/test/node/type-map.test.ts

@@ -60,6 +60,17 @@ describe('mapTypeRef', () => {
     expect(mapTypeRef(ref)).toBe('string | number');
   });
 
+  it('deduplicates union variants', () => {
+    const ref: TypeRef = {
+      kind: 'union',
+      variants: [
+        { kind: 'model', name: 'AuthenticationFactorTotp' },
+        { kind: 'model', name: 'AuthenticationFactorTotp' },
+      ],
+    };
+    expect(mapTypeRef(ref)).toBe('AuthenticationFactorTotp');
+  });
+
   it('maps map type', () => {
     const ref: TypeRef = {
       kind: 'map',

package/test/php/client.test.ts

@@ -0,0 +1,94 @@
+import { describe, it, expect } from 'vitest';
+import type { EmitterContext, ApiSpec, Service, Model } from '@workos/oagen';
+import { defaultSdkBehavior } from '@workos/oagen';
+import { generateClient } from '../../src/php/client.js';
+
+const models: Model[] = [
+  {
+    name: 'Organization',
+    fields: [{ name: 'id', type: { kind: 'primitive', type: 'string' }, required: true }],
+  },
+];
+
+const services: Service[] = [
+  {
+    name: 'Organizations',
+    operations: [
+      {
+        name: 'listOrganizations',
+        httpMethod: 'get',
+        path: '/organizations',
+        pathParams: [],
+        queryParams: [],
+        headerParams: [],
+        response: { kind: 'model', name: 'Organization' },
+        errors: [],
+        injectIdempotencyKey: false,
+      },
+    ],
+  },
+];
+
+const emptySpec: ApiSpec = {
+  name: 'Test',
+  version: '1.0.0',
+  baseUrl: 'https://api.example.com',
+  services,
+  models,
+  enums: [],
+  sdk: defaultSdkBehavior(),
+};
+
+const ctx: EmitterContext = {
+  namespace: 'workos',
+  namespacePascal: 'WorkOS',
+  spec: emptySpec,
+};
+
+describe('generateClient', () => {
+  it('only generates the main client file', () => {
+    const result = generateClient(emptySpec, ctx);
+
+    expect(result).toHaveLength(1);
+    expect(result[0].path).toBe('lib/WorkOS.php');
+  });
+
+  it('generates main client class with namespace', () => {
+    const result = generateClient(emptySpec, ctx);
+
+    expect(result[0].content).toContain('class WorkOS');
+    expect(result[0].content).toContain('namespace WorkOS;');
+  });
+
+  it('generates resource accessor methods', () => {
+    const result = generateClient(emptySpec, ctx);
+
+    expect(result[0].content).toContain('public function organizations(): Organizations');
+  });
+
+  it('includes constructor with config options', () => {
+    const result = generateClient(emptySpec, ctx);
+
+    expect(result[0].content).toContain('?string $apiKey = null');
+    expect(result[0].content).toContain('?string $clientId = null');
+    expect(result[0].content).toContain("string $baseUrl = 'https://api.example.com'");
+    expect(result[0].content).toContain('int $timeout = 60');
+    expect(result[0].content).toContain('int $maxRetries = 3');
+    expect(result[0].content).toContain(
+      'new HttpClient($apiKey, $clientId, $baseUrl, $timeout, $maxRetries, $handler)',
+    );
+    expect(result[0].content).not.toContain('self::$apiKey = $apiKey;');
+    expect(result[0].content).not.toContain('self::$clientId = $clientId;');
+  });
+
+  it('includes non-spec service accessors', () => {
+    const result = generateClient(emptySpec, ctx);
+
+    expect(result[0].content).toContain('public function passwordless(): Passwordless');
+    expect(result[0].content).toContain('public function vault(): Vault');
+    expect(result[0].content).toContain('public function webhookVerification(): WebhookVerification');
+    expect(result[0].content).toContain('public function actions(): Actions');
+    expect(result[0].content).toContain('public function sessionManager(): SessionManager');
+    expect(result[0].content).toContain('public function pkce(): PKCEHelper');
+  });
+});

package/test/php/enums.test.ts

@@ -0,0 +1,173 @@
+import { describe, it, expect } from 'vitest';
+import type { EmitterContext, ApiSpec, Enum } from '@workos/oagen';
+import { defaultSdkBehavior } from '@workos/oagen';
+import { generateEnums } from '../../src/php/enums.js';
+import { initializeEnumDedup } from '../../src/php/naming.js';
+
+const emptySpec: ApiSpec = {
+  name: 'Test',
+  version: '1.0.0',
+  baseUrl: '',
+  services: [],
+  models: [],
+  enums: [],
+  sdk: defaultSdkBehavior(),
+};
+
+const ctx: EmitterContext = {
+  namespace: 'workos',
+  namespacePascal: 'WorkOS',
+  spec: emptySpec,
+};
+
+describe('generateEnums', () => {
+  it('returns empty array for no enums', () => {
+    expect(generateEnums([], ctx)).toEqual([]);
+  });
+
+  it('generates a string-backed enum', () => {
+    const enums: Enum[] = [
+      {
+        name: 'OrganizationStatus',
+        values: [
+          { name: 'ACTIVE', value: 'active' },
+          { name: 'INACTIVE', value: 'inactive' },
+        ],
+      },
+    ];
+
+    const result = generateEnums(enums, ctx);
+
+    expect(result).toHaveLength(1);
+    expect(result[0].path).toBe('lib/Resource/OrganizationStatus.php');
+    expect(result[0].content).toContain('enum OrganizationStatus: string');
+    expect(result[0].content).toContain("case Active = 'active';");
+    expect(result[0].content).toContain("case Inactive = 'inactive';");
+  });
+
+  it('generates an int-backed enum', () => {
+    const enums: Enum[] = [
+      {
+        name: 'Priority',
+        values: [
+          { name: 'LOW', value: 1 },
+          { name: 'MEDIUM', value: 2 },
+          { name: 'HIGH', value: 3 },
+        ],
+      },
+    ];
+
+    const result = generateEnums(enums, ctx);
+
+    expect(result[0].content).toContain('enum Priority: int');
+    expect(result[0].content).toContain('case Low = 1;');
+    expect(result[0].content).toContain('case Medium = 2;');
+    expect(result[0].content).toContain('case High = 3;');
+  });
+
+  it('generates correct namespace', () => {
+    const enums: Enum[] = [
+      {
+        name: 'Status',
+        values: [{ name: 'ACTIVE', value: 'active' }],
+      },
+    ];
+
+    const result = generateEnums(enums, ctx);
+
+    expect(result[0].content).toContain('namespace WorkOS\\Resource;');
+  });
+
+  it('collapses duplicate enums with identical values into one file', () => {
+    const enums: Enum[] = [
+      {
+        name: 'Order',
+        values: [
+          { name: 'ASC', value: 'asc' },
+          { name: 'DESC', value: 'desc' },
+        ],
+      },
+      {
+        name: 'ConnectionOrder',
+        values: [
+          { name: 'ASC', value: 'asc' },
+          { name: 'DESC', value: 'desc' },
+        ],
+      },
+      {
+        name: 'ApiKeyOrder',
+        values: [
+          { name: 'ASC', value: 'asc' },
+          { name: 'DESC', value: 'desc' },
+        ],
+      },
+    ];
+
+    // Initialize dedup before generating
+    initializeEnumDedup(enums);
+    const result = generateEnums(enums, ctx);
+
+    // Should produce only one file (the shortest name: Order)
+    expect(result).toHaveLength(1);
+    expect(result[0].path).toBe('lib/Resource/Order.php');
+  });
+
+  it('adds PHPDoc @deprecated for deprecated enum values', () => {
+    const enums: Enum[] = [
+      {
+        name: 'ConnectionType',
+        values: [
+          { name: 'SAML', value: 'saml' },
+          { name: 'OAUTH', value: 'oauth', deprecated: true },
+        ],
+      },
+    ];
+
+    const result = generateEnums(enums, ctx);
+
+    expect(result).toHaveLength(1);
+    // The non-deprecated value should not have a PHPDoc
+    expect(result[0].content).not.toContain('/** @deprecated */\n    case Saml');
+    // The deprecated value should have a PHPDoc
+    expect(result[0].content).toContain('/** @deprecated */');
+    // Verify the deprecated case follows the PHPDoc
+    const lines = result[0].content.split('\n');
+    const deprecatedIdx = lines.findIndex((l: string) => l.includes('@deprecated'));
+    expect(deprecatedIdx).toBeGreaterThan(-1);
+    expect(lines[deprecatedIdx + 1]).toContain("= 'oauth';");
+  });
+
+  it('adds PHPDoc with description and @deprecated for enum values', () => {
+    const enums: Enum[] = [
+      {
+        name: 'ConnectionType',
+        values: [
+          { name: 'SAML', value: 'saml' },
+          { name: 'OAUTH', value: 'oauth', description: 'Use OIDC instead', deprecated: true },
+        ],
+      },
+    ];
+
+    const result = generateEnums(enums, ctx);
+
+    expect(result[0].content).toContain('Use OIDC instead');
+    expect(result[0].content).toContain('@deprecated');
+  });
+
+  it('deduplicates case names', () => {
+    const enums: Enum[] = [
+      {
+        name: 'DupEnum',
+        values: [
+          { name: 'FOO_BAR', value: 'foo_bar' },
+          { name: 'FOO__BAR', value: 'foo__bar' },
+        ],
+      },
+    ];
+
+    const result = generateEnums(enums, ctx);
+
+    expect(result[0].content).toContain('case FooBar =');
+    expect(result[0].content).toContain('case FooBar2 =');
+  });
+});

package/test/php/errors.test.ts

@@ -0,0 +1,9 @@
+import { describe, it, expect } from 'vitest';
+import { generateErrors } from '../../src/php/errors.js';
+
+describe('generateErrors', () => {
+  it('returns empty array (errors are now hand-maintained)', () => {
+    const result = generateErrors();
+    expect(result).toEqual([]);
+  });
+});