Add a /test skill to run singular tests or per module tests#5111
Add a /test skill to run singular tests or per module tests#5111
Conversation
Semver Impact of This PR⚪ None (no version bump detected) 📋 Changelog PreviewThis is how your changes will appear in the changelog.
🤖 This preview updates automatically when you update the PR. |
Instructions and example for changelogPlease add an entry to Example: ## Unreleased
### Features
- Add a /test skill to run singular tests or per module tests ([#5111](https://github.com/getsentry/sentry-java/pull/5111))If none of the above apply, you can opt out of this check by adding |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| |---------------|-----------| | ||
| | `sentry-android-*` | `testDebugUnitTest` | | ||
| | `sentry-compose*` | `testDebugUnitTest` | | ||
| | Everything else | `test` | |
There was a problem hiding this comment.
Test task detection misses Android module naming pattern
Medium Severity
The module-to-test-task mapping only handles sentry-android-* and sentry-compose* as needing testDebugUnitTest, but sentry-launchdarkly-android is also an Android library module (it uses the com.android.library Gradle plugin) that requires testDebugUnitTest. Since its name doesn't match either pattern, it falls through to the plain test task, which would fail. The pattern matching needs to also account for non-sentry-android-*-prefixed Android modules.
| --- | ||
| name: test | ||
| description: Run tests for a specific SDK module. Use when asked to "run tests", "test module", "run unit tests", "run system tests", "run e2e tests", or test a specific class. Auto-detects unit vs system tests. Supports interactive mode. | ||
| allowed-tools: Bash, Read, Glob |
There was a problem hiding this comment.
Interactive mode tool not listed in allowed-tools
Medium Severity
The skill's interactive mode (Steps 0, 1, and 3a) instructs Claude to use AskUserQuestion at multiple decision points, but allowed-tools only lists Bash, Read, Glob. Since AskUserQuestion is a distinct Claude Code tool that can be included in allowed-tools, its omission means the interactive mode feature described throughout the skill cannot function as intended when tool restrictions are enforced.


📜 Description
💡 Motivation and Context
💚 How did you test it?
📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps