@workos/oagen-emitters 0.12.2 → 0.12.3

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.12.2"
+  ".": "0.12.3"
 }

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.12.3](https://github.com/workos/oagen-emitters/compare/v0.12.2...v0.12.3) (2026-05-18)
+
+
+### Bug Fixes
+
+* **node:** generate tests for adopted services and fix related bugs ([#107](https://github.com/workos/oagen-emitters/issues/107)) ([abe877b](https://github.com/workos/oagen-emitters/commit/abe877bd6489365d0f145a1739a7c449b15760c2))
+
 ## [0.12.2](https://github.com/workos/oagen-emitters/compare/v0.12.1...v0.12.2) (2026-05-15)
 
 

package/dist/index.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.mts","names":[],"sources":["../src/node/index.ts","../src/python/index.ts","../src/php/index.ts","../src/go/index.ts","../src/dotnet/index.ts","../src/kotlin/index.ts","../src/ruby/index.ts","../src/rust/index.ts"],"mappings":";;;;;cAuVa,WAAA,EAAa,OAwFzB;;;cCxYY,aAAA,EAAe,OA+D3B;;;cClEY,UAAA,EAAY,OA2DxB;;;cClEY,SAAA,EAAW,OAyEvB;;;cCnCY,aAAA,EAAe,OAiR3B;;;cCrTY,aAAA,EAAe,OA6E3B;;;cChFY,WAAA,EAAa,OAkEzB;;;cC1DY,WAAA,EAAa,OA0DzB"}
+{"version":3,"file":"index.d.mts","names":[],"sources":["../src/node/index.ts","../src/python/index.ts","../src/php/index.ts","../src/go/index.ts","../src/dotnet/index.ts","../src/kotlin/index.ts","../src/ruby/index.ts","../src/rust/index.ts"],"mappings":";;;;;cAgWa,WAAA,EAAa,OAwFzB;;;cCjZY,aAAA,EAAe,OA+D3B;;;cClEY,UAAA,EAAY,OA2DxB;;;cClEY,SAAA,EAAW,OAyEvB;;;cCnCY,aAAA,EAAe,OAiR3B;;;cCrTY,aAAA,EAAe,OA6E3B;;;cChFY,WAAA,EAAa,OAkEzB;;;cC1DY,WAAA,EAAa,OA0DzB"}

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-eCuvoL1T.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-D2N2ZT5W.mjs";
 export { dotnetEmitter, dotnetExtractor, elixirExtractor, goEmitter, goExtractor, kotlinEmitter, kotlinExtractor, nodeEmitter, nodeExtractor, phpEmitter, phpExtractor, pythonEmitter, pythonExtractor, rubyEmitter, rubyExtractor, rustEmitter, rustExtractor, workosEmittersPlugin };

package/dist/{plugin-eCuvoL1T.mjs → plugin-D2N2ZT5W.mjs}

@@ -338,6 +338,9 @@ let adoptedModelNames = /* @__PURE__ */ new Set();
 function setAdoptedModelNames(names) {
 	adoptedModelNames = names;
 }
+function isAdoptedModelName(name) {
+	return adoptedModelNames.has(name);
+}
 /**
 * Wire/response interface name.
 *
@@ -4291,17 +4294,23 @@ function hasPathParams(segments) {
 *   "/orgs"            → `'orgs'`
 *   "/orgs/{id}"       → `` `orgs/${encodeURIComponent(id)}` ``
 *   "/orgs/{id}/foo"   → `` `orgs/${encodeURIComponent(id)}/foo` ``
+*
+* `paramNameMap` lets a caller override the local variable name used for a
+* spec parameter — used by the options-object code path so the URL template
+* references the SDK's public field name (e.g. `organizationMembershipId`)
+* instead of the spec's path-param name (e.g. `omId`), avoiding a
+* destructure rename in the method body.
 */
-function buildNodePathExpression(rawPath) {
+function buildNodePathExpression(rawPath, paramNameMap) {
 	const segments = parsePathTemplate(rawPath);
 	if (!hasPathParams(segments)) return `'${rawPath}'`;
 	let body = "";
-	for (const seg of segments) body += renderSegment(seg);
+	for (const seg of segments) body += renderSegment(seg, paramNameMap);
 	return `\`${body}\``;
 }
-function renderSegment(seg) {
+function renderSegment(seg, paramNameMap) {
 	if (seg.kind === "literal") return seg.value.replace(/\\/g, "\\\\").replace(/`/g, "\\`").replace(/\$\{/g, "\\${");
-	return `\${encodeURIComponent(${fieldName$6(seg.name)})}`;
+	return `\${encodeURIComponent(${paramNameMap?.get(seg.name) ?? fieldName$6(seg.name)})}`;
 }
 //#endregion
 //#region src/node/wrappers.ts
@@ -5092,8 +5101,8 @@ function renderOptionsObjectMethod(lines, op, plan, method, service, ctx, modelM
 	const optionParam = optionsObjectParam$1(baselineMethod);
 	if (!optionParam) return false;
 	const responseModel = plan.responseModelName ? resolveInterfaceName(plan.responseModelName, ctx) : null;
-	const pathStr = buildPathStr(op);
 	const pathBindings = buildOptionsObjectPathBindings(op, optionParam.type, ctx);
+	const pathStr = buildPathStr(op, buildOptionsObjectPathParamMap(op, optionParam.type, ctx));
 	if (plan.isPaginated && op.pagination && op.httpMethod === "get") {
 		let itemRawName = op.pagination.itemType.kind === "model" ? op.pagination.itemType.name : null;
 		if (itemRawName) {
@@ -5199,11 +5208,23 @@ function renderOptionsObjectDestructure(lines, pathBindings, restName) {
 	else if (restName) lines.push(`    const ${restName} = options;`);
 }
 function buildOptionsObjectPathBindings(op, optionType, ctx) {
-	return op.pathParams.map((param) => {
+	return op.pathParams.map((param) => resolveOptionsObjectField$1(fieldName$6(param.name), optionType, ctx));
+}
+/**
+* Map spec path-param names (e.g. `omId`) to the SDK field name exposed on
+* the options interface (e.g. `organizationMembershipId`). Empty when every
+* path param's spec name already matches the SDK field. Consumed by
+* `buildNodePathExpression` so the URL template binds to the same identifier
+* the destructure does.
+*/
+function buildOptionsObjectPathParamMap(op, optionType, ctx) {
+	const map = /* @__PURE__ */ new Map();
+	for (const param of op.pathParams) {
 		const localName = fieldName$6(param.name);
-		const optionField = resolveOptionsObjectField$1(localName, optionType, ctx);
-		return optionField === localName ? localName : `${optionField}: ${localName}`;
-	});
+		const sdkField = resolveOptionsObjectField$1(localName, optionType, ctx);
+		if (sdkField !== localName) map.set(param.name, sdkField);
+	}
+	return map;
 }
 function resolveOptionsObjectField$1(localName, optionType, ctx) {
 	const fields = ctx.apiSurface?.interfaces?.[optionType]?.fields;
@@ -5464,8 +5485,8 @@ function renderQueryExpr(queryParams) {
 	}
 	return `options ? { ${parts.join(", ")} } : undefined`;
 }
-function buildPathStr(op) {
-	return buildNodePathExpression(op.path);
+function buildPathStr(op, paramNameMap) {
+	return buildNodePathExpression(op.path, paramNameMap);
 }
 function buildPathParams(op, specEnumNames) {
 	const declaredNames = new Set(op.pathParams.map((p) => fieldName$6(p.name)));
@@ -5842,6 +5863,7 @@ function isSupportedFieldType(ref, ownerModelName, shared, ctx) {
 			if (ref.name === ownerModelName) return true;
 			const resolvedName = resolveInterfaceName(ref.name, ctx);
 			if (ctx.apiSurface?.interfaces?.[resolvedName] || ctx.apiSurface?.typeAliases?.[resolvedName]) return true;
+			if (isAdoptedModelName(ref.name)) return true;
 			return liveSurfaceHasManagedFile(`src/${shared.resolveDir(shared.modelToService.get(ref.name))}/interfaces/${fileName$3(ref.name)}.interface.ts`);
 		}
 		case "enum": {
@@ -6901,6 +6923,7 @@ function generateTests$7(spec, ctx) {
 		if (ops.length === 0) continue;
 		const propName = mountAccessors.get(mountName) ?? servicePropertyName$4(mountName);
 		if (ctx.apiSurface && baselineWorkOSProps.size > 0 && !baselineWorkOSProps.has(propName)) continue;
+		if (resolveResourceClassName$3(mergedService, ctx) !== mountName) continue;
 		const testService = ops.length < operations.length ? {
 			...mergedService,
 			operations: ops
@@ -7076,8 +7099,11 @@ function renderBodyTest(lines, op, plan, method, serviceProp, modelMap, ctx, ent
 	const payload = buildTestPayload(op, modelMap);
 	const payloadArg = payload ? payload.camelCaseObj : fallbackBodyArg(op, modelMap);
 	const allArgs = buildOptionsObjectTestArg(op, plan, baselineMethod, modelMap, ctx) ?? (pathArgs ? `${pathArgs}, ${payloadArg}` : payloadArg);
+	const isArrayResponse = !!plan.isArrayResponse;
+	const fixtureExpr = isArrayResponse ? `[${fixture}]` : fixture;
+	const accessor = isArrayResponse ? "result[0]" : "result";
 	lines.push("    it('sends the correct request and returns result', async () => {");
-	lines.push(`      fetchOnce(${fixture});`);
+	lines.push(`      fetchOnce(${fixtureExpr});`);
 	lines.push("");
 	lines.push(`      const result = await workos.${serviceProp}.${method}(${allArgs});`);
 	lines.push("");
@@ -7086,12 +7112,13 @@ function renderBodyTest(lines, op, plan, method, serviceProp, modelMap, ctx, ent
 	lines.push(`      expect(new URL(String(fetchURL())).pathname).toBe('${expectedPath}');`);
 	if (payload) lines.push(`      expect(fetchBody()).toEqual(expect.objectContaining(${payload.snakeCaseObj}));`);
 	else lines.push("      expect(fetchBody()).toBeDefined();");
+	if (isArrayResponse) lines.push("      expect(Array.isArray(result)).toBe(true);");
 	const bodyHelperName = ctx ? `expect${resolveInterfaceName(responseModelName, ctx)}` : null;
-	if (bodyHelperName && entityHelpers?.has(bodyHelperName)) lines.push(`      ${bodyHelperName}(result);`);
+	if (bodyHelperName && entityHelpers?.has(bodyHelperName)) lines.push(`      ${bodyHelperName}(${accessor});`);
 	else {
 		const responseModel = modelMap.get(responseModelName);
 		if (responseModel) {
-			const assertions = buildFieldAssertions(responseModel, "result", modelMap);
+			const assertions = buildFieldAssertions(responseModel, accessor, modelMap);
 			if (assertions.length > 0) for (const assertion of assertions) lines.push(`      ${assertion}`);
 			else lines.push("      expect(result).toBeDefined();");
 		} else lines.push("      expect(result).toBeDefined();");
@@ -7103,20 +7130,24 @@ function renderGetTest(lines, op, plan, method, serviceProp, modelMap, ctx, enti
 	const fixture = `${toCamelCase(responseModelName)}Fixture`;
 	const pathArgs = buildTestPathArgs(op);
 	const optionsArg = buildOptionsObjectTestArg(op, plan, baselineMethod, modelMap, ctx);
+	const isArrayResponse = !!plan.isArrayResponse;
+	const fixtureExpr = isArrayResponse ? `[${fixture}]` : fixture;
+	const accessor = isArrayResponse ? "result[0]" : "result";
 	lines.push("    it('returns the expected result', async () => {");
-	lines.push(`      fetchOnce(${fixture});`);
+	lines.push(`      fetchOnce(${fixtureExpr});`);
 	lines.push("");
 	lines.push(`      const result = await workos.${serviceProp}.${method}(${optionsArg ?? pathArgs});`);
 	lines.push("");
 	lines.push(`      expect(fetchMethod()).toBe('${op.httpMethod.toUpperCase()}');`);
 	const expectedPathGet = buildExpectedPath$4(op);
 	lines.push(`      expect(new URL(String(fetchURL())).pathname).toBe('${expectedPathGet}');`);
+	if (isArrayResponse) lines.push("      expect(Array.isArray(result)).toBe(true);");
 	const helperName = ctx ? `expect${resolveInterfaceName(responseModelName, ctx)}` : null;
-	if (helperName && entityHelpers?.has(helperName)) lines.push(`      ${helperName}(result);`);
+	if (helperName && entityHelpers?.has(helperName)) lines.push(`      ${helperName}(${accessor});`);
 	else {
 		const responseModel = modelMap.get(responseModelName);
 		if (responseModel) {
-			const assertions = buildFieldAssertions(responseModel, "result", modelMap);
+			const assertions = buildFieldAssertions(responseModel, accessor, modelMap);
 			if (assertions.length > 0) for (const assertion of assertions) lines.push(`      ${assertion}`);
 			else lines.push("      expect(result).toBeDefined();");
 		} else lines.push("      expect(result).toBeDefined();");
@@ -7219,23 +7250,28 @@ function generateEntityHelpers(plans, modelMap, ctx) {
 * nested models so we still get meaningful assertions instead of a bare
 * `toBeDefined()`.
 */
+function isDateTimeFieldType(type) {
+	if (type.kind === "primitive") return type.format === "date-time";
+	if (type.kind === "nullable") return isDateTimeFieldType(type.inner);
+	return false;
+}
 function buildFieldAssertions(model, accessor, modelMap) {
 	const assertions = [];
 	for (const field of model.fields) {
 		if (!field.required) continue;
+		const domainField = fieldName$6(field.name);
+		const fieldAccessor = isDateTimeFieldType(field.type) ? `${accessor}.${domainField}.toISOString()` : `${accessor}.${domainField}`;
 		if (field.example !== void 0) {
-			const domainField = fieldName$6(field.name);
 			if (typeof field.example === "object" && field.example !== null) assertions.push(`expect(${accessor}.${domainField}).toEqual(${JSON.stringify(field.example)});`);
 			else {
 				const exampleLiteral = typeof field.example === "string" ? `'${field.example}'` : String(field.example);
-				assertions.push(`expect(${accessor}.${domainField}).toBe(${exampleLiteral});`);
+				assertions.push(`expect(${fieldAccessor}).toBe(${exampleLiteral});`);
 			}
 			continue;
 		}
 		const value = fixtureValueForType(field.type, field.name, model.name);
 		if (value === null) continue;
-		const domainField = fieldName$6(field.name);
-		assertions.push(`expect(${accessor}.${domainField}).toBe(${value});`);
+		assertions.push(`expect(${fieldAccessor}).toBe(${value});`);
 	}
 	if (assertions.length === 0 && modelMap) for (const field of model.fields) {
 		if (!field.required) continue;
@@ -7700,9 +7736,12 @@ function applyLiveSurface(files, ctx, surface) {
 		if (policy.hasExistingSdk && !policy.managedPaths.has(f.path) && !canCreateNewPath(f.path, policy)) continue;
 		if (surface.files.has(f.path) && !surface.autogenFiles.has(f.path) && !ownedPath) continue;
 		if (isUserOwnedAfterFirstEmit(f.path)) {
+			const dir = topLevelDir(f.path);
+			const isAdoptedDir = dir !== void 0 && policy.adoptedServiceDirs.has(dir);
+			const isManagedDir = ownedPath || isAdoptedDir;
 			if (surface.files.has(f.path) && !surface.autogenFiles.has(f.path)) continue;
-			if (!ownedPath && !surface.autogenFiles.has(f.path)) continue;
-			if (ownedPath && !policy.regenerateOwnedTests) continue;
+			if (!isManagedDir && !surface.autogenFiles.has(f.path)) continue;
+			if (isManagedDir && !policy.regenerateOwnedTests) continue;
 		}
 		if (surface.autogenFiles.has(f.path) || ownedPath) {
 			f.overwriteExisting = true;
@@ -26520,4 +26559,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-eCuvoL1T.mjs.map
+//# sourceMappingURL=plugin-D2N2ZT5W.mjs.map