fix(smoke): remap supabase-realtime port 4000→4010 + widen grep window - #868
Conversation
Avoid port collision with tensorzero-ui (port 4000). Increase grep_context from 30→55 lines in realtime tests to capture healthcheck block below long env sections. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThe changes update the supabase-realtime service host port mapping from 4000 to 4010 in Docker Compose configuration and adjust test grep search windows from 30 to 55 lines to account for the configuration shift. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Docker Hardening ValidationHardening Validation ReportValidated: Wed Mar 11 18:36:37 UTC 2026Services CheckedPMOVES.AI Docker Hardening Validation[INFO] Checking: pmoves/docker-compose.hardened.yml [INFO] Validating: hi-rag-gateway-v2 [INFO] Validating: extract-worker [INFO] Validating: langextract [INFO] Validating: presign [INFO] Validating: render-webhook [INFO] Validating: retrieval-eval [INFO] Validating: pdf-ingest [INFO] Validating: jellyfin-bridge [INFO] Validating: invidious-companion-proxy [INFO] Validating: ffmpeg-whisper [INFO] Validating: media-video [INFO] Validating: media-audio [INFO] Validating: hi-rag-gateway-v2-gpu [INFO] Validating: hi-rag-gateway-gpu [INFO] Validating: deepresearch [INFO] Validating: supaserch [INFO] Validating: publisher-discord [INFO] Validating: mesh-agent [INFO] Validating: nats-echo-req [INFO] Validating: nats-echo-res [INFO] Validating: publisher [INFO] Validating: analysis-echo [INFO] Validating: graph-linker [INFO] Validating: comfy-watcher [INFO] Validating: grayjay-plugin-host [INFO] Validating: agent-zero [INFO] Validating: archon [INFO] Validating: channel-monitor [INFO] Validating: pmoves-yt [INFO] Validating: notebook-sync [INFO] Validating: supabase_service_role_key [INFO] Validating: supabase_jwt_secret ====================================== |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@pmoves/docker-compose.yml`:
- Line 712: The docker-compose mapping uses ${SUPABASE_REALTIME_PORT:-4010}:4000
which falls back to 4010, but the env files pmoves/env.supabase and
pmoves/env.tier-supabase.example still set SUPABASE_REALTIME_PORT=4000 and thus
override the fallback; update the SUPABASE_REALTIME_PORT default in both
pmoves/env.supabase (line ~167) and pmoves/env.tier-supabase.example (line ~114)
from 4000 to 4010 so the env-driven local setup no longer collides with
tensorzero-ui and preserves cross-platform bring-up behavior.
In `@pmoves/tests/smoke/test_supabase_realtime_tenant.py`:
- Line 135: The tests currently default SUPABASE_REALTIME_URL to
ws://localhost:4000/socket/websocket which, after the compose remap, resolves to
the wrong service; update the module-level default value of
SUPABASE_REALTIME_URL (the constant used by test_supabase_realtime_tenant.py and
referenced by tests test_supabase_realtime_http_accessible and
test_realtime_websocket_upgrade) to use the explicit compose hostname (e.g.,
ws://tensorzero-ui:4000/socket/websocket) and change the other occurrence noted
around the later block (the second default at the later lines) so both places
use the new host so the websocket tests target Realtime reliably.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 927c6919-07d9-4dff-945c-6d47ca91a5eb
📒 Files selected for processing (2)
pmoves/docker-compose.ymlpmoves/tests/smoke/test_supabase_realtime_tenant.py
Address CodeRabbit review: update SUPABASE_REALTIME_PORT from 4000 to 4010 in env.supabase, env.tier-supabase.example, PORT_REGISTRY, and derive test URL defaults from the port env var. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Docker Hardening ValidationHardening Validation ReportValidated: Wed Mar 11 18:45:22 UTC 2026Services CheckedPMOVES.AI Docker Hardening Validation[INFO] Checking: pmoves/docker-compose.hardened.yml [INFO] Validating: hi-rag-gateway-v2 [INFO] Validating: extract-worker [INFO] Validating: langextract [INFO] Validating: presign [INFO] Validating: render-webhook [INFO] Validating: retrieval-eval [INFO] Validating: pdf-ingest [INFO] Validating: jellyfin-bridge [INFO] Validating: invidious-companion-proxy [INFO] Validating: ffmpeg-whisper [INFO] Validating: media-video [INFO] Validating: media-audio [INFO] Validating: hi-rag-gateway-v2-gpu [INFO] Validating: hi-rag-gateway-gpu [INFO] Validating: deepresearch [INFO] Validating: supaserch [INFO] Validating: publisher-discord [INFO] Validating: mesh-agent [INFO] Validating: nats-echo-req [INFO] Validating: nats-echo-res [INFO] Validating: publisher [INFO] Validating: analysis-echo [INFO] Validating: graph-linker [INFO] Validating: comfy-watcher [INFO] Validating: grayjay-plugin-host [INFO] Validating: agent-zero [INFO] Validating: archon [INFO] Validating: channel-monitor [INFO] Validating: pmoves-yt [INFO] Validating: notebook-sync [INFO] Validating: supabase_service_role_key [INFO] Validating: supabase_jwt_secret ====================================== |
…est) PRs #868/#869 remapped supabase-realtime host port from 4000 to 4010 to avoid TensorZero UI collision, but the port registry table and smoke tests were not fully updated. Changes: - PORT_REGISTRY.md: update Supabase table (4000→4010), mark conflict resolved - test_port_conflicts.py: add test_supabase_realtime_port_changed_from_4000, update DOCUMENTED_CONFLICTS and supabase_standards entries - env.tier-supabase: fix runtime override (gitignored, not in this commit) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…est) (#870) PRs #868/#869 remapped supabase-realtime host port from 4000 to 4010 to avoid TensorZero UI collision, but the port registry table and smoke tests were not fully updated. Changes: - PORT_REGISTRY.md: update Supabase table (4000→4010), mark conflict resolved - test_port_conflicts.py: add test_supabase_realtime_port_changed_from_4000, update DOCUMENTED_CONFLICTS and supabase_standards entries - env.tier-supabase: fix runtime override (gitignored, not in this commit) Co-authored-by: Shaela Bello <slbello@uncg.edu> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
2 participants
Summary
supabase-realtimedefault host port from 4000 to 4010 to avoid collision withtensorzero-ui(port 4000)grep_contextfrom 30→55 lines in realtime smoke tests to capture healthcheck blocks below long env sectionsTest plan
pytest pmoves/tests/smoke/test_supabase_realtime_tenant.pypasses (all 5 tests)pytest pmoves/tests/smoke/test_port_conflicts.py— no port collision on 4000supabase-realtimecontainer starts on port 4010Supersedes smoke-test portion of PR #866.
🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes
Configuration
Tests