[Openapi-Diff] Bump autorest to ^3.8.0 and fix stderr handling (#459)

raych1 · Copilot · web-flow · commit 6d4dc5d18576 · 2026-03-16T17:03:52.000-07:00
* Bump autorest to ^3.8.0 and fix stderr handling

- Only throw on non-zero exit code; log stderr as warning otherwise
- autorest 3.8.0 emits deprecation message to stderr with exit code 0
- Bump version to 0.12.4

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;

* resolved review feedback

---------

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.12.4 2026-03-13
+
+- Only throw error on AutoRest stderr when exit code is non-zero; log warnings otherwise
+
 # 0.12.3 2025-10-13
 
 - Fix crash in logging when property named "$ref"
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@azure/oad",
-  "version": "0.12.3",
+  "version": "0.12.4",
   "author": {
     "name": "Microsoft Corporation",
     "email": "azsdkteam@microsoft.com",
@@ -15,7 +15,7 @@
     "@ts-common/json-parser": "0.9.0",
     "@ts-common/source-map": "0.5.0",
     "@ts-common/string-map": "0.3.0",
-    "autorest": "^3.6.1",
+    "autorest": "^3.8.0",
     "js-yaml": "^4.1.0",
     "json-pointer": "0.6.2",
     "json-refs": "^3.0.15",
diff --git a/src/lib/validators/openApiDiff.ts b/src/lib/validators/openApiDiff.ts
@@ -250,7 +250,8 @@ export class OpenApiDiff {
       env: { ...process.env, NODE_OPTIONS: "--max-old-space-size=8192" }
     })
     if (stderr) {
-      throw new Error(stderr)
+      // autorest 3.8.0 emits deprecation message to stderr with exit code 0
+      log.debug(`AutoRest completed with warnings on stderr: ${stderr}`)
     }
 
     const buffer = await asyncFs.readFile(outputMapFilePath)

Original file line number	Diff line number	Diff line change
`@@ -250,7 +250,8 @@ export class OpenApiDiff {`
`250`	`250`	`env: { ...process.env, NODE_OPTIONS: "--max-old-space-size=8192" }`
`251`	`251`	`})`
`252`	`252`	`if (stderr) {`
`253`		`- throw new Error(stderr)`
	`253`	`+ // autorest 3.8.0 emits deprecation message to stderr with exit code 0`
	`254`	+ log.debug(`AutoRest completed with warnings on stderr: ${stderr}`)
`254`	`255`	`}`
`255`	`256`
`256`	`257`	`const buffer = await asyncFs.readFile(outputMapFilePath)`