feat(workflow): ワークフロー自動化 13 件 — skill 4 + pre-commit hook 6 + pre-push gate

[ayutaz]

Summary

開発ワークフローで recurring に発生していた drift / cycle-of-shame パターンを fail-fast 化するため、 Claude Code skill 5 件 + pre-commit hook 6 件 + opt-in pre-push stage 1 件 + skill auto-chain 1 件 + cmake drift 解消 1 件 を追加 (9 コミット、16 ファイル、+2214 / -9 行)。各 hook は conservative 設計 (既存コードは ALLOWLIST or 検出ロジック緩和で通過、新規発生のみ block)。auto-invocation 可能な skill (/watch-pr, /reply-review, /create-pr, /check-review-backlog --pr <N>) は LLM が文脈判断で自動発動、 PR 作成時は /create-pr → /check-review-backlog --pr → /watch-pr の 3 段 auto-chain が連鎖。

新規 / 拡張機能

PR レビュー支援 (skill)

skill	動作	解決する問題	auto
/watch-pr <PR>	gh pr checks を polling + 失敗を format drift / test fail / build error / flake / contract drift に分類	CI 待ちの手動 refresh、 失敗原因の都度判定。 PR #419 で起きた silent skip パターンも検出	✓
/reply-review <PR> --stale-check	comment の originalCommit.oid を git log で HEAD と比較し、 該当ファイルが解決済みかを判定	Copilot が古い commit を指摘し続けて、 解決済みなのに review thread が closed されない問題 (memory feedback_copilot_stale_review)	✓
/reply-review <PR> --skip-copilot-style	^Consider (using|renaming) ^Optional: \bnit\b 等の Copilot 定型ノイズ regex を skip。 human reviewer には適用しない	人間の判断が要る指摘と Copilot の自動 nitpick の混在による集中度低下	✓
/check-review-backlog [--days N] [--author X]	全 open PR の isResolved=false thread を GraphQL で集計、 N 日 (default 7) 経過したものを表化。 read-only	レビュー対応漏れの可視化。 30 日超 unresolved の PR の active/abandoned 判定	—

PR 作成支援 (skill)

skill	動作	解決する問題	auto
/create-pr [base-branch]	push 済みブランチに対し、 機能カテゴリ別の表 + 設計判断 + Test plan を含む構造化 PR 本文を生成。 マイルストーン非付与、 auto-merge 非使用、 時系列フェーズ表記禁止を強制。 既存 PR は gh pr edit --body-file で本文書き換え	PR body の構造が PR ごとにバラバラ、 reviewer が読みづらい / 後から振り返れない問題。 マイルストーン誤付与・auto-merge 誤起動の予防	✓

PR 作成後の auto-chain (3 skill 連鎖)

/create-pr 完了直後に明示確認なしで 3 段連鎖。 PR #496 で「PR 作成後に review チェックを発動しなかった結果、 5 件の未対応 Copilot review に気づくのが遅れた」事例を再発防止する仕組み。

Step	skill	動作
6.1	/check-review-backlog --pr <N>	単一 PR モードで全未解決 review を即時表示 (PR 更新時の bot review 早期検出)
6.2	/watch-pr <N>	5 秒待機後 CI 状態 polling、 red なら失敗 job のログ分類
6.3	(集約報告)	PR URL + review 件数 + CI status を 1 メッセージにまとめ、 次のアクション提案

skip 条件: ユーザ明示拒否 / feature → feature の PR / dry-run。

cmake drift 解消

commit	変更	理由
3fe4112b	cmake/ExternalDeps.cmake の pyopenjtalk-plus URL を 0.4.1.post7 → 0.4.1.post8 に bump (SHA256 も 555fdf86... → f4dfbfbe...)	src/python_run/requirements.txt: pyopenjtalk-plus>=0.4.1.post8 との range constraint drift を、 強化された check_openjtalk_version_sync.py が検出したため解消。 Python / C++ で同じ OpenJTalk dict / behavior を共有する設計意図に従い PyPI 最新 (post8) に揃える

(auto ✓ = disable-model-invocation: false、 LLM が「PR を作って」「review に返信」「PR の CI を見て」等の文脈で自動発動。 auto — = 明示呼び出しのみ、 周期実行向け or 誤起動リスク回避)

リリース・ドキュメント整合性 (skill)

skill	動作	解決する問題
/sync-docs Agent 7 追加	既存 6 並列 agent に「Version Drift 監査」を追加。 docs/spec/release-versions.toml canonical で CLAUDE.md ランタイム表 / docs/spec の version comment / CHANGELOG [Unreleased] 残留 / README Feature Matrix を一括照合	リリース時の 9 manifest 間の version 同期漏れ。 release-versions.toml mode="warn" の段階で drift 検出を網羅化
/release-prep [runtime] [--target-version X.Y.Z]	9 manifest の現 version 一覧 → release-versions.toml の expected_prefix 照合 → CHANGELOG [Unreleased] → [X.Y.Z] 移行 markdown 生成 → PyPI / crates.io / NuGet / npm / Maven Central の最新公開 version 取得 → ローカル/公開/canonical 3-way 差分表を出力	リリース時の forensic check の手動性。 「local では bump 済みだが publish されてない」状態の即発見

Commit 時の drift gate (pre-commit hook)

.pre-commit-config.yaml に追加。 git commit 時点で staged files に対して fire。

hook	files filter	検出内容
language-parity	docker/webui/app.py, docker/python-inference/{inference,test_openai_api}.py	WebUI SAMPLE_TEXTS keys / CLI --language choices / FastAPI test 期待値の 3-way 不一致 (AST 解析)
changelog-unreleased-cleanup	CHANGELOG.md, docs/spec/release-versions.toml	[Unreleased] 内の release-style 見出し (### [vX.Y.Z]) 残留 検出 (conservative: 構造見出しのみ、 prose は flag しない)
secret-path-reference	*.{py,sh,yml,toml,rs,go,cs,...} 全 executable 拡張子	/data/piper/.env 等ホスト固有 secret env パスが executable code に漏洩していないか (CLAUDE.md / docs/ は exempt、 ALLOWLIST で legacy 許容)
openjtalk-version-sync	cmake/ExternalDeps.cmake, **/pyproject.toml	cmake URL の pyopenjtalk_plus-X.Y.Z pin と pyproject.toml の == / ~= 制約の同期検証 (bare name は warn-only)
pytest-skip-reason	src/**/tests/**/*.py	@pytest.mark.skip / pytest.skip() / skipif で reason= または positional arg が空でないか (AST、 f-string / 式 / kwarg すべて accept)
gofmt	*.go (vendor / *.pb.go 除外)	Go format drift。 scripts/run_gofmt_check.sh で gofmt -l の出力を exit code 化

Push 前の最終確認 (opt-in pre-push stage)

.pre-commit-config.yaml の stages: [pre-push]。 pre-commit install --hook-type pre-push で初めて有効化、 default では install されない。 git push 直前に full-repo で 4 contract gate (loanword / PUA / language parity / CHANGELOG unreleased) を実行 (~20-30 秒)。 commit 時に SKIP=... で hook を bypass しても push 前に drift を検出。

設計判断 / トレードオフ

• conservative changes: 既存 Rust source に rustfmt drift が pre-existing のため rustfmt hook は別 PR に deferral。 既存 prepare_*.py の docstring 内の /data/piper/ 参照は ALLOWLIST 化 (legacy 許容)。 既存の 20 件 pytest skip は f-string / 式 reason 検出で通過 (新規 reason-less skip のみ block)
• 誤検出回避: secret-path-reference は executable 拡張子のみ、 doc は exempt。 changelog-unreleased-cleanup は構造見出しのみ flag、 prose は除外。 openjtalk-version-sync は明示的 == / ~= 不一致のみ error、 bare name は warn-only
• bypass 経路: 全 hook は SKIP=hook-id git commit で個別 bypass 可。 pre-push stage は pre-commit install --hook-type pre-push を実行しなければそもそも fire しない
• CI mirror: 既存 .github/workflows/pre-commit.yml が pre-commit run --all-files を PR 時に実行するため、 install 忘れでも PR で drift を検出
• read-only skill: /check-review-backlog は GraphQL の query のみ実行、 POST / mutation 一切なし。 /release-prep の CHANGELOG 移行も markdown diff 提示のみ、 Edit 適用は明示確認後

Test plan

実装時に通過確認済み (各 commit の pre-commit hook で gate)

• pre-commit run --all-files で全 hook 通過 (個別 hook は実装時に通過確認済み)
• git commit 時の新規 hook fire を 9 commit 全件で確認 (0daeefaf f9e33992 40dc9fe4 7d720eaa b7cd1d74 27cc52f7 b7e71c8a 3fe4112b c204b55b)
• OpenJTalk pin sync gate: cmake bump 後 uv run python scripts/check_openjtalk_version_sync.py で OK: no explicit-pin mismatch against cmake 0.4.1.post8 を確認 (range violation WARN は drift 解消で消滅)
• secret-path-reference / language-parity / changelog-unreleased-cleanup / pytest-skip-reason / gofmt 各 hook 単体実行で Passed
• PR feat(workflow): ワークフロー自動化 13 件 — skill 4 + pre-commit hook 6 + pre-push gate #496 自身の /check-review-backlog --pr 496 単発モード: 5 thread が全件 resolved を確認
• PR feat(workflow): ワークフロー自動化 13 件 — skill 4 + pre-commit hook 6 + pre-push gate #496 自身の /reply-review 496: Copilot 5 件指摘に返信 + thread resolve 完了

CI 側で要確認 (本 PR で初めて gate 化される項目)

• cmake post8 bump 後の C++ build (Linux / macOS / Windows) が SHA256 mismatch / 構造変更で失敗しないこと — cpp-tests / C++ Release & Debug × macos-latest / windows-latest / ubuntu-24.04 全 6 組 + cpp-coverage + test-cpp-unit-tests + test-cpp-inference + iOS / macOS arm64 build が全件 pass
• .github/workflows/pre-commit.yml で 6 新規 hook 全件が PR 時に通過 — pre-commit run --all-files job が pass
• pre-push opt-in stage: pre-commit install --hook-type pre-push 後の git push で 4 contract gate が 20-30 秒で完走 (ローカル検証のみ可、 副作用あり)

skill chain の動作確認

• /create-pr フェーズ 6 auto chain: 別 PR で「PR を作って」要求から /create-pr → /check-review-backlog --pr <N> → /watch-pr <N> の 3 段連鎖が明示確認なしで発動 (本 PR 完了後に別 PR で要検証)
• /release-prep core 動作: 9 manifest (VERSION=1.12.0 / Rust=0.4.0 / C# Core/Cli=0.3.0 / npm synthesis=0.6.0 / npm g2p=0.4.0 / Swift=1.13.0 / Swift G2P=1.14.0 / Kotlin=1.0.0) の version 抽出成功 + PyPI registry curl で piper-plus 1.12.0 取得確認 (local と in sync)
• /check-review-backlog backlog 監視モード (引数なし): 全 4 open PR (feat(workflow): ワークフロー自動化 13 件 — skill 4 + pre-commit hook 6 + pre-push gate #496/feat: Hardware EP 自動選択 + PIPER_EXECUTION_PROVIDER env var 対応 (全5ランタイム) #386/feat: 感情条件付き TTS (Style Vector + PE-A Emotion Loss) — 実装完了 / 実 GPU 学習は未実施 #355/feat: Zero-Shot TTS — 参照音声から未知話者の音声を合成 #222) を scan、 +4 GraphQL req で rate limit 4981→4977 (影響 0.08%、 5000/h limit に対し余裕大)、 全 PR で unresolved=0 を確認
• /loop /check-review-backlog で週次自動監視 (spec 正当性は確認済み、 定期実行系は本 PR で計測不可)

Copilot レビュー対応検証 (commit b7e71c8a 関連)

• run_gofmt_check.sh: set -uo pipefail + rc=$? の明示捕捉で gofmt exit 2 (syntax error) が silent abort しない
• check_openjtalk_version_sync.py: packaging.SpecifierSet で == ~= >= <= > < != 全制約形式判定、 requirements*.txt も scan 対象に追加
• .pre-commit-config.yaml files filter: (^|/)(CMakeLists\.txt|Dockerfile|Makefile)$ で sub-directory の同名ファイルも検査対象 / .*requirements.*\.txt を openjtalk-version-sync filter に追加

参考

• canonical truth: docs/spec/release-versions.toml (expected_prefix per manifest)
• 関連 memory: feedback_conservative_changes.md, feedback_merge_caution.md, feedback_copilot_stale_review.md, feedback_ci_cancelled_baseline.md, feedback_pr_body_over_comments.md
• 関連 PR / issue: PR fix(ci): codespell baseline を整備 (format + 非英語 README skip + typo) #419 (silent skip patterns), PR test/ci: 全領域監査の Gap を解消 (parity gates / CLI smoke / Dependabot / CodeQL / attestation) #401 (cycle-of-shame ruff drift)

[Copilot]

Pull request overview

This PR adds local automation to catch recurring workflow drift earlier through pre-commit/pre-push gates and Claude Code skills, fitting into the repo’s existing contract-check and multi-runtime consistency tooling.

Changes:

• Adds new Python/Bash pre-commit helper scripts for language parity, changelog cleanup, secret-path detection, OpenJTalk version sync, pytest skip reasons, and gofmt checks.
• Extends .pre-commit-config.yaml with new commit-time hooks and an opt-in pre-push contract gate.
• Adds/updates Claude skill documentation for PR watching, review replies, release prep, review backlog checks, and docs sync version-drift auditing.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
scripts/run_gofmt_check.sh	Adds gofmt wrapper for pre-commit failures.
scripts/check_test_skip_reason.py	Adds AST-based pytest skip reason checker.
scripts/check_secret_path_reference.py	Adds host-specific secret path reference detector.
scripts/check_openjtalk_version_sync.py	Adds CMake vs Python OpenJTalk dependency sync checker.
scripts/check_language_parity.py	Adds WebUI/CLI/FastAPI language list parity checker.
scripts/check_changelog_unreleased.py	Adds CHANGELOG [Unreleased] release-heading drift checker.
CLAUDE.md	Documents opt-in pre-push contract gates.
.pre-commit-config.yaml	Registers new hooks and opt-in pre-push gate.
.gitignore	Unignores new check scripts.
.claude/skills/watch-pr/SKILL.md	Adds CI polling/triage skill.
.claude/skills/sync-docs/SKILL.md	Adds version-drift audit agent.
.claude/skills/reply-review/SKILL.md	Adds stale-check and Copilot-style filtering guidance.
.claude/skills/release-prep/SKILL.md	Adds release preparation helper skill.
.claude/skills/check-review-backlog/SKILL.md	Adds unresolved review-thread backlog skill.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.