@workos/oagen-emitters 0.12.4 → 0.12.5

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.12.4"
+  ".": "0.12.5"
 }

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.12.5](https://github.com/workos/oagen-emitters/compare/v0.12.4...v0.12.5) (2026-05-19)
+
+
+### Bug Fixes
+
+* **node:** skip dead deserializers and strengthen all-optional body tests ([#111](https://github.com/workos/oagen-emitters/issues/111)) ([632bd82](https://github.com/workos/oagen-emitters/commit/632bd8257d4139ab53895b5225fffd6913702e3b))
+
 ## [0.12.4](https://github.com/workos/oagen-emitters/compare/v0.12.3...v0.12.4) (2026-05-19)
 
 

package/dist/index.mjs

@@ -1,2 +1,2 @@
-import { _ as pythonEmitter, a as rustExtractor, c as pythonExtractor, d as rustEmitter, f as rubyEmitter, g as phpEmitter, h as goEmitter, i as kotlinExtractor, l as rubyExtractor, m as dotnetEmitter, n as elixirExtractor, o as goExtractor, p as kotlinEmitter, r as dotnetExtractor, s as phpExtractor, t as workosEmittersPlugin, u as nodeExtractor, v as nodeEmitter } from "./plugin-nmiHN7Ko.mjs";
+import { _ as pythonEmitter, a as rustExtractor, c as pythonExtractor, d as rustEmitter, f as rubyEmitter, g as phpEmitter, h as goEmitter, i as kotlinExtractor, l as rubyExtractor, m as dotnetEmitter, n as elixirExtractor, o as goExtractor, p as kotlinEmitter, r as dotnetExtractor, s as phpExtractor, t as workosEmittersPlugin, u as nodeExtractor, v as nodeEmitter } from "./plugin-Ca9LUkWW.mjs";
 export { dotnetEmitter, dotnetExtractor, elixirExtractor, goEmitter, goExtractor, kotlinEmitter, kotlinExtractor, nodeEmitter, nodeExtractor, phpEmitter, phpExtractor, pythonEmitter, pythonExtractor, rubyEmitter, rubyExtractor, rustEmitter, rustExtractor, workosEmittersPlugin };

package/dist/{plugin-nmiHN7Ko.mjs → plugin-Ca9LUkWW.mjs}

@@ -4073,6 +4073,7 @@ function buildSerializerImports(model, serializerPath, dirName, domainName, resp
 		const rel = relativeImport(serializerPath, depSerializerPath);
 		const canon = sctx.dedup.get(dep);
 		const depSkipSerialize = sctx.skippedSerializeModels.has(dep) || canon != null && sctx.skippedSerializeModels.has(canon);
+		const depSkipDeserialize = sctx.responseReachableModels !== void 0 && !sctx.responseReachableModels.has(dep) && (canon == null || !sctx.responseReachableModels.has(canon));
 		const hasDeser = liveSurfaceHasFunction(`deserialize${depName}`);
 		const hasSer = liveSurfaceHasFunction(`serialize${depName}`);
 		const fileExists = liveSurfaceHasFile(depSerializerPath);
@@ -4093,7 +4094,9 @@ function buildSerializerImports(model, serializerPath, dirName, domainName, resp
 			lines.push(`import { serialize${depName} } from '${rel}';`);
 			continue;
 		}
+		if (depSkipSerialize && depSkipDeserialize) continue;
 		if (depSkipSerialize) lines.push(`import { deserialize${depName} } from '${rel}';`);
+		else if (depSkipDeserialize) lines.push(`import { serialize${depName} } from '${rel}';`);
 		else lines.push(`import { deserialize${depName}, serialize${depName} } from '${rel}';`);
 	}
 	lines.push("");
@@ -4129,24 +4132,26 @@ function shouldSkipSerializeForModel(model, baselineResponse, baselineDomain, de
 	}
 	return shouldSkip;
 }
-function emitSerializerBody(model, domainName, responseName, typeParams, baselineDomain, baselineResponse, skipFormatFields, shouldSkipSerialize, ctx) {
+function emitSerializerBody(model, domainName, responseName, typeParams, baselineDomain, baselineResponse, skipFormatFields, shouldSkipSerialize, shouldSkipDeserialize, ctx) {
 	const lines = [];
-	const seenDeserFields = /* @__PURE__ */ new Set();
-	const deserParamPrefix = model.fields.length === 0 ? "_" : "";
-	lines.push(`export const deserialize${domainName} = ${typeParams.decl}(`);
-	lines.push(`  ${deserParamPrefix}response: ${responseName}${typeParams.usage},`);
-	lines.push(`): ${domainName}${typeParams.usage} => ({`);
-	for (const field of model.fields) {
-		const domain = fieldName$6(field.name);
-		if (seenDeserFields.has(domain)) continue;
-		seenDeserFields.add(domain);
-		const plan = planDeserializeField(field, baselineDomain, baselineResponse, skipFormatFields, ctx);
-		if (!plan.skip) lines.push(plan.line);
+	if (!shouldSkipDeserialize) {
+		const seenDeserFields = /* @__PURE__ */ new Set();
+		const deserParamPrefix = model.fields.length === 0 ? "_" : "";
+		lines.push(`export const deserialize${domainName} = ${typeParams.decl}(`);
+		lines.push(`  ${deserParamPrefix}response: ${responseName}${typeParams.usage},`);
+		lines.push(`): ${domainName}${typeParams.usage} => ({`);
+		for (const field of model.fields) {
+			const domain = fieldName$6(field.name);
+			if (seenDeserFields.has(domain)) continue;
+			seenDeserFields.add(domain);
+			const plan = planDeserializeField(field, baselineDomain, baselineResponse, skipFormatFields, ctx);
+			if (!plan.skip) lines.push(plan.line);
+		}
+		lines.push("});");
 	}
-	lines.push("});");
 	if (!shouldSkipSerialize) {
+		if (!shouldSkipDeserialize) lines.push("");
 		const serParamPrefix = model.fields.length === 0 ? "_" : "";
-		lines.push("");
 		lines.push(`export const serialize${domainName} = ${typeParams.decl}(`);
 		lines.push(`  ${serParamPrefix}model: ${domainName}${typeParams.usage},`);
 		lines.push(`): ${responseName}${typeParams.usage} => ({`);
@@ -6217,6 +6222,7 @@ function generateSerializers(models, ctx, shared) {
 	const projectedByName = new Map(projectedModels.map((model) => [model.name, model]));
 	const resourceUsage = buildGeneratedResourceModelUsage(models, ctx);
 	const serializerEligibleModels = resourceUsage ? expandModelRoots(resourceUsage.serializerRoots, projectedByName) : void 0;
+	const responseReachableModels = resourceUsage ? expandModelRoots(resourceUsage.responseRoots, projectedByName) : void 0;
 	const serializerReachable = computeNonEventReachable(ctx.spec.services, models);
 	const liveRoot = ctx.targetDir ?? ctx.outputDir;
 	if (liveRoot) for (const originalModel of models) {
@@ -6280,7 +6286,13 @@ function generateSerializers(models, ctx, shared) {
 			if (serializerPath === canonSerializerPath) continue;
 			if (domainName === canonDomainName) continue;
 			const rel = relativeImport(serializerPath, canonSerializerPath);
-			const reexportContent = skippedSerializeModels.has(canonicalName) || skippedSerializeModels.has(model.name) ? `export { deserialize${canonDomainName} as deserialize${domainName} } from '${rel}';` : `export { deserialize${canonDomainName} as deserialize${domainName}, serialize${canonDomainName} as serialize${domainName} } from '${rel}';`;
+			const canonSkipSerialize = skippedSerializeModels.has(canonicalName) || skippedSerializeModels.has(model.name);
+			const canonSkipDeserialize = responseReachableModels !== void 0 && !responseReachableModels.has(canonicalName) && !responseReachableModels.has(model.name);
+			if (canonSkipSerialize && canonSkipDeserialize) continue;
+			const parts = [];
+			if (!canonSkipDeserialize) parts.push(`deserialize${canonDomainName} as deserialize${domainName}`);
+			if (!canonSkipSerialize) parts.push(`serialize${canonDomainName} as serialize${domainName}`);
+			const reexportContent = `export { ${parts.join(", ")} } from '${rel}';`;
 			files.push({
 				path: serializerPath,
 				content: reexportContent,
@@ -6298,13 +6310,16 @@ function generateSerializers(models, ctx, shared) {
 		const baselineDomain = ctx.apiSurface?.interfaces?.[domainName];
 		const skipFormatFields = buildSkipFormatFields(model, baselineDomain);
 		const shouldSkipSerialize = skippedSerializeModels.has(model.name);
+		const shouldSkipDeserialize = responseReachableModels !== void 0 && !responseReachableModels.has(model.name);
+		if (shouldSkipSerialize && shouldSkipDeserialize) continue;
 		const lines = [...buildSerializerImports(model, serializerPath, dirName, domainName, responseName, {
 			modelToService,
 			resolveDir,
 			dedup,
 			skippedSerializeModels,
+			responseReachableModels,
 			ctx
-		}), ...emitSerializerBody(model, domainName, responseName, typeParams, baselineDomain, baselineResponse, skipFormatFields, shouldSkipSerialize, ctx)];
+		}), ...emitSerializerBody(model, domainName, responseName, typeParams, baselineDomain, baselineResponse, skipFormatFields, shouldSkipSerialize, shouldSkipDeserialize, ctx)];
 		files.push({
 			path: serializerPath,
 			content: pruneUnusedImports(lines).join("\n"),
@@ -6312,6 +6327,7 @@ function generateSerializers(models, ctx, shared) {
 		});
 	}
 	ctx._skippedSerializeModels = skippedSerializeModels;
+	ctx._responseReachableModels = responseReachableModels;
 	const serializersByDir = /* @__PURE__ */ new Map();
 	for (const f of files) {
 		const match = f.path.match(/^src\/([^/]+)\/serializers\/(.+)\.serializer\.ts$/);
@@ -6357,6 +6373,8 @@ function buildGeneratedResourceModelUsage(models, ctx) {
 	const modelMap = new Map(models.map((model) => [model.name, model]));
 	const interfaceRoots = /* @__PURE__ */ new Set();
 	const serializerRoots = /* @__PURE__ */ new Set();
+	const requestRoots = /* @__PURE__ */ new Set();
+	const responseRoots = /* @__PURE__ */ new Set();
 	const resolvedLookup = buildResolvedLookup(ctx);
 	const mountGroups = groupByMount(ctx);
 	const services = mountGroups.size > 0 ? [...mountGroups].map(([name, group]) => ({
@@ -6385,15 +6403,18 @@ function buildGeneratedResourceModelUsage(models, ctx) {
 					if (unwrapped) itemName = unwrapped.name;
 					interfaceRoots.add(itemName);
 					serializerRoots.add(itemName);
+					responseRoots.add(itemName);
 				}
 			} else if (plan.responseModelName) {
 				interfaceRoots.add(plan.responseModelName);
 				serializerRoots.add(plan.responseModelName);
+				responseRoots.add(plan.responseModelName);
 			}
 			const bodyInfo = extractRequestBodyModels(op, ctx);
 			for (const name of bodyInfo) {
 				interfaceRoots.add(name);
 				serializerRoots.add(name);
+				requestRoots.add(name);
 			}
 			for (const param of [
 				...op.pathParams,
@@ -6405,6 +6426,7 @@ function buildGeneratedResourceModelUsage(models, ctx) {
 				for (const name of collectWrapperResponseModels$1(resolved)) {
 					interfaceRoots.add(name);
 					serializerRoots.add(name);
+					responseRoots.add(name);
 				}
 				for (const wrapper of resolved.wrappers ?? []) for (const { field } of resolveWrapperParams(wrapper, ctx)) if (field) collectTypeRefModels(field.type, interfaceRoots);
 			}
@@ -6412,7 +6434,9 @@ function buildGeneratedResourceModelUsage(models, ctx) {
 	}
 	return {
 		interfaceRoots,
-		serializerRoots
+		serializerRoots,
+		requestRoots,
+		responseRoots
 	};
 }
 function extractRequestBodyModels(op, ctx) {
@@ -7393,12 +7417,20 @@ function buildTestPayload(op, modelMap) {
 		model = modelMap.get(firstVariant.name);
 	} else if (op.requestBody.kind === "model") model = modelMap.get(op.requestBody.name);
 	if (!model) return null;
-	const fields = model.fields.filter((f) => f.required);
-	const usableFields = fields.filter((f) => fixtureValueForType(f.type, f.name, model.name, modelMap) !== null);
-	if (usableFields.length === 0 || usableFields.length < fields.length) return null;
+	const requiredFields = model.fields.filter((f) => f.required);
+	const usableRequired = requiredFields.filter((f) => fixtureValueForType(f.type, f.name, model.name, modelMap) !== null);
+	let chosenFields;
+	if (requiredFields.length > 0) {
+		if (usableRequired.length < requiredFields.length) return null;
+		chosenFields = usableRequired;
+	} else {
+		const usableOptional = model.fields.filter((f) => !f.required && fixtureValueForType(f.type, f.name, model.name, modelMap) !== null);
+		if (usableOptional.length === 0) return null;
+		chosenFields = usableOptional.slice(0, 2);
+	}
 	const camelEntries = [];
 	const snakeEntries = [];
-	for (const field of usableFields) {
+	for (const field of chosenFields) {
 		const camelValue = fixtureValueForType(field.type, field.name, model.name, modelMap);
 		const wireValue = fixtureValueForType(field.type, field.name, model.name, modelMap, true);
 		const camelKey = fieldName$6(field.name);
@@ -7458,6 +7490,7 @@ function generateSerializerTests(spec, ctx) {
 	});
 	if (eligibleModels.length === 0) return files;
 	const serializeSkipped = ctx._skippedSerializeModels ?? /* @__PURE__ */ new Set();
+	const responseReachableModels = ctx._responseReachableModels;
 	const modelsByDir = /* @__PURE__ */ new Map();
 	for (const model of eligibleModels) {
 		const dirName = resolveDir(modelToService.get(model.name));
@@ -7472,15 +7505,20 @@ function generateSerializerTests(spec, ctx) {
 		const interfaceImports = [];
 		const fixtureImports = [];
 		const deserializeOnlyModels = /* @__PURE__ */ new Set();
+		const serializeOnlyModels = /* @__PURE__ */ new Set();
 		for (const model of models) {
 			const domainName = resolveInterfaceName(model.name, ctx);
 			const modelDir = resolveDir(modelToService.get(model.name));
 			const serializerPath = `src/${modelDir}/serializers/${fileName$3(model.name)}.serializer.ts`;
 			const interfacePath = `src/${modelDir}/interfaces/${fileName$3(model.name)}.interface.ts`;
 			const fixturePath = `src/${modelDir}/fixtures/${fileName$3(model.name)}.json`;
-			const deserializeOnly = serializeSkipped.has(model.name) || fixtureIsHandOwned(fixturePath, ctx);
+			const isRequestOnly = responseReachableModels !== void 0 && !responseReachableModels.has(model.name);
+			const deserializeOnly = !isRequestOnly && (serializeSkipped.has(model.name) || fixtureIsHandOwned(fixturePath, ctx));
+			const serializeOnly = isRequestOnly;
 			if (deserializeOnly) deserializeOnlyModels.add(model.name);
+			if (serializeOnly) serializeOnlyModels.add(model.name);
 			if (deserializeOnly) serializerImports.push(`import { deserialize${domainName} } from '${relativeImport(testPath, serializerPath)}';`);
+			else if (serializeOnly) serializerImports.push(`import { serialize${domainName} } from '${relativeImport(testPath, serializerPath)}';`);
 			else serializerImports.push(`import { deserialize${domainName}, serialize${domainName} } from '${relativeImport(testPath, serializerPath)}';`);
 			const wireName = wireInterfaceName(domainName);
 			interfaceImports.push(`import type { ${wireName} } from '${relativeImport(testPath, interfacePath)}';`);
@@ -7503,6 +7541,14 @@ function generateSerializerTests(spec, ctx) {
 				lines.push("    expect(deserialized).toBeDefined();");
 				lines.push("  });");
 				lines.push("});");
+			} else if (serializeOnlyModels.has(model.name)) {
+				lines.push(`describe('${domainName}Serializer', () => {`);
+				lines.push("  it('serializes correctly', () => {");
+				lines.push(`    const fixture = ${fixtureName} as ${wireName};`);
+				lines.push(`    const serialized = serialize${domainName}(fixture as any);`);
+				lines.push("    expect(serialized).toBeDefined();");
+				lines.push("  });");
+				lines.push("});");
 			} else {
 				lines.push(`describe('${domainName}Serializer', () => {`);
 				lines.push("  it('round-trips through serialize/deserialize', () => {");
@@ -26585,4 +26631,4 @@ const workosEmittersPlugin = {
 //#endregion
 export { pythonEmitter as _, rustExtractor as a, pythonExtractor as c, rustEmitter as d, rubyEmitter as f, phpEmitter as g, goEmitter as h, kotlinExtractor as i, rubyExtractor as l, dotnetEmitter as m, elixirExtractor as n, goExtractor as o, kotlinEmitter as p, dotnetExtractor as r, phpExtractor as s, workosEmittersPlugin as t, nodeExtractor as u, nodeEmitter as v };
 
-//# sourceMappingURL=plugin-nmiHN7Ko.mjs.map
+//# sourceMappingURL=plugin-Ca9LUkWW.mjs.map