194 lines
19 KiB
Markdown
194 lines
19 KiB
Markdown
---
|
|
schema_version: 2
|
|
sid: 20260622-170857
|
|
resumed_from: 20260622-092800
|
|
started_at: 2026-06-22T17:08:57+09:00
|
|
ended_at:
|
|
user_request: |
|
|
W3-2 댓글/리뷰 고도화 검증 부채 해소 (검증 전용 — 신규 기능 추가 금지).
|
|
직전 세션 코드 L1 43/43 GREEN. 미검증분 3가지만 닫는다:
|
|
(1) DDL 적용 + dev 스키마 4객체 검증
|
|
(2) L1 회귀 가드 (mvn test 43 GREEN 재확인)
|
|
(3) L3 브라우저 스모크 (XSS 미실행·재방문 영속·다축평점·페이지네이션·키보드·UX·A2 마스킹)
|
|
완료 시 changes 문서 §검증결과 표 + security checklist B3 갱신.
|
|
---
|
|
|
|
# Summary
|
|
W3-2 댓글/리뷰 고도화 검증 부채(DDL 적용 / L1 회귀 / L3 브라우저 스모크) 전량 해소. 검증 중 서버 집계의 실버그 2건(뷰 fan-out, 매퍼 alias 케이스 폴딩) 발견·최소수정. L3 12개 항목 전수 PASS. 검증 데이터는 세션 종료 시 정리해 원본 시드 복귀.
|
|
|
|
핵심 결과:
|
|
- STEP1 DDL: dev 4객체 검증 PASS.
|
|
- STEP2 L1: mvn test 43/43 GREEN(버그 수정 후 재실행도 43/43, 무회귀).
|
|
- STEP3 L3: XSS 미실행·재방문 영속·다축6행·SVG레이더·집계·페이지네이션·키보드·C1/C2/C4·B2·A2 전부 PASS.
|
|
- 버그수정: GameReviewStatsMapper.java(alias 3개 quote), docs/game-reviews-ddl.sql + db/schema.sql(뷰 fan-out 수정), 실행 DB CREATE OR REPLACE 적용.
|
|
- 환경: 가동 중이던 stale spring-boot:run(목요일 기동, 구 JSP)을 현재 빌드로 재기동. claude-in-chrome computer 도구는 document_idle 차단으로 javascript_tool/curl/psql 조합 검증.
|
|
|
|
# Advisor Invocation Decision Log
|
|
# 각 advisor 호출/스킵 판단 즉시 1줄 append
|
|
|
|
- advisor: requirements-advisor
|
|
decision: skip
|
|
rationale: '사용자가 STEP1/2/3 + 완료처리까지 정밀 명세. 요구 모호성 0.'
|
|
checked_at: 2026-06-22T17:09:00+09:00
|
|
- advisor: graphify-lookup-advisor / research-advisor
|
|
decision: skip
|
|
rationale: '검증 전용. 신규 설계·외부조사 불요. 배경문서(changes/local-setup) 직독으로 충분.'
|
|
checked_at: 2026-06-22T17:09:00+09:00
|
|
- advisor: design-advisor / implementation-advisor
|
|
decision: skip
|
|
rationale: '코드 변경 기본 없음(검증 전용). 버그 발견 시에만 최소수정 — 그때 재평가.'
|
|
checked_at: 2026-06-22T17:09:00+09:00
|
|
- advisor: verification-advisor
|
|
decision: call
|
|
rationale: 'STEP1(DDL 멱등적용+4객체 psql 검증) + STEP2(mvn test 43 GREEN) Bash 검증 위임. 출력 verbose → 컨텍스트 격리 가치.'
|
|
checked_at: 2026-06-22T17:09:30+09:00
|
|
|
|
# Invocations
|
|
- advisor: verification-advisor
|
|
phase: STEP1+STEP2
|
|
model_choice: { tier: medium, rationale: 'Bash 실행+AC 판정, 코드추론 불요' }
|
|
result: PASS
|
|
detail: |
|
|
AC-1 game_review_axes PASS (score CHECK 1-5, axis_key CHECK 6종, UNIQUE는 INDEX ux_game_review_axes_review_axis 로 구현 — constraint 카탈로그 미표기지만 기능동등)
|
|
AC-2 game_review_stats 뷰 9컬럼 PASS
|
|
AC-3 game_comments.updated_at timestamptz NOT NULL, mismatched=0/total=3 PASS
|
|
AC-4 game_reviews.is_rating_manual boolean DEFAULT false NOT NULL PASS
|
|
AC-5 mvn test: Tests run 43 Failures 0 Errors 0 (GameComment17/GameReview20/Csrf5/App1) PASS
|
|
concerns: 'AC-1 UNIQUE 구현형태=INDEX(not constraint). 기능동등, 비차단.'
|
|
|
|
# Bug Findings (L3 실게임 검증 중 발견 — L1 mock 사각)
|
|
## BUG-1 game_review_stats 뷰 fan-out (집계 왜곡)
|
|
- 근거: docs/game-reviews-ddl.sql:171-185 (수정전) — `LEFT JOIN game_review_axes` 직접 조인.
|
|
- 증상: 리뷰에 axes 행이 존재하면 리뷰가 axes 행수(최대 6)만큼 복제 → `COUNT(*)`=11(실제 6), `AVG(rating)`=2.8(실제 3.5). axes 0행일 땐 잠복(이전 "3.8(5)" 정상), 다축 리뷰 입력 순간 발현.
|
|
- 수정: 축 평균을 서브쿼리에서 game 단위 선집계 후 LEFT JOIN(MAX), 리뷰 단위 집계는 game_reviews 단독. docs/game-reviews-ddl.sql + db/schema.sql 동기 + 실행 DB CREATE OR REPLACE 적용.
|
|
- 수정후 검증: 뷰 game3 → review_count=6, avg_rating=3.5 (실제값 일치). PASS.
|
|
|
|
## BUG-2 GameReviewStatsMapper alias 케이스 폴딩 → summary 항상 null
|
|
- 근거: GameReviewStatsMapper.java:13-15 (수정전) — `AS gameId/avgRating/reviewCount` (따옴표 없음).
|
|
- 증상: Postgres가 따옴표 없는 alias 를 소문자(gameid/avgrating/reviewcount)로 폴딩 → MyBatis Map 키 소문자. buildSummary(GameReviewController.java:372-390) 가 `stats.get("reviewCount")` camelCase 조회 → null → reviewCount=0 → summary=null. JSP(game-detail.jsp:2188 `if(!summary||!summary.reviewCount)`)는 항상 "아직 평가 없음" 오표시. 서버집계 기능 전면 무력.
|
|
- 왜 L1 통과: BibimbapApplicationTests/컨트롤러테스트가 매퍼를 @MockBean 으로 대체 → 실제 Postgres alias 폴딩 미발생. mock-vs-reality 갭.
|
|
- 수정: alias 3개 따옴표(`AS "gameId"/"avgRating"/"reviewCount"`). axis alias 는 소문자=AXIS_KEYS 일치라 유지.
|
|
- 수정후 검증: 앱 재기동 후 curl 재확인(아래 verified_by_me).
|
|
|
|
# Decisions
|
|
- 버그 발견으로 "검증 전용·코드무수정 기본" 가정 반전 → task 의 "실제 버그 발견 시 최소수정" 사전승인 하에 수정. 수정 범위: 매퍼 alias 3개 + 뷰 정의(2파일) + 실행DB 재적용. 비파괴(CREATE OR REPLACE, alias quote).
|
|
- 회귀테스트: 두 버그 모두 DB-통합 계층(L1 mock 우회). 기존 단위 harness(@MockBean)로 재현 불가 → 자동 회귀 테스트는 별도 L2 contract(dev DB 연동) 인프라 필요. 본 세션은 L3 curl+브라우저 before/after(11/2.8/null → 6/3.5/정상)를 회귀 근거로 삼고, L2 contract 테스트 신설을 open_items 로 권고.
|
|
- DDL 적용은 §6 파괴 게이트 비해당: 멱등 CREATE TABLE/ALTER ADD/CREATE OR REPLACE VIEW (DROP/TRUNCATE/rollback 아님). changes 문서서 이미 일반 DDL 분류. → 사용자 재확인 불요.
|
|
- detail 라우트 = /game/{id} (numeric games.id). dev 가시게임 id=3.
|
|
- L3 브라우저 스모크는 claude-in-chrome 필요 → advisor 미보유 → orchestrator 직접 수행.
|
|
|
|
# verified_by_me
|
|
- L1: unit+regression — mvn test 43/43 GREEN (GameComment17/GameReview20/Csrf5/App1), 버그수정 후 재실행도 43/43 (verification-advisor 2회 독립 판정).
|
|
- DDL: dev 스키마 4객체 psql 검증 (game_review_axes UNIQUE index+CHECK 6종+score 1~5 / game_review_stats 9컬럼 / game_comments.updated_at timestamptz NOT NULL / game_reviews.is_rating_manual boolean DEFAULT false NOT NULL).
|
|
- L3 (javascript_tool+curl+psql+JSP정적): XSS 미실행(alert 0회, textContent 텍스트노드) / 재방문 영속(쿠키없는 GET 서버데이터) / 다축 6행+overall 자동(false)·수동(true) / SVG레이더+aria 6축 / 집계 3.5·(6) / 페이지네이션 21건 page0=20 hasMore=true page1=1 / 키보드 roving+preventDefault / C1 disabled+aria-busy / C2 상대시각+title / C4 0/200·0/1000 / B2 2자→400 / A2 테스터·탈퇴마스킹.
|
|
- 버그수정 검증: BUG-1 뷰 review_count 11→6, avg 2.8→3.5. BUG-2 summary null→{avgRating:3.5,reviewCount:6,axes}.
|
|
- 로그 스캔: clean (앱 기동 로그 ERROR 0, 환경성 WARN만).
|
|
|
|
# needs_user_verification
|
|
- (선택) claude-in-chrome computer/screenshot 도구가 본 dev 페이지에서 document_idle 미도달로 차단됨 — 시각적 스크린샷 증빙 없음(검증은 DOM/이벤트/서버계약으로 동치 수행). 원인이 WebGL iframe/page idle 휴리스틱인지 사용자 환경 확인 권장(비차단).
|
|
- 가동 앱: 본 세션이 stale spring-boot:run(구 JSP)을 현재 빌드로 재기동해 8080에서 실행 중. 사용자가 별도 기동 흐름이 있었다면 인지 필요.
|
|
|
|
# graph_refresh
|
|
fresh → 후속 없음. graph-refresh-checker 판정: 변경분(매퍼 alias 따옴표 + 뷰 내부 집계로직)이 심볼 시그니처·뷰/테이블/컬럼 토폴로지를 바꾸지 않음(구조 시그널 0). 재생성·삭제 불필요. source_commit 메타 갱신은 선택(필수 아님) — 본 세션 미수행.
|
|
|
|
# ended_at
|
|
2026-06-22T17:50:00+09:00
|
|
|
|
# open_items
|
|
- L2 contract 테스트 신설 권고: game_review_stats 뷰 집계 정합 + GameReviewStatsMapper Map 키를 dev DB 연동으로 가드. BUG-1/2가 L1 @MockBean 사각으로 누출된 근본원인 차단용. (changes 문서 §범위밖/이월 + security checklist 연계)
|
|
- 미커밋 잔여 0 목표 — 본 작업 단위 커밋으로 마감.
|
|
|
|
# user_signals
|
|
positive:
|
|
- quote: "dev 픽스처로 자동 로그인 (Recommended) 수락"
|
|
note: 제안한 검증 경로/권장안을 1회에 수락(이견 없음).
|
|
negative: []
|
|
|
|
# Retrospective
|
|
Retrospective:
|
|
signals:
|
|
positive:
|
|
- quote_or_paraphrase: "dev 픽스처로 자동 로그인 (Recommended) 수락"
|
|
about: "L3 브라우저 스모크 진입 전 인증 우회 경로로 'dev 픽스처 자동 로그인' 권장안을 제시 → 사용자가 1회에 수락(이견·재지시 0). 통상 검증 인증 셋업은 왕복 협의가 흔한데 권장안 단독 채택됨."
|
|
negative: []
|
|
what_went_well:
|
|
- "검증 전용 task 였으나 L3 실게임 스모크에서 서버 집계 실버그 2건(뷰 fan-out, 매퍼 alias 케이스 폴딩)을 잡아냄. L1 43/43 GREEN 만 믿지 않고 실제 런타임·DB 까지 내려간 판단이 사용자 노출 직전 차단으로 이어짐."
|
|
- "computer/screenshot 도구가 document_idle 미도달로 차단된 상황에서 멈추지 않고 javascript_tool + curl + psql + JSP 정적 조합으로 동치 검증을 구성해 L3 12항목 전수 PASS. 도구 1종 차단을 검증 포기 사유로 삼지 않음."
|
|
- "버그 발견 시 task 의 '실버그 발견 시 최소수정' 사전승인 범위 안에서만 비파괴 수정(CREATE OR REPLACE, alias quote)으로 한정. 검증 전용 scope 를 임의 확장하지 않음."
|
|
- "advisor 호출/스킵 판단을 즉시 1줄 로그로 남기고(검증 전용 → 설계/구현 advisor skip, verbose Bash 검증만 verification-advisor 위임), 버그 발견 시 'minimal-fix 재평가' 트리거를 사전 명시."
|
|
- "수정 후 회귀 근거를 before/after 수치(뷰 11/2.8/null → 6/3.5/정상)로 명시하고, L1 @MockBean 으로 재현 불가한 한계를 인정하며 L2 contract 신설을 open_item 으로 위임. 회귀 가드의 공백을 숨기지 않음."
|
|
what_to_improve:
|
|
- "BUG-2(매퍼 alias 케이스 폴딩→summary 항상 null)는 서버 집계 기능을 전면 무력화하는 사용자 가시 버그였는데 L1 43건 전부가 매퍼를 @MockBean 으로 대체해 우회했다. DB-통합 계층(매퍼 Map 키, 뷰 집계 정합)에 대한 자동 회귀 가드가 0인 상태가 구조적 사각. L2 contract(dev DB 연동) 부재가 근본원인."
|
|
- "stale spring-boot:run(목요일 기동, 구 JSP)이 8080 을 점유한 채로 검증을 시작하면 JSP docBase 가 동결돼 라이브 리로드가 안 되고 현재 빌드가 검증되지 않는다. 검증 진입 전 '가동 중 앱이 현재 빌드인지' 확인하는 절차가 부재했다(재기동으로 해소했으나 절차화 안 됨)."
|
|
- "claude-in-chrome computer/screenshot 가 본 dev 페이지(WebGL iframe 추정)에서 document_idle 미도달로 차단되는 원인이 미규명. 시각 증빙이 매번 막히면 L3 효율이 떨어지므로 idle 휴리스틱 우회/대체 증빙 표준이 필요."
|
|
memory_candidates:
|
|
- name: mock-vs-reality-db-contract-gap
|
|
type: feedback
|
|
description: "@MockBean 매퍼 대체 단위테스트는 DB-방언 계약 버그(alias 케이스 폴딩·뷰 fan-out)를 구조적으로 놓친다 — DB-통합 계약은 L2 로 가드."
|
|
body_draft: |
|
|
What: 컨트롤러/애플리케이션 단위테스트가 MyBatis 매퍼를 @MockBean 으로 대체하면,
|
|
실제 Postgres 가 일으키는 계약 결함이 전부 우회된다. 검증된 사각 2종:
|
|
1. 따옴표 없는 SQL alias(AS gameId)는 Postgres 가 소문자로 폴딩(gameid) → MyBatis Map 키 소문자
|
|
→ 컨트롤러의 camelCase get("reviewCount") 이 null → summary 전면 무력.
|
|
2. 집계 뷰의 1:N LEFT JOIN fan-out → COUNT/AVG 왜곡(자식행 0일 땐 잠복, 입력 순간 발현).
|
|
Why: 두 결함 다 L1 43/43 GREEN 을 통과했다. mock 은 '내가 부른 메서드가 불렸나'만 보고
|
|
'DB 가 그 SQL 을 우리 기대대로 해석하나'는 검증하지 못한다. 단위 통과율이 높을수록 오히려
|
|
DB-통합 계약 공백이 숨는다.
|
|
How to apply:
|
|
- 매퍼 메서드를 신규 추가/시그니처 변경하거나 집계 뷰를 정의·수정할 때 L1 GREEN 으로 끝내지 말 것.
|
|
- dev DB 연동 L2 contract 로 (a) 매퍼 반환 Map 키가 컨트롤러 조회 키와 정합, (b) 뷰 집계가
|
|
샘플 데이터에서 손계산과 일치, 를 가드한다.
|
|
- SQL alias 는 camelCase 가 필요하면 반드시 큰따옴표("gameId"). 소문자 폴딩 허용 alias 만 무따옴표.
|
|
rationale_for_saving: "동일 패턴(매퍼 추가/뷰 집계 변경) 재발 가능. 코드·git log 로 유도 불가(런타임 DB 방언 동작은 관찰로만 드러남). 기존 verification-strategies '신규 컨트롤러/매퍼 의존 변경 시 full test 의무'는 컨텍스트 로드 회귀만 다루고 DB 방언 계약 사각은 미커버."
|
|
signal_source: observation
|
|
docs_sync_target: /Users/wemadeplay/workspace/stz/bibimbap/docs/development/verification-strategies.md
|
|
memory_optional: true
|
|
- name: verify-against-current-build-not-stale-server
|
|
type: feedback
|
|
description: "L3 검증 진입 전 가동 중 앱이 현재 빌드인지 확인 — JSP docBase 동결로 stale spring-boot:run 은 라이브 리로드 불가, 재기동 필수."
|
|
body_draft: |
|
|
What: bibimbap 직접 실행(spring-boot:run / provided Tomcat)에서 JSP 는 docBase 가 기동 시점에
|
|
동결돼 라이브 리로드되지 않는다. 과거 세션에 띄워둔 stale 프로세스가 8080 을 점유한 채 검증을
|
|
시작하면 구 JSP 가 응답하고 현재 빌드는 검증되지 않는다(거짓 PASS/FAIL 위험).
|
|
Why: 본 세션은 목요일 기동된 구 JSP 프로세스가 떠 있어, 현재 빌드를 검증하려면 명시적으로
|
|
재기동해야 했다. flyway/liquibase 부재(local-setup §4)와 같은 결의 함정 — '띄워져 있음'이
|
|
'최신임'을 보장하지 않는다.
|
|
How to apply:
|
|
- L3/브라우저 스모크 진입 전: 8080 점유 프로세스의 기동 시각·아티팩트가 현재 작업 빌드인지 확인.
|
|
- 불일치/불확실하면 현재 빌드로 재기동(JSP 변경분은 재기동 없이 반영 불가).
|
|
- DDL 변경분은 별도로 db/apply-local-ddl.sh 로 실행 DB 에 적용(local-setup §4.1).
|
|
rationale_for_saving: "검증 세션마다 재발 가능한 함정. JSP docBase 동결·재기동 필요는 코드에 안 적혀 있고 런타임 관찰로만 드러남. local-setup §4.1 은 DDL 적용만 다루고 'stale 프로세스 vs 현재 빌드' 가드는 미기재."
|
|
signal_source: observation
|
|
docs_sync_target: /Users/wemadeplay/workspace/stz/bibimbap/docs/usage/local-setup.md
|
|
memory_optional: true
|
|
- name: recommended-option-accepted-low-friction
|
|
type: feedback
|
|
description: "검증 인증 우회 같은 비차단 셋업 결정은 (Recommended) 단독안으로 제시하면 왕복 협의 없이 1회 수락되는 경향."
|
|
body_draft: |
|
|
What: L3 검증 진입을 막는 비차단 셋업(인증 우회 경로 등)에서 'dev 픽스처 자동 로그인 (Recommended)'을
|
|
단일 권장안으로 제시 → 사용자가 즉시 수락(이견·재지시 0).
|
|
Why: 검증 진척을 위한 저위험·가역 셋업 결정은 옵션 나열보다 근거 붙인 권장안 단독 제시가 마찰을 줄였다.
|
|
본질적 산출물 결정이 아니라 검증 수단 결정이라 사용자 비용을 권장안에 위임하는 게 맞았다.
|
|
How to apply:
|
|
- 가역적·저위험·검증진척용 셋업 결정은 옵션 매트릭스 대신 (Recommended) 단독안 + 한 줄 근거로 제시.
|
|
- 단, 산출물 본질·비가역 결정은 이 패턴을 적용하지 말 것(옵션 제시 유지).
|
|
rationale_for_saving: "긍정 시그널이 검증한 비자명한 판단(옵션 나열 대신 권장안 단독). 재현성 있음 — 검증 세션마다 인증/픽스처 셋업 결정이 반복됨. 단 적용 경계(가역·저위험 한정)가 핵심이라 기록 가치."
|
|
signal_source: positive
|
|
docs_sync_target: null
|
|
memory_optional: true
|
|
protocol_feedback:
|
|
- "검증 사다리(verification-strategies §버그 범주→L 레벨)에 '신규 매퍼 메서드/집계 뷰 정의·변경'은 L1 + L2(dev DB contract) 의무로 명시 권고. 현재 표는 '순수 도메인 로직=L1', '외부 API=L1+L2' 만 있고 내부 DB-방언 계약(매퍼 Map 키·뷰 집계)은 L 레벨 매핑 공백. BUG-1/2 가 이 공백으로 누출됨 (structural)."
|
|
- "ATP §2.3 시그널 세탁 경계 관점: 본 세션 user_signals 는 positive 1/negative 0 로 기록됐고 회고에서 재검토 결과 세탁 정황은 없음(사용자가 사실·데이터 오류를 잡아낸 발화 없음, 버그 2건은 orchestrator 자가 발견). 다만 '검증 전용 task 에서 서버 실버그가 나왔는데 negative 시그널 0'은 사용자가 결함을 사후 인지하지 못한 정황일 수 있으므로, 검증 세션 회고에서는 '발견 버그 수 vs negative 시그널 수'를 교차 점검하는 절차를 protocol 에 추가 권고."
|
|
- "claude-in-chrome 미보유 advisor 상황에서 orchestrator 가 L3 브라우저 스모크를 직접 수행했고 computer 도구 차단을 javascript_tool 등으로 우회했다. document_idle 미도달 차단의 대체 증빙 표준(JS 이벤트/DOM/서버계약 동치)을 verification-strategies L3 항목에 등재 권고(긍정 패턴 보존)."
|
|
applied_changes:
|
|
- candidate: mock-vs-reality-db-contract-gap
|
|
decision: accepted
|
|
docs_applied: docs/development/verification-strategies.md (L레벨 표에 'DB-방언 계약' 행 + 'mock-vs-reality' 노트 추가)
|
|
- candidate: verify-against-current-build-not-stale-server
|
|
decision: accepted
|
|
docs_applied: docs/usage/local-setup.md (§4.2 '가동 앱이 현재 빌드인지 확인' 신설)
|
|
- candidate: recommended-option-accepted-low-friction
|
|
decision: docs-declined
|
|
rationale: 'docs_sync_target=null(소프트 작업흐름 패턴). 회고 기록으로 보존, 별도 docs sink 없음. memory 는 사용자 memory 설정 시에만 보조 — 본 세션 memory 미기록(docs-first 단독 마감).'
|
|
orchestrator_note: 'memory 기록은 사용자 memory 활성 신호 부재로 미수행(memory_optional). docs-first 로 #1·#2 반영 완료. protocol_feedback 3건은 외부 atp 번들 대상이라 본 repo 미적용(권고 보존).'
|