Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion apps/frontend/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ COPY packages/files/src/project-file-proxy-path.ts packages/files/src/project-fi
WORKDIR /app/apps/frontend
RUN bun run build

FROM node:24-alpine AS runner
FROM node:26-alpine AS runner

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Node 26 is not yet LTS

node:26-alpine is the current "stable" release but has not yet received LTS designation — that is expected in October 2026. Node 24 (krypton) remains the active LTS through April 2028. Upgrading to a non-LTS release in a production runner image means shorter community-tested lifetime and potential exposure to API churn before the release stabilises. Consider waiting for Node 26 to reach LTS, or keeping node:24-alpine until then.

Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/frontend/Dockerfile
Line: 51

Comment:
**Node 26 is not yet LTS**

`node:26-alpine` is the current "stable" release but has not yet received LTS designation — that is expected in October 2026. Node 24 (`krypton`) remains the active LTS through April 2028. Upgrading to a non-LTS release in a production runner image means shorter community-tested lifetime and potential exposure to API churn before the release stabilises. Consider waiting for Node 26 to reach LTS, or keeping `node:24-alpine` until then.

How can I resolve this? If you propose a fix, please make it concise.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Fix in Claude Code Fix in Codex

WORKDIR /app

ARG BUILD_LABEL=
Expand Down