요청 계약을 authorized 하나로 정리하고 헤드리스 게이트를 걷어낸다 - #46
Merged
Conversation
- headlessFirst(직행 최적화)를 제거한다. 불완전 승격이 들어온 뒤로는 정책 없이도 같은 결과에 도달해, 이 힌트는 도메인당 fetch 1회와 LLM 1회를 아끼는 값이 됐고 넣을 유인이 사라졌다. product.extract.headless_first 메트릭도 함께 제거 - headlessAllowed 게이트를 제거한다. 브라우저로 여는 것 자체는 신원을 밝히고 하는 일이라 허락을 전제하지 않는다 - 승격 3경로가 허가 뒤에 있던 구조를 풀어, 기본 상태에서도 CSR 페이지가 뚫린다 - 대신 authorized 를 받아 렌더 경계까지 전달만 한다. 이 값이 여는 것은 렌더 서비스의 우회 수단(지문 보정·프록시)이고 판정 원장은 호출자에 있다(무상태) - 전략 계약(LinkExtractionStrategy)에도 authorized 를 실었다. plain 은 우회 개념이 없어 무시하지만, 두 전략이 같은 시그니처를 갖는 편이 호출부가 분기를 들고 있는 것보다 낫다 - stub·fake 에 전달된 값을 기록해 렌더 경계까지 도달하는지 케이스로 고정한다 - 이 전달이 끊기면 허락받은 대상이 조용히 정직 모드로 가거나 그 반대가 되는데 반환값으로는 안 드러난다
|
Important Review available on request
Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 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 |
This was referenced Aug 16, 2026
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
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.
Situation
headlessFirst(직행 최적화)와headlessAllowed(허가 게이트) 둘을 받고 있었다. 뒤의 것은 헤드리스 진입 3경로(직행·차단 승격·불완전 승격)를 한꺼번에 닫는 게이트였다.Action
headlessFirst제거. 불완전 승격이 들어온 뒤로는 정책 없이도 같은 결과에 도달해, 이 힌트는 도메인당 fetch 1회와 LLM 1회를 아끼는 값이 됐고 넣을 유인이 사라졌다.product.extract.headless_first메트릭도 함께 제거.headlessAllowed게이트 제거. 승격 3경로가 허가 뒤에 있던 구조를 풀어, 기본 상태에서도 CSR 페이지가 뚫린다.authorized를 받아 렌더 경계까지 전달만 한다. 이 값이 여는 것은 렌더 서비스의 우회 수단이고 판정 원장은 호출자에 있다(무상태).LinkExtractionStrategy)에도authorized를 실었다. plain 은 우회 개념이 없어 무시하지만, 두 전략이 같은 시그니처를 갖는 편이 호출부가 분기를 들고 있는 것보다 낫다.Result
연관 이슈