Uh oh!
There was an error while loading. Please reload this page.
fix: 配布物の説明と実装の食い違いを解消 - #79
Merged
Merged
Conversation
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 を起動しようとするのを防ぐ
takemi-ohama
commented
Aug 8, 2026
takemi-ohama
left a comment
ContributorAuthor
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | APPROVE
修正が必要な指摘はありません。ドキュメントの変更内容は manifest、生成物、検証コマンドの結果と整合しています。
takemi-ohama
commented
Aug 8, 2026
takemi-ohama
left a comment
ContributorAuthor
There was a problem hiding this comment.
🤖 cross-review | round 1 | gemini | APPROVE
実装状態（各ランタイムの配備ファイルとフロントマター指定）とドキュメントの記述が正確に一致していることを確認しました。問題ありません。
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
release PR #66 の結合観点レビューで、個別 PR のマージ順に起因する記述の古さが 3 件見つかったため修正する。
plugins/ndf-codex/README.mdcherry-pick-prとdeployの 2 個mergedprpr-testsreviewを含む）plugins/ndf-kiro/README.md.kiro/skills/に並ぶのは 20 個ndf-policiesの移行対応表playwright-planning/playwright-evidenceは Codex にのみ配布1 件目は Task 0-8 が README を書いた時点では 6 個だったが、その後 Task 0-7 が
merged/pr/pr-tests/reviewからdisable-model-invocationを外したことによる。4 個を暗黙起動可にした理由（日常的に自然文で依頼されるため）と、代わりに取り消しの難しい手順の直前に同意を得ることを本文で必須化した点もあわせて書いた。3 件目は
ndf-policiesが 3 ランタイム共通で常時注入されるため、Claude / Kiro 環境で存在しない Skill を起動しようとするのを防ぐ。Test plan
grep -l 'disable-model-invocation: true' plugins/ndf-shared/skills/*/SKILL.mdの結果と Codex README の一覧が一致するbash scripts/build-runtime-plugins.sh --checkが差異を検出しないpython3 scripts/check-skill-frontmatter.pyがエラー 0 / 警告 0python3 scripts/check-markdown-links.py --root .が成功するbash scripts/validate-runtime-plugins.shが成功するrelease PR: #66