Initial release for Azure.ResourceManager.Resources.Bicep #50751

anthony-c-martin · 2025-06-20T16:07:30Z

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.

Copilot

Pull Request Overview

Initial release for the Azure.ResourceManager.Resources.Bicep client library, integrating Bicep management capabilities into the Azure SDK.

Updated CI to include the new Bicep package
Added project scaffolding: source, tests, samples, solution, and metadata files
Defined API surfaces, documentation template, changelog, and asset publishing configuration

Reviewed Changes

Copilot reviewed 16 out of 35 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
sdk/resources/ci.mgmt.yml	Include Bicep package in CI pipeline
sdk/resources/Azure.ResourceManager.Resources.Bicep/tsp-location.yaml	Register Bicep spec directory and commit in location map
sdk/resources/Azure.ResourceManager.Resources.Bicep/tests/BicepManagementTestEnvironment.cs	Add empty test environment class for Bicep tests
sdk/resources/Azure.ResourceManager.Resources.Bicep/tests/BicepManagementTestBase.cs	Add common test base with client and RG helper
sdk/resources/Azure.ResourceManager.Resources.Bicep/tests/Azure.ResourceManager.Resources.Bicep.Tests.csproj	Create test project referencing Bicep implementation
sdk/resources/Azure.ResourceManager.Resources.Bicep/src/Properties/AssemblyInfo.cs	Configure AzureResourceProviderNamespace and InternalsVisibleTo
sdk/resources/Azure.ResourceManager.Resources.Bicep/src/GlobalSuppressions.cs	Suppress naming-rule warnings for generated models
sdk/resources/Azure.ResourceManager.Resources.Bicep/src/Azure.ResourceManager.Resources.Bicep.csproj	Define package metadata (version, ID, tags)
sdk/resources/Azure.ResourceManager.Resources.Bicep/samples/Azure.ResourceManager.Resources.Bicep.Samples.csproj	Define sample project references
sdk/resources/Azure.ResourceManager.Resources.Bicep/assets.json	Configure asset publishing repository and prefixes
sdk/resources/Azure.ResourceManager.Resources.Bicep/api/Azure.ResourceManager.Resources.Bicep.netstandard2.0.cs	Define .NET Standard API surface for Bicep client
sdk/resources/Azure.ResourceManager.Resources.Bicep/api/Azure.ResourceManager.Resources.Bicep.net8.0.cs	Define .NET 8.0 API surface for Bicep client
sdk/resources/Azure.ResourceManager.Resources.Bicep/README.md	Add documentation template with placeholders
sdk/resources/Azure.ResourceManager.Resources.Bicep/Directory.Build.props	Import shared build props
sdk/resources/Azure.ResourceManager.Resources.Bicep/CHANGELOG.md	Add initial changelog entry for 1.0.0-beta.1
sdk/resources/Azure.ResourceManager.Resources.Bicep/Azure.ResourceManager.Resources.Bicep.sln	Add solution file including Bicep projects

Comments suppressed due to low confidence (2)

sdk/resources/Azure.ResourceManager.Resources.Bicep/README.md:3

Replace the placeholder on line 3 with a concise description of the Bicep management service to give users immediate context on what this library provides.

**[Describe the service briefly first.]**

sdk/resources/Azure.ResourceManager.Resources.Bicep/api/Azure.ResourceManager.Resources.Bicep.netstandard2.0.cs:11

Add unit or recorded tests that invoke BicepDecompileOperationGroup and its async overload to verify request construction and response handling for the new Bicep decompile operation.

        public static Azure.Response<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationSuccessResponse> BicepDecompileOperationGroup(this Azure.ResourceManager.Resources.SubscriptionResource subscriptionResource, Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationContent content, System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }

sdk/resources/Azure.ResourceManager.Resources.Bicep/assets.json

github-actions · 2025-06-20T16:20:50Z

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Azure.ResourceManager.Resources.Bicep

ArcturusZhang · 2025-06-27T07:53:10Z

...es/Azure.ResourceManager.Resources.Bicep/api/Azure.ResourceManager.Resources.Bicep.net8.0.cs

+    }
+    public static partial class ResourcesBicepExtensions
+    {
+        public static Azure.Response<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationSuccessResponse> BicepDecompileOperationGroup(this Azure.ResourceManager.Resources.SubscriptionResource subscriptionResource, Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationContent content, System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) { throw null; }


I think it would be more reasonable if we rename this to DecompileBicep.

ArcturusZhang · 2025-06-27T07:53:42Z

...es/Azure.ResourceManager.Resources.Bicep/api/Azure.ResourceManager.Resources.Bicep.net8.0.cs

+        public static Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationSuccessResponse DecompileOperationSuccessResponse(System.Collections.Generic.IEnumerable<Azure.ResourceManager.Resources.Bicep.Models.FileDefinition> files = null, string entryPoint = null) { throw null; }
+        public static Azure.ResourceManager.Resources.Bicep.Models.FileDefinition FileDefinition(string path = null, string contents = null) { throw null; }
+    }
+    public partial class DecompileOperationContent : System.ClientModel.Primitives.IJsonModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationContent>, System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationContent>


I think it would be more reasonable if we rename this to DecompileBicepContent.

ArcturusZhang · 2025-06-27T07:54:43Z

...es/Azure.ResourceManager.Resources.Bicep/api/Azure.ResourceManager.Resources.Bicep.net8.0.cs

+        string System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationContent>.GetFormatFromOptions(System.ClientModel.Primitives.ModelReaderWriterOptions options) { throw null; }
+        System.BinaryData System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationContent>.Write(System.ClientModel.Primitives.ModelReaderWriterOptions options) { throw null; }
+    }
+    public partial class DecompileOperationSuccessResponse : System.ClientModel.Primitives.IJsonModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationSuccessResponse>, System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationSuccessResponse>


Usually types in .net SDK should not end with Response. Could we rename this to DecompileBicepSuccessResult?

ArcturusZhang · 2025-06-27T07:54:58Z

...es/Azure.ResourceManager.Resources.Bicep/api/Azure.ResourceManager.Resources.Bicep.net8.0.cs

+        string System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationSuccessResponse>.GetFormatFromOptions(System.ClientModel.Primitives.ModelReaderWriterOptions options) { throw null; }
+        System.BinaryData System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.Resources.Bicep.Models.DecompileOperationSuccessResponse>.Write(System.ClientModel.Primitives.ModelReaderWriterOptions options) { throw null; }
+    }
+    public partial class FileDefinition : System.ClientModel.Primitives.IJsonModel<Azure.ResourceManager.Resources.Bicep.Models.FileDefinition>, System.ClientModel.Primitives.IPersistableModel<Azure.ResourceManager.Resources.Bicep.Models.FileDefinition>


Shall we rename this to BicepFileDefinition?

ArcturusZhang

Hi I left some renaming comments.
To resolve them, please open a PR in azure-rest-api-specs, and add some @@clientName statements to the client.tsp in your typespec.
We will review and merge that client renaming PR in the spec repo first.
In the meantime, could you update this PR by running the following command to apply those renamings to this PR so that we could see the change?

dotnet build /t:GenerateCode /p:LocalSpecRepo="The full directory path to your main.tsp"

Please note the path above should be the path of the directory of main.tsp, not the full filepath of main.tsp

anthony-c-martin · 2025-06-27T10:38:15Z

Hi I left some renaming comments. To resolve them, please open a PR in azure-rest-api-specs, and add some @@clientName statements to the client.tsp in your typespec. We will review and merge that client renaming PR in the spec repo first. In the meantime, could you update this PR by running the following command to apply those renamings to this PR so that we could see the change?
dotnet build /t:GenerateCode /p:LocalSpecRepo="The full directory path to your main.tsp"
Please note the path above should be the path of the directory of main.tsp, not the full filepath of main.tsp

Thanks! One thing I realized after seeing your comment is that we've actually already released this library Azure.ResourceManager.Resources - see #44181. There were some slightly different rename suggestions on that PR - see:

azure-sdk-for-net/sdk/resources/Azure.ResourceManager.Resources/src/autorest.md

Lines 29 to 30 in 684b650

    
           DecompileOperationSuccessResponse: DecompileOperationSuccessResult 
        
           FileDefinition: DecompiledFileDefinition

This is being split out from the Azure.ResourceManager.Resources package into a separate package as part of Azure/azure-sdk#8768 (see Azure/azure-rest-api-specs#34190).

Should I go with the exact rename suggestions you've made here, or should I copy the previous rename suggestions?

ArcturusZhang · 2025-07-01T03:29:08Z

@anthony-c-martin Ah I did not know that this was splitted from Resources.
If so, we really need to compare between those that have existed in Azure.ResourceManager.Resources and ensure they have the same name.
But before that, I think I should confirm this with Arthur since this is the first time that we do this splitting.

anthony-c-martin · 2025-07-02T22:24:22Z

In the meantime, I've submitted Azure/azure-rest-api-specs#35643 to align with original naming

Copilot AI review requested due to automatic review settings June 20, 2025 16:07

anthony-c-martin requested review from a team and jsquire as code owners June 20, 2025 16:07

github-actions bot added the ARM label Jun 20, 2025

Copilot AI reviewed Jun 20, 2025

View reviewed changes

sdk/resources/Azure.ResourceManager.Resources.Bicep/assets.json Outdated Show resolved Hide resolved

anthony-c-martin force-pushed the ant/bicep branch 2 times, most recently from e6fa0c8 to 1c71864 Compare June 26, 2025 09:38

Initial release for Azure.ResourceManager.Resources.Bicep

e765bf7

anthony-c-martin force-pushed the ant/bicep branch from 1c71864 to e765bf7 Compare June 26, 2025 10:06

Fix CI

1dc26ae

ArcturusZhang reviewed Jun 27, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Initial release for Azure.ResourceManager.Resources.Bicep #50751

Initial release for Azure.ResourceManager.Resources.Bicep #50751

anthony-c-martin commented Jun 20, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

github-actions bot commented Jun 20, 2025 •

edited

Loading

Uh oh!

ArcturusZhang Jun 27, 2025

Uh oh!

ArcturusZhang Jun 27, 2025

Uh oh!

ArcturusZhang Jun 27, 2025

Uh oh!

ArcturusZhang Jun 27, 2025

Uh oh!

ArcturusZhang left a comment

Uh oh!

anthony-c-martin commented Jun 27, 2025

Uh oh!

ArcturusZhang commented Jul 1, 2025 •

edited

Loading

Uh oh!

anthony-c-martin commented Jul 2, 2025

Uh oh!

Uh oh!

Initial release for Azure.ResourceManager.Resources.Bicep #50751

Are you sure you want to change the base?

Initial release for Azure.ResourceManager.Resources.Bicep #50751

Conversation

anthony-c-martin commented Jun 20, 2025

Contributing to the Azure SDK

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

github-actions bot commented Jun 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

API Change Check

Uh oh!

ArcturusZhang Jun 27, 2025

Choose a reason for hiding this comment

Uh oh!

ArcturusZhang Jun 27, 2025

Choose a reason for hiding this comment

Uh oh!

ArcturusZhang Jun 27, 2025

Choose a reason for hiding this comment

Uh oh!

ArcturusZhang Jun 27, 2025

Choose a reason for hiding this comment

Uh oh!

ArcturusZhang left a comment

Choose a reason for hiding this comment

Uh oh!

anthony-c-martin commented Jun 27, 2025

Uh oh!

ArcturusZhang commented Jul 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

anthony-c-martin commented Jul 2, 2025

Uh oh!

Uh oh!

github-actions bot commented Jun 20, 2025 •

edited

Loading

ArcturusZhang commented Jul 1, 2025 •

edited

Loading