Commit Graph

15 Commits

Author SHA1 Message Date
Kazuki Yamada 37fa9f75fb refactor(docker): Remove redundant npm run build from Dockerfile
The prepare lifecycle script already runs npm run build automatically
during npm ci, making the explicit build step redundant.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-21 21:44:43 +09:00
Kazuki Yamada 71581c13cd fix(core): Change prepack to prepare and use npm prune for dev deps removal
prepack does not reliably run when installing repomix as a git dependency,
causing the server Docker build to fail (lib/ directory missing).
Change to prepare which reliably runs for git dependency installation.

Also change npm ci --omit=dev to npm prune --omit=dev in Dockerfile and CI,
since npm prune does not trigger lifecycle scripts (avoiding prepare failure
when devDependencies are no longer available).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-21 21:34:22 +09:00
Claude 007a886095 fix(docker): Restore original order of npm link and npm ci --omit=dev
The previous commit unnecessarily changed the order of npm link and the
dev dependency removal step. Since npm link only creates a global symlink
to the package directory, it should come before npm ci --omit=dev, matching
the original order when npm prune --omit=dev was used.

https://claude.ai/code/session_01WorTDgSv8c9jvVpa1YsSeo
2026-03-10 14:16:48 +00:00
Claude 3e00d3f9e4 refactor(docker): Replace npm prune --omit=dev with npm ci --omit=dev
Now that `prepare` has been replaced with `prepack` (which only runs on
npm pack/publish), `npm ci --omit=dev` no longer triggers an unwanted
build. This allows a cleaner approach: install all deps, build, then
reinstall with only production dependencies via `npm ci --omit=dev`.

https://claude.ai/code/session_01WorTDgSv8c9jvVpa1YsSeo
2026-03-10 13:54:02 +00:00
yamadashy 749392cf2c fix(docker): Replace deprecated --production flag with --omit=dev
npm 10.x (shipped with Node 22) deprecates the --production flag.
Use --omit=dev instead for both Dockerfile and CI workflow.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-08 14:25:02 +09:00
yamadashy 72f1984ce0 refactor(docker): Use npm prune instead of npm ci for devDeps removal
Replace `npm ci --omit=dev --ignore-scripts` with `npm prune --production`
which is more efficient (removes packages in-place instead of reinstalling)
and does not trigger lifecycle scripts by default.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-08 14:19:37 +09:00
yamadashy 099d96e12d fix(docker): Add --ignore-scripts to prevent prepare script failure
The `prepare` script (`npm run build`) added in #1053 causes the Docker
build to fail because `npm ci --omit=dev` triggers it without
devDependencies (`rimraf`, `tsc`) being available. Adding
`--ignore-scripts` skips the unnecessary rebuild since the build output
already exists from the prior step.

Docker builds have been broken since 2025-12-31 (PR #1053).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-08 14:08:05 +09:00
Viktor Szépe 3ff6202404 Fix EC violations 2025-04-19 20:27:03 +00:00
Kazuki Yamada 071bae06fb feat(docker): Separate RUN commands 2024-12-30 12:45:08 +09:00
Juan Calderon-Perez 17d869bb71 Make sure help menu shows up during build 2024-12-30 12:45:08 +09:00
Kazuki Yamada dbaa95bb39 feat(docker): Optimize Docker image size by streamlining npm commands 2024-12-30 12:45:08 +09:00
Kazuki Yamada 2c7f84d3ed feat(docker): Provide repomix as a globally available tool in the Docker image 2024-12-30 12:45:08 +09:00
Juan Calderon-Perez fda9c5f210 Update Dockerfile 2024-12-30 12:45:08 +09:00
Juan Calderon-Perez 136292fd36 Update Dockerfile 2024-12-30 12:45:08 +09:00
Juan Calderon-Perez 2a0c083433 Add support for Docker 2024-12-30 12:45:08 +09:00