Skip to content

docs(ai): MCP integration plan and 22 Aug updates - #553

Merged
cursor[bot] merged 2 commits into
developfrom
docs/ai-mcp-plan
Aug 22, 2026
Merged

docs(ai): MCP integration plan and 22 Aug updates#553
cursor[bot] merged 2 commits into
developfrom
docs/ai-mcp-plan

Conversation

@tomymaritano

Copy link
Copy Markdown
Collaborator

Summary

Documents the MCP / other-AI plan in a new folder docs/ai/ (plan only, no product code).

Does not implement MCP HTTP. Existing packages/mcp-server stays as the current stdio path.

Test plan

  • Read the three files on the PR
  • Confirm no app/CI files changed

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@tomymaritano, you've reached your PR review limit, so we couldn't start this review.

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 @coderabbitai review or push new commits to the PR.

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 configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: df28aa08-03cc-4089-a0d4-4323a45f84bd

📥 Commits

Reviewing files that changed from the base of the PR and between 809a7a3 and 1e814cd.

📒 Files selected for processing (3)
  • docs/ai/README.md
  • docs/ai/mcp-plan.md
  • docs/ai/updates.md

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tomymaritano

Copy link
Copy Markdown
Collaborator Author

Completed the three files with mermaid: architecture, current vs target, AuthGate sequence, Now/Next/Later. Still plan-only; no product code.

@github-actions github-actions Bot added size/L and removed size/M labels Aug 22, 2026
tomymaritano added a commit that referenced this pull request Aug 22, 2026
## 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>
@cursor
cursor Bot enabled auto-merge (squash) August 22, 2026 17:52
@tomymaritano

Copy link
Copy Markdown
Collaborator Author

Rebased onto develop and resolved the add/add conflicts on docs/ai/README.md and docs/ai/updates.md.

develop had independently added both files with a condensed, ADR-linked version of the same 22 Aug decisions. Resolution keeps both:

  • README.mddevelop's ADR table (002–005) stays as the top-level index; this PR's How agents talk to notes section and mermaid diagram are added below it.
  • updates.mddevelop's ADR-linked bullets stay as the canonical summary for 2026-08-22; this PR's long-form sections follow as the reasoning behind them.

Two claims in the long form were superseded by ADRs merged in the meantime, so I corrected them rather than reintroducing contradictions:

  • "MCP … later, not v1" → noted that ADR 004 has since accepted the shape and PR feat(mcp): prefer Local HTTP over opening SQLite #554 is the code.
  • "iOS/iPad stays Later. When sync is real, new repo dripnex/ios" → the repo already exists (ADR 005, mobile bullet above it).

mcp-plan.md was a clean add, untouched. Please skim the two resolved files — the merge of your prose with develop's bullets is a judgment call and yours to overrule.

@cursor
cursor Bot merged commit a2ad6d9 into develop Aug 22, 2026
16 checks passed
@cursor
cursor Bot deleted the docs/ai-mcp-plan branch August 22, 2026 21:00
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 0.17.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant