Skip to content

.NET: docs/workflow fileinput sample dotnet - #7913

Merged
Vincent Biret (baywet) merged 7 commits into
mainfrom
docs/workflow-fileinput-sample-dotnet
Aug 27, 2026
Merged

.NET: docs/workflow fileinput sample dotnet#7913
Vincent Biret (baywet) merged 7 commits into
mainfrom
docs/workflow-fileinput-sample-dotnet

Conversation

@baywet

Copy link
Copy Markdown
Member

fixes #2772

Signed-off-by: Vincent Biret <vibiret@microsoft.com>
…ontent ones

Signed-off-by: Vincent Biret <vibiret@microsoft.com>
Copilot AI balanced review requested due to automatic review settings August 27, 2026 12:47
@agent-framework-automation agent-framework-automation Bot added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows labels Aug 27, 2026
@github-actions github-actions Bot changed the title docs/workflow fileinput sample dotnet .NET: docs/workflow fileinput sample dotnet Aug 27, 2026

This comment was marked as 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: No findings
Scope: full PR (2 commit(s)): c4d0259974e6, bc0eb76c8062
Model: gpt-5.6-sol

Overview

This PR adds a declarative workflow sample that uploads a local file, supplies it as HostedFileContent in a ChatMessage, and cleans up the uploaded file after execution. The shared runner overload delegates to the existing execution path, while the declarative root explicitly accepts ChatMessage inputs and existing integration coverage uses the same upload purpose. No publishable Critical, High, or Medium defect remains after reconciling the proposed failure modes against those guards.

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.

Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
Copilot AI and others added 3 commits August 27, 2026 13:04
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
Comment thread dotnet/samples/03-workflows/Declarative/README.md Outdated

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.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

Suppressed comments (1)

dotnet/samples/03-workflows/Declarative/FileInput/Program.cs:63

  • The sample uploads ProductBrief.txt, but this definition creates an agent with no tool that can read it. Foundry Responses direct file input supports PDFs, while .txt is supported through Code Interpreter; without that tool, the default dotnet run path cannot produce the advertised summary. Enable Code Interpreter with an automatic container so the file already present in the model input is made available to the agent.
    private static DeclarativeAgentDefinition DefineFileInputAgent(IConfiguration configuration) =>
        new(configuration.GetValue(Application.Settings.FoundryModel))
        {
            Instructions =

@baywet
Vincent Biret (baywet) added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit 341d0ec Aug 27, 2026
27 checks passed
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 workflows Usage: [Issues, PRs], Target: Workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.NET Workflows - Provide sample that demonstrate providing file based input to workflow

4 participants