Skip to content

Support zip installers natively without calling winget on target machines#259

Merged
svrooij merged 3 commits into
mainfrom
copilot/handle-nestedinstallertype-in-winget-manifest
Jul 3, 2026
Merged

Support zip installers natively without calling winget on target machines#259
svrooij merged 3 commits into
mainfrom
copilot/handle-nestedinstallertype-in-winget-manifest

Conversation

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Packages with InstallerType.Zip (e.g. Adobe Acrobat Pro) previously fell back to a winget-based install script, requiring winget on the target machine. This adds zip as a first-class installer type by downloading, extracting, and packaging the nested installer directly.

Changes

ComputeBestInstallerForPackageCommand.cs

  • Added case InstallerType.Zip in ComputeInstallerCommands() to derive InstallCommandLine/UninstallCommandLine from NestedInstallerFiles and InstallerSwitches
    • Nested MSI/WIX → msiexec /i "nestedPath" <switches>
    • Nested EXE → "nestedPath" <switches>, uninstall via MsiProductCode if present

IntuneManager.cs

  • Added else if branch in GenerateNoneMsiInstaller for zip packages with nested files: downloads the zip, extracts it to the output folder, sets InstallerFilename to the nested relative path

Tests

  • Added adobeAcrobatZipPackageInfo constant (zip with nested exe at Adobe Acrobat\setup.exe, EULA switches, product code)
  • Added GenerateInstallerPackage_ZipPackage_ExtractsAndPackagesNestedInstaller test covering the full flow

Copilot AI linked an issue Jul 3, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Implement NestedInstallerType handling in WinGet manifest Handle NestedInstallerType in WinGet manifest Jul 3, 2026
Copilot AI requested a review from svrooij July 3, 2026 18:48
Copilot AI changed the title Handle NestedInstallerType in WinGet manifest Support zip installers natively without calling winget on target machines Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Test Results

14 tests  ±0   13 ✅ ±0   1s ⏱️ ±0s
10 suites ±0    1 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 63fe424. ± Comparison against base commit 13d47f6.

@svrooij
svrooij marked this pull request as ready for review July 3, 2026 19:25
@svrooij
svrooij merged commit 0b7a139 into main Jul 3, 2026
7 checks passed
@svrooij
svrooij deleted the copilot/handle-nestedinstallertype-in-winget-manifest branch July 3, 2026 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle NestedInstallerType in WinGet manifest

2 participants