Remove Debian Docker variant, consolidate to Alpine-only (#231)

[nitrobass24]

Summary

• Remove Dockerfile.alpine and promote Alpine as the sole Dockerfile
• Simplify CI workflow by removing the variant matrix (Debian/Alpine), consolidating artifact names and metadata steps
• Update Makefile shell target from /bin/bash to /bin/sh (Alpine uses ash)
• Add BuildKit cache mount for npm to speed up frontend builds

Closes #231

Test plan

• CI builds the Docker image successfully on both amd64 and arm64
• Container starts and serves the web UI
• Verify image size is ~45 MB (Alpine baseline)
• make shell works correctly with /bin/sh

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Simplified CI/CD pipeline by consolidating build variants into a single, unified configuration
  • Migrated container images from Debian to Alpine Linux for improved efficiency and reduced image size
  • Optimized Docker build caching and removed unnecessary packages to streamline container builds

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: a779a9e9-a4f0-41d1-8def-14b758474aaa

📥 Commits

Reviewing files that changed from the base of the PR and between d8cb576 and c744316.

📒 Files selected for processing (4)

• .github/workflows/ci.yml
• Makefile
• src/docker/build/docker-image/Dockerfile
• src/docker/build/docker-image/Dockerfile.alpine

💤 Files with no reviewable changes (1)

• src/docker/build/docker-image/Dockerfile.alpine

---

📝 Walkthrough

Walkthrough

The pull request consolidates the Docker build pipeline from separate Debian and Alpine variants into a single Alpine-based approach. Changes include removing the Dockerfile.alpine file, simplifying the CI workflow by eliminating matrix strategy configurations, and optimizing the unified Dockerfile with Alpine-specific base images, caching strategies (npm cache mount), and dependency management. The shell in the Makefile dev container is switched from bash to sh.

Changes

Cohort / File(s)	Summary
CI Workflow Consolidation .github/workflows/ci.yml	Removed matrix strategy from build and publish jobs; eliminated per-variant (debian/alpine) artifact naming and metadata extraction steps; simplified artifact handling to architecture-based scopes (amd64/arm64); consolidated build conditions and cache sources; unified publish job naming and cross-job success checks.
Dockerfile Alpine Migration src/docker/build/docker-image/Dockerfile	Replaced Debian-based stages (bookworm-slim) with Alpine equivalents (alpine, node:22-alpine, python:3.12-alpine); added npm cache mount for layer efficiency; implemented aggressive Python cache and stdlib cleanup; switched runtime dependency installation from apt-get to apk; updated HEALTHCHECK from Python urllib to wget.
Removed Debian Variant src/docker/build/docker-image/Dockerfile.alpine	Deleted entire Alpine Dockerfile definition (113 lines); consolidation into main Dockerfile replaces this variant-specific file.
Shell Configuration Makefile	Changed interactive dev container shell from /bin/bash to /bin/sh.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• ci: publish develop images to ghcr.io on push #175 — Adds per-variant develop publish jobs for debian/alpine; directly opposes this PR's consolidation of publish job naming and artifact flows.
• Use pre-built docker-7zip image for RAR support #212 — Also modifies Docker build to copy prebuilt 7zz from Alpine-based image instead of compiling; aligned change in dependency strategy.
• feat: Alpine Docker image + dual-image CI #164 — Previously added the multi-variant matrix and Dockerfile.alpine that this PR removes; represents the prior state being reverted.

Poem

🐰 Farewell Debian, Alpine takes the stage,
One image slim—the future's turning page!
No matrix splits, no variants to keep,
Just alpine roots, lean builds, and lighter sleep. 🏔️

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely describes the primary change: removing the Debian Docker variant and consolidating to Alpine-only, matching the main objective of the PR.
Linked Issues check	✅ Passed	The PR addresses all major coding requirements from issue #231: removes Debian Dockerfile, simplifies CI matrix, updates artifact naming, adds npm cache mount, updates HEALTHCHECK to wget, and updates Makefile shell to /bin/sh.
Out of Scope Changes check	✅ Passed	All changes are directly related to consolidating to Alpine-only and CI optimization. The Makefile shell change, Dockerfile updates, and CI workflow simplification are all necessary to achieve the stated objectives.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/alpine-only-docker

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[nitrobass24]

@CodeRabbit review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.