Skip to content

feat(tools): NewsDOM JSON 처리를 위한 validate_dom 및 export_html 도구 추가 - #298

Closed
seonghobae wants to merge 1 commit into
developfrom
jules-7564294809414231963-00b1cacd
Closed

feat(tools): NewsDOM JSON 처리를 위한 validate_dom 및 export_html 도구 추가#298
seonghobae wants to merge 1 commit into
developfrom
jules-7564294809414231963-00b1cacd

Conversation

@seonghobae

Copy link
Copy Markdown
Collaborator

What (구현 내용)

  • 파싱된 NewsDOM JSON 출력 결과를 ParseResponse 스키마 기반으로 검증하는 CLI 도구 tools/validate_dom.py를 추가했습니다.
  • 파싱된 NewsDOM JSON을 사람이 읽기 편한 HTML 구조로 변환하는 CLI 도구 tools/export_html.py를 추가했습니다.
  • 새로 추가된 두 CLI 도구에 대한 포괄적인 단위 테스트(tests/test_tools_validate_dom.py, tests/test_tools_export_html.py)를 작성했습니다.

Why (해결하는 문제)

  • 기존에는 ParseResponse 구조가 제대로 생성되었는지 독립적으로 검증할 수단이 부족하여, 파싱 결과물의 유효성을 빠르게 확인하기 어려웠습니다.
  • JSON 형태의 데이터를 시각적으로 탐색하고 디버깅하기 어려워, 가시성이 높은 HTML 형식으로 내보내는 기능이 필요했습니다.

Impact (성능/영향)

  • 파싱 데이터의 무결성을 보장하고, 테스트 및 디버깅 시 개발자 경험(DX)을 크게 향상시킵니다.
  • 외부 라이브러리(HTML 이스케이핑 등)를 활용하여 XSS 등의 보안 이슈 없이 안전하게 데이터를 렌더링합니다.
  • tools/ 패키지에 대한 100% 테스트 커버리지를 유지합니다.

Measurement (측정/검증 방법)

  • uv run pytest --cov=tools --cov-report=term-missing 명령어를 통해 커버리지 100%를 달성했음을 확인했습니다.
  • uv run ruff formatuv run ruff check 명령어로 코딩 컨벤션 위반이 없음을 검증했습니다.

PR created automatically by Jules for task 7564294809414231963 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에는 validate/search 도구가 PR #280 commit f4bc8b0 및 PR #253 commit a760fa0으로, HTML export가 PR #284 commit eaa7fce로 이미 landing되어 있습니다. CodeGraph에서 validate_dom과 테스트 연결을 확인했고 export_html/validate_dom 관련 테스트를 포함한 98개 테스트가 통과했습니다. 과거 Trivy DS-0002도 current base에서 fixed이므로 중복 PR을 닫습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant