core/agent
8
0
Fork 0
Code Issues 5 Pull requests Projects Releases Packages Wiki Activity Actions
agent/codex/core/commands/code-review.md
Snider bb88604045 feat(core): wire Core framework into agentic + monitor subsystems 
Phase 2 of Core DI migration:
- Add *core.Core field + SetCore() to PrepSubsystem and monitor.Subsystem
- Register agentic/monitor/brain as Core services with lifecycle hooks
- Mark SetCompletionNotifier and SetNotifier as deprecated (removed in Phase 3)
- Fix monitor test to match actual event names
- initServices() now wires Core refs before legacy callbacks

Co-Authored-By: Virgil <virgil@lethean.io>
2026-03-24 14:44:53 +00:00

1.1 KiB
Raw Blame History

name description args
code-review Perform code review on staged changes or PRs
commit-range|--pr=N|--security

Code Review

Perform a thorough code review of the specified changes.

Arguments

  • No args: Review staged changes
  • HEAD~3..HEAD: Review last 3 commits
  • --pr=123: Review PR #123
  • --security: Focus on security issues

Process

  1. Gather changes from the requested diff target
  2. Analyse each changed file for correctness, security, maintainability, and test gaps
  3. Report findings with clear severity and file references

Review Checklist

Category Checks
Correctness Logic errors, edge cases, error handling
Security Injection, XSS, hardcoded secrets, CSRF
Performance N+1 queries, unnecessary loops, large allocations
Maintainability Naming, structure, complexity
Tests Coverage gaps, missing assertions

Output Format

## Code Review: [title]

### Critical
- **file:line** - Issue description

### Warning
- **file:line** - Issue description

### Suggestions
- **file:line** - Improvement idea

---
**Summary**: X critical, Y warnings, Z suggestions