Update flutter-navigation for auto_route#5
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the flutter-navigation plugin’s scope and guidance to target auto_route (instead of go_router) across the repo’s plugin listings, manifests, and navigation rule/skill content.
Changes:
- Replace
go_routerreferences withauto_routeacross plugin descriptions (repo README, AGENTS/CLAUDE docs, marketplace entries, plugin manifests). - Rewrite the navigation rule content to describe
auto_routeconcepts (AppRouter, generated routes, guards, AutoTabsRouter). - Update the
flutter-navigation-usageskill and plugin README to reflect the new guidance and terminology.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates the top-level plugin list to describe flutter-navigation as auto_route-based. |
| plugins/flutter-navigation/skills/flutter-navigation-usage/SKILL.md | Updates the usage skill to reference auto_route APIs and concepts. |
| plugins/flutter-navigation/rules/navigation.md | Replaces the rule’s content/globs to match auto_route router + generated routes + tabs/guards guidance. |
| plugins/flutter-navigation/README.md | Updates plugin scope summary to auto_route. |
| plugins/flutter-navigation/cursor-rules/navigation.mdc | Regenerated Cursor rule output reflecting the updated navigation rule. |
| plugins/flutter-navigation/claude-rules/navigation.md | Regenerated Claude rule output reflecting the updated navigation rule. |
| plugins/flutter-navigation/.cursor-plugin/plugin.json | Updates Cursor plugin manifest description/keywords for auto_route. |
| plugins/flutter-navigation/.claude-plugin/plugin.json | Updates Claude plugin manifest description/keywords for auto_route. |
| CLAUDE.md | Updates shipped plugin family description for flutter-navigation. |
| AGENTS.md | Updates shipped plugin list description for flutter-navigation. |
| .cursor-plugin/marketplace.json | Updates marketplace description for flutter-navigation to auto_route. |
| .claude-plugin/marketplace.json | Updates marketplace description for flutter-navigation to auto_route. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
That's actually extremely cool, I am curious how it plays out. It might be a great solution to our brown-field projects.
There was a problem hiding this comment.
I wonder if it would be suitable to add CI step to generate this post-merge, to not bloat PRs 🤔
wzslr321
left a comment
wzslr321
left a comment
There was a problem hiding this comment.
The only issue I see is that now both will still be pulled based on path. "
Especially
**/*_route.dart
Shall we remove globs/paths from navigation-go-router.md, navigation-auto-route.mdc, only leave it in main navigation file, for it to decide which should be pulled?
I will create an issue for it, to not block this PR as it should be addressed carefully, separately.
|
And of course... Thank you for you contribution @PiotrRogulski ❤️ 🎉 #team_go_auto_route |
No description provided.