Uh oh!
There was an error while loading. Please reload this page.
refactor: 利用実績のない Skill を削除し発動条件を見直す (0-6) - #72
Merged
Conversation
棚卸台帳の判断基準(issues/ndf-development-skills/02-skill-inventory.md)に従い、 起動ゼロまたは代替可能な Skill 8 個を削除する。sync-main は Task 0-4 の担当のため対象外。 削除: git-gh-operations / knowledge-reorg / google-chat / mcp-builder / data-analyst-sql-optimization / python-execution / data-analyst-export / deepwiki-transfer - data-analyst-export と data-analyst-sql-optimization の内容は plugins/ndf-claude/agents/data-analyst.md へ移設した - manifest 3 種、plugin.json、README、docs の参照を更新した 発動改善(削除しない): - deploy: disable-model-invocation は維持し description に用途とトリガを明記 - qa-security-scan: description に発動条件を含めて自動発動できるようにした - official-skills-autoloader: Claude Code 専用である旨と対応ランタイムを明記 - logging-guidelines: paths でコード変更時に限定 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AGejnYyYFuSkQjBhW2KQNy
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
修正必須の指摘はありません。
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
変更内容を網羅的に確認しました。
Skill の削除と 3 ランタイムの manifest (claude-skills.txt, codex-skills.txt, kiro-skills.txt) への反映漏れはなく、ビルドスクリプトを通じた生成物・プラグイン定義の同期も正常に行われる状態です。
また、統合元 (data-analyst-sql-optimization, data-analyst-export) の手順は、要点を損なうことなく data-analyst.md へ適切に要約・移植されていることを確認しました。
設計や後方互換性においても特に懸念点はありません。適切なクリーンアップです。
takemi-ohama
marked this pull request as ready for review
August 8, 2026 03:29
Uh oh!
There was an error while loading. Please reload this page.
takemi-ohama added a commit
that referenced
this pull request
Aug 8, 2026
release ブランチ側の削除 (PR #71 ブラウザ自動テスト集約 / PR #72 Skill 削除) と 本ブランチ側の統合 (review / fix) の双方を反映した。 - manifest 3 種と plugin.json は両者の削除を反映した積集合にした (claude 24 / codex 24 / kiro 23、shared 33) - browser-test は playwright-authoring へ統合済みのため削除し、 同 Skill に残っていた /ndf:review-branch 参照を /ndf:review --branch へ更新 - README の Skill 一覧と個数を実体に一致させた
takemi-ohama added a commit
that referenced
this pull request
Aug 8, 2026
並行 PR (#68 review/fix 系統合、#71 ブラウザ自動テスト集約、#72 Skill 削除) が 先に release へ入ったため、本ブランチの external-ai 統合と競合していた。 競合解消方針: - manifest / plugin.json の skills は両ブランチの削除を両方反映 (積集合)。 release が消した review-branch / review-pr-comments / resolve-pr-comments / browser-test と、本ブランチが消した codex / gemini をいずれも除去し、 release が追加した playwright-authoring と本ブランチの external-ai を残した。 - review / cross-review SKILL.md は release 側の再構成をベースに採用し、 本ブランチの /ndf:codex /ndf:gemini → /ndf:external-ai 参照差し替えを再適用。 - README.md の Skill 一覧と個数は plugins/ndf-shared/skills/ と manifest の 実数から数え直した (元Skills 32個 / Claude 23・Kiro 23・Codex 25)。 - plugins/ndf-{claude,codex,kiro}/skills/ は共有側を直したうえで scripts/build-runtime-plugins.sh で再生成。
7 tasks
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
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
棚卸台帳の「整理の判断基準」(起動数 × 機会数)に従い、
plugins/ndf-shared/skills/から 8 個を削除した。sync-mainは Task 0-4 の担当のためここには含まない。あわせて、起動ゼロだが機会があり削除対象にしない 4 個の発動条件を見直した。Skill 総数は 41 個(削除前 49 個 − 本 PR の 8 個。統合分は別 PR)。
削除した 8 個
git-gh-operationsgit addgit commitが広すぎることによる誤検出で、需要ではないknowledge-reorggoogle-chatmcp-builderdata-analyst-sql-optimizationpython-executionpythonスクリプトが広すぎ、他 Skill の発動を埋もれさせていたdata-analyst-exportdeepwiki-transfermanifest に載っていた(=実際に配布されていた)のは
python-executionとgit-gh-operationsの 2 個のみで、残る 6 個は共通編集元にのみ存在していた。data-analystエージェントへ移した内容data-analyst-exportとdata-analyst-sql-optimizationは「データ分析エージェントの定義に直接書けば足りる」が削除理由のため、削除前にplugins/ndf-claude/agents/data-analyst.mdへ次を移した。EXPLAINで実行計画を確認してから適用し再度確認する手順、DO / DON'T 表移さなかったもの:
scripts/export-csv.js。01-formats.mdはexport-json.js/export-excel.js/export-markdown.jsも案内していたが、実体はexport-csv.jsの 1 本しか存在せず、記載の 4 分の 3 が動かない状態だった。出力自体はモデルが直接書けるため、スクリプトは移さず削除した。mcp-builderのライセンス確認結果告知義務は残らない。削除して問題ない。
mcp-builderは上流anthropics/skills由来で、frontmatter にlicense: Complete terms in LICENSE.txtを持つ唯一の Skill だったLICENSE.txtは Apache License 2.0(Copyright 2026 Anthropic, PBC.)。NOTICEファイルは同梱されておらず、リポジトリ全体にも存在しないSKILL.md/reference//scripts//LICENSE.txtを含むディレクトリ全体を削除し、上流由来の記述を派生物として残していないため、削除後に伝達すべき告知は残らないmcp-builderの記載は、docs/claude-code-skills-survey.md/docs/claude-code-skills-official-reference.md/docs/official-skills-installation.mdの Apache-2.0 一覧など、上流カタログの紹介であり当リポジトリの同梱物を指していない。そのまま残したdocs/official-skills-installation.mdの「NDFが同梱している公式Skill」節は、同梱がなくなった旨とインストーラでの入手手順に書き換えたofficial-skills-autoloaderの対応マッピングに「MCPサーバー作成 →mcp-builder」を追加した発動改善した 4 個(削除しない)
deploy(0 / 340)環境ブランチへ書き込む破壊的操作のため
disable-model-invocation: trueは維持した。descriptionのみ改善している。"Create deploy PRs from feature to environment branches.""Create a deploy PR from the current feature branch to an environment branch such as qa/staging or release/v2. 破壊的操作のため、利用者が /ndf:deploy を明示的に指示したときのみ実行する(環境ブランチへデプロイ / qaに上げる / stagingに反映 / リリースブランチへPR)。deployブランチを作成し origin/main を取り込んでからPRを出す。"qa-security-scan(0 / 66)when_to_useに閉じ込めていたトリガをdescriptionへ移し、自動発動できるようにした。description"Run OWASP-focused security checks."/when_to_use"Use when conducting security testing or vulnerability assessment. Triggers: 'security scan', ..."description"Run an OWASP Top 10 security review of code, authentication, authorization, and data protection. Use when asked for a security scan, vulnerability assessment, or a security review of a change(セキュリティスキャン / 脆弱性チェック / セキュリティレビュー / OWASP / 認証認可の確認 / SQLインジェクションの確認)."(when_to_useはdescriptionの言い換えになるため削除)official-skills-autoloader(0 / 97)各ランタイムの公式 Skill 提供状況を確認した結果、Claude Code 専用と確定した。
.claude/skills/、プラグインのskills/~/.claude/skills/を読む).agents/skills/.kiro/skills//~/.kiro/skills/description"Install and use official document Skills on demand."/when_to_useに Word/Excel 等のトリガdescription"Install an Anthropic official Skill on demand and run it. Use when the request needs Word/Excel/PowerPoint/PDF creation or editing, frontend design, webapp testing, or MCP server scaffolding(Word作成 / Excel出力 / スライド生成 / PDF作成 / .docx / .pptx / .xlsx / .pdf / MCPサーバーを作りたい). Claude Code 専用。"(when_to_useは削除)logging-guidelines(0 / 112)pathsでコード変更時に限定した(Claude Code 独自項目。Codex / Kiro では無視される)。description"Design safe and useful application logging."/when_to_useに'logger''logging'等の広いトリガdescription"Choose log levels and write safe, useful application logs when adding or reworking logging in code(ログ追加 / logger / ログレベル / デバッグログ / エラーログ / print文をログに). Use when editing source code that emits logs, to pick the level and keep secrets and personal data out of the output."pathsに**/*.py**/*.ts**/*.tsx**/*.js**/*.jsx**/*.go**/*.rb**/*.java**/*.kt**/*.php**/*.rs**/*.shを追加4 個とも
descriptionは二重引用符で囲み、最初の 1 文に「何をするか + 主要トリガ」を置き、300 文字以内(244 / 262 / 288 / 270 文字)で、<>を含まない。併せて更新した参照
plugins/ndf-shared/manifests/{claude,codex,kiro}-skills.txtからpython-execution/git-gh-operationsを除去(27 / 28 / 26 個)plugins/ndf-claude/.claude-plugin/plugin.jsonのskills配列plugins/ndf-codex/.codex-plugin/plugin.jsonのdescription("GitHub operations" を除去)plugins/ndf-shared/skills/docker-container-access/SKILL.md/skill-stats/SKILL.mdの関連 Skill リンクREADME.mdの Skill 一覧(49 → 41 個。区分ごとの件数が合っていなかったため、未記載だったplaywright-browser-connect/playwright-evidence-drive/statuslineも補って合計を一致させた)docs/ndf-plugin-reference.md/docs/official-skills-installation.mdplugins/ndf-{claude,codex,kiro}/skills/はscripts/build-runtime-plugins.shで同期issues/配下のプラン文書とdocs/specifications/ndf-skill-inventory.md(棚卸台帳)は記録のため書き換えていない。本 PR では扱わなかった点(後続タスクへ)
qa-security-scanとofficial-skills-autoloaderはどの manifest にも載っておらず配布されていない。起動ゼロの直接の原因はここにあり、descriptionを直しても配布されるまで発動しない。配布セットの決定は Task 0-7 / 0-10 の担当のため、本 PR では manifest を変更していないdeployのargument-hintに<env-branch>という<>が残っている(frontmatter の<>禁止に抵触)。frontmatter の一括見直しは Task 0-7 の担当のため触っていないdisable-model-invocationの一括見直しも Task 0-7 の担当。本 PR はdeployの維持のみTest plan
grep -rnで削除した 8 個の Skill 名が、issues/と棚卸台帳を除いてヒットしない(残るのは上流カタログとしてのmcp-builderの記載のみ)descriptionに発動条件が含まれ、二重引用符・300 文字以内・<>なしを満たすbash scripts/build-runtime-plugins.sh実行後、bash scripts/build-runtime-plugins.sh --checkが差異なしpython3 scripts/check-markdown-links.py --root .が成功するbash scripts/validate-runtime-plugins.shが成功する(claude plugin validateを含む)release PR: #66
plan: issues/ndf-development-skills/07-tasks.md