Skip to content

fix: 結合観点で見つかった 3 ランタイム間の記述の食い違いを解消 - #77

Merged
takemi-ohama merged 1 commit into
release/skill-inventoryfrom
fix/release-integration-consistency
Aug 8, 2026
Merged

fix: 結合観点で見つかった 3 ランタイム間の記述の食い違いを解消#77
takemi-ohama merged 1 commit into
release/skill-inventoryfrom
fix/release-integration-consistency

Conversation

@takemi-ohama

Copy link
Copy Markdown
Contributor

Summary

release PR #66 の結合観点レビューで、個別 PR では見えなかった不整合が 3 件見つかったため修正する。

箇所内容
plugins/ndf-claude/.claude-plugin/plugin.jsondescription.claude-plugin/marketplace.json と一致させた。Task 0-10 で marketplace 側だけ更新しており、同じプラグインの説明が 2 通り存在していた
plugins/ndf-codex/.codex-plugin/plugin.jsondescription に版数と Skill 数を入れ、Playwright testingbrowser automation の表現を Claude 側と統一した
docs/presentations/2026-08-06-ai-plugins-intro.mdKiro 導入結果を 21 個 → 20 個へ修正。ndf-policies.kiro/skills/ へリンクせず .kiro/steering/ へ配置するため、実際に張られるリンクは 20 個になる

Skill 数の修正は推測ではなく、install.sh を一時ディレクトリで実行して実測した。

$ bash plugins/ndf-kiro/install.sh --project "$TD"
$ ls "$TD/.kiro/skills" | wc -l
20
$ ls "$TD/.kiro/skills" | grep -c ndf-policies
0
$ test -f "$TD/.kiro/steering/ndf-policies.md" && echo ok
ok

スライドの HTML と PDF も再生成した。

Test plan

  • marketplace.jsonplugins/ndf-claude/.claude-plugin/plugin.jsondescription が一致する
  • Kiro の install.sh が実際に張るリンク数が資料の記載と一致する(実測 20)
  • bash scripts/build-runtime-plugins.sh --check が差異を検出しない
  • python3 scripts/check-markdown-links.py --root . が成功する
  • bash scripts/validate-runtime-plugins.sh が成功する
  • bash scripts/runtime-smoke-test.sh が 3 ランタイムで成功する

release PR: #66

release ブランチ全体の cross-review で、個別 PR では見えなかった不整合が
3 件見つかった。
- plugins/ndf-claude/.claude-plugin/plugin.json の description を
.claude-plugin/marketplace.json と一致させた。0-10 で marketplace 側だけ
更新しており、同じプラグインの説明が 2 通り存在していた
- plugins/ndf-codex/.codex-plugin/plugin.json の description も版数と
Skill 数を含む形へ揃え、Playwright testing → browser automation の表現を
統一した
- プレゼン資料の Kiro 導入結果を 21 個 → 20 個へ修正。ndf-policies は
.kiro/skills/ へリンクせず .kiro/steering/ へ配置するため、実際に張られる
リンクは 20 個になる(install.sh を一時ディレクトリで実行して実測)

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 1 | codex | APPROVE

修正必須の指摘はありません。

@takemi-ohamatakemi-ohama left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

🤖 cross-review | round 1 | gemini | COMMENT

本 PR の変更範囲外ですが、3 ランタイム間の記述の整合性および実測値との一致の観点で 1 点報告します。
plugins/ndf-kiro/README.md の 156行目に 配布物(Skill 23 個)を導入しています。 という記載が残っています。マニフェストおよび実測では Kiro の Skill 数は 21個(.kiro/skills/ に 20個、steering に 1個)であるため、不一致が発生しています。次回以降の更新等で合わせて修正することを推奨します。

Comment threadplugins/ndf-claude/.claude-plugin/plugin.json
@takemi-ohama
takemi-ohama merged commit 3dce0bc into release/skill-inventoryAug 8, 2026
7 checks passed
@takemi-ohama
takemi-ohama deleted the fix/release-integration-consistency branch August 8, 2026 06:50
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@takemi-ohama