Skip to content

.NET: Revert ".NET: Support hosted code interpreter for skill script execution" - #4385

Merged
SergeyMenshykh merged 1 commit into
mainfrom
revert-4192-script-execution-by-code-interpreter
Mar 2, 2026
Merged

.NET: Revert ".NET: Support hosted code interpreter for skill script execution"#4385
SergeyMenshykh merged 1 commit into
mainfrom
revert-4192-script-execution-by-code-interpreter

Conversation

@SergeyMenshykh

Copy link
Copy Markdown
Contributor

Reverts #4192 until there's further clarity regarding the design of script executors.

CopilotAI review requested due to automatic review settings March 2, 2026 10:44
@SergeyMenshykhSergeyMenshykh self-assigned this Mar 2, 2026
@markwallace-microsoftMark Wallace (markwallace-microsoft) added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net labels Mar 2, 2026
@github-actionsgithub-actionsBot changed the title Revert ".NET: Support hosted code interpreter for skill script execution".NET: Revert ".NET: Support hosted code interpreter for skill script execution"Mar 2, 2026

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Reverts the earlier .NET “hosted code interpreter” script-execution support for Agent Skills while keeping the core file-based skills provider behavior.

Changes:

  • Removes hosted code interpreter script-execution types, options, tests, and the Step02 sample.
  • Updates FileAgentSkillsProvider to use composite-format ({0}) prompt templating and adds validation for custom templates.
  • Renames/internalizes some skills-loading implementation types (frontmatter/skill/loader) consistent with the revert.

Reviewed changes

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

Show a summary per file
FileDescription
dotnet/tests/Microsoft.Agents.AI.UnitTests/AgentSkills/HostedCodeInterpreterFileAgentSkillScriptExecutorTests.csRemoves tests for the hosted code interpreter script executor.
dotnet/tests/Microsoft.Agents.AI.UnitTests/AgentSkills/FileAgentSkillsProviderTests.csSwitches custom prompt template placeholder to {0} and adds invalid-template constructor test.
dotnet/tests/Microsoft.Agents.AI.UnitTests/AgentSkills/FileAgentSkillScriptExecutorTests.csRemoves integration tests for script executor wiring into the provider.
dotnet/tests/Microsoft.Agents.AI.UnitTests/AgentSkills/FileAgentSkillLoaderTests.csUpdates frontmatter type usage and removes tests tied to the reverted resource parsing behavior.
dotnet/src/Microsoft.Agents.AI/Skills/SkillFrontmatter.csRenames/replaces the prior frontmatter type with an internal SkillFrontmatter.
dotnet/src/Microsoft.Agents.AI/Skills/HostedCodeInterpreterFileAgentSkillScriptExecutor.csRemoves hosted code interpreter executor implementation.
dotnet/src/Microsoft.Agents.AI/Skills/FileAgentSkillsProviderOptions.csRemoves script-executor option and documents {0} placeholder format.
dotnet/src/Microsoft.Agents.AI/Skills/FileAgentSkillsProvider.csRemoves executor/tool merging logic; updates default prompt template and custom template handling.
dotnet/src/Microsoft.Agents.AI/Skills/FileAgentSkillScriptExecutor.csRemoves script executor abstraction.
dotnet/src/Microsoft.Agents.AI/Skills/FileAgentSkillScriptExecutionDetails.csRemoves script execution details model.
dotnet/src/Microsoft.Agents.AI/Skills/FileAgentSkillScriptExecutionContext.csRemoves script execution context model.
dotnet/src/Microsoft.Agents.AI/Skills/FileAgentSkillLoader.csInternalizes loader, adjusts resource link regex, and internalizes resource read API.
dotnet/src/Microsoft.Agents.AI/Skills/FileAgentSkill.csInternalizes the skill model and updates it to use SkillFrontmatter.
dotnet/samples/02-agents/AgentSkills/README.mdRemoves reference to the reverted Step02 sample.
dotnet/samples/02-agents/AgentSkills/Agent_Step02_ScriptExecutionWithCodeInterpreter/skills/password-generator/scripts/generate.pyDeletes script resource from the removed Step02 sample.
dotnet/samples/02-agents/AgentSkills/Agent_Step02_ScriptExecutionWithCodeInterpreter/skills/password-generator/references/PASSWORD_GUIDELINES.mdDeletes reference doc from the removed Step02 sample.
dotnet/samples/02-agents/AgentSkills/Agent_Step02_ScriptExecutionWithCodeInterpreter/skills/password-generator/SKILL.mdDeletes the sample skill definition for the removed Step02 sample.
dotnet/samples/02-agents/AgentSkills/Agent_Step02_ScriptExecutionWithCodeInterpreter/README.mdDeletes Step02 sample documentation.
dotnet/samples/02-agents/AgentSkills/Agent_Step02_ScriptExecutionWithCodeInterpreter/Program.csDeletes Step02 sample program.
dotnet/samples/02-agents/AgentSkills/Agent_Step02_ScriptExecutionWithCodeInterpreter/Agent_Step02_ScriptExecutionWithCodeInterpreter.csprojDeletes Step02 sample project.
dotnet/samples/02-agents/AgentSkills/Agent_Step01_BasicSkills/Program.csRemoves a warning comment block from the sample.
dotnet/agent-framework-dotnet.slnxRemoves Step02 sample project reference from the solution.

@SergeyMenshykh
SergeyMenshykh added this pull request to the merge queueMar 2, 2026
@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks Mar 2, 2026
@SergeyMenshykh
SergeyMenshykh added this pull request to the merge queueMar 2, 2026
Merged via the queue into main with commit 26cef55Mar 2, 2026
25 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationUsage: [Issues, PRs], Target: documentation in the code base and learn docs.NETUsage: [Issues, PRs], Target: .Net

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants

@SergeyMenshykh@rogerbarreto@westey-m@markwallace-microsoft