Skip to content

Python: Add Python package release skill#6356

Open
moonbox3 wants to merge 2 commits into
microsoft:mainfrom
moonbox3:add-py-release-skill
Open

Python: Add Python package release skill#6356
moonbox3 wants to merge 2 commits into
microsoft:mainfrom
moonbox3:add-py-release-skill

Conversation

@moonbox3
Copy link
Copy Markdown
Contributor

@moonbox3 moonbox3 commented Jun 5, 2026

Motivation and Context

Add Python package release skill

Description

Add Python package release skill

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

@moonbox3 moonbox3 self-assigned this Jun 5, 2026
Copilot AI review requested due to automatic review settings June 5, 2026 04:09
@moonbox3 moonbox3 added documentation Improvements or additions to documentation python labels Jun 5, 2026
@github-actions github-actions Bot changed the title Add Python package release skill Python: Add Python package release skill Jun 5, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new Python agent skill that documents the end-to-end workflow for preparing Python package release PRs in the microsoft/agent-framework monorepo, and wires it into the Python agent documentation index.

Changes:

  • Add agent-framework-py-release skill describing CHANGELOG-driven version bumping, lifecycle tier rules, and dependency-floor validation.
  • Update python/AGENTS.md to list the new release skill alongside existing Python skills.

Reviewed changes

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

File Description
python/AGENTS.md Adds the new agent-framework-py-release skill to the Python skills index.
python/.github/skills/agent-framework-py-release/SKILL.md Introduces a detailed Python release PR workflow skill (CHANGELOG-driven bumps, tier rules, validation steps).

Comment thread python/.github/skills/agent-framework-py-release/SKILL.md
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md
Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Automated Code Review

Reviewers: 4 | Confidence: 95%

✓ Correctness

This PR adds a new Python release skill document and a corresponding reference in AGENTS.md. Both changes are documentation-only. The skill file references real, verified repo infrastructure (PACKAGE_STATUS.md, validate-dependency-bounds-test poe task, the root pyproject.toml core pin, and the validator script). The AGENTS.md entry follows the existing format. No correctness issues found.

✓ Security Reliability

This PR adds a new AI agent skill document (SKILL.md) with release workflow instructions and updates AGENTS.md to reference it. The content is purely instructional documentation for AI coding agents — no executable code, no CI pipeline changes, no secrets, and no trust-boundary interactions. The shell commands shown are illustrative templates, not auto-executed scripts. No security or reliability issues identified.

✓ Test Coverage

This PR adds a new documentation-only skill file (SKILL.md) for Python release workflows and a one-line reference in AGENTS.md. No production code or behavioral logic is changed. The existing .github/skills/ directory already contains six other skill files (python-development, python-testing, python-code-quality, python-feature-lifecycle, python-package-management, python-samples), none of which have dedicated tests for their content or consistency with AGENTS.md. The core skills framework (FileSkill parsing, discovery) is well-tested in python/packages/core/tests/core/test_skills.py, but those tests cover the skill-loading machinery, not individual skill document content. Since this PR follows the established pattern for adding skills and introduces no executable code, there is no test coverage gap specific to this change.

✓ Design Approach

The new release skill is close, but its step 4a reconciliation workflow has two design flaws that make the changelog-driven bump check unreliable: the sample awk extraction does not actually capture the drafted section body, and the reconciliation logic still excludes root agent-framework changes even though the skill says root changes can drive an independent bump.


Automated review by moonbox3's agents

Comment thread python/.github/skills/agent-framework-py-release/SKILL.md Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants