Skip to content

docs(flutter): fix stale plugin name in pubspec dependency snippet#719

Merged
moven0831 merged 1 commit into
mainfrom
claude/mopro-flutter-bindings-path-b1ox36
Jul 11, 2026
Merged

docs(flutter): fix stale plugin name in pubspec dependency snippet#719
moven0831 merged 1 commit into
mainfrom
claude/mopro-flutter-bindings-path-b1ox36

Conversation

@moven0831

@moven0831 moven0831 commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Problem

Section 2-1 of the Flutter setup guide (docs/docs/setup/flutter-setup.md) tells users to add a pubspec.yaml dependency named mopro_flutter_plugin at path: ./mopro_flutter_plugin:

mopro_flutter_plugin:
    path: ./mopro_flutter_plugin

But the folder users actually copy in step 1 — and that's referenced everywhere else on the page — is mopro_flutter_bindings:

  • Step 1 copies the mopro_flutter_bindings folder
  • Imports use package:mopro_flutter_bindings/...
  • "What's next" references mopro_flutter_bindings

The mopro_flutter_plugin name is a leftover from the old, deprecated plugin-based flow. Since the Dart package is named mopro_flutter_bindings, this dependency entry would not resolve. The Mopro CLI that scaffolds these apps also writes path: ./mopro_flutter_bindings (cli/src/create/flutter.rs), confirming the intended value.

Fix

Update the section 2-1 snippet to match:

mopro_flutter_bindings:
    path: ./mopro_flutter_bindings

This was the only occurrence of mopro_flutter_plugin in the current docs. The versioned snapshots (versioned_docs/version-0.1, version-0.2) legitimately use mopro_flutter_plugin throughout for their era and are intentionally left untouched.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Smkp7CuhA7VjLwriicL12a


Generated by Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the Flutter setup guide to use the local mopro_flutter_bindings dependency in the pubspec.yaml example.

Section 2-1 of the Flutter setup guide told users to add a
`mopro_flutter_plugin` dependency at `path: ./mopro_flutter_plugin`, but
the folder copied in step 1 (and used everywhere else in the page, in the
imports, and by the Mopro CLI scaffolding) is `mopro_flutter_bindings`.
This leftover from the old plugin-based flow would fail to resolve. Update
the snippet to `mopro_flutter_bindings` / `path: ./mopro_flutter_bindings`.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Smkp7CuhA7VjLwriicL12a
@coderabbitai

coderabbitai Bot commented Jul 11, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 035ba9b3-2e6e-4fd9-923e-a2f2c7a128ee

📥 Commits

Reviewing files that changed from the base of the PR and between d7ed9f6 and 9fc1a45.

📒 Files selected for processing (1)
  • docs/docs/setup/flutter-setup.md

📝 Walkthrough

Walkthrough

The Flutter setup documentation updates its pubspec.yaml example to use the local mopro_flutter_bindings dependency path instead of mopro_flutter_plugin.

Changes

Flutter setup

Layer / File(s) Summary
Local Flutter dependency example
docs/docs/setup/flutter-setup.md
The dependency snippet now references mopro_flutter_bindings through path: ./mopro_flutter_bindings.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the doc-only Flutter setup change and describes the stale dependency rename in the pubspec snippet.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/mopro-flutter-bindings-path-b1ox36

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.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mopro with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9fc1a45
Status: ✅  Deploy successful!
Preview URL: https://03a8e8a5.mopro.pages.dev
Branch Preview URL: https://claude-mopro-flutter-binding.mopro.pages.dev

View logs

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
mopro 9fc1a45 Jul 11 2026, 03:41 AM

@moven0831
moven0831 merged commit 75efd15 into main Jul 11, 2026
29 of 31 checks passed
@moven0831
moven0831 deleted the claude/mopro-flutter-bindings-path-b1ox36 branch July 11, 2026 16:57
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