Remove namespaced command aliases; keep canonical commands only#24
Remove namespaced command aliases; keep canonical commands only#24
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c8c5c00644
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - Added external backend API integration workflow: | ||
| - command: `commands/integrate-mobile-api.md` | ||
| - skill: `skills/integrate-mobile-api/SKILL.md` | ||
| - Removed `flutter:*` command aliases and kept canonical commands only. |
There was a problem hiding this comment.
Bump major version after removing command aliases
Removing all flutter:* command files is a breaking API change for users invoking those commands, but this change does not include a major version bump in plugin.json/.cursor-plugin/plugin.json and the added changelog bullets do not mark the change as breaking. In this repo, .github/workflows/api-stability.yml runs .github/scripts/api_stability_guard.sh, which explicitly treats command deletions as breaking and requires both a major bump and a CHANGELOG.md addition containing the word breaking, so this commit will fail that guard and does not provide proper semver signaling.
Useful? React with 👍 / 👎.
Summary\n- remove all namespaced alias commands\n- remove alias maintenance artifacts (, )\n- update docs to reflect canonical-command-only model\n\n## Validation\n- PASS | A1 flutter-app-builder output expectations
PASS | A2 flutter-test-writer output expectations
PASS | A3 flutter-code-reviewer output expectations
PASS | A4 flutter-mobile-release-manager output expectations
PASS | S-build-flutter-features has workflow
PASS | S-build-flutter-features has output section
PASS | S-debug-flutter-issues has workflow
PASS | S-debug-flutter-issues has output section
PASS | S-integrate-firebase has workflow
PASS | S-integrate-firebase has output section
PASS | S-integrate-mobile-api has workflow
PASS | S-integrate-mobile-api has output section
PASS | S-migrate-flutter-code has workflow
PASS | S-migrate-flutter-code has output section
PASS | S-release-mobile-apps has workflow
PASS | S-release-mobile-apps has output section
PASS | S-review-flutter-code has workflow
PASS | S-review-flutter-code has output section
PASS | S-scaffold-flutter-architecture has workflow
PASS | S-scaffold-flutter-architecture has output section
PASS | S-security-audit has workflow
PASS | S-security-audit has output section
PASS | S-setup-flutter-environment has workflow
PASS | S-setup-flutter-environment has output section
PASS | S-sync-official-flutter-ai-rules has workflow
PASS | S-sync-official-flutter-ai-rules has output section
PASS | S-update-flutter-dependencies has workflow
PASS | S-update-flutter-dependencies has output section
PASS | S-write-flutter-tests has workflow
PASS | S-write-flutter-tests has output section
SUMMARY | total=30 passed=30 failed=0 (30/30 pass)\n- PASS | C-implement-flutter-feature.md has command name frontmatter
PASS | C-implement-flutter-feature.md has command description frontmatter
PASS | C-implement-figma-screen.md has command name frontmatter
PASS | C-implement-figma-screen.md has command description frontmatter
PASS | C-generate-flutter-tests.md has command name frontmatter
PASS | C-generate-flutter-tests.md has command description frontmatter
PASS | C-review-flutter-code.md has command name frontmatter
PASS | C-review-flutter-code.md has command description frontmatter
PASS | C-security-review.md has command name frontmatter
PASS | C-security-review.md has command description frontmatter
PASS | C-update-flutter-dependencies.md has command name frontmatter
PASS | C-update-flutter-dependencies.md has command description frontmatter
PASS | C-resolve-flutter-build-error.md has command name frontmatter
PASS | C-resolve-flutter-build-error.md has command description frontmatter
PASS | C-prepare-mobile-release.md has command name frontmatter
PASS | C-prepare-mobile-release.md has command description frontmatter
PASS | C-integrate-mobile-api.md has command name frontmatter
PASS | C-integrate-mobile-api.md has command description frontmatter
PASS | C-integrate-firebase.md has command name frontmatter
PASS | C-integrate-firebase.md has command description frontmatter
PASS | C-migrate-flutter-code.md has command name frontmatter
PASS | C-migrate-flutter-code.md has command description frontmatter
PASS | C-scaffold-flutter-feature.md has command name frontmatter
PASS | C-scaffold-flutter-feature.md has command description frontmatter
PASS | C-setup-mobile-github-pipeline.md has command name frontmatter
PASS | C-setup-mobile-github-pipeline.md has command description frontmatter
PASS | C-setup-flutter-environment.md has command name frontmatter
PASS | C-setup-flutter-environment.md has command description frontmatter
PASS | C-sync-official-flutter-ai-rules.md has command name frontmatter
PASS | C-sync-official-flutter-ai-rules.md has command description frontmatter
PASS | C-write-widget-test.md has command name frontmatter
PASS | C-write-widget-test.md has command description frontmatter
PASS | C-write-bloc-test.md has command name frontmatter
PASS | C-write-bloc-test.md has command description frontmatter
PASS | C-write-e2e-test.md has command name frontmatter
PASS | C-write-e2e-test.md has command description frontmatter
PASS | S-build-flutter-features has workflow
PASS | S-build-flutter-features has output contract
PASS | S-build-flutter-features has guardrails or scope limits
PASS | S-debug-flutter-issues has workflow
PASS | S-debug-flutter-issues has output contract
PASS | S-debug-flutter-issues has guardrails or scope limits
PASS | S-integrate-firebase has workflow
PASS | S-integrate-firebase has output contract
PASS | S-integrate-firebase has guardrails or scope limits
PASS | S-integrate-mobile-api has workflow
PASS | S-integrate-mobile-api has output contract
PASS | S-integrate-mobile-api has guardrails or scope limits
PASS | S-migrate-flutter-code has workflow
PASS | S-migrate-flutter-code has output contract
PASS | S-migrate-flutter-code has guardrails or scope limits
PASS | S-release-mobile-apps has workflow
PASS | S-release-mobile-apps has output contract
PASS | S-release-mobile-apps has guardrails or scope limits
PASS | S-review-flutter-code has workflow
PASS | S-review-flutter-code has output contract
PASS | S-review-flutter-code has guardrails or scope limits
PASS | S-scaffold-flutter-architecture has workflow
PASS | S-scaffold-flutter-architecture has output contract
PASS | S-scaffold-flutter-architecture has guardrails or scope limits
PASS | S-security-audit has workflow
PASS | S-security-audit has output contract
PASS | S-security-audit has guardrails or scope limits
PASS | S-setup-flutter-environment has workflow
PASS | S-setup-flutter-environment has output contract
PASS | S-setup-flutter-environment has guardrails or scope limits
PASS | S-sync-official-flutter-ai-rules has workflow
PASS | S-sync-official-flutter-ai-rules has output contract
PASS | S-sync-official-flutter-ai-rules has guardrails or scope limits
PASS | S-update-flutter-dependencies has workflow
PASS | S-update-flutter-dependencies has output contract
PASS | S-update-flutter-dependencies has guardrails or scope limits
PASS | S-write-flutter-tests has workflow
PASS | S-write-flutter-tests has output contract
PASS | S-write-flutter-tests has guardrails or scope limits
PASS | A-flutter-app-builder.md has output expectations
PASS | A-flutter-app-builder.md expects validation evidence
PASS | A-flutter-code-reviewer.md has output expectations
PASS | A-flutter-code-reviewer.md expects validation evidence
PASS | A-flutter-mobile-release-manager.md has output expectations
PASS | A-flutter-mobile-release-manager.md expects validation evidence
PASS | A-flutter-test-writer.md has output expectations
PASS | A-flutter-test-writer.md expects validation evidence
PASS | Priority policy rule exists and is alwaysApply
PASS | Priority policy enforces project-first state management
PASS | Priority policy allows Bloc/Riverpod style choices by project context
PASS | Sync script does not patch official state-management text
PASS | Scaffold architecture skill enforces project-first state-management selection
SUMMARY | total=88 passed=88 failed=0 (88/88 pass)\n- command audit confirms no remaining alias files ()