Conversation
beimnettes
requested changes
Feb 10, 2026
Contributor
beimnettes
left a comment
There was a problem hiding this comment.
I checked this branch locally and reviewed the changes.
The Docker setup and project structure look correct, and the application starts until required environment variables (such as JWT_SECRET) are needed. Those errors are expected locally and match the PR notes.
However, the CI checks are still failing and the branch is out of date with main. These issues should be resolved before merge.
✅ Full E2E Test Results
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary & Changes 📃
Summary
This PR introduces a Docker-based local development environment that standardizes the application’s infrastructure and workflow. The docker work flow establishes a consistent local development foundation by containerizing the application and routing traffic through a reverse proxy, consolidating the frontend and backend into a single, predictable local entry point.
Replacing the previous multi-port frontend-backend configuration, the application now runs entirely on http://localhost (port 80).
Resolving Issue
The monorepo exhibited cascading failures driven by unclear dependency boundaries and tooling conflicts, resulting in unstable builds and repeated troubleshooting.
This PR resolves dependency boundary and tooling conflict issues by removing the deduplicated and hoisted dependencies and allocating these to their correct execution environments before containerization. The result of these changes is a stable and maintainable development setup that aligns with production-ready deployment patterns.
Note
Follow-up work will continue with securely allocating environment variables into designated password files and
compose.yamlconfigurations.What is the expected behavior?
With these changes in place, developers should be able to spin up the full application locally using a consistent, repeatable workflow. The environment mirrors production-ready patterns while remaining focused on local development, enabling smoother onboarding and preparing the codebase for future cloud deployment work.
Relevant technical details
http://localhost/→ Next.js frontendhttp://localhost/api→ NestJS backendhttp://localhost:8080→ Traefik dashboardhttp://localhost:5050orhttp://db.localhost→ pgAdminChanges
How to Test 🧪
Steps to Reproduce
setup-macos-linux.shorsetup-windows.bat