/backend-review

/backend-review

On-demand, language-aware code review for backend code. Reviews code against the team's coding guidelines and uses LSP diagnostics.

Steps

1. Set up repository (if not already cloned):

   • Run the backend-repo-setup skill in runtime mode.
   • The skill reads the repository block from .backend-agent.json.
   • If no repository block is present, this step is a no-op.
   • If the block is present and localPath does not contain a .git directory, the skill clones the repository before proceeding.
   • On clone failure, the command halts here with a clear error message.

2. Determine scope:

   • If user specified files/paths, review those
   • If no files specified, review the current git diff (staged + unstaged)
   • If on a branch, review all changes vs the base branch

3. Resolve language and guidelines:

   • Run config-resolver skill to determine active language
   • Run guidelines-ingestion skill to load merged guidelines
   • If reviewing multiple files in different languages (monorepo), resolve per-file

4. For each file in scope:

   • Identify the language (from extension or directory mapping)
   • Load the corresponding language module's coding skill for language-specific checks
   • Run LSP diagnostics on the file (LSP.getDiagnostics)
   • Run LSP.getReferences to check for broken references

5. Review against guidelines:

   • Check each guideline rule against the code
   • For each violation, record: file, line, guideline rule, severity

6. Severity levels:

   • blocker — must fix before merge (security, correctness, guideline violation marked critical)
   • suggestion — should fix, improves quality
   • nit — style/preference, optional

7. Output format:

   ## Backend Code Review
   
   **Language:** <lang> | **Framework:** <framework> | **Guidelines:** <source>
   
   ### Blockers (X)
   - `file.cs:42` — [Guideline: ## Error Handling] Using throw instead of Result<T> pattern
   - `file.cs:87` — [LSP] CS0103: The name 'foo' does not exist in the current context
   
   ### Suggestions (X)
   - `file.cs:15` — [Guideline: ## Naming] Method name `DoStuff` is not descriptive
   
   ### Nits (X)
   - `file.cs:3` — [Guideline: ## Naming] Consider `IUserService` over `IUserSvc`
   
   ### LSP Diagnostics
   - Zero errors ✅ | X errors found ❌
   
   **Summary:** X blockers, Y suggestions, Z nits