github / github/copilot-sdk

Expose `disableModelInvocation` on `session.skills_loaded` skill entries

オープン
#1,850 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement
主要言語
Java
スター
10.5k
フォーク
1.5k
平均マージ
1日 11時間
マージ済み PR(30日)
127

説明

## 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 `SkillCustomization` already has a `disableModelInvocation` field, 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):

```jsonc
{
"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:

```jsonc
{
"name": "…",
"description": "…",
"source": "…",
"path": "…",
"argumentHint": "…",
"enabled": true,
"userInvocable": true,
"disableModelInvocation": false // NEW
}
```

Surfaces to touch:

- The `session.skills_loaded` event schema (the per-skill entry shape).
- Generated types in every language target (e.g. the Rust `SkillsLoadedSkill` and the TypeScript `session.skills_loaded` data type).
- For consistency, mirror the field onto the related skill API types that also omit it today (the `Skill` / `ServerSkill` discovery 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.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Start with the session.skills_loaded event schema and the per-skill entry types, including Rust’s SkillsLoadedSkill and the TypeScript session.skills_loaded data type. Then inspect the related Skill and ServerSkill discovery shapes and the producer’s existing model-invocation flag. Done means every listed shape exposes disableModelInvocation and the producer populates it with the existing flag, while older-producer defaults remain false.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java, rust, typescript
領域
api, backend-api-design
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
55/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。