<!-- task=m-stream-evidence-gate-core/24+20,21_edge_observation_sink plan=3 tag=REVIEW_EDGE_FILTER_OBSERVATION_SINK -->

# Code Review Reference - REVIEW_EDGE_FILTER_OBSERVATION_SINK

> **[IMPLEMENTING AGENT — READ FIRST] Filling in this file is the mandatory final step of implementation.**
> The task is NOT complete until every implementation-owned section below is filled in.
> Complete the `구현 체크리스트`; the final checklist item is mandatory before saving.
> Fill implementation-owned sections, then stop with active files in place and report ready for review.
> If implementation is blocked, record the exact blocker, attempted commands/output, and resume condition only in implementation-owned evidence fields.
> Do not ask the user directly, present choices, call user-input tools, create control-plane stop files, or classify the next state.
> Finalization (`코드리뷰 결과`, log rename, `complete.log`, archive moves, `코드리뷰 전용 체크리스트`) is review-agent-only, even after compaction/resume.
> Follow the ownership table at the bottom of this file for which sections you own.

## 개요

date=2026-07-27
task=m-stream-evidence-gate-core/24+20,21_edge_observation_sink, plan=3, tag=REVIEW_EDGE_FILTER_OBSERVATION_SINK

## Archive Evidence Snapshot

- 현재 pair archive: `agent-task/m-stream-evidence-gate-core/24+20,21_edge_observation_sink/plan_cloud_G03_2.log`, `agent-task/m-stream-evidence-gate-core/24+20,21_edge_observation_sink/code_review_cloud_G03_2.log`.
- 판정: FAIL; Required=1, Suggested=0, Nit=0.
- Required: `TestFilterObservationSink_MapsStableAllowlist`가 expected key 존재만 확인하고 대부분의 value, 4-entry cause order, `terminal_reason`을 검증하지 않으며 `OmitsRawFieldsAndSentinels`는 sentinel을 fixture에 주입하지 않는다.
- 실제 reviewer evidence: host Go `/config/opt/go/bin/go`, `go1.26.2 linux/arm64`; 네 named test, affected package, focused race, gofmt, `git diff --check`는 fresh PASS했다. 실패 원인은 실행 여부가 아니라 필수 assertion 공백이다.
- 선행 완료: `agent-task/archive/2026/07/m-stream-evidence-gate-core/20+17,19_edge_vertical_slice/complete.log`, `agent-task/archive/2026/07/m-stream-evidence-gate-core/21+14,19_filter_observation_contract/complete.log`.
- Roadmap carryover: SDD S08의 raw-free sink mapping 일부만 보강하므로 이 child는 Milestone `filter-observation` Task 완료를 주장하지 않는다. Full request timeline과 Roadmap Completion은 child `25`가 소유한다.

## 이 파일을 읽는 리뷰 에이전트에게

> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다.

각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다.

1. 판정과 `review_rework_count` / `evidence_integrity_failure` 라우팅 신호를 append한다.
2. `CODE_REVIEW-cloud-G03.md` → `code_review_cloud_G03_3.log`, `PLAN-cloud-G03.md` → `plan_cloud_G03_3.log`로 아카이브한다.
3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-stream-evidence-gate-core/24+20,21_edge_observation_sink/`로 이동한다. WARN/FAIL이면 code-review skill이 요구하는 다음 filesystem state를 완전히 작성한다.
4. PASS이고 task group이 `m-<milestone-slug>`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다.
5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다.

---

## 구현 항목별 완료 여부

| 항목 | 완료 여부 |
|------|---------|
| OBS-EDGE-3 closed allowlist oracle | [x] |
| OBS-EDGE-4 fresh verification evidence | [x] |

## 구현 체크리스트

- [x] OBS-EDGE-3 all-fields/base/terminal fixture가 exact key set과 모든 mapped value, 4-entry cause order/bound, evidence 값, raw-context sentinel 부재를 검증하도록 보강한다.
- [x] OBS-EDGE-4 host Go 확인과 fresh test-list/unit/package/race/gofmt/diff 검증을 수행한다.
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다.

## 코드리뷰 전용 체크리스트

> **[REVIEW AGENT ONLY]** 이 체크리스트는 코드리뷰 에이전트만 사용한다.
> 구현 에이전트는 이 섹션을 수정하거나 체크하지 않는다.

- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정과 검증된 `review_rework_count`, `evidence_integrity_failure`를 append한다.
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_cloud_G03_3.log`로 아카이브한다.
- [x] active `PLAN-*-G??.md`를 `plan_cloud_G03_3.log`로 아카이브한다.
- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다.
- [x] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다.
- [x] PASS이면 active task 디렉터리 `agent-task/m-stream-evidence-gate-core/24+20,21_edge_observation_sink/`를 `agent-task/archive/YYYY/MM/m-stream-evidence-gate-core/24+20,21_edge_observation_sink/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
- [x] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
- [x] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-stream-evidence-gate-core/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
- [ ] WARN/FAIL이면 code-review skill의 판정에 맞는 다음 filesystem state를 작성하고 `complete.log`를 작성하지 않는다.

## 계획 대비 변경 사항

없음. 계획(`PLAN-cloud-G03.md`)에 기술된 바와 같이 production 코드(`filter_observation_sink.go`)는 변경하지 않았으며, `filter_observation_sink_test.go`의 oracle 및 fixture만 exact-map assertion helper(`assertExactObservationFields`)와 4-entry cause order, terminal_reason fixture, context sentinel injection으로 보강하였습니다.

## 주요 설계 결정

1. **deterministic exact-map oracle 구축 (`assertExactObservationFields`)**: `entry.ContextMap()`의 key 개수를 expected map의 key 개수와 엄격히 비교(len equality)하여 unexpected extra key 주입을 사전에 차단하였습니다.
2. **cause chain 4-entry 입력 순서 검증**: 서로 다른 stage/code/consumer/filter/rule을 갖는 4개의 `FailureCause` 항목을 생성하고 `cause_stages`, `cause_codes`, `cause_consumers`, `cause_filters`, `cause_rule_ids` string slice의 순서와 값이 입력과 일치하는지 exact 비교하도록 보강했습니다.
3. **terminal_reason 및 base-only fixture 구분 검증**: `ObservationKindTerminalCommitted`에 대한 terminal fixture를 별도로 emit하여 `terminal_reason` key/value 존재 및 cause chain 부재를 확인하고, `ObservationKindReleaseCommitted` base fixture는 7개 필수 base key만 존재하는지 exact map으로 검증했습니다.
4. **context sentinel 주입 검증**: custom typed context key (`prompt`, `output`, `tool_args`, `auth`, `preparer_input`)에 raw sentinel 문자열을 설정해 `sequencer.Emit`에 전달한 뒤, zap encoder JSON string 및 observer context map 어디에도 forbidden key 및 sentinel value가 포함되지 않음을 검증했습니다.

## 리뷰어를 위한 체크포인트

- all-fields/base/terminal entry의 actual key set이 expected set과 정확히 같고 extra field를 허용하지 않는가.
- base/optional/recovery/preparer/evidence 값과 4-entry cause arrays의 순서가 입력과 정확히 일치하는가.
- raw prompt/output/tool/auth/preparer sentinel을 가진 context를 전달해도 key/value가 observation log에 나타나지 않는가.
- production sink, runtime injection, public OpenAI field, metric, config/storage가 변경되지 않았는가.

## 검증 결과

각 명령의 실제 stdout/stderr를 아래에 그대로 기록한다. 출력이 없으면 `출력 없음`이라고 적는다.

### OBS-EDGE-3 중간 검증

- `go test -count=1 -v ./apps/edge/internal/openai -run '^TestFilterObservationSink_(MapsStableAllowlist|OmitsRawFieldsAndSentinels)$'`

_실제 출력:_
```text
=== RUN   TestFilterObservationSink_MapsStableAllowlist
--- PASS: TestFilterObservationSink_MapsStableAllowlist (0.00s)
=== RUN   TestFilterObservationSink_OmitsRawFieldsAndSentinels
--- PASS: TestFilterObservationSink_OmitsRawFieldsAndSentinels (0.00s)
PASS
ok  	iop/apps/edge/internal/openai	0.006s
```

### OBS-EDGE-4 중간 검증

- `test -z "$(gofmt -l apps/edge/internal/openai/filter_observation_sink_test.go)"`

_실제 출력:_
```text
출력 없음
```

### 최종 검증

- `command -v go && readlink -f "$(command -v go)" && go version && go env GOROOT`
- `go test ./apps/edge/internal/openai -list '^TestFilterObservationSink_'`
- `go test -count=1 -v ./apps/edge/internal/openai -run '^TestFilterObservationSink_'`
- `go test -count=1 ./packages/go/streamgate ./packages/go/observability ./apps/edge/internal/openai`
- `go test -race -count=1 ./apps/edge/internal/openai -run '^TestFilterObservationSink_'`
- `test -z "$(gofmt -l apps/edge/internal/openai/filter_observation_sink.go apps/edge/internal/openai/filter_observation_sink_test.go)"`
- `git diff --check`

_실제 출력:_
```text
/config/.local/bin/go
/config/opt/go/bin/go
go version go1.26.2 linux/arm64
/config/opt/go

TestFilterObservationSink_MapsStableAllowlist
TestFilterObservationSink_OmitsRawFieldsAndSentinels
TestFilterObservationSink_PropagatesCoreWriteFailure
TestFilterObservationSink_SequencerIsolatesCoreWriteFailure
ok  	iop/apps/edge/internal/openai	0.007s

=== RUN   TestFilterObservationSink_MapsStableAllowlist
--- PASS: TestFilterObservationSink_MapsStableAllowlist (0.00s)
=== RUN   TestFilterObservationSink_OmitsRawFieldsAndSentinels
--- PASS: TestFilterObservationSink_OmitsRawFieldsAndSentinels (0.00s)
=== RUN   TestFilterObservationSink_PropagatesCoreWriteFailure
--- PASS: TestFilterObservationSink_PropagatesCoreWriteFailure (0.00s)
=== RUN   TestFilterObservationSink_SequencerIsolatesCoreWriteFailure
--- PASS: TestFilterObservationSink_SequencerIsolatesCoreWriteFailure (0.00s)
PASS
ok  	iop/apps/edge/internal/openai	0.006s

ok  	iop/packages/go/streamgate	0.884s
ok  	iop/packages/go/observability	0.013s
ok  	iop/apps/edge/internal/openai	6.994s

ok  	iop/apps/edge/internal/openai	1.016s

출력 없음

출력 없음
```

---

> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section?**
> If anything is blank, go back and fill it in before saving this file.
> Leave review-agent-only sections unchanged.

## 섹션 소유권

| Section | Owner | Note |
|---------|-------|------|
| Header comment, 개요, 리뷰 에이전트 지시 | Fixed at stub creation | Implementing agent must not modify or execute these (archive, complete.log, and task-directory archive move are review-agent only) |
| Roadmap Targets | Fixed at stub creation from plan when present | Implementing agent must not modify; code-review copies it into `complete.log` as `Roadmap Completion` only on PASS |
| Archive Evidence Snapshot | Fixed at stub creation from plan when present | Implementing agent uses it as default prior-loop context; read only the specific archive files cited there when more detail is required |
| Agent UI Completion | Mixed | Present only for plan-required agent-ui code work; implementing agent fills actual evidence, review agent applies `구현됨` status/evidence update on PASS and copies the section into `complete.log` |
| 구현 항목별 완료 여부 (item names) | Fixed at stub creation | Implementing agent checks `[ ]` → `[x]` only |
| 구현 체크리스트 (item text/order) | Fixed at stub creation from plan | Implementing agent checks `[ ]` → `[x]` only |
| 코드리뷰 전용 체크리스트 | Review agent only | Implementing agent must not modify or check this section |
| 계획 대비 변경 사항, 주요 설계 결정 | Implementing agent | Replace placeholder text with actual content |
| 리뷰어를 위한 체크포인트 | Fixed at stub creation | Pre-filled from plan |
| 검증 결과 (section headings + commands) | Fixed at stub creation | Implementing agent fills in command output only; command changes require a `계획 대비 변경 사항` entry |

## 코드리뷰 결과

### 종합 판정

PASS

### 차원별 평가

- correctness: Pass — exact allowlist helper가 key 수와 각 mapped value를 검증하고 all-fields/base/terminal 분기를 분리해 production mapping과 일치한다.
- completeness: Pass — 이전 Required였던 4-entry cause 순서·bound, `terminal_reason`, 실제 context sentinel 주입이 모두 구현되었다.
- test coverage: Pass — 네 개의 named sink test가 mapping, raw-free 경계, direct write failure, sequencer failure isolation을 검증한다.
- API contract: Pass — production sink와 public OpenAI/streamgate 계약을 변경하지 않고 테스트 오라클만 보강했다.
- code quality: Pass — target 파일에 debug 출력, dead code, TODO 또는 불필요한 범위 확장이 없다.
- implementation deviation: Pass — `PLAN-cloud-G03.md`의 test-only 범위와 검증 명령을 그대로 이행했다.
- verification trust: Pass — reviewer가 host Go `go1.26.2 linux/arm64`에서 test-list, fresh unit/package/race, gofmt, `git diff --check`를 재실행해 기록과 일치함을 확인했다.

### 발견된 문제

없음

### 라우팅 신호

- review_rework_count=2
- evidence_integrity_failure=false

### 다음 단계

- PASS: `complete.log`를 작성하고 task artifact를 월별 archive로 이동한다.
