Skip to content

docs(skill): stop implying load_skill is availability-gated #3468

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kojiwakayama merged 1 commit into from
Aug 8, 2026
Merged

docs(skill): stop implying load_skill is availability-gated #3468

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/api-reference/veryfront/skill.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ validateSkillMetadata(parsed.frontmatter, "review");

| Name | Description | Source |
| -------------------------- | ------------------------------------------------------------------------ | ------------------------------------------------------------------------------------------------ |
| `ActiveSkillContext` | Active skill context for runtime policy tracking | [source](https://github.com/veryfront/veryfront-code/blob/main/src/skill/types.ts#L310) |
| `ActiveSkillContext` | Active skill context for runtime availability and delegation tracking | [source](https://github.com/veryfront/veryfront-code/blob/main/src/skill/types.ts#L310) |
| `AgentCapabilityScope` | Caller scope used for owner-aware capability resolution. | [source](https://github.com/veryfront/veryfront-code/blob/main/src/skill/registry.ts#L59) |
| `ParsedSkillContent` | Result of splitting and decoding one bounded `SKILL.md` document. | [source](https://github.com/veryfront/veryfront-code/blob/main/src/skill/document-parser.ts#L48) |
| `Skill` | Registered skill instance | [source](https://github.com/veryfront/veryfront-code/blob/main/src/skill/types.ts#L203) |
Expand Down
7 changes: 4 additions & 3 deletions docs/architecture/21-agent-tool-registration-current-state.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,9 +190,10 @@ appends forwarded remote definitions, and applies `allowedRemoteToolNames`.
Provider-native tools are added later during model tool conversion. For
example, `web_search` is selected from the allowed name list, converted into an
Anthropic provider tool, and marked as provider-executed in stream handling.
Runtime skill tools are local platform tools. Local and project runtimes expose
the three-tool skill surface, gate it by what the loaded skill advertises, and
execute it through Veryfront. Hosted chat exposes its request-scoped `load_skill` tool
Runtime skill tools are local platform tools. Local and project runtimes always
expose `load_skill`. They gate `load_skill_reference` and `execute_skill_script`
Comment on lines +193 to +194

@chatgpt-codex-connector chatgpt-codex-connector Bot Aug 8, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Qualify load_skill exposure for disabled skill selectors

When an agent uses the explicit skills: false or skills: [] selector, shouldExposeSkillTools is false and the factory removes every skill tool, including load_skill; agent-runtime-step.test.ts also verifies this case. The current-state document therefore should limit “always expose” to skill-enabled agents rather than implying that the recovery loader exists in every local and project runtime.

AGENTS.md reference: AGENTS.md:L13-L14

Useful? React with 👍 / 👎.

by the references and scripts the loaded skill advertises, and execute them
through Veryfront. Hosted chat exposes its request-scoped `load_skill` tool
instead.

`executeConfiguredTool` resolves in this order:
Expand Down
2 changes: 1 addition & 1 deletion src/skill/types.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -306,7 +306,7 @@ export interface SkillScriptExecutor {
execute(input: SkillScriptExecutorInput): Promise<SkillScriptResult>;
}

/** Active skill context for runtime policy tracking */
/** Active skill context for runtime availability and delegation tracking */

@chatgpt-codex-connector chatgpt-codex-connector Bot Aug 8, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Describe ActiveSkillContext as identity-only

This exported interface contains only skillId and has no availability or delegation fields; the runtime tracks those separately as activeSkillToolAvailability and activeSkillDelegationOverrides. The regenerated public API now promises capabilities that consumers cannot obtain from this type, so describe it as active-skill identity context unless those fields are intended to become part of the interface.

AGENTS.md reference: AGENTS.md:L13-L14

Useful? React with 👍 / 👎.

export interface ActiveSkillContext {
skillId: string;
}