Added consolidated quickstart versions for model and agent of BasicVoiceAssistant sample … #53547

goergenj · 2025-10-29T00:20:34Z

…for Voice live

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

…for Voice live

github-actions · 2025-10-29T00:20:53Z

Thank you for your contribution @goergenj! We will review the pull request and get back to you soon.

Copilot

Pull Request Overview

This PR adds two standalone, consolidated quickstart samples for the Azure.AI.VoiceLive SDK, along with updating configuration templates in the main sample project. The quickstarts demonstrate voice assistant capabilities through self-contained examples that can be distributed independently.

Adds consolidated quickstart samples for model-based and agent-based voice assistants
Updates configuration templates with new Azure endpoint patterns and voice models
Adds root-level project files (appear to be unintentional)

Reviewed Changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
model_quickstart/Program.cs	Self-contained voice assistant sample using VoiceLive models directly
agent_quickstart/Program.cs	Self-contained voice assistant sample connecting to Azure AI Foundry agents
model_quickstart/README.md	Documentation for the model-based quickstart sample
agent_quickstart/README.md	Documentation for the agent-based quickstart sample
model_quickstart/appsettings.template.json	Configuration template for model quickstart
agent_quickstart/appsettings.template.json	Configuration template for agent quickstart (missing comma)
model_quickstart/*.csproj, Directory.Build.props, NuGet.Config	Build configuration for standalone model quickstart
agent_quickstart/*.csproj, Directory.Build.props, NuGet.Config	Build configuration for standalone agent quickstart
BasicVoiceAssistant/appsettings.template.json	Updated with new Azure endpoints and voice models
BasicVoiceAssistant/appsettings.json	File deleted (now gitignored)
azure-sdk-for-net.csproj, Program.cs	Unintentional root-level project files
.gitignore	Added global appsettings.json ignore

...ples/BasicVoiceAssistant/consolidated_quickstarts/agent_quickstart/appsettings.template.json

...I.VoiceLive/samples/BasicVoiceAssistant/consolidated_quickstarts/model_quickstart/Program.cs

...I.VoiceLive/samples/BasicVoiceAssistant/consolidated_quickstarts/agent_quickstart/Program.cs

azure-sdk-for-net.csproj

Program.cs

jsquire

@goergenj:

Your account lacks the public GitHub organization memberships and permissions required of an internal contributor. Please review the Azure SDK onboarding documentation and use the associated Teams channel for support.

Azure SDK onboarding (Microsoft internal)
Azure SDK onboarding assistance (Microsoft internal)

You can verify the state of your account by running the Validate-AzsdkCodeOwner script from the Azure SDK tools repository.

Please also be sure to add yourself to CODEOWNERS for this library, if you will be maintaining it going forward.

jsquire · 2025-10-29T16:31:00Z

.gitignore

 # Common toolchain intermediate files
 temp
+
+# Appsettings files


Please revert. You should not be making changes outside of your /sdk directory.

Should be reverted.

jsquire · 2025-10-29T16:31:15Z

Program.cs

@@ -0,0 +1,2 @@
+// See https://aka.ms/new-console-template for more information
+Console.WriteLine("Hello, World!");


Should be reverted.

jsquire · 2025-10-29T16:31:29Z

azure-sdk-for-net.csproj

@@ -0,0 +1,11 @@
+<Project Sdk="Microsoft.NET.Sdk">


Should be reverted.

jsquire · 2025-10-29T16:32:10Z

...VoiceLive/samples/BasicVoiceAssistant/consolidated_quickstarts/model_quickstart/NuGet.Config

+  <packageSources>
+    <clear />
+    <!-- Use only the public NuGet.org feed for standalone distribution -->
+    <add key="nuget.org" value="https://api.nuget.org/v3/index.json" protocolVersion="3" />


This needs to be removed. Only the approved root NuGet sources may be used in this repository.

jsquire · 2025-10-29T16:32:42Z

...VoiceLive/samples/BasicVoiceAssistant/consolidated_quickstarts/agent_quickstart/NuGet.Config

+  <packageSources>
+    <clear />
+    <!-- Use only the public NuGet.org feed for standalone distribution -->
+    <add key="nuget.org" value="https://api.nuget.org/v3/index.json" protocolVersion="3" />


This needs to be removed. Only the approved root source may be used in this repository.

jsquire · 2025-10-29T16:33:38Z

...zure.AI.VoiceLive/samples/AgentVoiceAssistantQuickstart/AgentVoiceAssistantQuickstart.csproj

+  </ItemGroup>
+
+  <!-- Use project reference (works in the repository context) -->
+  <ItemGroup>


This will fail to build outside of the repository.

I updated this based on the version in the parent folder and tested the new version. Please re-check.

The parent folder is incorrect and will not build outside of this repository. It also snuck in an unapproved dependency via use of override which will be remediated.

Stand-alone samples must be buildable inside the repository and stand-alone, as the docs process will export them as stand-alone samples.

jsquire · 2025-10-29T16:36:05Z

...icVoiceAssistant/consolidated_quickstarts/agent_quickstart/ConsolidatedVoiceAssistant.csproj

+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="Azure.Identity" Version="1.12.1"/>


This needs to be more robust such that the repository central versions get used when building in the repo and you have fallback versions outside the repository.

Example: Event Hubs

Example: Key Vault

Updated based on the version in parent folder present before now without the Version.

jsquire · 2025-10-29T16:38:38Z

...icVoiceAssistant/consolidated_quickstarts/agent_quickstart/ConsolidatedVoiceAssistant.csproj

+
+  <ItemGroup>
+    <PackageReference Include="Azure.Identity" Version="1.12.1"/>
+    <PackageReference Include="NAudio" Version="2.2.1"/>


This is not an approved dependency and may not be used in the repository. For consideration as an approved test/sample reference, please submit an email request to myself, Krzysztof Cwalina, Christopher Scott, and Scott Addie.

Updated based on the version in parent folder present before.

jsquire · 2025-10-29T16:39:14Z

...oiceAssistant/consolidated_quickstarts/model_quickstart/BasicVoiceAssistantQuickstart.csproj

+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="Azure.Identity" Version="1.12.1"/>


Please see comments from the other .csproj. They apply here as well.

I updated this based on the version in the parent folder and tested the new version. Please re-check.

sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistant/.gitignore

m-redding

please revert the root level changes

…ated_quickstarts/agent_quickstart/appsettings.template.json Co-authored-by: Copilot <[email protected]>

…ated_quickstarts/model_quickstart/Program.cs Co-authored-by: Copilot <[email protected]>

…ated_quickstarts/agent_quickstart/Program.cs Co-authored-by: Copilot <[email protected]>

Co-authored-by: Copilot <[email protected]>

…agent code issue

m-redding

root is resolved thanks!

resolved

...VoiceLive/samples/BasicVoiceAssistant/consolidated_quickstarts/agent_quickstart/NuGet.Config

jsquire · 2025-10-30T16:41:51Z

...zure.AI.VoiceLive/samples/AgentVoiceAssistantQuickstart/AgentVoiceAssistantQuickstart.csproj

+  </ItemGroup>
+
+  <!-- Use project reference (works in the repository context) -->
+  <ItemGroup>


The parent folder is incorrect and will not build outside of this repository. It also snuck in an unapproved dependency via use of override which will be remediated.

Stand-alone samples must be buildable inside the repository and stand-alone, as the docs process will export them as stand-alone samples.

jsquire · 2025-10-30T17:09:09Z

Related: #53586

… in the samples folder for clear structure

goergenj · 2025-10-31T23:00:13Z

@jsquire I removed the NAudio package from the csproj and put it as a pre-requisite requirement in then README until this topic can be solved.

Copilot

Pull Request Overview

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

sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistantQuickstart/README.md

sdk/ai/Azure.AI.VoiceLive/samples/AgentVoiceAssistantQuickstart/README.md

sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistantQuickstart/NuGet.Config

sdk/ai/Azure.AI.VoiceLive/samples/AgentVoiceAssistantQuickstart/NuGet.Config

sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistantQuickstart/README.md

sdk/ai/Azure.AI.VoiceLive/samples/AgentVoiceAssistantQuickstart/README.md

sdk/ai/Azure.AI.VoiceLive/samples/.gitignore

Co-authored-by: Copilot <[email protected]>

…t/NuGet.Config Co-authored-by: Copilot <[email protected]>

…t/README.md Co-authored-by: Copilot <[email protected]>

…t/NuGet.Config Co-authored-by: Copilot <[email protected]>

Added consolidated quickstart versions of BasicVoiceAssistant sample …

5502723

…for Voice live

goergenj requested review from jsquire, rhurey and xitzhang as code owners October 29, 2025 00:20

Copilot AI review requested due to automatic review settings October 29, 2025 00:20

goergenj requested a review from a team as a code owner October 29, 2025 00:20

github-actions bot added Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization. Voice Live labels Oct 29, 2025

Copilot AI reviewed Oct 29, 2025

View reviewed changes

jsquire requested changes Oct 29, 2025

View reviewed changes

m-redding previously requested changes Oct 29, 2025

View reviewed changes

goergenj and others added 9 commits October 29, 2025 09:46

Update sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistant/consolid…

d20a1ce

…ated_quickstarts/agent_quickstart/appsettings.template.json Co-authored-by: Copilot <[email protected]>

Update sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistant/consolid…

a3adad1

…ated_quickstarts/model_quickstart/Program.cs Co-authored-by: Copilot <[email protected]>

Update sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistant/consolid…

5b45252

…ated_quickstarts/agent_quickstart/Program.cs Co-authored-by: Copilot <[email protected]>

Update azure-sdk-for-net.csproj

65c7b57

Co-authored-by: Copilot <[email protected]>

Update Program.cs

1973045

Co-authored-by: Copilot <[email protected]>

Delete Program.cs

698a4af

Updated based on review comments

c1ddc59

Updated based on review comments

0c899b0

Fixing comments from review, cleaning up unrequired files and fixing …

dcd2242

…agent code issue

m-redding reviewed Oct 29, 2025

View reviewed changes

jsquire reviewed Oct 30, 2025

View reviewed changes

goergenj added 4 commits October 31, 2025 15:45

Renamed consolidated learn docs quickstart versions and moved them up…

997eff0

… in the samples folder for clear structure

Fixed review comment.

3be18d6

Fixed review comment.

957bfa2

Removed NAudio unapproved dependency package.

dc41415

goergenj requested a review from Copilot October 31, 2025 23:04

Copilot AI reviewed Oct 31, 2025

View reviewed changes

goergenj and others added 5 commits October 31, 2025 16:46

Update sdk/ai/Azure.AI.VoiceLive/samples/.gitignore

9d6f76d

Co-authored-by: Copilot <[email protected]>

Update sdk/ai/Azure.AI.VoiceLive/samples/AgentVoiceAssistantQuickstar…

8eb4d04

…t/NuGet.Config Co-authored-by: Copilot <[email protected]>

Update sdk/ai/Azure.AI.VoiceLive/samples/AgentVoiceAssistantQuickstar…

a81208d

…t/README.md Co-authored-by: Copilot <[email protected]>

Update sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistantQuickstar…

b4f98ef

…t/README.md Co-authored-by: Copilot <[email protected]>

Update sdk/ai/Azure.AI.VoiceLive/samples/BasicVoiceAssistantQuickstar…

a32cfcd

…t/NuGet.Config Co-authored-by: Copilot <[email protected]>

goergenj requested a review from jsquire October 31, 2025 23:48

		@@ -0,0 +1,2 @@
		// See https://aka.ms/new-console-template for more information
		Console.WriteLine("Hello, World!");

Added consolidated quickstart versions for model and agent of BasicVoiceAssistant sample … #53547

Are you sure you want to change the base?

Added consolidated quickstart versions for model and agent of BasicVoiceAssistant sample … #53547

Conversation

goergenj commented Oct 29, 2025

Contributing to the Azure SDK

Uh oh!

github-actions bot commented Oct 29, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jsquire left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

m-redding left a comment

Choose a reason for hiding this comment

Uh oh!

m-redding left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jsquire commented Oct 30, 2025

Uh oh!

goergenj commented Oct 31, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!