docs(direction): add community-providers policy section#1736
Conversation
Capture the acceptance criteria and maintenance policy for community providers in direction.md so PR triage stops devolving into ad-hoc 'should this match Pi or not' debates. Policy in brief: - Coding-agent SDK required (no raw chat.completions wrappers — Pi already covers ~20 LLM backends via one harness) - Match the Pi pattern: provider class + options translator + event bridge + capability matrix, registered with builtIn: false, tests at parity with the Pi suite, docs page in ai-assistants.md - No cap on acceptance - Contributor + community maintain; non-functional providers get deprecated and removed in the next minor unless someone fixes them Cite as direction.md §community-providers when triaging.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe PR adds a new "Community providers" section to the maintainer direction document, establishing acceptance criteria for community AI coding-agent providers including SDK-only scope, Pi pattern alignment requirements, parity tests, documentation standards, and a maintenance/deprecation policy with PR citation guidance. ChangesCommunity Providers Policy
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
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
Label Snapshot
Test plan
No tests — this is a maintainer-facing direction doc.
Security Impact
Compatibility / Migration
Human Verification
Side Effects / Blast Radius
Rollback Plan
Risks and Mitigations
Summary by CodeRabbit