Check branch changes for common PR readiness issues (missing tests, missing JSDoc, guideline violations). Use when the user asks to verify changes before opening a PR, check code quality, or audit a branch for missing items.
92
89%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Passed
No known issues
Scan the current branch diff for common issues that could be flagged during PR review. This is a non-blocking check — always report findings as warnings and let the user decide what to act on.
Collect the diff
git diff main...HEAD --name-only
git diff main...HEADCheck for missing tests
For each new or modified source file with non-trivial logic changes (not just config, docs, or styles), check whether a corresponding test file was added or updated.
Heuristic: a source file Foo.ts(x) should have a matching Foo.test.ts(x) (or Foo.view.test.tsx for components). Warn if:
Check for missing JSDoc
Scan new exported functions, types, and components in the diff. Warn if any lack JSDoc comments.
Check for guideline violations
Look for obvious violations of .github/guidelines/CODING_GUIDELINES.md and .cursor/rules/ patterns in the changed lines:
any type usage in TypeScriptStyleSheet.create() in new codeView or Text imports from react-native instead of design system Box/Textimport tw from 'twrnc' instead of useTailwind() hooknpx usage in scriptsPrint each finding as a warning line:
⚠ No tests detected for new logic in `app/core/Foo.ts`
⚠ Missing JSDoc on exported function `calculateFee` in `app/util/fees.ts`
⚠ `any` type used in `app/components/Bar.tsx:42`
⚠ `StyleSheet.create()` found in new file `app/components/Baz/Baz.tsx`If no issues found, confirm:
✅ No readiness issues detectedbee9b14
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.