From 71d280d570ea77c9330942722b0e34383de831d9 Mon Sep 17 00:00:00 2001 From: Dax Raad Date: Thu, 29 Jan 2026 23:10:50 -0500 Subject: [PATCH] ci: fix container build script Invoke docker build with Bun shell so commands run correctly, and document default automation behavior. --- AGENTS.md | 1 + packages/containers/script/build.ts | 9 ++++++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index c3f8e50d05..8cfe768da3 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -1,6 +1,7 @@ - To regenerate the JavaScript SDK, run `./packages/sdk/js/script/build.ts`. - ALWAYS USE PARALLEL TOOLS WHEN APPLICABLE. - The default branch in this repo is `dev`. +- Prefer automation: execute requested actions without confirmation unless blocked by missing info or safety/irreversibility. ## Style Guide diff --git a/packages/containers/script/build.ts b/packages/containers/script/build.ts index 18ae328e1c..8fd6a7d4db 100644 --- a/packages/containers/script/build.ts +++ b/packages/containers/script/build.ts @@ -15,9 +15,12 @@ for (const name of images) { const image = `${reg}/build/${name}:${tag}` const file = `packages/containers/${name}/Dockerfile` const arg = name === "base" ? "" : `--build-arg REGISTRY=${reg}` - const cmd = `docker build -f ${file} -t ${image} ${arg} .` - console.log(cmd) - await $`${cmd}` + console.log(`docker build -f ${file} -t ${image} ${arg} .`) + if (arg) { + await $`docker build -f ${file} -t ${image} --build-arg REGISTRY=${reg} .` + } else { + await $`docker build -f ${file} -t ${image} .` + } if (push) { await $`docker push ${image}`