Skip to content

fix: grilling skill の参照先を更新 - #111

Merged
version-1 merged 2 commits into
mainfrom
fix/update-grilling-skill-url
Jul 30, 2026
Merged

fix: grilling skill の参照先を更新#111
version-1 merged 2 commits into
mainfrom
fix/update-grilling-skill-url

Conversation

@version-1

Copy link
Copy Markdown
Owner

grilling skill の参照先更新

Summary

upstream で grill-me から grilling へ変更された外部 skill の名前、URL、配布先を更新します。
指定された GitHub の SKILL.md blob URL を deploy が直接扱えるようにしました。

主な変更点:

  • 外部 skill 名と配布先を grilling に統一
  • GitHub の blob/<ref>/<path>/SKILL.md URL に対応
  • skill ディレクトリの tree hash 検証を維持

やったこと

  • external-skills.json の URL、tree hash、配布先を更新
  • blob URL から親 skill ディレクトリを解決する処理を追加
  • URL パスを OS 非依存で解析
  • deploy の README、skill 一覧、依存関係を更新
  • blob URL の正常系・異常系と取得先解決の回帰テストを追加

動作確認

  • GOMODCACHE=/tmp/gomodcache GOCACHE=/tmp/gocache GOTOOLCHAIN=local go test ./...
  • GOMODCACHE=/tmp/gomodcache GOCACHE=/tmp/gocache GOTOOLCHAIN=local make deploy-dry-run
  • 指定 URL から grilling を取得し、tree hash と skill 名を検証

レビュー & 動作確認 チェックリスト

  • blob URL の互換性 - SKILL.md の親ディレクトリが取得対象になること
  • tree URL の後方互換性 - 既存の GitHub tree URL が従来どおり動作すること
  • 配布名の統一 - Codex / Claude の配布先と関連文書が grilling になっていること

推奨テスト計画:

  1. Go テスト一式を実行する
  2. make deploy-dry-rungrilling の取得元と配布先を確認する
  3. 必要に応じて make deploy 後の SKILL.md frontmatter を確認する

その他気になることや相談ごと

URL parser の追加的な境界条件テストは今後拡充できますが、tree URL の既存経路と今回の blob URL 経路はテスト済みです。

外部 skill 名と配布先を grilling に統一し、GitHub の SKILL.md blob URL を取得できるようにする。関連するテストとドキュメントも更新した。\n\n検証: GOMODCACHE=/tmp/gomodcache GOCACHE=/tmp/gocache GOTOOLCHAIN=local go test ./...\n検証: make deploy-dry-run
GitHub blob URL のパス判定に path パッケージを使い、Windows でも slash 区切りの URL を正しく解析する。\n\n検証: GOMODCACHE=/tmp/gomodcache GOCACHE=/tmp/gocache GOTOOLCHAIN=local go test ./...
@version-1
version-1 merged commit 64210f6 into main Jul 30, 2026
1 check passed
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