Repository navigation
feat: Pro 과목 관리와 풀이 기록 표시 개편 - #30
Merged
Merged
Conversation
- 기존 프리셋 기록을 로컬 과목으로 이전하고 과목 생성·수정·삭제·지정을 지원 - 무료 최신 10개, Pro 최대 100개와 과목별 보기 적용 - 기록 보존 및 접근 제한 테스트와 도메인 문서 갱신
AI 사전 코드 리뷰 (파일럿)검토 SHA: HistoryBrowser에 `shouldClose` 헬퍼 메서드를 추가하여 AppState의 히스토리 접근 제어(닫기 처리) 로직을 순수 함수로 분리하고, 이에 대한 단위 테스트를 작성한 refactoring PR입니다. 기존 동작과 동일하며 추가적인 문제점은 없습니다.
Finding 상세
사람의 판정: |
Contributor
Author
|
/ai-review false-positive AX-09ce21693308 SubjectPreset.licenseExam은 Paxo/Models.swift 47행에 이미 정의되어 있으며 동일 SHA의 macOS 빌드와 테스트가 통과했습니다. |
Contributor
Author
|
/ai-review false-positive AX-a03bc3e2d3be 테스트가 참조하는 SubjectPreset.licenseExam은 Paxo/Models.swift 47행에 정의되어 있고 동일 SHA의 전체 테스트 82개가 통과했습니다. |
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.
관련 이슈
Closes #29
변경사항
subjects.json의 편집 가능한 과목으로 이전하고, 일반 기록과 새 풀이는 미분류로 둡니다. 기존history.json형식은 유지합니다.serverTier == .pro에만 열고, 권한이 없어지면 과목 편집과 범위 밖의 기록 상세를 닫습니다. 과목 이름은 AI 요청에 넣지 않습니다.변경·영향 범위
담당 주 영역: 앱 기능 (Storage / UI)
AGENTS.md,docs/)PR 분류
서버 사용량·StoreKit·프록시 계약·설정 파일은 변경하지 않았습니다. 앱은 기존 서버 tier를 기록 화면의 접근 기준으로 사용합니다.
사전 예측
실제 결과·트러블슈팅
subjects.json으로 과목과 기록 ID의 관계를 분리하고, 무료 10개/Pro 100개 보기 및 과목별 묶음을 구현했습니다.검증 결과
xcrun swift-format lint --recursive --strict Paxo PaxoTests통과; macOS Debugxcodebuild ... CODE_SIGNING_ALLOWED=NO build통과; macOSxcodebuild ... CODE_SIGNING_ALLOWED=NO test83개 통과. 기존 기록 이전·중복 이전 방지, 과목 생성·이름 변경·재지정·삭제·재시작 후 복원, 무료/Pro 검색 범위, 구독 하향 시 열린 기록의 접근 범위, 과목별 묶음과 기록·이미지 보존을 자동 테스트했습니다. 실제 Pro 계정에서의 UI 수동 조작은 이 작업 환경에 해당 계정이 없어 실행하지 못했습니다.Paxo/Secrets.swiftgitignore 상태를 확인했습니다.