Add support for creating and updating portable manifests

src/WingetCreateCLI/Commands/NewCommand.cs

@@ -296,13 +296,17 @@ private static void PromptInstallerProperties<T>(T manifest, PropertyInfo proper
 
                 bool prompted = false;
 
-                // If we know the installertype is EXE, prompt the user for installer switches (silent and silentwithprogress)
+                // If the installerType is EXE, prompt the user for whether the package is a portable
                 if (installer.InstallerType == InstallerType.Exe)
                 {
                     Console.WriteLine();
-                    Logger.DebugLocalized(nameof(Resources.AdditionalMetadataNeeded_Message), installer.InstallerUrl);
-                    prompted = true;
-                    PromptInstallerSwitchesForExe(manifest);
+                    if (!PromptForPortableExe(installer))
+                    {
+                        // If we know the installertype is EXE, prompt the user for installer switches (silent and silentwithprogress)
+                        Logger.DebugLocalized(nameof(Resources.AdditionalMetadataNeeded_Message), installer.InstallerUrl);
+                        prompted = true;
+                        PromptInstallerSwitchesForExe(installer);
+                    }
                 }
 
                 foreach (var requiredProperty in requiredInstallerProperties)
@@ -326,7 +330,7 @@ private static void PromptInstallerProperties<T>(T manifest, PropertyInfo proper
             }
         }
 
-        private static void PromptInstallerSwitchesForExe<T>(T manifest)
+        private static void PromptInstallerSwitchesForExe<T>(T manifestInstaller)
         {
             InstallerSwitches installerSwitches = new InstallerSwitches();
 
@@ -335,7 +339,35 @@ private static void PromptInstallerSwitchesForExe<T>(T manifest)
 
             if (!string.IsNullOrEmpty(installerSwitches.Silent) || !string.IsNullOrEmpty(installerSwitches.SilentWithProgress))
             {
-                manifest.GetType().GetProperty(nameof(InstallerSwitches)).SetValue(manifest, installerSwitches);
+                manifestInstaller.GetType().GetProperty(nameof(InstallerSwitches)).SetValue(manifestInstaller, installerSwitches);
+            }
+        }
+
+        /// <summary>
+        /// Prompts the user to confirm whether the package is a portable.
+        /// If true, then prompts for related portable metadata.
+        /// </summary>
+        /// <typeparam name="T">Manifest installer.</typeparam>
+        /// <param name="manifestInstaller">Manifest installer object model.</param>
+        /// <returns>Boolean value indicating whether the package is a portable.</returns>
+        private static bool PromptForPortableExe<T>(T manifestInstaller)
+        {
+            if (Prompt.Confirm(Resources.ConfirmPortablePackage_Message))
+            {
+                manifestInstaller.GetType().GetProperty(nameof(Installer.InstallerType)).SetValue(manifestInstaller, InstallerType.Portable);
+                string portableCommandAlias = Prompt.Input<string>(Resources.PortableCommandAlias_Message);
+
+                if (!string.IsNullOrEmpty(portableCommandAlias))
+                {
+                    List<string> portableCommands = new List<string> { portableCommandAlias };
+                    manifestInstaller.GetType().GetProperty(nameof(Installer.Commands)).SetValue(manifestInstaller, portableCommands);
+                }
+
+                return true;
+            }
+            else
+            {
+                return false;
             }
         }
 

src/WingetCreateCLI/Properties/Resources.resx

@@ -919,4 +919,11 @@
     <value>List of winget arguments the installer does not support</value>
     <comment>"winget" is the proper name of the tool</comment>
   </data>
+  <data name="ConfirmPortablePackage_Message" xml:space="preserve">
+    <value>Is this a portable package?</value>
+  </data>
+  <data name="PortableCommandAlias_Message" xml:space="preserve">
+    <value>What is the command alias of the portable package |e.g. nuget|</value>
+    <comment>`command alias` refers to the name of the exe file that will be used when running the application from the command line.</comment>
+  </data>
 </root>

src/WingetCreateCore/Common/PackageParser.cs

@@ -10,8 +10,8 @@ namespace Microsoft.WingetCreateCore
     using System.Globalization;
     using System.IO;
     using System.Linq;
-    using System.Net.Http;
-    using System.Security.Cryptography;
+    using System.Net.Http;
+    using System.Security.Cryptography;
     using System.Text.RegularExpressions;
     using System.Threading.Tasks;
     using System.Xml;
@@ -615,14 +615,15 @@ private static CompatibilitySet GetCompatibilitySet(InstallerType type)
                 case InstallerType.Inno:
                 case InstallerType.Nullsoft:
                 case InstallerType.Exe:
-                case InstallerType.Burn:
+                case InstallerType.Burn:
+                case InstallerType.Portable:
                     return CompatibilitySet.Exe;
                 case InstallerType.Wix:
                 case InstallerType.Msi:
                     return CompatibilitySet.Msi;
                 case InstallerType.Msix:
                 case InstallerType.Appx:
-                    return CompatibilitySet.Msix;
+                    return CompatibilitySet.Msix;
                 default:
                     return CompatibilitySet.None;
             }

src/WingetCreateTests/WingetCreateTests/E2ETests/E2ETests.cs

@@ -42,7 +42,8 @@ public void Setup()
         /// <returns>A <see cref="Task"/> representing the asynchronous test.</returns>
         [TestCase(TestConstants.TestExePackageIdentifier, TestConstants.TestExeManifest, TestConstants.TestExeInstaller)]
         [TestCase(TestConstants.TestMsiPackageIdentifier, TestConstants.TestMsiManifest, TestConstants.TestMsiInstaller)]
-        [TestCase(TestConstants.TestMultifileMsixPackageIdentifier, TestConstants.TestMultifileMsixManifestDir, TestConstants.TestMsixInstaller)]
+        [TestCase(TestConstants.TestMultifileMsixPackageIdentifier, TestConstants.TestMultifileMsixManifestDir, TestConstants.TestMsixInstaller)]
+        [TestCase(TestConstants.TestPortablePackageIdentifier, TestConstants.TestPortableManifest, TestConstants.TestExeInstaller)]
         public async Task SubmitAndUpdateInstaller(string packageId, string manifestName, string installerName)
         {
             await this.RunSubmitAndUpdateFlow(packageId, TestUtils.GetTestFile(manifestName), installerName);

src/WingetCreateTests/WingetCreateTests/Resources/TestPublisher.Portable.yaml

@@ -0,0 +1,16 @@
+PackageIdentifier: TestPublisher.Portable
+PackageVersion: 0.1.2
+PackageName: Test portable app
+Publisher: Test publisher
+License: MIT
+ShortDescription: A manifest used for testing a portable package.
+Installers:
+- Architecture: x64
+  InstallerType: portable
+  InstallerUrl: https://fakedomain.com/WingetCreateTestExeInstaller.exe
+  InstallerSha256: 8A052767127A6E2058BAAE03B551A807777BB1B726650E2C7E92C3E92C8DF80D
+  Commands:
+    - portableCommand
+PackageLocale: en-US
+ManifestType: singleton
+ManifestVersion: 1.2.0

src/WingetCreateTests/WingetCreateTests/Resources/WingetCreateE2E.PortableTest.yaml

@@ -0,0 +1,26 @@
+PackageIdentifier: WingetCreateE2E.PortableTest
+PackageName: PortableTest
+PackageVersion: 1.2.3.4
+Publisher: WingetCreateE2E
+Author: Microsoft
+License: MIT
+LicenseUrl: https://github.com/microsoft/winget-create
+MinimumOSVersion: 10.0.0.0
+ShortDescription: Testing WingetCreate with a portable exe
+PackageLocale: en-US
+Tags: 
+  - Windows Package Manager
+  - winget create
+  - package manager
+  - utility
+  - tool
+  - test
+InstallerType: portable
+Installers:
+  - Architecture: x64
+    InstallerUrl: https://fakedomain.com/WingetCreateTestExeInstaller.exe
+    InstallerSha256: A7803233EEDB6A4B59B3024CCF9292A6FFFB94507DC998AA67C5B745D197A5DC
+    Commands:
+      - portableCommand
+ManifestType: singleton
+ManifestVersion: 1.2.0

src/WingetCreateTests/WingetCreateTests/TestConstants.cs

@@ -53,6 +53,11 @@ public static class TestConstants
         /// </summary>
         public const string TestMultifileMsixPackageIdentifier = "Multifile.MsixTest";
 
+        /// <summary>
+        /// PackageIdentifier for test portable.
+        /// </summary>
+        public const string TestPortablePackageIdentifier = "WingetCreateE2E.PortableTest";
+
         /// <summary>
         /// File name of the test exe manifest.
         /// </summary>
@@ -63,6 +68,11 @@ public static class TestConstants
         /// </summary>
         public const string TestMsiManifest = "WingetCreateE2E.MsiTest.yaml";
 
+        /// <summary>
+        /// File name of the test portable manifest.
+        /// </summary>
+        public const string TestPortableManifest = "WingetCreateE2E.PortableTest.yaml";
+
         /// <summary>
         /// Path of the directory with the multifile msix test manifests.
         /// </summary>

src/WingetCreateTests/WingetCreateTests/UnitTests/UpdateCommandTests.cs

@@ -539,6 +539,25 @@ public async Task UpdateResetsVersionSpecificFields()
             Assert.IsNull(updatedDefaultLocaleManifest.ReleaseNotesUrl, "ReleaseNotesUrl should be null.");
         }
 
+        /// <summary>
+        /// Ensures that updating a portable package preserves the portable installerType.
+        /// </summary>
+        /// <returns>A <see cref="Task"/> representing the result of the asynchronous operation.</returns>
+        [Test]
+        public async Task UpdatePortable()
+        {
+            TestUtils.InitializeMockDownloads(TestConstants.TestExeInstaller);
+            (UpdateCommand command, var initialManifestContent) = GetUpdateCommandAndManifestData("TestPublisher.Portable", null, this.tempPath, null);
+            var updatedManifests = await RunUpdateCommand(command, initialManifestContent);
+            Assert.IsNotNull(updatedManifests, "Command should have succeeded");
+
+            InstallerManifest updatedInstallerManifest = updatedManifests.InstallerManifest;
+            var updatedInstaller = updatedInstallerManifest.Installers.First();
+
+            Assert.IsTrue(updatedInstaller.InstallerType == InstallerType.Portable, "InstallerType should be portable");
+            Assert.IsTrue(updatedInstaller.Commands[0] == "portableCommand", "Command value should be preserved.");
+        }
+
         private static (UpdateCommand UpdateCommand, List<string> InitialManifestContent) GetUpdateCommandAndManifestData(string id, string version, string outputDir, IEnumerable<string> installerUrls)
         {
             var updateCommand = new UpdateCommand

src/WingetCreateTests/WingetCreateTests/WingetCreateTests.csproj

@@ -33,6 +33,9 @@
     <None Update="Resources\Multifile.MsixTest\Multifile.MsixTest.locale.en-US.yaml">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
+    <None Update="Resources\TestPublisher.Portable.yaml">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
     <None Update="Resources\TestPublisher.FullSingleton1_2.yaml">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
@@ -78,6 +81,9 @@
     <None Update="Resources\Multifile.MsixTest\Multifile.MsixTest.yaml">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
+    <None Update="Resources\WingetCreateE2E.PortableTest.yaml">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
     <None Update="Resources\WingetCreateTestExeInstaller.exe">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>