Contenuto dal repository con titoli, esempi, codice, tabelle, link e immagini preservati.
Code Review
Generic code review command that detects your project's tech stack and delegates to the appropriate framework-specific review orchestrator.
Config
TECH_STACK = config_read("tech_stack", "generic")DEFAULT_BRANCH=$(git remote show origin | grep 'HEAD branch' | awk '{print $NF}')Arguments
<inputarguments> $ARGUMENTS </inputarguments>
Scope modes:
#123or GitHub URL → PR changes- (no args) → unstaged changes (working directory)
--staged→ staged changes only--branch→ current branch vs default branchpath/to/file ...→ specific files
Step 1: Detect Tech Stack
Use "Tech stack" from Context above. If empty, auto-detect from project files:
Auto-Detection Fallback
If not configured in .agents.yml, detect from project files:
| Detection | Tech Stack |
|---|---|
Gemfile exists | rails |
pyproject.toml or requirements.txt exists | python |
package.json exists (no Gemfile) | javascript |
| None of the above | generic |
# Detection commands
[ -f Gemfile ] && echo "rails"
[ -f pyproject.toml ] || [ -f requirements.txt ] && echo "python"
[ -f package.json ] && echo "javascript"Step 2: Gather Changed Files
Based on scope from arguments:
# Default (unstaged changes)
git diff --name-only --diff-filter=d
# Staged mode
git diff --cached --name-only --diff-filter=d
# Branch mode (use "Default branch" from Context)
git diff <default_branch>...HEAD --name-only --diff-filter=d
# PR mode
gh pr diff <PR_NUMBER> --name-onlyStep 3: Delegate to Framework Orchestrator
Based on detected tech stack, invoke the appropriate orchestrator agent via Task tool:
Rails Projects
Task: majestic-rails:review/code-review-orchestrator
Prompt: "Review these Rails files. Scope: [scope mode]
Changed files:
[file list]"Python Projects
Task: majestic-python:review/code-review-orchestrator
Prompt: "Review these Python files. Scope: [scope mode]
Changed files:
[file list]"JavaScript Projects (Not Yet Supported)
**Tech Stack:** JavaScript
JavaScript code review orchestrator is not yet available.
For now, you can run the generic simplicity reviewer:
`Task(subagent_type: "simplicity-reviewer") [files]`
Or configure `tech_stack: generic` in AGENTS.md to use generic reviewers.Generic Projects
For unknown or generic tech stacks, run minimal reviewers directly:
- Load project topics (if configured)
- Run both reviewers in parallel:
- Synthesize output
# Run these tasks in parallel:
Task 1: majestic-engineer:review/simplicity-reviewer
Prompt: "Review these files for YAGNI violations, unnecessary complexity: [file list]"Step 4: Report Results
The orchestrator will return a synthesized report. Present it to the user with:
- Status - BLOCKED / NEEDS CHANGES / APPROVED
- Summary - Key findings by severity (P1/P2/P3)
- Details - Full agent reports in collapsible sections
Tech Stack Configuration
To explicitly set your tech stack (recommended for multi-language projects), add to AGENTS.md:
## Project Configuration
tech_stack: rails # rails | python | javascript | genericExamples
# Review unstaged changes (auto-detect tech stack)
Skill("code-review")
# Review staged changes
Skill("code-review") --staged
# Review current branch vs main
Skill("code-review") --branch
# Review a PR
Skill("code-review") #123
# Review specific files
Skill("code-review") app/models/user.rb app/controllers/users_controller.rb