Aimpress/Axil_Accountants
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: drop --no-require-module from migrator (added in Node 22.12.0)

Browse source 
Exit code 9 from Node.js means "Invalid argument", not SIGKILL/OOM.
--no-require-module was added in Node 22.12.0 — the cached node:22-slim
image on the VPS predates that release so Node rejects the flag immediately.

The minimal migrate.ts doesn't load lexical/form-builder, so there are no
TLA/ESM issues that required the flag in the first place.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Vadym Samoilenko 2026-02-23 21:55:04 +00:00
parent dc1b1efbab
commit 0f8de6479d
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Dockerfile
View file

@ -22,8 +22,11 @@ CMD ["pnpm", "dev"]
FROM base AS migrator
COPY --from=deps /app/node_modules ./node_modules
COPY . .
# Same Node.js native TS-stripping approach used by seed.ts (avoids TLA issues with lexical)
ENV NODE_OPTIONS="--experimental-strip-types --no-require-module"
# --experimental-strip-types: run migrate.ts + migration files as TypeScript natively.
# --no-require-module is intentionally omitted: it was added in Node 22.12.0 and the
# cached base image may be older; our minimal migrate script doesn't load lexical so
# there are no TLA/ESM issues that required that flag.
ENV NODE_OPTIONS="--experimental-strip-types"
CMD ["node", "src/scripts/migrate.ts"]
# --- Build ---