82 lines
8.3 KiB
Markdown
82 lines
8.3 KiB
Markdown
---
|
|
phase: implementation
|
|
agent: implementation-advisor
|
|
agent_version: 1
|
|
generated_at: 2026-06-24T10:55:00+09:00
|
|
workstream: W2-1-게임잼 엔티티/라이프사이클
|
|
concerns: []
|
|
concerns_checked: true
|
|
workers_spawned: 21
|
|
planned_workers: 21
|
|
actual_workers: 21
|
|
self_verification:
|
|
checklist_passed: true
|
|
unused_diagnostics: 0 # 정적 unused-import 스캔 0건 + 각 worker 자가확인. 통합 LSP/린터 실측은 verification 영역.
|
|
references:
|
|
design: .atp/work-session/20260623-104307/implementation/W2-1-jam-entity-design.md
|
|
ownership: .atp/work-session/20260624-100749/implementation/ownership.md
|
|
---
|
|
|
|
# 구현 보고 — W2-1 게임잼 엔티티/라이프사이클
|
|
|
|
설계 정본을 그대로 구현. 임의 변경 0(아래 "설계와의 차이"의 2건은 설계 계약을 깨지 않는 구현 디테일 명시).
|
|
|
|
## 변경 목록 (설계 파일영향맵 = 19파일, 신규 16 + 수정 3)
|
|
|
|
| 파일 | worker | 결과 요약 |
|
|
|---|---|---|
|
|
| docs/jam-ddl.sql | migration-writer w-001 | 신규. CREATE TABLE 5건, SEQ 5, ADD CONSTRAINT 14, INDEX 8. 멱등(IF NOT EXISTS/DO $$ guard) |
|
|
| db/schema.sql | migration-writer w-001 | 수정. 371행~ 잼 블록 append. docs/jam-ddl.sql 와 SEQ/CONSTRAINT/INDEX 갯수 동일(동기 사본). 기존 1~368행 무변경 |
|
|
| data/JamData.java | code-writer w-002 | 신규. jams 18필드 POJO(getIsVisible/getIsDelete) |
|
|
| data/JamEntryData.java | code-writer w-003 | 신규. jam_entries 8필드 + JOIN 표시 3필드(gameName/thumbnailUrl/entrantName) |
|
|
| data/JamTeamData.java | code-writer w-004 | 신규. jam_teams 6필드 + memberCount |
|
|
| jam/JamStatus.java | code-writer w-005 | 신규. enum 4멤버 + isValid/from(unknown→null) |
|
|
| jam/JamLifecycle.java | code-writer w-006 | 신규. @Component isAllowed(전이그래프) + isPeriodReady(좁은 3인자 + JamData 오버로드) |
|
|
| jam/JamSlugs.java | code-writer w-006 | 신규. generate(정규화/한글보존/72절단) + withSuffix(충돌재시도) |
|
|
| mapper/JamsMapper.java | code-writer w-007 | 신규. 9메서드. listVisibleKeyset 은 <script>+row-comparison keyset. ${} 0 |
|
|
| mapper/JamEntriesMapper.java | code-writer w-008 | 신규. insert/listByJam(JOIN games+COALESCE entrantName)/exists. ${} 0 |
|
|
| mapper/JamTeamsMapper.java | code-writer w-009 | 신규. insert/getById/listByJam(memberCount 서브쿼리). ${} 0 |
|
|
| mapper/JamTeamMembersMapper.java | code-writer w-010 | 신규. insert/exists. ${} 0 |
|
|
| mapper/JamStatusLogMapper.java | code-writer w-011 | 신규. insert(5파라미터 @Param). ${} 0 |
|
|
| controller/JamAdminController.java | code-writer w-012 | 신규. 6핸들러 전수 requireJamManage(=6) + 5쓰기 CSRF. slug 충돌 재시도 |
|
|
| config/InterceptorConfig.java | code-writer w-013 | 수정. .excludePathPatterns("/admin/jams/**") 추가(D4-A) |
|
|
| webapp/.../admin-jam-list.jsp | code-writer w-014 | 신규. 생성폼 + status/visibility/delete 액션. meta _csrf + htmlEscape |
|
|
| controller/JamController.java | code-writer w-015 | 신규. list(keyset)/detail/submitEntry/submitTeam/addMember. 쓰기 3 CSRF. resolveEntrant 인라인 |
|
|
| webapp/.../jam-list.jsp | code-writer w-016 | 신규. keyset 더보기(URLEncoder) + htmlEscape |
|
|
| webapp/.../jam-detail.jsp | code-writer w-017 | 신규. 출품/팀생성 폼(_csrf hidden) + htmlEscape, innerHTML 미사용 |
|
|
| test/.../BibimbapApplicationTests.java | code-writer w-018 | 수정. 신규 5매퍼 @MockBean 등록(기존 12빈 보존) |
|
|
| test/.../JamLifecycleTest.java | code-writer w-019 | 신규. 전이그래프 허용/거부/동일/null + 기간정합 8메서드 |
|
|
| test/.../JamAdminControllerTest.java | code-writer w-020 | 신규. 게이트 401/403/redirect + CSRF + 전이감사 10메서드(순수 Mockito) |
|
|
| test/.../JamControllerTest.java | code-writer w-021 | 신규. keyset/출품 소유·멤버·중복·상태 + CSRF 14메서드 |
|
|
|
|
## Bash 단계 (advisor 직접)
|
|
- 세션 디렉토리 생성: mkdir .atp/work-session/20260624-100749/implementation → OK
|
|
- 집합 전수 AC 정적 점검(테스트 실행 아님): AC-T1 테이블5/5 · AC-T2 상태4값(enum4=jams_check4=log_check4) · AC-T3 핸들러6=requireJamManage6 · AC-T4 매퍼5 ${} 0 · AC-T5 XOR+type CHECK 존재 · AC-T6 @MockBean Jam 5 → 전부 PASS
|
|
- DDL 동기 비교: docs/jam-ddl.sql ↔ schema.sql 잼블록 SEQ 5/5, ADD CONSTRAINT 14/14, INDEX 8/8 일치
|
|
- unused-import 정적 스캔 16 Java 파일 → 0건
|
|
- git status: 설계 파일영향맵과 정확히 일치(수정3+신규16), 범위 밖 변경 0
|
|
|
|
## 설계와의 차이 (계약 미파괴 — 구현 디테일 명시)
|
|
1. **상태전이 응답 키 `status`→`jamStatus`**: 설계 API 계약 표는 상태전이 200 응답을 `{status:toStatus}`로 적었으나, 동일 응답 Map 에 공통 `status`=HTTP 200(value) 와 잼 상태를 같은 키로 넣으면 후자가 HTTP value 를 덮어쓴다. w-012 가 데이터 손실 방지 위해 잼 상태를 `jamStatus` 키로 분리. **권장 정합**: verification 의 JamAdminControllerTest 가 `jamStatus`=전이상태 + `status`=200 을 검증하도록 작성됨(일관). 클라(admin-jam-list.jsp)는 reload 방식이라 응답 키 의존 없음. 설계 의도(상태 반환)는 충족, 키 이름만 충돌 회피.
|
|
2. **JamLifecycle.assertPeriodReady → isPeriodReady 좁힘(concern 1 반영)**: 설계 시그니처는 `assertPeriodReady(JamData jam, JamStatus to)`였으나 concern 1(dead parameter 방지)에 따라 ① 좁은 `isPeriodReady(JamStatus to, OffsetDateTime evalStartAt, OffsetDateTime evalEndAt)`(EVAL→start, CLOSED→end 두 인자 실사용) + ② 컨트롤러 편의용 `isPeriodReady(JamStatus to, JamData jam)` 오버로드(jam 의 두 getter 위임 실사용)로 구현. 예외 throw 대신 boolean(컨트롤러 ResponseEntity 패턴 정합). dead parameter 0.
|
|
|
|
> 위 2건 외 DDL/엔티티/매퍼 SQL/API 경로/시퀀스/keyset/게이트 enforcement 는 설계 정본 그대로.
|
|
|
|
## concerns 처리 결과 (설계 6개 concern)
|
|
- **concern 1 (시그니처 inflate)**: 해소. JamLifecycle.isPeriodReady 좁힘(위 #2). JamController.resolveEntrant 는 별도 헬퍼 추출 안 하고 submitEntry 인라인 유지(grep resolveEntrant private 0 — dead parameter 회피).
|
|
- **concern 2 (신규 컨트롤러/매퍼 의존 + @MockBean)**: 해소. BibimbapApplicationTests 에 신규 5매퍼 @MockBean 등록(grep Jam @MockBean = 5). contextLoads 보존. **단, full ./mvnw -o test 실측은 verification 영역(미실행)**.
|
|
- **concern 3 (DB-방언 L2: snake→camel alias / keyset)**: 매퍼 5개 모두 직접 alias(큰따옴표 0), listVisibleKeyset 은 PostgreSQL row-comparison `(created_at,id) < (...)` + <script> NULL 분기. **dev DB contract 실측은 verification 영역**.
|
|
- **concern 4 (games↔jam 활성 자연키)**: jam_entries 의 ux_jam_entries_jam_game_active(활성 UNIQUE) DDL 그대로. games 무변경 확인(git status 에 games 관련 변경 0).
|
|
- **concern 5 (자동전이 스케줄러)**: 본 구현은 수동 전이 enforcement 만. @Scheduled 빈 미도입 → BibimbapApplicationTests context 영향 0(신규 빈은 매퍼 5 + JamLifecycle/PermissionGate 기존). 범위 준수.
|
|
- **concern 6 (slug 충돌 재시도)**: 해소. JamSlugs.generate + withSuffix, JamAdminController.create 가 DuplicateKeyException catch 시 withSuffix(base,n) 재시도(최대 ~20).
|
|
|
|
## Verification 을 위한 힌트
|
|
- acceptance criteria: design.md §검증 포인트(VP-1~7) + §집합 전수 AC(AC-T1~T6) 참조.
|
|
- 영향받는 테스트 파일: JamLifecycleTest(VP-2), JamAdminControllerTest(VP-1/2/5), JamControllerTest(VP-3/4/5), BibimbapApplicationTests(VP-7 contextLoads).
|
|
- **반드시 full `./mvnw -o test`** (verification-strategies §30): 신규 컨트롤러/매퍼 의존 → test-compile 만으로 부족. contextLoads + 신규 테스트 컴파일·실행 판정.
|
|
- **dev DB contract(L2)** 권장 실측: listVisibleKeyset row-comparison, jam_entries XOR/type CHECK 위반 INSERT 거부, snake→camel alias 키 일치(VP-6).
|
|
- 미실측 리스크: ① keyset row-comparison SQL 의 PostgreSQL 실행 정합(매퍼 코드는 명세대로) ② JSP 의 게임상세 링크 경로 `/game/{gameId}`(게임 허브 실제 라우트와 대조 — gameId null 시 폴백 처리됨, 깨짐 없음) ③ datetime-local 입력의 OffsetDateTime 파싱(컨트롤러 parseOffset try/catch→422 처리).
|
|
|
|
## 미해결/리스크
|
|
- 없음(설계 범위 내 전부 구현). 위 "Verification 힌트"의 미실측 3건은 verification-advisor 가 full test + dev DB contract 로 판정.
|