$npx -y skills add luongnv89/asm --skill well-formedReview pull request diffs for code smells, style issues, and safety problems before merging.
| 1 | # Well-formed corpus skill |
| 2 | |
| 3 | ## When to Use |
| 4 | |
| 5 | - When the user asks to "review this PR" or "check the diff" |
| 6 | - Before merging any change larger than 10 lines |
| 7 | |
| 8 | ## Prerequisites |
| 9 | |
| 10 | - A git repository with the target branch checked out |
| 11 | - Read access to the files being reviewed |
| 12 | |
| 13 | ## Instructions |
| 14 | |
| 15 | 1. Run `git diff main...HEAD` to list files |
| 16 | 2. Read each file and check for common smells |
| 17 | 3. Emit a markdown report summarising findings |
| 18 | |
| 19 | ## Example |
| 20 | |
| 21 | ```bash |
| 22 | $ asm eval ./well-formed |
| 23 | Overall score: 95/100 |
| 24 | ``` |
| 25 | |
| 26 | ## Acceptance Criteria |
| 27 | |
| 28 | - Produces a markdown report with sections per file |
| 29 | - Flags any use of `eval()` or `exec` as dangerous |
| 30 | - Does not modify the working tree |
| 31 | |
| 32 | ## Edge cases |
| 33 | |
| 34 | - Empty diffs: emit a short "no changes" note |
| 35 | - Binary files: skip and mention the filename in the report |
| 36 | |
| 37 | ## Safety |
| 38 | |
| 39 | See `references/safety.md` for error handling rules. |
| 40 | Always confirm before writing. Never run destructive commands without a dry-run. |