Skip to content

수정: 문서 오류 처리를 보강하고 섹션 파서를 단순화 - #12

Merged
fakeminjun7321 merged 1 commit into
mainfrom
one-way/audit-site-20260906
Sep 16, 2026
Merged

fakeminjun7321 merged 1 commit into
mainfrom
one-way/audit-site-20260906

Conversation

@fakeminjun7321

Copy link
Copy Markdown
Member

한줄 요약

원본 문서 다운로드나 캐시 오류가 페이지 전체 실패로 이어지지 않게 하고, 문서 섹션 파서를 단순화해 코드 블록과 일반 이미지를 보존합니다.

배경

GitHub 응답 헤더를 받은 뒤 본문 연결이 끊기면 기존 오류 안내로 연결되지 않았습니다. 캐시 실패도 정상 원본 응답을 막았고 다운로드에 종료 시점이 없었습니다. 별도 섹션 파서는 코드 울타리의 종류와 길이를 구분하지 못했으며 이미지 문단은 뱃지 여부와 관계없이 제외했습니다.

무엇을 바꿨나

본문 읽기까지 5초 제한과 기존 오류 안내를 적용하고 캐시 실패를 처리합니다. Worker와 호환되는 manual redirect 처리로 리디렉션을 따르지 않습니다. 수제 섹션 파서를 이미 사용하는 marked 기반으로 56줄에서 24줄로 줄였습니다. 일반 이미지와 저장소 상대 이미지 주소, 섹션 안 참조 링크 정의를 보존합니다. 새 의존성은 없습니다.

어떻게 확인했나

  • Implemented: 5개 파일 수정과 12개 추가 테스트가 존재합니다.
  • Unit-verified: 52/52 테스트, ESLint·typegen·TypeScript·OpenNext Worker build·diff 검사 통과. 수정 전 회귀 7개 실패를 확인했습니다.
  • Simulator-verified: 원격 AI/D1 binding을 제거한 localhost workerd에서 실제 GitHub README를 읽어 문서 본문을 표시했습니다. HTTP 200과 console error 0을 확인했습니다.
  • 로컬 Chromium 데스크톱·모바일 크기에서 문서·검색·메뉴·deterministic assistant POST를 확인했습니다. 본문 단절·무응답은 fixture이며 무응답은 5.097초 뒤 안내로 전환됐습니다. 일반 이미지와 특수 fence도 fixture 검증입니다.
  • Physical-device-verified / 미검증: 실제 모바일 기기 경로는 확인하지 않았습니다.
  • 미검증: 운영 배포·colo 캐시·refresh Action, 실제 Workers AI/D1 응답·저장, 새 커밋 원격 CI. 기존 검색 이슈 #9와 UX 결정 이슈 #10은 해결한 것으로 처리하지 않습니다.
  • 운영 OMM 요청 0회. 검증용 로컬 서버는 종료했습니다.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
omm 53f56f2 Commit Preview URL

Branch Preview URL
Sep 06 2026, 12:42 PM

@fakeminjun7321
fakeminjun7321 merged commit a54b221 into main Sep 16, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant