docs: staleness sweep (fork drop, Hermes, synced models, memory model)#587
Conversation
…l management, system-wide memory model)
|
Warning Review limit reached
More reviews will be available in 16 minutes and 9 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
✨ 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 |
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (5 files)
|
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (5 files)
Reviewed by nemotron-3-super-120b-a12b-20230311:free · 200,871 tokens |
Summary
openclaw_acp_runtime.py), notjaylfc/openclawfork +taos-bridge.ts. Clarified that the bridge SSE endpoints survive for Hermes. Fixed Node version note (22.19 minimum, not 22.14). Marked §7 Step 1a install snippet as superseded.permitted_models,GET/PUT /api/agents/{name}/permitted-models,GET/POST /api/agents/me/models|model, andFrameworkModelReconcilerinframework_model_sync.py) and "Hermes environment variables" (documents the full.envset written byinstall_hermes.sh, including the requiredAPI_SERVER_KEY). Extended Related code list.tinyagentos.local→taos.local(matchestinyagentos/services/mdns_publisher.py).server.py, pip-installed FastAPI) with accurate description of the npm-based upstream install.FLAGGED — claims I suspect are stale but could not fully verify
docs/runbooks/framework-swap.mdtest reference: The runbook referencestests/test_framework_swap.py— I did not verify this test exists or still passes against the snapshot model. May need updating or creation.modelfield direction:tinyagentos/routes/librarian.pyandtaosmd.agents.set_librarian()both still accept a per-agentmodelfield on this branch. The task description says "memory model is SYSTEM-WIDE, not per-agent" — the system-wide/api/memory/modelendpoint is NOT present ondev; it may be on a separate unmerged branch. I documented the current state accurately and added a note in the Related code list, rather than documenting a non-existent API.docs/design/user-memory.md: References a QMD-based architecture (per-userport 7833, per-agentport 7832) that may be superseded by taosmd. No code contradiction found on this branch so left untouched — flagged for a future pass.docs/runbooks/framework-swap.mdtaos agentCLI: The procedure now usesincus execdirectly because I could not confirmtaos agent stop/undeploy/deployCLI commands exist. If that CLI is live, the procedure could be simplified.package.jsonclaim of>=22.14.0was not re-fetched from upstream; only the taOSinstall.shcheck was verified. The upstream minimum may have changed since April 2026.