Skip to content

Python: Add installable Agent Framework builder skill - #9374

Merged
Eduard van Valkenburg (eavanvalkenburg) merged 13 commits into
mainfrom
maf-consumer-skills
Oct 22, 2026
Merged

Eduard van Valkenburg (eavanvalkenburg) merged 13 commits into
mainfrom
maf-consumer-skills

Conversation

@eavanvalkenburg

@eavanvalkenburg Eduard van Valkenburg (eavanvalkenburg) commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Motivation & Context

Developers using coding agents currently have no installable, MAF-owned skill that explains the Python framework abstractions, helps select between agents, harness agents, workflows, and declarative definitions, and directs implementation toward current APIs and packages. Existing repository skills target contributors working on Agent Framework itself rather than customers building Agent Framework applications.

This adds a public Python consumer skill to the code repository while keeping it outside project-skill discovery locations. Repository coding agents therefore do not load it as contributor guidance, but contributor instructions require reviewing it when public Python APIs, packages, samples, integrations, or hosting behavior change.

This draft depends on #9363 for the documented Python 3.15 compatibility changes.

Description & Review Guide

  • What are the major changes? Added agent-framework-builder-python under the public skills/ catalog with a concise entry skill and focused references for agents, harness composition, workflows and orchestrations, declarative definitions, evaluation, observability, integrations, and hosting. Added install instructions, root README discovery, CODEOWNERS, contributor maintenance guidance, and a path-scoped workflow that validates the Agent Skills specification, relative resources, and GitHub publishing readiness.
  • What is the impact of these changes? Compatible coding agents can install one MAF-owned skill from microsoft/agent-framework and receive current Python guidance, including the post-Python: Add Python 3.15 support and refresh dependencies #9363 Python compatibility model. Normal coding-agent work inside this repository continues to use the existing .github/skills and AGENTS.md instructions without automatically loading the consumer skill. Changes under skills/ now receive dedicated automated validation.
  • What do you want reviewers to focus on? Whether the main skill contains the right minimal framework mental model and routing table, while each reference stays focused, current, and customer-facing rather than inheriting internal repository conventions.

Related Issue

Related to #7132

This PR delivers the Python consumer skill. The issue remains open for the .NET counterpart, and no competing open PR was found.

AI Assistance

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

AI assistance details: GitHub Copilot assisted with skill migration, API and documentation review, reference organization, validation, and the PR 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. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and the title prefix in sync automatically.

Copilot AI balanced review requested due to automatic review settings October 14, 2026 12:30
@agent-framework-automation agent-framework-automation Bot added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs python Usage: [Issues, PRs], Target: Python labels Oct 14, 2026

Copilot AI 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.

🟡 Changes recommended

The guidance misstates Python support and makes inaccurate security claims about telemetry redaction and HTTP response limits.

3 open findings
What changed in this PR

Adds an installable Python Agent Framework builder skill and contributor guidance for maintaining it.

Changes:

  • Adds guidance for agents, workflows, integrations, evaluation, observability, and hosting.
  • Documents installation and repository discovery.
  • Establishes ownership and maintenance expectations.
File Description
skills/​README.md Adds the consumer skill catalog.
skills/​agent-framework-builder-python/​SKILL.md Defines the skill’s routing and core guidance.
skills/​agent-framework-builder-python/​references/​workflow-builder.md Documents workflows and orchestrations.
skills/​agent-framework-builder-python/​references/​observability.md Documents telemetry configuration.
skills/​agent-framework-builder-python/​references/​integrations.md Covers providers and integrations.
skills/​agent-framework-builder-python/​references/​hosting-and-deployment.md Covers hosting and deployment.
skills/​agent-framework-builder-python/​references/​harness-agent-builder.md Documents harness agents.
skills/​agent-framework-builder-python/​references/​evaluation.md Documents evaluation APIs.
skills/​agent-framework-builder-python/​references/​declarative.md Covers declarative construction.
skills/​agent-framework-builder-python/​references/​agent-builder.md Documents plain agents.
README.md Advertises skill installation.
python/​AGENTS.md Adds consumer-skill maintenance guidance.
.github/​copilot-instructions.md Distinguishes public and contributor skills.
.github/​CODEOWNERS Assigns Python skill ownership.

🧠 Review effort: Balanced


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread skills/agent-framework-builder-python/SKILL.md Outdated
Comment thread skills/agent-framework-builder-python/references/declarative.md Outdated
Comment thread skills/agent-framework-builder-python/references/observability.md Outdated

@github-actions github-actions Bot 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.

MAF Automated Review — Iteration 1

Result: Findings reported
Scope: full PR (1 commit(s)): 1ca7119b63cf
Model: gpt-5.6-sol

Overview

The PR cleanly separates the installable consumer skill from contributor instructions and routes users through focused references, with several recommendations backed by current implementation guards and tests. The minimal harness example disables file memory, and the references include useful authorization, ownership, and sensitive-data warnings. Residual risk remains where later guidance permits the default unbounded file-memory store and where compatibility, response-size, and observability claims do not match current package metadata or runtime behavior.

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
6 verified findings remained after source verification (1 high, 5 medium) across 5 files. Details are attached to the affected lines below.

Affected areas: skills/agent-framework-builder-python/SKILL.md, skills/agent-framework-builder-python/references/declarative.md, skills/agent-framework-builder-python/references/harness-agent-builder.md, skills/agent-framework-builder-python/references/integrations.md, and 1 more

Comment thread skills/agent-framework-builder-python/references/harness-agent-builder.md Outdated
Comment thread skills/agent-framework-builder-python/references/declarative.md Outdated
Comment thread skills/agent-framework-builder-python/SKILL.md Outdated
Comment thread skills/agent-framework-builder-python/references/integrations.md Outdated
Comment thread skills/agent-framework-builder-python/references/integrations.md Outdated
Comment thread skills/agent-framework-builder-python/references/observability.md Outdated
Comment thread skills/agent-framework-builder-python/SKILL.md
@eavanvalkenburg
Eduard van Valkenburg (eavanvalkenburg) marked this pull request as ready for review October 15, 2026 12:10
Comment thread skills/agent-framework-builder-python/SKILL.md Outdated
Comment thread README.md Outdated

@jpalvarezl Jose Alvarez (jpalvarezl) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The Power Fx compatibility note is the substantive issue. The Python support wording, link-validator hardening, and release coordination comments are explicitly non-blocking.

Comment thread skills/agent-framework-builder-python/references/declarative.md
Comment thread skills/agent-framework-builder-python/SKILL.md Outdated
Comment thread .github/scripts/validate_agent_skill_links.py Outdated
Comment thread .github/workflows/agent-skills.yml
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 842af08a-f886-4564-b30a-9e7e9ebe324b
Merged via the queue into main with commit d0c3e6a Oct 22, 2026
33 checks passed
@eavanvalkenburg
Eduard van Valkenburg (eavanvalkenburg) deleted the maf-consumer-skills branch October 22, 2026 13:21

This branch was successfully deployed

1 active deployment
github-app-auth — b1df0aa8 Deployed Oct 22, 2026 by eavanvalkenburg via add_label #25929
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs python Usage: [Issues, PRs], Target: Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants