Skip to content

fix: Add warning about the caching of skills#170

Merged
raymondk merged 3 commits intomainfrom
rk/cache-skill-warning
Apr 16, 2026
Merged

fix: Add warning about the caching of skills#170
raymondk merged 3 commits intomainfrom
rk/cache-skill-warning

Conversation

@raymondk
Copy link
Copy Markdown
Collaborator

No description provided.

@raymondk raymondk requested review from a team and JoshDFN as code owners April 16, 2026 14:01
@marc0olo
Copy link
Copy Markdown
Member

The new ## Important section conflicts with step 2, which currently says:

"use it if already loaded in your context"

An agent reading both instructions gets contradictory guidance — one says reuse what's in context, the other says don't use cached versions blindly.

Rather than adding a separate section, I'd suggest folding the intent directly into step 2 and dropping the phrase that causes the conflict:

Current:

2. When a task matches a skill's description, use it if already loaded in your
   context. Otherwise, fetch the skill content from its url.
   Do not construct skill URLs manually.

Suggested:

2. When a task matches a skill's description, fetch the skill content from its url
   before using it. Skills are updated frequently — do not rely on a previously
   cached or trained version.
   Do not construct skill URLs manually.

Stale skill content causing broken code is a worse tradeoff than an extra HTTP request, so dropping "use it if already loaded" is the right call.

@github-actions
Copy link
Copy Markdown

Skill Validation Report

No skill files were changed in this PR — validation skipped.

Copy link
Copy Markdown
Member

@marc0olo marc0olo left a comment

Choose a reason for hiding this comment

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

this now removes the following sentence:

Do not construct skill URLs manually.

but I don't think it is an issue anymore since we include the URL now explicitly and even provide an example for the II skill.

@raymondk raymondk merged commit d1e4fb8 into main Apr 16, 2026
6 checks passed
@raymondk raymondk deleted the rk/cache-skill-warning branch April 16, 2026 14:57
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.

2 participants