@workos/oagen 0.10.2 → 0.10.4

package/dist/cli/index.mjs

@@ -6621,17 +6621,11 @@ function filterSurfaceByManifest(surface, manifest) {
 			filesExcluded++;
 			continue;
 		}
-		const generatedMethods = serviceMethodMap.get(normalize(name));
-		if (generatedMethods) {
-			const methods = {};
-			for (const [methodName, overloads] of Object.entries(cls.methods)) if (generatedMethods.has(methodName)) methods[methodName] = overloads;
-			else methodsExcluded++;
-			filteredClasses[name] = {
-				...cls,
-				methods,
-				constructorParams: []
-			};
-		} else filteredClasses[name] = cls;
+		if (serviceMethodMap.get(normalize(name))) filteredClasses[name] = {
+			...cls,
+			constructorParams: []
+		};
+		else filteredClasses[name] = cls;
 	}
 	const filteredExports = {};
 	for (const [filePath, symbols] of Object.entries(surface.exports)) if (manifestFiles.has(filePath)) filteredExports[filePath] = symbols;
@@ -6654,9 +6648,9 @@ async function compatExtractCommand(opts) {
 	let surface = await extractor.extract(opts.sdkPath);
 	const manifest = await readManifest(opts.sdkPath);
 	if (manifest) {
-		const { filtered, filesExcluded, methodsExcluded } = filterSurfaceByManifest(surface, manifest);
+		const { filtered, filesExcluded } = filterSurfaceByManifest(surface, manifest);
 		surface = filtered;
-		if (filesExcluded > 0 || methodsExcluded > 0) console.log(`Manifest filter: excluded ${filesExcluded} hand-written file(s), ${methodsExcluded} hand-written method(s)`);
+		if (filesExcluded > 0) console.log(`Manifest filter: excluded ${filesExcluded} hand-written file(s)`);
 	}
 	const snapshot = apiSurfaceToSnapshot(surface);
 	if (opts.spec) {
@@ -7185,6 +7179,9 @@ function filterSurface(surface, allowedNames, opts) {
 function diffSnapshots(baseline, candidate, policy) {
 	const effectivePolicy = policy ?? baseline.policies;
 	const changes = [];
+	const serviceAccessors = /* @__PURE__ */ new Set();
+	for (const sym of baseline.symbols) if (sym.kind === "service_accessor") serviceAccessors.add(sym.fqName);
+	for (const sym of candidate.symbols) if (sym.kind === "service_accessor") serviceAccessors.add(sym.fqName);
 	const candById = /* @__PURE__ */ new Map();
 	const candByFqName = /* @__PURE__ */ new Map();
 	for (const sym of candidate.symbols) {
@@ -7194,15 +7191,37 @@ function diffSnapshots(baseline, candidate, policy) {
 	const baseByFqName = /* @__PURE__ */ new Set();
 	for (const sym of baseline.symbols) baseByFqName.add(sym.fqName);
 	for (const baseSym of baseline.symbols) {
+		if (isServiceWrapperConstructor(baseSym, serviceAccessors)) continue;
 		const candSym = candById.get(baseSym.id) ?? candByFqName.get(baseSym.fqName);
 		changes.push(...classifySymbolChanges(baseSym, candSym, effectivePolicy));
 	}
-	for (const candSym of candidate.symbols) if (!baseByFqName.has(candSym.fqName)) changes.push(classifyAddedSymbol(candSym));
+	for (const candSym of candidate.symbols) if (!baseByFqName.has(candSym.fqName)) {
+		if (isServiceWrapperConstructor(candSym, serviceAccessors)) continue;
+		changes.push(classifyAddedSymbol(candSym));
+	}
 	return {
 		changes,
 		summary: summarizeChanges(changes)
 	};
 }
+/**
+* Check if a symbol is a constructor belonging to a service wrapper class.
+*
+* Service wrapper constructors are internal plumbing (taking a client/config
+* object) — users interact with services via `client.admin_portal`, not
+* `new AdminPortal(...)`.  Changes to these constructors should not be
+* reported as breaking.
+*
+* Catches two patterns:
+*  - Ruby: kind === 'constructor', ownerFqName is a service_accessor
+*  - PHP:  kind === 'callable' with fqName ending in '.__construct'
+*/
+function isServiceWrapperConstructor(sym, serviceAccessors) {
+	if (!sym.ownerFqName || !serviceAccessors.has(sym.ownerFqName)) return false;
+	if (sym.kind === "constructor") return true;
+	if (sym.kind === "callable" && sym.fqName.endsWith(".__construct")) return true;
+	return false;
+}
 //#endregion
 //#region src/compat/concepts.ts
 /**