Skip to content

[fix] yarn sdk 파일 복구#17

Merged
2seb2 merged 1 commit into
mainfrom
fix/yarn-sdks
Apr 27, 2026
Merged

[fix] yarn sdk 파일 복구#17
2seb2 merged 1 commit into
mainfrom
fix/yarn-sdks

Conversation

@zeon0xx0
Copy link
Copy Markdown
Contributor

@zeon0xx0 zeon0xx0 commented Apr 26, 2026

#️⃣ 관련 이슈

연관된 이슈 번호를 적어주세요.
이슈를 함께 종료하려면 Closes #이슈번호 형식으로 작성해주세요.

  • Closes #이슈번호

⏰ 작업 시간

예상과 실제 시간이 다르다면 이유를 간단히 적어주세요.

  • 예상 작업 시간 : 2h
  • 실제 작업 시간 : 2h

💻 작업 내용

이번 작업에서 진행한 내용을 정리해주세요.

  • .yarn/sdks 폴더 복구
    • eslint, typescript SDK 파일 누락 문제 해결
  • .gitignore 수정
    • .yarn/* 규칙으로 인해 SDK까지 무시되던 문제 수정
    • .yarn/sdks는 추적되도록 예외 처리 추가
  • Yarn PnP 환경에서 VSCode TypeScript 인식 문제 해결

필요한 경우 스크린샷이나 캡처 화면을 함께 첨부해주세요.


🪏 작업하면서 고민한 부분

작업 중 겪은 문제나 고민, 그리고 그에 대한 해결 과정을 정리해주세요.
관련 트러블슈팅 문서가 있다면 링크로 연결해주세요.

문제 상황

테스트 환경 설정 과정에서 .gitignore 수정으로 인해 .yarn/sdks 폴더가 Git에서 제외되었고, 이후 merge 과정에서 SDK 파일들이 삭제됨

이로 인해:

  • VSCode에서 react, @emotion/styled 등 의존성 인식 실패
  • react/jsx-runtime 관련 타입 에러 발생

해결 과정

  • git diff로 변경 이력 확인
  • 삭제 이전 커밋 복구
  • gitignore 수정

👀 리뷰 포인트

리뷰어가 중점적으로 확인해주길 바라는 부분이 있다면 작성해주세요.

  • .gitignore 에서 .yarn/sdks 예외 처리 부분 위주로 확인해주세요
  • sdks 폴더 안에 기존에 있던 파일들이 제대로 들어갔는지 확인해주세요🥹
  • 이 부분은 상의해봐야할 것 같아요!! 아직 머지하지 않는게 좋을 것 같습니다!!!

📘 참고

작업하면서 참고한 문서, 링크, 자료가 있다면 작성해주세요.

Summary by CodeRabbit

Chores

  • Yarn Berry SDK 저장소 추적 설정이 업데이트되었습니다. SDK 아티팩트 파일이 이제 버전 제어에 포함될 수 있도록 변경되었습니다.

@zeon0xx0 zeon0xx0 added the 🐞 Bug 버그 수정 및 오류 해결 label Apr 26, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 26, 2026

📝 Walkthrough

Walkthrough

.gitignore 파일이 Yarn Berry SDK 아티팩트 규칙을 수정합니다. .yarn/sdks/ 디렉토리에 대한 일괄 무시 규칙을 제거하고, .yarn/sdks/**와 모든 중첩된 내용을 명시적으로 제외합니다.

Changes

Cohort / File(s) Summary
Yarn SDK 무시 규칙 수정
.gitignore
.yarn/sdks/ 일괄 무시 규칙 제거 및 .yarn/sdks/** 명시적 제외 추가로 SDK 파일 버전 관리 포함 변경

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

hotfix

Suggested reviewers

  • 2seb2

Poem

🐰 Yarn의 SDK들이 이제 밝은 빛을 보네,
.gitignore의 벽을 넘어 버전 관리로 돌아와,
중첩된 파일들이 함께 춤을 추며,
코드의 정원에 새로운 생명을 불어넣는다! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 .yarn/sdks 파일 복구라는 주요 변경사항을 명확하게 설명하고 있으며, 변경 내용과 직접적으로 관련되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description check ✅ Passed PR 설명이 템플릿 구조를 대부분 따르고 있으며, 작업 내용, 문제 상황, 해결 과정, 리뷰 포인트 등이 상세하게 작성되어 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/yarn-sdks

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 and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.gitignore:
- Line 37: Update the misleading comment that currently reads "SDK와 Unplugged
폴더를 무시하여 Diff 방지" to reflect the actual .gitignore behavior: either remove the
claim about ignoring the SDK folder or change it to state that the Unplugged
folder is ignored while the SDK folder is intentionally tracked (the tracked SDK
entries introduced around the SDK inclusion lines). Locate the comment line
containing "SDK와 Unplugged 폴더를 무시하여 Diff 방지" and edit its text to match the
reality of the SDK entries (or delete the contradictory phrase) so the comment
no longer contradicts the SDK tracking rules.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b4c3cb31-cddb-4f03-bce7-682672179125

📥 Commits

Reviewing files that changed from the base of the PR and between 8fdcf31 and df08e99.

⛔ Files ignored due to path filters (19)
  • .yarn/sdks/eslint/bin/eslint.js is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/api.js is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/config-api.js is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/types/config-api.d.ts is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/types/index.d.ts is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/types/rules.d.ts is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/types/universal.d.ts is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/types/use-at-your-own-risk.d.ts is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/universal.js is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/lib/unsupported-api.js is excluded by !**/.yarn/**
  • .yarn/sdks/eslint/package.json is excluded by !**/.yarn/**
  • .yarn/sdks/integrations.yml is excluded by !**/.yarn/**
  • .yarn/sdks/typescript/bin/tsc is excluded by !**/.yarn/**
  • .yarn/sdks/typescript/bin/tsserver is excluded by !**/.yarn/**
  • .yarn/sdks/typescript/lib/tsc.js is excluded by !**/.yarn/**
  • .yarn/sdks/typescript/lib/tsserver.js is excluded by !**/.yarn/**
  • .yarn/sdks/typescript/lib/tsserverlibrary.js is excluded by !**/.yarn/**
  • .yarn/sdks/typescript/lib/typescript.js is excluded by !**/.yarn/**
  • .yarn/sdks/typescript/package.json is excluded by !**/.yarn/**
📒 Files selected for processing (1)
  • .gitignore

Comment thread .gitignore
@zeon0xx0 zeon0xx0 requested review from 2seb2 and jjhparkk April 26, 2026 07:37
@zeon0xx0 zeon0xx0 closed this Apr 26, 2026
@zeon0xx0 zeon0xx0 reopened this Apr 26, 2026
Copy link
Copy Markdown
Contributor

@2seb2 2seb2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 한번 얘기 따로 나눠봐요!

@2seb2 2seb2 merged commit 661cbd4 into main Apr 27, 2026
3 checks passed
@zeon0xx0 zeon0xx0 self-assigned this Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Bug 버그 수정 및 오류 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants