Skip to content

Commit 4924e94

Browse files
committed
fix(wasm): correct spawn result property access in WASM build scripts
Fix incorrect spawn result access in WASM-related build scripts. Changed result.status to result.code to match @socketsecurity/lib/spawn API. Files fixed: - scripts/wasm/build-unified-wasm.mjs - scripts/wasm/check-rust-toolchain.mjs - scripts/wasm/optimize-embedded-wasm.mjs - scripts/wasm/benchmark-build.mjs - scripts/wasm/convert-codet5.mjs The spawn function returns {code, stdout?, stderr?}, not {status, ...}. Using result.status evaluates to undefined, causing incorrect error handling.
1 parent e72bcb4 commit 4924e94

File tree

5 files changed

+8
-8
lines changed

5 files changed

+8
-8
lines changed

scripts/wasm/benchmark-build.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,8 @@ async function benchmarkBuild(mode) {
6565
stdioString: true,
6666
})
6767

68-
if (result.status !== 0) {
69-
throw new Error(`Build failed with exit code ${result.status}`)
68+
if (result.code !== 0) {
69+
throw new Error(`Build failed with exit code ${result.code}`)
7070
}
7171
} catch (e) {
7272
logger.error(`${mode} build failed:`, e.message)

scripts/wasm/build-unified-wasm.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ async function exec(command, args, options = {}) {
5757
})
5858

5959
return {
60-
code: result.status ?? 0,
60+
code: result.code ?? 0,
6161
stderr: result.stderr ?? '',
6262
stdout: result.stdout ?? '',
6363
}

scripts/wasm/check-rust-toolchain.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ async function exec(command, args, options = {}) {
3030
})
3131

3232
return {
33-
code: result.status ?? 0,
33+
code: result.code ?? 0,
3434
stderr: result.stderr ?? '',
3535
stdout: result.stdout ?? '',
3636
}

scripts/wasm/convert-codet5.mjs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,12 @@ async function exec(command, args, options = {}) {
4242
...options,
4343
})
4444

45-
if (result.status !== 0) {
46-
throw new Error(`Command failed with code ${result.status}`)
45+
if (result.code !== 0) {
46+
throw new Error(`Command failed with code ${result.code}`)
4747
}
4848

4949
return {
50-
code: result.status ?? 0,
50+
code: result.code ?? 0,
5151
stderr: result.stderr ?? '',
5252
stdout: result.stdout ?? '',
5353
}

scripts/wasm/optimize-embedded-wasm.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ async function exec(command, args, options = {}) {
3737
})
3838

3939
return {
40-
code: result.status ?? 0,
40+
code: result.code ?? 0,
4141
stderr: result.stderr ?? '',
4242
stdout: result.stdout ?? '',
4343
}

0 commit comments

Comments
 (0)