feat: ml-model-structure skill 追加 (ndf v4.10.0)#23
Conversation
- 新規 skill `ml-model-structure` を追加 (i19 登録依頼) - 機械学習モデルの版ごと自己完結構造 + 版内 feature SSoT + 推論コンテナ規約 - レビュー指摘を反映 (.dockerignore 文言の正確化 / 受け入れ基準に import 解決・docker代替を追加) - plugin.json: version 4.9.0 → 4.10.0、skills 配列追加、description を 45 skills に更新 - marketplace.json / README / AGENTS.md / ndf-plugin-reference: skill 数・version 表記を整合 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- README/AGENTS/ndf README/docs reference の skill 数・カテゴリ個数を実数45に修正 - 欠落していたカテゴリ (issue-plan-strategy / Playwright E2E 6個 / Google連携 2個 / AIクロスレビュー 2個 / skill-stats) を各ドキュメント本文へ追記し45個を網羅 - ndf README のセクション番号衝突 (### 5 重複) を解消し連番化 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | codex | REQUEST_CHANGES
docs/ndf-plugin-reference.md の「現行バージョン」が v4.7.0 のままです。PR で ndf を 4.10.0 に上げているため、README / marketplace / plugin.json と同じ版に揃えてください。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 1 | gemini | REQUEST_CHANGES
PR #23 における ml-model-structure スキルの追加と各ドキュメントの更新内容を確認しました。スキル定義自体は SageMaker や Docker の制約を考慮した非常に実用的な内容になっています。
一方で、v4.10.0 リリースにあたりドキュメント内のバージョン表記やエージェント一覧に不整合が残っているため、以下の修正を提案します。
- README.md: 冒頭の NDFプラグイン バージョン表記を v4.0.0 → v4.10.0 - docs/ndf-plugin-reference.md: 現行バージョンを v4.7.0 → v4.10.0、 ml-model-structure skill の記載追加、バージョン変遷表に v4.10.0 を追記 - plugins/ndf/README.md: 実体のない scanner エージェントを削除し、 欠落していた debugger / devops-engineer / code-reviewer の詳細を追加(実在8エージェントに整合) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
🤖 cross-review fix | round 1 対応報告codex / gemini のレビュー指摘に対応しました(commit 77a3c1a)。
deferred(nit) / rejected はありません。`claude plugin validate .` も通過済みです。 |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | gemini | APPROVE
新規 skill ml-model-structure の追加および、ドキュメント全般の整合性確保が非常に丁寧に実施されています。
- 設計の妥当性: 版ごとの自己完結構造や train/serve skew 対策など、実プロジェクトの知見に基づいた高度な基準が言語化されており、有用性が高い。
- 整合性: 45 スキルの総数およびカテゴリ分けが、ルート README / ndf README / reference / plugin.json 等の全ドキュメントで完全に一致していることを確認しました。
- クリーンアップ: 実態と乖離していた
scannerエージェントの記述削除や、セクション番号の重複解消も適切に行われています。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 2 | codex | REQUEST_CHANGES
CHANGELOG とリファレンスの整合を修正してください。
ndf-plugin-reference.md が v4.10.0 の詳細参照先として CHANGELOG.md を案内するが、 CHANGELOG の最新項目が v4.7.6 のままだった codex レビュー指摘 (minor) に対応。 ml-model-structure skill 追加・skill 数 45 更新の v4.10.0 エントリを追記。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | gemini | REQUEST_CHANGES
plugins/ndf/README.md の末尾にある比較表(L764付近)において、スキル数が古いバージョンの表記 (23個) のまま残っています。PR の趣旨である「45個」への統一と整合させるため、この箇所の更新をお願いします。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 3 | codex | APPROVE
修正必須の新規指摘はありません。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Round 3 gemini 指摘への対応ご指摘ありがとうございます。
ご確認をお願いいたします。 |
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 4 | codex | APPROVE
修正必須の新規指摘はありません。
takemi-ohama
left a comment
There was a problem hiding this comment.
🤖 cross-review | round 4 | gemini | APPROVE
統合プラグイン ndf のバージョンを v4.10.0 に、スキル総数を 45 個に統一し、新規スキル ml-model-structure を追加する本 PR を承認します。各種ドキュメント(README, AGENTS.md, ndf-plugin-reference 等)の整合性が取れており、README のセクション番号の修正など細部まで配慮されています。
Summary
ml-model-structureを ndf に追加(issue i19 の登録依頼)。vN/が独自のfeatures.py/train//inference/を持つ・symlink/shared なし)vN/features.pyを学習・推論・テストが共有し train/serve skew を版内で防ぐ)COPY features.py/.dockerignore配置 /model.tar.gzは成果物のみ)references/inference-and-contract.mdに推論コンテナ規約 + TRAIN_SPEC.md テンプレを分離plugin.json: version 4.9.0 → 4.10.0(新規 skill 追加の minor bump)、skills配列に追加、description を 45 skills に更新marketplace.json/ ルートREADME.md/AGENTS.md/docs/ndf-plugin-reference.md/plugins/ndf/README.md):issue-plan-strategy/ Playwright E2E 6個 / Google連携 2個 / AIクロスレビュー 2個 /skill-stats)を各ドキュメント本文へ追記し、45スキルを漏れなく掲載plugins/ndf/README.mdのセクション番号衝突(### 5の重複)を解消し連番化.dockerignoreの説明を references と整合させ技術的に正確化(context ルート or<dockerfile>.dockerignoreのみ有効)Test plan
claude plugin validate .がパス(既存の marketplace description warning のみ)plugin.json/marketplace.jsonの JSON 構文 OKplugin.jsonのskills配列が実数 45 個/ndf:ml-model-structure等の文脈で参照されること(インストール後の動作確認)やらないこと
carmo-screening側の実装には触れない(本 PR は登録と文書整合のみ)。