Python: Forward runtime kwargs to skill resource functions

[SergeyMenshykh]

Note: This PR is based on #4387 and should be merged after it.

Summary

Enables @skill.resource decorated functions to receive runtime keyword arguments passed via agent.run(), matching the existing @tool kwargs pattern.

Problem

Currently, skill resource functions (decorated with @skill.resource) are always called with zero arguments. In contrast, regular @tool functions can receive runtime kwargs passed via agent.run(user_id="123") through **kwargs. This makes it impossible to inject request-scoped context (app version, auth tokens, correlation IDs) into skill resources.

Changes

Core (_skills.py)

• Added **kwargs: Any to _read_skill_resource() method signature
• Detects if resource functions accept **kwargs via inspect.signature (same pattern as FunctionTool)
• Forwards kwargs to resource functions that accept them; functions without **kwargs are unaffected (backward compatible)

Tests (test_skills.py)

• Sync resource function receiving kwargs
• Async resource function receiving kwargs
• Backward compatibility: resource function without **kwargs still works when extra args are passed

Sample (code_skill.py)

• Updated project_info_skill's environment() resource to accept **kwargs and use app_version
• Updated main() to pass app_version="2.4.1" via agent.run()
• Added Pattern 3 documentation for dynamic resources with kwargs

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

[markwallace-microsoft]

Python Test Coverage Report •

File	Stmts	Miss	Cover	Missing
packages/core/agent_framework
_skills.py	289	8	97%	577, 616–617, 690, 695, 847–848, 1058
TOTAL	22611	2795	87%

Python Unit Test Overview

Tests	Skipped	Failures	Errors	Time
4702	25 💤	0 ❌	0 🔥	1m 16s ⏱️

[Copilot]

Pull request overview

This PR significantly refactors the Agent Skills system by: (1) replacing FileAgentSkillsProvider with a unified SkillsProvider that supports both file-based and code-defined skills, (2) introducing Skill and SkillResource classes as public API, and (3) enabling @skill.resource-decorated functions to receive runtime kwargs passed via agent.run(), matching the existing @tool kwargs pattern.

Changes:

• Core refactor of _skills.py: replaces FileAgentSkillsProvider/_FileAgentSkill with SkillsProvider/Skill/SkillResource classes; adds **kwargs forwarding to callable resource functions; changes resource discovery from markdown-link parsing to filesystem scanning
• Comprehensive test rewrite in test_skills.py: updates all existing tests for the new API and adds new test cases for code-defined skills, kwargs forwarding, and edge cases
• New code_skill sample and README demonstrating code-defined skills with dynamic resources and kwargs injection; updates basic_skill sample to use the renamed SkillsProvider

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
python/packages/core/agent_framework/_skills.py	Full refactor: new Skill/SkillResource/SkillsProvider classes; kwargs forwarding in _read_skill_resource; filesystem-based resource discovery
python/packages/core/agent_framework/__init__.py	Exports Skill, SkillResource, SkillsProvider; removes FileAgentSkillsProvider
python/packages/core/tests/core/test_skills.py	Updated tests for new API + new tests for code-defined skills and kwargs
python/samples/02-agents/skills/code_skill/code_skill.py	New sample demonstrating three skill patterns including kwargs-enabled resources
python/samples/02-agents/skills/code_skill/README.md	Documentation for new code_skill sample
python/samples/02-agents/skills/basic_skill/basic_skill.py	Updated to use renamed SkillsProvider
python/samples/02-agents/skills/basic_skill/README.md	Updated references to SkillsProvider and corrected file/directory names