docs: 벤더링 절차 — 엔진 SHA를 7자리로 - #41
Conversation
전체 40자를 적을 필요가 없다. 추적에 필요한 것은 "어느 커밋을 복사했나"이고 이 저장소 규모에서 7자리면 충분히 유일하다 (충돌하면 git이 더 긴 약칭을 요구하므로 그때 늘리면 된다). 2단계의 rev-parse HEAD도 --short로 바꾼다. 엔진 repo 쪽 축약본(PLAYBOOKS.md §PB-5)도 같은 값으로 맞춘다 — 이번 v2.26.1 벤더링에서 그 축약본에 VENDORED_ENGINE_VERSION 갱신 · npm test · SHA 명시 세 단계가 빠져 있어 CI가 버전 핀 불일치를 잡았다. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 38 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough엔진 업데이트 절차에서 기록하는 커밋 SHA를 전체 SHA에서 7자리 약칭 SHA로 변경했다. 커밋 메시지 안내에는 7자리 SHA를 기본값으로 사용하고, 충돌 시 더 긴 약칭을 사용하도록 명시했다. Changes엔진 SHA 문서 갱신
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/fig-extract-integration.md`:
- Line 230: Update the git rev-parse command in the SHA-recording instruction to
use the explicit --short=7 option, preserving the documented seven-character SHA
requirement while allowing Git to extend the abbreviation only when collision
resolution requires it.
🪄 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 Plus
Run ID: 6862ad27-3d9c-4993-b614-3b446e0b5392
📒 Files selected for processing (1)
docs/fig-extract-integration.md
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
docs/fig-extract-integration.md§갱신 절차의 SHA 표기를 40자 전체 → 7자리로 바꿉니다. 문서 한 파일, 4줄 변경입니다.추적에 필요한 것은 "어느 커밋을 복사했나"이고 이 저장소 규모에서 7자리면 충분히 유일합니다. 충돌하면 git이 더 긴 약칭을 요구하므로 그때 늘리면 됩니다. 2단계의
rev-parse HEAD도rev-parse --short HEAD로 맞췄습니다.배경
직전 v2.26.1 벤더링(#40)에서 CI가
test/fig-engine.test.ts의 버전 핀 불일치를 잡았습니다 —VENDORED_ENGINE_VERSION을 안 올린 채 엔진 파일만 복사한 상태였습니다.원인은 엔진 repo의 축약본(
docs/PLAYBOOKS.md§PB-5)에 이 문서 §갱신 절차의 세 단계가 빠져 있던 것입니다.VENDORED_ENGINE_VERSION을 같은 커밋에서 갱신 (계약 태그와 무관하게 매번)npm test(PB-5에는typecheck && build만 있었습니다 — 그 둘은 이 결함을 통과시키고 테스트만 잡습니다)엔진 repo 쪽 PB-5는 이 문서를 정본으로 삼아 8단계로 맞추는 수정을 함께 올립니다. 이 PR은 그중 SHA 표기 규칙만 정본에 반영하는 것입니다.
🤖 Generated with Claude Code
Summary by CodeRabbit