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 change.
• Recommendation: No change needed.

---

✅ Commit Type

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

---

✅ Risk Level

• The submitted risk level (Low) matches the scope of the diff reasonably well.
• The PR is a targeted hotfix revert with tests and a CI-only coverage adjustment, so low is acceptable.

---

✅ What & Why

• Current: Well filled out and clearly explains the revert, motivation, affected areas, and CI coverage fix.
• Issue: None.
• Recommendation: No change needed.

---

✅ Impact of Change

• The section is sufficiently detailed for users, developers, and system impact.
• Recommendation:
  • Users: Good as written.
  • Developers: Good as written.
  • System: Good as written.

---

✅ Test Plan

• The diff contains the claimed unit test updates/restorations and additional coverage-related changes.
• There is also a reasonable explanation for the CI heap adjustment and the PR-coverage rerun note.
• No E2E tests were added, but unit test coverage is present and the explanation is adequate for this PR type.

---

✅ Contributors

• N/A is acceptable here.
• No action required.

---

✅ Screenshots/Videos

• Not applicable for this revert/CI/test-only change.
• No action required.

---

Summary Table

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

---

PR passes. Advised risk is not higher than the submitter’s estimate.

---

Last updated: Wed, 20 May 2026 21:43:08 GMT