Skip to content

.NET: Enable Microsoft.Agents.AI.FoundryMemory for NuGet release#4559

Merged
rogerbarreto merged 5 commits intomicrosoft:mainfrom
rogerbarreto:issues/foundry-memory-package-release-enablement
Mar 10, 2026
Merged

.NET: Enable Microsoft.Agents.AI.FoundryMemory for NuGet release#4559
rogerbarreto merged 5 commits intomicrosoft:mainfrom
rogerbarreto:issues/foundry-memory-package-release-enablement

Conversation

@rogerbarreto
Copy link
Member

Motivation and Context

Enable Release of Package Microsoft.Agents.AI.FoundryMemory.

  • Remove IsPackable=false override from .csproj to inherit IsPackable=true from nuget-package.props
  • Add project to agent-framework-release.slnf for inclusion in build/sign/publish pipeline

- Remove IsPackable=false override from .csproj to inherit IsPackable=true from nuget-package.props
- Add project to agent-framework-release.slnf for inclusion in build/sign/publish pipeline

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 9, 2026 14:29
@rogerbarreto rogerbarreto enabled auto-merge March 9, 2026 14:29
@github-actions github-actions bot changed the title .Net: Enable Microsoft.Agents.AI.FoundryMemory for NuGet release .NET: Enable Microsoft.Agents.AI.FoundryMemory for NuGet release Mar 9, 2026
@rogerbarreto rogerbarreto disabled auto-merge March 9, 2026 14:30
Copy link
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

Enables NuGet packaging and release pipeline inclusion for Microsoft.Agents.AI.FoundryMemory, aligning it with the repo’s standard nuget-package.props packaging configuration.

Changes:

  • Removed the IsPackable=false override so the project inherits IsPackable=true from dotnet/nuget/nuget-package.props.
  • Added Microsoft.Agents.AI.FoundryMemory.csproj to dotnet/agent-framework-release.slnf to include it in build/sign/publish.

Reviewed changes

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

File Description
dotnet/src/Microsoft.Agents.AI.FoundryMemory/Microsoft.Agents.AI.FoundryMemory.csproj Removes the explicit opt-out from packing so the project becomes packable via shared NuGet props.
dotnet/agent-framework-release.slnf Includes FoundryMemory in the release solution filter used by the release pipeline.

- StoreAIContextAsync fires UpdateMemoriesAsync immediately (non-accumulation)
- WhenUpdatesCompletedAsync polls last updateId via GetUpdateResultAsync
- Updated FoundryAgents_Step22_MemorySearch sample to create/destroy memory store
  (matching features/foundry-agent-client pattern)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
rogerbarreto and others added 2 commits March 9, 2026 16:45
- Sample now creates/destroys memory store (self-contained lifecycle)
- Uses WaitForMemoriesUpdateAsync for seeding memories
- Cleanup in finally block deletes both agent and memory store
- Matches features/foundry-agent-client pattern

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@rogerbarreto rogerbarreto added this pull request to the merge queue Mar 9, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 9, 2026
@rogerbarreto rogerbarreto added this pull request to the merge queue Mar 9, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 9, 2026
Copilot AI review requested due to automatic review settings March 10, 2026 13:25
@rogerbarreto rogerbarreto enabled auto-merge March 10, 2026 13:25
Copy link
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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants