docs: 온보딩 구현 기준 하네스를 정리한다 - #12
Open
pbk95120 wants to merge 3 commits into
Open
Conversation
pbk95120
requested review from
d6nggyun,
easyhooon,
hoooonshub and
jongse7
and removed request for
easyhooon
August 25, 2026 05:28
내 정보 수정 규칙을 마이페이지가 온보딩·목표 정책으로 위임하는데, PUT /api/onboarding/profile로 완료 후 순자산·목표 기간·월 저축액을 바꾸면 확정된 목표를 재계산(목표금액=순자산+월 저축 목표×목표 기간)하는 동작이 POLICY-ONBOARDING-001/002에 누락돼 있어 규칙·불변조건을 보강한다.
d6nggyun
force-pushed
the
codex/harness-onboarding
branch
from
August 25, 2026 15:55
d7ea773 to
88c35a3
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 작업 내용 요약
main구현을 기준으로 프로필 입력과 목표 준비·확정 정책 및 플로우를 역문서화했습니다.150%상한의 정수 내림 규칙을 명시했습니다.✅ 체크리스트
main브랜치의 최신 코드를pull받았나요?templates/를 복사해 작성했나요?domain/glossary.md에 등록했나요?🤖 AI 리뷰
독립 AI 리뷰
150%상한 반올림 기준을 고정 FE/BE SHA로 재검증했습니다.Codex 교차 리뷰
P2: 급여 없는 프로필의 목표 상한 fallback 미정 — ADDRESSEDP2: 0.5만원이 생기는150%상한의 반올림 기준 미정 — ADDRESSED💬 기타 코멘트
Harness 변경 병합 후 FE와 BE 저장소에서 서브모듈 포인터 갱신 PR이 필요합니다.