Skip to content

feat: 어법 노트와 분석 구간·계층 계약 정의 - #162

Merged
Yelihi merged 2 commits into
mainfrom
feature/149-grammar-domain
Oct 5, 2026
Merged

Yelihi merged 2 commits into
mainfrom
feature/149-grammar-domain

Conversation

@Yelihi

@Yelihi Yelihi commented Oct 5, 2026

Copy link
Copy Markdown
Owner

사용자 입력(영어 문장·핵심 어법)과 AI 제목·태그를 분리하고, 문장 분석의 구간/계층을 편집할 때 지켜야 할 계약을 추가합니다. 공백·구두점을 포함한 원문 복원, UTF-16 경계, 고유 ID, 부모 포함관계/순환, 원문·revision 일치를 검증합니다.

Closes #149

이슈별 구현 계획(#149–#161)과 승인된 화면 설계도 포함합니다. 이 PR에는 화면·외부 AI·DB 연결이 없으므로 기존 제품 동작은 바뀌지 않습니다. 컴포넌트/Storybook은 후속 #150에서 시작합니다.

검증: 도메인 Jest 24개, npm run typecheck, 변경 범위 ESLint, git diff --check 통과. 시작 전 기존 Jest 120 suites / 406 tests 및 타입 검사 통과. 유료 AI·원격 DB 호출 없음.

검토/병합 순서: 이 PR을 먼저 검토해 주세요. 후속 PR은 이 브랜치 기반으로 변경 범위를 분리합니다. 병합은 사용자가 직접 수행합니다.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Comment thread src/entities/grammar-note/models/entity.ts
Comment thread src/entities/grammar-note/models/validation.ts
@Yelihi
Yelihi merged commit f5bef12 into main Oct 5, 2026
2 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