Skip to content

feat: ✨ Image very big to be correctly updated#627

Merged
robvanderleek merged 2 commits into
mainfrom
issue-626-Image_very_big_to_be_correctly_updated
Jan 17, 2026
Merged

feat: ✨ Image very big to be correctly updated#627
robvanderleek merged 2 commits into
mainfrom
issue-626-Image_very_big_to_be_correctly_updated

Conversation

@robvanderleek

Copy link
Copy Markdown
Owner

No description provided.

@robvanderleek robvanderleek requested a review from Copilot January 17, 2026 08:19
@create-issue-branch create-issue-branch Bot changed the title feat: ✨ Smaller container image feat: ✨ Image very big to be correctly updated Jan 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces the container image size by introducing a multi-stage Docker build process that bundles the application using esbuild before creating the runtime container.

Changes:

  • Introduced esbuild bundling to reduce deployment artifact size
  • Migrated from Yarn to NPM as the package manager
  • Updated Node.js version requirement from 20 to 24

Reviewed changes

Copilot reviewed 6 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
standalone/package.json Added esbuild dependency and build script; updated Node.js engine requirement to 24
standalone/Dockerfile Implemented multi-stage build with esbuild bundling; switched from Yarn to NPM
README.md Updated documentation to reflect NPM usage instead of Yarn
.github/workflows/main.yml Updated Node.js version and package manager in CI workflows
.github/workflows/codelimit.yml Updated checkout action version and removed unnecessary quotes
.github/dependabot.yml Added GitHub Actions ecosystem to dependabot configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread standalone/Dockerfile Outdated
FROM node:20-slim
FROM node:24 AS build-cwa
RUN mkdir /app
COPY package.json package-lock.json app/

Copilot AI Jan 17, 2026

Copy link

Choose a reason for hiding this comment

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

Missing forward slash before 'app/' directory. The COPY command should use '/app/' to correctly reference the absolute path created in line 2.

Suggested change
COPY package.json package-lock.json app/
COPY package.json package-lock.json /app/

Copilot uses AI. Check for mistakes.
@github-actions

github-actions Bot commented Jan 17, 2026

Copy link
Copy Markdown

CodeLimit Report

Overview

Language Files Functions Lines of Code
JavaScript 8 24 299 0 0

Summary

Easy / Verbose Hard-to-maintain ⚠ Unmaintainable ⛌
100% 0% 0%

✅ 100% of the functions are maintainable, no refactoring necessary.

Findings

Function Length File

Generated by CodeLimit

@robvanderleek robvanderleek merged commit 0249e6c into main Jan 17, 2026
3 checks passed
@robvanderleek robvanderleek deleted the issue-626-Image_very_big_to_be_correctly_updated branch January 17, 2026 08:30
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