[MatthewHsu1]

Problem

A model read "the service skill" in its instructions and called load_skill('service'), although service was pinned and its body was already in the prompt inside <skill name="service">. MAF answered Error: Skill 'service' not found., and the model went on to guess through 10 tool calls (SpiritAI production, 2026-10-02).

Change

• PinnedSkillRedirect wraps MAF's load_skill. A pinned name gets: The 'service' skill is already in your instructions, inside <skill name="service">. Use it from there. Every other name goes to MAF unchanged.
• ReadOnlySkillsProvider already rewrites the skill tools on each turn. It now applies the wrapper, but only when the agent has pinned skills.
• Pinned skills are not added to <available_skills>, and their body is not returned a second time.

Tests

SkillsProviderFactoryTests covers three cases:

• a pinned name, with the argument given both as a JsonElement and as a string;
• a loadable name, which returns its body;
• an unknown name, which still gets MAF's not-found answer.