<!-- task=m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness plan=11 tag=REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_TEST milestone-task=hot-smoke -->

# Revalidate the Hot Path harness with current-checkout evidence

## For the Implementing Agent

Run every checklist item and fill the implementation-owned sections of `CODE_REVIEW-cloud-G03.md` with fresh stdout/stderr. Keep the active PLAN/CODE_REVIEW files in place and report ready for review. This is verification-only: do not repair or overwrite shared Edge/config source in this task. If the shared checkout is still compile-inconsistent, record the exact blocker, attempted commands/output, and resume condition in implementation-owned evidence fields and leave the verification item incomplete. Do not ask the user, call user-input tools, create control-plane stop files, classify the next state, archive logs, or write `complete.log`; finalization is owned by the code-review skill.

## Background

The harness syntax, fixed schema, credential-free oracle, and diff-integrity checks pass, but the SDD-required common Go regression still fails during compilation. The previous review evidence is also stale for the current checkout: `server.go` now contains the previously cited Hot Path fields, while the current compiler reports a different set of missing symbols. This follow-up records exact evidence from one current checkout and closes only when all required commands exit 0.

## Archive Evidence Snapshot

- The current pair will archive as `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/plan_cloud_G03_10.log` and `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/code_review_cloud_G03_10.log`.
- The current review verdict is `FAIL` with two Required findings, zero Suggested findings, and zero Nits.
- Fresh verification: `bash -n`, the fixed-schema `jq` assertion, `./scripts/e2e-hot-path-agents.sh --self-test`, and `git diff --check` exit 0. The SDD common regression exits 1 because `packages/go/config` and `apps/edge/internal/openai` do not compile.
- `review_rework_count=9`; `evidence_integrity_failure=true` because the active review's exact compiler output and cited `Server`-field blocker do not match the current source and fresh output.
- The contribution remains `milestone-task=hot-smoke`; the SDD contribution is S16. This deterministic harness task does not claim the separate credentialed Claude/Pi streaming evidence.

## Analysis

### Files Read

- `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/PLAN-cloud-G03.md`
- `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/CODE_REVIEW-cloud-G03.md`
- `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/plan_cloud_G03_9.log`
- `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/code_review_cloud_G03_9.log`
- `scripts/e2e-hot-path-agents.sh`
- `scripts/fixtures/hot-path-agent-smoke-manifest.schema.json`
- `apps/edge/internal/openai/server.go`
- `apps/edge/internal/openai/hot_path_terminal_control.go`
- `packages/go/config/edge_cli_config_test.go`
- `agent-roadmap/current.md`
- `agent-roadmap/phase/knowledge-tool-optimization-extension/PHASE.md`
- `agent-roadmap/phase/knowledge-tool-optimization-extension/milestones/iop-hot-path-one-shot-execution.md`
- `agent-roadmap/sdd/knowledge-tool-optimization-extension/iop-hot-path-one-shot-execution/SDD.md`
- `agent-spec/runtime/stream-evidence-gate.md`
- `agent-spec/input/openai-compatible-surface.md`
- `agent-contract/outer/openai-compatible-api.md`
- `agent-ops/rules/project/domain/testing/rules.md`
- `agent-ops/rules/project/domain/edge/rules.md`
- `agent-ops/rules/project/domain/platform-common/rules.md`
- `agent-test/local/rules.md`
- `agent-test/local/testing-smoke.md`
- `agent-test/local/edge-smoke.md`
- `agent-test/local/platform-common-smoke.md`

### SDD Criteria

- SDD: `agent-roadmap/sdd/knowledge-tool-optimization-extension/iop-hot-path-one-shot-execution/SDD.md`; status `[승인됨]`; SDD lock is released and no SDD `USER_REVIEW.md` exists.
- Contribution id: `hot-smoke`; targeted Acceptance Scenario: S16.
- S16 requires actual Claude/Pi streaming smoke plus workspace before/after evidence. This child supplies only the deterministic fail-closed harness prerequisite and must not claim the downstream credentialed run.
- The S16 Evidence Map requires the `hot-smoke` two-protocol final validation. The common completion evidence additionally requires `go test -race -count=1 ./packages/go/streamgate ./packages/go/config ./apps/edge/internal/openai ./apps/edge/internal/service` and `git diff --check`; therefore the checklist keeps the harness oracle and integrated regression in one indivisible verification item.

### Verification Context

- No external verification handoff was supplied. Repository-native fallback is based on the testing, Edge, and platform-common domain rules, local test rules and smoke profiles, the approved SDD, and the fresh commands run from `/config/workspace/iop-s0`.
- Toolchain: `go1.26.2 linux/arm64`, Bash, jq, GNU coreutils. Deterministic package verification needs no credential, provider, network, deployment, or installed Claude/Pi execution.
- Fresh results: syntax, schema, credential-free self-test, and diff integrity exit 0. The common regression exits 1 with missing `AdaptersConf.CLI`, `CompletionMarkerConf`, `normalizedStageDelta`, `reasonArtifactRequired`, `Server.emitHotPathObservation`, `hotPathLightStore.cleanupStage`, and `openAIRunEventSource.observeRunEvents`.
- Preconditions: the shared config/OpenAI checkout must be internally consistent before the required package command can pass. This task owns evidence only and must not edit shared production/config/test source.
- External Verification Preflight: not applicable. Credentialed Claude/Pi streaming remains a separate downstream S16 evidence run.
- Evidence confidence is high for the current failure because all five commands were rerun from the same dirty checkout, with `-count=1` on the Go command. The previous exact compiler transcript is not trusted for this loop because its cited `Server` state is contradicted by current `server.go:72-74,109-111`.

### Test Coverage Gaps

- The credential-free harness self-test covers the fixed 2x5 matrix, schema rejection, identity checks, redaction, cancellation, cleanup/orphan classification, and workspace digest behavior.
- No new behavior is introduced by this follow-up, so no test is added. The remaining gap is the failing SDD common package regression; OpenAI package tests cannot execute until compilation succeeds.

### Symbol References

- This follow-up renames no symbol.
- Current compiler references include `packages/go/config/edge_cli_config_test.go:180,242,289,352,364`; `apps/edge/internal/openai/hot_path_terminal_control.go:1016`; `apps/edge/internal/openai/hot_path_observation.go:645,694,716,739,754,767,786,804`; and `apps/edge/internal/openai/hot_path_stage_stream.go:144`.
- The previously cited `Server.requestCoordinator`, `Server.artifactFrontiers`, and `Server.lightFlows` are currently present and initialized at `apps/edge/internal/openai/server.go:72-74,109-111`; the next evidence must not repeat the stale diagnosis.

### Split Judgment

- Keep one verification-only plan. The harness checks and the SDD common regression are one completion invariant: PASS requires all five commands to exit 0 in the same checkout, and splitting would allow completion without integrated evidence.

### Scope Rationale

- Modify only the active review evidence file `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/CODE_REVIEW-cloud-G03.md`.
- Do not edit `scripts/e2e-hot-path-agents.sh`, its schema, `apps/edge/internal/openai/**`, `packages/go/config/**`, `proto/**`, config, Makefile, deployment, credentials, or tracked smoke output. The compile reconciliation belongs to the owning production/config work.

### Final Routing

- `evaluation_mode=isolated-reassessment`; finalizer=`finalize-task-policy.sh pair`.
- Build closures are all true. Scores are scope/state/blast/evidence/verification=`1/0/0/1/1`, grade G03, base route `local-fit`, final route `recovery-boundary`, lane `cloud`, filename `PLAN-cloud-G03.md`.
- Review closures are all true. Scores are `1/0/0/1/1`, grade G03, route `official-review`, lane `cloud`, filename `CODE_REVIEW-cloud-G03.md`.
- `large_indivisible_context=false`; positive loop risks are `boundary_contract` and `variant_product` (2). `review_rework_count=9`; `evidence_integrity_failure=true`; recovery boundary matches.
- No capability gap is claimed. The deterministic check is repository-local once the shared source is reconciled.

## Implementation Checklist

- [ ] [REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_TEST-1] Rerun the unchanged fail-closed harness and the SDD common regression from one current checkout, replacing the stale blocker transcript with exact output.
- [ ] Fill implementation-owned sections in CODE_REVIEW-*-G??.md with actual implementation notes and verification output.

### [REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_TEST-1] Revalidate mandatory integrated verification

**Problem:** The current pair's recorded SDD common regression transcript is stale, and the exact command still exits 1. Current output reports missing config and OpenAI symbols rather than the previously cited omitted `Server` fields.

**Solution:** Do not edit shared production/config/test source in this task. Rerun the complete deterministic verification set in the current checkout, paste exact stdout/stderr into the new review evidence, and leave this item incomplete if the common regression remains non-zero. PASS requires all five commands to exit 0.

**Before (current evidence):**

```text
go test -race -count=1 ./packages/go/streamgate ./packages/go/config ./apps/edge/internal/openai ./apps/edge/internal/service
exit=1
packages/go/config: AdaptersConf.CLI and CompletionMarkerConf undefined
apps/edge/internal/openai: normalizedStageDelta, reasonArtifactRequired, emitHotPathObservation, cleanupStage, and observeRunEvents undefined
```

**After:**

```text
go test -race -count=1 ./packages/go/streamgate ./packages/go/config ./apps/edge/internal/openai ./apps/edge/internal/service
ok ... all four packages
exit=0
```

**Modified Files and Checklist:**

- [ ] Keep all harness, schema, production/config, protocol, deployment, credential, and tracked smoke-output files unchanged by this task.
- [ ] Run every Final Verification command with fresh output and fill `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/CODE_REVIEW-cloud-G03.md`.

**Test Strategy:** Add no new test. The existing credential-free self-test is the behavioral oracle; the existing race-enabled package command is the SDD-required integration oracle. Cached Go output is not accepted because the command uses `-count=1`.

**Verification:** Every Final Verification command exits 0. A shared-worktree compiler error remains a blocker, never PASS evidence.

## Dependencies and Execution Order

1. The shared config/OpenAI owner must restore a compile-consistent checkout or remove the corresponding incomplete feature/test set consistently.
2. After that state change, rerun REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_TEST-1.

## Modified Files Summary

| File | Item |
|---|---|
| `agent-task/m-iop-hot-path-one-shot-execution/20+17,19_smoke_harness/CODE_REVIEW-cloud-G03.md` | REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_REVIEW_TEST-1 |

## Final Verification

```bash
bash -n scripts/e2e-hot-path-agents.sh
jq -e '(.properties.cases.prefixItems | length) == 10 and .properties.cases.items == false and ([.properties.cases.prefixItems[].properties.id.const] | length == 10) and ([.properties.cases.prefixItems[].properties.id.const] | unique | length == 10)' scripts/fixtures/hot-path-agent-smoke-manifest.schema.json
./scripts/e2e-hot-path-agents.sh --self-test
go test -race -count=1 ./packages/go/streamgate ./packages/go/config ./apps/edge/internal/openai ./apps/edge/internal/service
git diff --check
```

Expected: every command exits 0 with fresh evidence from the same checkout. If the common regression fails, preserve exact output and leave the implementation item incomplete.

After completing all code changes, fill implementation-owned sections in CODE_REVIEW-*-G??.md.
