docs(ai): MCP integration plan and 22 Aug updates - #553
Conversation
|
Warning Review limit reached
Next review available in: 45 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
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 |
|
Completed the three files with mermaid: architecture, current vs target, AuthGate sequence, Now/Next/Later. Still plan-only; no product code. |
## Summary Writes the 22 Aug product decisions into `docs/` (not chat). Mobile is **started** (Tomás said go). Repo [`dripnex/ios`](https://github.com/dripnex/ios) exists. No iOS code in this repo. - ADRs 002–005: AuthGate stays, SQLite not CouchDB, MCP over Local HTTP, mobile is `dripnex/ios` - `docs/mobile/PLAN.md`: P1 skeleton in dripnex/ios. Two-profile desktop sync is a P3 risk, not a P1 blocker. - `docs/mobile/CONTRACT.md`: iOS copies `NoteSnapshot` + `NoteStatus` + `sync-core` wire. Do not invent fields. - `docs/ai/README.md` + `updates.md` log - Prettier table padding (same lint that failed on #553) Points at #551. Closes nothing. Prefer this over #556 (duplicate, closed). ## Test plan - [ ] CI lint green (do not merge if red) - [ ] Docs render on GitHub (mermaid in PLAN.md) - [ ] No app/code changes --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com>
7a6616a to
1e814cd
Compare
|
Rebased onto
Two claims in the long form were superseded by ADRs merged in the meantime, so I corrected them rather than reintroducing contradictions:
|
|
🎉 This PR is included in version 0.17.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
Documents the MCP / other-AI plan in a new folder
docs/ai/(plan only, no product code).docs/ai/README.mdindexdocs/ai/mcp-plan.mdInkdrop shape: local HTTP after AuthGate + one MCP; Grok Bot / Claude / Cursor share itdocs/ai/updates.md22 Aug decisions (AuthGate stays, test(desktop): type into CodeMirror in Playwright e2e #541/ci: rewrite git SSH to HTTPS so Dependabot e2e can install #552, Project 3, no CouchDB, mobile Later)Does not implement MCP HTTP. Existing
packages/mcp-serverstays as the current stdio path.Test plan