feat: add Alpine image

[pat-s]

fix #1946

[Copilot]

Pull request overview

Adds an Alpine-based Docker image variant for prek (to cover common hook runtime dependencies) and updates docs/CI publishing to support it.

Changes:

• Document two published images (scratch minimal and alpine) in Docker integration docs.
• Extend the root Dockerfile with an alpine:3.23 target that installs common hook dependencies and includes the prek binary.
• Update the Docker build/publish GitHub Actions workflow to build and publish both minimal and alpine image variants for multiple platforms.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
docs/integrations.md	Documents the new ghcr.io/j178/prek-alpine image and clarifies the minimal scratch image section.
Dockerfile	Adds minimal and alpine build targets; Alpine target installs common hook dependencies and copies prek to /usr/local/bin.
.github/workflows/build-docker.yml	Builds/publishes both image variants across linux/amd64 and linux/arm64, and separates digest artifacts per image+platform.

[Copilot]

IMAGE_NAME is set via $GITHUB_ENV in a previous step, but later you reference it via the expression context (${{ env.IMAGE_NAME }}) in with: inputs. Environment variables written to $GITHUB_ENV are available to subsequent run: steps, but they are not reliably available for expression evaluation in later steps, which can cause images:/outputs: to be empty or incorrect. Prefer computing the image name directly in the expression (e.g., conditional on matrix.image), or set it as a step output and reference steps.<id>.outputs.* instead.

[Copilot]

Same issue as in the build job: IMAGE_NAME is set via $GITHUB_ENV, but later used in expression contexts (${{ env.IMAGE_NAME }}) for docker/metadata-action and actions/attest. This can result in publishing/attesting with an empty or wrong image name. Compute IMAGE_NAME via expressions (based on matrix.image) or expose it as a step output and reference steps.<id>.outputs.image_name.

[Copilot]

Because the alpine stage is now the last stage in the Dockerfile, running docker build . without --target will produce the Alpine image rather than the previously default scratch/minimal image. If consumers rely on the default being the minimal image, consider keeping the minimal stage as the final/default stage (e.g., add a final FROM minimal stage at the end, or reorder stages) while still allowing --target alpine for the Alpine variant.

Suggested change

	ENTRYPOINT ["prek"]
	ENTRYPOINT ["prek"]
	FROM minimal

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.25%. Comparing base (de7ac64) to head (ac73e8d).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1949   +/-   ##
=======================================
  Coverage   93.25%   93.25%           
=======================================
  Files         127      127           
  Lines       27988    27988           
=======================================
  Hits        26099    26099           
  Misses       1889     1889           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

[j178]

Sorry for the back and forth here. After thinking more about the long-term maintenance tradeoff, I think the official prek image should stay as the minimal scratch-based image.

That keeps the image contract clear: we ship the prek binary, not a maintained set of hook runtimes and system packages. For CI setups that need git, Python, Node, etc., I think the better path is to copy/install prek into the project’s own base image.

Thanks again for the proposal and the PR. I’m going to close this and #1946 for now.

[pat-s]

Uff, that's quite a bummer.

I didn't mean to replace the scratch image, just add an Alpine one on the side which will save multiple seconds and bandwith in CI on every run. There's almost no effort to build these in addition to the already existing base image.

It's easy to do this in the main source repo. By denying this, you indirectly force many individual's to build the same images and keep up with releases in a dynamic way etc. That is a lot of (unnecessary) overhead.

For CI setups that need git, Python, Node, etc., I think the better path is to copy/install prek into the project’s own base image.

A sophisticated use of prek in CI almost always needs these. The scratch image can't be used out-of-the box in a meaningful way.

[prek-ci-bot]

📦 Cargo Bloat Comparison

Binary size change: +0.00% (25.8 MiB → 25.8 MiB)

Expand for cargo-bloat output

Head Branch Results

 File  .text     Size             Crate Name
 1.3%   2.6% 332.0KiB        aws_lc_sys aws_lc_0_39_1_aes_gcm_encrypt_avx512
 1.3%   2.6% 332.0KiB        aws_lc_sys aws_lc_0_39_1_aes_gcm_decrypt_avx512
 0.3%   0.7%  84.0KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.3%   0.7%  82.0KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.2%   0.5%  60.9KiB             prek? <prek::cli::Command as clap_builder::derive::Subcommand>::augment_subcommands
 0.2%   0.4%  56.8KiB              prek prek::languages::<impl prek::config::Language>::install::{{closure}}
 0.2%   0.4%  53.3KiB annotate_snippets annotate_snippets::renderer::render::render
 0.2%   0.4%  46.8KiB              prek prek::run::{{closure}}
 0.2%   0.3%  41.3KiB              prek prek::cli::run::run::run::{{closure}}
 0.1%   0.3%  33.3KiB             prek? <prek::cli::RunArgs as clap_builder::derive::Args>::augment_args
 0.1%   0.2%  28.0KiB        aws_lc_sys aws_lc_0_39_1_edwards25519_scalarmuldouble_alt
 0.1%   0.2%  28.0KiB             prek? <prek::config::_::<impl serde_core::de::Deserialize for prek::config::Config>::deserialize::__Visitor as serde_core::de::Visitor>::visit_map
 0.1%   0.2%  27.6KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  27.5KiB        aws_lc_sys aws_lc_0_39_1_edwards25519_scalarmuldouble
 0.1%   0.2%  26.9KiB               std core::ptr::drop_in_place<prek::languages::<impl prek::config::Language>::install::{{closure}}>
 0.1%   0.2%  26.3KiB              prek prek::cli::try_repo::try_repo::{{closure}}
 0.1%   0.2%  22.8KiB              prek prek::hooks::meta_hooks::MetaHooks::run::{{closure}}
 0.1%   0.2%  22.5KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  22.3KiB         [Unknown] Lp384_montjscalarmul_alt_p384_montjadd
 0.1%   0.2%  21.5KiB      clap_builder clap_builder::parser::parser::Parser::get_matches_with
41.2%  86.3%  10.6MiB                   And 23718 smaller methods. Use -n N to show more.
47.8% 100.0%  12.3MiB                   .text section size, the file size is 25.8MiB

Base Branch Results

 File  .text     Size             Crate Name
 1.3%   2.6% 332.0KiB        aws_lc_sys aws_lc_0_39_1_aes_gcm_encrypt_avx512
 1.3%   2.6% 332.0KiB        aws_lc_sys aws_lc_0_39_1_aes_gcm_decrypt_avx512
 0.3%   0.7%  88.9KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.3%   0.6%  81.6KiB              prek prek::languages::<impl prek::config::Language>::run::{{closure}}::{{closure}}
 0.3%   0.6%  71.8KiB             prek? <prek::cli::Command as clap_builder::derive::Subcommand>::augment_subcommands
 0.2%   0.5%  56.8KiB              prek prek::languages::<impl prek::config::Language>::install::{{closure}}
 0.2%   0.4%  53.3KiB annotate_snippets annotate_snippets::renderer::render::render
 0.2%   0.4%  46.7KiB              prek prek::run::{{closure}}
 0.2%   0.3%  40.9KiB              prek prek::cli::run::run::run::{{closure}}
 0.1%   0.3%  32.6KiB             prek? <prek::cli::RunArgs as clap_builder::derive::Args>::augment_args
 0.1%   0.2%  30.0KiB             prek? <prek::config::_::<impl serde_core::de::Deserialize for prek::config::Config>::deserialize::__Visitor as serde_core::de::Visitor>::visit_map
 0.1%   0.2%  28.0KiB        aws_lc_sys aws_lc_0_39_1_edwards25519_scalarmuldouble_alt
 0.1%   0.2%  27.6KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  27.5KiB        aws_lc_sys aws_lc_0_39_1_edwards25519_scalarmuldouble
 0.1%   0.2%  27.1KiB               std core::ptr::drop_in_place<prek::languages::<impl prek::config::Language>::install::{{closure}}>
 0.1%   0.2%  26.4KiB              prek prek::cli::try_repo::try_repo::{{closure}}
 0.1%   0.2%  23.2KiB              prek prek::hooks::meta_hooks::MetaHooks::run::{{closure}}
 0.1%   0.2%  22.5KiB      serde_saphyr saphyr_parser_bw::scanner::Scanner<T>::fetch_more_tokens
 0.1%   0.2%  22.3KiB         [Unknown] Lp384_montjscalarmul_alt_p384_montjadd
 0.1%   0.2%  21.5KiB      clap_builder clap_builder::parser::parser::Parser::get_matches_with
41.0%  86.1%  10.6MiB                   And 23756 smaller methods. Use -n N to show more.
47.6% 100.0%  12.3MiB                   .text section size, the file size is 25.8MiB

[Copilot]

Pull request overview

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

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 233a2b871a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

[pat-s]

@j178 Tried to make all automated reviewers happy. Would be great if you could take a look again if this would work for you!

[Timple]

Containerized hooks are also quite common. Would it make sense to add podman here as well?

[j178]

Thanks @pat-s for your work in this PR! I continued the work in #2352 with a revised image and publishing setup, and I’ve added you as a co-author on that commit to credit your contribution. Thanks again!