Skip to content

feat: 후속 어법 화면을 위한 분석 UI·서비스 선행 변경 연결 - #167

Open
Yelihi wants to merge 11 commits into
feature/152-grammar-note-persistencefrom
feature/grammar-analysis-foundation
Open

Yelihi wants to merge 11 commits into
feature/152-grammar-note-persistencefrom
feature/grammar-analysis-foundation

Conversation

@Yelihi

@Yelihi Yelihi commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

어법 등록 화면이 사용할 분석 편집 UI(#163)와 사전 체크·분석 서비스(#164)를 저장 기반 PR 위에 연결합니다. 두 PR은 feature/149-grammar-domain에 병합되었지만 main에는 반영되지 않았으므로, 후속 기능이 검토 가능한 연속 PR을 구성하도록 선행 변경을 연결합니다.

선행 PR: #166 (feature/152-grammar-note-persistence). 먼저 #166을 main에 병합한 다음 이 PR의 base를 main으로 변경해 병합합니다. 직접 병합하지 않습니다.

Closes #150
Closes #151

검증된 기존 변경을 merge commit으로 연결했으며 새 기능 코드는 추가하지 않습니다. 이후 #153부터 이 브랜치를 기준으로 별도 PR을 구성합니다. 마지막 #161에서 통합 테스트와 빌드 결과를 갱신합니다.

코드 관례 검토 반영 (2026-10-06)

분석 공급자 schema 파일명을 schema.ts로 정리하고 늦은 응답 차단 등 판단 근거를 주석으로 보완했습니다.

반영 커밋: fc24d78. 해당 PR 코드에서 TypeScript 검사와 어법 관련 Jest 109개를 통과했습니다. 선행 수정 커밋을 부모로 연결해 연속 PR의 변경 범위를 유지했습니다. main 병합은 하지 않았습니다.

@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.

Normalize schema/error/mapper naming and preserve the preceding PR's reviewed changes.
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