Skip to content

[Docs] 인터페이스정의서 5-2 매핑표 DASH-W01 구분 확인 - #245

Merged
yslee4601 merged 2 commits into
developfrom
docs/84-dashboard-mpa-mapping
Aug 19, 2026
Merged

[Docs] 인터페이스정의서 5-2 매핑표 DASH-W01 구분 확인#245
yslee4601 merged 2 commits into
developfrom
docs/84-dashboard-mpa-mapping

Conversation

@yslee4601

Copy link
Copy Markdown
Collaborator

📌 Pull Request

📖 1. 변경 사항 요약

  • 인터페이스정의서 5-2 매핑표가 DASH-W01을 Ajax(IF-API-03)로 적었으나 구현은 진입·기준월 변경 모두 서버 렌더링 + 전체 리로드.
  • 구현을 문서에 맞춤.

🔗 2. 관련 이슈

🛠 3. 구현 내용

  • docs/05_인터페이스정의서_v2_0.md
    • 4-2 IF-API-03 구분 Ajax → MPA
    • 5-2 DASH-W01 → 없음 / — (진입·기준월 변경 모두 페이지 전체 리로드) (AUTH-W01 선례)
    • 5-1 총칙 → "입력·목록·대시보드는 MPA" + 왜 MPA인지·IF-API-03은 2차용으로 남는다는 문단
    • 4-1 → 구분 열이 "1차 화면의 사용 방식"일 뿐 API 생사와 무관함을 한 줄 명시 (MPA 표기를 "React에서 못 쓴다"로 오독하는 것 차단)
  • DashboardViewController.java — javadoc 에 결정 근거 2줄 (PR [Feat] 공통 셸(Thymeleaf 레이아웃) 및 DASH-W01 업무 대시보드 화면 구현 #59 리뷰에서 질문이 나온 바로 그 자리)
  • 근거대장.md(로컬 vault) — "DASH-W01 MPA/Ajax 판정 (2026-08-19)" 절 추가, SRC-028(구현 기술표준). 규제가 아니라 렌더링 방식 판단이라 사용 규칙 6에 해당.

🗄️ 4. DB / Flyway 변경

  • DB 변경 없음
  • 새로운 Flyway 마이그레이션 파일 추가
  • 시드 데이터 변경
  • 기존 데이터에 영향을 줄 수 있음

추가된 마이그레이션 파일

  • 없음

✅ 5. 체크리스트

  • develop 최신 내용을 반영했습니다.
  • 로컬 빌드에 성공했습니다.
  • 애플리케이션 실행을 확인했습니다.
  • 관련 기능을 직접 테스트했습니다.
  • 테스트 코드가 필요한 경우 작성했습니다.
  • 기존 기능에 영향이 없는지 확인했습니다.
  • 커밋 메시지 컨벤션을 준수했습니다.
  • 민감정보를 커밋하지 않았습니다.
  • 기존 Flyway 마이그레이션 파일을 수정하지 않았습니다.
  • 불필요한 주석과 디버깅 코드를 제거했습니다.

🖼️ 6. 화면 변경

  • 없음

💬 7. 참고 사항

  • 4-2 IF-API-04~08(BASE-W01)이 같은 종류의 반대 방향 충돌입니다. 4-2는 MPA인데 5-2는 Ajax 엔드포인트로 적고, 실제 js/features/base/base-api.js는 /api/v1/base/*를 fetch 합니다. [Docs] 인터페이스정의서 5-2 매핑표 DASH-W01 구분 확인 #84 범위 밖이라 손대지 않고 근거대장 "남은 것"에만 기록했습니다. 별도 이슈로 올릴까요?
  • vault 사본 30_산출물/05_...md 는 repo 판보다 약 44행 뒤처져 있었습니다(3장 오류코드·§7-6 등). 근거대장의 "diff 0" 기재가 낡아 있어 사실대로 고쳤고, DASH-W01 판정분만 양쪽에 같이 반영했습니다. 전체 재동기화는 미처리.

인터페이스정의서 5-2 매핑표가 DASH-W01을 Ajax(IF-API-03)로 적었으나
구현은 진입·기준월 변경 모두 서버 렌더링 + 전체 리로드다. 구현을 문서에 맞춘다.

- 4-2 IF-API-03 구분: Ajax → MPA (API 자체는 2차 React용으로 유지)
- 5-2 DASH-W01: Ajax 엔드포인트 없음, 갱신 트리거를 전체 리로드로 명시
- 5-1 총칙에 대시보드 추가 — 총칙 누락이 이 충돌의 원인이었다
- 4-1에 구분 열이 "1차 화면의 사용 방식"임을 명시
- DashboardViewController javadoc에 결정 근거 링크

기준월 변경은 대시보드 전용이 아니라 공통 셸 동작(app-shell.js
navigateToGlobalMonth)이라 Ajax로 바꾸면 화면 하나 때문에 21개 화면이
공유하는 메커니즘을 이원화해야 한다.

근거: SRC-028 (구현 기술표준) · 근거대장 "DASH-W01 MPA/Ajax 판정"

Closes #84
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: d9b1be13-1dd6-4c44-87a7-dd662086daa1


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@claude

claude Bot commented Aug 19, 2026

Copy link
Copy Markdown

FGC 도메인 문서 기준 검토

범위

이 PR은 코드 로직 변경이 없다. docs/05_인터페이스정의서_v2_0.md의 DASH-W01/IF-API-03 표기를 실제 구현(MPA)에 맞춰 고치고, DashboardViewController.java에 근거 javadoc 2줄을 추가한 것이 전부다. 계산 로직·반올림·초년도 경계·DB 스키마 변경이 없으므로 해당 체크리스트 항목은 대상 외.

확인한 내용

  1. 구현-문서 일치 여부DashboardViewController.java:31-36dashboardService.summarize(...)를 서버에서 직접 호출해 dashboard/index.html을 렌더링하며, dashboard.js(src/main/resources/static/js/features/dashboard/dashboard.js)에는 fetch/Ajax 호출이 전혀 없다(진행바 폭 계산만 함). 기준월 변경도 ShellAdvice.java:64-71에서 ?month= 쿼리로 세션에 저장 후 MPA 전체 리로드로 처리된다. → 이번에 고친 문서 표기(“없음 / — (진입·기준월 변경 모두 페이지 전체 리로드)”, docs/05_인터페이스정의서_v2_0.md:378)가 실제 동작과 일치한다.

  2. IF-API-03 생사 여부DashboardController.javaGET /api/v1/dashboard/summary가 그대로 남아 있고, 응답 필드(DashboardSummaryResponse.java: capViolation, capWarning, arbitrageCandidate, reconMismatch, journalImbalance, openException, recentExceptions[], recentRuns[])가 docs/05_인터페이스정의서_v2_0.md:288의 IF-API-03 응답 필드와 정확히 일치한다. “MPA로 적혀 있어도 API 자체는 살아 있다”(docs/05_인터페이스정의서_v2_0.md:280)는 새 안내문이 사실과 맞는다.

  3. 요구사항·화면정의서와의 충돌 여부FGC-FUN-057(요구사항명세서 52행)과 FGC-UI-DASH-W01(화면정의서:376-422)은 Ajax/MPA 방식을 강제하지 않고 “집계 조회”만 요구한다. 렌더링 방식은 규제가 아닌 구현 판단이라는 PR 설명과 문서상 근거(SRC-028)가 상충하지 않는다.

  4. 문서 내부 정합성grep -rn "DASH-W01" docs/로 확인한 결과, 인터페이스정의서 4-2/5-1/5-2 세 곳 모두 이번 수정으로 “MPA / 없음”으로 통일되었고, 남아있는 Ajax 표기가 없다.

결론

지적 사항 없음. 문서-구현 정합성 교정 목적에 부합하며 검증 가능한 근거(코드·기존 문서)와 일치한다.

PR 설명에 언급된 IF-API-04~08(BASE-W01) 쪽 4-2/5-2 불일치는 이번 PR 범위(#84) 밖이라는 판단에 동의하며, 별도 이슈로 분리하는 편이 맞아 보인다.

@claude

claude Bot commented Aug 19, 2026

Copy link
Copy Markdown

FGC 도메인 문서 기준 검토 결과

변경 범위: docs/05_인터페이스정의서_v2_0.md(4-1, 4-2, 5-1, 5-2 문구/표 수정) + DashboardViewController.java javadoc 2줄 추가. 로직 변경 없음(순수 문서·주석 정정), DB 변경 없음.

확인한 내용

  1. 문서-구현 일치성 (FGC-FUN-057 / DASH-W01)

    • DashboardViewController.java:31-35DashboardService를 직접 호출하며 HTTP로 /api/v1/dashboard/summary를 다시 부르지 않음을 확인했습니다. → 수정된 docs/05_인터페이스정의서_v2_0.md:375 ("DASH-W01 | ... | 없음 | — (진입·기준월 변경 모두 페이지 전체 리로드)")와 일치합니다.
    • src/main/resources/static/js/features/dashboard/dashboard.js에도 fetch/Ajax 호출이 없고 진행률 바 렌더링만 수행 → "Ajax 없음" 서술과 일치합니다.
    • IF-API-03 REST 엔드포인트(DashboardController.java:49)는 PR에서 제거되지 않고 그대로 남아 있어, 새로 추가된 5-1 문단의 "IF-API-03은 4-2에 그대로 남아 2차 React가 씁니다" 서술(docs/05_인터페이스정의서_v2_0.md:370)과도 모순 없습니다.
  2. 4-1 "구분 열은 사용 방식만 표기" 신설 문구와 4-2 표의 MPA 값, 5-3 SIR-006 인수조건표(docs/05_인터페이스정의서_v2_0.md:399-406) 사이에 충돌 없음을 확인했습니다.

  3. FGC_화면정의서_v2_0.md의 DASH-W01 항목(L376-421)은 API 호출 방식(MPA/Ajax)을 규정하지 않고 GET /api/v1/dashboard/summary 사용만 명시하므로, 이번 문서 정정과 배치되지 않습니다.

  4. 금액 반올림·1,200% 한도·초년도 경계·DB 스키마 등 REG-01~24 및 ERD 관련 규칙은 이번 PR 변경 범위(문서 서술 정정 + javadoc)에 해당 사항이 없습니다.

결론: 검토 항목 17 기준으로 지적사항 없음. PR 설명에 언급된 IF-API-0408(BASE-W01) 쪽 반대 방향 불일치는 이번 diff에 포함되지 않은 기존 문제이므로 이번 PR 승인 여부와 무관하며, 이미 별도 이슈 검토를 저자가 제안한 상태입니다.

@yslee4601
yslee4601 merged commit eb691fa into develop Aug 19, 2026
5 checks passed
@yslee4601
yslee4601 deleted the docs/84-dashboard-mpa-mapping branch August 19, 2026 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] 인터페이스정의서 5-2 매핑표 DASH-W01 구분 확인

2 participants