Remove deprecated Docker configuration files and update README to ref…

[Johnny-Dang]

…lect changes in local development setup. The docker-compose.yml and related .dockerignore and Dockerfile have been deleted as they are no longer needed. The README has been updated to clarify the current structure and usage of docker-compose files.

Summary by CodeRabbit

• Deployment

  • Production deployments now include the production Docker Compose configuration.
  • Deployment scripts validate and install the required production configuration before starting services.

• Documentation

  • Updated project documentation to distinguish local and production Docker Compose configurations.

• Chores

  • Removed obsolete local Docker Compose and backend container configuration files.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
menu-green-system	Ready	Preview, Comment	Jul 12, 2026 8:10am

[coderabbitai]

📝 Walkthrough

Walkthrough

The CD workflow now uploads docker-compose.prod.yml for server deployment. The deploy script installs the uploaded file instead of generating an embedded configuration. README Docker entries were updated, while the backend Dockerfile, local Compose file, and Docker ignore rules were removed.

Changes

Deployment and Docker setup

Layer / File(s)	Summary
Production Compose deployment handoff .github/workflows/backend-cd.yml, backend/scripts/deploy-server.sh	The CD upload includes docker-compose.prod.yml; the deploy script searches supported staging layouts, fails when the file is absent, and copies it into the application directory.
Docker setup documentation and removals README.md, backend/Dockerfile, backend/.dockerignore, docker-compose.yml	The README lists local and production Compose configurations, while the backend Dockerfile, Docker ignore rules, and local Compose stack are removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant GitHubActions
  participant DeployServer
  participant AppDirectory
  GitHubActions->>DeployServer: Upload docker-compose.prod.yml
  DeployServer->>DeployServer: Locate uploaded Compose file
  DeployServer->>AppDirectory: Install docker-compose.prod.yml
  DeployServer->>AppDirectory: Run deployment using installed file

Loading

Possibly related PRs

• EXE201-MenuGreen/MenuGreenSystem#38: Related Dockerization changes involving Compose files and the backend Dockerfile.
• EXE201-MenuGreen/MenuGreenSystem#106: Related CD staging changes under /tmp/nginx-deploy.
• EXE201-MenuGreen/MenuGreenSystem#107: Related deploy-script handling of variable staging directory layouts.

🚥 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 summarizes the main changes: removing deprecated Docker config files and updating the README.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch Tuan

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@backend/scripts/deploy-server.sh`:
- Around line 134-138: Update the diagnostic echo in the COMPOSE_DEPLOYED check
to preserve the literal `source:` text by escaping the backticks or using a
quoting approach that prevents command substitution; keep the existing message
and failure behavior unchanged.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 5a7dff18-84a8-4102-941b-45e99a69001f

📥 Commits

Reviewing files that changed from the base of the PR and between 20273c8 and 15b20ba.

📒 Files selected for processing (6)

• .github/workflows/backend-cd.yml
• README.md
• backend/.dockerignore
• backend/Dockerfile
• backend/scripts/deploy-server.sh
• docker-compose.yml

💤 Files with no reviewable changes (3)

• backend/Dockerfile
• docker-compose.yml
• backend/.dockerignore

[coderabbitai]

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Backticks in double-quoted string cause unintended command substitution.

Line 136 uses backticks around source: inside a double-quoted echo. Bash interprets these as command substitution, so the shell will try to execute source: as a command (which fails), and the backtick expression is replaced with empty output. The error message will print as >>> Check the list in .github/workflows/backend-cd.yml instead of the intended formatted text.

🔧 Fix: escape backticks or use single quotes

-  echo ">>> Check the `source:` list in .github/workflows/backend-cd.yml"
+  echo ">>> Check the 'source:' list in .github/workflows/backend-cd.yml"

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	if [ -z "$COMPOSE_DEPLOYED" ]; then
	echo ">>> FATAL: docker-compose.prod.yml not found under /tmp/nginx-deploy/"
	echo ">>> Check the `source:` list in .github/workflows/backend-cd.yml"
	exit 1
	fi
	if [ -z "$COMPOSE_DEPLOYED" ]; then
	echo ">>> FATAL: docker-compose.prod.yml not found under /tmp/nginx-deploy/"
	echo ">>> Check the 'source:' list in .github/workflows/backend-cd.yml"
	exit 1
	fi

🧰 Tools

🪛 Shellcheck (0.11.0)

[style] 136-136: Use $(...) notation instead of legacy backticks ....

(SC2006)

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@backend/scripts/deploy-server.sh` around lines 134 - 138, Update the
diagnostic echo in the COMPOSE_DEPLOYED check to preserve the literal `source:`
text by escaping the backticks or using a quoting approach that prevents command
substitution; keep the existing message and failure behavior unchanged.

Source: Linters/SAST tools