confluence-mdx: Confluence 동기화 기반을 복구합니다 - #1053
Merged
Merged
Conversation
## Summary QM 변환의 cross-profile preflight가 빈 QCP catalog를 YAML null로 해석하지 않도록 유효한 빈 목록으로 명시합니다. - `pages.qcp.yaml`을 빈 YAML list로 초기화합니다. - QM Confluence 동기화 workflow가 sibling catalog를 안전하게 검사하도록 합니다. ## Test plan - [x] `confluence-mdx/venv/bin/python -m pytest tests/test_convert_all_folders.py -q` - [x] `git diff --check` 🤖 Generated with Codex Co-Authored-By: Atlas <atlas@jk.agent>
## Summary Container가 atomic replace로 생성한 conversion manifest를 후속 Git 단계가 읽을 수 있도록 파일 권한을 명시합니다. - 임시 manifest를 교체하기 전에 권한을 `0644`로 설정합니다. - 생성된 manifest 권한을 회귀 테스트로 검증합니다. ## Test plan - [x] `confluence-mdx/venv/bin/python -m pytest tests/test_convert_all_folders.py -q` - [x] `git diff --check` 🤖 Generated with Codex Co-Authored-By: Atlas <atlas@jk.agent>
## Summary main의 Space catalog가 변경되면 Docker image hierarchy cache도 전체 갱신하도록 fetch mode를 선택합니다. - main push에서 `pages.*.yaml` 변경을 감지하면 `--remote --attachments`를 사용합니다. - 일반 PR build와 catalog 변경이 없는 push는 `--recent --attachments`를 유지합니다. - workflow_dispatch에서는 사용자가 선택한 fetch arguments를 그대로 사용합니다. ## Test plan - [x] catalog 변경 commit이 remote mode로 선택되는지 확인합니다. - [x] converter-only commit이 recent mode로 선택되는지 확인합니다. - [x] `git diff --check` 🤖 Generated with Codex Co-Authored-By: Atlas <atlas@jk.agent>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5105c1ed01
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
## Summary catalog 변경이 있는 main push에서 해당 Space의 hierarchy를 정확히 갱신합니다. - 변경된 catalog 파일명에서 sync code를 도출하고 복수 Space를 순차 fetch합니다. - fetch하지 않는 Space의 tracked catalog를 Docker builder에 보존합니다. - 이전 commit을 조회할 수 없으면 모든 catalog를 full fetch 대상으로 사용합니다. ## Test plan - [x] QCP-only, catalog 미변경, missing-before, 복수 catalog fetch plan을 검증합니다. - [x] `docker buildx build --check confluence-mdx`를 통과합니다. - [x] `confluence-mdx/venv/bin/python -m pytest confluence-mdx/tests/test_convert_all_folders.py -q`를 통과합니다. 🤖 Generated with Codex
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
1 participant
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.
Summary
Confluence Space recent sync를 실행해 최신 문서 상태를 확인하고, 자동 동기화를 막던 catalog와 manifest 문제를 해결합니다.
pages.qcp.yaml을 유효한 YAML list로 초기화해 cross-profile preflight 오류를 제거합니다.0644로 저장해 container 다음의 Git 단계가 읽을 수 있도록 합니다.pages.*.yaml이 변경되면 Docker image build가--remote --attachments를 선택해 hierarchy cache를 전체 갱신합니다.--recent --attachments를 유지합니다.최근 수정 후보 3개는 모두 main과 동일했으며, 최신 QM catalog 316개 중 root를 제외한 315개 문서를 변환한 결과 실제 ko/en/ja 문서 변경은 없었습니다.
Test plan
confluence-mdx/venv/bin/python bin/fetch_cli.py --recent --attachmentsconfluence-mdx/venv/bin/python bin/convert_all.py --sync-code qm— 315개 문서 변환 성공confluence-mdx/venv/bin/python bin/convert_all.py --verify-translationsconfluence-mdx/venv/bin/python -m pytest tests/test_convert_all_folders.py -q— 16 passedgit diff --checkAdditional notes
null로 해석되어 실패했습니다.0600manifest를git add가 읽지 못해 실패했습니다.2>&1은 PR confluence-mdx: Space 계층과 canonical 경로를 동기화합니다 #1052 리뷰로 추가된 수동 보정이며 Confluence 원문에는 아직 반영되지 않았습니다. 재변환으로 발생한 redirection 회귀는 이번 PR에서 제외했습니다.Related tickets & links
🤖 Generated with Codex
Co-Authored-By: Atlas atlas@jk.agent