Skip to content

feat: ml-model-structure skill 追加 (ndf v4.10.0)#23

Open
takemi-ohama wants to merge 5 commits into
mainfrom
feature/add-ml-model-structure-skill
Open

feat: ml-model-structure skill 追加 (ndf v4.10.0)#23
takemi-ohama wants to merge 5 commits into
mainfrom
feature/add-ml-model-structure-skill

Conversation

@takemi-ohama
Copy link
Copy Markdown
Contributor

@takemi-ohama takemi-ohama commented May 30, 2026

Summary

  • ML モデル構築・推論API開発の標準ディレクトリ構造を定める新規 skill ml-model-structure を ndf に追加(issue i19 の登録依頼)。
    • 版ごと自己完結 (vN/ が独自の features.py/train//inference/ を持つ・symlink/shared なし)
    • 版内 feature SSoT (vN/features.py を学習・推論・テストが共有し train/serve skew を版内で防ぐ)
    • 推論コンテナ規約 (SageMaker 標準ハンドラ / build context=版ルート / 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 に更新
  • ドキュメントの skill 数・カテゴリ個数を実数45に完全整合marketplace.json / ルート README.md / AGENTS.md / docs/ndf-plugin-reference.md / plugins/ndf/README.md):
    • 既存でバラバラだった総数表記(33/36/39/44)を全て 45 に統一
    • 欠落していたカテゴリ(issue-plan-strategy / Playwright E2E 6個 / Google連携 2個 / AIクロスレビュー 2個 / skill-stats)を各ドキュメント本文へ追記し、45スキルを漏れなく掲載
    • カテゴリ個数の合計を 13+9+12+6+2+2+1 = 45 に一致させた
    • plugins/ndf/README.md のセクション番号衝突(### 5 の重複)を解消し連番化
  • レビュー指摘の反映(skill 本体):
    • .dockerignore の説明を references と整合させ技術的に正確化(context ルート or <dockerfile>.dockerignore のみ有効)
    • 受け入れ基準に「import 解決チェック」と「Docker 不可環境での代替手段」を追加

Test plan

  • claude plugin validate . がパス(既存の marketplace description warning のみ)
  • plugin.json / marketplace.json の JSON 構文 OK
  • plugin.jsonskills 配列が実数 45 個
  • 3 ドキュメント(root README / ndf README / docs reference)に 45 スキル名が全掲載(機械検証済み)
  • root README / ndf README のカテゴリ個数合計が 45 に一致(機械検証済み)
  • skill が /ndf:ml-model-structure 等の文脈で参照されること(インストール後の動作確認)

やらないこと

  • skill 本体が由来する carmo-screening 側の実装には触れない(本 PR は登録と文書整合のみ)。

takemi-ohama and others added 2 commits May 30, 2026 07:33
- 新規 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>
Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

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 | REQUEST_CHANGES

docs/ndf-plugin-reference.md の「現行バージョン」が v4.7.0 のままです。PR で ndf を 4.10.0 に上げているため、README / marketplace / plugin.json と同じ版に揃えてください。

Comment thread README.md Outdated
Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

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 | REQUEST_CHANGES

PR #23 における ml-model-structure スキルの追加と各ドキュメントの更新内容を確認しました。スキル定義自体は SageMaker や Docker の制約を考慮した非常に実用的な内容になっています。

一方で、v4.10.0 リリースにあたりドキュメント内のバージョン表記やエージェント一覧に不整合が残っているため、以下の修正を提案します。

Comment thread README.md
Comment thread plugins/ndf/README.md
- 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>
@takemi-ohama
Copy link
Copy Markdown
Contributor Author

🤖 cross-review fix | round 1 対応報告

codex / gemini のレビュー指摘に対応しました(commit 77a3c1a)。

  • codex (docs/ndf-plugin-reference.md): 「現行バージョン」を v4.7.0 → v4.10.0 に更新。ml-model-structure skill の記載を追加し、バージョン変遷表にも v4.10.0 を追記しました。
  • gemini (README.md 冒頭バージョン): `NDFプラグイン v4.0.0` → v4.10.0 に修正(インラインスレッドで返信・Resolve 済み)。
  • gemini (plugins/ndf/README.md エージェント一覧): 実体のない `scanner` を削除し、欠落していた `debugger` / `devops-engineer` / `code-reviewer` の詳細を追加。実在8エージェントに整合(インラインスレッドで返信・Resolve 済み)。

deferred(nit) / rejected はありません。`claude plugin validate .` も通過済みです。

Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

Choose a reason for hiding this comment

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

🤖 cross-review | round 2 | gemini | APPROVE

新規 skill ml-model-structure の追加および、ドキュメント全般の整合性確保が非常に丁寧に実施されています。

  • 設計の妥当性: 版ごとの自己完結構造や train/serve skew 対策など、実プロジェクトの知見に基づいた高度な基準が言語化されており、有用性が高い。
  • 整合性: 45 スキルの総数およびカテゴリ分けが、ルート README / ndf README / reference / plugin.json 等の全ドキュメントで完全に一致していることを確認しました。
  • クリーンアップ: 実態と乖離していた scanner エージェントの記述削除や、セクション番号の重複解消も適切に行われています。

Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

Choose a reason for hiding this comment

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

🤖 cross-review | round 2 | codex | REQUEST_CHANGES

CHANGELOG とリファレンスの整合を修正してください。

Comment thread docs/ndf-plugin-reference.md
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>
Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

Choose a reason for hiding this comment

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

🤖 cross-review | round 3 | gemini | REQUEST_CHANGES

plugins/ndf/README.md の末尾にある比較表(L764付近)において、スキル数が古いバージョンの表記 (23個) のまま残っています。PR の趣旨である「45個」への統一と整合させるため、この箇所の更新をお願いします。

Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

Choose a reason for hiding this comment

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

🤖 cross-review | round 3 | codex | APPROVE

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

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@takemi-ohama
Copy link
Copy Markdown
Contributor Author

Round 3 gemini 指摘への対応

ご指摘ありがとうございます。plugins/ndf/README.md 末尾の比較表(L764)に旧スキル数表記「23個」が残っていた点を確認し、PR の趣旨(45個統一)に合わせて 45個 に修正しました。

  • 修正コミット: 8f7e740
  • 文書全体を grep -n で再確認し、他に旧スキル数(23個等)の残存箇所がないことを確認済みです。
  • claude plugin validate . パス(既存の marketplace description 警告のみ、本修正とは無関係)。

ご確認をお願いいたします。

Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

Choose a reason for hiding this comment

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

🤖 cross-review | round 4 | codex | APPROVE

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

Copy link
Copy Markdown
Contributor Author

@takemi-ohama takemi-ohama left a comment

Choose a reason for hiding this comment

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

🤖 cross-review | round 4 | gemini | APPROVE

統合プラグイン ndf のバージョンを v4.10.0 に、スキル総数を 45 個に統一し、新規スキル ml-model-structure を追加する本 PR を承認します。各種ドキュメント(README, AGENTS.md, ndf-plugin-reference 等)の整合性が取れており、README のセクション番号の修正など細部まで配慮されています。

Sign up for free to 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