@shinaps/skills 0.34.0 → 0.34.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +7 -7
- package/package.json +1 -1
- package/plugins/shinaps-claude/.claude-plugin/plugin.json +1 -1
- package/plugins/shinaps-claude/agents/shinaps-split-reviewer.md +4 -4
- package/plugins/shinaps-claude/references/dev/workflow.md +1 -1
- package/plugins/shinaps-claude/references/human-review.md +25 -16
- package/plugins/shinaps-claude/references/linear/bridge.md +6 -6
- package/plugins/shinaps-claude/references/linear/configuration.md +7 -7
- package/plugins/shinaps-claude/references/linear/execution.md +1 -1
- package/plugins/shinaps-claude/references/linear/workflow.md +9 -9
- package/plugins/shinaps-claude/skills/commit/SKILL.md +7 -7
- package/plugins/shinaps-claude/skills/linear-execute/SKILL.md +4 -4
- package/plugins/shinaps-codex/.codex-plugin/plugin.json +1 -1
- package/plugins/shinaps-codex/references/dev/workflow.md +1 -1
- package/plugins/shinaps-codex/references/human-review.md +25 -16
- package/plugins/shinaps-codex/references/linear/bridge.md +6 -6
- package/plugins/shinaps-codex/references/linear/configuration.md +7 -7
- package/plugins/shinaps-codex/references/linear/execution.md +1 -1
- package/plugins/shinaps-codex/references/linear/workflow.md +9 -9
- package/plugins/shinaps-codex/skills/commit/SKILL.md +7 -7
- package/plugins/shinaps-codex/skills/linear-execute/SKILL.md +3 -3
package/README.md
CHANGED
|
@@ -16,10 +16,10 @@ sharedで採用する `instructions.md` は共有JSONの `references.specificati
|
|
|
16
16
|
|
|
17
17
|
0.33.0から `.shinaps/linear.json`（`config_version: 2`）を使います。`config_version: 1`・`implementation_skill`・`review_flow.tool`・許可の `codex_task`/`managed_session:*`、旧YAML・scalar review_flowは受け付けず、[移行手順](plugins/shinaps-claude/references/linear/configuration.md#config_version-1-からの移行)に従います。
|
|
18
18
|
|
|
19
|
-
CLIはpluginに含まれないため、利用中のskillと同じ `@shinaps/skills@0.34.
|
|
19
|
+
CLIはpluginに含まれないため、利用中のskillと同じ `@shinaps/skills@0.34.1` を導入し、`shinaps-skills --version` で確認します。`shinaps-skills validate-linear-config <明示ファイル>` は同梱schemaだけで検証し、ネットワーク取得・正本探索・Git操作・入力変更は行いません。`$schema` は固定URLによるエディタ補助で、契約の正本は[同梱schema](plugins/shinaps-claude/references/linear/configuration.schema.json)です。0.34.0から、Managed Agentsの `.shinaps/managed.json`（`config_version: 1`）も `shinaps-skills validate-managed-config <明示ファイル>` で同じ方式で検証します（[Managedの設定](plugins/shinaps-claude/references/managed/configuration.md)）。
|
|
20
20
|
|
|
21
21
|
```sh
|
|
22
|
-
npm install -g @shinaps/skills@0.34.
|
|
22
|
+
npm install -g @shinaps/skills@0.34.1
|
|
23
23
|
shinaps-skills --version
|
|
24
24
|
shinaps-skills validate-linear-config /absolute/repository/.shinaps/linear.json
|
|
25
25
|
shinaps-skills validate-managed-config /absolute/repository/.shinaps/managed.json
|
|
@@ -37,7 +37,7 @@ shinaps-skills validate-managed-config /absolute/repository/.shinaps/managed.jso
|
|
|
37
37
|
| `refine` | Shinaps AIのruleと実ファイルを照合する |
|
|
38
38
|
| `extract-rules` | PRレビューからruleを抽出する |
|
|
39
39
|
| `simplify` | 目的から見直して不要な構造や手順を削る |
|
|
40
|
-
| `commit` | 各単位のreview_flowに従い、autoはAIがレビュー・コミット、manual
|
|
40
|
+
| `commit` | 各単位のreview_flowに従い、autoはAIがレビュー・コミット、manualは人の開始依頼で分割案を示し、人が単位ごとに依頼したコミットを行う |
|
|
41
41
|
| `create-pr` | リポジトリの規約に合わせてPRを作成する |
|
|
42
42
|
| `release` | 公開条件を確定し、対象バージョンの利用可能性を確認する |
|
|
43
43
|
| `linear-plan` | 相談・必要なspec/explore・Issue作成と整理・プロジェクト設定を行い、実装前で止める |
|
|
@@ -47,9 +47,9 @@ shinaps-skills validate-managed-config /absolute/repository/.shinaps/managed.jso
|
|
|
47
47
|
|
|
48
48
|
コミット・PR・リリースは、利用者の依頼またはLinear進行で合意済みの継続許可に従う。[共通設定](plugins/shinaps-claude/references/linear/configuration.md)・[担当と工程](plugins/shinaps-claude/references/linear/workflow.md)・[終了と待機](plugins/shinaps-claude/references/linear/execution.md)を参照し、必要な前工程が揃えば再確認せずに進む。単独スキル利用にはLinear進行や新しいゴールを必須にしない。`autonomous-commit-review` の役割は `commit` に統合した。個人用の別AI呼び出し・ブラウザ管理・ゴール管理・動画制作はこのpluginに含めない。
|
|
49
49
|
|
|
50
|
-
`linear-execute` は初版では利用者本人の環境向けで、Codexが進行役として進行・Agent QA・Issueの記録・人への窓口を担当し、Claude Codeが実装・修正・コミットを行う。主フローは着手・実装・差分確定・公開確認・結果報告の5工程。運用項目は明示必須で、未記載を既定値で補わない。経路は着手時に通常の `review_flow.mode` で決まる。autoはCodexがローカル `claude -p` を起動して同じsessionをresumeし（許可 `claude_cli`、[CLI手順](plugins/shinaps-claude/references/linear/claude-cli.md)）、CLIの既存権限・model・effort・pluginsを保持する。manualはWebStorm内の対話Claude CodeセッションへBridgeで依頼する（許可 `bridge_request`、[Bridge](plugins/shinaps-claude/references/linear/bridge.md)
|
|
50
|
+
`linear-execute` は初版では利用者本人の環境向けで、Codexが進行役として進行・Agent QA・Issueの記録・人への窓口を担当し、Claude Codeが実装・修正・コミットを行う。主フローは着手・実装・差分確定・公開確認・結果報告の5工程。運用項目は明示必須で、未記載を既定値で補わない。経路は着手時に通常の `review_flow.mode` で決まる。autoはCodexがローカル `claude -p` を起動して同じsessionをresumeし（許可 `claude_cli`、[CLI手順](plugins/shinaps-claude/references/linear/claude-cli.md)）、CLIの既存権限・model・effort・pluginsを保持する。manualはWebStorm内の対話Claude CodeセッションへBridgeで依頼する（許可 `bridge_request`、[Bridge](plugins/shinaps-claude/references/linear/bridge.md)）。manualではQA後もCodexはcommitを依頼せず、人の依頼でClaudeがcommitする。初期体制はClaudeのメインが決めて報告し、Codexは判断の材料を渡す。Bridge本体は同梱せず、導入済みのMCPとWebStormプラグインを使う。`SKILL.md` は担当ごとに両版で書き分け、共通の手順と書式は `references/linear/` に置く。`references/linear/` はClaude版を編集元としてCodex版へ同じ内容を複製し、テストで一致を確認する。
|
|
51
51
|
|
|
52
|
-
`review_flow: { "mode": "auto" }` はAIが自然な単位を決め、独立レビュー・QA後にstage・commitする。`review_flow: { "mode": "manual" }` はAIが単位外を安全に退避して単位をstage
|
|
52
|
+
`review_flow: { "mode": "auto" }` はAIが自然な単位を決め、独立レビュー・QA後にstage・commitする。`review_flow: { "mode": "manual" }` はAIがレビュー待ちを明示し、人がWebStormで差分を確認する。人のコミット開始依頼でAIが分割案を示し、人がその単位のコミットを依頼したらAIが単位外を安全に退避して単位をstage・検証・commitする。両方のstage・AI commit権限は利用者との明示合意に含まれ、未合意ファイルからは得ない。`path_policy_overrides` は `paths` と `review_flow` だけを持つ。全変更が一つのルールで覆われる単位だけ適用し、改名は旧新両パスを判定する。競合時は止める。旧 `human_code_review`・`commit_mode`・permissionsのstage/commit・パス別ai_commitは明示移行が必要。[共通設定](plugins/shinaps-claude/references/linear/configuration.md#パス別の方針)を参照する。
|
|
53
53
|
|
|
54
54
|
0.29.0では旧 `document_only_changes: ai_commit` を廃止した。利用先では依存工程を止め、具体的なパス例外への移行または例外廃止を合意する。説明だけの変更の許可とは同値でないため自動変換しない。旧 `none` もJSONへ明示移行し、新規設定は `path_policy_overrides: []` にする。
|
|
55
55
|
|
|
@@ -178,9 +178,9 @@ npm pack --dry-run
|
|
|
178
178
|
|
|
179
179
|
公開前にはnpmスコープ`@shinaps`の利用権限とtarball内容を確認する。CLIのバージョンと両pluginのバージョンを揃える。開発中の同一バージョンも`update codex`で再配置できる。
|
|
180
180
|
|
|
181
|
-
### manual
|
|
181
|
+
### manualのレビュー待ちとコミットの依頼
|
|
182
182
|
|
|
183
|
-
manualではWebStorm内の対話Claude Code
|
|
183
|
+
manualではWebStorm内の対話Claude Codeセッションで、作業が終わるとClaudeが短く報告して「ユーザーのレビュー待ち」と明示します。人はWebStormの差分表示で全体を確認し、Claudeと修正を相談します。人がコミットの開始を依頼すると、Claudeが自然なコミット分割案を示します。分割案への合意はコミットの承認ではありません。人がその単位のコミットを依頼すると、Claudeが単位外を退避して単位をstage・検証し、stage時のtreeと一致することを確かめてコミットします。stage時のtreeを記録した後に単位の差分が変われば再提示と改めての依頼が必要です。会話には差分本文を貼らず、単位名・目的・検証結果・推奨メッセージなど必要最小限だけを示します。手順は[共通手順](plugins/shinaps-claude/references/human-review.md)に従います。autoには人レビュー待ちを追加しません。
|
|
184
184
|
|
|
185
185
|
### CIとnpm自動公開
|
|
186
186
|
|
package/package.json
CHANGED
|
@@ -1,13 +1,13 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: shinaps-split-reviewer
|
|
3
|
-
description: shinaps:commit のmanual
|
|
3
|
+
description: shinaps:commit のmanualで、人が単位ごとにコミットを依頼する前に分割計画を確認するレビューエージェント。メインスレッドが提案したコミット単位の分割を、各単位の diff だけを読んで変更の妥当性を判断できるか (下限) / 一言で説明できる範囲か (上限) / 依存順 / 網羅性の観点で批判的にレビューし、指摘を反映した修正版最終分割計画までを返す。worktree は read-only で扱い、stash / add / checkout / Edit は一切行わない
|
|
4
4
|
model: opus
|
|
5
5
|
effort: medium
|
|
6
6
|
color: blue
|
|
7
7
|
---
|
|
8
8
|
|
|
9
|
-
あなたは `/shinaps:commit` のmanual
|
|
10
|
-
メインスレッドが提案した **コミット単位の分割計画** を、「この単位の diff だけを渡された人間レビュアーが、変更の妥当性を判断できるか」という視点で **批判的に検証** し、Critical / Warning / Info の指摘を出したうえで、**指摘を反映した修正版最終分割計画 (Part 2)**
|
|
9
|
+
あなたは `/shinaps:commit` のmanualで、人が単位ごとにコミットを依頼する前に分割計画を確認するレビューエージェントです。
|
|
10
|
+
メインスレッドが提案した **コミット単位の分割計画** を、「この単位の diff だけを渡された人間レビュアーが、変更の妥当性を判断できるか」という視点で **批判的に検証** し、Critical / Warning / Info の指摘を出したうえで、**指摘を反映した修正版最終分割計画 (Part 2)** までを返してください。人が単位ごとにコミットを依頼する前に区切り方を確かめる立場なので（人のレビュー・コミットの依頼は代行しない）、「概ね良い」で済ませずに各単位を実際に読んで判定します。
|
|
11
11
|
|
|
12
12
|
## 作業前の必須確認
|
|
13
13
|
|
|
@@ -55,7 +55,7 @@ color: blue
|
|
|
55
55
|
|
|
56
56
|
- hunk 分割する単位は「対象ファイル」に hunk の範囲 (関数名や行範囲) を明記する
|
|
57
57
|
- 150 行を超える単位は「補足」にまとめる理由を書く
|
|
58
|
-
- 差し戻しの未解決単位は「補足」に「未解決:
|
|
58
|
+
- 差し戻しの未解決単位は「補足」に「未解決: <理由>」を書く。メインスレッドはこれを分割案とともに人へ提示し、人が判断材料の追加やスキップを決める
|
|
59
59
|
|
|
60
60
|
## 動作原則
|
|
61
61
|
|
|
@@ -105,7 +105,7 @@ Issue・既存spec・調査記録・関連コード・検証方法を必要な
|
|
|
105
105
|
|
|
106
106
|
初期体制と変更理由、実際の担当・確認範囲、計画と実装後のレビュー回数、採否・修正、検証結果、未実施・未照合・未解決事項を報告する。
|
|
107
107
|
|
|
108
|
-
linear-executeから受けた作業は未コミットの差分と結果を進行役へ返す。devの完了はLocal Agent QAへの引き渡しであり、進行全体の終了ではない。[Linear共通設定](../linear/configuration.md)と[担当と工程](../linear/workflow.md)に従い、QA
|
|
108
|
+
linear-executeから受けた作業は未コミットの差分と結果を進行役へ返す。devの完了はLocal Agent QAへの引き渡しであり、進行全体の終了ではない。[Linear共通設定](../linear/configuration.md)と[担当と工程](../linear/workflow.md)に従い、autoはQA後の差分確定の依頼を受けて、manualは人のコミット開始依頼で `commit` を始め、単位の退避・stage・commitはQA結果の引き渡し後に行う。auto経路は [Claude CLI](../linear/claude-cli.md) のresultへ報告し、同じsessionのresumeで質問・修正を受ける。manual経路は同じ対話セッションで人と作業し、[Bridge](../linear/bridge.md)で依頼元Codexへ報告する。manualの完了時は人へユーザーのレビュー待ちと明示する（[manualの手順](../human-review.md)）。
|
|
109
109
|
|
|
110
110
|
単独のdevではコミットが許可されている場合だけ範囲とモードを `commit` へ渡す。許可がなければ未コミットで返し、形式的な許可確認を追加しない。PR・リリースも許可された範囲で各スキルへ渡す。単独devにLinear進行・新しいゴールを必須にしない。
|
|
111
111
|
|
|
@@ -1,26 +1,35 @@
|
|
|
1
|
-
# manual
|
|
1
|
+
# manualのレビュー待ちと単位ごとのコミット
|
|
2
2
|
|
|
3
|
-
実効 `review_flow.mode: manual`
|
|
3
|
+
実効 `review_flow.mode: manual` の差分を、人がWebStormの差分表示でレビューし、人の依頼に従ってWebStorm内の対話Claude Codeセッションで単位ごとにClaudeがcommitする手順。review_flow・操作許可は、プロジェクト設定と今回の依頼に従う。この手順はそれらを変更しない。
|
|
4
4
|
|
|
5
|
-
|
|
5
|
+
会話では単位名・目的・対象ファイル・検証結果・短い推奨コミットメッセージなど判断に必要な最小限を示し、差分本文を貼らない。Claude自身の差分の読取りと独立レビューは制限しない。
|
|
6
6
|
|
|
7
|
-
|
|
8
|
-
2. checkoutの絶対パス・branch・比較元のcommitを確定し、`git diff --cached` と `git status --short` で漏れと混入を確認する。未追跡ファイルも対象に含める。
|
|
9
|
-
3. 会話で単位の差分・目的・確認してほしい点・検証結果・推奨コミットメッセージを提示する。Linear進行ではIssueも示す。人はWebStormの差分表示で同じstaged差分を確認できる。
|
|
10
|
-
4. 提示時に `git write-tree` の値を記録する。これが承認の対象になる。
|
|
7
|
+
## レビュー待ち
|
|
11
8
|
|
|
12
|
-
|
|
9
|
+
devと必要な検証が終わったら、作業内容と検証結果を短く報告し、「ユーザーのレビュー待ち」と明示する。人はWebStormの差分表示で全体を確認し、Claudeと相談する。
|
|
10
|
+
|
|
11
|
+
人がコミットの開始を依頼するまで、分割案の提示・単位外の退避・単位のstage・単位のコミット依頼の要求を始めない。Linear進行でCodexへの完了報告に含める単位案と予定差分はQAの材料であり、人への分割案の提示ではない。Linear進行では、分割案の提示は人の開始依頼だけで行え、単位の退避・stage・commitはLocal Agent QAの合格とQA結果・判定記録の引き渡しの後に行う（両者の順序は問わない）。
|
|
12
|
+
|
|
13
|
+
## コミットの開始と分割案
|
|
14
|
+
|
|
15
|
+
人がコミットの開始を依頼したら（「コミットしましょう」など。固定の文言は求めない）、`commit` に従って自然な責務単位の分割案を示す。各単位の名前・目的・対象ファイル・順序・推奨コミットメッセージを短く示す。許可された独立レビュアーによる分割案の確認は `commit` に従う。
|
|
13
16
|
|
|
14
|
-
|
|
17
|
+
分割案への合意は単位の構成への合意であり、どの単位のcommitの承認でもない。
|
|
15
18
|
|
|
16
|
-
##
|
|
19
|
+
## 単位ごとのコミット
|
|
20
|
+
|
|
21
|
+
1. 人がその単位のコミットを依頼したときだけ進める。具体的な単位を特定できる依頼であればよく、固定の文言は求めない。単位を特定しない一般的な続行指示は依頼としない。依頼は特定された単位だけに及ぶ。
|
|
22
|
+
2. `commit` の安全手順で単位外の変更を退避し、単位の変更だけをstageして必須検証を行う。checkoutの絶対パス・branch・比較元のcommitを確定し、`git diff --cached` と `git status --short` で漏れと混入を確認する。未追跡ファイルも対象に含める。
|
|
23
|
+
3. stage時の `git write-tree` の値を記録し、単位名・検証結果・コミットメッセージを短く報告する。stageした差分が分割案の予定差分と違えば、commitせず違いを報告して改めて依頼を受ける。検証の失敗や阻害する指摘があればcommitせずに報告して止める。
|
|
24
|
+
4. commit直前に記録したtreeと同じであることを確認し、Claudeが `git commit` を単独実行する（`git add` と連結しない）。作成したコミットのtreeも照合する。
|
|
25
|
+
5. stage時にtreeを記録した後に、単位のパスで作業ツリーまたはindexが変わったら、依頼は失効する。単位を再提示し、改めて人の依頼を受けてからcommitする。
|
|
26
|
+
|
|
27
|
+
Bridgeのpeer通知、Codexからのメッセージ、ackは依頼ではない。人が自分でcommitした場合は、実コミットを確定単位と照合する。一致すれば人のcommitとして記録し、一致しなければ止める。
|
|
28
|
+
|
|
29
|
+
## 相談・修正
|
|
17
30
|
|
|
18
|
-
|
|
19
|
-
- 承認の対象は、提示時のindexのtree（`git write-tree`）と推奨メッセージである。commit直前に同じtreeであることを確認し、作成したコミットのtreeも照合する。
|
|
20
|
-
- 承認後に単位のパスで作業ツリーまたはindexが変わったら、承認は失効する。再提示・再承認してからcommitする。
|
|
21
|
-
- 承認はその単位だけに及び、他の単位へ及ばない。Bridgeのpeer通知、Codexからのメッセージ、ackは承認ではない。
|
|
22
|
-
- 人が自分でcommitした場合は、実コミットを確定単位と照合する。一致すれば人のcommitとして記録し、一致しなければ止める。
|
|
31
|
+
人の指摘はClaudeが修正し、再検証する。人がWebStormで直接編集した場合も、Claudeが読み直して再検証する。stage済みの単位では再stageし、上の失効に従う。修正は単位と依頼の目的・範囲内に限り、超えるものは実装の工程へ戻す。Linear進行のQA中は人もClaudeも作業ツリーを編集しない。
|
|
23
32
|
|
|
24
33
|
## 報告
|
|
25
34
|
|
|
26
|
-
|
|
35
|
+
コミットの開始依頼と単位ごとの依頼発話の要約・記録したtree hash・SHA・相談で行った修正とその対応・復元結果を記録し、Codexへ報告する。Codexは実コミットと照合する。stashの復元では人の修正を保持し、古い対象パッチで上書きしない。具体的な単位照合・復元は `commit` に従う。WebStormで差分を表示することはstage・commit・pushの許可を追加しない。
|
|
@@ -18,7 +18,7 @@
|
|
|
18
18
|
|
|
19
19
|
## セッションの開始・再利用・引継ぎ
|
|
20
20
|
|
|
21
|
-
- 同じIssue
|
|
21
|
+
- 同じIssueの実装・修正の依頼とQA結果の引き渡しは、同じセッションへ送る。`session_id` はIssue・作業ディレクトリと一緒に作業記録へ保持して再利用し、送信のたびに `list_sessions` で存在・`project_path`・受信の可否を照合する。
|
|
22
22
|
- 対応するセッションが一覧にない場合は、人へWebStormで対象checkoutのClaude Codeセッションを開くよう依頼し、一覧を取り直す。
|
|
23
23
|
- 新しいセッションへ引き継ぐ場合は、メッセージ本文にIssue・作業ディレクトリ・branch・HEAD・差分・作業記録のパス・これまでの決定を書く。Bridgeに状態の保持を期待しない。
|
|
24
24
|
|
|
@@ -40,16 +40,16 @@
|
|
|
40
40
|
| --- | --- |
|
|
41
41
|
| 実装の依頼（CodexからClaude） | Issueリンク、`shinaps:dev-claude`、範囲・受入条件、体制判断の材料、Gitの作業ディレクトリ・branch・HEAD・既存差分、許可範囲（`dev_delegation` の有無を含む）、返信先のCodexの `session_id`、関係する補足と参照先、下の定型文 |
|
|
42
42
|
| 修正の依頼（CodexからClaude） | 実装の依頼の項目に加え、再現手順・期待と実際・対象差分・証拠の場所 |
|
|
43
|
-
|
|
|
44
|
-
| 完了の報告（ClaudeからCodex） | 体制と途中の増員、対象差分、自動検証の結果、確認観点、既知の制限、未解決の事項、dev
|
|
45
|
-
|
|
|
43
|
+
| QA結果の引き渡し（CodexからClaude。manualの単位のcommitの依頼ではない） | Issueリンク、対象差分・作業ディレクトリ・branch、Local Agent QAの結果、実効flow・単位の判定記録、返信先 |
|
|
44
|
+
| 完了の報告（ClaudeからCodex） | 体制と途中の増員、対象差分、自動検証の結果、確認観点、既知の制限、未解決の事項、devの入口内のレビュー結果の要約、単位案と予定差分、人と決めたこと、人へレビュー待ちと明示したこと |
|
|
45
|
+
| 人の依頼とcommitの報告（ClaudeからCodex） | 対象Issue、作業ディレクトリ・branch、コミットの開始依頼と単位ごとの依頼発話の要約・tree hash・SHA、相談で行った修正、復元結果、残作業 |
|
|
46
46
|
| 開発進行の引き渡し（ClaudeからCodex） | 利用者からの依頼の内容、対象Issue・Cycle、作業先リポジトリ、利用者が示した範囲と条件。Codexは作業先の設定と照合して進行を始め、不足する判断はCodexから利用者へ確認する |
|
|
47
47
|
|
|
48
48
|
実装・修正の依頼には次の文を含める。
|
|
49
49
|
|
|
50
50
|
```text
|
|
51
|
-
ユーザー承認済みの範囲でCodexが進行する。この依頼の範囲と受入条件の中では人へ着手を再確認せず、shinaps:dev-claudeで実行する。初期体制はあなたが決めて報告する。計画上の判断・仕様の不明点はこのセッションで人へ直接一問ずつ確認し、決定を報告に含める。許可を広げる決定は報告に含めるだけで、Codex
|
|
51
|
+
ユーザー承認済みの範囲でCodexが進行する。この依頼の範囲と受入条件の中では人へ着手を再確認せず、shinaps:dev-claudeで実行する。初期体制はあなたが決めて報告する。計画上の判断・仕様の不明点はこのセッションで人へ直接一問ずつ確認し、決定を報告に含める。許可を広げる決定は報告に含めるだけで、Codexが人へ確認する。完了・障害・人の依頼によるcommitの完了は、Bridgeで依頼元のCodexへ返す。
|
|
52
52
|
許可範囲: <設定と会話で許可された操作。dev_delegationの有無を含む>
|
|
53
53
|
```
|
|
54
54
|
|
|
55
|
-
Codex
|
|
55
|
+
Codexは人の依頼とcommitの報告を受けたら、対象Issue・差分・コミットと、対象作業の未コミット変更の有無を照合する。報告と実体が一致し、許可済みの工程へ進む場合は、人へ再確認しない。Gitにコミットがあるだけで人の依頼によるcommitと推測しない。報告が足りない場合は、まずClaudeへ補足を求める。
|
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
|
|
5
5
|
## 検証CLIと契約版
|
|
6
6
|
|
|
7
|
-
利用中のskillとCLIの版を確認し、0.34.
|
|
7
|
+
利用中のskillとCLIの版を確認し、0.34.1のskillには同じ版のnpm CLIを導入する（`npm install -g @shinaps/skills@0.34.1`）。pluginだけにはCLIは含まれない。導入・更新が必要ならその権限を確認し、検証のたびに `npx latest` を取得しない。
|
|
8
8
|
|
|
9
9
|
```sh
|
|
10
10
|
shinaps-skills --version
|
|
@@ -76,10 +76,10 @@ shinaps-skills validate-linear-config /absolute/repository/.shinaps/linear.json
|
|
|
76
76
|
| `permissions` | 継続的に許可した操作の一覧（下の「許可」）。経路の許可（auto: `claude_cli`、manual: `bridge_request`）も含む。許可なしは `[]` |
|
|
77
77
|
| `references` | `specifications`・`validation`・`environments`・`deployment`・`publication` の5分類をすべて記載し、既存資料のパスまたはURLの一覧を指定する。該当資料なしは `[]` |
|
|
78
78
|
| `cycle_report` | Cycle報告の設定（下の「Cycle報告」）。報告を使わない場合は `null` |
|
|
79
|
-
| `review_flow` | `{ "mode": "auto" }`（AIが自然な単位を決め、独立レビュー・QA後にstage・commit）または `{ "mode": "manual" }`（AI
|
|
79
|
+
| `review_flow` | `{ "mode": "auto" }`（AIが自然な単位を決め、独立レビュー・QA後にstage・commit）または `{ "mode": "manual" }`（AIがレビュー待ちを明示し、人がWebStormで差分を確認する。人のコミット開始依頼でAIが分割案を示し、人が単位ごとに依頼したらAIがstage・commit）。開始時の値で経路も決まる（下の「経路」） |
|
|
80
80
|
| `path_policy_overrides` | パス別に `review_flow` を上書きするルールの一覧。例外なしは `[]`（下の「パス別の方針」） |
|
|
81
81
|
|
|
82
|
-
- 利用者と合意した `review_flow` は両方とも対象のstageとAIのcommitを許可する。`auto` は独立レビュー後、`manual`
|
|
82
|
+
- 利用者と合意した `review_flow` は両方とも対象のstageとAIのcommitを許可する。`auto` は独立レビュー後、`manual` は人がその単位のコミットを依頼した後にcommitする。未合意の設定ファイルから許可を得ない。その場の明示指示が優先し、push・公開・履歴変更は別の許可に従う。
|
|
83
83
|
- どちらのflowもdev内の確認、必須検証、Agent QA、必要なHuman QAを維持する。`auto` は独立レビューを必須とし、利用できなければ自己確認で代用せず止める。人レビュー省略を人のレビュー済みと記録しない。
|
|
84
84
|
- 状態キーは [workflow.md](workflow.md) の「工程とステータス」の表と同じ。`statuses` の値はチームに実在する正確な名前にする。操作時はLinearから取得した状態のIDを使う。対応が曖昧・欠落・実物と不一致なら、その対応に依存する更新を止める。ステータスを自動で新設しない。
|
|
85
85
|
- `team_settings: manage_on_setup` の場合も、通常実行ではチーム設定を変更しない。
|
|
@@ -94,8 +94,8 @@ shinaps-skills validate-linear-config /absolute/repository/.shinaps/linear.json
|
|
|
94
94
|
| `manual` | WebStorm内の対話Claude CodeセッションへBridgeで依頼する（[Bridge](bridge.md)） | `bridge_request` |
|
|
95
95
|
|
|
96
96
|
- 経路はIssueの途中で切り替えない。`permissions`・`stop_after`・`dev_delegation`・worktreeや別セッションの作成の許可は、経路によって変わらない。
|
|
97
|
-
- manual経路でパス別の方針によりauto
|
|
98
|
-
- auto経路でmanual単位が生じた場合、非対話CLI
|
|
97
|
+
- manual経路でパス別の方針によりauto単位が生じた場合は、人のコミット開始依頼の後、対話session内のClaudeがその単位を単位ごとの依頼なしに独立レビュー後にcommitし、人のレビューの省略を記録する。
|
|
98
|
+
- auto経路でmanual単位が生じた場合、非対話CLIは人との相談・コミットの依頼を扱えない。CodexはLocal Agent QAで判定記録を照合した時点で検出し、差分確定の依頼前に人へ一問で確認する: 「今回の明示指示でその単位をautoとして進める」か「Issueを中止し、未コミット差分を保全したまま設定（通常設定またはルール）を見直してから、manual経路でやり直す」か。黙ってautoへ落とさない。`commit` の中で検出した場合は手順を進めず、判定を記録してresultへ返す。
|
|
99
99
|
|
|
100
100
|
## パス別の方針
|
|
101
101
|
|
|
@@ -125,9 +125,9 @@ shinaps-skills validate-linear-config /absolute/repository/.shinaps/linear.json
|
|
|
125
125
|
|
|
126
126
|
### 判定記録と変更時の扱い
|
|
127
127
|
|
|
128
|
-
各予定単位について、設定の正本と内容、適用した全ルール（または適用なしの理由）、単位の比較元と全差分、実効 `review_flow`、今回の明示指示とflowの合意根拠、検証・レビューの結果を記録する。manual
|
|
128
|
+
各予定単位について、設定の正本と内容、適用した全ルール（または適用なしの理由）、単位の比較元と全差分、実効 `review_flow`、今回の明示指示とflowの合意根拠、検証・レビューの結果を記録する。manualの単位は依頼記録（コミットの依頼発話の要約・stage時のtree hash・作成したSHA）も残す。差分の同一性はパスだけでなく内容・mode等も含めて確認する。Linear進行での担当間の引き渡しは [workflow.md](workflow.md)、コミット前後の単位照合と完了時の全体照合は `commit` に従う。
|
|
129
129
|
|
|
130
|
-
予定単位の内容・分割・統合を変更したら、変更した単位を再判定して必要なレビュー・許可確認・検証へ戻す。設定変更時も影響する未実行単位を再判定する。以前の判定や別単位の方針は流用しない。manual
|
|
130
|
+
予定単位の内容・分割・統合を変更したら、変更した単位を再判定して必要なレビュー・許可確認・検証へ戻す。設定変更時も影響する未実行単位を再判定する。以前の判定や別単位の方針は流用しない。manualで人の修正・依頼を経てcommitした場合は `commit` に従い、commit後に目的・範囲・実効flowと影響する挙動を確認する。正当な修正は元の予定との差と対応を残して確定単位へ反映する。不明・対象外の変更、確定単位との不一致は停止し、履歴を無断で書き換えない。
|
|
131
131
|
|
|
132
132
|
### config_version 1 からの移行
|
|
133
133
|
|
|
@@ -28,7 +28,7 @@ Claudeのメインは途中の新しい根拠で必要な担当を増やし、
|
|
|
28
28
|
|
|
29
29
|
原則、確定した順序で一つずつ実行する。Issue単体の依頼では完了後に別Issueへ進まない。複数対象では、現在のIssueのQA・必要な人の対応・コミットを残して次へ切り替えない。
|
|
30
30
|
|
|
31
|
-
- 実装担当の作業中や、manual
|
|
31
|
+
- 実装担当の作業中や、manualの人のレビュー・コミットの依頼・Human QA・本人認証・通常の承認待ちは必要な工程として扱い、必要な依頼を行って会話側のゴールを `active` のまま待機する。回答がないことや待ち時間をブロッカー到達・承認・完了と扱わない。
|
|
32
32
|
- 観測結果が得られないことだけでは終了条件を満たさない。同じsessionの状態を確認して待機を続け、新しいsessionやゴールを作る根拠にしない。
|
|
33
33
|
- QAで見つけた不具合は[共通ルール](workflow.md#失敗・許可・停止の共通ルール)に従って実装担当へ戻し、再検証する。テストの失敗だけでバックログ実行を打ち切らない。
|
|
34
34
|
- 未決の仕様判断、許可不足、対象外の依存、外部障害など、現在の許可と対象内では解消できない進行不能をブロッカーとする。
|
|
@@ -5,8 +5,8 @@
|
|
|
5
5
|
| 担当 | 責務 |
|
|
6
6
|
| --- | --- |
|
|
7
7
|
| Codex（進行役） | Issueの作成・整理、仕様相談、Ready判定、依存・Cycle管理、進行判断、作業先設定からの経路と許可の確定、体制判断の材料（受入条件・関連範囲・既存検証・リスク）の提示、Claudeへの依頼（auto: CLI、manual: Bridge）、Local・Staging・ProductionのAgent QA、単位・全体の照合、許可されたpush・PR・merge・配備・公開、Issueへの記録、人への依頼 |
|
|
8
|
-
| Claude（実装担当・Claude Code） | 初期体制の決定と途中の増員、`shinaps:dev-claude` による技術調査・計画・実装・検証・独立レビュー、QA指摘の修正、`shinaps:commit`（auto: 独立レビュー後にcommit、manual:
|
|
9
|
-
| 人 | 未合意の仕様・権限の判断、manual
|
|
8
|
+
| Claude（実装担当・Claude Code） | 初期体制の決定と途中の増員、`shinaps:dev-claude` による技術調査・計画・実装・検証・独立レビュー、QA指摘の修正、`shinaps:commit`（auto: 独立レビュー後にcommit、manual: 人の開始依頼で分割案、人が依頼した単位ごとにcommit）、結果の報告。実装を伴わない文書整理は、作業先リポジトリの規約に従いdevの入口の対象外にできる。スキル・コードの変更は文書の形でも実装としてdevの入口を使う |
|
|
9
|
+
| 人 | 未合意の仕様・権限の判断、manualのWebStormでの差分レビュー・修正相談・コミットの開始と単位ごとの依頼、本人の操作が必要な認証等、必要なHuman QA |
|
|
10
10
|
|
|
11
11
|
経路は着手時に通常設定の `review_flow.mode` で決まる（[経路](configuration.md#経路)）。autoはCodexが[ローカルClaude CLI](claude-cli.md)の同じsessionをresumeし、manualはWebStorm内の対話Claude Codeセッションへ[Bridge](bridge.md)で依頼する。どちらでもClaudeはdev内レビューと検証を終えて結果を返し、CodexがLocal Agent QAを行う。Claudeの報告をQA合格の根拠にせず、この引き渡しを進行全体の終了と扱わない。
|
|
12
12
|
|
|
@@ -23,8 +23,8 @@ Linear MCPで扱えない画面操作（添付、並べ替え、Cycleへの紐
|
|
|
23
23
|
| 工程 | 状態キー | 内容と次へ進む条件 |
|
|
24
24
|
| --- | --- | --- |
|
|
25
25
|
| 着手 | `ready` | Readyと依存を確認し、作業先設定を読み、通常設定の `review_flow` から経路と必要な許可（auto: `claude_cli`、manual: `bridge_request`）を確定する。checkout・branch・HEAD・既存差分を固定し、体制判断の材料を添えてClaudeへ依頼する。通常設定がautoでmanualのパス別ルールがあれば、起こり得る停止（[経路](configuration.md#経路)）を依頼に書く |
|
|
26
|
-
| 実装 | `coding`, `local_agent_qa` | Claudeのメインが初期体制を決めて報告し、`shinaps:dev-claude` で実装・検証・独立レビューを行い、未コミット差分・単位案・予定差分・判定記録を返す。CodexがLocal Agent QA（判定記録と経路の整合を含む）を行い、不合格なら同じsessionへ戻す |
|
|
27
|
-
| 差分確定 | `code_review` | QA合格後、Codexが実効flow・判定記録・QA結果を同じsessionへ渡し、Claudeが `shinaps:commit` を行う。autoは単位ごとに独立レビュー後にcommitする。manual
|
|
26
|
+
| 実装 | `coding`, `local_agent_qa` | Claudeのメインが初期体制を決めて報告し、`shinaps:dev-claude` で実装・検証・独立レビューを行い、未コミット差分・単位案・予定差分・判定記録を返す。manualでは人へユーザーのレビュー待ちと明示する。CodexがLocal Agent QA（判定記録と経路の整合を含む）を行い、不合格なら同じsessionへ戻す |
|
|
27
|
+
| 差分確定 | `code_review` | QA合格後、Codexが実効flow・判定記録・QA結果を同じsessionへ渡し、Claudeが `shinaps:commit` を行う。autoは単位ごとに独立レビュー後にcommitする。manualではCodexはcommitを依頼せず、渡した後は報告を待つ。人のコミット開始依頼でClaudeが分割案を示し、人が単位ごとに依頼したcommitを行う。Codexは各コミットの単位照合と全体照合を行う。差分確定中の修正は単位の目的・範囲内に限り、超えるものは実装へ戻す。差分確定で差分が変わったIssueは、確定後にCodexのAgent QAを必須にする（公開確認のpush前の最終Agent QAで兼ねてよい） |
|
|
28
28
|
| 公開確認 | `post_commit`, `staging_agent_qa`, `staging_human_qa`, `release`, `production_agent_qa`, `production_human_qa` | 受入条件と許可に応じて必要なものだけ行う: push前の最終Agent QA、push・PR作成、環境配備とStaging/ProductionのAgent QA、必要なHuman QA、merge・配備・公開。適用しない項目は理由を記録する |
|
|
29
29
|
| 結果報告 | （状態キーなし） | 結果・証拠・省略した工程と理由・残項目をIssueへ記録し、終了地点に到達する。必須受入条件をすべて満たしたIssueは `done` にする |
|
|
30
30
|
|
|
@@ -60,7 +60,7 @@ Linear MCPで扱えない画面操作（添付、並べ替え、Cycleへの紐
|
|
|
60
60
|
| Codexから許可範囲と受入条件を伴う実装・修正の依頼を受けた | Claudeは人へ着手を再確認せず、指定されたスキルで実行する |
|
|
61
61
|
| devの入口の中で計画上の判断が必要になった | autoはClaudeがresultでCodexへ相談し、Codexは既存の合意で判断できれば回答する。manualはClaudeが人へ直接確認し、決定を報告する |
|
|
62
62
|
| QAで受入条件の範囲内の不具合が見つかった | Codexが同じClaude sessionへ修正を依頼し、Claudeは人の着手許可を改めて取らずに修正する。Codexが再検証する |
|
|
63
|
-
| ClaudeがCLIのresultでcommitの完了を、またはBridge
|
|
63
|
+
| ClaudeがCLIのresultでcommitの完了を、またはBridgeで人の依頼によるcommitの完了を報告した | Codexは人に同じ事実を聞き直さず、報告の対象とGitの状態を照合して次の工程へ進む |
|
|
64
64
|
| 次の操作の継続的な許可があり、必要な工程の結果が揃った | Codexは「進めてもよいか」を再質問せずに実行する |
|
|
65
65
|
| Cycle全体を任され、対象Issueが設定された終了地点に達した | Codexは開始時に確定した対象と順序を照合し、その範囲の次のIssueへ進む |
|
|
66
66
|
|
|
@@ -72,9 +72,9 @@ Linear MCPで扱えない画面操作（添付、並べ替え、Cycleへの紐
|
|
|
72
72
|
|
|
73
73
|
1. Claudeは成果引き渡しに、モードによらず自然な責務で決めたコミット単位案と予定差分を含める。CodexはLocal Agent QAで対象を検証し、単位案が依頼全体を漏れなく覆い対象外変更を含まないことと、各単位の[判定記録](configuration.md#判定記録と変更時の扱い)を確認する。依頼の比較元と全体差分も最終照合用に記録する。判定記録に経路と異なるmanual単位があれば、差分確定の依頼前に[経路](configuration.md#経路)に従って人へ確認する。
|
|
74
74
|
2. QA合格後、Codexは実効flow・単位の判定記録・QA結果を同じClaude sessionへ渡す。Claudeは設定・予定差分・合意根拠を照合する。
|
|
75
|
-
3. Claudeは `commit` に従う。autoでは独立レビュー後にClaudeがstage・commitする。manual
|
|
75
|
+
3. Claudeは `commit` に従う。autoでは独立レビュー後にClaudeがstage・commitする。manualでは人のコミット開始依頼で分割案を示し、人がその単位のコミットを依頼したら単位外を安全に退避して単位をstage・検証し、Claudeがcommitする。各コミット直前・作成後の実差分と確定した予定単位を照合し、予定単位・SHA・検証・レビュー・依頼記録・復元結果をCodexへ返す。
|
|
76
76
|
4. 全単位完了後、Codexは各コミットの単位照合結果を実差分で独立確認したうえで、依頼の比較元からの全体差分が、`commit` の手順で正当な人の修正を反映した確定済みの全体期待差分と一致し、対象の未コミット変更が残っていないことを照合する。最終差分だけでは途中で混入し後で相殺された対象外変更を検出できないため、単位照合を省かない。人レビューを省略した単位は、上の省略記録を完成させる。
|
|
77
|
-
5. 単位の組み替え・内容・設定の変更は[変更時の扱い](configuration.md#判定記録と変更時の扱い)
|
|
77
|
+
5. 単位の組み替え・内容・設定の変更は[変更時の扱い](configuration.md#判定記録と変更時の扱い)に従って再判定し、必要なレビュー・許可確認・検証へ戻す。人の依頼によるcommit（または人自身のcommit）の後は、`commit` に従って正当な修正と元の予定差分の対応を記録し、確定単位に更新する。確定単位と実コミットに不一致があれば次工程を止め、SHA・予定差分・実差分の違いを記録して人に確認する。履歴を無断で書き換えない。
|
|
78
78
|
|
|
79
79
|
## 待ち状態
|
|
80
80
|
|
|
@@ -90,13 +90,13 @@ manual経路の依頼・報告と、別会話との任意の進行の引き渡
|
|
|
90
90
|
|
|
91
91
|
## 人への依頼
|
|
92
92
|
|
|
93
|
-
manual
|
|
93
|
+
manualの差分レビュー・相談・コミットの依頼は、WebStormの差分表示と対話Claude Codeセッションで[manualのレビュー待ちと単位ごとのコミット](../human-review.md)に従って行う。
|
|
94
94
|
|
|
95
95
|
会話でIssueリンクと必要な作業を明示し、ステータスの変更だけで依頼を済ませない。
|
|
96
96
|
|
|
97
97
|
| 依頼 | 文例 |
|
|
98
98
|
| --- | --- |
|
|
99
|
-
| manual
|
|
99
|
+
| manualのレビュー | できるようになったことと重点確認点を添えて「[Issueリンク]の差分をWebStormでレビューし、コミットはWebStormのClaude Codeセッションで依頼してください。」 |
|
|
100
100
|
| Human QA | 「[Issueリンク]のHuman QAをお願いします。」 |
|
|
101
101
|
| 外部対応 | 「[Issueリンク]の〇〇をお願いします。」 |
|
|
102
102
|
|
|
@@ -22,11 +22,11 @@ description: 許可された差分を依存順のローカルコミットにし
|
|
|
22
22
|
|
|
23
23
|
## 範囲と進め方
|
|
24
24
|
|
|
25
|
-
コミットは利用者の明示依頼、または[Linear共通設定](../../references/linear/configuration.md)の合意済みの許可に従う。Linear進行では[担当と工程](../../references/linear/workflow.md)の差分確定に従い、Local Agent QA
|
|
25
|
+
コミットは利用者の明示依頼、または[Linear共通設定](../../references/linear/configuration.md)の合意済みの許可に従う。Linear進行では[担当と工程](../../references/linear/workflow.md)の差分確定に従い、Local Agent QA後に進行役から対象を指定した依頼を受けて行う。manualでは進行役はcommitを依頼せず、QA結果と判定記録の引き渡しの後に人の依頼で行う。単独の依頼にはLinear進行や新しいゴールを必須にしない。
|
|
26
26
|
|
|
27
27
|
どちらのモードも、許可された対象以外の変更と秘密情報、push、PR、公開、既存履歴の書き換えを含まない。
|
|
28
28
|
|
|
29
|
-
Git準備・修正・commitはAIが現在のセッションで行い、manual
|
|
29
|
+
Git準備・修正・commitはAIが現在のセッションで行い、manualのレビューとコミットの依頼は人が行う。autoは独立レビューを必須とする。必要な担当・許可が利用できなければ、セルフレビューを同等と称さず止める。別worktreeの作成や別セッションへの委託を、このスキルから許可しない。
|
|
30
30
|
|
|
31
31
|
## 回復できる状態を作る
|
|
32
32
|
|
|
@@ -43,23 +43,23 @@ Git準備・修正・commitはAIが現在のセッションで行い、manualの
|
|
|
43
43
|
|
|
44
44
|
予定単位ごとに、[パス別の方針](../../references/linear/configuration.md#パス別の方針)でそのコミットの全変更を判定し、同資料の判定記録を残す。Linear進行から記録を受け取った場合は、設定とその単位の予定差分（内容・mode等を含む）の一致を確認して引き継ぐ。単独の依頼でも同じ判定を使い、Linear専用の工程は追加しない。設定が真にない単独依頼では、今回の明示許可とプロジェクト規約を通常方針とし、パス例外は適用しない。
|
|
45
45
|
|
|
46
|
-
合意済みの実効 `review_flow` と今回の明示指示から進め方を確定する。両方とも対象のstageとAIのcommitを許可し、`auto` では独立レビュー後、`manual`
|
|
46
|
+
合意済みの実効 `review_flow` と今回の明示指示から進め方を確定する。両方とも対象のstageとAIのcommitを許可し、`auto` では独立レビュー後、`manual` では人がその単位のコミットを依頼した後にAIがcommitする。設定の存在だけを合意と扱わない。旧キー・不備・競合は共通設定に従って止める。
|
|
47
47
|
|
|
48
|
-
設定がない単独依頼でも「各単位をAI
|
|
48
|
+
設定がない単独依頼でも「各単位をAIがレビューして自動コミット」「人がレビューしてから単位ごとにコミット」など明確な意図を引き継ぐ。「コミットして」「進めて」だけからautoを推測しない。flowが未解決のときだけ対象を示し「auto（AIがレビュー・コミット）とmanual（人がレビューして単位ごとにコミットを依頼し、AIが修正・コミット）のどちらで進めますか？」と一問聞き、回答まで依存操作を待つ。未回答を承認と扱わない。同じ依頼内でもパス例外により異なるflowの単位があってよい。
|
|
49
49
|
|
|
50
50
|
モード決定後にレビュー可能性や依存関係のため単位を分割・統合する場合は、[変更時の扱い](../../references/linear/configuration.md#判定記録と変更時の扱い)に従って変更した単位を再判定し、必要なレビュー・許可確認・検証へ戻す。設定変更時も影響する未実行単位を再判定する。「autoだからまとめる」「manualだから分ける」を先に適用して単位決定と判定を循環させない。
|
|
51
51
|
|
|
52
52
|
## 単位ごとの検証・レビュー・コミット
|
|
53
53
|
|
|
54
|
-
manualでは[manual
|
|
54
|
+
manualでは[manualのレビュー待ちと単位ごとのコミット](../../references/human-review.md)に従い、人と対話できるセッションで人の依頼を受ける。手順1は人のコミット開始依頼の後に、手順2以降は人がその単位のコミットを依頼した後に行う。ローカルClaude CLIの非対話実行でmanualの単位を検出した場合は、手順を進めず、判定を記録してresultへ返す（[経路](../../references/linear/configuration.md#経路)）。
|
|
55
55
|
|
|
56
56
|
以下を、その単位のモードで行う。
|
|
57
57
|
|
|
58
58
|
1. 単位案を短く示す。manualで複数の単位に分ける場合、許可された独立レビュアーがいれば、各単位の差分だけで人が妥当性を判断できるかを差分自体から確認させ、結論はメインが実際の差分で検証する。autoでは単位案の独立レビューを求めない（プロジェクトの必須条件にある場合は従う）。
|
|
59
59
|
2. コミットと検証の内容が単位だけで決まるようにする。manualでは、単位以外の未コミット変更（対象外、後続の単位、untracked）を一意な名前のstashへ退避し、作業ツリーとindexに単位の変更だけを残す。autoでは、対象外の変更を作業ツリーとindexに残してよく、検証結果が残る変更に依存しないことを根拠（例: 残る変更が検証の入力に含まれない）とともに判断する。依存する、または判断できない場合は、許可された範囲で単位をまとめ直すか、manualと同じ方法で退避する。退避が他の進行中の作業（他のセッションや人がステージ・編集中の変更）を妨げる場合は退避せず、所有者と調整できるまで止める。manualは別の隔離方法へ置き換えない。退避するときは、stashのindexにコミット対象のステージ済み変更を入れない。`git stash push` はパスを指定してもindex全体を保存し、対象のステージ済み変更が入ると後の `git stash apply --index` が失敗するため。対象のステージ済み差分のパッチを保存し、対象のステージを外してから退避し、退避後にそのパッチで対象のステージを戻す。作成前後のstash一覧を比べて今回作成されたものだけのSHAを記録し、既存stashの先頭を自分のものと決めつけない。共有ファイルのhunk分割は控えから行い、対象のhunkだけのステージを外せない場合や、対象外のステージ済みの変更が変わる場合は止める。
|
|
60
60
|
3. コミットする内容に必要な検証を行う。後続や対象外の未コミット変更に依存した成功を、その単位の検証としない。範囲を限った検証はプロジェクトが認める場合だけ使い、必須検証を省かない。後続や対象外への依存で失敗するなら、手順2に戻って単位か隔離を見直す。
|
|
61
|
-
4. 対象だけをステージし、実際にコミットされる差分の全体（indexのままコミットする場合は完全なcached diff）と影響するファイルをレビューする。indexに対象外の変更を残す場合は、それを含めずに対象だけをコミットする方法を使い、対象を限定できなければ止める。manual
|
|
62
|
-
5. 必須検証と独立レビュー（auto必須）が通り、阻害指摘がないことを確認する。autoでは直前の全差分が予定単位と一致してからAIが `git commit` を単独実行する（`git add` と連結しない）。manual
|
|
61
|
+
4. 対象だけをステージし、実際にコミットされる差分の全体（indexのままコミットする場合は完全なcached diff）と影響するファイルをレビューする。indexに対象外の変更を残す場合は、それを含めずに対象だけをコミットする方法を使い、対象を限定できなければ止める。manualで人がWebStormで確認できるstaged差分は、コミットされる差分と同じものにする。指摘は実物で確認し、依頼範囲の修正後は再ステージ・再検証する。外部仕様の調査は変更がそこに依存するときに行う。
|
|
62
|
+
5. 必須検証と独立レビュー（auto必須）が通り、阻害指摘がないことを確認する。autoでは直前の全差分が予定単位と一致してからAIが `git commit` を単独実行する（`git add` と連結しない）。manualではstage時の `git write-tree` を記録し、単位名・目的・検証結果・推奨メッセージを短く示す（差分本文は貼らない）。記録時の差分が分割案の予定単位と違えばcommitせず報告して依頼を取り直す。直前のindexのtreeが記録したtreeと一致することを確認してAIが `git commit` を単独実行し、作成したコミットのtreeも照合する。treeの記録後に単位のパスで作業ツリーまたはindexが変わったら（人との相談による修正を含む）依頼は失効し、再stage・再検証・再提示して改めて人の依頼を受ける。依頼はその単位にだけ及ぶ。commit後に（人自身のcommitを含む）、比較元からの実コミット一覧と各SHA・全差分を調べ、相談で行った修正が許可された目的・範囲を満たすことを確認する。実差分で実効flowを再判定し、影響する挙動を再検証する。正当な人の修正は確定単位と全体の期待差分へ反映し、元の予定差分・依頼記録・変更理由・実コミットとの対応を残す。意図や範囲が不明、対象外の混入、flowの適用が未解決なら人へ確認して止める。Gitにcommitがあるだけで人レビュー完了としない。各コミットを確定単位と照合してSHA・結果を記録し、不一致なら次工程を止める。履歴を無断で書き換えない。
|
|
63
63
|
6. 退避した場合は、コミット後または中断時に、indexにコミット対象のステージ済み変更がない状態で、記録したSHAのstashを `git stash apply --index` で戻す。中断時は対象のステージを外し、stashの復元を妨げる対象の未コミット変更（共有ファイルの対象のhunkなど）もパッチに保存して作業ツリーから一時的に外してから戻す。その後、外した対象の変更を戻し、対象のステージ済み差分のパッチで対象のステージを戻す。復元前に人が修正した最新のHEAD・作業ツリー・indexを記録して保全する。控えで扱った対象外変更だけを復元し、古い対象パッチで人の修正を上書きしたり、コミット済みの変更を再び加えたりしない。退避の有無によらず、コミットしていない変更（ステージ済み・未ステージ・untracked）と既存stashが、操作前の記録と確定済みの人の修正からコミットした内容を除いたものと一致することを確認する。一致しない、曖昧な競合がある、ステージ境界を戻せない場合は推測で消さず、一致しないパスと、退避した場合は回復資料を示して止める。一致を確認してから今回作成したstashだけをdropし、HEADを確認して次へ進む。
|
|
64
64
|
|
|
65
65
|
全単位の完了後、各単位の照合結果に加え、元の比較元からの全体差分が、人の正当な修正を反映した確定済みの全体期待差分と一致し、対象の未コミット変更が残っていないことを確認する。最終差分だけで途中コミットの照合を代用しない。予定どおりのコミットによる残差分の減少は不一致と扱わない。
|
|
@@ -12,7 +12,7 @@ Claudeは実装担当で、Codexが進行・Agent QA・Issueの記録・人へ
|
|
|
12
12
|
経路は着手時に通常設定の `review_flow.mode` で決まり、Issueの途中で切り替えない（[経路](../../references/linear/configuration.md#経路)）。
|
|
13
13
|
|
|
14
14
|
- auto: Codexが起動する[ローカルClaude CLI](../../references/linear/claude-cli.md)で依頼を受け、質問・障害・完了はresultへ返す。回答・修正・コミットの依頼は同じsessionのresumeで受ける。
|
|
15
|
-
- manual: WebStorm内の対話Claude Codeセッションで[Bridge](../../references/linear/bridge.md)の依頼を受ける。人と直接作業し、計画上の判断・仕様の不明点は人へ `AskUserQuestion`
|
|
15
|
+
- manual: WebStorm内の対話Claude Codeセッションで[Bridge](../../references/linear/bridge.md)の依頼を受ける。人と直接作業し、計画上の判断・仕様の不明点は人へ `AskUserQuestion` で一問ずつ確認する。作業の完了時は人へユーザーのレビュー待ちと明示する（[manualの手順](../../references/human-review.md)）。完了・障害・人の依頼によるcommitの完了は、Bridgeで依頼元のCodexへ報告する。
|
|
16
16
|
|
|
17
17
|
## 読む資料
|
|
18
18
|
|
|
@@ -29,12 +29,12 @@ Claudeは実装担当で、Codexが進行・Agent QA・Issueの記録・人へ
|
|
|
29
29
|
- devの入口の委託（サブエージェント）は、依頼の許可範囲に `dev_delegation` が書かれている場合だけ使う。worktreeの作成は含まない。
|
|
30
30
|
- QAで見つかった不具合の修正依頼も同じ扱いにし、人の着手許可を改めて取らない。
|
|
31
31
|
- 同じ作業ツリーをCodexや人と同時に編集しない。人や他のAIの変更を保持する。
|
|
32
|
-
- コミットの許可は[共通設定](../../references/linear/configuration.md)、担当とQA後の条件は[担当と工程](../../references/linear/workflow.md)の差分確定に従う。実装完了の直後に先行してコミットせず、Codex
|
|
32
|
+
- コミットの許可は[共通設定](../../references/linear/configuration.md)、担当とQA後の条件は[担当と工程](../../references/linear/workflow.md)の差分確定に従う。実装完了の直後に先行してコミットせず、Codexへ結果を返す。autoはCodexから差分確定の依頼を受けたら、manualは人がコミットの開始を依頼したら `shinaps:commit` を使い（単位の退避・stage・commitはQA結果の引き渡し後）、許可済みの操作を再確認しない。
|
|
33
33
|
|
|
34
34
|
## 結果と人の対応を報告する
|
|
35
35
|
|
|
36
|
-
-
|
|
37
|
-
- 人がClaude
|
|
36
|
+
- 完了の報告と、人の依頼とcommitの報告に含める項目は、経路の資料に従う。
|
|
37
|
+
- 人がClaudeへ直接伝えた指示・決定・コミットの依頼は保持し、Codexへ報告する。Gitにコミットがあるだけで、人の依頼によるcommitと報告しない。設定で人のレビューを省略した単位は、省略と報告する。
|
|
38
38
|
|
|
39
39
|
## Codexの依頼なしで起動された場合
|
|
40
40
|
|
|
@@ -105,7 +105,7 @@ Issue・既存spec・調査記録・関連コード・検証方法を必要な
|
|
|
105
105
|
|
|
106
106
|
初期体制と変更理由、実際の担当・確認範囲、計画と実装後のレビュー回数、採否・修正、検証結果、未実施・未照合・未解決事項を報告する。
|
|
107
107
|
|
|
108
|
-
linear-executeから受けた作業は未コミットの差分と結果を進行役へ返す。devの完了はLocal Agent QAへの引き渡しであり、進行全体の終了ではない。[Linear共通設定](../linear/configuration.md)と[担当と工程](../linear/workflow.md)に従い、QA
|
|
108
|
+
linear-executeから受けた作業は未コミットの差分と結果を進行役へ返す。devの完了はLocal Agent QAへの引き渡しであり、進行全体の終了ではない。[Linear共通設定](../linear/configuration.md)と[担当と工程](../linear/workflow.md)に従い、autoはQA後の差分確定の依頼を受けて、manualは人のコミット開始依頼で `commit` を始め、単位の退避・stage・commitはQA結果の引き渡し後に行う。auto経路は [Claude CLI](../linear/claude-cli.md) のresultへ報告し、同じsessionのresumeで質問・修正を受ける。manual経路は同じ対話セッションで人と作業し、[Bridge](../linear/bridge.md)で依頼元Codexへ報告する。manualの完了時は人へユーザーのレビュー待ちと明示する（[manualの手順](../human-review.md)）。
|
|
109
109
|
|
|
110
110
|
単独のdevではコミットが許可されている場合だけ範囲とモードを `commit` へ渡す。許可がなければ未コミットで返し、形式的な許可確認を追加しない。PR・リリースも許可された範囲で各スキルへ渡す。単独devにLinear進行・新しいゴールを必須にしない。
|
|
111
111
|
|
|
@@ -1,26 +1,35 @@
|
|
|
1
|
-
# manual
|
|
1
|
+
# manualのレビュー待ちと単位ごとのコミット
|
|
2
2
|
|
|
3
|
-
実効 `review_flow.mode: manual`
|
|
3
|
+
実効 `review_flow.mode: manual` の差分を、人がWebStormの差分表示でレビューし、人の依頼に従ってWebStorm内の対話Claude Codeセッションで単位ごとにClaudeがcommitする手順。review_flow・操作許可は、プロジェクト設定と今回の依頼に従う。この手順はそれらを変更しない。
|
|
4
4
|
|
|
5
|
-
|
|
5
|
+
会話では単位名・目的・対象ファイル・検証結果・短い推奨コミットメッセージなど判断に必要な最小限を示し、差分本文を貼らない。Claude自身の差分の読取りと独立レビューは制限しない。
|
|
6
6
|
|
|
7
|
-
|
|
8
|
-
2. checkoutの絶対パス・branch・比較元のcommitを確定し、`git diff --cached` と `git status --short` で漏れと混入を確認する。未追跡ファイルも対象に含める。
|
|
9
|
-
3. 会話で単位の差分・目的・確認してほしい点・検証結果・推奨コミットメッセージを提示する。Linear進行ではIssueも示す。人はWebStormの差分表示で同じstaged差分を確認できる。
|
|
10
|
-
4. 提示時に `git write-tree` の値を記録する。これが承認の対象になる。
|
|
7
|
+
## レビュー待ち
|
|
11
8
|
|
|
12
|
-
|
|
9
|
+
devと必要な検証が終わったら、作業内容と検証結果を短く報告し、「ユーザーのレビュー待ち」と明示する。人はWebStormの差分表示で全体を確認し、Claudeと相談する。
|
|
10
|
+
|
|
11
|
+
人がコミットの開始を依頼するまで、分割案の提示・単位外の退避・単位のstage・単位のコミット依頼の要求を始めない。Linear進行でCodexへの完了報告に含める単位案と予定差分はQAの材料であり、人への分割案の提示ではない。Linear進行では、分割案の提示は人の開始依頼だけで行え、単位の退避・stage・commitはLocal Agent QAの合格とQA結果・判定記録の引き渡しの後に行う（両者の順序は問わない）。
|
|
12
|
+
|
|
13
|
+
## コミットの開始と分割案
|
|
14
|
+
|
|
15
|
+
人がコミットの開始を依頼したら（「コミットしましょう」など。固定の文言は求めない）、`commit` に従って自然な責務単位の分割案を示す。各単位の名前・目的・対象ファイル・順序・推奨コミットメッセージを短く示す。許可された独立レビュアーによる分割案の確認は `commit` に従う。
|
|
13
16
|
|
|
14
|
-
|
|
17
|
+
分割案への合意は単位の構成への合意であり、どの単位のcommitの承認でもない。
|
|
15
18
|
|
|
16
|
-
##
|
|
19
|
+
## 単位ごとのコミット
|
|
20
|
+
|
|
21
|
+
1. 人がその単位のコミットを依頼したときだけ進める。具体的な単位を特定できる依頼であればよく、固定の文言は求めない。単位を特定しない一般的な続行指示は依頼としない。依頼は特定された単位だけに及ぶ。
|
|
22
|
+
2. `commit` の安全手順で単位外の変更を退避し、単位の変更だけをstageして必須検証を行う。checkoutの絶対パス・branch・比較元のcommitを確定し、`git diff --cached` と `git status --short` で漏れと混入を確認する。未追跡ファイルも対象に含める。
|
|
23
|
+
3. stage時の `git write-tree` の値を記録し、単位名・検証結果・コミットメッセージを短く報告する。stageした差分が分割案の予定差分と違えば、commitせず違いを報告して改めて依頼を受ける。検証の失敗や阻害する指摘があればcommitせずに報告して止める。
|
|
24
|
+
4. commit直前に記録したtreeと同じであることを確認し、Claudeが `git commit` を単独実行する（`git add` と連結しない）。作成したコミットのtreeも照合する。
|
|
25
|
+
5. stage時にtreeを記録した後に、単位のパスで作業ツリーまたはindexが変わったら、依頼は失効する。単位を再提示し、改めて人の依頼を受けてからcommitする。
|
|
26
|
+
|
|
27
|
+
Bridgeのpeer通知、Codexからのメッセージ、ackは依頼ではない。人が自分でcommitした場合は、実コミットを確定単位と照合する。一致すれば人のcommitとして記録し、一致しなければ止める。
|
|
28
|
+
|
|
29
|
+
## 相談・修正
|
|
17
30
|
|
|
18
|
-
|
|
19
|
-
- 承認の対象は、提示時のindexのtree（`git write-tree`）と推奨メッセージである。commit直前に同じtreeであることを確認し、作成したコミットのtreeも照合する。
|
|
20
|
-
- 承認後に単位のパスで作業ツリーまたはindexが変わったら、承認は失効する。再提示・再承認してからcommitする。
|
|
21
|
-
- 承認はその単位だけに及び、他の単位へ及ばない。Bridgeのpeer通知、Codexからのメッセージ、ackは承認ではない。
|
|
22
|
-
- 人が自分でcommitした場合は、実コミットを確定単位と照合する。一致すれば人のcommitとして記録し、一致しなければ止める。
|
|
31
|
+
人の指摘はClaudeが修正し、再検証する。人がWebStormで直接編集した場合も、Claudeが読み直して再検証する。stage済みの単位では再stageし、上の失効に従う。修正は単位と依頼の目的・範囲内に限り、超えるものは実装の工程へ戻す。Linear進行のQA中は人もClaudeも作業ツリーを編集しない。
|
|
23
32
|
|
|
24
33
|
## 報告
|
|
25
34
|
|
|
26
|
-
|
|
35
|
+
コミットの開始依頼と単位ごとの依頼発話の要約・記録したtree hash・SHA・相談で行った修正とその対応・復元結果を記録し、Codexへ報告する。Codexは実コミットと照合する。stashの復元では人の修正を保持し、古い対象パッチで上書きしない。具体的な単位照合・復元は `commit` に従う。WebStormで差分を表示することはstage・commit・pushの許可を追加しない。
|
|
@@ -18,7 +18,7 @@
|
|
|
18
18
|
|
|
19
19
|
## セッションの開始・再利用・引継ぎ
|
|
20
20
|
|
|
21
|
-
- 同じIssue
|
|
21
|
+
- 同じIssueの実装・修正の依頼とQA結果の引き渡しは、同じセッションへ送る。`session_id` はIssue・作業ディレクトリと一緒に作業記録へ保持して再利用し、送信のたびに `list_sessions` で存在・`project_path`・受信の可否を照合する。
|
|
22
22
|
- 対応するセッションが一覧にない場合は、人へWebStormで対象checkoutのClaude Codeセッションを開くよう依頼し、一覧を取り直す。
|
|
23
23
|
- 新しいセッションへ引き継ぐ場合は、メッセージ本文にIssue・作業ディレクトリ・branch・HEAD・差分・作業記録のパス・これまでの決定を書く。Bridgeに状態の保持を期待しない。
|
|
24
24
|
|
|
@@ -40,16 +40,16 @@
|
|
|
40
40
|
| --- | --- |
|
|
41
41
|
| 実装の依頼（CodexからClaude） | Issueリンク、`shinaps:dev-claude`、範囲・受入条件、体制判断の材料、Gitの作業ディレクトリ・branch・HEAD・既存差分、許可範囲（`dev_delegation` の有無を含む）、返信先のCodexの `session_id`、関係する補足と参照先、下の定型文 |
|
|
42
42
|
| 修正の依頼（CodexからClaude） | 実装の依頼の項目に加え、再現手順・期待と実際・対象差分・証拠の場所 |
|
|
43
|
-
|
|
|
44
|
-
| 完了の報告（ClaudeからCodex） | 体制と途中の増員、対象差分、自動検証の結果、確認観点、既知の制限、未解決の事項、dev
|
|
45
|
-
|
|
|
43
|
+
| QA結果の引き渡し（CodexからClaude。manualの単位のcommitの依頼ではない） | Issueリンク、対象差分・作業ディレクトリ・branch、Local Agent QAの結果、実効flow・単位の判定記録、返信先 |
|
|
44
|
+
| 完了の報告（ClaudeからCodex） | 体制と途中の増員、対象差分、自動検証の結果、確認観点、既知の制限、未解決の事項、devの入口内のレビュー結果の要約、単位案と予定差分、人と決めたこと、人へレビュー待ちと明示したこと |
|
|
45
|
+
| 人の依頼とcommitの報告（ClaudeからCodex） | 対象Issue、作業ディレクトリ・branch、コミットの開始依頼と単位ごとの依頼発話の要約・tree hash・SHA、相談で行った修正、復元結果、残作業 |
|
|
46
46
|
| 開発進行の引き渡し（ClaudeからCodex） | 利用者からの依頼の内容、対象Issue・Cycle、作業先リポジトリ、利用者が示した範囲と条件。Codexは作業先の設定と照合して進行を始め、不足する判断はCodexから利用者へ確認する |
|
|
47
47
|
|
|
48
48
|
実装・修正の依頼には次の文を含める。
|
|
49
49
|
|
|
50
50
|
```text
|
|
51
|
-
ユーザー承認済みの範囲でCodexが進行する。この依頼の範囲と受入条件の中では人へ着手を再確認せず、shinaps:dev-claudeで実行する。初期体制はあなたが決めて報告する。計画上の判断・仕様の不明点はこのセッションで人へ直接一問ずつ確認し、決定を報告に含める。許可を広げる決定は報告に含めるだけで、Codex
|
|
51
|
+
ユーザー承認済みの範囲でCodexが進行する。この依頼の範囲と受入条件の中では人へ着手を再確認せず、shinaps:dev-claudeで実行する。初期体制はあなたが決めて報告する。計画上の判断・仕様の不明点はこのセッションで人へ直接一問ずつ確認し、決定を報告に含める。許可を広げる決定は報告に含めるだけで、Codexが人へ確認する。完了・障害・人の依頼によるcommitの完了は、Bridgeで依頼元のCodexへ返す。
|
|
52
52
|
許可範囲: <設定と会話で許可された操作。dev_delegationの有無を含む>
|
|
53
53
|
```
|
|
54
54
|
|
|
55
|
-
Codex
|
|
55
|
+
Codexは人の依頼とcommitの報告を受けたら、対象Issue・差分・コミットと、対象作業の未コミット変更の有無を照合する。報告と実体が一致し、許可済みの工程へ進む場合は、人へ再確認しない。Gitにコミットがあるだけで人の依頼によるcommitと推測しない。報告が足りない場合は、まずClaudeへ補足を求める。
|
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
|
|
5
5
|
## 検証CLIと契約版
|
|
6
6
|
|
|
7
|
-
利用中のskillとCLIの版を確認し、0.34.
|
|
7
|
+
利用中のskillとCLIの版を確認し、0.34.1のskillには同じ版のnpm CLIを導入する（`npm install -g @shinaps/skills@0.34.1`）。pluginだけにはCLIは含まれない。導入・更新が必要ならその権限を確認し、検証のたびに `npx latest` を取得しない。
|
|
8
8
|
|
|
9
9
|
```sh
|
|
10
10
|
shinaps-skills --version
|
|
@@ -76,10 +76,10 @@ shinaps-skills validate-linear-config /absolute/repository/.shinaps/linear.json
|
|
|
76
76
|
| `permissions` | 継続的に許可した操作の一覧（下の「許可」）。経路の許可（auto: `claude_cli`、manual: `bridge_request`）も含む。許可なしは `[]` |
|
|
77
77
|
| `references` | `specifications`・`validation`・`environments`・`deployment`・`publication` の5分類をすべて記載し、既存資料のパスまたはURLの一覧を指定する。該当資料なしは `[]` |
|
|
78
78
|
| `cycle_report` | Cycle報告の設定（下の「Cycle報告」）。報告を使わない場合は `null` |
|
|
79
|
-
| `review_flow` | `{ "mode": "auto" }`（AIが自然な単位を決め、独立レビュー・QA後にstage・commit）または `{ "mode": "manual" }`（AI
|
|
79
|
+
| `review_flow` | `{ "mode": "auto" }`（AIが自然な単位を決め、独立レビュー・QA後にstage・commit）または `{ "mode": "manual" }`（AIがレビュー待ちを明示し、人がWebStormで差分を確認する。人のコミット開始依頼でAIが分割案を示し、人が単位ごとに依頼したらAIがstage・commit）。開始時の値で経路も決まる（下の「経路」） |
|
|
80
80
|
| `path_policy_overrides` | パス別に `review_flow` を上書きするルールの一覧。例外なしは `[]`（下の「パス別の方針」） |
|
|
81
81
|
|
|
82
|
-
- 利用者と合意した `review_flow` は両方とも対象のstageとAIのcommitを許可する。`auto` は独立レビュー後、`manual`
|
|
82
|
+
- 利用者と合意した `review_flow` は両方とも対象のstageとAIのcommitを許可する。`auto` は独立レビュー後、`manual` は人がその単位のコミットを依頼した後にcommitする。未合意の設定ファイルから許可を得ない。その場の明示指示が優先し、push・公開・履歴変更は別の許可に従う。
|
|
83
83
|
- どちらのflowもdev内の確認、必須検証、Agent QA、必要なHuman QAを維持する。`auto` は独立レビューを必須とし、利用できなければ自己確認で代用せず止める。人レビュー省略を人のレビュー済みと記録しない。
|
|
84
84
|
- 状態キーは [workflow.md](workflow.md) の「工程とステータス」の表と同じ。`statuses` の値はチームに実在する正確な名前にする。操作時はLinearから取得した状態のIDを使う。対応が曖昧・欠落・実物と不一致なら、その対応に依存する更新を止める。ステータスを自動で新設しない。
|
|
85
85
|
- `team_settings: manage_on_setup` の場合も、通常実行ではチーム設定を変更しない。
|
|
@@ -94,8 +94,8 @@ shinaps-skills validate-linear-config /absolute/repository/.shinaps/linear.json
|
|
|
94
94
|
| `manual` | WebStorm内の対話Claude CodeセッションへBridgeで依頼する（[Bridge](bridge.md)） | `bridge_request` |
|
|
95
95
|
|
|
96
96
|
- 経路はIssueの途中で切り替えない。`permissions`・`stop_after`・`dev_delegation`・worktreeや別セッションの作成の許可は、経路によって変わらない。
|
|
97
|
-
- manual経路でパス別の方針によりauto
|
|
98
|
-
- auto経路でmanual単位が生じた場合、非対話CLI
|
|
97
|
+
- manual経路でパス別の方針によりauto単位が生じた場合は、人のコミット開始依頼の後、対話session内のClaudeがその単位を単位ごとの依頼なしに独立レビュー後にcommitし、人のレビューの省略を記録する。
|
|
98
|
+
- auto経路でmanual単位が生じた場合、非対話CLIは人との相談・コミットの依頼を扱えない。CodexはLocal Agent QAで判定記録を照合した時点で検出し、差分確定の依頼前に人へ一問で確認する: 「今回の明示指示でその単位をautoとして進める」か「Issueを中止し、未コミット差分を保全したまま設定（通常設定またはルール）を見直してから、manual経路でやり直す」か。黙ってautoへ落とさない。`commit` の中で検出した場合は手順を進めず、判定を記録してresultへ返す。
|
|
99
99
|
|
|
100
100
|
## パス別の方針
|
|
101
101
|
|
|
@@ -125,9 +125,9 @@ shinaps-skills validate-linear-config /absolute/repository/.shinaps/linear.json
|
|
|
125
125
|
|
|
126
126
|
### 判定記録と変更時の扱い
|
|
127
127
|
|
|
128
|
-
各予定単位について、設定の正本と内容、適用した全ルール（または適用なしの理由）、単位の比較元と全差分、実効 `review_flow`、今回の明示指示とflowの合意根拠、検証・レビューの結果を記録する。manual
|
|
128
|
+
各予定単位について、設定の正本と内容、適用した全ルール（または適用なしの理由）、単位の比較元と全差分、実効 `review_flow`、今回の明示指示とflowの合意根拠、検証・レビューの結果を記録する。manualの単位は依頼記録（コミットの依頼発話の要約・stage時のtree hash・作成したSHA）も残す。差分の同一性はパスだけでなく内容・mode等も含めて確認する。Linear進行での担当間の引き渡しは [workflow.md](workflow.md)、コミット前後の単位照合と完了時の全体照合は `commit` に従う。
|
|
129
129
|
|
|
130
|
-
予定単位の内容・分割・統合を変更したら、変更した単位を再判定して必要なレビュー・許可確認・検証へ戻す。設定変更時も影響する未実行単位を再判定する。以前の判定や別単位の方針は流用しない。manual
|
|
130
|
+
予定単位の内容・分割・統合を変更したら、変更した単位を再判定して必要なレビュー・許可確認・検証へ戻す。設定変更時も影響する未実行単位を再判定する。以前の判定や別単位の方針は流用しない。manualで人の修正・依頼を経てcommitした場合は `commit` に従い、commit後に目的・範囲・実効flowと影響する挙動を確認する。正当な修正は元の予定との差と対応を残して確定単位へ反映する。不明・対象外の変更、確定単位との不一致は停止し、履歴を無断で書き換えない。
|
|
131
131
|
|
|
132
132
|
### config_version 1 からの移行
|
|
133
133
|
|
|
@@ -28,7 +28,7 @@ Claudeのメインは途中の新しい根拠で必要な担当を増やし、
|
|
|
28
28
|
|
|
29
29
|
原則、確定した順序で一つずつ実行する。Issue単体の依頼では完了後に別Issueへ進まない。複数対象では、現在のIssueのQA・必要な人の対応・コミットを残して次へ切り替えない。
|
|
30
30
|
|
|
31
|
-
- 実装担当の作業中や、manual
|
|
31
|
+
- 実装担当の作業中や、manualの人のレビュー・コミットの依頼・Human QA・本人認証・通常の承認待ちは必要な工程として扱い、必要な依頼を行って会話側のゴールを `active` のまま待機する。回答がないことや待ち時間をブロッカー到達・承認・完了と扱わない。
|
|
32
32
|
- 観測結果が得られないことだけでは終了条件を満たさない。同じsessionの状態を確認して待機を続け、新しいsessionやゴールを作る根拠にしない。
|
|
33
33
|
- QAで見つけた不具合は[共通ルール](workflow.md#失敗・許可・停止の共通ルール)に従って実装担当へ戻し、再検証する。テストの失敗だけでバックログ実行を打ち切らない。
|
|
34
34
|
- 未決の仕様判断、許可不足、対象外の依存、外部障害など、現在の許可と対象内では解消できない進行不能をブロッカーとする。
|
|
@@ -5,8 +5,8 @@
|
|
|
5
5
|
| 担当 | 責務 |
|
|
6
6
|
| --- | --- |
|
|
7
7
|
| Codex（進行役） | Issueの作成・整理、仕様相談、Ready判定、依存・Cycle管理、進行判断、作業先設定からの経路と許可の確定、体制判断の材料（受入条件・関連範囲・既存検証・リスク）の提示、Claudeへの依頼（auto: CLI、manual: Bridge）、Local・Staging・ProductionのAgent QA、単位・全体の照合、許可されたpush・PR・merge・配備・公開、Issueへの記録、人への依頼 |
|
|
8
|
-
| Claude（実装担当・Claude Code） | 初期体制の決定と途中の増員、`shinaps:dev-claude` による技術調査・計画・実装・検証・独立レビュー、QA指摘の修正、`shinaps:commit`（auto: 独立レビュー後にcommit、manual:
|
|
9
|
-
| 人 | 未合意の仕様・権限の判断、manual
|
|
8
|
+
| Claude（実装担当・Claude Code） | 初期体制の決定と途中の増員、`shinaps:dev-claude` による技術調査・計画・実装・検証・独立レビュー、QA指摘の修正、`shinaps:commit`（auto: 独立レビュー後にcommit、manual: 人の開始依頼で分割案、人が依頼した単位ごとにcommit）、結果の報告。実装を伴わない文書整理は、作業先リポジトリの規約に従いdevの入口の対象外にできる。スキル・コードの変更は文書の形でも実装としてdevの入口を使う |
|
|
9
|
+
| 人 | 未合意の仕様・権限の判断、manualのWebStormでの差分レビュー・修正相談・コミットの開始と単位ごとの依頼、本人の操作が必要な認証等、必要なHuman QA |
|
|
10
10
|
|
|
11
11
|
経路は着手時に通常設定の `review_flow.mode` で決まる（[経路](configuration.md#経路)）。autoはCodexが[ローカルClaude CLI](claude-cli.md)の同じsessionをresumeし、manualはWebStorm内の対話Claude Codeセッションへ[Bridge](bridge.md)で依頼する。どちらでもClaudeはdev内レビューと検証を終えて結果を返し、CodexがLocal Agent QAを行う。Claudeの報告をQA合格の根拠にせず、この引き渡しを進行全体の終了と扱わない。
|
|
12
12
|
|
|
@@ -23,8 +23,8 @@ Linear MCPで扱えない画面操作（添付、並べ替え、Cycleへの紐
|
|
|
23
23
|
| 工程 | 状態キー | 内容と次へ進む条件 |
|
|
24
24
|
| --- | --- | --- |
|
|
25
25
|
| 着手 | `ready` | Readyと依存を確認し、作業先設定を読み、通常設定の `review_flow` から経路と必要な許可（auto: `claude_cli`、manual: `bridge_request`）を確定する。checkout・branch・HEAD・既存差分を固定し、体制判断の材料を添えてClaudeへ依頼する。通常設定がautoでmanualのパス別ルールがあれば、起こり得る停止（[経路](configuration.md#経路)）を依頼に書く |
|
|
26
|
-
| 実装 | `coding`, `local_agent_qa` | Claudeのメインが初期体制を決めて報告し、`shinaps:dev-claude` で実装・検証・独立レビューを行い、未コミット差分・単位案・予定差分・判定記録を返す。CodexがLocal Agent QA（判定記録と経路の整合を含む）を行い、不合格なら同じsessionへ戻す |
|
|
27
|
-
| 差分確定 | `code_review` | QA合格後、Codexが実効flow・判定記録・QA結果を同じsessionへ渡し、Claudeが `shinaps:commit` を行う。autoは単位ごとに独立レビュー後にcommitする。manual
|
|
26
|
+
| 実装 | `coding`, `local_agent_qa` | Claudeのメインが初期体制を決めて報告し、`shinaps:dev-claude` で実装・検証・独立レビューを行い、未コミット差分・単位案・予定差分・判定記録を返す。manualでは人へユーザーのレビュー待ちと明示する。CodexがLocal Agent QA（判定記録と経路の整合を含む）を行い、不合格なら同じsessionへ戻す |
|
|
27
|
+
| 差分確定 | `code_review` | QA合格後、Codexが実効flow・判定記録・QA結果を同じsessionへ渡し、Claudeが `shinaps:commit` を行う。autoは単位ごとに独立レビュー後にcommitする。manualではCodexはcommitを依頼せず、渡した後は報告を待つ。人のコミット開始依頼でClaudeが分割案を示し、人が単位ごとに依頼したcommitを行う。Codexは各コミットの単位照合と全体照合を行う。差分確定中の修正は単位の目的・範囲内に限り、超えるものは実装へ戻す。差分確定で差分が変わったIssueは、確定後にCodexのAgent QAを必須にする（公開確認のpush前の最終Agent QAで兼ねてよい） |
|
|
28
28
|
| 公開確認 | `post_commit`, `staging_agent_qa`, `staging_human_qa`, `release`, `production_agent_qa`, `production_human_qa` | 受入条件と許可に応じて必要なものだけ行う: push前の最終Agent QA、push・PR作成、環境配備とStaging/ProductionのAgent QA、必要なHuman QA、merge・配備・公開。適用しない項目は理由を記録する |
|
|
29
29
|
| 結果報告 | （状態キーなし） | 結果・証拠・省略した工程と理由・残項目をIssueへ記録し、終了地点に到達する。必須受入条件をすべて満たしたIssueは `done` にする |
|
|
30
30
|
|
|
@@ -60,7 +60,7 @@ Linear MCPで扱えない画面操作（添付、並べ替え、Cycleへの紐
|
|
|
60
60
|
| Codexから許可範囲と受入条件を伴う実装・修正の依頼を受けた | Claudeは人へ着手を再確認せず、指定されたスキルで実行する |
|
|
61
61
|
| devの入口の中で計画上の判断が必要になった | autoはClaudeがresultでCodexへ相談し、Codexは既存の合意で判断できれば回答する。manualはClaudeが人へ直接確認し、決定を報告する |
|
|
62
62
|
| QAで受入条件の範囲内の不具合が見つかった | Codexが同じClaude sessionへ修正を依頼し、Claudeは人の着手許可を改めて取らずに修正する。Codexが再検証する |
|
|
63
|
-
| ClaudeがCLIのresultでcommitの完了を、またはBridge
|
|
63
|
+
| ClaudeがCLIのresultでcommitの完了を、またはBridgeで人の依頼によるcommitの完了を報告した | Codexは人に同じ事実を聞き直さず、報告の対象とGitの状態を照合して次の工程へ進む |
|
|
64
64
|
| 次の操作の継続的な許可があり、必要な工程の結果が揃った | Codexは「進めてもよいか」を再質問せずに実行する |
|
|
65
65
|
| Cycle全体を任され、対象Issueが設定された終了地点に達した | Codexは開始時に確定した対象と順序を照合し、その範囲の次のIssueへ進む |
|
|
66
66
|
|
|
@@ -72,9 +72,9 @@ Linear MCPで扱えない画面操作（添付、並べ替え、Cycleへの紐
|
|
|
72
72
|
|
|
73
73
|
1. Claudeは成果引き渡しに、モードによらず自然な責務で決めたコミット単位案と予定差分を含める。CodexはLocal Agent QAで対象を検証し、単位案が依頼全体を漏れなく覆い対象外変更を含まないことと、各単位の[判定記録](configuration.md#判定記録と変更時の扱い)を確認する。依頼の比較元と全体差分も最終照合用に記録する。判定記録に経路と異なるmanual単位があれば、差分確定の依頼前に[経路](configuration.md#経路)に従って人へ確認する。
|
|
74
74
|
2. QA合格後、Codexは実効flow・単位の判定記録・QA結果を同じClaude sessionへ渡す。Claudeは設定・予定差分・合意根拠を照合する。
|
|
75
|
-
3. Claudeは `commit` に従う。autoでは独立レビュー後にClaudeがstage・commitする。manual
|
|
75
|
+
3. Claudeは `commit` に従う。autoでは独立レビュー後にClaudeがstage・commitする。manualでは人のコミット開始依頼で分割案を示し、人がその単位のコミットを依頼したら単位外を安全に退避して単位をstage・検証し、Claudeがcommitする。各コミット直前・作成後の実差分と確定した予定単位を照合し、予定単位・SHA・検証・レビュー・依頼記録・復元結果をCodexへ返す。
|
|
76
76
|
4. 全単位完了後、Codexは各コミットの単位照合結果を実差分で独立確認したうえで、依頼の比較元からの全体差分が、`commit` の手順で正当な人の修正を反映した確定済みの全体期待差分と一致し、対象の未コミット変更が残っていないことを照合する。最終差分だけでは途中で混入し後で相殺された対象外変更を検出できないため、単位照合を省かない。人レビューを省略した単位は、上の省略記録を完成させる。
|
|
77
|
-
5. 単位の組み替え・内容・設定の変更は[変更時の扱い](configuration.md#判定記録と変更時の扱い)
|
|
77
|
+
5. 単位の組み替え・内容・設定の変更は[変更時の扱い](configuration.md#判定記録と変更時の扱い)に従って再判定し、必要なレビュー・許可確認・検証へ戻す。人の依頼によるcommit（または人自身のcommit）の後は、`commit` に従って正当な修正と元の予定差分の対応を記録し、確定単位に更新する。確定単位と実コミットに不一致があれば次工程を止め、SHA・予定差分・実差分の違いを記録して人に確認する。履歴を無断で書き換えない。
|
|
78
78
|
|
|
79
79
|
## 待ち状態
|
|
80
80
|
|
|
@@ -90,13 +90,13 @@ manual経路の依頼・報告と、別会話との任意の進行の引き渡
|
|
|
90
90
|
|
|
91
91
|
## 人への依頼
|
|
92
92
|
|
|
93
|
-
manual
|
|
93
|
+
manualの差分レビュー・相談・コミットの依頼は、WebStormの差分表示と対話Claude Codeセッションで[manualのレビュー待ちと単位ごとのコミット](../human-review.md)に従って行う。
|
|
94
94
|
|
|
95
95
|
会話でIssueリンクと必要な作業を明示し、ステータスの変更だけで依頼を済ませない。
|
|
96
96
|
|
|
97
97
|
| 依頼 | 文例 |
|
|
98
98
|
| --- | --- |
|
|
99
|
-
| manual
|
|
99
|
+
| manualのレビュー | できるようになったことと重点確認点を添えて「[Issueリンク]の差分をWebStormでレビューし、コミットはWebStormのClaude Codeセッションで依頼してください。」 |
|
|
100
100
|
| Human QA | 「[Issueリンク]のHuman QAをお願いします。」 |
|
|
101
101
|
| 外部対応 | 「[Issueリンク]の〇〇をお願いします。」 |
|
|
102
102
|
|
|
@@ -22,11 +22,11 @@ description: 許可された差分を依存順のローカルコミットにし
|
|
|
22
22
|
|
|
23
23
|
## 範囲と進め方
|
|
24
24
|
|
|
25
|
-
コミットは利用者の明示依頼、または[Linear共通設定](../../references/linear/configuration.md)の合意済みの許可に従う。Linear進行では[担当と工程](../../references/linear/workflow.md)の差分確定に従い、Local Agent QA
|
|
25
|
+
コミットは利用者の明示依頼、または[Linear共通設定](../../references/linear/configuration.md)の合意済みの許可に従う。Linear進行では[担当と工程](../../references/linear/workflow.md)の差分確定に従い、Local Agent QA後に進行役から対象を指定した依頼を受けて行う。manualでは進行役はcommitを依頼せず、QA結果と判定記録の引き渡しの後に人の依頼で行う。単独の依頼にはLinear進行や新しいゴールを必須にしない。
|
|
26
26
|
|
|
27
27
|
どちらのモードも、許可された対象以外の変更と秘密情報、push、PR、公開、既存履歴の書き換えを含まない。
|
|
28
28
|
|
|
29
|
-
Git準備・修正・commitはAIが現在のセッションで行い、manual
|
|
29
|
+
Git準備・修正・commitはAIが現在のセッションで行い、manualのレビューとコミットの依頼は人が行う。autoは独立レビューを必須とする。必要な担当・許可が利用できなければ、セルフレビューを同等と称さず止める。別worktreeの作成や別セッションへの委託を、このスキルから許可しない。
|
|
30
30
|
|
|
31
31
|
## 回復できる状態を作る
|
|
32
32
|
|
|
@@ -43,23 +43,23 @@ Git準備・修正・commitはAIが現在のセッションで行い、manualの
|
|
|
43
43
|
|
|
44
44
|
予定単位ごとに、[パス別の方針](../../references/linear/configuration.md#パス別の方針)でそのコミットの全変更を判定し、同資料の判定記録を残す。Linear進行から記録を受け取った場合は、設定とその単位の予定差分（内容・mode等を含む）の一致を確認して引き継ぐ。単独の依頼でも同じ判定を使い、Linear専用の工程は追加しない。設定が真にない単独依頼では、今回の明示許可とプロジェクト規約を通常方針とし、パス例外は適用しない。
|
|
45
45
|
|
|
46
|
-
合意済みの実効 `review_flow` と今回の明示指示から進め方を確定する。両方とも対象のstageとAIのcommitを許可し、`auto` では独立レビュー後、`manual`
|
|
46
|
+
合意済みの実効 `review_flow` と今回の明示指示から進め方を確定する。両方とも対象のstageとAIのcommitを許可し、`auto` では独立レビュー後、`manual` では人がその単位のコミットを依頼した後にAIがcommitする。設定の存在だけを合意と扱わない。旧キー・不備・競合は共通設定に従って止める。
|
|
47
47
|
|
|
48
|
-
設定がない単独依頼でも「各単位をAI
|
|
48
|
+
設定がない単独依頼でも「各単位をAIがレビューして自動コミット」「人がレビューしてから単位ごとにコミット」など明確な意図を引き継ぐ。「コミットして」「進めて」だけからautoを推測しない。flowが未解決のときだけ対象を示し「auto（AIがレビュー・コミット）とmanual（人がレビューして単位ごとにコミットを依頼し、AIが修正・コミット）のどちらで進めますか？」と一問聞き、回答まで依存操作を待つ。未回答を承認と扱わない。同じ依頼内でもパス例外により異なるflowの単位があってよい。
|
|
49
49
|
|
|
50
50
|
モード決定後にレビュー可能性や依存関係のため単位を分割・統合する場合は、[変更時の扱い](../../references/linear/configuration.md#判定記録と変更時の扱い)に従って変更した単位を再判定し、必要なレビュー・許可確認・検証へ戻す。設定変更時も影響する未実行単位を再判定する。「autoだからまとめる」「manualだから分ける」を先に適用して単位決定と判定を循環させない。
|
|
51
51
|
|
|
52
52
|
## 単位ごとの検証・レビュー・コミット
|
|
53
53
|
|
|
54
|
-
manualでは[manual
|
|
54
|
+
manualでは[manualのレビュー待ちと単位ごとのコミット](../../references/human-review.md)に従い、人と対話できるセッションで人の依頼を受ける。手順1は人のコミット開始依頼の後に、手順2以降は人がその単位のコミットを依頼した後に行う。ローカルClaude CLIの非対話実行でmanualの単位を検出した場合は、手順を進めず、判定を記録してresultへ返す（[経路](../../references/linear/configuration.md#経路)）。
|
|
55
55
|
|
|
56
56
|
以下を、その単位のモードで行う。
|
|
57
57
|
|
|
58
58
|
1. 単位案を短く示す。manualで複数の単位に分ける場合、許可された独立レビュアーがいれば、各単位の差分だけで人が妥当性を判断できるかを差分自体から確認させ、結論はメインが実際の差分で検証する。autoでは単位案の独立レビューを求めない（プロジェクトの必須条件にある場合は従う）。
|
|
59
59
|
2. コミットと検証の内容が単位だけで決まるようにする。manualでは、単位以外の未コミット変更（対象外、後続の単位、untracked）を一意な名前のstashへ退避し、作業ツリーとindexに単位の変更だけを残す。autoでは、対象外の変更を作業ツリーとindexに残してよく、検証結果が残る変更に依存しないことを根拠（例: 残る変更が検証の入力に含まれない）とともに判断する。依存する、または判断できない場合は、許可された範囲で単位をまとめ直すか、manualと同じ方法で退避する。退避が他の進行中の作業（他のセッションや人がステージ・編集中の変更）を妨げる場合は退避せず、所有者と調整できるまで止める。manualは別の隔離方法へ置き換えない。退避するときは、stashのindexにコミット対象のステージ済み変更を入れない。`git stash push` はパスを指定してもindex全体を保存し、対象のステージ済み変更が入ると後の `git stash apply --index` が失敗するため。対象のステージ済み差分のパッチを保存し、対象のステージを外してから退避し、退避後にそのパッチで対象のステージを戻す。作成前後のstash一覧を比べて今回作成されたものだけのSHAを記録し、既存stashの先頭を自分のものと決めつけない。共有ファイルのhunk分割は控えから行い、対象のhunkだけのステージを外せない場合や、対象外のステージ済みの変更が変わる場合は止める。
|
|
60
60
|
3. コミットする内容に必要な検証を行う。後続や対象外の未コミット変更に依存した成功を、その単位の検証としない。範囲を限った検証はプロジェクトが認める場合だけ使い、必須検証を省かない。後続や対象外への依存で失敗するなら、手順2に戻って単位か隔離を見直す。
|
|
61
|
-
4. 対象だけをステージし、実際にコミットされる差分の全体（indexのままコミットする場合は完全なcached diff）と影響するファイルをレビューする。indexに対象外の変更を残す場合は、それを含めずに対象だけをコミットする方法を使い、対象を限定できなければ止める。manual
|
|
62
|
-
5. 必須検証と独立レビュー（auto必須）が通り、阻害指摘がないことを確認する。autoでは直前の全差分が予定単位と一致してからAIが `git commit` を単独実行する（`git add` と連結しない）。manual
|
|
61
|
+
4. 対象だけをステージし、実際にコミットされる差分の全体（indexのままコミットする場合は完全なcached diff）と影響するファイルをレビューする。indexに対象外の変更を残す場合は、それを含めずに対象だけをコミットする方法を使い、対象を限定できなければ止める。manualで人がWebStormで確認できるstaged差分は、コミットされる差分と同じものにする。指摘は実物で確認し、依頼範囲の修正後は再ステージ・再検証する。外部仕様の調査は変更がそこに依存するときに行う。
|
|
62
|
+
5. 必須検証と独立レビュー（auto必須）が通り、阻害指摘がないことを確認する。autoでは直前の全差分が予定単位と一致してからAIが `git commit` を単独実行する（`git add` と連結しない）。manualではstage時の `git write-tree` を記録し、単位名・目的・検証結果・推奨メッセージを短く示す（差分本文は貼らない）。記録時の差分が分割案の予定単位と違えばcommitせず報告して依頼を取り直す。直前のindexのtreeが記録したtreeと一致することを確認してAIが `git commit` を単独実行し、作成したコミットのtreeも照合する。treeの記録後に単位のパスで作業ツリーまたはindexが変わったら（人との相談による修正を含む）依頼は失効し、再stage・再検証・再提示して改めて人の依頼を受ける。依頼はその単位にだけ及ぶ。commit後に（人自身のcommitを含む）、比較元からの実コミット一覧と各SHA・全差分を調べ、相談で行った修正が許可された目的・範囲を満たすことを確認する。実差分で実効flowを再判定し、影響する挙動を再検証する。正当な人の修正は確定単位と全体の期待差分へ反映し、元の予定差分・依頼記録・変更理由・実コミットとの対応を残す。意図や範囲が不明、対象外の混入、flowの適用が未解決なら人へ確認して止める。Gitにcommitがあるだけで人レビュー完了としない。各コミットを確定単位と照合してSHA・結果を記録し、不一致なら次工程を止める。履歴を無断で書き換えない。
|
|
63
63
|
6. 退避した場合は、コミット後または中断時に、indexにコミット対象のステージ済み変更がない状態で、記録したSHAのstashを `git stash apply --index` で戻す。中断時は対象のステージを外し、stashの復元を妨げる対象の未コミット変更（共有ファイルの対象のhunkなど）もパッチに保存して作業ツリーから一時的に外してから戻す。その後、外した対象の変更を戻し、対象のステージ済み差分のパッチで対象のステージを戻す。復元前に人が修正した最新のHEAD・作業ツリー・indexを記録して保全する。控えで扱った対象外変更だけを復元し、古い対象パッチで人の修正を上書きしたり、コミット済みの変更を再び加えたりしない。退避の有無によらず、コミットしていない変更（ステージ済み・未ステージ・untracked）と既存stashが、操作前の記録と確定済みの人の修正からコミットした内容を除いたものと一致することを確認する。一致しない、曖昧な競合がある、ステージ境界を戻せない場合は推測で消さず、一致しないパスと、退避した場合は回復資料を示して止める。一致を確認してから今回作成したstashだけをdropし、HEADを確認して次へ進む。
|
|
64
64
|
|
|
65
65
|
全単位の完了後、各単位の照合結果に加え、元の比較元からの全体差分が、人の正当な修正を反映した確定済みの全体期待差分と一致し、対象の未コミット変更が残っていないことを確認する。最終差分だけで途中コミットの照合を代用しない。予定どおりのコミットによる残差分の減少は不一致と扱わない。
|
|
@@ -12,7 +12,7 @@ Issueの責務・受入条件・ステータスを、実際の成果と一致さ
|
|
|
12
12
|
着手時に作業先設定の通常の `review_flow.mode` で経路を決め、Issueの途中で切り替えない（[経路](../../references/linear/configuration.md#経路)）。
|
|
13
13
|
|
|
14
14
|
- auto: [ローカルClaude CLI](../../references/linear/claude-cli.md)を起動し、質問への回答・修正・差分確定を同じsessionのresumeで依頼する。許可は `claude_cli`。
|
|
15
|
-
- manual: WebStorm内の対話Claude Codeセッションへ[Bridge](../../references/linear/bridge.md)で依頼する。人とClaudeの作業中は同じIssue
|
|
15
|
+
- manual: WebStorm内の対話Claude Codeセッションへ[Bridge](../../references/linear/bridge.md)で依頼する。人とClaudeの作業中は同じIssueへ指示を送らず、報告を待つ。QA合格後もcommitを依頼せず、QA結果と判定記録を渡して人の依頼によるcommitの報告を待つ。許可は `bridge_request`。
|
|
16
16
|
|
|
17
17
|
体制判断の材料（受入条件・関連範囲・既存検証・リスク）を依頼に添える。初期体制はClaudeのメインが決めて報告する。
|
|
18
18
|
|
|
@@ -22,7 +22,7 @@ Issueの責務・受入条件・ステータスを、実際の成果と一致さ
|
|
|
22
22
|
|
|
23
23
|
開発進行には作業先の明示された設定と `issue_update`、経路の許可が必要（[configuration.md](../../references/linear/configuration.md) の「許可」）。設定の許可・`stop_after` を超えない。Issue未作成の明示的な実装依頼は共通の紐づけ手順でIssueを作って続けられるが、仕様が未決なら `shinaps:linear-plan` へ整理を戻す。
|
|
24
24
|
|
|
25
|
-
実行には対象と終了条件を持つゴールの設定成功が必須。環境が明示許可を要求する場合は、既存の依頼で満たされるか確認し、なければ開始前に一度確認する。ゴール未設定のまま実装へ進まない。Claudeの作業・manual
|
|
25
|
+
実行には対象と終了条件を持つゴールの設定成功が必須。環境が明示許可を要求する場合は、既存の依頼で満たされるか確認し、なければ開始前に一度確認する。ゴール未設定のまま実装へ進まない。Claudeの作業・manualの人のレビュー・コミットの依頼・Human QA待ちは予定された工程として待機する。スキルの実行だけで予約実行・定期監視は開始しない。人への確認は会話で一問ずつ行う。
|
|
26
26
|
|
|
27
27
|
## 設定と補足を読む
|
|
28
28
|
|
|
@@ -42,4 +42,4 @@ Issueの責務・受入条件・ステータスを、実際の成果と一致さ
|
|
|
42
42
|
| QAの実施・記録・素材 | [references/qa.md](../../references/linear/qa.md) |
|
|
43
43
|
| Cycle完了QA・報告 | [references/cycle-report.md](../../references/linear/cycle-report.md) |
|
|
44
44
|
|
|
45
|
-
コミットはClaudeへ差分確定を依頼して `shinaps:commit`
|
|
45
|
+
コミットはClaudeへ差分確定を依頼して `shinaps:commit` で行い（manualは上の経路の項に従う）、Codexは各コミットの単位照合と全体照合を行う。pushとPRの作成は `shinaps:create-pr`、公開は `shinaps:release` に従い、このスキルで手順を持たない。これらのスキルには[共通設定](../../references/linear/configuration.md)の許可と前工程の結果を渡す。必要な条件が揃えば再確認せずに進み、個別スキルの完了だけで進行を終えず、[終了条件](../../references/linear/execution.md)まで継続する。
|