[docs] Update interaction file upload docs for GetFiles() and upload limits - #1554
Merged
James Newton-King (JamesNK) merged 5 commits intoAug 20, 2026
Conversation
… limit Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
14 tasks
Contributor
Author
Frontend HTML artifact readyThe latest frontend build uploaded the This comment updates automatically when a new frontend build artifact is uploaded. |
Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com>
James Newton-King (JamesNK)
approved these changes
Aug 20, 2026
James Newton-King (JamesNK)
marked this pull request as ready for review
August 20, 2026 09:16
James Newton-King (JamesNK)
requested a review
from David Pine (IEvangelist)
as a code owner
August 20, 2026 09:16
Copilot started reviewing on behalf of
James Newton-King (JamesNK)
August 20, 2026 09:16
View session
James Newton-King (JamesNK)
enabled auto-merge (squash)
August 20, 2026 09:18
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Interaction Service documentation to reflect the new file-upload API patterns and lifecycle semantics introduced in microsoft/aspire (moving from InteractionInput.Files to GetFiles() and documenting temporary-file cleanup expectations).
Changes:
- Updated the C# file-upload example to use
InteractionInput.GetFiles()and index into the returned collection. - Added guidance about temporary file ownership/disposal for C# and release semantics for TypeScript (
releaseFiles()in afinallyblock).
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
James Newton-King (JamesNK)
deleted the
docs/pr-19488-32349097745-1-440da2a5711588ec
branch
August 20, 2026 10:06
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
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.
Documents changes from microsoft/aspire#19488
@JamesNKTargeting
release/17.0— the latest release branch onmicrosoft/aspire.dev— becauserelease/13.6(from the source PR milestone13.6) does not exist there.Why
The source PR hardens interaction file uploads and changes the public API used to read uploaded files:
InteractionInput.GetFiles(), returning a new disposableInteractionFileCollection.InteractionInput.Files[Obsolete]in favor ofGetFiles().InteractionFile.OpenRead()/ReadAllBytesAsync()throwObjectDisposedException, though already-open streams remain usable until disposed.AllowMultipleFiles = true(1 file otherwise), enforced during upload with in-progress uploads counted as reserved slots.The existing
extensibility/interaction-service.mdx"Prompt for file upload" section used the oldresult.Data.Files?[0]pattern and didn't mention the new ownership/disposal model or the file-count limit, so it no longer matched the shipped API.Changes
src/frontend/src/content/docs/extensibility/interaction-service.mdx:using var files = result.Data.GetFiles();instead of the obsoleteFilesproperty.GetFiles()ownership/disposal semantics and thatFilesis obsolete.AllowMultipleFilesbullet to document the 100-file limit (1 file when not multi-file) and that in-progress uploads count toward it.Only this existing page was updated; no new pages were created.