From 052c5aded2ebc739826091d585dd065b4c2face5 Mon Sep 17 00:00:00 2001
From: Anton Golub <a.golub@cian.ru>
Date: Tue, 25 Feb 2025 18:39:51 +0300
Subject: [PATCH] feat: pass origin error as `ProcessOuput` cause

---
 .size-limit.json  |  4 ++--
 src/core.ts       | 10 ++++++----
 src/goods.ts      |  2 +-
 test/core.test.js |  4 +++-
 4 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/.size-limit.json b/.size-limit.json
index 5772945e1a..c4b74716c1 100644
--- a/.size-limit.json
+++ b/.size-limit.json
@@ -2,7 +2,7 @@
   {
     "name": "zx/core",
     "path": ["build/core.cjs", "build/util.cjs", "build/vendor-core.cjs"],
-    "limit": "77.5 kB",
+    "limit": "77.6 kB",
     "brotli": false,
     "gzip": false
   },
@@ -30,7 +30,7 @@
   {
     "name": "all",
     "path": "build/*",
-    "limit": "850 kB",
+    "limit": "850.2 kB",
     "brotli": false,
     "gzip": false
   }
diff --git a/src/core.ts b/src/core.ts
index 171d0c8fee..f73c66371b 100644
--- a/src/core.ts
+++ b/src/core.ts
@@ -675,6 +675,11 @@ type ProcessDto = {
 
 export class ProcessOutput extends Error {
   private readonly _dto: ProcessDto
+  cause!: Error | null
+  message!: string
+  stdout!: string
+  stderr!: string
+  stdall!: string
   constructor(dto: ProcessDto)
   constructor(
     code: number | null,
@@ -704,6 +709,7 @@ export class ProcessOutput extends Error {
       : { code, signal, duration, error, from, store }
 
     Object.defineProperties(this, {
+      cause: { value: dto.error, enumerable: false, writable: true, configurable: true },
       stdout: { get: once(() => bufArrJoin(dto.store.stdout)) },
       stderr: { get: once(() => bufArrJoin(dto.store.stderr)) },
       stdall: { get: once(() => bufArrJoin(dto.store.stdall)) },
@@ -715,10 +721,6 @@ export class ProcessOutput extends Error {
       },
     })
   }
-  message!: string
-  stdout!: string
-  stderr!: string
-  stdall!: string
 
   get exitCode(): number | null {
     return this._dto.code
diff --git a/src/goods.ts b/src/goods.ts
index f46bf1fd0f..8f6539a950 100644
--- a/src/goods.ts
+++ b/src/goods.ts
@@ -194,7 +194,7 @@ export async function spinner<T>(
   title: string | (() => T),
   callback?: () => T
 ): Promise<T> {
-  if (typeof title == 'function') {
+  if (typeof title === 'function') {
     callback = title
     title = ''
   }
diff --git a/test/core.test.js b/test/core.test.js
index 8552d1db3d..46191e9ec9 100644
--- a/test/core.test.js
+++ b/test/core.test.js
@@ -253,15 +253,17 @@ describe('core', () => {
         assert.equal(o1.exitCode, 1)
         assert.match(o1.message, /exit code: 1/)
 
+        const err = new Error('BrokenSpawn')
         const o2 = await $({
           nothrow: true,
           spawn() {
-            throw new Error('BrokenSpawn')
+            throw err
           },
         })`echo foo`
         assert.equal(o2.ok, false)
         assert.equal(o2.exitCode, null)
         assert.match(o2.message, /BrokenSpawn/)
+        assert.equal(o2.cause, err)
       })
 
       test('handles `input` option', async () => {