<!-- task=m-iop-hot-path-one-shot-execution/05+01,02,03_artifact_pair plan=0 tag=API milestone-task=artifact-pair -->

# Code Review Reference - API

> **[IMPLEMENTING AGENT — READ FIRST] Filling in this file is the mandatory final step of implementation.**
> Fill item statuses, deviations, decisions, and actual output, then stop with active files and report ready. Record blockers only in implementation evidence. Do not ask the user, create control state, classify, archive, or write `complete.log`; review owns finalization.

## Overview

date=2026-08-02
task=m-iop-hot-path-one-shot-execution/05+01,02,03_artifact_pair, plan=0, tag=API

## For the Review Agent

> **[REVIEW AGENT ONLY]** Implementers must not execute this section.

Compare source/evidence, append verdict/signals, archive the pair, and on PASS write `complete.log`, preserve metadata, archive the directory, and update the final `.log` checklist. WARN/FAIL must create the exact next state.
## Implementation Item Completion

| Item | Status |
|------|---------|
| API-1 Compile request-local workspace operation bindings | [ ] |
| API-2 Validate directory prepare and exact pair continuation frontier | [ ] |

## Implementation Checklist

- [ ] Select and pin a declarative workspace tool binding from actual Chat/Anthropic schemas with safe deterministic argument/result transforms.
- [ ] Enforce prepare and exact Plan/Review expected sets, paths, public/provider ids, and one-frontier result success before local eligibility.
- [ ] Run focused mapping/frontier, common race, vet, and diff verification exactly as written.
- [ ] Fill implementation-owned sections in CODE_REVIEW-*-G??.md with actual implementation notes and verification output.

## Review-Only Checklist

> **[REVIEW AGENT ONLY]** Implementers must not modify or check this section.

- [ ] Append one PASS/WARN/FAIL verdict with verified `review_rework_count` and `evidence_integrity_failure`.
- [ ] Verify verdict, Dimension Assessment, and Required/Suggested/Nit classifications match.
- [ ] Archive the active review to `code_review_cloud_G10_0.log`.
- [ ] Archive the active plan to `plan_cloud_G10_0.log`.
- [ ] Verify the Agent-Ops `.gitignore` block.
- [ ] On PASS write `complete.log` from `agent-ops/skills/common/code-review/templates/complete-log-template.md` and leave no active `.md`.
- [ ] On PASS archive to `agent-task/archive/YYYY/MM/m-iop-hot-path-one-shot-execution/05+01,02,03_artifact_pair/` and update this checklist there.
- [ ] On PASS preserve/report `milestone-task=artifact-pair` without direct roadmap mutation.
- [ ] On PASS remove the active parent only if no siblings/files remain.
- [ ] On WARN/FAIL create the mandatory next state without `complete.log`.

## Deviations from Plan

_Implementer: replace with actual deviations or “None”._

## Key Design Decisions

_Implementer: replace with actual decisions._

## Reviewer Checkpoints

- Bindings match actual schemas and remain pinned/fingerprinted.
- Paths/commands are deterministic; Edge performs lexical checks and validates the exact receipt from a caller-executed containment guard, but never inspects the workspace or executes the tool.
- Only exact prepare or exact two-result pair advances, once and order-independently.

## Verification Results

Paste actual stdout/stderr below.

### API-1 item verification

```bash
go test -count=1 ./apps/edge/internal/openai -run 'TestWorkspace(Tool|Command)'
```

_Actual stdout/stderr:_

### API-2 item verification

```bash
go test -race -count=1 ./apps/edge/internal/openai -run 'Test(Workspace|ArtifactPair)'
```

_Actual stdout/stderr:_

### Dependencies and focused race

```bash
test -f agent-task/m-iop-hot-path-one-shot-execution/01_preset_catalog/complete.log
test -f agent-task/m-iop-hot-path-one-shot-execution/02+01_preset_model/complete.log
test -f agent-task/m-iop-hot-path-one-shot-execution/03+01,02_request_identity/complete.log
go test -race -count=1 ./apps/edge/internal/openai -run 'Test(Workspace|ArtifactPair)'
```

_Actual stdout/stderr:_

### Common race tests

```bash
go test -race -count=1 ./packages/go/streamgate ./packages/go/config ./apps/edge/internal/openai ./apps/edge/internal/service
```

_Actual stdout/stderr:_

### Vet and diff

```bash
go vet ./apps/edge/internal/openai
git diff --check
```

_Actual stdout/stderr:_

---

> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section?** Leave review-only sections unchanged.

## Section Ownership

| Section | Owner | Note |
|---------|-------|------|
| Fixed structure, item names/checklist/checkpoints/commands | Fixed | Do not rewrite |
| Item status, deviations, decisions, actual output | Implementer | Must complete |
| Review checklist and verdict/finalization | Review agent | Implementer must not modify |
