@workos/oagen 0.29.4 → 0.30.1

package/dist/cli/index.mjs

@@ -6360,6 +6360,43 @@ function resolveResponseModelName(op) {
 	if (ref.kind === "nullable" && ref.inner.kind === "model") return ref.inner.name;
 	return null;
 }
+/**
+* Find every pair of distinct paths that resolve to the same SDK method name.
+*
+* Emitters already refuse to generate on a collision, but they only run inside
+* the per-language build matrix — so a spec addition that collides fails every
+* matrix job minutes in, with no cheap way to catch it first. Surfacing the same
+* check from `oagen resolve` turns that into a seconds-long precheck.
+*
+* Returns every collision rather than throwing on the first, so a spec that
+* introduces several is fixed in one pass instead of one CI round-trip each.
+*
+* Callers that generate a subset of services must filter to the operations they
+* will actually emit before calling this: a collision between two out-of-scope
+* paths is not that run's problem.
+*/
+function findResolvedMethodCollisions(resolved) {
+	const seen = /* @__PURE__ */ new Map();
+	const collisions = [];
+	for (const r of resolved) {
+		const key = `${r.mountOn}.${r.methodName}`;
+		const current = {
+			httpMethod: r.operation.httpMethod.toUpperCase(),
+			path: r.operation.path
+		};
+		const existing = seen.get(key);
+		if (existing && (existing.path !== current.path || existing.httpMethod !== current.httpMethod)) {
+			collisions.push({
+				key,
+				first: existing,
+				second: current
+			});
+			continue;
+		}
+		if (!existing) seen.set(key, current);
+	}
+	return collisions;
+}
 //#endregion
 //#region src/engine/scoped-services.ts
 /**
@@ -9192,7 +9229,8 @@ function classifyParameterChanges(baseline, candidate, policy, specRef, language
 			old: { parameter: "(absent)" },
 			new: {
 				parameter: candParam.publicName,
-				required: "true"
+				required: "true",
+				position: String(candParam.position)
 			},
 			message: `Required parameter "${candParam.publicName}" added to "${baseline.displayName}"`,
 			policy,
@@ -9202,7 +9240,10 @@ function classifyParameterChanges(baseline, candidate, policy, specRef, language
 			category,
 			symbol: baseline.fqName,
 			old: { parameter: "(absent)" },
-			new: { parameter: candParam.publicName },
+			new: {
+				parameter: candParam.publicName,
+				position: String(candParam.position)
+			},
 			message: `Optional parameter "${candParam.publicName}" added to "${baseline.displayName}"`,
 			policy,
 			specRef
@@ -10910,6 +10951,11 @@ async function resolveCommand(opts) {
 	});
 	if (opts.docUrl) ir = expandDocUrls(ir, opts.docUrl);
 	const resolved = resolveOperations(ir, opts.operationHints, opts.mountRules);
+	const collisions = findResolvedMethodCollisions(resolved);
+	if (collisions.length > 0) {
+		const detail = collisions.map((c) => `  ${c.key}: ${c.first.httpMethod} ${c.first.path} conflicts with ${c.second.httpMethod} ${c.second.path}`).join("\n");
+		throw new CommandError(`Resolved operation name collision${collisions.length > 1 ? "s" : ""}:\n${detail}`, "Give one of each pair an explicit `name` in operationHints. Rename the NEW path — renaming an already-shipped method breaks released SDKs.", 1);
+	}
 	if ((opts.format ?? "table") === "json") {
 		const output = resolved.map(toJsonRow);
 		console.log(JSON.stringify(output, null, 2));
@@ -10994,7 +11040,10 @@ function applyConfig(config) {
 function handleError(err) {
 	const exitCode = err instanceof CommandError ? err.exitCode : 1;
 	const message = err instanceof Error ? err.message : String(err);
-	if (message) console.error(message);
+	if (message) {
+		const label = err instanceof CommandError ? "" : `${err instanceof Error ? err.name : "Error"}: `;
+		console.error(`${label}${message}`);
+	}
 	process.exit(exitCode);
 }
 const configArgIdx = process.argv.indexOf("--config");