Giles Odigwe
1249275997
* Python: Remove experimental marker from Skills API Promote the Skills feature from experimental to stable, mirroring .NET PR #6861. Removes the @experimental(SKILLS) decorators from the skills APIs and the SKILLS ExperimentalFeature enum member, updates tests and samples accordingly. MCP skills (MCP_SKILLS) remain experimental, matching the .NET change. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add experimental-stage assertions for MCP skills types Guard MCPSkill, MCPSkillResource, and MCPSkillsSource against accidental promotion by asserting their docstring warning block and __feature_stage__/__feature_id__ metadata remain experimental (MCP_SKILLS). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Remove redundant stable-stage test for Skills API Drop TestSkillsStableStage: asserting the absence of experimental markers on a released API is not meaningful, and the feature-stage decorator machinery is already covered by test_feature_stage.py. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
73 lines
2.7 KiB
Python
73 lines
2.7 KiB
Python
# Copyright (c) Microsoft. All rights reserved.
|
|
|
|
"""Sample subprocess-based skill script runner.
|
|
Executes file-based skill scripts as local Python subprocesses.
|
|
This is provided for demonstration purposes only.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import subprocess
|
|
import sys
|
|
from pathlib import Path
|
|
from typing import Any
|
|
|
|
from agent_framework import FileSkill, FileSkillScript
|
|
|
|
|
|
def subprocess_script_runner(
|
|
skill: FileSkill, script: FileSkillScript, args: dict[str, Any] | list[str] | None = None
|
|
) -> str:
|
|
"""Run a skill script as a local Python subprocess.
|
|
Uses ``FileSkillScript.full_path`` as the script path, converts the
|
|
``args`` to CLI arguments, and returns captured output.
|
|
Args:
|
|
skill: The file-based skill that owns the script.
|
|
script: The file-based script to run.
|
|
args: Optional arguments. A ``list[str]`` is forwarded as
|
|
positional CLI arguments. Passing a ``dict`` or any other
|
|
type raises :class:`TypeError` — file-based scripts expect
|
|
positional arguments as a JSON array of strings.
|
|
Returns:
|
|
The combined stdout/stderr output, or an error message.
|
|
Raises:
|
|
TypeError: If ``args`` is not a ``list[str]`` or ``None``, or if
|
|
any list element is not a string.
|
|
"""
|
|
script_path = Path(script.full_path)
|
|
if not script_path.is_file():
|
|
return f"Error: Script file not found: {script_path}"
|
|
cmd = [sys.executable, str(script_path)]
|
|
if isinstance(args, list):
|
|
for item in args:
|
|
if not isinstance(item, str):
|
|
raise TypeError(
|
|
f"File-based skill scripts only accept string CLI arguments "
|
|
f"but received a {type(item).__name__}. "
|
|
f"All array elements must be strings."
|
|
)
|
|
cmd.extend(args)
|
|
elif args is not None:
|
|
raise TypeError(
|
|
f"Expected a list of CLI arguments but received {type(args).__name__}. "
|
|
f"File-based skill scripts expect positional arguments as a list of strings."
|
|
)
|
|
try:
|
|
result = subprocess.run(
|
|
cmd,
|
|
capture_output=True,
|
|
text=True,
|
|
timeout=30,
|
|
cwd=str(script_path.parent),
|
|
)
|
|
output = result.stdout
|
|
if result.stderr:
|
|
output += f"\nStderr:\n{result.stderr}"
|
|
if result.returncode != 0:
|
|
output += f"\nScript exited with code {result.returncode}"
|
|
return output.strip() or "(no output)"
|
|
except subprocess.TimeoutExpired:
|
|
return f"Error: Script '{script.name}' timed out after 30 seconds."
|
|
except OSError as e:
|
|
return f"Error: Failed to execute script '{script.name}': {e}"
|