fix Dockerfile

[DMarinhoCodacy]

No description provided.

[codacy-production]

Up to standards ✅

🟢 Issues 1 medium

Results:
1 new issue

Category	Results
BestPractice	1 medium

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

_{TIP This summary will be updated as you push new changes.}

[gemini-code-assist]

Code Review

This pull request updates the .gitignore file, removes the --static flag from build.sbt, deletes numerous documentation files, and significantly refactors the Dockerfile by switching the base image from Alpine to Debian-based php:8.5-cli (with corresponding package manager and user setup changes). Feedback on the changes suggests keeping the .sbt and .ivy2/cache cache mounts in the Dockerfile to optimize build times, and simplifying the redundant and relative COPY commands used for copying documentation.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[codacy-production]

Pull Request Overview

This PR introduces significant architectural changes by migrating the Docker base image from Alpine to Debian and transitioning to build-time documentation generation. Codacy analysis indicates the changes are currently not up to standards due to package management issues in the new Debian environment.

The primary technical concerns involve redundant and potentially incorrect COPY instructions in the doc-generator stage and the loss of build reproducibility due to unpinned package versions. Additionally, the removal of static linking for the 'codacy-codesniffer' binary could impact portability. Given the scale of these changes—including the deletion of nearly 400 documentation files—the lack of a PR description is a notable oversight that complicates the validation of the intended build behavior.

About this PR

• The PR lacks a description despite making significant architectural changes, such as switching the OS distribution from Alpine to Debian and removing static linking. This makes it difficult to assess the impact of these changes.
• Nearly 400 documentation files were deleted. While the Dockerfile suggests these are now generated during build-time, the lack of explicit confirmation in the PR metadata makes this a high-impact change to verify.

Test suggestions

• Verify that the 'codacy-codesniffer' binary, now dynamically linked, executes correctly within the Debian php:8.5-cli environment.
• Verify that the documentation generation process in the 'doc-generator' stage successfully produces the expected files in the final image at /docs.
• Verify that the 'docker' user created with 'useradd' has correct ownership and permissions for the /app and /docs directories.

Prompt proposal for missing tests

Consider implementing these tests if applicable:
1. Verify that the 'codacy-codesniffer' binary, now dynamically linked, executes correctly within the Debian php:8.5-cli environment.
2. Verify that the documentation generation process in the 'doc-generator' stage successfully produces the expected files in the final image at /docs.
3. Verify that the 'docker' user created with 'useradd' has correct ownership and permissions for the /app and /docs directories.

Low confidence findings

• The removal of the '--static' flag in 'build.sbt' may impact the portability of the generated 'codacy-codesniffer' binary if it is ever required to run outside of this specific Debian environment.

_{TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback}