Skip to content

test: workflow paths-mirror の drift 検証テストを追加#358

Merged
ymnao merged 1 commit into
mainfrom
test/workflow-paths-mirror
Jul 18, 2026
Merged

test: workflow paths-mirror の drift 検証テストを追加#358
ymnao merged 1 commit into
mainfrom
test/workflow-paths-mirror

Conversation

@ymnao

@ymnao ymnao commented Jul 18, 2026

Copy link
Copy Markdown
Owner

概要

PR #357 で codex-review qa-fixture が REPORT-ONLY で挙げた「ci.yml + ci-skip.yml / codeql.yml + codeql-skip.yml の paths リスト drift を検証するテスト不在」を、pair 全体の pattern 昇格として解消する。yaml から paths 定義を読み取り、pair 間補完と 4 workflow 全体の path list 統一を Vitest で assert する軽量テストを追加。

関連 Issue

なし (HANDOFF §2 追跡タスク)

移行 Stage

  • Stage 0: 雛形 + フロント表示確認
  • Stage 1: ファイル I/O
  • Stage 2: ワークスペース・ファイルツリー・ファイル監視
  • Stage 3: 全文検索(ripgrep sidecar)
  • Stage 4: Git Sync
  • Stage 5: OGP / PDF / アップデート
  • Stage 6: 仕上げ・配布・切り替え

Stage 対象外 (CI インフラ・テスト追加)。

変更内容

  • test/workflows/paths-mirror.test.ts (新規): 4 workflow を js-yaml でパースして以下を assert
    • ci.yml: push.paths-ignore === pull_request.paths-ignore (順序独立の Set 比較)
    • ci-skip.yml: push.paths === pull_request.paths
    • ci.yml paths-ignore === ci-skip.yml paths (補完 pair の drift 検出)
    • codeql.yml + codeql-skip.yml も同様
    • 4 workflow 全体で path list が同一 (pattern 統一の骨子)
  • vitest.config.ts: 新規 workflows project (node env, test/**/*.test.ts) を追加
  • tsconfig.node.json: include に test を追加 (typecheck 対象化)

動作確認

  • pnpm vitest run --project workflows — 6/6 pass
  • pnpm vitest run (全 project) — 2584 pass / 2 skipped、既存テスト regression なし
  • pnpm tsc -b tsconfig.node.json — エラーなし
  • pnpm biome check test/ vitest.config.ts tsconfig.node.json — エラーなし

スクリーンショット

なし (テスト追加のみ、UI 影響なし)

検証エビデンス

リスク分類

tier: medium — classify-risk.sh (reasons 明示なし、default medium)

実行した検証

種別 コマンド 結果
ユニットテスト (workflows 単体) ./node_modules/.bin/vitest run --project workflows PASS 6/6
ユニットテスト (全 project) ./node_modules/.bin/vitest run PASS 2584 / SKIP 2、regression なし
Typecheck (node) ./node_modules/.bin/tsc -b tsconfig.node.json エラーなし
Lint / Format ./node_modules/.bin/biome check test/ vitest.config.ts tsconfig.node.json エラーなし (No fixes applied)
レビュー codex-review security PASS 0 findings

Lint / typecheck の web / e2e 側と e2e 実行はローカル sandbox 制約により未実施 (memory: project_verify_sandbox_eperm)、CI (ci.yml pull_request trigger) で確認する。

レビュー指摘と対応

codex-review security 観点で 0 finding (PASS)。shell-senior / qa-fixture は tier=medium のため未実施。

追跡先

なし

Draft 判定

  • 判定: normal
  • 根拠: step 4 — tier=medium レビュー PASS、fix 必要 finding も追跡別 PR に回す finding も 0 件

- ci.yml + ci-skip.yml と codeql.yml + codeql-skip.yml の
  paths-ignore / paths を js-yaml でパースし、pair 間の補完関係と
  4 workflow 全体で path list が同一であることを assert
- vitest.config.ts に workflows project (node env, test/**/*.test.ts)
  を追加
- tsconfig.node.json の include に test を追加

PR #357 で codex-review qa-fixture が REPORT-ONLY で指摘した
「paths リスト drift 検証テスト不在」を pair 全体の pattern 昇格として
解消 (HANDOFF §2)。
@ymnao
ymnao merged commit 55e8dca into main Jul 18, 2026
9 checks passed
@ymnao
ymnao deleted the test/workflow-paths-mirror branch July 18, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant