Skip to content

chore: add code-review-flutter agent skill#217

Merged
tompsota merged 2 commits into
masterfrom
ai-skills
May 13, 2026
Merged

chore: add code-review-flutter agent skill#217
tompsota merged 2 commits into
masterfrom
ai-skills

Conversation

@tompsota
Copy link
Copy Markdown
Member

Summary

  • Adds a project-scoped agent skill at .agents/skills/code-review-flutter/SKILL.md that 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).
  • Adds .pubignore so .agents/ (and any future .cursor/) tooling directories are excluded from pub 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

.pubignore is combined with .gitignore by 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:

  • 0 warnings
  • Neither .agents/ nor .cursor/ appears in the package file listing
  • All lib/, test/, pigeons/, android/, ios/, etc. content is unchanged

Test plan

  • flutter pub publish --dry-run reports 0 warnings and excludes .agents/
  • Open a fresh agent session in this repo and confirm the skill appears in the agent's available-skills list (verified by the maintainer once the branch is checked out locally)
  • Trigger the skill with "do a code review for PR #N" and confirm it follows the workflow defined in SKILL.md

Made with Cursor

tompsota and others added 2 commits May 13, 2026 06:03
… 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>
@tompsota tompsota merged commit 7ca7d88 into master May 13, 2026
4 checks passed
@tompsota tompsota deleted the ai-skills branch May 13, 2026 04:17
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.

1 participant