Skip to content

feat: add Purple case on explicit consent for self retry billing terms - #7

Merged
labyrinth30 merged 3 commits into
mainfrom
feat/purple-terms-consent
Oct 5, 2026
Merged

labyrinth30 merged 3 commits into
mainfrom
feat/purple-terms-consent

Conversation

@labyrinth30

Copy link
Copy Markdown
Owner

변경

  • Purple CASE 04 추가: 재결제 약관 동의를 서버가 대신하지 않게 하기 (PA-1868)
    • 문제 → 리스크 보고 및 정식 과제화 → 명시 동의·청구 직전 차단·우회 방지 → Toss 상점별 동의·약관 조회 API 특성 → 검증(Jest 22건, DEV 실검증) → 범위(관리자·배치 자동 동의는 범위 밖)
    • 다이어그램 consent.mmd 추가
  • 사례 수 규칙 일반화: 프로젝트당 3개 이상, 1부터 빈 번호 없이 (src/lib/cases.ts, 스키마 order 상한 제거)
  • 다이어그램 수·PDF 사례 수를 고정값(6) 대신 실제 파일 수 기준으로 검사
  • Purple 담당 범위에 셀프 재결제 약관 명시 동의 추가

검증

  • build 에러 0, 두 PDF 생성, e2e 31/31 통과 (프로젝트별 사례 수 gguk 3 / purple 4)
  • 목차·사례 헤더·다이어그램 스크린샷 확인

🤖 Generated with Claude Code

Allow projects to hold three or more cases numbered without gaps, and
derive diagram and PDF case counts from the content instead of fixed
numbers. Add the consent case and its diagram as Purple case 04.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 18 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6ea70b74-6c6b-4921-a4f5-95f14e8d81aa
📥 Commits

Reviewing files that changed from the base of the PR and between 7f90685 and b5d5dd3.

📒 Files selected for processing (9)
  • README.md
  • scripts/build-diagrams.mjs
  • scripts/export-pdf.mjs
  • src/content.config.ts
  • src/content/cases/purple-consent.mdx
  • src/content/projects/purple.json
  • src/diagrams/consent.mmd
  • src/lib/cases.ts
  • tests/portfolio.spec.mjs
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

labyrinth30 and others added 2 commits October 5, 2026 15:46
Replace the DEV-only scope note with the production confirmation and drop
mentions of other billing paths.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@labyrinth30
labyrinth30 merged commit 634a676 into main Oct 5, 2026
3 checks passed
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.

1 participant