[BlueRaddish]

Motivation & Context

The file-based and mixed skills guides tell users to set AZURE_OPENAI_MODEL, but both samples read FOUNDRY_MODEL. Following either guide with a custom deployment silently leaves the sample on gpt-4o-mini. The mixed skills guide also supplies an Azure OpenAI account URL where the sample needs a Foundry project endpoint.

Description & Review Guide

• What are the major changes? Correct the model environment variable in both READMEs and use the canonical project-scoped Foundry endpoint placeholder in the mixed skills guide. Three documentation lines change.
• What is the impact of these changes? The setup instructions match the unchanged samples and the existing consolidated environment-variable inventory.
• What do you want reviewers to focus on? Running both complete, unchanged sample main() functions against a recording FoundryChatClient boundary reproduced two original custom-deployment failures and four passing controls. The corrected guides pass all six cases; the endpoint placeholder also satisfies the sample documentation convention. These checks stop before any agent or provider call and do not test live Foundry connectivity. The frozen all-workspace environment setup, ordinary scoped hooks, and normal native commit hooks passed, including the required whole-samples Ruff and Pyright checks. The resulting source commit was independently imported from its bundle against the pristine base and its exact three-line patch verified. The full unit suite and build were not run for this documentation-only change.

Related Issue

N/A - trivial change. These are obvious corrections to existing sample configuration names and the required project endpoint format. No runtime API or design decision changes.

AI Assistance

• This is an AI-assisted contribution. I reviewed, understood, and verified all submitted content and accept responsibility for it.

AI assistance details: OpenAI Codex assisted with diagnosis, the three documentation corrections, deterministic verification, and this description.

Contribution Checklist

• The code builds clean without any errors or warnings
• All unit tests pass, and I have added new tests where possible
• The PR follows the Contribution Guidelines
• This PR links to an agreed issue with no competing open PR, or the Related Issue section documents a trivial-change or repository-automation exception.
• This is not a breaking change.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[Eduard van Valkenburg (eavanvalkenburg)]

/review

[github-actions]

MAF Automated Review — Iteration 1

Result: No findings
Scope: full PR (1 commit(s)): eb647fef2417
Model: gpt-5.6-sol

Overview

The documentation changes align both skills guides with the unchanged sample entry points and the FoundryChatClient environment contract. The mixed-skills endpoint also follows the repository's required project-scoped Foundry format, while existing client guards and tests confirm that the corrected values are forwarded or fail fast when absent. No publishable risk remains in the reviewed changes.

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
No publishable findings remained after source verification for this scope.