You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
No actionable comments were generated in the recent review. 🎉
ℹ️ Recent review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Advanced
Run ID: 41061005-2b62-405b-a69e-28d8a9259f09
📥 Commits
Reviewing files that changed from the base of the PR and between 5a47459 and 5c59f79.
📒 Files selected for processing: 1
apps/app/package.json
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
📝 Walkthrough
앱 패키지에 tsc --noEmit을 실행하는 check-types 스크립트를 추가합니다. 기존 lint 스크립트는 유지합니다.
Changes: 앱 타입 검사
Layer / File(s)
Summary
타입 검사 명령 추가 apps/app/package.json
check-types 스크립트가 TypeScript를 출력 없이 검사하도록 추가되었습니다. 기존 lint 스크립트는 유지됩니다.
This change adds the requested app TypeScript check without an identified merge-blocking issue.
Pre-merge checks: 5
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
PR 제목은 앱에 타입 체크를 추가하고 PR CI에서 실행하도록 변경한 주요 내용을 정확하고 간결하게 설명합니다.
Linked Issues check
직접 연결된 활성 이슈는 [#740]입니다. apps/app/package.json에 요구된 "check-types": "tsc --noEmit" 스크립트가 추가되었습니다. 기존 lint 스크립트도 유지됩니다. PR 설명은 루트 pnpm check-types가 앱을 포함해 통과했다고 보고합니다. packages/core의 스크립트를 추가하…
Out of Scope Changes check
전체 PR 변경은 apps/app/package.json의 check-types 스크립트 추가와 기존 lint 스크립트 유지로 한정됩니다. 이 변경은 [#740]의 앱 TypeScript 검사 요구를 직접 구현합니다. 관련 없는 변경은 확인되지 않습니다.
Docstring Coverage
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Linked Issues check
Explanation
직접 연결된 활성 이슈는 [#740]입니다. apps/app/package.json에 요구된 "check-types": "tsc --noEmit" 스크립트가 추가되었습니다. 기존 lint 스크립트도 유지됩니다. PR 설명은 루트 pnpm check-types가 앱을 포함해 통과했다고 보고합니다. packages/core의 스크립트를 추가하지 않은 내용도 이슈의 범위와 일치합니다.
Full details: Docstring Coverage
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches
🧪 Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · 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.
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
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.
작업 요약
apps/app에check-types스크립트를 추가해 PR CI 의pnpm check-types가 앱 타입도 검사하게 합니다작업 세부 내용
배경
PR CI 는
pnpm check-types로 turbo 가 전 워크스페이스의check-types를 돌리지만, 앱에는 스크립트가 없어 건너뛰고 있었습니다. 그래서@piki/core타입 변경으로 앱이 깨져도 CI 가 초록이었습니다.앱의
lint(expo lint)는 ESLint 라 타입 오류를 잡지 못합니다.const n: number = 'oops'로 확인하면 lint 는 통과하고tsc만TS2322를 냅니다.변경
apps/app/package.json"check-types": "tsc --noEmit"추가.lint는 그대로 둡니다next typegen이 없어tsc만 실행합니다영향
check-types스텝이piki-app도 실행합니다. 로컬에서 약 2초, CI 추가분은 10초 이내로 예상합니다tsc --noEmit은 빌드가 아니라 env 가 필요 없습니다.process.env.X는string | undefined라 값 없이도 통과합니다확인
pnpm check-types:piki-app·piki-web·@piki/hooks3개 실행, 모두 통과pnpm lint: 영향 없음 (기존 web 경고 1건만 그대로)범위 밖
packages/core자체check-types는 이번에 추가하지 않습니다. 웹·앱tsc가 import 경로로 간접 검사합니다연관 이슈
closes #740
Summary by CodeRabbit
check-types명령을 추가했습니다.