Skip to content

docs: clarify mcp_dart positioning versus dart_mcp#103

Merged
leehack merged 4 commits into
mainfrom
docs/mcp-dart-positioning
May 12, 2026
Merged

docs: clarify mcp_dart positioning versus dart_mcp#103
leehack merged 4 commits into
mainfrom
docs/mcp-dart-positioning

Conversation

@leehack

@leehack leehack commented May 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add README guidance for choosing between the community mcp_dart SDK and the official dart_mcp package.
  • Position mcp_dart around production-oriented Dart/Flutter MCP servers, clients, transports, auth/security, tooling, and latest protocol compatibility.
  • Remind readers to re-check current pub.dev/docs before making a production decision.

Test Plan

  • Not run (documentation-only README change).

Notes

  • This handles the Obsidian candidate “Document positioning vs official dart_mcp” directly in the repository.

Copilot AI review requested due to automatic review settings May 12, 2026 09:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds README guidance to help users choose between this community mcp_dart SDK and the official dart_mcp package, clarifying intended use cases and encouraging readers to verify current pub.dev/docs before making production decisions.

Changes:

  • Introduces a new “Choosing between mcp_dart and the official dart_mcp package” section in the README.
  • Adds a short comparison table describing when each package is a better fit.
  • Adds a reminder that both packages may evolve quickly and to re-check current releases/docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
@codecov

codecov Bot commented May 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.64%. Comparing base (2d32beb) to head (cab488b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #103   +/-   ##
=======================================
  Coverage   80.64%   80.64%           
=======================================
  Files          43       43           
  Lines        5801     5801           
=======================================
  Hits         4678     4678           
  Misses       1123     1123           
Flag Coverage Δ
unittests 80.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread README.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread README.md Outdated
Comment thread README.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@leehack leehack merged commit 8ef0637 into main May 12, 2026
10 checks passed
@leehack leehack deleted the docs/mcp-dart-positioning branch May 12, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants