← ClaudeAtlas

reviewlisted

Review existing ABAP code against the Clean ABAP rule set in ../CLAUDE.md (loaded on invocation). Use when the user asks to review, audit, or check ABAP for clean-code violations — magic numbers, naming, method shape, error handling, table reads, classes, etc. Outputs a structured report grouped by severity, ATC-checkable findings first. Targets modern ABAP only — BTP ABAP Environment or S/4HANA on-prem in the ABAP Cloud development model.
matt1as/claude-abap-skills · ★ 9 · Code & Development · score 77
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