Add default module root setting for configuration

doc/Settings.md

@@ -200,6 +200,21 @@ The `purgePortablePackage` behavior affects the default behavior for uninstallin
     },
 ```
 
+## Configure Behavior
+
+The `configureBehavior` settings affect the default behavior of applying a configuration.
+
+### Default Module Root
+The `defaultModuleRoot` behavior affects the default root directory where modules are installed to. Defaults to `%LOCALAPPDATA%/Microsoft/WinGet/Configuration/Modules` if value is not set or is invalid.
+
+> Note: This setting value must be an absolute path.
+
+```json
+    "configureBehavior": {
+        "defaultModuleRoot": "C:/Program Files/Modules/"
+    },
+```
+
 ## Telemetry
 
 The `telemetry` settings control whether winget writes ETW events that may be sent to Microsoft on a default installation of Windows.

schemas/JSON/settings/settings.schema.0.2.json

@@ -198,6 +198,17 @@
         }
       }
     },
+    "ConfigureBehavior": {
+      "description": "Configure settings",
+      "type": "object",
+      "properties": {
+        "defaultModuleRoot": {
+          "description": "The default root directory where PowerShell modules are installed to when applying a configuration.",
+          "type": "string",
+          "maxLength": 32767
+        }
+      }
+    },
     "DownloadBehavior": {
       "description": "Download settings",
       "type": "object",

src/AppInstallerCLICore/ConfigurationCommon.cpp

@@ -48,6 +48,13 @@ namespace AppInstaller::CLI
                 }
             }
 
+            std::filesystem::path defaultModuleRoot = Settings::User().Get<Settings::Setting::ConfigureDefaultModuleRoot>();
+
+            if (!defaultModuleRoot.empty())
+            {
+                return { SetProcessorFactory::PwshConfigurationProcessorLocation::Custom, defaultModuleRoot.u8string() };
+            }
+
             return { SetProcessorFactory::PwshConfigurationProcessorLocation::WinGetModulePath, {} };
         }
     }

src/AppInstallerCLIE2ETests/ConfigureCommand.cs

@@ -115,6 +115,24 @@ public void ConfigureFromTestRepo_Location(TestCommon.TestModuleLocation locatio
                 Constants.SimpleTestModuleName)));
         }
 
+        /// <summary>
+        /// Simple test to confirm that the module was installed to the location specified in the DefaultModuleRoot settings.
+        /// </summary>
+        [Test]
+        public void ConfigureFromTestRepo_SettingsDefaultModuleRoot()
+        {
+            TestCommon.EnsureModuleState(Constants.SimpleTestModuleName, present: false);
+            string moduleTestDir = TestCommon.GetRandomTestDir();
+            WinGetSettingsHelper.ConfigureConfigureBehavior(Constants.DefaultModuleRoot, moduleTestDir);
+
+            string args = TestCommon.GetTestDataFile("Configuration\\Configure_TestRepo_Location.yml");
+            var result = TestCommon.RunAICLICommand(CommandAndAgreementsAndVerbose, args);
+
+            WinGetSettingsHelper.ConfigureInstallBehavior(Constants.DefaultModuleRoot, string.Empty);
+            Assert.AreEqual(0, result.ExitCode);
+            Assert.True(Directory.Exists(Path.Combine(moduleTestDir, Constants.SimpleTestModuleName)));
+        }
+
         /// <summary>
         /// One resource fails, but the other is not dependent and should be executed.
         /// </summary>

src/AppInstallerCLIE2ETests/Constants.cs

@@ -125,6 +125,7 @@ public class Constants
         public const string PortablePackageMachineRoot = "portablePackageMachineRoot";
         public const string InstallBehaviorScope = "scope";
         public const string InstallerTypes = "installerTypes";
+        public const string DefaultModuleRoot = "defaultModuleRoot";
 
         // Configuration
         public const string PSGalleryName = "PSGallery";

src/AppInstallerCLIE2ETests/Helpers/WinGetSettingsHelper.cs

@@ -108,6 +108,20 @@ public static void ConfigureInstallBehavior(string settingName, string value)
             SetWingetSettings(settingsJson);
         }
 
+        /// <summary>
+        /// Configure the configuration behavior.
+        /// </summary>
+        /// <param name="settingName">Setting name.</param>
+        /// <param name="value">Setting value.</param>
+        public static void ConfigureConfigureBehavior(string settingName, string value)
+        {
+            JObject settingsJson = GetJsonSettingsObject("configureBehavior");
+            var configureBehavior = settingsJson["configureBehavior"];
+            configureBehavior[settingName] = value;
+
+            SetWingetSettings(settingsJson);
+        }
+
         /// <summary>
         /// Configure the install behavior preferences.
         /// </summary>

src/AppInstallerCLITests/UserSettings.cpp

@@ -526,6 +526,21 @@ TEST_CASE("SettingsDownloadDefaultDirectory", "[settings]")
     }
 }
 
+TEST_CASE("SettingsConfigureDefaultModuleRoot", "[settings]")
+{
+    auto again = DeleteUserSettingsFiles();
+
+    SECTION("Valid path")
+    {
+        std::string_view json = R"({ "configureBehavior": { "defaultModuleRoot": "C:/Foo/Bar" } })";
+        SetSetting(Stream::PrimaryUserSettings, json);
+        UserSettingsTest userSettingTest;
+
+        REQUIRE(userSettingTest.Get<Setting::ConfigureDefaultModuleRoot>() == "C:/Foo/Bar");
+        REQUIRE(userSettingTest.GetWarnings().size() == 0);
+    }
+}
+
 TEST_CASE("SettingsArchiveExtractionMethod", "[settings]")
 {
     auto again = DeleteUserSettingsFiles();

src/AppInstallerCommonCore/Public/winget/UserSettings.h

@@ -108,6 +108,8 @@ namespace AppInstaller::Settings
         UninstallPurgePortablePackage,
         // Download behavior
         DownloadDefaultDirectory,
+        // Configure behavior
+        ConfigureDefaultModuleRoot,
         // Interactivity
         InteractivityDisable,
 #ifndef AICLI_DISABLE_TEST_HOOKS
@@ -185,6 +187,8 @@ namespace AppInstaller::Settings
         SETTINGMAPPING_SPECIALIZATION(Setting::UninstallPurgePortablePackage, bool, bool, false, ".uninstallBehavior.purgePortablePackage"sv);
         // Download behavior
         SETTINGMAPPING_SPECIALIZATION(Setting::DownloadDefaultDirectory, std::string, std::filesystem::path, {}, ".downloadBehavior.defaultDownloadDirectory"sv);
+        // Configure behavior
+        SETTINGMAPPING_SPECIALIZATION(Setting::ConfigureDefaultModuleRoot, std::string, std::filesystem::path, {}, ".configureBehavior.defaultModuleRoot"sv);
 
         // Network
         SETTINGMAPPING_SPECIALIZATION(Setting::NetworkDownloader, std::string, InstallerDownloader, InstallerDownloader::Default, ".network.downloader"sv);

src/AppInstallerCommonCore/UserSettings.cpp

@@ -401,6 +401,11 @@ namespace AppInstaller::Settings
             return ValidatePathValue(value);
         }
 
+        WINGET_VALIDATE_SIGNATURE(ConfigureDefaultModuleRoot)
+        {
+            return ValidatePathValue(value);
+        }
+
         WINGET_VALIDATE_SIGNATURE(NetworkDownloader)
         {
             static constexpr std::string_view s_downloader_default = "default";