Python: Make foundry toolbox MCP skills sample self-contained#7099
Merged
giles17 merged 3 commits intoJul 14, 2026
Conversation
Rework sample 12 (foundry_toolbox_mcp_skills) so users can build it from zero with azd, mirroring samples 04 and 09: - Bundle two single-file SKILL.md skills (support-style, escalation-policy) and a skills-only toolbox.yaml (with one connectionless code_interpreter tool, required by `azd ai toolbox create`). - Rewrite the README as an azd-native, from-zero guide (create skills -> create toolbox -> set TOOLBOX_ENDPOINT -> run) and fix the stale MCPSkillsSource API description to match main.py. - Switch config from TOOLBOX_NAME to the versioned TOOLBOX_ENDPOINT (.env.example, agent.yaml, agent.manifest.yaml); add .azdignore. Also enable the sample to run unattended behind ResponsesHostServer: - Forward disable_load_skill_approval / disable_read_skill_resource_approval / disable_run_skill_script_approval from FoundryToolbox.as_skills_provider() to the underlying SkillsProvider, so load_skill needs no approval round-trip (the Responses host runs without an AgentSession, which the default approval flow requires). main.py now uses as_skills_provider(disable_load_skill_approval=True). - Add unit tests covering the default and overridden approval behaviour. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4f14f83d-1868-45c1-be1a-12f49a58ac36
giles17
added
documentation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Python 12_foundry_toolbox_mcp_skills Responses-hosted sample so it can be run end-to-end starting from an empty Foundry project (bundled skill sources + toolbox definition), and extends FoundryToolbox.as_skills_provider() to optionally disable skill-tool approval requirements for unattended hosting scenarios.
Changes:
- Bundles two single-file
SKILL.mdskills and atoolbox.yaml, and rewrites the sample README into an azd-native “from zero” walkthrough usingTOOLBOX_ENDPOINT. - Updates the sample’s config assets (
agent.yaml,agent.manifest.yaml,.env.example) to use a versioned toolbox endpoint instead ofTOOLBOX_NAME, and excludes authoring inputs from deployment via.azdignore/.dockerignore. - Extends
FoundryToolbox.as_skills_provider()to forwarddisable_*_skill_*_approvaloptions toSkillsProvider, and adds unit tests validating default/override behavior.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/toolbox.yaml | Adds a toolbox definition referencing the bundled skills (plus a placeholder tool to satisfy CLI constraints). |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/skills/support-style/SKILL.md | Adds bundled “support-style” skill source. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/skills/escalation-policy/SKILL.md | Adds bundled “escalation-policy” skill source. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/README.md | Rewrites docs to a from-zero azd walkthrough and explains progressive disclosure stages/constraints. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/main.py | Uses as_skills_provider(disable_load_skill_approval=True) for unattended hosting. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/agent.yaml | Switches configuration from TOOLBOX_NAME to TOOLBOX_ENDPOINT. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/agent.manifest.yaml | Switches manifest parameterization from TOOLBOX_NAME to TOOLBOX_ENDPOINT. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/.env.example | Documents and switches to TOOLBOX_ENDPOINT. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/.dockerignore | Excludes bundled skill/toolbox authoring inputs from container builds. |
| python/samples/04-hosting/foundry-hosted-agents/responses/12_foundry_toolbox_mcp_skills/.azdignore | Excludes authoring inputs and local env files from azd packaging/deploy context. |
| python/packages/foundry_hosting/tests/test_toolbox.py | Adds tests for approval override forwarding behavior. |
| python/packages/foundry_hosting/agent_framework_foundry_hosting/_toolbox.py | Adds approval-disabling kwargs to as_skills_provider() and forwards them into SkillsProvider. |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
Python Test Coverage Report •
Python Unit Test Overview
|
||||||||||||||||||||||||||||||
4 tasks
giles17
marked this pull request as ready for review
July 14, 2026 04:21
moonbox3
approved these changes
Jul 14, 2026
eavanvalkenburg
approved these changes
Jul 14, 2026
giles17
enabled auto-merge
July 14, 2026 15:18
TaoChenOSU
reviewed
Jul 14, 2026
giles17
disabled auto-merge
July 14, 2026 15:47
- Remove the unused parameters section from agent.manifest.yaml (TOOLBOX_ENDPOINT is supplied via environment_variables, matching sample 04). - README: state the sample is self-contained directly instead of contrasting with the C# sample. - README: describe skill discovery behaviour without naming the internal MCPSkillsSource class. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4f14f83d-1868-45c1-be1a-12f49a58ac36
TaoChenOSU
approved these changes
Jul 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation & Context
The
12_foundry_toolbox_mcp_skillssample was not self-contained and did not work end-to-end:04_foundry_toolbox(bundles atoolbox.yaml) and sample09_foundry_skills(bundlesSKILL.mdsources). A user starting from zero was stuck.SkillsProvider(MCPSkillsSource(...))) that did not matchmain.py(FoundryToolbox(...).as_skills_provider()), and used the fragileTOOLBOX_NAME(unversioned) env var instead of a versioned endpoint.SkillsProviderregistersload_skillwith approval required by default, butResponsesHostServerruns the agent without anAgentSession, so the default approval flow (andToolApprovalMiddleware) cannot complete — every skill-loading request returned an unansweredmcp_approval_request.This makes the sample a complete, from-zero, azd-native walkthrough and lets it run unattended behind the Responses host.
Description & Review Guide
What are the major changes?
SKILL.mdskills (support-style,escalation-policy, reused from sample 09) and atoolbox.yamlthat references them. The toolbox includes one connectionlesscode_interpretertool becauseazd ai toolbox createrejects a skills-only toolbox;load_tools=Falsekeeps it hidden from the model, so it stays effectively skills-only. Add.azdignoreand update.dockerignoreso authoring inputs are excluded from the deployed container.TOOLBOX_ENDPOINT→ run), fixing the staleMCPSkillsSourcedescription and documenting the two progressive-disclosure stages this path supports (advertise + load). Config switched fromTOOLBOX_NAMEto the versionedTOOLBOX_ENDPOINTacross.env.example,agent.yaml, andagent.manifest.yaml.agent-framework-foundry-hosting):FoundryToolbox.as_skills_provider()now forwardsdisable_load_skill_approval/disable_read_skill_resource_approval/disable_run_skill_script_approvalto the underlyingSkillsProvider.main.pyusesas_skills_provider(disable_load_skill_approval=True)so the hosted agent loads skills without an approval round-trip. Adds unit tests for the default and overridden behaviour.What is the impact of these changes?
support-style(canarySTYLE-CANARY-3318); an escalation prompt loads both skills (canaryESC-CANARY-7742). The package change is purely additive (new keyword-only args defaulting toFalse), so existing callers are unaffected.type: archive, which the PythonMCPSkillsSourcedoes not currently discover (it only handlestype: skill-md). The README explains this.What do you want reviewers to focus on?
as_skills_provider()is the preferred fix for theResponsesHostServer(noAgentSession) scenario, vs. an alternative.Related Issue
Contribution Checklist
breaking changelabel (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.