Revert .NET 10 custom code support for v5.959 hotfix

[lambrianmsft]

What & Why

Reverts .NET 10 custom code project support from the v5.959 hotfix line and keeps the restored unit coverage in the same PR so this can merge directly into hotfix/v5.959.

Why:

• Issue VS Code: Deployed logic app fails to open in the portal with prolonged loading and no content displayed. #9144 reports Azure-deployed Logic Apps workflows not loading after SQL + NuGet deployment.
• VSIX 5.273.0 is reported as known-good, and the strongest regression boundary is the .NET 10 custom code support change.
• The hotfix should restore the previous .NET 8 codeless NuGet conversion defaults while preserving later hotfix-era conflict resolutions.
• Restored/adapted tests keep the coverage gate strict instead of weakening PR coverage policy.
• PR coverage CI hit Node heap OOM while running the full VS Code extension unit suite with Istanbul coverage, so the coverage workflow now gives that step a larger Node heap without changing thresholds or exclusions.

Affected areas:

• apps/vs-code-designer: custom code function/project steps, code project base, codeful workflow creation, workflow switchToDotnetProject, debug/custom-code/dotnet/codeless utility tests, and reverted .NET 8 defaults/templates.
• apps/vs-code-react: create workspace .NET framework step tests and reverted .NET 8 framework UI behavior.
• .github/workflows/pr-coverage.yml: CI-only Node heap increase for the coverage test step to avoid VS Code extension coverage OOM.

Supersedes #9169, which was closed so this PR can carry both the revert and coverage fix directly into hotfix/v5.959.

Fixes #9144

Type of Change

• fix

Impact of Change

Users

• Restores prior .NET 8-based custom-code/codeless NuGet conversion behavior for the v5.959 hotfix line.
• Intended to unblock SQL + NuGet deployed Logic Apps workflows that fail to load in Azure Portal.

Developers

• Restores and adapts unit tests removed by the revert so changed-file coverage remains enforced.
• Keeps the hotfix branch reviewable as one PR: production revert plus corresponding coverage restoration.
• Makes the PR coverage workflow more stable for the existing full VS Code extension unit coverage run.

System

• Reverts .NET 10 workflow/template/default behavior back to the previously supported .NET 8 hotfix behavior.
• No coverage-threshold changes, coverage exclusions, runtime dependencies, or product runtime changes from the CI heap adjustment.

Risk Level

• Low

Risk notes:

• Low risk because this is a targeted hotfix revert to previously shipped behavior plus tests.
• The product change is scoped to VS Code custom-code/codeless NuGet conversion and workspace framework selection paths.
• The CI change only increases Node heap for the PR coverage test step and does not weaken validation.

Test Plan

• Updated/restored unit tests for the affected VS Code designer and VS Code React flows.
• Ran pnpm install --frozen-lockfile during the hotfix setup.
• Ran pnpm exec biome check --write <changed test files>.
• Ran targeted apps/vs-code-designer Vitest coverage: 12 files, 192 tests passed.
• Ran targeted apps/vs-code-react Vitest coverage: 2 files, 25 tests passed.
• Verified final changed-file coverage table: all target files are >= 80%.
• Ran pnpm run build:extension during the revert validation.
• PR Coverage Check rerun after CI heap adjustment.

Screenshots / Videos

Not applicable - VS Code hotfix revert, unit coverage restoration, and CI coverage stability update; no new UI surface.

Contributors

N/A

[github-actions]

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

✅ PR Title

• Current: Revert .NET 10 custom code support for v5.959 hotfix
• Issue: None. The title is clear, specific, and accurately describes the revert-focused hotfix scope.
• Recommendation: No change needed.

---

✅ Commit Type

• Properly selected (fix).
• Only one commit type is selected, which is correct.

---

✅ Risk Level

• The selected risk label risk:low matches the PR body’s low-risk assessment and the revert is scoped to previously shipped behavior plus tests.

---

✅ What & Why

• Current: Well populated and concise.
• Issue: None.
• Recommendation: No change needed.

---

✅ Impact of Change

• The impact section is filled out and appropriately scoped to users, developers, and system behavior.
• Recommendation:
  • Users: Good as written.
  • Developers: Good as written.
  • System: Good as written.

---

✅ Test Plan

• The diff includes restored/updated unit tests, which satisfies the PR test-plan requirement.
• Manual testing remaining unchecked is acceptable because unit tests are present in the diff.

---

✅ Contributors

• N/A is acceptable here.

---

✅ Screenshots/Videos

• Not applicable for this non-visual hotfix/revert.

---

Summary Table

Section	Status	Recommendation
Title	✅	No change needed
Commit Type	✅	No change needed
Risk Level	✅	No change needed
What & Why	✅	No change needed
Impact of Change	✅	No change needed
Test Plan	✅	No change needed
Contributors	✅	No change needed
Screenshots/Videos	✅	No change needed

---

This PR passes review as submitted.

---

Last updated: Wed, 27 May 2026 01:17:13 GMT