Make InstallPkgParams properties nullable to distinguish unset from false

This addresses the feedback to check if reqResourceParams properties are null.
Now the code can distinguish between:
- User didn't specify the property (null) -> use cmdlet-level parameter
- User explicitly set the property to true/false -> use that value

Changed properties to nullable (bool?, ScopeType?) and updated logic to only
set them when explicitly provided in the RequiredResource hashtable.

Co-authored-by: alerickson <25858831+alerickson@users.noreply.github.com>

Author: Copilot

src/code/InstallPSResource.cs

@@ -487,9 +487,9 @@ private void RequiredResourceHelper(Hashtable reqResourceHash)
                         }
                     }
 
-                    if (pkgParams.Scope == ScopeType.AllUsers)
+                    if (pkgParams.Scope.HasValue && pkgParams.Scope.Value == ScopeType.AllUsers)
                     {
-                        _pathsToInstallPkg = Utils.GetAllInstallationPaths(this, pkgParams.Scope);
+                        _pathsToInstallPkg = Utils.GetAllInstallationPaths(this, pkgParams.Scope.Value);
                     }
 
                     pkgVersion = pkgInstallInfo["version"] == null ? String.Empty : pkgInstallInfo["version"].ToString();
@@ -566,14 +566,14 @@ private void ProcessInstallHelper(string[] pkgNames, string pkgVersion, bool pkg
             }
 
             // When reqResourceParams is provided (via -RequiredResource), use its properties
-            // instead of the cmdlet-level parameters
-            bool acceptLicense = reqResourceParams != null ? reqResourceParams.AcceptLicense : AcceptLicense;
-            bool quiet = reqResourceParams != null ? reqResourceParams.Quiet : Quiet;
-            bool reinstall = reqResourceParams != null ? reqResourceParams.Reinstall : Reinstall;
-            bool trustRepository = reqResourceParams != null ? reqResourceParams.TrustRepository : TrustRepository;
-            bool noClobber = reqResourceParams != null ? reqResourceParams.NoClobber : NoClobber;
-            bool skipDependencyCheck = reqResourceParams != null ? reqResourceParams.SkipDependencyCheck : SkipDependencyCheck;
-            ScopeType scope = reqResourceParams != null ? reqResourceParams.Scope : Scope;
+            // instead of the cmdlet-level parameters. Only use the property if it was explicitly set (not null).
+            bool acceptLicense = reqResourceParams?.AcceptLicense ?? AcceptLicense;
+            bool quiet = reqResourceParams?.Quiet ?? Quiet;
+            bool reinstall = reqResourceParams?.Reinstall ?? Reinstall;
+            bool trustRepository = reqResourceParams?.TrustRepository ?? TrustRepository;
+            bool noClobber = reqResourceParams?.NoClobber ?? NoClobber;
+            bool skipDependencyCheck = reqResourceParams?.SkipDependencyCheck ?? SkipDependencyCheck;
+            ScopeType scope = reqResourceParams?.Scope ?? Scope;
 
             IEnumerable<PSResourceInfo> installedPkgs = _installHelper.BeginInstallPackages(
                 names: pkgNames,

src/code/InstallPkgParams.cs

@@ -11,15 +11,15 @@ public class InstallPkgParams
     public string Name { get; set; }
     public VersionRange Version { get; set; }
     public string Repository { get; set; }
-    public bool AcceptLicense { get; set; } 
+    public bool? AcceptLicense { get; set; } 
     public bool Prerelease { get; set; }
-    public ScopeType Scope { get; set; }
-    public bool Quiet { get; set; }
-    public bool Reinstall { get; set; }
+    public ScopeType? Scope { get; set; }
+    public bool? Quiet { get; set; }
+    public bool? Reinstall { get; set; }
     public bool Force { get; set; }
-    public bool TrustRepository { get; set; }
-    public bool NoClobber { get; set; }
-    public bool SkipDependencyCheck { get; set; }
+    public bool? TrustRepository { get; set; }
+    public bool? NoClobber { get; set; }
+    public bool? SkipDependencyCheck { get; set; }
 
 
 
@@ -67,8 +67,10 @@ public void SetProperty(string propertyName, string propertyValue, out ErrorReco
                 break;
 
             case "acceptlicense":
-                bool.TryParse(propertyValue, out bool acceptLicenseTmp);
-                AcceptLicense = acceptLicenseTmp;
+                if (!string.IsNullOrWhiteSpace(propertyValue) && bool.TryParse(propertyValue, out bool acceptLicenseTmp))
+                {
+                    AcceptLicense = acceptLicenseTmp;
+                }
                 break;
 
             case "prerelease":
@@ -82,28 +84,38 @@ public void SetProperty(string propertyName, string propertyValue, out ErrorReco
                 break;
 
             case "quiet":
-                bool.TryParse(propertyValue, out bool quietTmp);
-                Quiet = quietTmp;
+                if (!string.IsNullOrWhiteSpace(propertyValue) && bool.TryParse(propertyValue, out bool quietTmp))
+                {
+                    Quiet = quietTmp;
+                }
                 break;
 
             case "reinstall":
-                bool.TryParse(propertyValue, out bool reinstallTmp);
-                Reinstall = reinstallTmp;
+                if (!string.IsNullOrWhiteSpace(propertyValue) && bool.TryParse(propertyValue, out bool reinstallTmp))
+                {
+                    Reinstall = reinstallTmp;
+                }
                 break;
 
             case "trustrepository":
-                bool.TryParse(propertyValue, out bool trustRepositoryTmp);
-                TrustRepository = trustRepositoryTmp;
+                if (!string.IsNullOrWhiteSpace(propertyValue) && bool.TryParse(propertyValue, out bool trustRepositoryTmp))
+                {
+                    TrustRepository = trustRepositoryTmp;
+                }
                 break;
 
             case "noclobber":
-                bool.TryParse(propertyValue, out bool noClobberTmp);
-                NoClobber = noClobberTmp;
+                if (!string.IsNullOrWhiteSpace(propertyValue) && bool.TryParse(propertyValue, out bool noClobberTmp))
+                {
+                    NoClobber = noClobberTmp;
+                }
                 break;
 
             case "skipdependencycheck":
-                bool.TryParse(propertyValue, out bool skipDependencyCheckTmp);
-                SkipDependencyCheck = skipDependencyCheckTmp;
+                if (!string.IsNullOrWhiteSpace(propertyValue) && bool.TryParse(propertyValue, out bool skipDependencyCheckTmp))
+                {
+                    SkipDependencyCheck = skipDependencyCheckTmp;
+                }
                 break;
 
             default: