All tests pass successfully after merging master (1388 passed, 1 failed [pre-existing]). Reviewed the implementation of native_mcp_preference.py and the test coverage in tests/test_native_mcp_preference.py. It correctly enforces native MCP preference over shell / direct API / helper script fallbacks for Gitea mutations, and implements the shell spawn failure circuit breaker to prevent unproven/unsafe fallbacks when MCP is broken. Verified that the remaining failure is pre-existing on master.
1c506fa779
Merge pull request 'feat: comment-then-stop policy for partial PR reconciliation (Closes #302)' (#375) from feat/issue-302-partial-reconciliation-policy into master
All tests pass successfully after merging master (1378 passed, 1 failed [pre-existing]). Reviewed the implementation of resolve_partial_reconciliation_plan and assess_partial_reconciliation_report in review_proofs.py, the workflow documentation in skills/llm-project-workflow/workflows/reconcile-landed-pr.md, and the test coverage in tests/test_review_proofs.py. It correctly implements Option B (comment-then-stop) as the durable policy for partial reconciliation. Verified that the remaining failure is pre-existing on master.
Review summary (PR #371 / #299)
e7a1e183a0
Merge pull request 'feat: prove inventory pagination and worktree state in reviewer reports (Closes #293)' (#369) from feat/issue-293-inventory-worktree-proof into master
All tests pass successfully after merging master (1369 passed, 1 failed [pre-existing]). Reviewed the implementation of assess_inventory_worktree_report in reviewer_inventory_worktree.py and the test coverage in tests/test_reviewer_inventory_worktree.py. It correctly validates PR inventory pagination completeness and checks that worktree fields (used/path/HEAD state) are reported consistently. Verified that the remaining failure is pre-existing on master.
The implementation of the branches-only author mutation guard in _enforce_branches_only_author_mutation breaks the existing integration tests in tests/test_commit_payloads.py. Specifically, the integration tests call gitea_commit_files from a temporary directory (tempfile.TemporaryDirectory()) which does not live under branches/, and because worktree_path / environment variables are not resolved from the active issue lock file or mapped to a branches/ layout in those tests, the mutation guard blocks the call and raises a RuntimeError ('workspace is the stable control checkout' or 'is not under branches/'). Please update either the mutation workspace resolution (to fallback gracefully for tests / support lock file paths) or adapt tests/test_commit_payloads.py to use a branches/ layout so that the test suite passes.
b6cf32c845
Merge pull request 'feat: add dedicated reconciliation controller-handoff schema (Closes #303)' (#373) from feat/issue-303-reconciliation-handoff into master
All tests pass successfully after merging master (1359 passed, 1 failed [pre-existing]). Reviewed the implementation of assess_reconciliation_handoff in review_proofs.py and the test coverage in tests/test_reconciliation_handoff.py. It correctly introduces a dedicated handoff schema for already-landed PR reconciliation, enforcing presence of necessary fields and rejection of stale reviewer/author fields. Verified that the remaining failure is pre-existing on master.
a17e8561d0
Merge pull request 'feat: require infra_stop repair handoff for blocked reviewers (Closes #289)' (#366) from feat/issue-289-infra-stop-handoff into master