majiayu000 Quick Code Review
Context
- Current branch: !
git branch --show-current - Git status: !
git status --porcelain - Base branch: !
(git show-branch | grep '*' | grep -v "$(git rev-parse --abbrev-ref HEAD)" | head -1 | sed 's/.*\[\([^]]*\)\].*/\1/' | sed 's/\^.*//' 2>/dev/null) || echo "develop" - Changes since base: !
BASE=$(git merge-base HEAD develop 2>/dev/null || git merge-base HEAD main 2>/dev/null) && git log --oneline $BASE..HEAD - Files changed since base: !
BASE=$(git merge-base HEAD develop 2>/dev/null || git merge-base HEAD main 2>/dev/null) && git diff --name-only $BASE..HEAD - Test commands available: !
([ -f package.json ] && echo "npm/pnpm/yarn test") || ([ -f Cargo.toml ] && echo "cargo test") || ([ -f pyproject.toml ] && echo "pytest/uv run pytest") || ([ -f go.mod ] && echo "go test") || echo "no standard test framework detected"
Requirements
- Use @tech-lead-reviewer — architectural impact assessment — to scope the review and decide which specialized agents are required.
- Launch only the necessary specialized reviews to minimize turnaround time.
- Summarize results by priority (Critical → High → Medium → Low) and confidence (High → Medium → Low).
- Offer optional implementation support and ensure resulting commits follow Git commit conventions (详见
skills/references/git-commit-conventions.md).
Your Task
IMPORTANT: You MUST use the Task tool to complete ALL tasks.
- Run an initial assessment with @tech-lead-reviewer — architectural impact assessment — to gauge architectural, security, and UX risk, and determine if a deeper review is needed.
- Trigger the relevant specialized reviews via the Task tool, gather targeted feedback, and resolve conflicting recommendations.
- Present a concise summary, ask whether the user wants fixes implemented, and if confirmed, apply changes, refactor with @code-simplifier — code simplification and optimization —, test, and stage commits before reporting outcomes.
Targeted Review Flow
- Selective Agents:
- @code-reviewer — logic correctness, tests, error handling.
- @security-reviewer — authentication, data protection, validation.
- @ux-reviewer — usability and accessibility (skip if purely backend/CLI).
- Results Analysis: Organize findings using the priority/confidence matrix and provide actionable steps.
- Optional Implementation: Execute requested fixes, optimize the code, rerun tests, and prepare commits that adhere to the standards fragment.
- Closure: Push updates if changes were made and confirm review completion with the user.