<!-- task=m-iop-agent-cli-runtime/09+05_workflow_evidence plan=0 tag=REFACTOR -->

# Code Review Reference - REFACTOR

> **[IMPLEMENTING AGENT — READ FIRST]** Confirm `05` completion, fill actual evidence, and do not finalize/archival work.

## Overview

date=2026-07-28
task=m-iop-agent-cli-runtime/09+05_workflow_evidence, plan=0, tag=REFACTOR

## Roadmap Targets

- Milestone: `agent-roadmap/phase/automation-runtime-bridge/milestones/iop-agent-cli-runtime.md`
- Milestone link: [Milestone 문서](agent-roadmap/phase/automation-runtime-bridge/milestones/iop-agent-cli-runtime.md)
- Task ids:
  - `workflow-evidence`: common artifact matcher, review gate, Pi evidence repair
- Completion mode: check-on-pass

## Implementation Item Completion

| Item | Status |
|---|---|
| REFACTOR-1 artifact gate와 Pi repair를 strict port로 추가한다 | [ ] |

## Implementation Checklist

- [ ] Define normalized artifact identity, completeness and repair intent port types.
- [ ] Implement provider-neutral matcher for active pair, placeholders and exact work/attempt identity.
- [ ] Permit Pi-only same-context repair with durable locator/ordinal and a mandatory rematch.
- [ ] Gate reviewer invocation on a matched submission and retain typed blocker evidence otherwise.
- [ ] Add matcher, repair and manager integration matrices.
- [ ] Fill implementation-owned sections in CODE_REVIEW-*-G??.md with actual implementation notes and verification output.

## Review-Only Checklist

> **[REVIEW AGENT ONLY]**

- [x] Append PASS/WARN/FAIL and verified routing signals.
- [x] Archive as `code_review_cloud_G09_0.log` and `plan_cloud_G09_0.log`.
- [ ] On PASS create `complete.log`, report `workflow-evidence`, then archive this subtask.

## Deviations from Plan

_Record actual deviations and rationale here._

## Key Design Decisions

_Record actual decisions here._

## Reviewer Checkpoints

- Review is never invoked before matcher success; Pi repair is same-context-only and rematched.

## Verification Results

### `go test -count=1 ./packages/go/agenttask`

_Paste actual stdout/stderr._

### `rg --sort path -n 'USER_REVIEW|complete\.log' packages/go/agenttask`

_Paste actual stdout/stderr._

### `git diff --check`

_Paste actual stdout/stderr._

## Section Ownership

| Section | Owner |
|---|---|
| Implementation checklist/results/deviations/decisions | Implementing agent |
| Review checklist, verdict, archive and completion log | Review agent |

## Code Review Result

- Overall Verdict: FAIL
- Dimension Assessment:
  - Correctness: Pass
  - Completeness: Fail
  - Test Coverage: Pass
  - API Contract: Pass
  - Code Quality: Pass
  - Implementation Deviation: Fail
  - Verification Trust: Pass
  - Spec Conformance: Pass
- Findings:
  - Required — `agent-task/m-iop-agent-cli-runtime/09+05_workflow_evidence/CODE_REVIEW-cloud-G09.md:19`: the implementation item, every implementation checklist entry, deviations, design decisions, and all verification-result fields remain unfilled, so this active pair is an unfilled review stub and cannot satisfy the mandatory implementation-evidence step. Fix the follow-up stub by reconciling the implemented S08 source/tests, checking each supported item, replacing all implementation placeholders with actual notes, and pasting fresh command stdout/stderr.
- Routing Signals: `review_rework_count=1`, `evidence_integrity_failure=false`
- Next Step: Invoke the plan skill in `prepare-follow-up` mode with this Required finding and the fresh reviewer verification output.
