chore: revert "refactor: migrate all subprocess call sites to the shared subprocess module"

[mrgrain]

Reverts #1758

We decided this is too risky right now since we don't have windows integration tests as part of the release flow. We will prioritize the Windows integ tests first.

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[github-actions]

Dependency Review

The following issues were found:

• ✅ 0 vulnerable package(s)
• ✅ 0 package(s) with incompatible licenses
• ✅ 0 package(s) with invalid SPDX license definitions
• ⚠️ 1 package(s) with unknown licenses.

See the Details below.

License Issues

packages/@aws-cdk/toolkit-lib/package.json

Package	Version	License	Issue Type
split2	^4.2.0	Null	Unknown License

OpenSSF Scorecard

Package	Version	Score	Details
npm/@types/split2	^4.2.3	Unknown	Unknown
npm/split2	^4.2.0	Unknown	Unknown
npm/@types/split2	4.2.3	🟢 6.5	Details Check Score Reason Maintained 🟢 10 30 commit(s) and 4 issue activity found in the last 90 days -- score normalized to 10 Code-Review 🟢 8 Found 24/30 approved changesets -- score normalized to 8 Packaging ⚠️ -1 packaging workflow not detected CII-Best-Practices ⚠️ 0 no effort to earn an OpenSSF best practices badge detected Dangerous-Workflow 🟢 10 no dangerous workflow patterns detected Token-Permissions ⚠️ 0 detected GitHub workflow tokens with excessive permissions Security-Policy 🟢 10 security policy file detected License 🟢 9 license file detected Signed-Releases ⚠️ -1 no releases found Branch-Protection ⚠️ -1 internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md SAST ⚠️ 0 SAST tool is not run on all commits -- score normalized to 0 Pinned-Dependencies 🟢 8 dependency not pinned by hash detected -- score normalized to 8 Binary-Artifacts 🟢 10 no binaries found in the repo Fuzzing ⚠️ 0 project is not fuzzed
npm/split2	4.2.0	🟢 4.7	Details Check Score Reason Maintained ⚠️ 0 1 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 0 Code-Review 🟢 3 Found 8/22 approved changesets -- score normalized to 3 Binary-Artifacts 🟢 10 no binaries found in the repo Dangerous-Workflow 🟢 10 no dangerous workflow patterns detected Packaging ⚠️ -1 packaging workflow not detected Security-Policy 🟢 10 security policy file detected Token-Permissions 🟢 9 detected GitHub workflow tokens with excessive permissions Pinned-Dependencies ⚠️ 0 dependency not pinned by hash detected -- score normalized to 0 CII-Best-Practices ⚠️ 0 no effort to earn an OpenSSF best practices badge detected Fuzzing ⚠️ 0 project is not fuzzed License 🟢 10 license file detected Branch-Protection ⚠️ 0 branch protection not enabled on development/release branches Signed-Releases ⚠️ -1 no releases found SAST ⚠️ 0 SAST tool is not run on all commits -- score normalized to 0

Scanned Files

• packages/@aws-cdk/cdk-assets-lib/package.json
• packages/@aws-cdk/toolkit-lib/package.json
• yarn.lock

[iankhou]

The world wasn't ready for this change

[codecov-commenter]

Codecov Report

❌ Patch coverage is 89.56522% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.04%. Comparing base (e8aaa6b) to head (1809d3c).

Files with missing lines	Patch %	Lines
packages/aws-cdk/lib/commands/init/os.ts	85.88%	12 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1774   +/-   ##
=======================================
  Coverage   90.03%   90.04%           
=======================================
  Files          78       78           
  Lines       12035    12110   +75     
  Branches     1685     1698   +13     
=======================================
+ Hits        10836    10904   +68     
- Misses       1169     1177    +8     
+ Partials       30       29    -1     

Flag	Coverage Δ
suite.unit	90.04% <89.56%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.