@workos/oagen 0.18.2 → 0.19.1

package/dist/cli/index.mjs

@@ -4983,6 +4983,7 @@ function buildEmitterContext(spec, options) {
 		overlayLookup: options.overlayLookup,
 		resolvedOperations: resolveOperations(spec, options.operationHints, options.mountRules),
 		modelHints: options.modelHints,
+		emitterOptions: options.emitterOptions,
 		targetDir: options.target,
 		priorTargetManifestPaths: options.priorTargetManifestPaths
 	};
@@ -5421,8 +5422,9 @@ async function removeEmptyParents(startDir, stopDir) {
 //#endregion
 //#region src/engine/orchestrator.ts
 async function generate(spec, emitter, options) {
+	const outputPrevManifestForCtx = options.noPrune ? null : await readManifest(options.outputDir);
 	const targetManifestForCtx = options.target && !options.noPrune ? await readManifest(options.target) : null;
-	const priorTargetManifestPaths = targetManifestForCtx ? new Set(targetManifestForCtx.files) : void 0;
+	const priorTargetManifestPaths = targetManifestForCtx ? new Set(targetManifestForCtx.files) : outputPrevManifestForCtx ? new Set(outputPrevManifestForCtx.files) : void 0;
 	const { files: withHeaders, header, operations } = generateFiles(spec, emitter, {
 		...options,
 		priorTargetManifestPaths
@@ -5434,7 +5436,7 @@ async function generate(spec, emitter, options) {
 		}
 		return withHeaders;
 	}
-	const outputPrevManifest = options.noPrune ? null : await readManifest(options.outputDir);
+	const outputPrevManifest = outputPrevManifestForCtx;
 	const writeResult = await writeFiles(withHeaders, options.outputDir, {
 		language: emitter.language,
 		header
@@ -6217,6 +6219,7 @@ async function generateCommand(opts) {
 		operationHints: opts.operationHints,
 		mountRules: opts.mountRules,
 		modelHints: opts.modelHints,
+		emitterOptions: opts.emitterOptions,
 		noPrune: opts.prune === false
 	});
 	if (opts.dryRun) for (const f of files) console.log(f.path);
@@ -6927,16 +6930,17 @@ function apiSurfaceToSnapshot(surface) {
 /** Convert a legacy ApiParam to a CompatParameter. */
 function apiParamToCompatParam(param, position, language) {
 	const policy = getDefaultPolicy(language);
+	const passing = param.passingStyle ?? inferPassingStyle(language);
 	return {
 		publicName: param.name,
 		position,
 		required: !param.optional,
 		nullable: false,
 		hasDefault: param.optional,
-		passing: param.passingStyle ?? inferPassingStyle(language),
+		passing,
 		type: { name: param.type },
 		sensitivity: {
-			order: policy.constructorOrderMatters,
+			order: passing === "positional",
 			publicName: policy.methodParameterNamesArePublicApi,
 			requiredness: true,
 			type: true
@@ -7270,10 +7274,9 @@ function classifyParameterChanges(baseline, candidate, policy, specRef) {
 			policy,
 			specRef
 		}));
-		if (baseParam.position !== candParam.position) if (isConstructor ? policy.constructorOrderMatters : baseParam.sensitivity.order) {
-			const category = isConstructor ? "constructor_position_changed_order_sensitive" : "parameter_position_changed_order_sensitive";
-			changes.push(makeChange({
-				category,
+		if (baseParam.position !== candParam.position) {
+			if (isConstructor) if (policy.constructorOrderMatters) changes.push(makeChange({
+				category: "constructor_position_changed_order_sensitive",
 				symbol: baseline.fqName,
 				old: {
 					parameter: baseParam.publicName,
@@ -7287,21 +7290,52 @@ function classifyParameterChanges(baseline, candidate, policy, specRef) {
 				policy,
 				specRef
 			}));
-		} else changes.push(makeChange({
-			category: "constructor_reordered_named_friendly",
-			symbol: baseline.fqName,
-			old: {
-				parameter: baseParam.publicName,
-				position: String(baseParam.position)
-			},
-			new: {
-				parameter: candParam.publicName,
-				position: String(candParam.position)
-			},
-			message: `Parameter "${baseParam.publicName}" reordered on "${baseline.displayName}" (named-friendly language)`,
-			policy,
-			specRef
-		}));
+			else changes.push(makeChange({
+				category: "constructor_reordered_named_friendly",
+				symbol: baseline.fqName,
+				old: {
+					parameter: baseParam.publicName,
+					position: String(baseParam.position)
+				},
+				new: {
+					parameter: candParam.publicName,
+					position: String(candParam.position)
+				},
+				message: `Parameter "${baseParam.publicName}" reordered on "${baseline.displayName}" (named-friendly language)`,
+				policy,
+				specRef
+			}));
+			else if (baseParam.passing === "positional") changes.push(makeChange({
+				category: "parameter_position_changed_order_sensitive",
+				symbol: baseline.fqName,
+				old: {
+					parameter: baseParam.publicName,
+					position: String(baseParam.position)
+				},
+				new: {
+					parameter: candParam.publicName,
+					position: String(candParam.position)
+				},
+				message: `Parameter "${baseParam.publicName}" moved from position ${baseParam.position} to ${candParam.position} on "${baseline.displayName}"`,
+				policy,
+				specRef
+			}));
+			else if (baseParam.passing === "named") changes.push(makeChange({
+				category: "constructor_reordered_named_friendly",
+				symbol: baseline.fqName,
+				old: {
+					parameter: baseParam.publicName,
+					position: String(baseParam.position)
+				},
+				new: {
+					parameter: candParam.publicName,
+					position: String(candParam.position)
+				},
+				message: `Parameter "${baseParam.publicName}" reordered on "${baseline.displayName}" (named-friendly language)`,
+				policy,
+				specRef
+			}));
+		}
 	}
 	for (const candParam of candParams) if (!baseByName.has(candParam.publicName)) {
 		if (changes.some((c) => c.category === "parameter_renamed" && c.new.parameter === candParam.publicName)) continue;
@@ -9133,6 +9167,7 @@ let configDocUrl;
 let configOperationHints;
 let configMountRules;
 let configModelHints;
+let configEmitterOptions;
 let configCompat;
 try {
 	const config = await loadConfig(explicitConfigPath);
@@ -9146,6 +9181,7 @@ try {
 		configOperationHints = config.operationHints;
 		configMountRules = config.mountRules;
 		configModelHints = config.modelHints;
+		configEmitterOptions = config.emitterOptions;
 		configCompat = config.compat;
 	}
 } catch (err) {
@@ -9179,7 +9215,8 @@ program.command("generate").description("Run a registered emitter against an Ope
 		docUrl: configDocUrl,
 		operationHints: configOperationHints,
 		mountRules: configMountRules,
-		modelHints: configModelHints
+		modelHints: configModelHints,
+		emitterOptions: configEmitterOptions?.[opts.lang]
 	}).catch(handleError);
 });
 program.command("diff").description("Compare two OpenAPI specs and output a diff report").requiredOption("--old <path>", "Path to old spec").requiredOption("--new <path>", "Path to new spec").action((opts) => {