Skip to content

Add GET /health endpoint#11

Merged
sebastientaggart merged 1 commit intomainfrom
feature/health-endpoint
Apr 7, 2026
Merged

Add GET /health endpoint#11
sebastientaggart merged 1 commit intomainfrom
feature/health-endpoint

Conversation

@sebastientaggart
Copy link
Copy Markdown
Member

Adds an unauthenticated GET /health endpoint for monitoring and container orchestration.

Response includes:

  • Service uptime and version
  • WebSocket client count
  • Agent count and per-agent statuses
  • Plugin modules, action count, signal count
  • State store entry count, persist path, and writable check

Closes #6

@sebastientaggart sebastientaggart linked an issue Apr 7, 2026 that may be closed by this pull request
@sebastientaggart
Copy link
Copy Markdown
Member Author

Review Summary

Verdict: APPROVE

Findings

No issues found. Code looks correct and follows project conventions.

  • Health endpoint correctly guards against uninitialized globals with a 503.
  • No secrets, no auth bypass concerns beyond the intentional unauthenticated /health.
  • is_writable and entry_count additions are minimal and well-scoped.
  • Test covers the unauthenticated path and key response fields.

@sebastientaggart sebastientaggart merged commit ae74f9e into main Apr 7, 2026
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.

Add health check endpoint

1 participant