[PS] Migrate SecurityInsights module to autorest v4 by JoyerJin · Pull Request #28447 · Azure/azure-powershell

JoyerJin · 2025-08-25T02:24:04Z

Description

Preannouncement PR:

Mandatory Checklist

Please choose the target release of Azure PowerShell. (⚠️Target release is a different concept from API readiness. Please click below links for details.)
- General release
- Public preview
- Private preview
- Engineering build
- No need for a release
Check this box to confirm: I have read the Submitting Changes section of CONTRIBUTING.md and reviewed the following information:

SHOULD update ChangeLog.md file(s) appropriately
- Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
  - A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
- Should not change ChangeLog.md if no new release is required, such as fixing test case only.
SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
SHOULD have proper test coverage for changes in pull request.
SHOULD NOT adjust version of module manually in pull request

azure-client-tools-bot-prd · 2025-08-25T02:24:07Z

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copilot

Pull Request Overview

This PR migrates the SecurityInsights module from AutoRest v3 to v4, updating API models and cmdlet structure. The migration involves significant changes to parameter types, output types, and method signatures to align with the newer AutoRest framework.

Key Changes

Type simplification from strongly-typed enums to string parameters for improved flexibility
Output type namespace updates from versioned API models to unversioned ones
Addition of new parameter sets and JSON-based input methods for cmdlets

Reviewed Changes

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

File	Description
`tools/StaticAnalysis/Exceptions/Az.SecurityInsights/BreakingChangeIssues.csv`	Documents expected breaking changes from the AutoRest v4 migration
Multiple help markdown files	Updated parameter types, output types, and documentation to reflect API changes

github-actions · 2025-09-02T03:50:36Z

To the author of the pull request,
This PR was labeled "Breaking Change Release" because it contains breaking changes.

According to our policy, breaking changes can only take place during major release and they must be preannounced.
Please follow our guide on the detailed steps.
Required: Please fill in the task below to facilitate our contact,you will receive notifications related to breaking changes.

Copilot

Pull request overview

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

isra-fel · 2026-01-29T02:09:36Z

/azp run

azure-pipelines · 2026-01-29T02:09:49Z

Azure Pipelines successfully started running 3 pipeline(s).

NoriZC · 2026-03-16T05:39:28Z

/azp run

azure-pipelines · 2026-03-16T05:39:43Z

Azure Pipelines successfully started running 3 pipeline(s).

Copilot

Pull request overview

Copilot reviewed 152 out of 153 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/SecurityInsights/SecurityInsights.Autorest/test/Update-AzSentinelOnboardingState.Tests.ps1:24

Both test cases are marked -skip and contain only a NotImplementedException placeholder, so there is currently no automated coverage for Update-AzSentinelOnboardingState despite the cmdlet being introduced/updated in this PR.

Suggested fix: replace these placeholders with real tests (record/playback or mocking) that execute the cmdlet for the supported parameter sets, and remove -skip once they pass reliably.

+        [Microsoft.Azure.PowerShell.Cmdlets.SecurityInsights.PSArgumentCompleterAttribute("Activity","Expansion")]
        [Microsoft.Azure.PowerShell.Cmdlets.SecurityInsights.Category('Body')]


azure-pipelines · 2026-03-31T02:42:54Z

Azure Pipelines successfully started running 3 pipeline(s).

…m/Azure/azure-powershell into jiaseng/main/SecurityInsight-v4

Copilot

Pull request overview

Copilot reviewed 120 out of 232 changed files in this pull request and generated 1 comment.

Copilot · 2026-04-08T02:56:45Z

+    # The incidents/relations endpoint returns 409 when updating an existing relation's target.
+
    It 'UpdateExpanded' {
        $bookmark = New-AzSentinelBookmark -ResourceGroupName $env.resourceGroupName `
            -Id $env.UpdateincidentRelationBookmarkId2 -WorkspaceName $env.workspaceName -DisplayName $env.UpdateincidentRelationBookmarkName2 -Query "SecurityEvent\n| take 1" `
            -QueryStartTime (get-date).ToUniversalTime() -QueryEndTime (get-date).AddDays(-1).ToUniversalTime() -EventTime (get-date).ToUniversalTime()
-        $incidentRelation = Update-AzSentinelIncidentRelation -ResourceGroupName $env.resourceGroupName -WorkspaceName $env.workspaceName `
-            -IncidentId $env.UpdateincidentRelationIncidentId -RelationName $env.UpdateincidentRelationId -RelatedResourceId $bookmark.Id
-        $incidentRelation.RelatedResourceId | should -be $bookmark.id
+        { Update-AzSentinelIncidentRelation -ResourceGroupName $env.resourceGroupName -WorkspaceName $env.workspaceName `
+            -IncidentId $env.UpdateincidentRelationIncidentId -RelationName $env.UpdateincidentRelationId -RelatedResourceId $bookmark.Id } | Should -Throw "already exists on incident"
    }


The test now asserts Should -Throw "already exists on incident", which couples the test to an exact/partial error message that can change with service wording, localization, or SDK error formatting. Prefer asserting on a stable signal (e.g., HTTP status code 409, an error code, or exception type) while still documenting the expected conflict behavior in the comment.

VeryEarly · 2026-04-08T03:08:58Z

/azp run

azure-pipelines · 2026-04-08T03:09:05Z

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

…3->v4 (#28572) Co-authored-by: JunKai-v <v-juphang@microsoft.com>

…shell v3->v4 (#28376) Co-authored-by: JunKai-v <v-juphang@microsoft.com>

Co-authored-by: Azure PowerShell <65331932+azure-powershell-bot@users.noreply.github.com> Co-authored-by: Thomas Temby <thomastemby@microsoft.com> Co-authored-by: Audrey Toney <audreytoney@microsoft.com> Co-authored-by: Meghana Gupta <93553451+meghanagupta95@users.noreply.github.com> Co-authored-by: Yeming Liu <11371776+isra-fel@users.noreply.github.com> Co-authored-by: Yeming Liu <yeliu@microsoft.com> Co-authored-by: JunKai-v <v-juphang@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Yabo Hu <yabhu@microsoft.com> Co-authored-by: Sam Lee <32776898+minhsuanlee@users.noreply.github.com> Co-authored-by: Sam Lee <samlee3@microsoft.com> Co-authored-by: Ptnan7 <148527582+Ptnan7@users.noreply.github.com> Co-authored-by: JingnanXu <jingnanxu@microsoft.com> Co-authored-by: Matthew Metcalf <mrm9084@gmail.com> Co-authored-by: DevanshG1 <108400981+DevanshG1@users.noreply.github.com> Co-authored-by: Devansh Goswami <degoswami@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Jeffrey Bulanadi <41933086+jeffreybulanadi@users.noreply.github.com> Co-authored-by: Bhaskar Brahma <bhbrahma@microsoft.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: audreyttt <225061541+audreyttt@users.noreply.github.com> Co-authored-by: Auðunn <audunnb@gmail.com> Co-authored-by: Komla-Ansah <selasiekmortey@gmail.com> Co-authored-by: Selasie Mortey <smortey@microsoft.com> Co-authored-by: MaddyMicrosoft <maddisondas@microsoft.com> Co-authored-by: jruttle <jruttle+github@microsoft.com> Co-authored-by: Jonathan Ruttle <jruttle@microsoft.com> Co-authored-by: Balashivaram Ganesan <71939272+Balashivaram@users.noreply.github.com> Co-authored-by: Balashivaram Ganesan <bganesan@microsoft.com> Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com> Co-authored-by: JaysonTaiMicrosoft <jaysontai@microsoft.com>

Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

Co-authored-by: JunKai-v <v-juphang@microsoft.com>

Co-authored-by: JunKai-v <v-juphang@microsoft.com> Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

…29510)

Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

JoyerJin added 2 commits August 21, 2025 10:33

migrate SecurityInsights module to autorest v4

f6c8ee6

update example

4e9517f

JoyerJin added 2 commits August 25, 2025 17:51

breaking change issues exceptions

83763b6

hide Update variant to customize

e98c4ee

JoyerJin marked this pull request as ready for review August 26, 2025 07:24

Copilot AI review requested due to automatic review settings August 26, 2025 07:24

Copilot AI reviewed Aug 26, 2025

View reviewed changes

Comment thread tools/StaticAnalysis/Exceptions/Az.SecurityInsights/BreakingChangeIssues.csv

Comment thread src/SecurityInsights/SecurityInsights/help/New-AzSentinelAlertRule.md Outdated

Comment thread src/SecurityInsights/SecurityInsights/help/New-AzSentinelIncidentTeam.md Outdated

update examples

d3f8439

VeryEarly added the Contains Breaking Change This PR contains breaking change label Sep 2, 2025

Merge branch 'main' into joyer/SecurityInsights-migrate-v4

7ccdeb2

isra-fel added the autorest v4 migration pr migrating module from generated by autorest.powershell v3 to v4 label Oct 2, 2025

Pan-Qi marked this pull request as draft January 8, 2026 07:19

Pan-Qi marked this pull request as ready for review January 9, 2026 05:13

Copilot AI review requested due to automatic review settings January 9, 2026 05:13

Copilot started reviewing on behalf of Pan-Qi January 9, 2026 05:16 View session

Copilot AI reviewed Jan 9, 2026

View reviewed changes

isra-fel assigned Pan-Qi Jan 28, 2026

isra-fel added this to the Az 16.0.0 (TBD) milestone Jan 28, 2026

JiaSeng-v added 2 commits January 29, 2026 09:39

rebuild with Autorest 4.0.752

08737e6

solve conflict

51a797c

fix merge conflict

bc4ddda

Copilot AI review requested due to automatic review settings March 16, 2026 05:35

Copilot started reviewing on behalf of JiaSeng-v March 16, 2026 05:37 View session

Copilot AI reviewed Mar 16, 2026

View reviewed changes

JiaSeng-v added 2 commits April 8, 2026 10:47

fix conflict

6ebb26a

Merge branch 'joyer/SecurityInsights-migrate-v4' of https://github.co…

12be642

…m/Azure/azure-powershell into jiaseng/main/SecurityInsight-v4

Copilot AI review requested due to automatic review settings April 8, 2026 02:49

Copilot started reviewing on behalf of JiaSeng-v April 8, 2026 02:51 View session

Copilot AI reviewed Apr 8, 2026

View reviewed changes

lijinpei2008 and others added 4 commits April 28, 2026 16:53

Az.CloudService update generation tool version: autorest.powershell v…

f03e00d

…3->v4 (#28572) Co-authored-by: JunKai-v <v-juphang@microsoft.com>

[Az.ContainerInstance] Update generation tool version: autorest.power…

a90681b

…shell v3->v4 (#28376) Co-authored-by: JunKai-v <v-juphang@microsoft.com>

fix merge conflict

f756213

khang-11 changed the base branch from main to autorest-v4-breakingchange May 6, 2026 00:51

khang-11 requested review from NoriZC, audreyttt, haagha, ookoka, teresaritorto and yanzhudd as code owners May 6, 2026 00:51

Francisco-Gamino and others added 10 commits May 6, 2026 10:53

Migrate Az.Functions from AutoRest PowerShell v3 to v4 (#29321)

a621d71

[PS] migrate MachineLearningServices module to autorest v4 (#28818)

d390d7d

Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

[PS] Migrate ManagedServices module to autorest v4 (#27929)

5f75abd

Co-authored-by: JunKai-v <v-juphang@microsoft.com>

[PS] Migrate MySQL module to autorest v4 (#28784)

2f29e98

Co-authored-by: JunKai-v <v-juphang@microsoft.com> Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

[PS] Migrate RedisEnterpriseCache module to autorest v4 (#28618)

5e29af3

Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

[Az.SignalR] Migrate Autorest Generation v3 to v4 (#29050)

308addc

[Az.SqlVirtualMachine] Update Autorest v3 to v4 (#29156)

05e8c8c

Revert: Migrate RedisEnterpriseCache module to autorest v4 (#28618) (#…

e2f5484

…29510)

[PS] Migrate StackHCI module to autorest v4 (#28363)

5913fad

Co-authored-by: JiaSeng-v <v-jiasenglaw@microsoft.com>

fix merge conflict

32a5749

khang-11 changed the base branch from autorest-v4-breakingchange to main May 12, 2026 03:47

		[Microsoft.Azure.PowerShell.Cmdlets.SecurityInsights.PSArgumentCompleterAttribute("Activity","Expansion")]
		[Microsoft.Azure.PowerShell.Cmdlets.SecurityInsights.Category('Body')]

Conversation

JoyerJin commented Aug 25, 2025 • edited by JiaSeng-v Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Mandatory Checklist

Uh oh!

azure-client-tools-bot-prd Bot commented Aug 25, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Key Changes

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Sep 2, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

isra-fel commented Jan 29, 2026

Uh oh!

azure-pipelines Bot commented Jan 29, 2026

Uh oh!

NoriZC commented Mar 16, 2026

Uh oh!

azure-pipelines Bot commented Mar 16, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

azure-pipelines Bot commented Mar 31, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

VeryEarly commented Apr 8, 2026

Uh oh!

azure-pipelines Bot commented Apr 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

JoyerJin commented Aug 25, 2025 •

edited by JiaSeng-v

Loading