<!-- task=m-iop-hot-path-one-shot-execution/03+01,02_request_identity plan=0 tag=API milestone-task=request-identity -->

# Code Review Reference - API

> **[IMPLEMENTING AGENT — READ FIRST] Filling in this file is the mandatory final step of implementation.**
> The task is not complete until item statuses, Deviations, Key Design Decisions, and actual verification output are filled. Then stop with active files and report ready. Blockers belong only in those evidence fields. Do not ask the user, create control state, classify next state, archive, or write `complete.log`; review owns finalization.

## Overview

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

## For the Review Agent

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

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

| Item | Status |
|------|---------|
| API-1 Build the bounded logical-request store and lineage fence | [ ] |
| API-2 Join preset-backed endpoint ingress to the coordinator | [ ] |

## Implementation Checklist

- [ ] Implement opaque request/call/stage identity, owner affinity, immutable lineage/toolset fingerprints, and exactly-once frontier state.
- [ ] Integrate preset-backed Chat and Messages ingress without changing legacy/provider paths or trusting caller metadata as identity.
- [ ] Run deterministic concurrency, focused handler, 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_G09_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/03+01,02_request_identity/` and update this checklist there.
- [ ] On PASS preserve/report `milestone-task=request-identity` without direct roadmap mutation.
- [ ] On PASS remove the active parent only if no siblings/files remain.
- [ ] On WARN/FAIL write the mandatory next state and no `complete.log`.

## Deviations from Plan

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

## Key Design Decisions

_Implementer: replace with actual decisions._

## Reviewer Checkpoints

- IDs are server-generated, path-safe, collision-resistant, and never authorization secrets.
- Lineage/toolset/principal mutation and missing state dispatch nothing.
- Exactly one concurrent resume consumes a frontier; legacy routes bypass the store.

## Verification Results

Paste actual stdout/stderr below.

### API-1 item verification

```bash
go test -race -count=1 ./apps/edge/internal/openai -run 'TestLogicalRequest'
```

_Actual stdout/stderr:_

### API-2 item verification

```bash
go test -count=1 ./apps/edge/internal/openai -run 'TestPresetRequestIdentity'
```

_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
go test -race -count=1 ./apps/edge/internal/openai -run 'Test(LogicalRequest|PresetRequestIdentity)'
```

_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 |
