fix: add try/catch around client build and inline-builds to yarn

ludomikula · ludomikula · commit 6b7f01ca60f5 · 2025-06-24T09:07:55.000+02:00
diff --git a/client/scripts/build.js b/client/scripts/build.js
@@ -91,8 +91,12 @@ buildVars.forEach(({ name, defaultValue }) => {
   shell.env[name] = shell.env[name] ?? defaultValue;
 });
 
-shell.exec(`BUILD_TARGET=browserCheck yarn workspace lowcoder build`, { fatal: true });
-shell.exec(`yarn workspace lowcoder build`, { fatal: true });
+try {
+  shell.exec(`BUILD_TARGET=browserCheck yarn workspace lowcoder build`, { fatal: true });
+  shell.exec(`yarn workspace lowcoder build`, { fatal: true });
+} catch(e) {
+  console.error("Command failed: ", e.message);
+}
 
 if (process.env.REACT_APP_BUNDLE_BUILTIN_PLUGIN) {
   for (const pluginName of builtinPlugins) {
diff --git a/deploy/docker/Dockerfile b/deploy/docker/Dockerfile
@@ -68,7 +68,7 @@ RUN apt-get update \
 # Copy and build the node-service app
 COPY server/node-service/ /lowcoder/node-service/app/
 WORKDIR /lowcoder/node-service/app/
-RUN yarn --immutable
+RUN yarn --immutable --inline-builds
 RUN yarn build
 
 # Copy startup script
@@ -118,7 +118,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends curl ca-certifi
 # Build client
 COPY ./client /lowcoder-client
 WORKDIR /lowcoder-client
-RUN yarn --immutable
+RUN yarn --immutable --inline-builds
 
 ARG REACT_APP_COMMIT_ID=test
 ARG REACT_APP_ENV=production
