skills: lead with uv for all Python tooling + SDK on PyPI + AGENTS.md#1741
Open
paulyuk wants to merge 4 commits into
Open
skills: lead with uv for all Python tooling + SDK on PyPI + AGENTS.md#1741paulyuk wants to merge 4 commits into
paulyuk wants to merge 4 commits into
Conversation
The azure-containerapps-sandbox SDK has been published to PyPI ahead of GA. Replace the curl + local-wheel install workaround with the canonical 'pip install azure-containerapps-sandbox' (also works with uv). The 'Python SDK - install + use' section further down in SKILL.md already documents the PyPI path; this aligns the cue-row in the acceptance criteria table with that. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The repo standardizes on 'uv' as the canonical Python tool. Drop 'or pip install' branches and bare 'python3' invocations from skill docs and scenarios; replace with 'uv pip install' / 'uv run'. Adds AGENTS.md at the repo root documenting the policy for AI agents (Copilot CLI, Claude Code, etc.) and human contributors: - Lead with uv pip install / uv run / uv venv / uvx - Never lead with pip, pip3, python, python3 - No 'or pip install ...' alternative branches - Pre-PR checklist Affected files: - AGENTS.md (new) - plugin/skills/aca-sandboxes/SKILL.md: SDK install cue + SDK section - plugin/skills/aca-sandboxes/references/scenarios.md: web-app + code-interpreter sketches now use 'uv run' instead of 'python3' Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
paulyuk
changed the title
Adds the 'check before invoke' preflight pattern to the SDK install guidance in SKILL.md (both the acceptance-criteria cue row and the 'Python SDK - install + use' section show bash + PowerShell variants), and adds a corresponding hard rule + checklist item to AGENTS.md. This closes the gap noted in PR review: previously the docs assumed 'uv' was on PATH and would fail with 'command not found' for users who don't have it. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…n floor to 3.13+ Adds a new section to AGENTS.md, 'Language runtime versions: recommend mainstream LTS only', with a per-stack floor table (Python 3.13+, Node Active LTS, .NET LTS, Go latest two, Java latest LTS), 4 rules for keeping doc floors honest, and a rationale. Bumps scripts/region-discovery/README.md from Python 3.12+ to 3.13+ to match. (CI workflow region-discovery.yml still pins 3.12 for now - that bump should happen in a separate infra PR after testing.) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
paulyuk
changed the title
annaji-msft
approved these changes
Jun 1, 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
2 participants
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.
What
Three related doc changes prepping for prod / release:
1. SDK install:
uv pip install azure-containerapps-sandboxazure-containerapps-sandboxis now on PyPI. Dropped the old curl-and-install-wheel workaround from the Install the Python SDK acceptance-criteria row inplugin/skills/aca-sandboxes/SKILL.mdand the matching## Python SDK — install + usesection.2. Repo-wide policy: lead with
uvNew
AGENTS.mdat the repo root documents the canonical Python tooling for this repo. Hard rules:uv pip install/uv run/uv venv/uvx.pip,pip3,python,python3.uv pip install foo(orpip/pythonare acceptable when quoting third-party docs or showing legacy/error output — but never as the recommended command.This file is read by GitHub Copilot CLI, Claude Code, Cursor, and other agents working in the repo.
3. Fix existing instances
plugin/skills/aca-sandboxes/SKILL.md— removed the(oruv pip install …)branch in the cue row; switched the SDK code block frompip installtouv pip install.plugin/skills/aca-sandboxes/references/scenarios.md— Scenario 1 (web app) and Scenario 3 (code interpreter) now invokeuv run /tmp/step.py/uv run /app/server.pyinstead ofpython3 ….--sandbox-groupaudit (also requested)Verified every remaining occurrence is legitimate — no stale uses:
SKILL.mdL33–L39aca sandbox-group(hyphenated verb) is fabricated;--sandbox-groupis the real flagSKILL.mdL67containerapps.azure.com/sandbox-groupsreferences/quickstart.mdmy-sandbox-groupreferences/reference.md(5 lines)--sandbox-groupflag — still present inaca --helpscripts/verify-aca-verbs.mjs+scripts/README.mdFiles changed
AGENTS.md(new)plugin/skills/aca-sandboxes/SKILL.mdplugin/skills/aca-sandboxes/references/scenarios.mdTest
aca --helpconfirms--sandbox-groupglobal flag still exists.uv pip install azure-containerapps-sandboxresolves from PyPI.grepfor\bpip3?\s+installand\bpython3?\s+acrossplugin/returns no leading-instruction matches after this change.Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com