@workos/oagen-emitters 0.2.0 → 0.3.0

package/src/node/errors.ts

@@ -1,223 +1,31 @@
 import type { EmitterContext, GeneratedFile } from '@workos/oagen';
 import { fileName } from './naming.js';
+import { buildNodeStatusExceptions } from './sdk-errors.js';
 
+/**
+ * Static exception classes are now hand-maintained in the target SDK.
+ * Only typed exceptions derived from spec error models are generated here.
+ */
 export function generateErrors(ctx?: EmitterContext): GeneratedFile[] {
-  const files: GeneratedFile[] = [
-    {
-      path: 'src/common/exceptions/bad-request.exception.ts',
-      content: `export class BadRequestException extends Error {
-  readonly status = 400;
-  readonly name = 'BadRequestException';
-  readonly requestID: string;
-  readonly code?: string;
+  if (!ctx?.spec) return [];
 
-  constructor({
-    code,
-    message,
-    requestID,
-  }: {
-    code?: string;
-    message?: string;
-    requestID: string;
-  }) {
-    super();
-    this.message = message ?? 'Bad request';
-    this.requestID = requestID;
-    if (code) this.code = code;
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/unauthorized.exception.ts',
-      content: `export class UnauthorizedException extends Error {
-  readonly status = 401;
-  readonly name = 'UnauthorizedException';
-  readonly requestID: string;
-
-  constructor(requestID: string) {
-    super();
-    this.message = 'Unauthorized';
-    this.requestID = requestID;
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/not-found.exception.ts',
-      content: `export class NotFoundException extends Error {
-  readonly status = 404;
-  readonly name = 'NotFoundException';
-  readonly requestID: string;
-  readonly code?: string;
-
-  constructor({
-    code,
-    message,
-    path,
-    requestID,
-  }: {
-    code?: string;
-    message?: string;
-    path: string;
-    requestID: string;
-  }) {
-    super();
-    this.message =
-      message ?? \`The requested path '\${path}' could not be found.\`;
-    this.requestID = requestID;
-    if (code) this.code = code;
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/conflict.exception.ts',
-      content: `export class ConflictException extends Error {
-  readonly status = 409;
-  readonly name = 'ConflictException';
-  readonly requestID: string;
+  const typedErrors = collectTypedErrors(ctx);
+  if (typedErrors.length === 0) return [];
 
-  constructor({
-    message,
-    requestID,
-  }: {
-    message?: string;
-    error?: string;
-    requestID: string;
-  }) {
-    super();
-    this.message = message ?? 'Conflict';
-    this.requestID = requestID;
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/unprocessable-entity.exception.ts',
-      content: `export interface UnprocessableEntityError {
-  code: string;
-}
+  const files: GeneratedFile[] = [];
+  const exportLines: string[] = [];
 
-export class UnprocessableEntityException extends Error {
-  readonly status = 422;
-  readonly name = 'UnprocessableEntityException';
-  readonly requestID: string;
-  readonly code?: string;
+  for (const { modelName, statusCode, baseException } of typedErrors) {
+    const exceptionClassName = `${modelName}Exception`;
+    const filePath = `src/common/exceptions/${fileName(modelName)}.exception.ts`;
+    const baseImport = baseException
+      ? `import { ${baseException} } from './${fileName(baseException.replace(/Exception$/, '')).replace(/^/, '')}.exception';\n\n`
+      : '';
+    const baseClass = baseException ?? 'Error';
 
-  constructor({
-    code,
-    errors,
-    message,
-    requestID,
-  }: {
-    code?: string;
-    errors?: UnprocessableEntityError[];
-    message?: string;
-    requestID: string;
-  }) {
-    super();
-    this.requestID = requestID;
-    this.message = message ?? 'Unprocessable entity';
-    if (code) this.code = code;
-    if (errors) {
-      const requirement =
-        errors.length === 1 ? 'requirement' : 'requirements';
-      this.message = \`The following \${requirement} must be met:\\n\`;
-      for (const { code: errCode } of errors) {
-        this.message = this.message.concat(\`\\t\${errCode}\\n\`);
-      }
-    }
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/rate-limit-exceeded.exception.ts',
-      content: `export class RateLimitExceededException extends Error {
-  readonly status = 429;
-  readonly name = 'RateLimitExceededException';
-  readonly requestID: string;
-  readonly retryAfter?: number;
-
-  constructor(message: string, requestID: string, retryAfter?: number) {
-    super();
-    this.message = message ?? 'Too many requests';
-    this.requestID = requestID;
-    this.retryAfter = retryAfter;
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/generic-server.exception.ts',
-      content: `export class GenericServerException extends Error {
-  readonly status: number;
-  readonly name = 'GenericServerException';
-  readonly requestID: string;
-
-  constructor(status: number, message: string, requestID: string) {
-    super();
-    this.status = status;
-    this.message = message ?? 'Server error';
-    this.requestID = requestID;
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/no-api-key-provided.exception.ts',
-      content: `export class NoApiKeyProvidedException extends Error {
-  readonly name = 'NoApiKeyProvidedException';
-
-  constructor() {
-    super();
-    this.message =
-      'No API key provided. Pass it to the WorkOS constructor or set the WORKOS_API_KEY environment variable.';
-  }
-}`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-    {
-      path: 'src/common/exceptions/index.ts',
-      content: `export { BadRequestException } from './bad-request.exception';
-export { UnauthorizedException } from './unauthorized.exception';
-export { NotFoundException } from './not-found.exception';
-export { ConflictException } from './conflict.exception';
-export {
-  UnprocessableEntityException,
-  type UnprocessableEntityError,
-} from './unprocessable-entity.exception';
-export { RateLimitExceededException } from './rate-limit-exceeded.exception';
-export { GenericServerException } from './generic-server.exception';
-export { NoApiKeyProvidedException } from './no-api-key-provided.exception';`,
-      skipIfExists: true,
-      integrateTarget: false,
-    },
-  ];
-
-  // Fix 6: Generate typed exception classes from ErrorResponse.type
-  if (ctx?.spec) {
-    const typedErrors = collectTypedErrors(ctx);
-    for (const { modelName, statusCode, baseException } of typedErrors) {
-      const exceptionClassName = `${modelName}Exception`;
-      const filePath = `src/common/exceptions/${fileName(modelName)}.exception.ts`;
-      const baseImport = baseException
-        ? `import { ${baseException} } from './${fileName(baseException.replace(/Exception$/, '')).replace(/^/, '')}.exception';\n\n`
-        : '';
-      const baseClass = baseException ?? 'Error';
-
-      files.push({
-        path: filePath,
-        content: `${baseImport}export class ${exceptionClassName} extends ${baseClass} {
+    files.push({
+      path: filePath,
+      content: `${baseImport}export class ${exceptionClassName} extends ${baseClass} {
   readonly status = ${statusCode};
   readonly name = '${exceptionClassName}';
   readonly requestID: string;
@@ -230,35 +38,37 @@ export { NoApiKeyProvidedException } from './no-api-key-provided.exception';`,
     if (data) this.data = data;
   }
 }`,
-        skipIfExists: true,
-        integrateTarget: false,
-      });
+      skipIfExists: true,
+      integrateTarget: false,
+    });
 
-      // Update the barrel index to export the new exception
-      const existingIndex = files.find((f) => f.path === 'src/common/exceptions/index.ts');
-      if (existingIndex) {
-        existingIndex.content += `\nexport { ${exceptionClassName} } from './${fileName(modelName)}.exception';`;
-      }
-    }
+    exportLines.push(`export { ${exceptionClassName} } from './${fileName(modelName)}.exception';`);
+  }
+
+  // Generate a barrel for typed errors only (appended to existing exceptions/index.ts
+  // via region preservation if needed)
+  if (exportLines.length > 0) {
+    files.push({
+      path: 'src/common/exceptions/typed-errors.ts',
+      content: exportLines.join('\n'),
+      skipIfExists: true,
+      integrateTarget: false,
+    });
   }
 
   return files;
 }
 
-const STATUS_TO_BASE_EXCEPTION: Record<number, string> = {
-  400: 'BadRequestException',
-  401: 'UnauthorizedException',
-  404: 'NotFoundException',
-  409: 'ConflictException',
-  422: 'UnprocessableEntityException',
-  429: 'RateLimitExceededException',
-};
-
 function collectTypedErrors(
   ctx: EmitterContext,
 ): { modelName: string; statusCode: number; baseException: string | null }[] {
+  const statusToBase = buildNodeStatusExceptions(ctx.spec.sdk);
   const seen = new Set<string>();
-  const results: { modelName: string; statusCode: number; baseException: string | null }[] = [];
+  const results: {
+    modelName: string;
+    statusCode: number;
+    baseException: string | null;
+  }[] = [];
 
   for (const service of ctx.spec.services) {
     for (const op of service.operations) {
@@ -268,8 +78,7 @@ function collectTypedErrors(
           results.push({
             modelName: err.type.name,
             statusCode: err.statusCode,
-            baseException:
-              STATUS_TO_BASE_EXCEPTION[err.statusCode] ?? (err.statusCode >= 500 ? 'GenericServerException' : null),
+            baseException: statusToBase[err.statusCode] ?? (err.statusCode >= 500 ? 'GenericServerException' : null),
           });
         }
       }