fix: 링크 파서의 경계 입력과 이미지 대체 텍스트 처리 - #111
Merged
Merged
Conversation
라벨 깊이만 보고 안쪽 링크를 글자로 내려서, 바깥 링크가 차단된 스킴이거나 텔레그램 한도를 넘어 글자로 나갈 때 안쪽 링크까지 사라졌다. 바깥이 <a>를 낼지(makes_anchor)를 먼저 판정해 in_anchor 로 넘긴다. - 라벨용 안쪽 인라인 상태가 in_cell 을 이어받는다. 표 칸 안 링크 라벨의 <br> 이 줄바꿈이 되어 행이 갈리던 것을 막는다. - 대괄호 색인 버퍼를 Inline 에 두고 재사용한다. 조각마다 새로 할당하지 않는다. - 불러오는 이미지일 때만 Plain Vocab 을 만든다. - Go 이식도 같게 고치고, 내장 close 를 가리던 지역 변수 이름을 바꾼다. - 코퍼스: blocked-outer-link-label, cell-link-label-break Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
minjun0219
marked this pull request as ready for review
October 5, 2026 23:29
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
닫히지 않은 대괄호를 반복하면 링크 탐색이 O(n²)으로 늘고, 링크 라벨 안의 링크를 깊게 중첩하면 호출 스택이 고갈됩니다. 또한 서식이 있는 이미지 대체 텍스트가 HTML 속성에 태그로 들어가 React에서 이미지 대신 태그 문자열이 보였습니다. Rust와 Go를 함께 수정하고 재현 입력을 코퍼스에 추가했습니다.
변경
재현 및 결과
기준은 main의
87fd02b입니다. 동일 환경의 debug Rust CLI에서 대괄호 1만·2만·4만 자가 약 0.19·0.74·3.00초였고, 수정 후에는 약 0.002·0.003·0.005초였습니다. 1만 단계 중첩 링크는 기존 CLI에서 stack overflow로 중단됐으며 수정 후 정상 종료했습니다.의 불러오기 출력은alt="<strong>cat</strong>"에서alt="cat"으로 바뀝니다. npm 스모크에서 WASM → 이벤트 → React 전체 경로와 스트리밍·미리보기를 확인합니다.검증
cargo test --workspace --locked통과: 코퍼스, 스트리밍 일치, 퍼즈, 산문 할당 회귀 포함cargo clippy --workspace --all-targets --locked -- -D warnings통과wasm32-unknown-unknown빌드 통과CI=1 ./scripts/smoke.sh통과: Node, Bun, TypeScript, React, lezer깊이 제한과 중첩 앵커 폴백은 SPEC에 명시했습니다. 코어 의존성 및 공개 API 변경은 없습니다.