@workos/oagen 0.17.0 → 0.17.2

package/dist/cli/index.mjs

@@ -534,6 +534,14 @@ function buildInlineModel(name, schema) {
 * to apply the same name transform to $ref model/enum references.
 */
 let activeSchemaNameTransform = null;
+/**
+* Module-level set of resolved schema names whose components/schemas entry is
+* an enum (i.e. has `enum:`). Populated by extractSchemas() so schemaToTypeRef()
+* can classify a `$ref` to a top-level enum schema as `kind: 'enum'` instead of
+* `kind: 'model'`. Without this, every `$ref` was naively returned as a model
+* ref, even when the target was a shared enum like `PaginationOrder`.
+*/
+let activeEnumSchemaNames = /* @__PURE__ */ new Set();
 /** Apply cleanSchemaName + the active transform (if any) to a raw schema name. */
 function resolveSchemaName(rawName) {
 	let name = cleanSchemaName(toPascalCase(rawName));
@@ -559,6 +567,8 @@ function extractSchemas(schemas, options) {
 		for (const [, originals] of transformedToOriginals) if (originals.length > 1) for (const n of originals) unsafeToTransform.add(n);
 		activeSchemaNameTransform = (name) => unsafeToTransform.has(name) ? name : transform(name);
 	} else activeSchemaNameTransform = null;
+	activeEnumSchemaNames = /* @__PURE__ */ new Set();
+	for (const [name, schema] of Object.entries(schemas)) if (schema.enum) activeEnumSchemaNames.add(resolveSchemaName(name));
 	const modelsByCleanName = /* @__PURE__ */ new Map();
 	for (const [name, schema] of Object.entries(schemas)) {
 		const pascalName = resolveSchemaName(name);
@@ -593,6 +603,7 @@ function extractSchemas(schemas, options) {
 /** Clear the module-level schemaNameTransform. Called by parseSpec() after all extraction phases. */
 function clearSchemaNameTransform() {
 	activeSchemaNameTransform = null;
+	activeEnumSchemaNames = /* @__PURE__ */ new Set();
 }
 /**
 * Walk a single TypeRef and extract inline enum refs as top-level enum definitions.
@@ -1157,9 +1168,14 @@ function schemaToTypeRef(schema, contextName, parentModelName) {
 	if (schema.$ref) {
 		const segments = schema.$ref.split("/");
 		const rawName = segments[segments.length - 1];
+		const resolved = resolveSchemaName(rawName);
+		if (activeEnumSchemaNames.has(resolved)) return {
+			kind: "enum",
+			name: resolved
+		};
 		return {
 			kind: "model",
-			name: resolveSchemaName(rawName)
+			name: resolved
 		};
 	}
 	if (Array.isArray(schema.type)) {
@@ -1847,10 +1863,23 @@ function inferServiceName(path, tag) {
 	if (first.startsWith("{")) return "Default";
 	return toPascalCase(first);
 }
+/**
+* Strip overload indices (e.g. NestJS emits `Foo_bar[0]` when a controller
+* method handles multiple routes) from an operationId so derived names never
+* leak the bracketed digit. Without this, `authenticate[0]` would surface as
+* `authenticate0` in method names and `Authenticate0Request` in body model
+* names. Brackets at the end (or with trailing whitespace) are common in
+* NestJS-style backends; this regex is intentionally narrow to avoid eating
+* legitimate digits like `oauth2`.
+*/
+function normalizeOperationIdForNaming(operationId) {
+	return operationId.replace(/\[\d+\]\s*$/, "");
+}
 function inferOperationName(method, path, operationId, operationIdTransform) {
 	if (operationId) {
-		if (operationIdTransform) return operationIdTransform(operationId);
-		return toCamelCase(operationId);
+		const normalized = normalizeOperationIdForNaming(operationId);
+		if (operationIdTransform) return operationIdTransform(normalized);
+		return toCamelCase(normalized);
 	}
 	const verb = inferVerb(method, path);
 	const resource = inferResourceFromPath(path);
@@ -2168,7 +2197,7 @@ function extractRequestBody(body, op, inlineModels) {
 		};
 		return {};
 	}
-	const contextName = cleanSchemaName(op?.operationId ? toPascalCase(op.operationId) + "Request" : "RequestBody");
+	const contextName = cleanSchemaName(op?.operationId ? toPascalCase(normalizeOperationIdForNaming(op.operationId)) + "Request" : "RequestBody");
 	if (schema.properties && (schema.type === "object" || !schema.type)) {
 		const requiredSet = new Set(schema.required ?? []);
 		const fields = [];
@@ -2249,7 +2278,7 @@ function deriveOneOfVariantName(variant, baseName, existingModels) {
 	return name;
 }
 function deriveResponseName(op, path, method) {
-	if (op?.operationId) return cleanSchemaName(toPascalCase(op.operationId) + "Response");
+	if (op?.operationId) return cleanSchemaName(toPascalCase(normalizeOperationIdForNaming(op.operationId)) + "Response");
 	return toPascalCase(path.split("/").filter(Boolean)[0] ?? "Unknown") + toPascalCase(method) + "Response";
 }
 function extractResponses(responses, op, path, method) {