Skip to content

Bump base image to Node 22 and reduce image size#83

Open
nvdk wants to merge 2 commits into
mu-semtech:masterfrom
nvdk:chore/bump-to-node-22
Open

Bump base image to Node 22 and reduce image size#83
nvdk wants to merge 2 commits into
mu-semtech:masterfrom
nvdk:chore/bump-to-node-22

Conversation

@nvdk

@nvdk nvdk commented Jun 17, 2026

Copy link
Copy Markdown
Member

Switches the base image from node:20-bookworm to node:22-bookworm-slim, reducing the image size from ~452 MB to ~168 MB (~3× smaller).

Tested:

  • Production build runs correctly
  • Development mode live reload works correctly (file changes trigger recompile and clean restart without port conflicts)
  • chrome debugger connected correctly

Notes:

  • replaced ps -p check with equivalent kill -0 check (ps is not available in slim image)
  • installs ca-certificates and wget explicitly since those were missing from the slim image

Niels V added 2 commits June 17, 2026 15:15
This also switches to bookworm-slim for 3x smaller image.
replaced `ps -p` check with equivalent `kill -0` check
(ps is not available in slim image)
@nvdk

nvdk commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

Note: I think we should also bump typescript to 5.x but I kept it out of this PR for simplicity

@erikap

erikap commented Jun 17, 2026

Copy link
Copy Markdown
Member

We definitely need to test the integration with the Node inspector, as that's why we switched from alpine version to bookworm in the past. See commit aa69f40

@nvdk

nvdk commented Jun 18, 2026

Copy link
Copy Markdown
Member Author

We definitely need to test the integration with the Node inspector, as that's why we switched from alpine version to bookworm in the past. See commit aa69f40

Yes I was aware :). This is why I kept bookworm, but switched to bookworm-slim. In my testing the node inspector now works correctly, but extra testing is always welcome 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants