Bump: detect file-actually-changed in AutoUpdatedVersions.props

The dependency loop's per-edit hasChanges flag was stale when cross-family
edits cancelled out (e.g. Metalama.Compiler toggled 2026.0->2026.1->2026.0
on Vsx 2026.1 PrePublish), causing an empty <<DEPENDENCIES_UPDATED>>
commit that failed with "nothing to commit". WriteIfDifferent is now
called unconditionally and its return value drives both the dry-run log
and the commit gate. WriteIfDifferent gains an optional dry parameter
to support the dry-mode path.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: PostSharpAgent

src/PostSharp.Engineering.BuildTools/Build/Files/AutoUpdatedVersionsFile.cs

@@ -288,12 +288,11 @@ public static bool TryWrite(
             thisMainVersionElement.Value = versionComponents.MainVersion;
         }
 
-        // Write changes (always try to write to handle formatting and condition changes).
-        if ( !dry )
-        {
-            hasChanges = TextFileHelper.WriteIfDifferent( thisAutoUpdatedVersionsFilePath, thisAutoUpdatedVersionsDocument, context );
-        }
-        else if ( hasChanges )
+        // Always compare against the file on disk to handle formatting and condition changes,
+        // and to reset the stale per-edit hasChanges flag when in-memory edits cancel out.
+        hasChanges = TextFileHelper.WriteIfDifferent( thisAutoUpdatedVersionsFilePath, thisAutoUpdatedVersionsDocument, context, dry );
+
+        if ( dry && hasChanges )
         {
             context.Console.WriteMessage( $"New content for '{thisAutoUpdatedVersionsFilePath}':" );
             context.Console.WriteMessage( thisAutoUpdatedVersionsDocument.ToNiceString() );
@@ -308,13 +307,15 @@ public static bool TryWrite(
     public static bool TryWriteAndCommit( BuildContext context, bool dry )
     {
         // Go through all dependencies and update their fixed version in AutoUpdatedVersions.props file.
-        if ( !TryWrite( context, dry, out var dependenciesUpdated, out _, out _, out _ ) )
+        // Gate on hasChanges (whether the file was actually written), not on the per-dependency edit flag:
+        // cross-family edits can cancel out, leaving the file unchanged on disk.
+        if ( !TryWrite( context, dry, out _, out var hasChanges, out _, out _ ) )
         {
             return false;
         }
 
-        // Commit and push if dependencies versions were updated in previous step.
-        if ( dependenciesUpdated )
+        // Commit and push if the AutoUpdatedVersions.props file was changed.
+        if ( hasChanges )
         {
             if ( dry )
             {

src/PostSharp.Engineering.BuildTools/Utilities/TextFileHelper.cs

@@ -1,30 +1,38 @@
-// Copyright (c) SharpCrafters s.r.o. See the LICENSE.md file in the root directory of this repository root for details.
-
-using PostSharp.Engineering.BuildTools.Build;
-using System.IO;
-using System.Xml.Linq;
-
-namespace PostSharp.Engineering.BuildTools.Utilities;
-
-internal static class TextFileHelper
-{
-    public static bool WriteIfDifferent( string path, XDocument content, BuildContext context ) => WriteIfDifferent( path, content.ToNiceString(), context );
-
-    public static bool WriteIfDifferent( string path, string content, BuildContext context )
-    {
-        if ( File.Exists( path ) && content == File.ReadAllText( path ) )
-        {
-            context.Console.WriteMessage( $"The file '{path}' is up to date." );
-
-            return false;
-        }
-        else
-        {
-            context.Console.WriteMessage( $"Writing '{path}'." );
-            Directory.CreateDirectory( Path.GetDirectoryName( path )! );
-            File.WriteAllText( path, content );
-
-            return true;
-        }
-    }
+// Copyright (c) SharpCrafters s.r.o. See the LICENSE.md file in the root directory of this repository root for details.
+
+using PostSharp.Engineering.BuildTools.Build;
+using System.IO;
+using System.Xml.Linq;
+
+namespace PostSharp.Engineering.BuildTools.Utilities;
+
+internal static class TextFileHelper
+{
+    public static bool WriteIfDifferent( string path, XDocument content, BuildContext context, bool dry = false )
+        => WriteIfDifferent( path, content.ToNiceString(), context, dry );
+
+    public static bool WriteIfDifferent( string path, string content, BuildContext context, bool dry = false )
+    {
+        if ( File.Exists( path ) && content == File.ReadAllText( path ) )
+        {
+            context.Console.WriteMessage( $"The file '{path}' is up to date." );
+
+            return false;
+        }
+        else
+        {
+            if ( dry )
+            {
+                context.Console.WriteMessage( $"Dry run: would write '{path}'." );
+            }
+            else
+            {
+                context.Console.WriteMessage( $"Writing '{path}'." );
+                Directory.CreateDirectory( Path.GetDirectoryName( path )! );
+                File.WriteAllText( path, content );
+            }
+
+            return true;
+        }
+    }
 }