reviewlisted
Install: claude install-skill matt1as/claude-abap-skills
# clean-abap:review
Review existing ABAP code against the Clean ABAP rule set in `../CLAUDE.md` (relative to this skill's directory).
## What this command does
You are reviewing ABAP source code for compliance with the Clean ABAP rule set. Your job is to produce a **structured, prioritised review** — not to refactor the code (that is `/clean-abap:refactor`).
## Inputs
Accept any of the following. If none are provided, ask the user which they want.
- A code block pasted into the conversation (the most reliable path today)
- An ABAP object name — only works if a community MCP exposes source reads. The official `SAPSE.adt-vscode` 1.0 MCP does not.
- A package name — only works with a community MCP that exposes repository search. The official MCP does not.
If only the official MCP is connected, ask the user to paste the source from ADT in VS Code.
## Procedure
1. **Load the rule set.** Read every rule under `## RULE:` in `../CLAUDE.md`. These are the only rules in scope for this review.
2. **Get the source.** If pasted inline, use it. If only an object name was given, attempt source-read via whichever MCP is connected; if no MCP exposes source reads, ask the user to paste it from ADT. The official `SAPSE.adt-vscode` 1.0 MCP does not have a source-read tool.
3. **Scan the source against every rule.** Record each violation with: the rule name, the exact line or block in the source, and a one-sentence diagnosis.
4. **Prioritise ATC-checkable violations first.** Rules with a