Skip to content

Enhanced Health Checks#27

Merged
lbedner merged 1 commit intomainfrom
enhanced-health-checks
Aug 27, 2025
Merged

Enhanced Health Checks#27
lbedner merged 1 commit intomainfrom
enhanced-health-checks

Conversation

@lbedner
Copy link
Copy Markdown
Owner

@lbedner lbedner commented Aug 27, 2025

Enhanced database health checks

Screenshot 2025-08-26 at 11 09 54 PM

Copilot AI review requested due to automatic review settings August 27, 2025 03:12
@lbedner lbedner self-assigned this Aug 27, 2025
@lbedner lbedner added this to the Add DB Component milestone Aug 27, 2025
@lbedner lbedner linked an issue Aug 27, 2025 that may be closed by this pull request
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 enhances database health checks by adding comprehensive metadata collection for SQLite databases, including version information, file size, connection pool details, and PRAGMA settings. The changes also improve the frontend display to show this enhanced database information.

  • Enhanced database health check service to collect SQLite version, file size, connection pool size, and PRAGMA settings
  • Added utility function format_bytes for human-readable file size formatting
  • Updated frontend to display detailed database information in the health dashboard

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
app/services/system/health.py.j2 Added enhanced database metadata collection including SQLite version, file size, and PRAGMA settings
app/components/frontend/main.py Enhanced frontend display to show detailed database information in health dashboard
tests/services/test_health_logic.py.j2 Added comprehensive tests for new utility function and enhanced database health check metadata
tests/services/test_component_integration.py.j2 Fixed conditional imports to only include health check functions when corresponding features are enabled
tests/conftest.py.j2 Minor variable naming improvement and comment cleanup

@lbedner lbedner force-pushed the enhanced-health-checks branch from cd18f6a to d684e2c Compare August 27, 2025 03:26
@lbedner lbedner merged commit 3973bf9 into main Aug 27, 2025
6 checks passed
@lbedner lbedner deleted the enhanced-health-checks branch August 27, 2025 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhanced Database Health Check Metadata Display

2 participants