chore: bump fig-extract to v2.26.1 - #40
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthrough벤더링된 그림 추출 엔진 버전을 Changes그림 추출 엔진
Estimated code review effort: 4 (Complex) | ~60 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ast-grep (0.45.0)src/core/fig-extract.jsast-grep timed out on this file 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 |
engine: onetwothr1/PDFViewer-Figure-Extract@732f2d2 엔진 repo가 v2.19.4 이후 11개 버전 앞서 있어 한 번에 올린다. 2.19.5 · 2.20.0 · 2.21.0 · 2.21.1 · 2.22.0 · 2.23.0 · 2.24.0 · 2.24.1 · 2.25.0 · 2.26.0 · 2.26.1 — 전부 [계약 무변경]/[행동 변경]이고 [BREAKING]·[필드 추가]는 하나도 없어 fig-engine.ts 타입· docs/fig-extract-integration.md의 계약 서술은 그대로다. 엔진 export 목록(파일 끝 return)도 v2.19.4 대비 무변경이다. 주요 감지 변화 (엔진 repo 규칙 유래 표가 정본): 2.19.5 12-B captionCompetition 가드를 진단 독립으로 교정 2.20.0 SI 목록 페이지 거부 (false_positive 19행 제거) 2.21.0 경계 벗기기 — 바깥 블록이 남의 텍스트면 방출 상자에서 뺀다 2.22.0 패널형 soft 캡션 앵커 2.23.0 컬럼 인식 블록 분할 (기각 전용) 2.24.x table 캡션 융합 판정을 그림 잉크 실측으로 + 본체 롤백 공유 2.25.0 본문 단락 기하 객체 + 단락 상단 클램프 2.26.x 단락 x 클램프 + 캡션 컬럼 확정 2.20.0·2.22.0은 방출 집합 자체를 바꾸고, 2.21.0 이후는 집합을 유지한 채 상자만 줄이는 계약이다. 검증: 복사 시점 엔진 repo clean · byte-identical(335,512 bytes) typecheck · test 48/48 · build 통과 · [사람] 스모크 정상 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
3d13603 to
74a2438
Compare
`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`만 있었습니다 — 그 둘은 이 결함을 통과시키고 테스트만 잡습니다) - ⑧ 커밋 메시지에 엔진 SHA 명시 엔진 repo 쪽 PB-5는 이 문서를 정본으로 삼아 8단계로 맞추는 수정을 함께 올립니다. 이 PR은 그중 SHA 표기 규칙만 정본에 반영하는 것입니다. 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **문서** * 엔진 업데이트 절차에서 커밋 SHA를 7자리 약칭 형식으로 기록하도록 안내를 수정했습니다. * 커밋 메시지에 복사한 엔진의 7자리 SHA를 포함하는 예시와 설명을 업데이트했습니다. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
엔진 repo가 v2.19.4 이후 11개 버전 앞서 있어 한 번에 올립니다.
2.19.5 · 2.20.0 · 2.21.0 · 2.21.1 · 2.22.0 · 2.23.0 · 2.24.0 · 2.24.1 · 2.25.0 · 2.26.0 · 2.26.1계약 영향 없음
11개 항목을 태그 단위로 감사한 결과 전부
[계약 무변경]/[행동 변경]이고[BREAKING]·[필드 추가]는 하나도 없습니다. 따라서fig-engine.ts·fig-extract.d.ts·docs/fig-extract-integration.md는 손대지 않았습니다. 출력 필드·좌표계·전역 노출 방식 모두 그대로입니다.변경은
src/core/fig-extract.js한 파일이고 엔진 repo와 byte-identical(335,512 bytes)입니다.주요 감지 변화
정본은 엔진 repo의 규칙 유래 표이고, 여기서는 이번 점프로 크롭이 달라질 수 있는 이유만 적습니다.
captionCompetition가드를 진단 독립으로 교정 — 소비자 쪽에서 꺼져 있던 가드가 켜집니다false_positive19행 제거)FIG. N a, …) — 앵커 0이던 문서가 살아납니다그림 개수·번호·경계가 달라지는 논문이 있습니다 — 특히 2.20.0(오탐 제거)과 2.22.0(앵커 복구)은 방출 집합 자체를 바꿉니다. 2.21.0 이후는 방출 집합을 유지한 채 상자만 줄이는 계약입니다.
검증
npm run typecheck통과npm run build성공 (viewer 588.96 kB / gzip 183.19 kB)엔진 repo 게이트(v2.24.1 → v2.26.1):
sCriticalgate 0.9301→0.9313 · bench 0.9263→0.9282 · all 0.9274→0.9291, critical 회귀 0, 앵커 악화 0.🤖 Generated with Claude Code
Summary by CodeRabbit