Repository navigation
既定の設定を Action に持たせ、各文書で .include しなくても効くようにする - #11
Conversation
- _setup.qd を Action 同梱の setup.qd にし、default-setup (既定 true) のとき
各文書の先頭に足した写しを同じフォルダに作って変換する
- 文書の中で同じ関数を呼べば上書きできる
- sample.qd から .include {_setup.qd} を外す
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USCeESvgLneCai7g8399GM
|
Claude finished @5ym's task in 27s —— View job レビュー結果
全体: 方針 (同じフォルダに一時コピーを作り、相対パスを保つ) は妥当で、README・spec・sample も揃っています。指摘は 2 点です。
セキュリティ上の問題は見当たりません。 |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
| base=$(git show "$PDF:convertdone.txt") | ||
| # _ で始まるファイル(_setup.qd など)は単体では変換しない | ||
| # _ で始まるファイル(.include で読む分割用)は単体では変換しない | ||
| list() { git diff --name-only --diff-filter="$1" "$base" "$SRC" | grep -E '\.qd$' | grep -v '\(^\|/\)_' || true; } |
There was a problem hiding this comment.
不具合 (要確認): _setup.qd を setup.qd に改名したため、_ 除外 (grep -v '\(^\|/\)_') に掛からなくなります。このリポジトリ自身のワークフロー (uses: ./) では、マージ時の push で setup.qd が変更扱いになり、setup.pdf (中身のほぼ空の PDF) が pdf ブランチに作られる見込みです。default-setup により先頭に自分自身が二重に足されます。
対応案: setup.qd を _setup.qd のまま Action 同梱にする、または list() の除外に setup.qd を足す、あるいは置き場所を .github/ などに移す。
| --out "$out" --out-name document | ||
| mv "$out/document.pdf" "${f%.*}.pdf" | ||
| rm -rf "$out" | ||
| [ "$in" = "$f" ] || rm -f "$in" |
There was a problem hiding this comment.
軽微 (要確認): quarkdown c が失敗した場合 (--strict は失敗し得ます)、ここに届く前にステップが落ちるため .seisho.<名前>.qd が作業ツリーに残ります。後続の cleanup でブランチを戻す際に邪魔にならないか確認してください。trap などで消すと安全です。また先頭に設定が 6 行足されるので、エラーメッセージの行番号が文書とずれます。
- setup.qd だと _ 始まりの除外から外れ、このリポジトリで setup.pdf が作られる - 変換が途中で落ちたときに残る .seisho.*.qd を restore で消す - 先頭に足した行のぶんエラーの行番号がずれることを仕様に書く Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01USCeESvgLneCai7g8399GM
|
Claude finished @5ym's task in 15s —— View job 再レビュー結果 (最新コミット 3c39542)前回の 2 点はどちらも解消されています。
残っている点 (軽微・対応は任意): 先頭に設定が足されるぶん、 セキュリティ: 問題は見当たりません。 テスト: このリポジトリにテストはありません。PR 本文の手元確認と、マージ後の 問題無しです。 |
1 participant
今まで各文書の先頭に書いていた
.include {_setup.qd}を要らなくする。変えたこと
_setup.qdを Action 同梱の既定の設定にした (_始まりなので、このリポジトリでは変換されない)。中身は A4・paperwhite (latex)・日本語・BIZ UDP明朝 / M PLUS 1 Codedefault-setup(既定true) を足した。有効なときは、_setup.qdを先頭に足した写しを同じフォルダに.seisho.<名前>.qdとして作って変換し、終わったら消す。同じフォルダに作るので、画像などの相対パスはそのまま使える.include {_setup.qd}が残っている文書でもそのまま動く確かめたこと
手元の Quarkdown 2.6.3 で、
--strict付きでコンパイルした:.includeを外したもの).include {_setup.qd}が残った文書に、既定の設定を足したもの (二重になる場合)マージ時の push で、このリポジトリのワークフロー (
uses: ./) が sample.qd を PDF にする。🤖 Generated with Claude Code
https://claude.ai/code/session_01USCeESvgLneCai7g8399GM