From 032c3bd07aadfa38e30743a832162ae50a79db0d Mon Sep 17 00:00:00 2001 From: "takemi.ohama" Date: Sat, 8 Aug 2026 07:02:17 +0000 Subject: [PATCH] =?UTF-8?q?Fix:=20=E9=85=8D=E5=B8=83=E7=89=A9=E3=81=AE?= =?UTF-8?q?=E8=AA=AC=E6=98=8E=E3=81=A8=E5=AE=9F=E8=A3=85=E3=81=AE=E9=A3=9F?= =?UTF-8?q?=E3=81=84=E9=81=95=E3=81=84=E3=82=92=E8=A7=A3=E6=B6=88?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit release ブランチ全体の cross-review で、個別 PR のマージ順に起因する記述の 古さが 3 件見つかった。 - plugins/ndf-codex/README.md: 暗黙起動を抑止する Skill を 6 個と書いていたが、 Task 0-7 で merged / pr / pr-tests / review から disable-model-invocation を 外したため、実際の生成対象は cherry-pick-pr と deploy の 2 個。4 個を許可へ 変えた理由と、代わりに実行前確認を必須化したことも明記した - plugins/ndf-kiro/README.md: 占有率の測定条件を「Skill 23 個」から、Kiro manifest の 21 個のうち .kiro/skills/ に並ぶのは 20 個（ndf-policies は steering へ回す）へ修正した - ndf-policies の移行対応表: playwright-planning と playwright-evidence は Codex にのみ配布しているため、その旨を注記した。ndf-policies は 3 ランタイム 共通なので、Claude / Kiro で存在しない Skill を起動しようとするのを防ぐ --- plugins/ndf-claude/skills/ndf-policies/SKILL.md | 4 ++-- plugins/ndf-codex/README.md | 8 +++----- plugins/ndf-codex/skills/ndf-policies/SKILL.md | 4 ++-- plugins/ndf-kiro/README.md | 2 +- plugins/ndf-kiro/skills/ndf-policies/SKILL.md | 4 ++-- plugins/ndf-shared/skills/ndf-policies/SKILL.md | 4 ++-- 6 files changed, 12 insertions(+), 14 deletions(-) diff --git a/plugins/ndf-claude/skills/ndf-policies/SKILL.md b/plugins/ndf-claude/skills/ndf-policies/SKILL.md index 176736e2..775032d3 100644 --- a/plugins/ndf-claude/skills/ndf-policies/SKILL.md +++ b/plugins/ndf-claude/skills/ndf-policies/SKILL.md @@ -32,9 +32,9 @@ Skill の棚卸で 49 個を 29 個へ整理した。旧コマンドは存在し | `/ndf:sync-main` | `/ndf:merged` | | `/ndf:branch-fix-strategy` | `/ndf:cherry-pick-pr`（原則は本 Skill の「ブランチ運用の原則」） | | `/ndf:codex` `/ndf:gemini` | `/ndf:external-ai` | -| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning` | +| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning`（Codex のみ配布） | | `/ndf:playwright-script-creation` `/ndf:playwright-execution` `/ndf:browser-test` `/ndf:playwright-browser-connect` | `/ndf:playwright-authoring` | -| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence` | +| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence`（Codex のみ配布） | 移行先を用意せず削除したもの（いずれも起動実績がなく、現在のモデルの標準能力か 汎用コマンドで足りる）: diff --git a/plugins/ndf-codex/README.md b/plugins/ndf-codex/README.md index 63313731..8b031502 100644 --- a/plugins/ndf-codex/README.md +++ b/plugins/ndf-codex/README.md @@ -22,16 +22,14 @@ Claude Code 専用の agents、statusline 自動設定、transcript retention ## 暗黙起動を抑止する Skill -破壊的操作を伴う以下 6 個の Skill は、`skills//agents/openai.yaml` の `policy.allow_implicit_invocation: false` によって **Codex の暗黙起動 (モデルが自分で選んで起動する経路) を抑止**しています。共有 Skill の frontmatter が `disable-model-invocation: true` のものが対象で、`scripts/build-runtime-plugins.sh` が自動生成します。 +取り消しが難しい以下 2 個の Skill は、`skills//agents/openai.yaml` の `policy.allow_implicit_invocation: false` によって **Codex の暗黙起動 (モデルが自分で選んで起動する経路) を抑止**しています。共有 Skill の frontmatter が `disable-model-invocation: true` のものが対象で、`scripts/build-runtime-plugins.sh` が自動生成します。 | Skill | 内容 | |-------|------| | `cherry-pick-pr` | 環境ブランチへの cherry-pick PR 作成 | | `deploy` | 環境ブランチ (qa/staging, release/v2 等) への deploy PR 作成 | -| `merged` | PR マージ後のクリーンアップ | -| `pr` | commit / push / PR 作成・更新 | -| `pr-tests` | PR テストプランの実行と結果コメント | -| `review` | PR / ブランチ差分のレビューと approve・request-changes 投稿 | + +`merged` / `pr` / `pr-tests` / `review` は日常的に自然文で依頼されるため、v5.0.0 で暗黙起動を許可しました。代わりに、取り消しの難しい手順 (push、PR 作成、ブランチ・worktree の削除) の直前に対象を提示して同意を得ることを各 Skill の本文で必須化しています。 ### 利用者への影響と起動方法 diff --git a/plugins/ndf-codex/skills/ndf-policies/SKILL.md b/plugins/ndf-codex/skills/ndf-policies/SKILL.md index 176736e2..775032d3 100644 --- a/plugins/ndf-codex/skills/ndf-policies/SKILL.md +++ b/plugins/ndf-codex/skills/ndf-policies/SKILL.md @@ -32,9 +32,9 @@ Skill の棚卸で 49 個を 29 個へ整理した。旧コマンドは存在し | `/ndf:sync-main` | `/ndf:merged` | | `/ndf:branch-fix-strategy` | `/ndf:cherry-pick-pr`（原則は本 Skill の「ブランチ運用の原則」） | | `/ndf:codex` `/ndf:gemini` | `/ndf:external-ai` | -| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning` | +| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning`（Codex のみ配布） | | `/ndf:playwright-script-creation` `/ndf:playwright-execution` `/ndf:browser-test` `/ndf:playwright-browser-connect` | `/ndf:playwright-authoring` | -| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence` | +| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence`（Codex のみ配布） | 移行先を用意せず削除したもの（いずれも起動実績がなく、現在のモデルの標準能力か 汎用コマンドで足りる）: diff --git a/plugins/ndf-kiro/README.md b/plugins/ndf-kiro/README.md index 7d8feead..453e7c92 100644 --- a/plugins/ndf-kiro/README.md +++ b/plugins/ndf-kiro/README.md @@ -153,7 +153,7 @@ kiro-cli **2.16.1** / 検証日 **2026-08-07**（ランタイム規約の調査 | `kiro-cli agent set-default` の保存先 | `~/.local/share/kiro-cli/data.sqlite3`（マシン全体の設定） | 実行した cwd に `.kiro/settings.json` は生成されず、`find ~/.kiro ~/.aws` にも差分が出なかった | | 既定エージェントが cwd 依存で復旧できるか | 導入先から実行すれば復旧できる | 対象プロジェクト限定の workspace エージェントを既定にした状態では、別 cwd からの `set-default` が `No agent with name … found` になりつつ終了コード 0 を返し、既定が戻らなかった | -コンテキスト占有率を `kiro-cli chat --agent <名前> --no-interactive '/context show'` で実測しました。測定用プロジェクトには本リポジトリの `AGENTS.md` と `README.md` を置き、`install.sh --project <測定用ディレクトリ>` で配布物（Skill 23 個）を導入しています。`一致ファイル数` と `合計文字数` は `/context show` が列挙したファイルを数え上げた値、`占有率` は `Context files total` の表示値です。 +コンテキスト占有率を `kiro-cli chat --agent <名前> --no-interactive '/context show'` で実測しました。測定用プロジェクトには本リポジトリの `AGENTS.md` と `README.md` を置き、`install.sh --project <測定用ディレクトリ>` で配布物を導入しています（Kiro manifest の 21 個のうち、`ndf-policies` は `.kiro/steering/` へ回すため `.kiro/skills/` に並ぶのは 20 個）。`一致ファイル数` と `合計文字数` は `/context show` が列挙したファイルを数え上げた値、`占有率` は `Context files total` の表示値です。 | 構成 | 一致ファイル数 | `ndf-policies` の注入回数 | 占有率 | 文脈ファイルの合計文字数 | | --- | --- | --- | --- | --- | diff --git a/plugins/ndf-kiro/skills/ndf-policies/SKILL.md b/plugins/ndf-kiro/skills/ndf-policies/SKILL.md index 176736e2..775032d3 100644 --- a/plugins/ndf-kiro/skills/ndf-policies/SKILL.md +++ b/plugins/ndf-kiro/skills/ndf-policies/SKILL.md @@ -32,9 +32,9 @@ Skill の棚卸で 49 個を 29 個へ整理した。旧コマンドは存在し | `/ndf:sync-main` | `/ndf:merged` | | `/ndf:branch-fix-strategy` | `/ndf:cherry-pick-pr`（原則は本 Skill の「ブランチ運用の原則」） | | `/ndf:codex` `/ndf:gemini` | `/ndf:external-ai` | -| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning` | +| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning`（Codex のみ配布） | | `/ndf:playwright-script-creation` `/ndf:playwright-execution` `/ndf:browser-test` `/ndf:playwright-browser-connect` | `/ndf:playwright-authoring` | -| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence` | +| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence`（Codex のみ配布） | 移行先を用意せず削除したもの（いずれも起動実績がなく、現在のモデルの標準能力か 汎用コマンドで足りる）: diff --git a/plugins/ndf-shared/skills/ndf-policies/SKILL.md b/plugins/ndf-shared/skills/ndf-policies/SKILL.md index 176736e2..775032d3 100644 --- a/plugins/ndf-shared/skills/ndf-policies/SKILL.md +++ b/plugins/ndf-shared/skills/ndf-policies/SKILL.md @@ -32,9 +32,9 @@ Skill の棚卸で 49 個を 29 個へ整理した。旧コマンドは存在し | `/ndf:sync-main` | `/ndf:merged` | | `/ndf:branch-fix-strategy` | `/ndf:cherry-pick-pr`（原則は本 Skill の「ブランチ運用の原則」） | | `/ndf:codex` `/ndf:gemini` | `/ndf:external-ai` | -| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning` | +| `/ndf:playwright-test-planning` `/ndf:playwright-scenario-test` | `/ndf:playwright-planning`（Codex のみ配布） | | `/ndf:playwright-script-creation` `/ndf:playwright-execution` `/ndf:browser-test` `/ndf:playwright-browser-connect` | `/ndf:playwright-authoring` | -| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence` | +| `/ndf:playwright-report` `/ndf:playwright-evidence-drive` | `/ndf:playwright-evidence`（Codex のみ配布） | 移行先を用意せず削除したもの（いずれも起動実績がなく、現在のモデルの標準能力か 汎用コマンドで足りる）: