Skip to content

chore: 앱 타입 체크를 PR CI 에 추가 - #743

Merged
iOdiO89 merged 2 commits into
devfrom
iOdiO89/app-check-types
Oct 20, 2026
Merged

iOdiO89 merged 2 commits into
devfrom
iOdiO89/app-check-types

Conversation

@iOdiO89

@iOdiO89 iOdiO89 commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

작업 요약

  • 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 만 실행합니다

영향

  • CI 의 check-types 스텝이 piki-app 도 실행합니다. 로컬에서 약 2초, CI 추가분은 10초 이내로 예상합니다
  • tsc --noEmit 은 빌드가 아니라 env 가 필요 없습니다. process.env.X 는 string | undefined 라 값 없이도 통과합니다

확인

  • 루트 pnpm check-types: piki-app · piki-web · @piki/hooks 3개 실행, 모두 통과
  • 루트 pnpm lint: 영향 없음 (기존 web 경고 1건만 그대로)

범위 밖

  • packages/core 자체 check-types 는 이번에 추가하지 않습니다. 웹·앱 tsc 가 import 경로로 간접 검사합니다

연관 이슈

closes #740

Summary by CodeRabbit

  • Chores
    • 타입 오류를 확인할 수 있는 check-types 명령을 추가했습니다.
    • 기존 린트 검사 명령은 유지됩니다.

@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
piki Ready Ready Preview Oct 20, 2026 12:16am UTC
1 Skipped Deployment
Project Deployment Actions Updated
piki-notify Skipped Skipped Oct 20, 2026 12:16am UTC

@github-actions github-actions Bot added APP Good for newcomers chore labels Oct 20, 2026
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

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 스크립트는 유지됩니다.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other · Severity of issue fixed: Medium

Suggested reviewers: ychany

Merge Risk: ⚪ Minimal · up to fee20

Merge Risk: ⚪ Minimal · up to fee20

This change adds the requested app TypeScript check without an identified merge-blocking issue.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed PR 제목은 앱에 타입 체크를 추가하고 PR CI에서 실행하도록 변경한 주요 내용을 정확하고 간결하게 설명합니다.
Linked Issues check Passed 직접 연결된 활성 이슈는 [#740]입니다. apps/app/package.json에 요구된 "check-types": "tsc --noEmit" 스크립트가 추가되었습니다. 기존 lint 스크립트도 유지됩니다. PR 설명은 루트 pnpm check-types가 앱을 포함해 통과했다고 보고합니다. packages/core의 스크립트를 추가하…
Out of Scope Changes check Passed 전체 PR 변경은 apps/app/package.json의 check-types 스크립트 추가와 기존 lint 스크립트 유지로 한정됩니다. 이 변경은 [#740]의 앱 TypeScript 검사 요구를 직접 구현합니다. 관련 없는 변경은 확인되지 않습니다.
Docstring Coverage Passed 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@vercel
vercel Bot temporarily deployed to Preview – piki-notify October 20, 2026 00:14 Inactive
@iOdiO89
iOdiO89 merged commit c039e4b into dev Oct 20, 2026
6 checks passed
@iOdiO89
iOdiO89 deleted the iOdiO89/app-check-types branch October 20, 2026 00:17

This branch was successfully deployed

1 active and 1 inactive deployments
Preview – piki — fee202f4 Deployed Oct 20, 2026 by vercel[bot]
Preview – piki-notify — fee202f4 Deployed Oct 20, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

APP Good for newcomers chore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

앱 타입 체크를 PR CI 에 추가

2 participants