Skip to content

Fix packaged skill resource reads#369

Merged
FredKSchott merged 1 commit into
mainfrom
fix/packaged-skill-resource-reader
Jun 25, 2026
Merged

Fix packaged skill resource reads#369
FredKSchott merged 1 commit into
mainfrom
fix/packaged-skill-resource-reader

Conversation

@FredKSchott

Copy link
Copy Markdown
Member

Summary

  • expose packaged skill supporting files through a dedicated read_skill_resource framework tool
  • stop synthesizing a packaged-only tool named read for sandbox adapters that intentionally replace default filesystem tools
  • advertise the dedicated resource tool during progressive disclosure and reserve its framework-owned name

Why

Sandbox adapter tools replace Flue's default model-facing filesystem tools. When an adapter omitted read, registering a packaged skill caused Flue to inject a packaged-only tool named read. Models could reasonably pass workspace paths to it and receive a misleading Packaged skill file not found error even though the workspace file existed.

A dedicated tool matches the Agent Skills client guidance: activate instructions first, then load bundled supporting resources on demand without implying general sandbox filesystem authority.

Verification

  • pnpm run test in packages/runtime (759 tests)
  • pnpm run check:types in packages/runtime
  • Biome check for all changed files

@FredKSchott FredKSchott merged commit 1af058b into main Jun 25, 2026
2 checks passed
@FredKSchott FredKSchott deleted the fix/packaged-skill-resource-reader branch June 25, 2026 10:06
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