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 1 commit
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
46 changes: 46 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,46 @@
# Phase B 実走スモークテスト用の一時ドキュメント

> **このファイルは削除前提の検証用アーティファクトです。** ADR-067 (Phase B 無人 fix push) の
> スモーク段 2 で、CodeRabbit の docs 指摘 → fix agent の編集 → 4 軸ゲート → workflow による
> push、という allow 経路を実走観測するために作成しました。観測完了後、この PR はマージせず
> クローズし、ブランチごと削除します。プロダクションの手順書として参照しないでください。
>
> **本文には意図的な不整合を 3 点含めてあります** (ステップ数の不一致 / variable 設定が任意と
> 必須で矛盾 / 判断基準のない TODO)。これは CodeRabbit に docs 指摘を出させ、Phase B の
> findings → fix → gate → push 経路を実走させるための**観測装置**です。
>
> **補足（人間のレビュアー向け）**: 本文中の不整合は Phase B スモークテスト (ADR-067 段 2) の
> 観測装置として意図的に埋め込んだものです。観測対象は post-PR の CodeRabbit → Phase B fix job
> 経路であり、pre-push レビュー/fix パイプラインがこれらを検出・修正しても観測の成立を妨げま
> せん。段 2 の観測完了後にブランチごと破棄するため、修正の有無にかかわらず master に不整合が
> 残ることはありません。

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

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 になる

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

@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

ステップ数の記載を 4 に統一してください。

Lines 22-25 は 4 項目を列挙しています。Line 27 は 3 ステップと記載しています。観測結果の解釈が不明確になります。

修正案 
-以上の 3 ステップを順に観測します。
+以上の 4 ステップを順に観測します。
📝 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
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 になる
以上の 3 ステップを順に観測します。
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 ステップを順に観測します。
🤖 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 22 - 27, docs/phase-b-smoke-test.md
の観測説明で、列挙されている 4 項目と一致するよう「3 ステップ」の記載を「4 ステップ」に更新してください。


## 実行手順

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

なお `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: 観測結果の記録先を決める

@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.

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

TODO に記録先と完了条件を追加してください。

現在の TODO は「記録先を決める」とだけ記載しています。どこに記録するか、何を記録したら完了とするかが不明です。PR とブランチを削除する前に、記録先と完了条件を明記してください。

🤖 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` at line 40, Update the TODO in the phase-B smoke
test documentation to specify the exact location where observation results must
be recorded and the concrete criteria that define completion, ensuring both are
documented before deleting the PR and branch.


## 補足

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