Skip to content

feat: Add production deployment to CI, include all dependencies in Dockerfile, and introduce a health check API endpoint.#3

Merged
ilramdhan merged 1 commit intomutugading:mainfrom
ilramdhan:feat/integration-with-proto-uom-service
Feb 2, 2026
Merged

feat: Add production deployment to CI, include all dependencies in Dockerfile, and introduce a health check API endpoint.#3
ilramdhan merged 1 commit intomutugading:mainfrom
ilramdhan:feat/integration-with-proto-uom-service

Conversation

@ilramdhan
Copy link
Member

No description provided.

…ckerfile, and introduce a health check API endpoint.
@ilramdhan ilramdhan requested a review from Copilot February 2, 2026 22:13
@ilramdhan ilramdhan self-assigned this Feb 2, 2026
@ilramdhan ilramdhan added the enhancement New feature or request label Feb 2, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds production deployment capabilities to the CI/CD pipeline, ensures complete dependency installation in the Docker build process, and introduces a health check endpoint for container orchestration.

Changes:

  • Added a new health check API endpoint at /api/health for Kubernetes liveness and readiness probes
  • Modified Dockerfile to install all dependencies (including devDependencies) instead of production-only dependencies
  • Added production deployment job to GitHub Actions workflow with ArgoCD integration

Reviewed changes

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

File Description
src/app/api/health/route.ts New health check endpoint returning service status and timestamp
Dockerfile Updated dependency installation to include devDependencies required for build
.github/workflows/ci.yml Added production deployment job that triggers after staging deployment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +9 to +17
export async function GET() {
return NextResponse.json(
{
status: 'healthy',
timestamp: new Date().toISOString()
},
{ status: 200 }
);
}
Copy link

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The GET function is declared as async but doesn't perform any asynchronous operations. Remove the async keyword to avoid unnecessary promise wrapping.

Copilot uses AI. Check for mistakes.
@ilramdhan ilramdhan merged commit eae25bd into mutugading:main Feb 2, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant