Skip to content

Python: Clarify binary content purview classification - #10103

Merged
westey (westey-m) merged 1 commit into
microsoft:mainfrom
westey-m:purview-binary-content-docs-clarification
Nov 4, 2026
Merged

westey (westey-m) merged 1 commit into
microsoft:mainfrom
westey-m:purview-binary-content-docs-clarification

Conversation

@westey-m

Copy link
Copy Markdown
Collaborator

Motivation & Context

Improve documentation to explain how binary content is classified.

Description & Review Guide

Improve documentation to explain how binary content is classified.

Related Issue

Fixes #

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:

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 title prefix in sync automatically.

Copilot AI balanced review requested due to automatic review settings November 4, 2026 16:39
@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 .NET Usage: [Issues, PRs], Target: .Net labels Nov 4, 2026
@github-actions github-actions Bot changed the title Clarify binary content purview classification .NET: Clarify binary content purview classification Nov 4, 2026
@github-actions github-actions Bot changed the title .NET: Clarify binary content purview classification Python: Clarify binary content purview classification Nov 4, 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.

🟢 Approval recommended

No active review issues were identified in the documentation updates clarifying Purview binary-content classification and DLP policy behavior.

No open issues 🎉

What changed in this PR

This documentation-only PR clarifies how the Python and .NET Purview integrations classify binary content, aligning package documentation with consumer integration guidance.

Changes:

  • Explains that binary prompts use UploadText and binary responses use DownloadText, rather than file activities.
  • Warns that DLP policies targeting only UploadFile or DownloadFile do not block binary content through these integrations.
File Description
skills/​agent-framework-builder-python/​references/​integrations.md Adds Python Purview activity and DLP policy guidance.
skills/​agent-framework-builder-dotnet/​references/​integrations.md Adds equivalent .NET Purview guidance.
python/​packages/​purview/​README.md Documents binary-content activity mapping for both Python middleware classes.
dotnet/​src/​Microsoft.Agents.AI.Purview/​README.md Documents binary-content activity mapping for agent and chat middleware.

Review effort: Balanced

@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: No findings
Scope: full PR (1 commit(s)): 4bd1f3ddf42e
Model: gpt-5.6-sol

Overview

This documentation-only change accurately clarifies that both Purview middleware implementations retain UploadText and DownloadText activities when submitting binary content. The .NET and Python conversion paths preserve non-empty binary payloads as Purview binary content without changing the middleware-selected activity, and existing tests pin both aspects of that contract. No publishable risk remains in the authoritative diff.

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.

@westey-m
westey (westey-m) added this pull request to the merge queue Nov 4, 2026
Merged via the queue into microsoft:main with commit 0452e0d Nov 4, 2026
64 of 66 checks passed
@westey-m
westey (westey-m) deleted the purview-binary-content-docs-clarification branch November 4, 2026 17:12

This branch was previously deployed

1 inactive deployment
github-app-auth — 4bd1f3dd Deployed Nov 4, 2026 by westey-m via team_check #8767
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 .NET Usage: [Issues, PRs], Target: .Net python Usage: [Issues, PRs], Target: Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants