docs: refresh kernel/provider docs after v0.3.0#50
Conversation
- docs/api.md: note Retry-After header (seconds) on 429 responses - AGENTS.md, docs/architecture.md: list src/core/errors.ts in core layout - AGENTS-providers.md: document RateLimitError / AgentNotFoundError contract providers must throw, with kernel-reaction table and pointer to the toRateLimitError helpers in the Discord and Slack adapters - AGENTS-providers.md: drop stale "169 tests at time of writing" figure Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThis PR documents the typed error handling pattern for the kernel and providers. It adds architectural references to the error module, specifies provider contracts for throwing ChangesTyped Error Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
Summary
Post-v0.3.0 doc sync — picks up the contract surface added by the four merged PRs (#46–#49) that the docs didn't follow.
docs/api.md— 429 row now notes theRetry-Afterheader (seconds) added in fix(bridge): provider-agnostic rate-limit / not-found errors + hardening #47.AGENTS.md+docs/architecture.md— core file lists now includesrc/core/errors.ts.AGENTS-providers.md— new "Typed errors providers throw" subsection under theBridgeProvidercontract, with a kernel-reaction table forRateLimitError/AgentNotFoundErrorand a pointer to thetoRateLimitErrorhelpers in the Discord/Slack adapters. Stale "169 tests at time of writing" figure dropped (current count is 233 and would just keep drifting).No code changes; doc-only.
Test plan
git diff mainshows only the four expected files (+15 / −2)src/Summary by CodeRabbit