Expose `disableModelInvocation` on `session.skills_loaded` skill entries
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 55/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- java, rust, typescript
調査の方向性
session.skills_loaded イベントスキーマとスキルごとのエントリ型から始め、Rust の SkillsLoadedSkill と TypeScript の session.skills_loaded データ型を含めて確認します。次に、関連する Skill と ServerSkill の discovery 形状、および producer に既存の model-invocation フラグを調べます。完了条件は、一覧にあるすべての形状が disableModelInvocation を公開し、producer が既存のフラグを使ってそれを設定し、古い producer のデフォルト値は false のままであることです。
索引モデルが issue の本文から書いたものです。
説明
Summary
The session.skills_loaded event reports each skill's enabled and userInvocable flags, but not its model-invocation flag (the disable-model-invocation frontmatter gate). Please add disableModelInvocation to each skill entry so consumers can tell whether the model may auto-invoke a skill — not just whether the user can invoke it as a slash command.
Why
userInvocable (user slash-command availability) and model-invocability are two independent axes:
userInvocable: true/false— can the user invoke it as a slash command.disableModelInvocation: true/false— may the model auto-invoke it.
A skill can be any combination (user-only, model-only, both). Consumers today only receive userInvocable, so they can't distinguish a model-auto-invoked skill from a user-only one. This matters for:
- Rendering an accurate command palette / skill list (which entries are slash commands vs. silently model-driven).
- Projecting skills into surfaces that have a native model-invocation concept (for example, AHP's
SkillCustomizationalready has adisableModelInvocationfield, which currently can't be populated from this event).
The flag is already part of the skill model — it's used internally to decide whether a skill is visible to the model — so this is a surfacing change, not new behavior.
Current shape
session.skills_loaded skill entries today (camelCase wire):
{
"name": "…",
"description": "…",
"source": "project | inherited | personal-copilot | personal-agents | plugin | custom | builtin",
"path": "…", // optional
"argumentHint": "…", // optional
"enabled": true,
"userInvocable": true
}
Proposed change
Add disableModelInvocation: boolean to each entry:
{
"name": "…",
"description": "…",
"source": "…",
"path": "…",
"argumentHint": "…",
"enabled": true,
"userInvocable": true,
"disableModelInvocation": false // NEW
}
Surfaces to touch:
- The
session.skills_loadedevent schema (the per-skill entry shape). - Generated types in every language target (e.g. the Rust
SkillsLoadedSkilland the TypeScriptsession.skills_loadeddata type). - For consistency, mirror the field onto the related skill API types that also omit it today (the
Skill/ServerSkilldiscovery shapes). - Populate it at the producer from the skill model's existing model-invocation flag.
Backwards compatibility
Additive boolean; existing consumers are unaffected. A sensible default for older producers is false (model invocation allowed), matching the default semantics of the disable-model-invocation frontmatter flag.
- 主要言語
- Java
- スター
- 10.5k
- フォーク
- 1.5k
- 平均マージ
- 1日 9時間
- マージ済み PR(30日)
- 131
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/copilot-sdk のほかの issue
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/copilot-sdk#2709 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
github/copilot-sdk#2673 ·
-
bug testing
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
github/copilot-sdk#2628 ·
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/copilot-sdk#2627 · コメント 1 件 ·
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
github/copilot-sdk#2493 ·
github/copilot-sdk の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
elastic/gradle-plugins#157 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
cryptomator/hub#497 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
johanhaleby/occurrent#1120 ·