Conversation
… package Adds a project-scoped agent skill at `.agents/skills/code-review-flutter/` that codifies a strict review standard for the plugin: no hand-edits to generated files, no hardcoded fallbacks in the native layer, single source of truth across Dart/Kotlin/Swift, mandatory documentation of platform-specific features, and tests for new public API. Adds `.pubignore` so the new `.agents/` (and any future `.cursor/`) tooling directories are excluded from `pub publish`. Verified clean via `flutter pub publish --dry-run` — 0 warnings, neither directory appears in the published archive. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.agents/skills/code-review-flutter/SKILL.mdthat codifies a strict review standard for this plugin (no hand-edits to generated files, no hardcoded fallbacks in the native layer, single source of truth across Dart/Kotlin/Swift, identifier and API alignment across all three languages, mandatory documentation of platform-specific features, and tests for new public API)..pubignoreso.agents/(and any future.cursor/) tooling directories are excluded frompub publish.Why
The repository didn't have a written, repo-scoped review standard. The skill captures the conventions already followed in recent reviews so any agent session opened in this repo applies them automatically. Keeping the skill in-repo (rather than personal) means it travels with the codebase and stays in lockstep with the project's structure (
pigeons/,*.g.dart,android/.../generated/,ios/Classes/).Publish-package safety
.pubignoreis combined with.gitignoreby pub, so adding.agents/and.cursor/there is sufficient to keep them out of the published archive while still letting them be tracked in git.Verified with
flutter pub publish --dry-run:.agents/nor.cursor/appears in the package file listinglib/,test/,pigeons/,android/,ios/, etc. content is unchangedTest plan
flutter pub publish --dry-runreports 0 warnings and excludes.agents/SKILL.mdMade with Cursor