From 349fa114222c87910e92c32e93f6de4f8ff1330d Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Wed, 11 Feb 2026 03:57:45 +0000 Subject: [PATCH] fix(scripts): make version injection failure fatal in build-dist.mjs Previously, `scripts/build-dist.mjs` caught errors during version injection and logged them but allowed the build to complete with a success exit code. This masked issues in CI/CD environments (like 4everland) where version injection failed silently, leading to deployments without version hashes. This change modifies `injectVersionInfo` to `process.exit(1)` upon failure, ensuring that any issue with version injection halts the build pipeline and surfaces the error in the logs. Tested locally by simulating injection failure (verified exit code 1) and by running a standard build (verified success and correct injection). Co-authored-by: PR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com> --- scripts/build-dist.mjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/build-dist.mjs b/scripts/build-dist.mjs index 9438345b..947f83cb 100644 --- a/scripts/build-dist.mjs +++ b/scripts/build-dist.mjs @@ -191,7 +191,7 @@ function injectVersionInfo() { } } catch (error) { console.error('Failed to inject version info:', error); - // Continue even if injection fails, as it's not critical for functionality + process.exit(1); } }