[AISOS-741] Add auto review primitive to Forge#119

Open

JoshSalomon wants to merge 43 commits into

forge-sdlc:mainfrom

JoshSalomon:forge/aisos-741

Commits on Jul 21, 2026

[AISOS-2053] Create containers/review.py module with ReviewConfig and ReviewCycleData dataclasses

Forge
authored and
JoshSalomon
committed
[AISOS-2054] Add detect_review_md with project override precedence

Forge
authored and
JoshSalomon
committed
[AISOS-2055] Implement parse_verdict for APPROVED/REJECTED extraction

Forge
authored and
JoshSalomon
committed
[AISOS-2056] Implement write_cycle_file for review cycle JSON output

Forge
authored and
JoshSalomon
committed
[AISOS-2057] Integrate review loop logic into entrypoint.py

Forge
authored and
JoshSalomon
committed
[AISOS-2058] Add terminal progress display for review cycles in local mode

Forge
authored and
JoshSalomon
committed
[AISOS-2059] Create ReviewCyclePoller class with async polling loop

Forge
authored and
JoshSalomon
committed
[AISOS-2060] Implement ReviewCycleData and ReviewCycleRecorder with mode switching

Forge
authored and
JoshSalomon
committed
[AISOS-2061] Add Prometheus metrics for review cycles

Forge
authored and
JoshSalomon
committed
[AISOS-2062] Implement ReviewJiraNotifier for cycle comment posting

Forge
authored and
JoshSalomon
committed
[AISOS-2063] Integrate review polling into ContainerRunner with step-name parameter

Forge
authored and
JoshSalomon
committed
[AISOS-2064] Add post-execution sync sweep for missed review cycle files

Forge
authored and
JoshSalomon
committed
[AISOS-2065] Pass step_name from workflow nodes to ContainerRunner

Forge
authored and
JoshSalomon
committed
[AISOS-2066] Define review.md file format schema and validation specification

Forge
authored and
JoshSalomon
committed
[AISOS-2067] Create sample review.md for implement-task skill

Forge
authored and
JoshSalomon
committed
[AISOS-2068] Create sample review.md for local-code-review skill

Forge
authored and
JoshSalomon
committed
[AISOS-2069] Create sample project-specific review.md override for implement-task

Forge
authored and
JoshSalomon
committed
[AISOS-2070] Create comprehensive auto-review documentation

Forge
authored and
JoshSalomon
committed
[AISOS-2071] Update skills documentation to reference auto-review

Forge
authored and
JoshSalomon
committed
[AISOS-741-review] Fix import ordering in observability module

Forge
authored and
JoshSalomon
committed
[AISOS-741-docs] docs: add auto-review configuration settings to config reference

Forge
authored and
JoshSalomon
committed
fix: copy review.py module into container image

JoshSalomon
and
claude
committed
fix: add containers/ to sys.path in test conftest for CI

JoshSalomon
and
claude
committed
fix: make ReviewCyclePoller.poll() a sync method

JoshSalomon
and
claude
committed
fix: await poller.poll() before async iteration

JoshSalomon
and
claude
committed
fix: use AsyncMock for poll() in runner tests

JoshSalomon
and
claude
committed
fix: make ReviewCyclePoller.poll() sync for Python 3.11 compat

JoshSalomon
and
claude
committed
feat: surface review exhaustion data in PR body

JoshSalomon
and
claude
committed
fix: sort imports in review_report.py

JoshSalomon
and
claude
committed
fix: resolve ruff lint errors for review_report re-export

JoshSalomon
and
claude
committed
fix: apply ruff formatting to modified workflow nodes

JoshSalomon
and
claude
committed
feat: pass skill_name to container for review loop discovery

JoshSalomon
and
claude
committed
fix: use null-byte separators and guard ls-files result

JoshSalomon
and
claude
committed
fix: search mounted skill paths for review.md instead of hardcoded dirs

JoshSalomon
and
claude
committed
fix: align review cycle file paths between container and orchestrator

JoshSalomon
and
claude
committed
fix: replace operator.add with operator.or_ for review_exhaustion_report

JoshSalomon
and
claude
committed
refactor: consolidate review types, remove dead code, break up oversized functions

JoshSalomon
and
claude
committed
fix: forward step_name/skill_name through review_utils.run_review_container

JoshSalomon
and
claude
committed
fix: add exhaustion collection to task_takeover_review, fix mock data in tests

JoshSalomon
and
claude
committed
fix: resolve lint errors in test files

JoshSalomon
and
claude
committed
style: apply ruff formatting to all PR files

JoshSalomon
and
claude
committed
fix: address code review findings and test coverage gaps

JoshSalomon
and
claude
committed
fix: address max-effort code review findings

JoshSalomon
and
claude
committed