chore(atp): retrospective 섹션 + memory_candidates 기록

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K3FeMrbtxfTScjrwUukyHD
This commit is contained in:
이정수 2026-06-29 19:24:33 +09:00
parent 7047e06791
commit 2c48184fa7
1 changed files with 201 additions and 0 deletions

View File

@ -89,3 +89,204 @@ positive:
- '계획 합의 게이트(AskUserQuestion 3축) 1회에 즉시 수락 — 옵션·Recommended 제시가 적중.' - '계획 합의 게이트(AskUserQuestion 3축) 1회에 즉시 수락 — 옵션·Recommended 제시가 적중.'
negative: [] negative: []
# 세션 중 부정 시그널(지적·반복요청·오류지적) 없음. 명시 피드백은 합의 답변 3건뿐. # 세션 중 부정 시그널(지적·반복요청·오류지적) 없음. 명시 피드백은 합의 답변 3건뿐.
## Summary
3트랙 완료. 8커밋(메인스트림 아님 feat/v2, push 미수행).
- **B2**: abstracts(4)·header.jspf·GameCatalog 삭제 + 없는 게임 ID HTTP 404(ResponseStatusException, advice 핸들러 status 보존). 회귀 가드 추가.
- **B4**: Spring Boot 3.5.16 + snapshot repo 제거 + OWASP DC 12.2.2(비bind) + 세션쿠키 하드닝(HttpOnly/SameSite=Lax, live secure) + ibatis 로그 WARN(dev TRACE) + multipart 1GB 유지(리스크 문서화).
- **FE**: index 검색 `<details>` 접이식 위계 정리 + 다크 input 대비 강화 + 체크박스 터치타깃(4화면).
검증: mvn test 361 PASS, 런타임 404/쿠키/FE 브라우저 스모크 확인, snapshot 없이 빌드 재현. graph src 재생성.
부수: 직전 B3 사전결함(BibimbapApplicationTests mock 누락) 보강. dev override offline prefetch 함정 문서화.
ended_at: 2026-06-29T19:16:00+09:00
## Retrospective
```yaml
Retrospective:
signals:
positive:
- quote_or_paraphrase: "계획 합의 게이트(AskUserQuestion 3축) 1회에 즉시 수락"
about: >
orchestrator 가 B4 설정 위치 불일치(application-dev/live.properties 부재)를
plan-gate 에서 옵션+Recommended 형식으로 제시했고, 사용자가 재질의 없이
1라운드에 D-B4-cfgloc 확정. 옵션 정렬 + Recommended 명시 패턴이 유효함.
negative:
- quote_or_paraphrase: "(B3 직전 세션) GameLikesMapper @MockBean 누락 — 이번 verification 에서 표면화"
about: >
B3 에서 GameController 에 GameLikesMapper 의존 추가 후 BibimbapApplicationTests
@MockBean 미갱신. verification-strategies 규약("신규 컨트롤러/매퍼 의존 변경 시
full test 의무")이 B3 implementation 단계에서 지켜지지 않아, 다음 세션(B2/B4)
검증 단계에서야 발견됨.
structural: true
- quote_or_paraphrase: "404 전역핸들러 가림 — mvn test 통과 후 런타임 스모크에서 발견"
about: >
ResponseStatusException(404) 이 기존 @ExceptionHandler(Exception.class) 에
HTTP 500 으로 먹혔으나 L1(단위/회귀)은 PASS. 런타임 브라우저 스모크에서야 발견.
"HTTP 상태코드 변경 AC 는 L1 만으로 검증 불충분" 이라는 갭.
structural: true
- quote_or_paraphrase: "advisor summary 불완전 종료 — FE implementation-advisor 가 'Worker C만 완료, A/B 대기'로 보고했으나 실제론 4파일 적용됨"
about: >
advisor 의 최종 summary 가 실제 적용 상태와 불일치. orchestrator 가 git diff 로
실변경을 직접 확인해 진행했으나, summary 신뢰성 문제는 구조적.
structural: true
- quote_or_paraphrase: "graphify 스킬이 graphify-out/ 에 출력 — src/ 소스트리 오염, 수동 삭제"
about: >
graphify 스킬의 출력 디렉토리가 graphify-out/ 고정인데, 프로젝트 관례는
docs/graph/<scope>/. 소스트리 루트에 graphify-out/ 이 생성돼 수동 삭제.
structural: false
what_went_well:
- "plan-gate 에서 B4 가정불일치(profile properties 부재)를 research 단계에서 선제 발견해 사용자 옵션+Recommended 로 제시 → 1라운드 결정 수렴."
- "OWASP Dependency-Check executions 없이 비bind 로 설계(평소 빌드 비영향) + autoUpdate=false 오프라인 친화 설정 — research-advisor 의 사실조사 품질이 설계 결정을 뒷받침."
- "B2 dead code 삭제 안전성: abstracts 패키지 외부 참조 0, header.jspf include 0 를 rg 전수 + 상속 키워드 보강으로 더블체크. 삭제 후 컴파일 무파손."
- "graph src 재생성을 세션 내에서 완료(partial-stale 판정 → 재생성 → index.md 갱신), docs scope 이월은 근거 명시 후 투명 기록."
what_to_improve:
- >
[구조적] B3 에서 매퍼 의존 추가 후 @MockBean 미갱신: verification-strategies 규약이
이미 존재하나 이전 세션 implementation 단계에서 지켜지지 않았음. implementation-advisor
체크리스트에 "컨트롤러 매퍼 의존 변경 시 BibimbapApplicationTests @MockBean 목록 동기화
확인" 단계를 명시적 항목으로 포함 필요.
- >
[구조적] HTTP 상태코드 변경(404 등) AC 는 런타임 스모크 없이 L1 만으로 검증 불충분.
@ExceptionHandler 계층 우선순위는 단위테스트로는 탐지 안 됨. 검증 레지스트리에
"HTTP status 변경 변경 포함 트랙 → 런타임 스모크 의무" 행 추가 필요.
- >
[구조적] advisor 최종 summary 가 실제 파일 적용 상태를 정확히 반영하지 못할 수 있음.
orchestrator 는 implementation-advisor 완료 보고를 git diff/실파일 확인으로 교차검증하는
습관이 필요(특히 multi-worker 세션).
- >
[단발] graphify 스킬 출력 경로(graphify-out/)와 프로젝트 관례(docs/graph/<scope>/)
불일치. 스킬 호출 전 출력 경로를 프로젝트 관례로 사전 지정하거나, 호출 후 이동하는
절차가 필요. src/ 소스트리 오염 방지.
memory_candidates:
- name: http-status-change-requires-runtime-smoke
type: feedback
description: >
HTTP 상태코드 변경(4xx/5xx 전환) 트랙은 L1 단위테스트만으로 검증 불충분
@ExceptionHandler 계층 우선순위는 런타임에서만 드러남.
body_draft: |
## Why
ResponseStatusException(404) 이 기존 @ExceptionHandler(Exception.class) 에 HTTP 500
으로 가려진 사례(세션 20260629-175705 B2). L1(mvn test) 은 PASS 했으나 런타임
브라우저 스모크에서야 발견. @ExceptionHandler 계층 우선순위 + ResponseStatusException
전파 경로는 단위테스트 mock 환경에서 재현되지 않는다.
## How to apply
- HTTP 상태코드 변경이 포함된 트랙(4xx 신규 도입, 에러 핸들러 수정 등)은
검증 레지스트리 "버그 범주 → L 레벨" 표에서 "런타임 스모크 의무" 로 분류.
- 런타임 스모크: 해당 AC 엔드포인트에 curl/브라우저로 기대 상태코드 직접 확인.
- verification-advisor 는 이 분류 트랙에 대해 "L1 PASS = 검증 완료" 로 종료하지 않고
L3 스모크 항목을 needs_user_verification 에 포함.
rationale_for_saving: >
@ExceptionHandler 계층 가림은 단위테스트로 탐지 불가 — L1 GREEN 이 "404 동작함"을
보장하지 않는다는 비자명한 패턴. 재발 가능성 높음(향후 에러 핸들러 변경마다).
signal_source: negative
docs_sync_target: /Users/wemadeplay/workspace/stz/bibimbap/docs/development/verification-strategies.md
memory_optional: true
- name: plan-gate-option-recommended-pattern
type: feedback
description: >
가정불일치(현실-설계 gap)를 plan-gate 에서 옵션+Recommended 형식으로 제시하면
사용자가 1라운드에 수렴하는 패턴이 이번 세션에서 검증됨.
body_draft: |
## Why
B4 트랙에서 research-advisor 가 "application-dev/live.properties 파일 없음"
가정불일치를 선제 발견. orchestrator 가 plan-gate AskUserQuestion 을
"옵션A / 옵션B / Recommended: B" 형식으로 구조화 → 사용자가 3축 질문을 1회에 수락.
재질의 0, 트랙 지연 0.
## How to apply
- research 단계에서 가정불일치·ambiguity 를 발견하면 설계/구현 단계로 넘기지 말고
plan-gate(AskUserQuestion)에서 표면화.
- 질문 형식: 옵션 나열 + 각 옵션 1줄 설명 + Recommended 명시.
- 한 번에 묻기: 독립 결정 여러 개를 한 메시지로 묶어 라운드 수를 최소화.
rationale_for_saving: >
1라운드 수렴 패턴이 실증(재질의 0). 비자명한 선택(가정불일치를 plan-gate에서 처리)이
검증된 긍정 패턴으로 재현 가치 있음.
signal_source: positive
docs_sync_target: /Users/wemadeplay/workspace/stz/bibimbap/docs/development/verification-strategies.md
memory_optional: true
- name: implementation-advisor-summary-cross-check
type: feedback
description: >
multi-worker implementation 세션에서 advisor 최종 summary 가 실파일 적용 상태와
불일치할 수 있음 — orchestrator 는 git diff 로 교차검증 필요.
body_draft: |
## Why
FE implementation-advisor 가 "Worker C만 완료, A/B 대기" 로 불완전 종료 보고했으나
실제론 4파일이 적용돼 있었음(세션 20260629-175705). summary 의존 시 롤백 오판
또는 불필요한 재작업 위험.
## How to apply
- implementation 완료 보고를 받은 직후 orchestrator 는 `git diff --name-only HEAD~1`
또는 `git status` 로 실변경 파일 목록을 교차 확인한다.
- summary 와 실파일 불일치 시 advisor 에 보정 요청(재summary)하거나
orchestrator 가 직접 diff 기반으로 진행.
- multi-worker(workers_spawned >= 2) 세션에서 특히 의무화.
rationale_for_saving: >
advisor summary 신뢰성 갭은 구조적이며 재발 가능. 단발 실수가 아니라
multi-worker 세션 구조에서 발생 가능한 패턴.
signal_source: negative
docs_sync_target: /Users/wemadeplay/workspace/stz/bibimbap/docs/development/verification-strategies.md
memory_optional: true
- name: graphify-output-path-project-convention
type: project
description: >
graphify 스킬 출력 경로(graphify-out/) 와 프로젝트 관례(docs/graph/<scope>/) 불일치.
스킬 호출 시 출력 경로 사전 지정 또는 호출 후 이동 필요.
body_draft: |
## 사실
graphify 스킬은 출력을 graphify-out/ 고정 디렉토리에 생성한다.
bibimbap 프로젝트 관례는 docs/graph/<scope>/ (예: docs/graph/src/, docs/graph/docs/).
스킬 호출 위치에 따라 src/ 루트 아래 graphify-out/ 이 생성돼 소스트리 오염.
세션 20260629-175705 에서 src/graphify-out 수동 삭제.
## How to apply
- /graphify 호출 전: 스킬 문서(~/.claude/skills/graphify/SKILL.md) 에서
출력 경로 파라미터 확인 후 docs/graph/<scope>/ 로 지정.
- 또는 스킬 완료 후 즉시 mv graphify-out/ docs/graph/<scope>/.
- docs/graph/index.md frontmatter 에 source_commit 갱신.
rationale_for_saving: >
스킬-프로젝트 관례 불일치는 스킬 호출마다 재발하는 구조적 함정.
코드/git log 에서 유도 불가(관례 vs 스킬 기본 경로).
signal_source: negative
docs_sync_target: /Users/wemadeplay/workspace/stz/bibimbap/docs/development/local-dev-setup.md
memory_optional: true
protocol_feedback:
- target: implementation-advisor (체크리스트)
description: >
[구조적] 컨트롤러 매퍼 의존 추가/변경 후 BibimbapApplicationTests @MockBean 동기화
확인을 implementation 단계 체크리스트 필수 항목으로 포함. 현재 verification-strategies
에 규약이 있으나 implementation 단계 체크리스트에는 없어 이전 세션에서 누락됨.
근거: B3 @MockBean 미갱신 → B2/B4 세션 verification FAIL.
- target: verification-strategies.md (버그 범주 → L 레벨 표)
description: >
[구조적] "HTTP 상태코드 변경(4xx 신규·에러 핸들러 수정)" 행 추가 — "L1 + 런타임 스모크
(curl/browser 기대 status 직접 확인)" 의무. @ExceptionHandler 계층 우선순위는
L1 단위테스트로 탐지 불가.
근거: ResponseStatusException(404) 이 @ExceptionHandler(Exception.class) 에 500 으로
가려진 사례(세션 20260629-175705).
- target: agent-team-protocol (orchestrator 체크리스트 또는 §implementation 완료 처리)
description: >
[구조적] multi-worker implementation 세션 완료 시 orchestrator 의
git diff 교차검증을 의무 단계로 명시. advisor summary 만 신뢰하면 실변경과 다를 수 있음.
"summary != diff 시 처리 경로(재summary 요청 또는 diff 기반 진행)" 명시.
근거: FE advisor summary 불완전 종료 사례(세션 20260629-175705).
- target: graphify skill (SKILL.md 또는 호출 가이드)
description: >
[단발 → 패턴] graphify-out/ 고정 출력 vs 프로젝트 docs/graph/<scope>/ 관례 불일치.
스킬에 출력 경로 파라미터 지원 추가 또는, 호출 가이드에 "프로젝트 관례 경로로
이동 후 index.md 갱신" 절차 명시 권고.
applied_changes: []
```