Skip to content

feat: add export_html CLI tool with 100% test coverage - #311

Closed
seonghobae wants to merge 2 commits into
developfrom
feat/add-export-html-tool-16224716909917454010
Closed

feat: add export_html CLI tool with 100% test coverage#311
seonghobae wants to merge 2 commits into
developfrom
feat/add-export-html-tool-16224716909917454010

Conversation

@seonghobae

Copy link
Copy Markdown
Collaborator

What (구현 내용)

  • NewsDOM JSON 구조를 파싱하여 스타일이 적용된 HTML 문서로 내보내는 tools/export_html.py 스크립트를 추가했습니다.
  • 입력 JSON 및 출력 파일 옵션 처리를 위한 인자 파싱 기능을 포함했습니다.
  • 보안 강화를 위해 생성되는 모든 HTML 콘텐츠에 html.escape를 적용하여 XSS 등 잠재적인 인젝션 리스크를 방지했습니다.
  • 새롭게 추가된 스크립트에 대한 예외 테스트 및 분기 테스트를 모두 포함한 tests/test_tools_export_html.py 파일을 작성했습니다.
  • CHANGELOG.md[Unreleased] 항목에 신규 기능 추가 내용을 한국어로 명시했습니다.

Why (해결하는 문제)

  • 파싱된 결과를 사람이 시각적으로 빠르게 검토할 수 있도록 브라우저에서 바로 열람 가능한 HTML 변환 도구가 필요했습니다.
  • 별도의 웹 애플리케이션이나 마크다운 렌더러 없이도 CLI 단에서 DOM 트리 결과를 손쉽게 확인할 수 있도록 지원합니다.

Impact (성능/영향)

  • 코어 모듈(API 서버나 모델 추론 등)과는 완전히 독립된 오프라인 도구이므로 런타임 성능에 영향을 주지 않습니다.
  • 제공된 단위 테스트가 모든 분기를 통과(코드 커버리지 100%)하므로 기존 테스트 신뢰도 지표를 낮추지 않습니다.

Measurement (측정/검증 방법)

  • CLI 환경에서 uv run ruff checkuv run ruff format 명령어를 사용하여 정적 분석과 서식을 검증했습니다.
  • uv run pytest --cov=tools tests/ 명령을 수행하여 새로 작성한 모듈(tools/export_html.py)을 포함한 전체 tools 패키지의 단위 테스트 커버리지가 100%를 충족하는 것을 직접 확인했습니다.

PR created automatically by Jules for task 16224716909917454010 started by @seonghobae

@google-labs-jules

Copy link
Copy Markdown

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@seonghobae

Copy link
Copy Markdown
Collaborator Author

현재 develop에는 NewsDOM HTML export가 PR #284 commit eaa7fce로 이미 반영되어 있고 tests/test_tools_export_html.py로 보존됩니다. CodeGraph 및 관련 98개 테스트로 current base 동작을 확인했습니다. 이 오래된 head의 Trivy DS-0002는 current develop에서 non-root container로 이미 fixed입니다. 중복 queue를 제거하기 위해 닫습니다.

@seonghobae seonghobae closed this Jul 13, 2026
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