Skip to content

ci: add health endpoint, safety-load mock data, update README + screenshot - #1

Open
OneByJorah wants to merge 1 commit into
mainfrom
pipeline/auto-review-ADSentinel-2026-06-30
Open

ci: add health endpoint, safety-load mock data, update README + screenshot#1
OneByJorah wants to merge 1 commit into
mainfrom
pipeline/auto-review-ADSentinel-2026-06-30

Conversation

@OneByJorah

Copy link
Copy Markdown
Owner

Summary

  • Adds a `/health` endpoint for monitoring and deploy hooks.
  • Makes mock-data loading resilient to missing / malformed JSON files.
  • Loads data via `file`-relative path so cwd moves do not break the app.
  • Replaces README screenshot asset with a live local render.
  • Documented stack manifest and reviewer findings.

Deploy status

  • Before: app crashes if `mock_dc_status.json` is missing; no health endpoint.
  • After: safe defaults on missing data; `/health` returns `{"status":"ok"}`.

Findings fixed

  • ADS-001: explicit error handling for missing/malformed data file.
  • ADS-002: new `/health` endpoint.
  • ADS-003: path-relative data lookup.

Open / needs human review

  • None.

Models used:

  • Writer/subagent: ornith:9b

…nshot

ADS-001, ADS-003: use pathlib/__file__ and explicit error handling
ADS-002: expose /health for monitoring/deploy hooks
Reviewer: captured review_findings.json
Deploy: assets/screenshot.png replaced with local live render
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant