Skip to content

Conversation

@jk-kim0
Copy link
Contributor

@jk-kim0 jk-kim0 commented Feb 11, 2026

Summary

  • 4단계 변환 파이프라인을 2단계로 단순화하는 구현 계획을 추가합니다.
  • 리뷰 후 구현을 진행합니다.

Description

현재 MDX 변환에 4개 명령이 필요합니다:

  1. fetch_cli.py → 데이터 수집
  2. translate_titles.py → 제목 번역 (list.txt → list.en.txt)
  3. generate_commands_for_xhtml2markdown.py → 셸 스크립트 생성
  4. bash xhtml2markdown.ko.sh → 변환 실행

pages.yaml에 이미 영어 breadcrumbs와 slugified path가 포함되어 있어, 2~4단계는 중복입니다.

계획 요약

  • convert_all.py 신규 작성: pages.yaml 기반 일괄 변환
  • --verify-translations: 번역 누락 검증 기능
  • entrypoint.sh full 워크플로우 단순화

계획 문서: docs/plan-simplify-convert-pipeline.md

Added/updated tests?

  • No, and this is why: 구현 계획 문서만 추가합니다.

🤖 Generated with Claude Code

## Description
- 4단계 파이프라인(fetch → translate → generate_commands → convert)을 2단계(fetch → convert_all)로 단순화하는 구현 계획을 작성합니다.
- `pages.yaml` 기반 일괄 변환, 번역 검증 기능 등 요구사항을 정리합니다.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@vercel
Copy link

vercel bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
querypie-docs Canceled Canceled Comment Feb 11, 2026 9:19am

Request Review

@jk-kim0 jk-kim0 self-assigned this Feb 11, 2026
@jk-kim0 jk-kim0 merged commit 6f572c1 into main Feb 11, 2026
5 of 7 checks passed
@jk-kim0 jk-kim0 deleted the feat/simplify-convert-pipeline branch February 11, 2026 09:19
jk-kim0 added a commit that referenced this pull request Feb 11, 2026
## Description
- `bin/convert_all.py`를 추가하여, `pages.yaml` 기반으로 전체 Confluence 페이지를 MDX로 일괄 변환합니다.
  - `translate_titles.py` → `generate_commands_for_xhtml2markdown.py` → `xhtml2markdown.ko.sh` 3단계를 대체합니다.
  - `--verify-translations`: 한국어 제목 번역 누락 검증만 수행합니다.
  - `--generate-list`: 디버깅용 `list.txt` / `list.en.txt`를 선택적으로 생성합니다.
- `scripts/entrypoint.sh`의 `full` 워크플로우를 `fetch_cli.py` → `convert_all.py` 2단계로 단순화합니다.
- `README.md`의 변환 절차를 2단계로 간소화합니다.

## Related tickets & links
- docs/plan-simplify-convert-pipeline.md (#680)

## Added/updated tests?
- [x] No, and this is why: 기존 converter 테스트 18개, render 테스트 21개 모두 통과 확인. `convert_all.py`는 기존 `converter/cli.py`를 호출하는 오케스트레이터이므로, 별도 단위 테스트 없이 통합 검증으로 충분합니다.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
jk-kim0 added a commit that referenced this pull request Feb 11, 2026
## Summary
- 기존 4단계 파이프라인(`fetch_cli.py` → `translate_titles.py` →
`generate_commands...py` → `xhtml2markdown.ko.sh`)을 2단계(`fetch_cli.py` →
`convert_all.py`)로 단순화합니다.

## Commits

### 1. `feat(convert_all): pages.yaml 기반 일괄 변환 스크립트 추가`
- `bin/convert_all.py` 추가: `pages.yaml` 기반으로 전체 Confluence 페이지를 MDX로 일괄
변환
- `--verify-translations`: 한국어 제목 번역 누락 검증
- `--generate-list`: 디버깅용 `list.txt` / `list.en.txt` 선택적 생성
- `scripts/entrypoint.sh`의 `full` 워크플로우를 `fetch_cli.py` →
`convert_all.py` 2단계로 단순화
- `README.md`의 변환 절차를 2단계로 간소화

### 2. `chore: 구 파이프라인 도구 삭제 및 참조 정리`
- 불필요해진 구 파이프라인 도구 삭제:
- `bin/translate_titles.py`,
`bin/generate_commands_for_xhtml2markdown.py`, `bin/generated/` 디렉토리,
`var/list.en.txt`
- `scripts/entrypoint.sh`에서 `title)`, `generate_commands)`, `convert)`
case 제거
- `compose.yml`에서 구 파이프라인 주석과 `list.en.txt` volume mount 제거
- `Dockerfile`에서 `bin/generated/*.sh` chmod 제거
- `CONTAINER_DESIGN.md`를 단순화된 2단계 파이프라인으로 업데이트

## Related
- 구현 계획: docs/plan-simplify-convert-pipeline.md (#680)

## Test plan
- [x] `convert_all.py --help` 정상 출력
- [x] `convert_all.py --verify-translations` 번역 검증 통과 (121개 번역, 누락 0건)
- [x] `convert_all.py --generate-list` list.txt/list.en.txt 생성 확인 (289건)
- [x] `convert_all.py` 전체 변환 289개 페이지 성공
- [x] 기존 converter 테스트 18개, render 테스트 21개 통과

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 4.6 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant