[AI Generated] Ensure correct DSC export of admin settings

doc/ReleaseNotes.md

@@ -38,4 +38,4 @@ The PowerShell module now automatically uses `GH_TOKEN` or `GITHUB_TOKEN` enviro
 
 ## Bug Fixes
 
-<!-- Nothing yet! -->
+* DSC export now correctly exports WinGet Admin Settings

src/AppInstallerCommonCore/AdminSettings.cpp

@@ -180,7 +180,22 @@ namespace AppInstaller::Settings
 
         void AdminSettingsInternal::LoadAdminSettings()
         {
-            auto stream = m_settingStream.Get();
+            std::unique_ptr<std::istream> stream;
+            try
+            {
+                stream = m_settingStream.Get();
+            }
+            catch (const std::exception& e)
+            {
+                AICLI_LOG(Core, Error, << "Failed to read admin settings: " << e.what() << ". Falling back to default values.");
+                return;
+            }
+            catch (...)
+            {
+                AICLI_LOG(Core, Error, << "Failed to read admin settings due to unknown exception. Falling back to default values.");
+                return;
+            }
+
             if (!stream)
             {
                 AICLI_LOG(Core, Verbose, << "Admin settings was not found");
@@ -223,6 +238,8 @@ namespace AppInstaller::Settings
             {
                 m_settingValues.DefaultProxy.emplace(std::move(defaultProxy));
             }
+
+            AICLI_LOG(Core, Verbose, << "Admin settings loaded successfully");
         }
 
         bool AdminSettingsInternal::SaveAdminSettings()

src/AppInstallerCommonCore/Settings.cpp

@@ -293,14 +293,19 @@ namespace AppInstaller::Settings
                 return result;
             }
 
-            void SetVerificationData(VerificationData data)
+            bool SetVerificationData(VerificationData data)
             {
                 YAML::Emitter out;
                 out << YAML::BeginMap;
                 out << YAML::Key << NodeName_Sha256 << YAML::Value << SHA256::ConvertToString(data.Hash);
                 out << YAML::EndMap;
 
-                m_secure.Set(out.str());
+                bool result = m_secure.Set(out.str());
+                if (!result)
+                {
+                    AICLI_LOG(Core, Warning, << "Failed to write verification data for '" << m_name << "'");
+                }
+                return result;
             }
 
         public:
@@ -338,7 +343,15 @@ namespace AppInstaller::Settings
                     VerificationData verData;
                     verData.Hash = m_hash.value();
 
-                    SetVerificationData(verData);
+                    if (!SetVerificationData(verData))
+                    {
+                        AICLI_LOG(Core, Error, << "Failed to write verification data for '" << m_name << "'. Reverting setting write to maintain consistency.");
+                        AICLI_LOG(Core, Warning, << "This failure may be caused by insufficient permissions or disk space issues");
+                        // Verification data write failed, so we need to revert the main write
+                        // to maintain consistency
+                        Remove();
+                        return false;
+                    }
                 }
 
                 return exchangeResult;