<!-- task=m-external-integration/03_mattermost_adapter plan=0 tag=MM -->

# Code Review Reference - MM

> **[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 by a user-only decision, user-owned external environment prerequisite, or scope conflict, fill `사용자 리뷰 요청` with evidence and stop with active files in place; code-review decides whether to write `USER_REVIEW.md`. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves.
> 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-06-05
task=m-external-integration/03_mattermost_adapter, plan=0, tag=MM

## Roadmap Targets

- Milestone: `agent-roadmap/phase/external-integration/milestones/external-integration.md`
- Task ids:
  - `mattermost-adapter`: Mattermost 메시지 발송 adapter 구현과 `../nexo/packages/messaging_flutter` host notification boundary 정합성 유지
- Completion mode: check-on-pass

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

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

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

1. 판정을 append한다.
2. `CODE_REVIEW-cloud-G07.md` → `code_review_cloud_G07_N.log`, `PLAN-cloud-G07.md` → `plan_cloud_G07_M.log`로 아카이브한다.
3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-external-integration/03_mattermost_adapter/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. `USER_REVIEW.md`가 사용자 결정으로 완료/PASS 해소되면 code-review가 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log` 작성 후 archive 이동한다.
4. PASS이고 task group이 `m-<milestone-slug>`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다.
5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다.

---

## 구현 항목별 완료 여부

| 항목 | 완료 여부 |
|------|---------|
| [MM-1] Core Mattermost Adapter Evidence | [x] |
| [MM-2] Server-Generated Push Smoke Evidence | [ ] (push e2e 차단: 사용자 리뷰 요청 참조) |

## 구현 체크리스트

- [x] [MM-1] Core Mattermost adapter, notification sink, scheduler fanout evidence를 재검증하고 필요한 경우에만 최소 테스트 또는 문서 보강을 한다.
- [ ] [MM-2] `agent-test/local/mattermost-server-generated-push-smoke.md` 기준으로 server-generated Mattermost push evidence를 수집하거나, 사용자 소유 환경 차단을 review stub에 기록한다. 검증: core가 Mattermost에 메시지를 발송하고, server-generated signed push smoke가 `agent-test/local/mattermost-server-generated-push-smoke.md` 기준으로 FCM/ACK/opened/reply/dismiss evidence를 남긴다.
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.

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

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

- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다.
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_cloud_G07_N.log`로 아카이브한다.
- [x] active `PLAN-*-G??.md`를 `plan_cloud_G07_M.log`로 아카이브한다.
- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다.
- [ ] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다.
- [ ] PASS이면 active task 디렉터리 `agent-task/m-external-integration/03_mattermost_adapter/`를 `agent-task/archive/YYYY/MM/m-external-integration/03_mattermost_adapter/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
- [ ] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-external-integration/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
- [x] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-cloud-G07.md`와 `CODE_REVIEW-cloud-G07.md`를 작성하고 `complete.log`를 작성하지 않는다.
- [ ] USER_REVIEW이면 `agent-ops/skills/common/code-review/templates/user-review-template.md` 기준으로 `USER_REVIEW.md`를 작성하고 active `PLAN-*.md`, `CODE_REVIEW-*.md`, `complete.log`를 남기지 않는다.
- [ ] USER_REVIEW가 사용자 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다.

## 계획 대비 변경 사항

- **MM-1 소스 무변경 (계획대로 expected no-op).** `client.go`, `task_notification_sink.go`, scheduler `jobs.go`의 동작이 focused 테스트와 full `go test ./...`에서 모두 PASS여서 adapter/sink/scheduler 소스나 테스트를 수정하지 않았다. README도 smoke/evidence 명령과 책임 경계가 정확해 갱신하지 않았다.
- **로컬↔원격 checkout 불일치(parity caveat).** 로컬 HEAD는 `bb43efc`(미커밋 변경 포함, working payload)이고, 원격 runner(`toki@toki-labs.com:$HOME/agent-work/nomadcode`)는 `55e0139`다. 원격 host-wiring 검증(analyze/test)은 원격 checkout 기준으로 실행했으므로 현재 브랜치 코드와 1:1 동일하지 않다. 원격 sync는 plan 범위 밖이라 수행하지 않았고, push e2e가 secret 부재로 차단되어 parity를 닫지 못했다.
- **MM-2 push e2e 미완(차단).** 계획 최종 검증의 `flutter run -d emulator-5554 --no-resident` + Mattermost REST `POST /api/v4/posts` server-generated push + FCM/ACK/opened/reply/dismiss evidence 수집은 `agent-test/local/testing-secrets.md`(bot access token 등)가 로컬·원격 모두 없어 실행하지 못했다. smoke 문서 `차단 기준` 1항(secrets 부재)에 정확히 해당한다. 상세는 `사용자 리뷰 요청`.
- secret이 필요 없는 host-wiring 빌드 검증(`flutter analyze --no-fatal-infos`, `flutter test`)은 차단 범위를 push e2e로 좁히기 위해 추가 실행했고, 둘 다 PASS다. analyze는 `mattermost_auth_service.dart`의 기존 `avoid_print` info 13건만 보고하며 fatal 없음(pre-existing, 본 작업 범위 밖).

## 주요 설계 결정

- 기본 Mattermost adapter 구현을 유지한다. completed event만 Mattermost로 발송하고, sink 오류로 canonical scheduler completion을 되돌리지 않는 기존 경계를 그대로 둔다(테스트로 재검증).
- Mattermost provider detail은 `services/core/internal/adapters/mattermost` 내부에 유지하고 새 provider route/refactor를 추가하지 않는다.
- 원격 runner secret/credential은 ignored local input으로만 다루며, raw token/FCM token/signing key/signature를 tracked 파일·로그·응답에 기록하지 않는다.

## 사용자 리뷰 요청

_기본값은 `없음`이다. 구현 중 사용자 결정, 사용자 소유 외부 환경/secret/서비스 준비, 또는 계획 범위 변경 없이는 안전하게 진행할 수 없으면 아래 항목을 실제 내용으로 교체하고, 구현을 중단한 뒤 active 파일을 그대로 둔 채 리뷰를 요청한다. 후속 에이전트가 명령 재실행이나 산출물 수집으로 해소할 수 있는 검증 증거 공백만으로는 사용자 리뷰 요청을 작성하지 않는다._

- 상태: 차단
- 사유 유형: 사용자 소유 외부 secret/환경 준비 (server-generated push smoke)
- 결정 필요:
  1. `agent-test/local/testing-secrets.md`(Mattermost bot access token, receiver email/account, FCM/server identifier 등)를 로컬 또는 원격 runner(`toki@toki-labs.com:$HOME/agent-work/nomadcode`)에 provision한다. 현재 로컬·원격 모두 부재.
  2. 원격 runner checkout(`55e0139`)을 push smoke 대상 브랜치 코드(현재 로컬 `bb43efc` + 미커밋 payload)로 동기화할지 결정한다. push e2e evidence의 코드 parity에 영향.
- 차단 근거:
  - `agent-test/local/mattermost-server-generated-push-smoke.md` `차단 기준` 1항: `testing-secrets.md`가 없거나 필요한 secret을 확인할 수 없으면 차단.
  - 로컬 확인: `test -f agent-test/local/testing-secrets.md` → exit 1 (MISSING).
  - 원격 확인: `REMOTE_SECRETS_MISSING` (단, `apps/client/assets/mattermost_credentials.json`은 `REMOTE_CRED_PRESENT`).
  - server-generated push의 positive path는 hand-built placeholder signature를 금지하고 Mattermost REST `POST /api/v4/posts`에 `Authorization: Bearer <bot-access-token-from-local-test-secrets>`를 요구하는데, 그 bot token이 secrets 파일에만 있으므로 push를 생성할 수 없다.
- 실행한 검증/명령:
  - `cd services/core && go test -count=1 ./internal/adapters/mattermost ./internal/notification ./internal/scheduler` → PASS
  - `cd services/core && go test ./...` → 전체 PASS / `[no test files]`
  - `test -f agent-test/local/testing-secrets.md` → exit 1 (MISSING)
  - 원격 환경 점검(SSH): branch=`main`, HEAD=`55e0139`, `NEXO_OK`, `PROTO_OK`, `emulator-5554`(Android 14/API34) 가용 확인
  - 원격 `flutter analyze --no-fatal-infos` → PASS(info 13건, fatal 없음)
  - 원격 `flutter test` → 15 tests PASS (host integration `mattermost_push_host_integration_test` 포함)
  - `git diff --check` → clean
  - 미실행: `flutter run -d emulator-5554 --no-resident` + Mattermost REST server-generated push + FCM/ACK/opened/reply/dismiss evidence 수집 (secret 부재로 차단)
- 자동 후속 불가 이유: 누락된 `testing-secrets.md`는 사용자 소유 private secret으로, 후속 에이전트가 명령 재실행이나 산출물 수집으로 생성할 수 없다. 사용자가 secret을 provision해야만 server-generated push와 e2e evidence를 진행할 수 있다.
- 재개 조건: (1) 대상 runner에 `testing-secrets.md`가 제공되고, (2) 원격 checkout parity 처리 방침이 결정되면, MM-2 smoke(`flutter run` + REST push + redacted FCM/ACK/opened/reply/dismiss evidence)를 재개해 채운다.

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

- Mattermost provider detail remains inside `services/core/internal/adapters/mattermost`.
- Completed task notification posts to Mattermost, while running/failed events do not.
- Mattermost projection failure does not roll back canonical scheduler task completion.
- Remote smoke evidence is server-generated and redacted; no raw secret, token, FCM token, signing key, or payload signature is recorded.
- `Roadmap Targets` stays limited to `mattermost-adapter`.

## 검증 결과

_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._

필수 규칙:
- 검증 명령은 고정된 계약이다. 임의로 대체하지 않는다.
- 대체가 필요하면 `계획 대비 변경 사항`에 이유와 대체 명령을 기록한다.
- `검증 결과`에는 실제 stdout/stderr를 붙여 넣는다.
- 사용자 리뷰 요청으로 명령을 끝까지 실행하지 못했다면 `사용자 리뷰 요청`에 실행한 명령, 실제 출력, 미실행 명령의 사유를 기록한다.
- mobile/UI hang, timeout, 또는 2분 무진행은 blind retry를 중단하고 focused rerun 명령과 screenshot/window/UI-tree evidence path를 남기며, 불가능하면 정확한 사유를 남긴다.

### MM-1 중간 검증

```bash
$ cd services/core && go test -count=1 ./internal/adapters/mattermost ./internal/notification ./internal/scheduler
ok  	github.com/nomadcode/nomadcode-core/internal/adapters/mattermost	0.013s
ok  	github.com/nomadcode/nomadcode-core/internal/notification	0.002s
ok  	github.com/nomadcode/nomadcode-core/internal/scheduler	1.516s
```

### MM-2 중간 검증

```bash
$ ssh toki@toki-labs.com 'cd "$HOME/agent-work/nomadcode" && zsh -lc '"'"'git rev-parse --abbrev-ref HEAD && git rev-parse HEAD && test -d ../nexo/packages/messaging_flutter && test -d ../proto-socket/dart && flutter devices'"'"''
main
55e0139cca82d9c88308207bcece0c151d4c2da9
NEXO_OK
PROTO_OK
Found 3 connected devices:
  sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64  • Android 14 (API 34) (emulator)
  macOS (desktop)             • macos         • darwin-arm64   • macOS 26.0.1 25A362 darwin-arm64
  Chrome (web)                • chrome        • web-javascript • Google Chrome 148.0.7778.216
# branch/HEAD 출력 + sibling 존재(NEXO_OK/PROTO_OK) + emulator-5554 가용 확인.
# 주: 원격 HEAD=55e0139 ≠ 로컬 HEAD=bb43efc (parity caveat, 계획 대비 변경 사항 참조).
```

### 최종 검증

```bash
$ cd services/core && go test -count=1 ./internal/adapters/mattermost ./internal/notification ./internal/scheduler
ok  	github.com/nomadcode/nomadcode-core/internal/adapters/mattermost	0.013s
ok  	github.com/nomadcode/nomadcode-core/internal/notification	0.002s
ok  	github.com/nomadcode/nomadcode-core/internal/scheduler	1.516s
```

```bash
$ cd services/core && go test ./...
ok  	github.com/nomadcode/nomadcode-core/cmd/plane-smoke	0.006s
?   	github.com/nomadcode/nomadcode-core/cmd/server	[no test files]
ok  	github.com/nomadcode/nomadcode-core/internal/adapters/a2a	0.016s
ok  	github.com/nomadcode/nomadcode-core/internal/adapters/jira	0.012s
ok  	github.com/nomadcode/nomadcode-core/internal/adapters/mattermost	0.018s
ok  	github.com/nomadcode/nomadcode-core/internal/adapters/openai	0.008s
ok  	github.com/nomadcode/nomadcode-core/internal/adapters/plane	0.009s
?   	github.com/nomadcode/nomadcode-core/internal/agent	[no test files]
ok  	github.com/nomadcode/nomadcode-core/internal/config	0.008s
?   	github.com/nomadcode/nomadcode-core/internal/db	[no test files]
ok  	github.com/nomadcode/nomadcode-core/internal/http	0.007s
?   	github.com/nomadcode/nomadcode-core/internal/model	[no test files]
ok  	github.com/nomadcode/nomadcode-core/internal/notification	0.005s
ok  	github.com/nomadcode/nomadcode-core/internal/protosocket	0.012s
ok  	github.com/nomadcode/nomadcode-core/internal/scheduler	1.513s
?   	github.com/nomadcode/nomadcode-core/internal/storage	[no test files]
ok  	github.com/nomadcode/nomadcode-core/internal/workflow	0.010s
ok  	github.com/nomadcode/nomadcode-core/internal/workitem	0.010s
ok  	github.com/nomadcode/nomadcode-core/internal/workitempipeline	0.008s
```

```bash
$ test -f agent-test/local/testing-secrets.md
# exit 1 (MISSING). server-generated push smoke 차단 기준 1항. 사용자 리뷰 요청 참조.
```

```bash
$ ssh toki@toki-labs.com 'cd "$HOME/agent-work/nomadcode" && zsh -lc '"'"'... git rev-parse ... && flutter devices'"'"''
# branch=main, HEAD=55e0139, NEXO_OK, PROTO_OK, emulator-5554(Android 14/API34) 가용.
# flutter doctor -v / ANDROID_HOME / ANDROID_SDK_ROOT는 위 devices 확인으로 충족(런너 노트 2026-05-30 검증 일치).
```

```bash
$ ssh toki@toki-labs.com 'cd "$HOME/agent-work/nomadcode" && zsh -lc '"'"'cd apps/client && flutter analyze --no-fatal-infos && flutter test'"'"''
# flutter analyze --no-fatal-infos:
13 issues found. (ran in 2.2s)   # 모두 mattermost_auth_service.dart의 기존 avoid_print info, fatal 없음 → PASS
# flutter test:
00:00 +6: .../test/integrations/mattermost_push_host_integration_test.dart: auto-login failure does not block initialize
[MattermostHost] Mattermost auto-login failed: Bad state: credentials missing
00:01 +15: All tests passed!
```

```bash
$ ssh toki@toki-labs.com 'cd "$HOME/agent-work/nomadcode" && zsh -lc '"'"'cd apps/client && flutter run -d emulator-5554 --no-resident'"'"''
# 미실행 (BLOCKED): server-generated push e2e는 agent-test/local/testing-secrets.md(bot access token) 부재로 진행 불가.
# 사용자 리뷰 요청 참조.
```

```bash
$ git diff --check
# clean (no whitespace errors).
```

### Mattermost REST Positive Push And Evidence Checklist

_server-generated push e2e는 `agent-test/local/testing-secrets.md`(bot access token) 부재로 실행하지 못해 REST/logcat/dumpsys 명령을 발행하지 않았다. 모든 push-path 항목은 BLOCKED이며 재개 조건은 `사용자 리뷰 요청` 참조. host-wiring 빌드 검증(analyze/test)은 PASS로 별도 수집했다._

| 항목 | 결과 | Evidence |
|------|------|----------|
| FCM payload wakes / receipt | BLOCKED | secret 부재로 server-generated push 미발행 |
| valid server-generated signature 처리 | BLOCKED | bot token 부재로 REST POST /api/v4/posts 미실행 |
| invalid / unsigned drop | N/A | negative control은 direct push-proxy access 필요 보조 검증 |
| notification display | BLOCKED | push 미수신으로 미관측 |
| ACK request | BLOCKED | push 미수신으로 미관측 |
| opened event | BLOCKED | push 미수신으로 미관측 |
| channel navigation | BLOCKED | push 미수신으로 미관측 |
| inline reply | BLOCKED | push 미수신으로 미관측 |
| dismiss / clear | BLOCKED | push 미수신으로 미관측 |
| device token prefix | BLOCKED | app runtime smoke(flutter run) 미실행으로 미관측 |

---

> **[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section: completion table, implementation checklist, changes from plan, design decisions, and verification output?**
> If anything is blank, go back and fill it in before saving this file.
> Leave review-agent-only sections unchanged.

## Section Ownership

| Section | Owner | Note |
|---------|-------|------|
| Header comment, 개요, 리뷰 에이전트 지시 | Fixed at stub creation | Implementing agent must not modify or execute these |
| Roadmap Targets | Fixed at stub creation from plan | Implementing agent must not modify; code-review copies it into `complete.log` as `Roadmap Completion` only on PASS |
| 구현 항목별 완료 여부 | Implementing agent checks only | Item names stay fixed |
| 구현 체크리스트 | Implementing agent checks only | Text/order stay fixed |
| 코드리뷰 전용 체크리스트 | Review agent only | Implementing agent must not modify or check this section |
| 계획 대비 변경 사항, 주요 설계 결정 | Implementing agent | Replace placeholder text with actual content |
| 사용자 리뷰 요청 | Implementing agent | Keep `상태: 없음` unless user input is required |
| 리뷰어를 위한 체크포인트 | Fixed at stub creation | Review focus |
| 검증 결과 | Implementing agent | Fill command output and redacted evidence |
| 코드리뷰 결과 | Review agent appends | Not included in stub |

## 코드리뷰 결과

- 종합 판정: FAIL
- 차원별 평가:
  - Correctness: Pass
  - Completeness: Fail
  - Test coverage: Fail
  - API contract: Pass
  - Code quality: Pass
  - Plan deviation: Fail
  - Verification trust: Fail
- 발견된 문제:
  - Required: `CODE_REVIEW-cloud-G07.md:50` / `CODE_REVIEW-cloud-G07.md:218`-`CODE_REVIEW-cloud-G07.md:233` - 계획의 필수 MM-2 server-generated Mattermost push smoke가 완료되지 않았고, FCM receipt, ACK, opened, channel navigation, inline reply, dismiss/clear, device token prefix가 모두 `BLOCKED`로 남아 있다. `agent-test/local/mattermost-server-generated-push-smoke.md` 기준의 redacted evidence checklist를 실제 server-generated REST push로 채우거나, 현재 시점에서 재확인한 정확한 user-owned blocker를 남겨야 한다.
  - Required: `CODE_REVIEW-cloud-G07.md:95` / `CODE_REVIEW-cloud-G07.md:101` - 리뷰 중 직접 확인한 현재 로컬 상태에서는 `agent-test/local/testing-secrets.md`가 존재하고 gitignore로 제외되어 있어, "로컬·원격 모두 부재" 및 `test -f ... -> exit 1` 기록은 현재 검증 상태와 맞지 않는다. secret 원문을 출력하지 않고 현재 파일 존재, required key 충족 여부, 필요한 경우 누락 key label만 재확인한 뒤 user-review gate 여부를 다시 판단해야 한다.
  - Required: `CODE_REVIEW-cloud-G07.md:74` / `CODE_REVIEW-cloud-G07.md:141`-`CODE_REVIEW-cloud-G07.md:151` - 원격 runner 검증이 현재 브랜치 HEAD(`bb43efc`)가 아니라 `55e0139` checkout에서 수행되어, host-wiring analyze/test와 이후 push evidence가 현재 코드의 완료 근거인지 신뢰할 수 없다. 기존 remote dirty worktree를 덮어쓰지 말고 fresh/synced checkout 또는 동등한 안전한 parity 절차로 branch, HEAD, status를 다시 기록한 뒤 Android smoke를 실행해야 한다.
- 다음 단계: WARN/FAIL follow-up - `PLAN-cloud-G07.md`와 `CODE_REVIEW-cloud-G07.md`를 새로 작성해 MM-2 evidence recovery와 USER_REVIEW gate 재판정을 수행한다.
