Skip to content

chore(security): 의존성 취약점과 서명키 노출을 줄인다 - #259

Merged
pbk95120 merged 1 commit into
developfrom
chore(security)/dependencies-and-signing-ignore
Aug 30, 2026
Merged

chore(security): 의존성 취약점과 서명키 노출을 줄인다#259
pbk95120 merged 1 commit into
developfrom
chore(security)/dependencies-and-signing-ignore

Conversation

@pbk95120

@pbk95120 pbk95120 commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

📝 작업 내용 요약

  • Next.js를 16.3.3으로 올려 critical 취약점 1건을 제거했습니다.
  • EAS CLI를 22.6.0으로 올려 high 취약점을 줄였습니다.
  • Android 서명키와 로컬 credentials 파일을 git ignore 처리했습니다.
  • Next 16.3의 정식 retry 에러 경계 API로 마이그레이션했습니다.

✅ 체크리스트

  • develop 브랜치의 최신 코드를 pull 받았나요?
  • 빌드가 통과했나요?
  • vitest 테스트가 통과했나요?
  • Playwright e2e가 통과했나요?

🤖 AI 리뷰

Claude 리뷰 (Codex 구현)

  • 판정: 승인
  • 최초 리뷰에서 에러 경계의 reset 사용을 회귀로 지적했습니다. Next 16.3의 정식 retry API로 수정하고 전체 게이트를 다시 통과했습니다.
  • 서명키 ignore와 의존성 보안 업데이트에 blocking 이슈가 없음을 확인했습니다.

💬 기타 코멘트

  • pnpm audit: critical 1 → 0, high 47 → 26. 남은 high는 Expo/EAS/Sentry 하위 의존성이 중심이라 강제 override하지 않았습니다.
  • peer 경고 2건(TypeScript 6/Expo, React 19.2.3/Expo DOM)은 기존 정합 경고입니다.

변경 사항

  • next^16.3.3, eas-cli^22.6.0으로 업데이트했습니다.
  • Next.js 16.3의 retry API를 사용하도록 오류 경계를 변경했습니다.
    • apps/web/app/error.tsx
    • apps/web/app/global-error.tsx
  • apps/admin/next-env.d.tsapps/web/next-env.d.tsroot-params.d.ts 타입 참조를 추가했습니다.
  • .gitignore*.jks, *.keystore, credentials.json을 추가했습니다.
  • eas-cli@22.6.0minimumReleaseAgeExclude 예외 목록에 추가했습니다.

@pbk95120 pbk95120 self-assigned this Aug 27, 2026
@pbk95120
pbk95120 requested a review from jongse7 August 27, 2026 06:23
@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
28th-web-team-3-fe-web-dev Ready Ready Preview Aug 27, 2026 6:28am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
28th-web-team-3-fe-admin Ignored Ignored Aug 27, 2026 6:28am

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: e91a17fe-3ebf-476b-bf8c-0c0e87e9aa8a

📥 Commits

Reviewing files that changed from the base of the PR and between 6ba701f and 814a82f.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml, !pnpm-lock.yaml
📒 Files selected for processing (6)
  • .gitignore
  • apps/admin/next-env.d.ts
  • apps/web/app/error.tsx
  • apps/web/app/global-error.tsx
  • apps/web/next-env.d.ts
  • pnpm-workspace.yaml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

Next.js를 16.3.3으로 업데이트하고 오류 경계의 재시도 API를 retry로 변경했습니다. 생성된 root-params.d.ts 타입 import를 추가했습니다. EAS CLI 버전과 인증서·자격 증명 파일의 무시 규칙도 갱신했습니다.

Changes

Next 16.3 마이그레이션

Layer / File(s) Summary
Next 16.3 오류 재시도 및 타입 계약
apps/web/app/error.tsx, apps/web/app/global-error.tsx, apps/admin/next-env.d.ts, apps/web/next-env.d.ts, pnpm-workspace.yaml
next 버전을 ^16.3.3으로 올렸습니다. 오류 경계와 재시도 버튼이 retry 콜백을 사용합니다. 두 앱의 Next.js 생성 타입 import를 추가했습니다.

저장소 및 EAS CLI 업데이트

Layer / File(s) Summary
저장소 보호 및 EAS CLI 설정
.gitignore, pnpm-workspace.yaml
*.jks, *.keystore, credentials.json을 버전 관리에서 제외합니다. eas-cli^22.6.0으로 올리고 eas-cli@22.6.0을 출시 제한 예외에 추가했습니다.

Estimated code review effort: 2 (Simple) | ~10분

Merge Risk: ⚪ Minimal · up to 814a8

This change updates dependencies, adjusts error-boundary retry behavior, and ignores local signing credentials; no actionable merge-blocking risk remains based on the current evidence.

Suggested reviewers: jongse7

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 4 files. (2 skipped: 2 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 의존성 취약점 감소와 서명키·credentials 파일 노출 방지라는 주요 변경 사항을 정확히 요약합니다. retry API 마이그레이션은 포함하지 않지만 제목이 모든 변경 사항을 설명할 필요는 없습니다.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 4 files. (2 skipped: 2 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore(security)/dependencies-and-signing-ignore

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.

@pbk95120
pbk95120 merged commit 6dbf981 into develop Aug 30, 2026
6 checks passed
@pbk95120
pbk95120 deleted the chore(security)/dependencies-and-signing-ignore branch August 30, 2026 07:36
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