Rendu depuis le dépôt source en conservant titres, exemples, code, tableaux, liens et images.
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