Add Appium mobile test automation rule#307
Conversation
📝 WalkthroughWalkthroughThis PR adds a new Cursor rule set for Appium mobile testing. A new .mdc file provides comprehensive guidance for writing cross-platform Android and iOS UI tests, and the README Testing section is updated with a link to the new rule. Appium Mobile Testing Rule
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
rules/appium-mobile-testing-cursorrules-prompt-file.mdc (1)
3-4: ⚡ Quick winNarrow the rule scope for better auto-attach precision.
Line 3 uses
globs: **/*withalwaysApply: false, which effectively targets the whole repo and can cause this Appium-specific rule to attach in unrelated contexts. Consider narrowing to likely mobile test/config files (for example, Java/Kotlin/Python test paths and Appium config files).Suggested frontmatter adjustment
-description: "Cursor rules for Appium development with mobile (Android and iOS) test automation." -globs: **/* +description: "Cursor rules for Appium development with mobile (Android and iOS) test automation." +globs: **/*test*.py, **/*test*.java, **/*test*.kt, **/appium*.{js,ts,json,yml,yaml}, **/*capabilities*.{json,yml,yaml} alwaysApply: false🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@rules/appium-mobile-testing-cursorrules-prompt-file.mdc` around lines 3 - 4, The rule's frontmatter currently uses a broad glob pattern ("globs: **/*") causing it to match the entire repo; narrow the scope by replacing that pattern with targeted globs that match likely mobile test and Appium config files (for example test/source folders and filetypes) such as Java/Kotlin/Python test directories and Appium config filenames so the rule only auto-attaches for mobile-test contexts; update the frontmatter entries that reference "globs" and "alwaysApply" (the current symbols to change) to include specific patterns for test paths and Appium config files instead of a repository-wide "**/*".
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@rules/appium-mobile-testing-cursorrules-prompt-file.mdc`:
- Around line 3-4: The rule's frontmatter currently uses a broad glob pattern
("globs: **/*") causing it to match the entire repo; narrow the scope by
replacing that pattern with targeted globs that match likely mobile test and
Appium config files (for example test/source folders and filetypes) such as
Java/Kotlin/Python test directories and Appium config filenames so the rule only
auto-attaches for mobile-test contexts; update the frontmatter entries that
reference "globs" and "alwaysApply" (the current symbols to change) to include
specific patterns for test paths and Appium config files instead of a
repository-wide "**/*".
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 95d6d911-4b3a-4bad-bea8-8074ead6c937
📒 Files selected for processing (2)
README.mdrules/appium-mobile-testing-cursorrules-prompt-file.mdc
Adds an Appium mobile test automation rule to the Testing section.
The Testing list currently covers web and API testing (Cypress, Playwright, Jest, Vitest) but has no mobile testing entry. This adds cross-platform mobile coverage for Android and iOS.
The rule follows the same structure as the existing testing rules: frontmatter, an expert persona, language and platform auto-detection, and focused sections for the Page Object Model, locator and context strategy, waits and app state, capabilities and devices, reliability with retries, and CI/CD reporting.
Adapted from an Appium framework setup I maintain here: https://github.com/tugkanboz/awesome-cursorrules
Summary by CodeRabbit