Skip to content

Services Health Check Initial Implementation#174

Merged
lbedner merged 1 commit intomainfrom
services-health-check
Sep 18, 2025
Merged

Services Health Check Initial Implementation#174
lbedner merged 1 commit intomainfrom
services-health-check

Conversation

@lbedner
Copy link
Copy Markdown
Owner

@lbedner lbedner commented Sep 18, 2025

Screenshot 2025-09-18 at 11 05 15 AM

Copilot AI review requested due to automatic review settings September 18, 2025 15:08
@lbedner lbedner self-assigned this Sep 18, 2025
Copy link
Copy Markdown

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 introduces a comprehensive services health check system that enables monitoring and management of business logic services (like authentication) alongside existing component health checks. The implementation adds service-specific health monitoring with dedicated UI cards and integration into the existing health check architecture.

  • Adds service health check registration system separate from component health checks
  • Implements authentication service health monitoring with JWT configuration validation
  • Expands UI to display service cards with service-specific metrics and status indicators

Reviewed Changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tests/cli/test_services_cli.py Updates test mock to include database confirmation for auth service
tests/services/*.py Updates all health test files to support new grouped component structure
app/services/system/models.py Adds service-specific properties and flattens component hierarchy recursively
app/services/system/health.py.j2 Implements service health check registry and grouped component structure
app/services/auth/health.py New authentication service health check implementation
app/components/frontend/*.py Adds service card components and updates UI to handle grouped structure
app/components/backend/startup/component_health.py.j2 Registers service health checks during startup
app/components/backend/api/health.py Exposes service information in health API responses
aegis/commands/init.py Improves service dependency resolution for interactive and non-interactive modes
aegis/cli/interactive.py Adds database confirmation flow for auth service selection

Comment thread aegis/commands/init.py Outdated
@lbedner lbedner linked an issue Sep 18, 2025 that may be closed by this pull request
10 tasks
@lbedner lbedner force-pushed the services-health-check branch from 2480f3d to 51b8da0 Compare September 18, 2025 15:21
@lbedner lbedner merged commit 7fde038 into main Sep 18, 2025
7 checks passed
@lbedner lbedner deleted the services-health-check branch September 18, 2025 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

2 participants