Chore/deepresearch health - #234
Merged
Merged
Conversation
added 6 commits
November 6, 2025 12:15
… expose port and document
…esearch in up-agents-ui
…g probe; CI matrix + local multi-arch; docs + make targets
Contributor
|
Caution Review failedThe pull request is closed. WalkthroughThe PR introduces DeepResearch and SupaSerch services to the PMOVES architecture, adding multi-architecture Docker build support, health check endpoints, Docker Compose configurations, monitoring dashboards, Prometheus scrape jobs, CI/CD pipeline entries, and supporting documentation and smoke tests. Changes
Sequence DiagramsequenceDiagram
participant ext as External Monitor
participant worker as DeepResearch Worker
participant nats as NATS
participant app as SupaSerch App
participant prom as Prometheus
rect rgb(200, 220, 255)
Note over worker: Worker Startup
worker->>nats: connect()
nats-->>worker: connected
worker->>worker: create FastAPI health app
worker->>worker: asyncio.create_task(_serve_health)
worker->>worker: run uvicorn on :8098
end
rect rgb(220, 255, 220)
Note over ext,prom: Health Check Loop
prom->>ext: GET /healthz (via blackbox)
ext->>worker: GET :8098/healthz
worker-->>ext: {status, nats_connected, ...}
ext-->>prom: 200 OK / healthz response
prom->>app: GET :8099/healthz (via blackbox)
app-->>prom: {status, service: supaserch, ...}
end
rect rgb(255, 240, 220)
Note over nats,app: SupaSerch Search Request
app->>nats: subscribe(RESULT_SUBJECT)
nats-->>app: subscribed
app->>nats: publish(REQUEST_SUBJECT, envelope)
nats->>worker: message received
worker->>nats: publish(RESULT_SUBJECT, result)
nats-->>app: result with correlation_id
app-->>app: match & return
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes
Areas requiring extra attention:
Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (15)
Note Free review on us!CodeRabbit is offering free reviews until Fri Nov 07 2025 to showcase some of the refinements we've made. Comment |
POWERFULMOVES
added a commit
that referenced
this pull request
Jan 18, 2026
Chore/deepresearch health
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Branded images + SUPASERCH scaffolding are in, and I updated CI, monitoring, and docs. Summary and what’s next:
What I implemented
and CI to use context pmoves and Dockerfile services/deepresearch/Dockerfile.
How to build branded images right now (local)
Ensure env creds are loaded from pmoves/env.shared:
Pin in env.shared after push
Set:
Bring up + smokes
Summary by CodeRabbit
Release Notes
New Features
Chores