Unblock C# data-plane upgrade PRs on management approval mismatches

[Copilot]

Dashboard regeneration validates management dependencies even during independent data-plane upgrades, aborting PR submission on unrelated approval mismatches.

• Approval isolation: Allow data-plane PR submission to continue after dashboard failures. Keep failures blocking management PR submission without advancing unapproved dependencies.
• Failure reporting: Report automated submission failures as warnings / SucceededWithIssues, preserving actionable error details and cleanup. Keep manual-run failures fatal.
• Error handling: Capture dashboard exceptions, nonterminating errors, and nonzero exit codes; restore the caller’s error preference.

• Fixes [Bug]: Management emitter approval validation blocks C# data-plane upgrade PRs #12192

[azure-pipelines]

Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-client-csharp@12193

commit: 46be8d3

[Copilot]

🟡 Changes recommended

Changed protected dependencies can bypass approval checks, and handled dashboard lookup failures can incorrectly block management submissions.

2 open findings

• Regeneration can commit an unapproved Azure generator version · New
• Failed commit lookup leaves nonzero exit status · New

What changed in this PR

Updates C# SDK upgrade automation to let data-plane PRs proceed despite unrelated dashboard approval failures.

Changes:

• Catches dashboard errors and restores the caller’s error preference.
• Keeps dashboard failures blocking management submissions.
• Reports automated submission failures as warnings while retaining fatal manual-run behavior.

File	Description
packages/​http-client-csharp/​eng/​scripts/​Submit-AzureSdkForNetPr.ps1	Adds dashboard failure isolation and warning-only automated failure reporting.

🧠 Review effort: Balanced

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟡 Changes recommended

Approval validation parses a local package reference as a semantic version, still blocking otherwise valid data-plane submissions.

1 open finding

• Preserve approved generator version before rewriting descriptor · New

2 resolved since last review

• Regeneration can commit an unapproved Azure generator version
• Failed commit lookup leaves nonzero exit status

🧠 Review effort: Balanced

[Jorge Rangel (jorgerangel-msft)]

Copilot why do we need this ? Shouldn't the try catch around the emitter version dashboard script allow us to avoid doing this at the end of the outer script ?

[Copilot]

The inner catch only handles dashboard-generation errors. The outer catch covers failures elsewhere in PR submission (for example, commit, push, or gh pr create) and preserves automated-run SucceededWithIssues versus manual-run failure behavior. Dashboard errors caught by the inner handler never reach the outer catch.

[Jorge Rangel (jorgerangel-msft)]

Copilot right, so why are we touching the out catch ? The outer catch works today and shouldn't be touched. The inner catch is the only thing we should be editing. Help me understand otherwise.

[Jorge Rangel (jorgerangel-msft)]

Copilot where are we on this ?