WIP: System health diagnostics #1

2026-10-14T10:53:59-05:00

lussierj commented

2026-10-14 10:53:59 -05:00

Source: codex/review-01-system-health
Target: main

Summary

Adds system-health diagnostics, backend health reporting, application metadata, and the corresponding frontend.

This PR establishes the first review boundary in a five-PR dependency stack. It introduces no database migration.

Exact included commits

c3501b9 — feat: add system health diagnostics

Dependencies

No earlier PR in this stack is required. PR 2 targets this source branch.

Available validation evidence

The commit includes system-health unit and PostgreSQL integration tests, frontend-shell coverage, and scripts/test-system-health.sh.

The Git comparison and commit scope were verified. Tests were not rerun during branch preparation, and no isolated passing result for this endpoint is claimed.

Outstanding review concerns

Run the focused health and frontend tests against this exact committed endpoint.
Verify diagnostic permissions, secret redaction, database/Redis failure handling, and migration reporting.
At the final stack endpoint, the committed health integration test still expects migration 0017_agent_enrollment, while the database head is 0020_existing_agent_transition. Resolve and validate that mismatch before final-stack approval.
Startup schema compatibility enforcement remains separate, uncommitted work and is not included here.

Deployment status

This PR is not approved for production deployment. Review boundaries do not establish deployment readiness.

Source: `codex/review-01-system-health` Target: `main` ### Summary Adds system-health diagnostics, backend health reporting, application metadata, and the corresponding frontend. This PR establishes the first review boundary in a five-PR dependency stack. It introduces no database migration. ### Exact included commits - `c3501b9` — feat: add system health diagnostics ### Dependencies No earlier PR in this stack is required. PR 2 targets this source branch. ### Available validation evidence The commit includes system-health unit and PostgreSQL integration tests, frontend-shell coverage, and `scripts/test-system-health.sh`. The Git comparison and commit scope were verified. Tests were not rerun during branch preparation, and no isolated passing result for this endpoint is claimed. ### Outstanding review concerns - Run the focused health and frontend tests against this exact committed endpoint. - Verify diagnostic permissions, secret redaction, database/Redis failure handling, and migration reporting. - At the final stack endpoint, the committed health integration test still expects migration `0017_agent_enrollment`, while the database head is `0020_existing_agent_transition`. Resolve and validate that mismatch before final-stack approval. - Startup schema compatibility enforcement remains separate, uncommitted work and is not included here. ### Deployment status This PR is not approved for production deployment. Review boundaries do not establish deployment readiness.

lussierj added 1 commit 2026-10-14 10:54:00 -05:00

feat: add system health diagnostics c3501b987e

lussierj referenced this issue from a commit

2026-10-19 09:40:18 -05:00

security(discovery): enforce Zone isolation and retire unsafe legacy writers

lussierj referenced this pull request

2026-10-19 09:41:00 -05:00

WIP: Blocker #1 - Agent Discovery Zone Isolation #7

You are not authorized to merge this pull request.

This pull request can be merged automatically.

This branch is out-of-date with the base branch

This pull request is marked as a work in progress.