docs(ops): align data services and release cadence - #1124
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
📝 WalkthroughWalkthroughThis pull request updates documentation across eight files to establish Release Notes/CVE intake funnel workflows, standardize modular data-plane provisioning guidance (Supabase, Qdrant, Meilisearch, Neo4j, n8n), clarify Supabase runtime selection, and document new operational Make targets and cadence-based review processes. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
pmoves/docs/services/README.md (1)
5-44:⚠️ Potential issue | 🟠 MajorAddress duplicate service entries between "Core data plane" and other sections.
The new "Core data plane" section lists
neo4j(line 9),n8n(line 10), andsupabase(line 44), but these services also appear in later sections:
- Line 33:
neo4jin "Implemented (compose-managed)"- Line 43:
n8nin "Auxiliary / Libraries / Adapters"- Line 44:
supabasein "Auxiliary / Libraries / Adapters"This duplication creates ambiguity about which categorization is authoritative.
Recommended approach:
- Keep services in "Core data plane" only (preferred), OR
- Add a note like
(also listed in Core data plane above)for the duplicate entries, OR- Remove the duplicates from the lower sections if "Core data plane" is the new canonical grouping
As per coding guidelines, the services index should reflect the "core data plane" set without creating parallel/copy runbooks.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@pmoves/docs/services/README.md` around lines 5 - 44, The services index lists neo4j, n8n, and supabase in the "Core data plane" section but repeats them later, causing ambiguity; update pmoves/docs/services/README.md so the "Core data plane" is canonical by removing the duplicate entries for neo4j from "Implemented (compose-managed)" and for n8n and supabase from "Auxiliary / Libraries / Adapters" (or alternatively add “(also listed in Core data plane)” next to those entries if you prefer to keep them), and verify the remaining lists and links (section headings "Core data plane", "Implemented (compose-managed)", "Auxiliary / Libraries / Adapters") remain correctly ordered and formatted.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@pmoves/docs/services/README.md`:
- Around line 5-44: The services index lists neo4j, n8n, and supabase in the
"Core data plane" section but repeats them later, causing ambiguity; update
pmoves/docs/services/README.md so the "Core data plane" is canonical by removing
the duplicate entries for neo4j from "Implemented (compose-managed)" and for n8n
and supabase from "Auxiliary / Libraries / Adapters" (or alternatively add
“(also listed in Core data plane)” next to those entries if you prefer to keep
them), and verify the remaining lists and links (section headings "Core data
plane", "Implemented (compose-managed)", "Auxiliary / Libraries / Adapters")
remain correctly ordered and formatted.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: d7a947fa-8802-4916-8cb0-007e137cd65e
📒 Files selected for processing (8)
docs/hardening/PMOVES-hardening-tracker.mdpmoves/docs/NEXT_STEPS.mdpmoves/docs/PMOVES.AI PLANS/README_DOCS_INDEX.mdpmoves/docs/PMOVES.AI PLANS/ROADMAP.mdpmoves/docs/operations/FIRST_RUN.mdpmoves/docs/operations/MAKE_TARGETS.mdpmoves/docs/services/README.mdpmoves/docs/services/supabase/README.md
…ity hardening Fleet Networking & RustDesk: - KVM2 self-hosted RustDesk relay (hbbs+hbbr, systemd, UFW-locked) - Scripts: fix-kvm2-rustdesk-relay.sh, restart-jetson-rustdesk.sh (env var driven — HOSTINGER_KVM2_IP, RUSTDESK_RELAY_KEY, JETSON_IPS) - RUSTDESK_SELF_HOSTED.md deployment guide (sanitized, no secrets) - QR code generation instructions for mobile enrollment Security Hardening: - Network hardening: localhost defaults for all service bindings - 4090 laptop host hardening script - SSH key-only auth across 8 nodes (password disabled) - Gitignore: rustdesk QR images excluded from repo Infrastructure: - DeepResearch Dockerfile: add pmoves.chit COPY for CGP_SPEC_VERSION (Python files only, secrets manifests excluded from image layers) - TOPOLOGY.md: KVM2 RustDesk ports, updated node hostnames, Jetson status - AGNOTE4482 ACK: Z890-CLAUDE fleet networking claim signed Also includes merged PR content: - Hi-RAG embedding defaults aligned to Qwen3 2560d (#1122) - Pinokio Codex plugin + Agent Zero launcher (#1121) - Publishing approval handoff state (#1120) - TTS MCP bridge + expression registry (#1116) - Damage-control hooks Windows compat (#1123) - AGNOTE4482 fleet claim docs (#1127) - Dependency bumps (#1111, #1113, #1128) - Bootstrap SUPERSEDED_VALUES (#1112) - Various docs updates (#1115, #1117, #1119, #1124, #1125) - Publisher RPC idempotent guards (#1126) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Summary
Testing
Reviewer Notes