Refactor CI/CD workflow for improved deployment

[blebelo]

Summary by CodeRabbit

• Chores
  • Optimized CI/CD pipeline to allow tests and builds to execute in parallel for faster feedback.
  • Removed sensitive environment variables from deployment job configuration.
  • Updated smoke test endpoint targeting.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
moipone	Ready	Preview, Comment	Jun 3, 2026 8:14pm

[coderabbitai]

Walkthrough

The CI/CD workflow configuration receives targeted updates to improve job parallelization, remove unnecessary secrets from environment variables, simplify deployment naming, and adjust health-check verification targeting for smoke tests.

Changes

CI/CD Workflow Configuration

Layer / File(s)	Summary
Test job parallel execution .github/workflows/ci-cd.yml	The test job's needs: build dependency is removed, allowing tests to execute in parallel with the build job instead of sequentially.
Database and deployment configuration .github/workflows/ci-cd.yml	The db_update job's env block removes two secret-backed environment variables (App__S3Prefix and App__CorsOrigins). The deploy job's display name is shortened from "Deploy DLL to EC2" to "Deploy DLL".
Smoke test health check targeting .github/workflows/ci-cd.yml	The smoke-test step's curl endpoint changes from 127.0.0.1:8080 to 0.0.0.0:8080, broadening the network interface used for deployment health verification.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• blebelo/moipone#52: Modifies the same CI/CD workflow including deploy job dependencies and job setup.
• blebelo/moipone#44: Modifies the same workflow file's db_update job environment handling and deploy step curl invocation.
• blebelo/moipone#51: Modifies the same workflow's deploy job naming and smoke-test curl target configuration.

Suggested reviewers

• codespace-cloudmaindesk
• mblebelo

Poem

🐰 A workflow refined, with jobs set free,
No more shall tests for builders wait with glee,
Secrets removed, names simplified so bright,
Health checks now listen to the world's own light,
CI/CD flows, more parallel and light! ✨

🚥 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 'Refactor CI/CD workflow for improved deployment' is partially related to the changeset. It accurately describes refactoring of the CI/CD workflow, but does not highlight the most important specific changes (job dependency removal, environment variable removal, and deployment target changes).
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.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch hotfix/deployJob

---

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

[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 @.github/workflows/ci-cd.yml:
- Around line 113-115: Update the "Smoke test" CI step to connect to the local
server using a loopback address instead of the bind address: replace the curl
target "http://0.0.0.0:8080" in the "Smoke test" run block with
"http://127.0.0.1:8080" (or "http://localhost:8080") so the health check
connects to the server correctly.

🪄 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: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 396b8b7c-010a-4237-b29c-41f50f2563a0

📥 Commits

Reviewing files that changed from the base of the PR and between dcea436 and 7c88831.

📒 Files selected for processing (1)

• .github/workflows/ci-cd.yml

[mblebelo]

All checks passed and conversations resolved ✅

[mblebelo]

All checks passed and conversations resolved ✅