Bump: download dep AutoUpdatedVersions.props from VCS when not local

Reverts the artifact-dep approach from 0cebead: AutoUpdatedVersions.props
is build-independent source code, so coupling bump to a CI artifact made
the version pipeline build-dependent for no reason.

New behavior in AutoUpdatedVersionsFile.TryWrite: if neither local
candidate path exists (source-dependencies/<dep>/<eng>/... or
../<dep>/<eng>/...), fall back to VcsRepository.TryDownloadTextFile on
the dep's ReleaseBranch (or Branch when null). XML is parsed in-memory
via the new TryParse helper, which TryRead now also delegates to.

This unblocks bump for repos like Metalama.Vsx 2026.1 where Metalama and
Metalama.Compiler are artifact-only deps and never appear under
source-dependencies.

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

Author: PostSharpAgent

.teamcity/settings.kts

@@ -29,7 +29,6 @@ object PublicBuild : BuildType({
 +:artifacts/testResults/**/*=>artifacts/testResults
 +:artifacts/logs/**/*=>logs
 +:artifacts/dumps/**/*=>dumps
-+:eng/AutoUpdatedVersions.props=>eng
 """
 
     params {

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

@@ -31,9 +31,16 @@ private static bool TryRead(
         string path,
         [NotNullWhen( true )] out string? dependencyReleasedVersion,
         [NotNullWhen( true )] out string? releasedMainVersionPropertyValue )
-    {
-        var theirAutoUpdatedVersionsDocument = XDocument.Load( path );
+        => TryParse( context, dependency, path, XDocument.Load( path ), out dependencyReleasedVersion, out releasedMainVersionPropertyValue );
 
+    private static bool TryParse(
+        BuildContext context,
+        DependencyDefinition dependency,
+        string source,
+        XDocument theirAutoUpdatedVersionsDocument,
+        [NotNullWhen( true )] out string? dependencyReleasedVersion,
+        [NotNullWhen( true )] out string? releasedMainVersionPropertyValue )
+    {
         var releasedVersionPropertyName = $"{dependency.NameWithoutDot}ReleaseVersion";
 
         dependencyReleasedVersion = theirAutoUpdatedVersionsDocument.Root
@@ -43,7 +50,7 @@ private static bool TryRead(
 
         if ( string.IsNullOrEmpty( dependencyReleasedVersion ) )
         {
-            context.Console.WriteError( $"The '{releasedVersionPropertyName}' property in '{path}' is not defined." );
+            context.Console.WriteError( $"The '{releasedVersionPropertyName}' property in '{source}' is not defined." );
 
             releasedMainVersionPropertyValue = null;
             dependencyReleasedVersion = null;
@@ -60,7 +67,7 @@ private static bool TryRead(
 
         if ( string.IsNullOrEmpty( releasedMainVersionPropertyValue ) )
         {
-            context.Console.WriteError( $"The '{releasedMainVersionPropertyName}' property in '{path}' is not defined." );
+            context.Console.WriteError( $"The '{releasedMainVersionPropertyName}' property in '{source}' is not defined." );
 
             releasedMainVersionPropertyValue = null;
             dependencyReleasedVersion = null;
@@ -125,28 +132,48 @@ public static bool TryWrite(
                         dependency.Name,
                         dependency.EngineeringDirectory,
                         FileName ) ),
-                Path.GetFullPath( Path.Combine( context.RepoDirectory, "..", dependency.Name, dependency.EngineeringDirectory, FileName ) ),
-
-                // Artifact-dep flow: producer publishes its own AutoUpdatedVersions.props as a build artifact;
-                // consumer pulls it to dependencies/<key>/AutoUpdatedVersions.props (see TeamCitySettingsFile and ConfigurationProperties).
-                Path.GetFullPath( Path.Combine( context.RepoDirectory, "dependencies", dependencyConfiguration.Key, FileName ) )
+                Path.GetFullPath( Path.Combine( context.RepoDirectory, "..", dependency.Name, dependency.EngineeringDirectory, FileName ) )
             ];
 
             var theirAutoUpdatedVersionsFilePath = filePathCandidates.FirstOrDefault( File.Exists );
 
-            if ( theirAutoUpdatedVersionsFilePath == null )
+            string source;
+            XDocument theirAutoUpdatedVersionsDocument;
+
+            if ( theirAutoUpdatedVersionsFilePath != null )
+            {
+                source = theirAutoUpdatedVersionsFilePath;
+                theirAutoUpdatedVersionsDocument = XDocument.Load( theirAutoUpdatedVersionsFilePath );
+            }
+            else
             {
-                context.Console.WriteError( $"None of these files exists: {string.Join( ", ", filePathCandidates.Select( x => $"'{x}'" ) )}." );
+                // Fallback for artifact-only dependencies: AutoUpdatedVersions.props is build-independent source code,
+                // so download it directly from the dependency's VCS repository on its release branch (or development branch
+                // when no release branch is set). Keeps bump independent of any CI artifact pipeline.
+                var branch = dependency.ReleaseBranch ?? dependency.Branch;
+                var pathInRepo = $"{dependency.EngineeringDirectory.Replace( '\\', '/' )}/{FileName}";
+                source = $"{dependency.VcsRepository}/{branch}/{pathInRepo}";
 
-                errors++;
+                context.Console.WriteMessage(
+                    $"Local '{FileName}' for '{dependency.Name}' not found at any of [{string.Join( ", ", filePathCandidates.Select( x => $"'{x}'" ) )}]; downloading from '{source}'." );
 
-                continue;
+                if ( !dependency.VcsRepository.TryDownloadTextFile( context.Console, branch, pathInRepo, out var text ) )
+                {
+                    context.Console.WriteError( $"Failed to download '{source}'." );
+
+                    errors++;
+
+                    continue;
+                }
+
+                theirAutoUpdatedVersionsDocument = XDocument.Parse( text );
             }
 
-            if ( !TryRead(
+            if ( !TryParse(
                     context,
                     dependency,
-                    theirAutoUpdatedVersionsFilePath,
+                    source,
+                    theirAutoUpdatedVersionsDocument,
                     out var dependencyReleasedVersion,
                     out var releasedMainVersionPropertyValue ) )
             {

src/PostSharp.Engineering.BuildTools/ContinuousIntegration/TeamCity/Generation/ConfigurationProperties.cs

@@ -37,10 +37,7 @@ public ConfigurationProperties( Product product, BuildConfiguration configuratio
             .Select( d => new TeamCitySnapshotDependency(
                          d.Definition.CiConfiguration.BuildTypes[d.Configuration],
                          true,
-                         string.Join(
-                             "\\n",
-                             $"+:{d.Definition.GetPrivateArtifactsDirectory( d.Configuration ).Replace( Path.DirectorySeparatorChar, '/' )}/**/*=>dependencies/{d.Key}",
-                             $"+:{d.Definition.EngineeringDirectory.Replace( '\\', '/' )}/AutoUpdatedVersions.props=>dependencies/{d.Key}/AutoUpdatedVersions.props" ),
+                         $"+:{d.Definition.GetPrivateArtifactsDirectory( d.Configuration ).Replace( Path.DirectorySeparatorChar, '/' )}/**/*=>dependencies/{d.Key}",
                          ReuseBuilds: d.ArtifactPickup == DependencyArtifactPickup.LastSuccessful ? ReuseBuilds.LastSuccessful : ReuseBuilds.Default,
                          Branch: d.ArtifactPickup == DependencyArtifactPickup.LastSuccessful && configuration == BuildConfiguration.Public
                              ? d.Definition.ReleaseBranch

src/PostSharp.Engineering.BuildTools/ContinuousIntegration/TeamCity/Generation/TeamCitySettingsFile.cs

@@ -52,10 +52,6 @@ internal static bool TryWrite( BuildContext context )
             publishedArtifactRules += $@"\n+:{productProperties.LogsDirectory}/**/*=>logs";
             publishedArtifactRules += $@"\n+:{productProperties.DumpsDirectory}/**/*=>dumps";
 
-            // Publish AutoUpdatedVersions.props so consumers can read released versions of auto-updated dependencies
-            // without needing them as source dependencies. Read by AutoUpdatedVersionsFile.TryRead during bump.
-            publishedArtifactRules += $@"\n+:{product.AutoUpdatedVersionsFilePath.Replace( '\\', '/' )}=>{product.EngineeringDirectory.Replace( '\\', '/' )}";
-
             var teamCityBuildConfiguration = CreateBuildConfiguration(
                 context,
                 productProperties,
@@ -273,27 +269,12 @@ private static TeamCityBuildConfiguration CreateUpstreamMergeConfiguration( Prod
 
     private static TeamCityBuildConfiguration CreateBumpConfiguration( ProductProperties productProperties )
     {
-        var product = productProperties.Product;
-
-        // Bump reads each auto-update dependency's AutoUpdatedVersions.props to learn its latest released version.
-        // Pull only the props file from the dependency's last successful Public build on its release branch.
-        var bumpDependencies = product.DependencyDefinition.GetAllDependencies( BuildConfiguration.Public )
-            .Where( d => d.Definition.AutoUpdateVersion && d.Definition.GenerateSnapshotDependency )
-            .Select( d => new TeamCitySnapshotDependency(
-                         d.Definition.CiConfiguration.BuildTypes[d.Configuration],
-                         true,
-                         $"+:{d.Definition.EngineeringDirectory.Replace( '\\', '/' )}/AutoUpdatedVersions.props=>dependencies/{d.Key}/AutoUpdatedVersions.props",
-                         ReuseBuilds: ReuseBuilds.LastSuccessful,
-                         Branch: d.Definition.ReleaseBranch ) )
-            .OrderBy( d => d.ObjectId )
-            .ToArray();
-
         var bumpConfiguration = new TeamCityBuildConfiguration(
             objectName: "VersionBump",
             name: $"Version Bump",
             productProperties.DefaultBranch,
             productProperties.VcsId,
-            buildAgentRequirements: product.ResolvedBuildAgentRequirements )
+            buildAgentRequirements: productProperties.Product.ResolvedBuildAgentRequirements )
         {
             BuildSteps =
             [
@@ -302,10 +283,9 @@ private static TeamCityBuildConfiguration CreateBumpConfiguration( ProductProper
                     "Bump",
                     "bump",
                     areCustomArgumentsAllowed: true,
-                    dockerSpec: product.DockerSpec,
-                    timeout: product.VersionBumpTimeout )
+                    dockerSpec: productProperties.Product.DockerSpec,
+                    timeout: productProperties.Product.VersionBumpTimeout )
             ],
-            SnapshotDependencies = bumpDependencies,
             IsSshAgentRequired = productProperties.IsRepoRemoteSsh
         };