実装: deploy-pr の pnpm self-installer failure を決定論的に解消する

Machine-Readable Contract

contract_schema_version: v1
issue_kind: implementation
parent_issue: none
goal_ref: "follow-up:squne121/loop-protocol:pr-1463:deploy-pr-pnpm-action-setup"
change_kind: workflow
dedupe_key: "follow-up:squne121/loop-protocol:pr-1463:deploy-pr-pnpm-action-setup"

Parent Issue

なし（PR #1463 の CI infrastructure follow-up）

Parent Goal Ref

Goal: PR fix: legacy contract snapshot を更新可能にする #1463 の deploy-pr で反復した pnpm bootstrap failure を、PATH 上で実行される pnpm のバージョンまで検証して解消する。
Desired Destination: deploy-pages の deploy-main / deploy-pr が同一の pnpm package-manager contract を消費し、current-head の deploy-pr 証跡でその実行結果を確認できる。
Source review: fix: legacy contract snapshot を更新可能にする #1463 (review)
Anchor decision: 実装: deploy-pr の pnpm self-installer failure を決定論的に解消する #1476 (comment)

Current Validated Scope

package.json には packageManager がなく、deploy-main / deploy-pr は pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093 に可変の version: 11 を渡している。
同 SHA は upstream tag v6.0.8 であり、packageManager pin を bootstrap に反映する upstream fix 改善: SERENA_DANGEROUS_TOOLS のツール名と Serena upstream 公式ツール名の乖離を定期検証・更新する仕組みを整備する #256（Fixes process: GitHub native issue dependencies を blocker SSOT とし Depends On 本文欄を fallback/mirror に降格するポリシーを docs に記述する #233）を含む。今回の修正はこの SHA を変えず、repository 側に単一の exact pnpm contract を追加して action に消費させる。
repository 内の既存 CI は pnpm@11.7.0 を明示しているため、今回の単一の正規値は pnpm@11.7.0 とする。

Remaining Parent Gaps

PR fix: legacy contract snapshot を更新可能にする #1463 の実装内容、contract snapshot producer、trusted publisher identity は本 Issue では扱わない。
snapshot author trust hardening は Issue 実装: contract snapshotのtrusted publisher identityとcomment ID bindingを検証する #1475 が追跡する。

Outcome

package.json の pnpm@11.7.0 を唯一の正規値として deploy-main / deploy-pr が同じ pnpm を bootstrap し、install 前の実行時照合と current-head deploy-pr 証跡で drift を fail-closed に検出できる状態にする。

In Scope

package.json に packageManager: pnpm@11.7.0 を追加し、workflow の pnpm version input は持たない単一 source of truth にする。
deploy-main / deploy-pr で pnpm/action-setup@0e279bb959325dab635dd2c09392533439d90093 を維持し、install 前に pnpm --version と expected 11.7.0 の完全一致を fail-closed で検査する。
scripts/ci/tests/test_deploy_pages_pnpm_setup.py を追加し、両 job の bootstrap contract、exact semver、順序、error suppression 不在を検査する。
current-head deploy-pr の run ID / attempt / job URL / head SHA / expected と actual の pnpm version / frozen install 成功を PR evidence に残す。
package.json#packageManager 導入で pnpm/action-setup が「Multiple versions of pnpm specified」で fail-closed になる他の pnpm consumer（.github/actions/setup-node-pnpm/action.yml、.github/workflows/session-manifest.yml、.github/workflows/agent-retro-index.yml）を、明示 version: input を持たず packageManager から version を導出する形に揃え、単一 pnpm contract の Outcome を repository 全体で成立させる。
agent-retro-index.yml に pull_request trigger を追加して bootstrap の CI 検証を可能にしたことで新たに顕在化した .claude/skills/pr-review-judge/scripts/ci_verdict_summary.py の HEAD_SHA_NULL_SKIPPED_EXCLUDE_RULES allowlist gap（agent-retro-index の build-index / upsert-parent-comment が非対応写像パターンとして未登録で誤って failed_checks に計上される）を修正し、同スクリプトの単体テスト（.claude/skills/pr-review-judge/scripts/tests/test_ci_verdict_summary.py）に回帰テストを追加する。

Out of Scope

PR fix: legacy contract snapshot を更新可能にする #1463 の変更コード、pnpm 以外への package manager 移行、lockfile 更新。
deploy artifact、Pages publish、preview URL semantics。
continue-on-error、|| true、失敗 check の非 blocking 化、runtime evidence のローカル代替 PASS。
pnpm/action-setup の mutable tag（setup-node-pnpm/action.yml の @v6、agent-retro-index.yml の @v4）を commit SHA へ pin する対応（別 follow-up Issue で扱う）。
ci_verdict_summary.py の allowlist gap 修正以外の ci_verdict_summary.py / ci_verdict_summary_v2.py の設計変更。

Acceptance Criteria

Verification Commands

# AC1
# baseline-expect: fail
$ uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py::test_deploy_jobs_require_package_manager_pinned_bootstrap -v

# AC2
# preflight-scope: pr_review_only
$ uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v

# AC3
# preflight-scope: pr_review_only
$ uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v

# AC4
# preflight-scope: pr_review_only
$ uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v

# AC5
# preflight-scope: pr_review_only
$ uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v

# AC6
# preflight-scope: runtime_only
$ gh pr checks --repo squne121/loop-protocol --required

# AC7
$ pnpm typecheck

# AC8
$ pnpm lint

# AC9
$ pnpm test

# AC10
$ pnpm build

# AC11
# baseline-expect: fail
$ uv run --locked pytest .claude/skills/pr-review-judge/scripts/tests/test_ci_verdict_summary.py -v

Runtime Verification Applicability

decision: immediate
runtime_required: true
applicable_acs:
  - AC3
  - AC6
execution_environment: GitHub Actions deploy-pages workflow on the implementation PR; Actions read access and the current PR number / head SHA are required.
skip_conditions:
  - GitHub Actions が利用不能、または current-head の PR number / head SHA / deploy-pr check を取得できない場合は exit 77 相当の SKIP として停止する
fallback_policy: ローカル test、workflow 再実行、または stale head の成功を deploy-pr runtime PASS の代替にしない
artifact_requirements:
  - current head SHA、PR number、workflow run ID、run attempt、job URL
  - expected / actual pnpm version と pnpm install --frozen-lockfile 成功を示す deploy-pr log
reason: action の static pin だけでは PATH から解決された pnpm executable を証明できないため。

Allowed Paths

.github/workflows/deploy-pages.yml
scripts/ci/tests/test_deploy_pages_pnpm_setup.py
package.json
.github/actions/setup-node-pnpm/action.yml
.github/workflows/session-manifest.yml
.github/workflows/agent-retro-index.yml
.claude/skills/pr-review-judge/scripts/ci_verdict_summary.py
.claude/skills/pr-review-judge/scripts/tests/test_ci_verdict_summary.py

Stop Conditions

Allowed Paths 外の変更が必要と判明した場合
In Scope の固定契約（キー集合・スキーマ・型定義）の変更が必要になった場合
新規 Issue の起票が必要と判断した場合（スコープ分割が発生する場合）
後続 Phase / 別スコープへの波及が判明した場合
nested SubAgent delegation が必要になった場合
外部サービス利用・権限昇格・既存テスト大規模改変が必要になった場合
action SHA 0e279bb959325dab635dd2c09392533439d90093、pnpm 11.7.0、lockfile、または packageManager 以外の repository-wide package-manager policy を変更する必要が生じた場合
deploy-pr を非 blocking 化する必要、または current-head deploy-pr evidence を取得できない状態になった場合
ci_verdict_summary.py の allowlist gap 修正以外の設計変更（判定ロジック全体の再設計等）が必要と判明した場合

Required Skills

ci-test-performance

Required Design References

docs/dev/test-lane-policy.md
docs/dev/ci-performance.md
docs/dev/runtime-verification-policy.md
.github/workflows/deploy-pages.yml
fix: self-update bootstrap to packageManager-pinned version (#233) pnpm/action-setup#256

Scope Delta（任意）

anchor comment に従い package.json を Allowed Paths に追加する。これは packageManager の単一 source of truth を定義するための最小変更であり、lockfile には広げない。

追加の Scope Delta（PR #1480 CI 実行で判明・人間承認済み、1回目）: package.json#packageManager 導入後、実際に CI を走らせたところ .github/actions/setup-node-pnpm/action.yml（デフォルト pnpm-version: '11'）、.github/workflows/session-manifest.yml（version: '11.1.2'）、.github/workflows/agent-retro-index.yml（pnpm/action-setup@v4 に version: 10'）の3ファイルが明示 version: を渡しており、pnpm/action-setup の「Multiple versions of pnpm specified」で fail-closed になることが確認された（PR #1480 CI run 29175334036 等）。これらは本 Issue の Outcome である「単一の pnpm package-manager contract」の一部であり、pnpm 11.7.0 という正規値そのものは変更せず、各 consumer を packageManager から version を導出する形に揃えるだけの最小拡張のため、上記3ファイルを Allowed Paths に追加する。lockfile や pnpm 以外の package-manager policy 変更は引き続き対象外。

追加の Scope Delta（PR #1480 敵対的レビュー対応・人間承認済み、2回目）: agent-retro-index.yml に pull_request trigger を追加して pnpm bootstrap の CI 検証を可能にした結果、ci_verdict_summary.py の HEAD_SHA_NULL_SKIPPED_EXCLUDE_RULES allowlist が agent-retro-index の新規 skipped job（build-index / upsert-parent-comment）を認識せず誤って failed_checks に計上する gap が判明した（reviewer 独自の ci_verdict_summary.py 再実行で確認、exit 10）。既存の ("deploy-pages", "deploy-main") / ("deploy-pages", "cleanup-pr") と同一パターンの allowlist 追加のみであり、判定ロジック自体の再設計は伴わないため、.claude/skills/pr-review-judge/scripts/ci_verdict_summary.py と対応する単体テストを Allowed Paths に追加する。pnpm/action-setup の mutable tag を SHA pin する対応は別 follow-up Issue に切り出し、本 Issue のスコープには含めない。

CI_TEST_PERFORMANCE_DECISION_V1

schema: CI_TEST_PERFORMANCE_DECISION_V1
issue_number: 1476
pr_number: 1480
decision_scope: ci_change
changed_paths:
  - .github/workflows/deploy-pages.yml
  - scripts/ci/tests/test_deploy_pages_pnpm_setup.py
  - package.json
  - .github/actions/setup-node-pnpm/action.yml
  - .github/workflows/session-manifest.yml
  - .github/workflows/agent-retro-index.yml
  - .claude/skills/pr-review-judge/scripts/ci_verdict_summary.py
  - .claude/skills/pr-review-judge/scripts/tests/test_ci_verdict_summary.py
lane_classification:
  fast_static:
    applicable: true
    evidence:
      - workflow YAML と packageManager contract の静的整合
    required_commands:
      - pnpm typecheck
      - pnpm lint
  python_unit:
    applicable: true
    evidence:
      - deploy-pages bootstrap contract test
      - ci_verdict_summary.py allowlist 回帰テスト
    required_commands:
      - uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v
      - uv run --locked pytest .claude/skills/pr-review-judge/scripts/tests/test_ci_verdict_summary.py -v
  contract_artifact:
    applicable: true
    evidence:
      - exact pnpm、action SHA、bootstrap order、error suppression 不在
    required_commands:
      - uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v
  integration:
    applicable: true
    evidence:
      - current-head deploy-pr の pnpm actual version と frozen install
      - ci.yml / session-manifest.yml / agent-retro-index.yml の pnpm bootstrap 成功
    required_commands:
      - gh pr checks --repo squne121/loop-protocol --required
baseline_inputs:
  ci_runtime_baseline_v1_available: false
  run_count: 0
  p50_p95_ready: false
artifact_consistency:
  ci_test_selection_v1_checked: true
  missing_pytest_args: []
risk_flags:
  - workflow_ci_lane_change
  - deploy_gate_change
  - external_action_bootstrap
reviewer_gate:
  approve_allowed: false
  required_evidence:
    - TEST_VERDICT_MACHINE
    - CI_CHECK_RUN_SCOPED
follow_up_required: []

Delivery Rule

1 Issue = 1 PR。専用 worktree / branch で実装し、Draft PR を既定とする。
current-head deploy-pr 成功証跡が揃うまで merge-ready と判定しない。

総合判定

REQUEST_CHANGES。現状の Issue #1476 は、そのまま実装に渡すべきではありません。

障害の切り分け方向は妥当ですが、「決定論的に解消する」という目標に対して、pnpm の正規バージョン、採用する action 系列、実行時に確認すべき pnpm 実体が契約化されていません。現在の AC と Allowed Paths のままでは、表面的な YAML 固定だけで完了扱いになる可能性があります。

Issue #1476 自体にはコメントはなく、直接の根拠は PR #1463 の反復失敗とレビューです。PR #1463 では同一 head の attempt 1/2 が同じ self-installer error で失敗し、workflow/bootstrap infrastructure failure と分類されています。

重大な指摘

P0-1: `Allowed Paths` が正しい修正を禁止している

Issue は「repository の package manager contract と整合」と要求していますが、現在の package.json には packageManager フィールドがありません。

一方、pnpm/action-setup の公式仕様では、次のどちらかが必要です。

workflow の version
package.json の packageManager

また version: 11 は exact version ではなく、npm semver のメジャーレンジとして解釈されます。公式 README も 10、10.x.x、^10.9.8 などの範囲指定を許容すると明記しています。

現在の Allowed Paths は以下だけです。

.github/workflows/deploy-pages.yml
scripts/ci/tests/test_deploy_pages_pnpm_setup.py

これでは repository 全体の正規 pnpm バージョンを package.json に定義できません。

必須修正

次のどちらを採用するか、Issue 本文で明示してください。

推奨案

{
  "packageManager": "pnpm@<exact-version>"
}

そのうえで Allowed Paths に package.json を追加する。

または、package.json を変更しない方針なら、workflow の exact version を repository contract の正規値とすることを明記する必要があります。ただし、その場合は deploy workflow だけがバージョンの source of truth になり、他の CI やローカル環境との drift を防げません。

P0-2: 原因を「暗黙更新」と表現しており、既知の action v6 不具合を正確に捉えていない

現在利用している action は commit pin された v6 系です。両ジョブとも version: 11 を指定しています。

pnpm/action-setup 側では、v6 が指定された pnpm version を正しく反映せず、bootstrap pnpm v11 が残る問題が報告されています。具体的には、self-update されたバイナリより bootstrap バイナリが PATH 上で優先される問題と分析されています。

したがって、次のような変更だけでは不十分です。

version: 11.1.2

action v6 の PATH/bootstrap 不具合が該当するなら、exact version を書いても実際に呼ばれる pnpm がそのバージョンになる保証がありません。

必須修正

Issue に候補実装を明示してください。

修正済み action commit を検証して pin する
未修正なら v5 系の安全な commit へ一時的に pin する
action を使わず、Corepack等で exact pnpm を有効化する
いずれの場合も pnpm --version を runtime assertion として追加する

単に「action commit pin と pnpm version の整合性」では弱すぎます。実際に PATH から解決された pnpm の version が期待値と一致することが必要です。

P0-3: AC2 は実行された pnpm のバージョンを検証していない

AC2 は次の状態です。

v11 self-installer の暗黙更新に依存せず、pnpm install --frozen-lockfile まで成功する。

しかし pnpm install が成功するだけでは、期待した pnpm が使われた証明になりません。

現在の lockfile は lockfileVersion: '9.0' です。複数の pnpm major がこの形式を扱える可能性があるため、install 成功だけではバージョン drift を検知できません。

AC2 の修正版

AC2:
deploy-main / deploy-pr の pnpm setup 後に `pnpm --version` が
repository contract に定義された exact version と完全一致し、
続く `pnpm install --frozen-lockfile` が成功する。

workflow には最低でも次のステップが必要です。

- name: Verify pnpm version
  shell: bash
  run: |
    set -euo pipefail
    actual="$(pnpm --version)"
    expected="<exact-version>"
    test "$actual" = "$expected"

pnpm --version のログも CI 証跡要件に含めるべきです。

高優先度の指摘

P1-1: AC3 の contract test が実装詳細の固定に寄りすぎている

現在の AC3:

action commit pin、pnpm version、install 順序の drift を fail-closed で検出する。

問題は、「安全性」ではなく単なる YAML の形を固定していることです。

例えば以下の状態でも、浅い静的テストなら PASS し得ます。

action v6 の既知不具合を含む commit を pin
version: 11 という可変レンジを固定
packageManager と workflow version が不一致
pnpm --version assertion がない
deploy-main と deploy-pr の一方だけ環境変数や setup input が異なる

contract test が検証すべき意味論

action ref が40桁の commit SHA
action commit が承認済み SHA と一致
pnpm version が exact semver
package.json#packageManager と workflow の期待値が一致
deploy-main / deploy-pr の bootstrap step が意味論的に同一
pnpm --version assertion が install より前にある
pnpm install --frozen-lockfile が assertion より後にある
continue-on-error が setup/version-check/install に付いていない
shell 内で || true 等による握りつぶしがない

「ステップ順序」だけでなく、失敗を伝播する構造まで対象にしてください。

P1-2: baseline test が有効な Red test になっていない

Verification Commands には次があります。

# baseline-expect: fail
uv run --locked pytest \
  scripts/ci/tests/test_deploy_pages_pnpm_setup.py::test_deploy_jobs_pin_reproducible_pnpm_setup -v

しかしテストファイル自体が新規作成対象です。

実装前には、以下のどちらでも失敗します。

ファイルが存在しない
test node が存在しない

これは「既存 workflow が不正なため失敗した」という Red 証跡ではありません。

修正方法

テスト追加後、workflow 修正前の中間 commit または fixture に対して、期待する assertion failure が発生することを証明してください。

最低でも証跡に以下が必要です。

baseline_failure:
  command: ...
  exit_code: 1
  expected_failure_key: non_exact_pnpm_version
  unexpected_collection_error: false

P1-3: `gh pr checks` は current-head 証跡として曖昧

現在の AC4 コマンド:

gh pr checks

これは実行コンテキスト依存です。

current branch の特定に依存する
対象 PR が曖昧
check がどの head SHA に紐づくか本文だけでは証明できない
rerun 前の古い成功結果との混同を排除できない

修正版

gh pr checks <PR_NUMBER> \
  --repo squne121/loop-protocol \
  --required

加えて以下を取得してください。

gh pr view <PR_NUMBER> \
  --repo squne121/loop-protocol \
  --json headRefOid,statusCheckRollup

AC4 は次を machine-readable evidence に含めるべきです。

head_sha: <40-char SHA>
workflow: deploy-pages
job: deploy-pr
conclusion: success
run_attempt: <number>
pnpm_expected_version: <exact>
pnpm_actual_version: <exact>
frozen_lockfile_install: pass
check_url: ...

P1-4: `deploy-main` の runtime verification が欠落している

AC1 では deploy-main と deploy-pr の同一条件を要求していますが、runtime evidence は deploy-pr だけです。

静的に同一と判定しても、event context、permissions、checkout条件、cache条件の違いで main push 時だけ失敗する可能性があります。

ただし Issue PR から deploy-main を直接実行すると main 相当の publish権限を伴い、スコープが膨らみます。

現実的な整理

この Issue では以下のどちらかに明確化してください。

deploy-main は静的同値性のみ保証し、実 runtime はマージ後観測とする
reusable workflow または non-publishing bootstrap job を切り出して両経路で実行する

現状は AC1 が deploy-main runtime 成功まで含むようにも読める一方、証跡は deploy-pr のみで、検証境界が曖昧です。

P1-5: `version: 11` と「決定論的」が矛盾する

action の公式仕様上、11 は exact version ではありません。

したがって、Issue 本文にある以下の表現は不正確です。

commit pin 済み action に version: 11 を渡している

action commit を固定していても、npm registry 上で解決される pnpm 11 系のバージョンは将来変化します。

必須条件

「exact version」の定義を明記してください。

^[0-9]+\.[0-9]+\.[0-9]+(?:-[0-9A-Za-z.-]+)?$

少なくとも以下は reject すべきです。

11
11.x
^11.0.0
latest
*

中優先度の指摘

P2-1: Node と runner は完全には固定されていない

workflow は以下です。

runs-on: ubuntu-latest
node-version: 22

これも major/routing alias です。

今回の pnpm 障害スコープで OS image digest や Node patch まで固定する必要はありませんが、「bootstrap が完全に再現可能」と主張するのは過大です。

Outcome は次のように限定した方が正確です。

GitHub-hosted ubuntu-latest と Node 22 のサポート範囲内で、
pnpm の action implementation と package-manager version を固定し、
pnpm executable drift を検知する。

P2-2: action の更新方針がない

修正済み upstream commit を pin して終わると、将来の更新時に同じ問題を再導入できます。

contract test は SHA の固定だけでなく、更新手順を要求すべきです。

Dependabot 更新時に pnpm actual version test が走る
action ref の変更は contract test の期待値更新を伴う
upstream issue/PR または release note を PR に記録する
一時的に v5へ戻す場合は、v6復帰条件を追跡する

P2-3: `packageManager` 追加時の scope delta が Stop Condition と衝突する

Issue の Stop Conditions は「Allowed Paths 外が必要なら停止」です。

しかし、今回の adversarial review により package.json が必要となる可能性が高いため、現状の契約で実装Agentを開始すると、正しく停止するだけで成果が出ません。

実装開始前に Issue を更新すべきです。

推奨実装方針

最も堅実なのは次です。

repository の正規 pnpm exact version を決定
package.json#packageManager に exact versionを記録
pnpm/action-setup v6 の修正済み commitが確認できる場合だけ採用
未修正なら、既知の正常な v5 commitへ一時 pin
workflow内で pnpm --version を exact comparison
その後に actions/setup-node cache と frozen installを実行
deploy-main / deploy-pr の bootstrap部分をcontract testで比較
current-head の deploy-pr jobで actual versionとinstall成功を記録

概念例:

- name: Install pnpm
  uses: pnpm/action-setup@<reviewed-40-char-sha>
  with:
    version: 11.1.2

- name: Verify pnpm bootstrap
  shell: bash
  run: |
    set -euo pipefail
    expected='11.1.2'
    actual="$(pnpm --version)"
    printf 'expected=%s\nactual=%s\n' "$expected" "$actual"
    if [[ "$actual" != "$expected" ]]; then
      echo "::error::pnpm version mismatch"
      exit 1
    fi

- uses: actions/setup-node@<reviewed-40-char-sha>
  with:
    node-version: 22
    cache: pnpm

- name: Install dependencies
  run: pnpm install --frozen-lockfile

実際の exact version と action SHA は、リポジトリで現在サポートする pnpm versionおよび upstream 修正状態を確認して決定してください。

修正版 Acceptance Criteria

- [ ] AC1: `package.json#packageManager` または明示された単一の
  repository contract が、exact pnpm version を定義する。
  major range、`latest`、wildcard は許可しない。

- [ ] AC2: deploy-main / deploy-pr は、reviewed commit SHA に固定された
  同一の pnpm bootstrap implementation と exact pnpm version を使用する。

- [ ] AC3: 両ジョブは install 前に `pnpm --version` を実行し、
  repository contract の exact version と不一致なら fail-closed で停止する。

- [ ] AC4: contract test は、action SHA、exact pnpm version、
  repository contractとの一致、version assertion、frozen install順序、
  error suppression禁止を検証する。

- [ ] AC5: baseline test は collection errorではなく、
  既存 `version: 11` または version assertion欠落を理由として失敗する。

- [ ] AC6: current PR head SHAに紐づく deploy-pr jobが成功し、
  ログに期待値と一致する `pnpm --version` および
  `pnpm install --frozen-lockfile` 成功が記録される。

- [ ] AC7: evidenceには head SHA、workflow run ID、run attempt、
  job URL、expected/actual pnpm versionを含める。

- [ ] AC8: typecheck、lint、test、buildがPASSする。

Allowed Paths 修正版

- `.github/workflows/deploy-pages.yml`
- `scripts/ci/tests/test_deploy_pages_pnpm_setup.py`
- `package.json`

packageManager を採用しない設計判断を明示する場合だけ、package.json を除外できます。

最終評価

Issue #1476 は、障害を independent infrastructure failure として分離した点は適切です。しかし、現状は以下の3点が未解決です。

正規 pnpm exact version がない
pnpm/action-setup v6 の既知 bootstrap/PATH問題を実装判断に反映していない
実際に使用された pnpm executable を検証しない

このままでは「workflow に exact-looking な文字列を書き、静的テストを通したが、PATH上では別の pnpmが動いている」という再発を防げません。実装開始前に Issue 契約の修正が必要です。

Contract Review Result

CONTRACT_REVIEW_RESULT_V1:
  status: go
  generated_at: "2026-07-12T00:18:45Z"
  generated_by: issue-contract-review
  issue_url: https://github.com/squne121/loop-protocol/issues/1476
  body_sha256: "sha256:16535438e54d25adec21ee4bff5c1efb028493164ca2b7239014f1cf38d64ad5"
  checks:
    readiness: go
    blockers: pass
    product_spec: pass
    vc_preflight:
      decision: pass
      classifications: [{"ac":"AC1","line":3,"raw_command":"uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py::test_deploy_jobs_require_package_manager_pinned_bootstrap -v","command_hash":"sha256:47f9037ff05149be30679d0cb5d54576189ea5db387101df86496465fa67796d","runner":"exec","exit_code":4,"classification":"expected_fail","category":"expected_baseline_fail","decision":"go","scope_class":"baseline_fail_expected","confidence":"high","stdout_head":["============================= test session starts ==============================","platform linux -- Python 3.12.3, pytest-9.0.3, pluggy-1.6.0 -- /home/squne/projects/LOOP_PROTOCOL/.venv/bin/python","cachedir: .pytest_cache","rootdir: /home/squne/projects/LOOP_PROTOCOL","configfile: pyproject.toml","plugins: xdist-3.8.0","collecting ... collected 0 items","============================ no tests ran in 0.01s ============================="],"stdout_truncated":false,"stdout_original_line_count":10,"stderr_head":["ERROR: file or directory not found: scripts/ci/tests/test_deploy_pages_pnpm_setup.py::test_deploy_jobs_require_package_manager_pinned_bootstrap"],"stderr_truncated":false,"stderr_original_line_count":3,"runner_env_delta":{},"duration_ms":192,"fix_hint":null,"annotations":{"baseline_expect":"fail","vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":2,"raw":"# baseline-expect: fail"},"strict":null,"repair":null},{"ac":"AC2","line":7,"raw_command":"uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v","command_hash":"sha256:30a0d6b858160d309e05112f625eada334370cd8428463316c767e543fe5b1e7","runner":"skipped","exit_code":null,"classification":"skipped","category":"preflight_scope_pr_review_only","decision":"go","scope_class":"pr_review_only","confidence":"low","stdout_head":[],"stdout_truncated":false,"stdout_original_line_count":1,"stderr_head":[],"stderr_truncated":false,"stderr_original_line_count":1,"runner_env_delta":{},"duration_ms":0,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null,"verification_owner":"pr-review-judge","deferred_reason":"VC marked pr_review_only; verification deferred to PR review","runtime_verification_required":false},{"ac":"AC3","line":11,"raw_command":"uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v","command_hash":"sha256:30a0d6b858160d309e05112f625eada334370cd8428463316c767e543fe5b1e7","runner":"skipped","exit_code":null,"classification":"skipped","category":"preflight_scope_pr_review_only","decision":"go","scope_class":"pr_review_only","confidence":"low","stdout_head":[],"stdout_truncated":false,"stdout_original_line_count":1,"stderr_head":[],"stderr_truncated":false,"stderr_original_line_count":1,"runner_env_delta":{},"duration_ms":0,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null,"verification_owner":"pr-review-judge","deferred_reason":"VC marked pr_review_only; verification deferred to PR review","runtime_verification_required":false},{"ac":"AC4","line":15,"raw_command":"uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v","command_hash":"sha256:30a0d6b858160d309e05112f625eada334370cd8428463316c767e543fe5b1e7","runner":"skipped","exit_code":null,"classification":"skipped","category":"preflight_scope_pr_review_only","decision":"go","scope_class":"pr_review_only","confidence":"low","stdout_head":[],"stdout_truncated":false,"stdout_original_line_count":1,"stderr_head":[],"stderr_truncated":false,"stderr_original_line_count":1,"runner_env_delta":{},"duration_ms":0,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null,"verification_owner":"pr-review-judge","deferred_reason":"VC marked pr_review_only; verification deferred to PR review","runtime_verification_required":false},{"ac":"AC5","line":19,"raw_command":"uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v","command_hash":"sha256:30a0d6b858160d309e05112f625eada334370cd8428463316c767e543fe5b1e7","runner":"skipped","exit_code":null,"classification":"skipped","category":"preflight_scope_pr_review_only","decision":"go","scope_class":"pr_review_only","confidence":"low","stdout_head":[],"stdout_truncated":false,"stdout_original_line_count":1,"stderr_head":[],"stderr_truncated":false,"stderr_original_line_count":1,"runner_env_delta":{},"duration_ms":0,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null,"verification_owner":"pr-review-judge","deferred_reason":"VC marked pr_review_only; verification deferred to PR review","runtime_verification_required":false},{"ac":"AC6","line":23,"raw_command":"gh pr checks --repo squne121/loop-protocol --required","command_hash":"sha256:6dd040346c01ab260900ce312454007dd7192737dcfb9bf24de2a53598b30c42","runner":"skipped","exit_code":null,"classification":"skipped","category":"preflight_scope_runtime_only","decision":"go","scope_class":"runtime_only","confidence":"low","stdout_head":[],"stdout_truncated":false,"stdout_original_line_count":1,"stderr_head":[],"stderr_truncated":false,"stderr_original_line_count":1,"runner_env_delta":{},"duration_ms":0,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null,"verification_owner":"impl-review-loop","deferred_reason":"VC marked runtime_only; verification deferred to post-implementation runtime","runtime_verification_required":true},{"ac":"AC7","line":26,"raw_command":"pnpm typecheck","command_hash":"sha256:1b65adab2d69e0f148ddd11c15c7dcd73ca087b66cc02dac2672f9493093cc6d","runner":"exec","exit_code":0,"classification":"expected_pass","category":"regression_gate","decision":"go","scope_class":"regression_gate","confidence":"low","stdout_head":[],"stdout_truncated":false,"stdout_original_line_count":1,"stderr_head":["$ tsc --noEmit"],"stderr_truncated":false,"stderr_original_line_count":2,"runner_env_delta":{"CI":"true"},"duration_ms":930,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null},{"ac":"AC8","line":29,"raw_command":"pnpm lint","command_hash":"sha256:ede823a2d5f2814db6ddd8a2969504d513bf5a4428b806aa4d1a0499fb38462f","runner":"exec","exit_code":0,"classification":"expected_pass","category":"regression_gate","decision":"go","scope_class":"regression_gate","confidence":"low","stdout_head":[],"stdout_truncated":false,"stdout_original_line_count":1,"stderr_head":["$ eslint ."],"stderr_truncated":false,"stderr_original_line_count":2,"runner_env_delta":{"CI":"true"},"duration_ms":5382,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null},{"ac":"AC9","line":32,"raw_command":"pnpm test","command_hash":"sha256:ae7aca4de98e885127fe392c0b60056f3472ed0f0be972870a06df422b3f140a","runner":"exec","exit_code":0,"classification":"expected_pass","category":"regression_gate","decision":"go","scope_class":"regression_gate","confidence":"low","stdout_head":[" RUN  v4.1.6 /home/squne/projects/LOOP_PROTOCOL"," Test Files  96 passed (96)","      Tests  1575 passed (1575)","   Start at  09:18:27","   Duration  13.75s (transform 22.46s, setup 0ms, import 36.05s, tests 121.28s, environment 10.20s)"],"stdout_truncated":false,"stdout_original_line_count":10,"stderr_head":["$ vitest run --exclude 'tests/e2e/**' --exclude '.claude/worktrees/**'"],"stderr_truncated":false,"stderr_original_line_count":2,"runner_env_delta":{"CI":"true"},"duration_ms":16305,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null},{"ac":"AC10","line":35,"raw_command":"pnpm build","command_hash":"sha256:702e948d947e109310cb2bf19c157b5bb8142b88c87e02ae87a7594c9b797893","runner":"exec","exit_code":0,"classification":"expected_pass","category":"regression_gate","decision":"go","scope_class":"regression_gate","confidence":"low","stdout_head":["vite v8.0.13 building client environment for production...","\u001b[2K","transforming...✓ 41 modules transformed.","rendering chunks...","computing gzip size...","dist/index.html                  0.46 kB │ gzip:  0.30 kB","dist/assets/index-BMsLd5fy.css   3.49 kB │ gzip:  1.47 kB","dist/assets/index-FsMPCAOe.js   61.16 kB │ gzip: 18.85 kB","✓ built in 116ms"],"stdout_truncated":false,"stdout_original_line_count":11,"stderr_head":["$ tsc && vite build"],"stderr_truncated":false,"stderr_original_line_count":2,"runner_env_delta":{"CI":"true"},"duration_ms":1192,"fix_hint":null,"annotations":{"baseline_expect":null,"vc_role":null,"missing_baseline_expect":false},"annotation_source":{"line":null,"raw":null},"strict":null,"repair":null}]
  source: ensure_contract_snapshot_auto

Contract Review Result (serialization repair)

The controlled publisher posted the same fresh review result, but its inline JSON cannot be consumed as YAML because of a control-character escape. This normalized block preserves the validated go result and VC classifications without changing the Issue body.

CONTRACT_REVIEW_RESULT_V1:
  status: go
  generated_at: "2026-07-12T00:18:45Z"
  generated_by: issue-contract-review
  issue_url: https://github.com/squne121/loop-protocol/issues/1476
  body_sha256: "sha256:16535438e54d25adec21ee4bff5c1efb028493164ca2b7239014f1cf38d64ad5"
  checks:
    readiness: go
    blockers: pass
    product_spec: pass
    vc_preflight:
      decision: pass
      classifications:
        - ac: AC1
          classification: expected_fail
          category: expected_baseline_fail
          decision: go
          scope_class: baseline_fail_expected
        - ac: AC2
          classification: skipped
          category: preflight_scope_pr_review_only
          decision: go
          scope_class: pr_review_only
        - ac: AC3
          classification: skipped
          category: preflight_scope_pr_review_only
          decision: go
          scope_class: pr_review_only
        - ac: AC4
          classification: skipped
          category: preflight_scope_pr_review_only
          decision: go
          scope_class: pr_review_only
        - ac: AC5
          classification: skipped
          category: preflight_scope_pr_review_only
          decision: go
          scope_class: pr_review_only
        - ac: AC6
          classification: skipped
          category: preflight_scope_runtime_only
          decision: go
          scope_class: runtime_only
        - ac: AC7
          classification: expected_pass
          category: regression_gate
          decision: go
          scope_class: regression_gate
        - ac: AC8
          classification: expected_pass
          category: regression_gate
          decision: go
          scope_class: regression_gate
        - ac: AC9
          classification: expected_pass
          category: regression_gate
          decision: go
          scope_class: regression_gate
        - ac: AC10
          classification: expected_pass
          category: regression_gate
          decision: go
          scope_class: regression_gate
  source: contract_snapshot_serialization_repair

Refinement Loop: Approved

The issue has been approved and is ready for implementation.

Issue: 実装: deploy-pr の pnpm self-installer failure を決定論的に解消する #1476
Final iteration: 1
Termination reason: approved
Generated at: 2026-07-12T00:22:56Z

LOOP_HANDOFF_RESULT_V1:
  status: impl_ready
  routing_action: run_impl_review_loop
  contract_review:
    status: go
    gate_result: fresh_go
    latest_comment_url: https://github.com/squne121/loop-protocol/issues/1476#issuecomment-4949300954
    generated_at: '2026-07-12T00:18:45Z'
    body_sha256: sha256:16535438e54d25adec21ee4bff5c1efb028493164ca2b7239014f1cf38d64ad5
  metadata:
    title_prefix_ready: true
    phase_label_ready: true
  auto_fixes:
    result: auto_fixed
    required: []
    skipped: []
  blockers: []
  permissions:
    unavailable: []
  generated_at: '2026-07-12T00:22:02Z'
  checked_body_sha256: sha256:16535438e54d25adec21ee4bff5c1efb028493164ca2b7239014f1cf38d64ad5
  checker_exit_code: 0
  missing_sections: []
  missing_contract_keys: []

impl-review-loop 停止報告

実装前の Scope Collision Preflight が fail-closed となったため、Issue 本文を変更せずに停止します。

source_complete: false
source_saturated: true
候補 Issue #481、#666、#667、#901 に missing_allowed_paths が検出されました。
この状態では overlap 判定の入力完全性を証明できず、worktree 作成・実装・PR 起票・current-head deploy-pr runtime evidence 取得には進みません。

IMPL_REVIEW_LOOP_RESULT_V1:
  status: human_escalation
  termination_reason: human_escalation
  iteration: 0
  last_step: implementation
  pr_url: null
  merge_ready: false
  required_auto_actions: []
  blocker:
    kind: scope_collision_preflight_incomplete
    source_complete: false
    source_saturated: true
    candidates:
      - issue_number: 481
        reason: missing_allowed_paths
      - issue_number: 666
        reason: missing_allowed_paths
      - issue_number: 667
        reason: missing_allowed_paths
      - issue_number: 901
        reason: missing_allowed_paths

Scope Collision Preflight override

ユーザーの明示判断「競合判定は無視して作業を進めて」に基づき、Scope Collision Preflight のみを今回の実装で override します。

対象: IMPLEMENT_SCOPE_COLLISION_PREFLIGHT_V1 の human_review_required
除外する停止理由: 候補 fix: verify_acp_roundtrip.sh の scenario 2b (fake ACP + --approve-edits) FAIL を修正する #481 / 実装: plan_refinement_loop の _extract_missing_contract_keys を PyYAML 不在時に fail-closed にする（#661 follow-up） #666 / 実装: plan_refinement_loop の fail_closed reason code exhaustiveness invariant をテストする（#661 follow-up） #667 / 実装: baseline-expect annotation の pytest テストを CI required ジョブに追加し regression gate 化する #901 の missing_allowed_paths、および候補取得の source_saturated
維持するゲート: Issue contract / Allowed Paths / Runtime Verification Applicability / CI / current-head evidence / publish safety
この override は実装: deploy-pr の pnpm self-installer failure を決定論的に解消する #1476 の実装にのみ適用し、Scope Collision Preflight の恒久仕様変更ではありません。

実装再開前に、上記以外の契約ゲートを引き続き検証します。

Contract Review Result（再実行）

既存の go コメントには product_spec_check の詳細オブジェクトがなかったため、issue-contract-review の fresh preflight 結果から完全なプロダクト仕様ゲートを再投稿します。Issue 本文は変更していません。

CONTRACT_REVIEW_RESULT_V1:
  status: go
  generated_at: "2026-07-12T00:50:20Z"
  generated_by: issue-contract-review
  issue_url: https://github.com/squne121/loop-protocol/issues/1476
  body_sha256: "sha256:16535438e54d25adec21ee4bff5c1efb028493164ca2b7239014f1cf38d64ad5"
  checks:
    template_compliance: pass
    state_label: pass
    allowed_paths_present: pass
    vc_present: pass
    stop_conditions_complete: pass
    ac_verifiability: pass
    worktree_branch_collision: pass
    readiness: go
    blockers: pass
    product_spec: pass
    product_spec_check:
      schema: product_spec_check/v1
      applicability: applicable
      decision: pass
      triggers:
        docs_product_allowed_paths: false
        tasks_md_mentioned: false
        specify_artifact_mentioned: false
        generated_task_mentioned: false
        product_spec_context_present: false
        machine_readable_contract_present: true
      conditions:
        docs_product_requires_spec_evidence:
          status: pass
          evidence: []
        tasks_md_not_direct_source:
          status: pass
          evidence: []
        specify_not_canonical:
          status: pass
          evidence: []
        diff_first_rationale_present:
          status: pass
          evidence: []
        generated_task_trace_present:
          status: pass
          evidence: []
        task_dependencies_materialized:
          status: pass
          evidence: []
      blocked_reasons: []
      body_sha256: "sha256:16535438e54d25adec21ee4bff5c1efb028493164ca2b7239014f1cf38d64ad5"
      source_provenance:
        source_type: github_api
        body_file: null
    vc_preflight:
      decision: pass
      classifications:
        - {ac: AC1, classification: expected_fail, category: expected_baseline_fail, decision: go, scope_class: baseline_fail_expected}
        - {ac: AC2, classification: skipped, category: preflight_scope_pr_review_only, decision: go, scope_class: pr_review_only}
        - {ac: AC3, classification: skipped, category: preflight_scope_pr_review_only, decision: go, scope_class: pr_review_only}
        - {ac: AC4, classification: skipped, category: preflight_scope_pr_review_only, decision: go, scope_class: pr_review_only}
        - {ac: AC5, classification: skipped, category: preflight_scope_pr_review_only, decision: go, scope_class: pr_review_only}
        - {ac: AC6, classification: skipped, category: preflight_scope_runtime_only, decision: go, scope_class: runtime_only}
        - {ac: AC7, classification: expected_pass, category: regression_gate, decision: go, scope_class: regression_gate}
        - {ac: AC8, classification: expected_pass, category: regression_gate, decision: go, scope_class: regression_gate}
        - {ac: AC9, classification: expected_pass, category: regression_gate, decision: go, scope_class: regression_gate}
        - {ac: AC10, classification: expected_pass, category: regression_gate, decision: go, scope_class: regression_gate}
    next_action: implement_issue
    blocked_reasons: []
  next_action: implement_issue
  warnings: []
  source: issue_contract_review_forced_refresh

implement-issue: overlap preflight が human_review_required を返したため実装を停止

implement-issue/SKILL.md Step 2（Contract-aware overlap preflight）を実行した結果、route: human_review_required となり Step 3（worktree 作成）へ進めませんでした。

IMPLEMENT_SCOPE_COLLISION_PREFLIGHT_V1:
  route: human_review_required
  source:
    complete: true
    saturated: false
  candidates_parsed: 13
  candidates_policy_class: all C1 (disjoint — deploy-pages/pnpm scope と無関係)
  validation_errors:
    "150": [missing_allowed_paths]
    "152": [missing_allowed_paths]
    "154": [missing_allowed_paths]
    "173": [missing_allowed_paths]
    "481": [missing_allowed_paths]
    "666": [missing_allowed_paths]
    "667": [missing_allowed_paths]
    "901": [missing_allowed_paths]
  decision_inputs_sha256: "sha256:2c18cd85b786ab573a83ae17a8cb517c9fc35beb113248f6eea183ee340d609a"
  evidence_sha256: "sha256:30bdd0d54f76064f90e9de74133d88c49f405b2d8924f2f890a6430eeb9aea5a"

根拠

--limit 250（デフォルト100を超えて phase/implementation OPEN Issue 全165件を取得）で再実行すると source.complete: true / saturated: false になり、飽和による human_review_required は解消しました。
残る human_review_required の唯一の理由は、legacy Issue（実装: post-merge-cleanup に parent body materialization guard を追加する #150 実装: issue-contract-review に parent body consistency check を追加する #152 実装: open-pr に tracking/parent issue への Closes 自動付与を防ぐ guard を追加する #154 chore: #145/#146/#147 を milestone-ops.md 確定後に issue-contract-review で再 preflight する #173 fix: verify_acp_roundtrip.sh の scenario 2b (fake ACP + --approve-edits) FAIL を修正する #481 実装: plan_refinement_loop の _extract_missing_contract_keys を PyYAML 不在時に fail-closed にする（#661 follow-up） #666 実装: plan_refinement_loop の fail_closed reason code exhaustiveness invariant をテストする（#661 follow-up） #667 実装: baseline-expect annotation の pytest テストを CI required ジョブに追加し regression gate 化する #901）が ## Allowed Paths セクションを持たず missing_allowed_paths で候補スキーマ検証が失敗するためです。
これら 8 件のタイトルを確認しましたが、いずれも .github/workflows/deploy-pages.yml / scripts/ci/tests/test_deploy_pages_pnpm_setup.py / package.json とは無関係（post-merge-cleanup guard、issue-contract-review parent body check、ACP roundtrip fix、plan_refinement_loop fail-closed 化等）です。
この fail-closed パターンは既存 OPEN Issue 実装: Scope Collision Preflight で Allowed Paths 未記載 Issue を候補から除外する #1477「実装: Scope Collision Preflight で Allowed Paths 未記載 Issue を候補から除外する」が正確に追跡している既知の構造的問題です。

停止理由

implement-issue/SKILL.md の route 契約表では human_review_required は「人間判断へ停止」が唯一の許容アクションであり、implementation-worker 自身が false positive と判断して override することは許可されていません。よって Step 3（worktree 作成）以降を実行せず、ここで停止します。

推奨対応（人間判断）

本 Issue 実装: deploy-pr の pnpm self-installer failure を決定論的に解消する #1476 は実質的にスコープ衝突なし（13 候補すべて C1 disjoint）と評価できるため、人間の目視確認の上で implement-issue の再実行（または実装: Scope Collision Preflight で Allowed Paths 未記載 Issue を候補から除外する #1477 のマージ後の再実行）を承認していただければ、そのまま実装を継続できます。

implement-issue: overlap preflight の human_review_required は人間レビューで解消済み

前回の実行で check_implementation_overlap.py が route: human_review_required を返しました（証跡）。原因は legacy Issue（#150 #152 #154 #173 #481 #666 #667 #901）が ## Allowed Paths セクションを持たず missing_allowed_paths で候補スキーマ検証が失敗したことによる fail-closed（既知の構造的問題、Issue #1477 が追跡）です。

repo owner（squne121）が gh issue view --json title,body で該当 8 件全ての本文を目視確認し、以下の通り本 Issue #1476 のスコープ（.github/workflows/deploy-pages.yml / package.json / scripts/ci/tests/test_deploy_pages_pnpm_setup.py）と意味的に disjoint であることを確認しました。

実装: post-merge-cleanup に parent body materialization guard を追加する #150: post-merge-cleanup parent body materialization guard
実装: issue-contract-review に parent body consistency check を追加する #152: issue-contract-review parent body consistency check
実装: open-pr に tracking/parent issue への Closes 自動付与を防ぐ guard を追加する #154: open-pr tracking/parent issue Closes 自動 attach 防止 guard
chore: #145/#146/#147 を milestone-ops.md 確定後に issue-contract-review で再 preflight する #173: milestone-ops.md finalization re-preflight（実装: M1 Foundation Gate GitHub Milestone object を materialize する (M-A2) #145/実装: 既存 Issue 群の GitHub Milestone 割当設計と readback を行う (M-A3) #146/実装: milestone rollup / next-action 導線を作る (M-A4) #147）
fix: verify_acp_roundtrip.sh の scenario 2b (fake ACP + --approve-edits) FAIL を修正する #481: verify_acp_roundtrip.sh scenario 2b FAIL 修正
実装: plan_refinement_loop の _extract_missing_contract_keys を PyYAML 不在時に fail-closed にする（#661 follow-up） #666: plan_refinement_loop PyYAML 不在時の fail-closed 処理
実装: plan_refinement_loop の fail_closed reason code exhaustiveness invariant をテストする（#661 follow-up） #667: plan_refinement_loop fail_closed reason code 網羅性テスト
実装: baseline-expect annotation の pytest テストを CI required ジョブに追加し regression gate 化する #901: .github/workflows/ci.yml python-test job annotation テストカバレッジ + 新規 docs/dev/ci-operations.md

#901 は ci.yml を触るが deploy-pages.yml とは別ファイルであり package.json にも触れません。他 13 件の real candidate は前回実行で既に policy_class C1（disjoint）と確認済みです。

この人間レビューをもって human_review_required route の要求する人間判断とみなし、Step 3（worktree 作成）以降を継続します。

implement-issue: 実装は完了・PR 作成のみ open-pr の Overlap Preflight Hard Gate でブロック

Allowed Paths 内での実装（package.json / .github/workflows/deploy-pages.yml / scripts/ci/tests/test_deploy_pages_pnpm_setup.py）、TDD による契約テスト追加、Verification Commands（AC1〜AC5, AC7〜AC10）全 PASS、commit、push まで完了しました。

branch: worktree-agent-afda59fa16381334b
head commit: cb5c65e73acae2c4a1ec2f4a85d9aa969184ce2f

検証結果

uv run --locked pytest scripts/ci/tests/test_deploy_pages_pnpm_setup.py -v
  実装前(baseline): 6 failed, 1 passed
  実装後: 7 passed

pnpm typecheck   PASS (exit 0)
pnpm lint        PASS (exit 0)
pnpm test        PASS (exit 0, 96 files / 1575 tests)
pnpm build       PASS (exit 0)

ブロック理由

open-pr skill（open_pr.py）の Overlap Preflight Hard Gate（.claude/skills/open-pr/SKILL.md 4.5 節）は gh pr create 直前に check_implementation_overlap.py をオンラインで再実行し、fresh route が proceed / proceed_with_collision_evidence であることを要求します。本 Issue は phase/implementation ラベルを持つため、この gate は forced_by_label により caller の overlap_preflight 指定の有無に関わらず必ず起動し、バイパスするオプション引数は存在しません。

先行コメント（#1476 (comment) ・ https://github.com/squne121/loop-protocol/issues/1476#issuecomment-4949421980）で implement-issue/SKILL.md Step 2 の human_review_required は人間レビューで解消済みとされていますが、これは implement-issue の worker 判断（Step 3 = worktree 作成へ進むかどうか）に対する承認であり、open-pr 側が PR 作成直前に独立して行うオンライン再検証（Overlap Preflight Hard Gate）は別の強制ポイントです。実際に本セッションで再実行したところ、依然として以下の通りです:

route: human_review_required
validation_errors:
  "150": [missing_allowed_paths]
  "152": [missing_allowed_paths]
  "154": [missing_allowed_paths]
  "173": [missing_allowed_paths]
  "481": [missing_allowed_paths]
  "666": [missing_allowed_paths]
  "667": [missing_allowed_paths]
  "901": [missing_allowed_paths]

原因は前回と同じ構造的問題（Issue #1477 が追跡・未マージ）です。open_pr.py にはこの route を override する CLI フラグが存在しないため、gh pr create を直接呼ぶ以外に回避策がありません。ガードレール上 open-pr を経由せず gh pr create を直接呼ぶことは禁止されているため、本セッションではこれ以上進めず停止します。

推奨対応（人間判断）

Issue 実装: Scope Collision Preflight で Allowed Paths 未記載 Issue を候補から除外する #1477（Scope Collision Preflight で Allowed Paths 未記載 Issue を候補から除外する）をマージすれば、この Overlap Preflight Hard Gate は自然に解消される見込みです。
または、open_pr.py に人間承認済みの human_review_required route を明示的に許容する override 経路を追加する（別 Issue 化が必要、本 Issue の Allowed Paths 外）。
上記のいずれかの後、本 branch (worktree-agent-afda59fa16381334b, head cb5c65e73acae2c4a1ec2f4a85d9aa969184ce2f) から open-pr を再実行すれば PR を起票できます。実装自体の再実行は不要です。

fix(workflow): pnpm/action-setup の明示 version pin を削除し packageManager …

fix: baseline-red fixture・SSOT version derivation・全pnpm consumer契約・be…

fix(ci): actionlint SC2016 false positive を shellcheck disable コメントで解消する

fix(pr-review-judge): agent-retro-index の skipped job を allowlist に追加する

fix: deploy-pr の pnpm self-installer failure を決定論的に解消する (#1480)

実装: deploy-pr の pnpm self-installer failure を決定論的に解消する #1476

Description

Machine-Readable Contract

Parent Issue

Parent Goal Ref

Current Validated Scope

Remaining Parent Gaps

Outcome

In Scope

Out of Scope

Acceptance Criteria

Verification Commands

Runtime Verification Applicability

Allowed Paths

Stop Conditions

Required Skills

Required Design References

Scope Delta（任意）

CI_TEST_PERFORMANCE_DECISION_V1

Delivery Rule

Activity

squne121 commented on Jul 12, 2026

総合判定

重大な指摘

P0-1: Allowed Paths が正しい修正を禁止している

必須修正

P0-2: 原因を「暗黙更新」と表現しており、既知の action v6 不具合を正確に捉えていない

必須修正

P0-3: AC2 は実行された pnpm のバージョンを検証していない

AC2 の修正版

高優先度の指摘

P1-1: AC3 の contract test が実装詳細の固定に寄りすぎている

contract test が検証すべき意味論

P1-2: baseline test が有効な Red test になっていない

修正方法

P1-3: gh pr checks は current-head 証跡として曖昧

修正版

P1-4: deploy-main の runtime verification が欠落している

現実的な整理

P1-5: version: 11 と「決定論的」が矛盾する

必須条件

中優先度の指摘

P2-1: Node と runner は完全には固定されていない

P2-2: action の更新方針がない

P2-3: packageManager 追加時の scope delta が Stop Condition と衝突する

推奨実装方針

修正版 Acceptance Criteria

Allowed Paths 修正版

最終評価

squne121 commented on Jul 12, 2026

Contract Review Result

squne121 commented on Jul 12, 2026

Contract Review Result (serialization repair)

squne121 commented on Jul 12, 2026

Refinement Loop: Approved

squne121 commented on Jul 12, 2026

impl-review-loop 停止報告

squne121 commented on Jul 12, 2026

Scope Collision Preflight override

squne121 commented on Jul 12, 2026

Contract Review Result（再実行）

squne121 commented on Jul 12, 2026

implement-issue: overlap preflight が human_review_required を返したため実装を停止

根拠

停止理由

推奨対応（人間判断）

squne121 commented on Jul 12, 2026

implement-issue: overlap preflight の human_review_required は人間レビューで解消済み

squne121 commented on Jul 12, 2026

implement-issue: 実装は完了・PR 作成のみ open-pr の Overlap Preflight Hard Gate でブロック

検証結果

ブロック理由

推奨対応（人間判断）

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

P0-1: `Allowed Paths` が正しい修正を禁止している

P1-3: `gh pr checks` は current-head 証跡として曖昧

P1-4: `deploy-main` の runtime verification が欠落している

P1-5: `version: 11` と「決定論的」が矛盾する

P2-3: `packageManager` 追加時の scope delta が Stop Condition と衝突する