Skip to content

test(phase-b): スモーク段 2 用の観測装置 — マージせずクローズする一時 PR (ADR-067) #355

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
aloekun wants to merge 3 commits into from
Closed

test(phase-b): スモーク段 2 用の観測装置 — マージせずクローズする一時 PR (ADR-067) #355

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6c5b1fa
test(phase-b): スモーク段 2 用の観測装置となる docs を追加 (ADR-067)
aloekun Aug 4, 2026
ee9557a
test(phase-b): 観測装置の宣言から fix agent の編集抑止要因を除去 (ADR-067 残課題 4)
aloekun Aug 4, 2026
86f424c
docs: PR #355 のレビュー指摘に対する無人 fix (pr-monitor Phase B)
github-actions[bot] Aug 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 36 additions & 0 deletions docs/phase-b-smoke-test.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
# Phase B 実走スモークテスト用の一時ドキュメント

> **このファイルは削除前提の検証用アーティファクトです。** ADR-067 (Phase B 無人 fix push) の
> スモーク段 2 で、CodeRabbit の docs 指摘 → fix agent の編集 → 4 軸ゲート → workflow による
> push、という allow 経路を実走観測するために作成しました。観測完了後、この PR はマージせず
> クローズし、ブランチごと削除します。プロダクションの手順書として参照しないでください。

## このテストで観測すること

Phase B の fix job が `claude/` prefix ブランチに対して起動し、以下を通過することを確認します。

1. `Decide whether Phase B applies` で `proceed=true` になる
2. CodeRabbit の指摘が決定論的な著者フィルタを通って findings になる
3. fix agent が `pr/docs/**` の範囲で編集する
4. `cli-fix-push-gate` の 4 軸 AND がすべて満たされ exit 0 になる

以上の 4 ステップを順に観測します。

## 実行手順

Actions タブから pr-monitor workflow を手動起動します。ref は master を選び、`pr_number` に
この PR の番号を入れてください。

なお `AUTONOMY_ENABLED` が `true` でなければ fix job は起動しないため、事前設定は必須です。
Comment on lines +21 to +24

@coderabbitai coderabbitai Bot Aug 4, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

AUTONOMY_ENABLED の設定条件を矛盾なく記載してください。

Line 32 は variable の設定を任意と記載しています。Line 34 は AUTONOMY_ENABLED=true が必須と記載しています。.github/workflows/pr-monitor.yml の fix.if は vars.AUTONOMY_ENABLED == 'true' を要求します。Line 32 に従うと fix job は起動しません。

修正案 
-この PR の番号を入れてください。variable の設定は任意です。
+この PR の番号を入れてください。Actions variable `AUTONOMY_ENABLED` を `true` に設定してください。
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Actions タブから pr-monitor workflow を手動起動します。ref は master を選び、`pr_number` に
この PR の番号を入れてください。variable の設定は任意です。
なお `AUTONOMY_ENABLED` が `true` でなければ fix job は起動しないため、事前設定は必須です。
Actions タブから pr-monitor workflow を手動起動します。ref は master を選び、`pr_number` に
この PR の番号を入れてください。Actions variable `AUTONOMY_ENABLED` を `true` に設定してください。
なお `AUTONOMY_ENABLED` が `true` でなければ fix job は起動しないため、事前設定は必須です。
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/phase-b-smoke-test.md` around lines 31 - 34, Update the pr-monitor
workflow instructions so variable configuration is not described as optional:
explicitly require setting the AUTONOMY_ENABLED variable to true before manually
starting the workflow, while preserving the existing ref and pr_number guidance.


## 観測後の後始末

- この PR をクローズする
- ブランチを削除する
- TODO: 観測結果はこの PR のコメントに記録する。上記 4 ステップすべての通過を確認できた時点で完了とする

## 補足

段 1 では非 `claude/` ブランチに対して prefix 層の deny を確認済みです。段 2 はその対になる
allow 経路の確認にあたります。ゲートの 4 軸の詳細は [ADR-067](adr/adr-067-phase-b-unattended-fix-push.md)
を参照してください。