commit 469eb6193c0a49495ea2ce7432cf749f077ad596
author Sigurd Meldgaard <sigurdm@google.com> Mon Aug 18 13:14:55 2025 +0200
committer GitHub <noreply@github.com> Mon Aug 18 13:14:55 2025 +0200
tree b3632a21c803a5515bcb483767a2131d04dce804
parent befc0a1bb283806c026505e7a703e175bfe38873

Fix tag_pattern lockfile deserialization (#4646)

lib/src/source/git.dart

diff --git a/lib/src/source/git.dart b/lib/src/source/git.dart
index 39ea66d..9087944 100644
--- a/lib/src/source/git.dart
+++ b/lib/src/source/git.dart
@@ -171,7 +171,7 @@
       );
     }
 
-    final tagPattern = description['tag_pattern'];
+    final tagPattern = description['tag-pattern'];
     if (tagPattern is! String?) {
       throw const FormatException(
         "The 'tag_pattern' field of the description "

test/get/git/tag_pattern_test.dart

diff --git a/test/get/git/tag_pattern_test.dart b/test/get/git/tag_pattern_test.dart
index 0f36ae4..732ca40 100644
--- a/test/get/git/tag_pattern_test.dart
+++ b/test/get/git/tag_pattern_test.dart
@@ -313,4 +313,70 @@
       ),
     );
   });
+
+  // Regression test for https://github.com/dart-lang/pub/issues/4644.
+  test(
+    'Lockfile persists tag-pattern when `pub get`ing already locked file',
+    () async {
+      ensureGit();
+      await d.git('foo.git', [
+        d.libPubspec('foo', '1.0.0', sdk: '^3.9.0'),
+      ]).create();
+      await d.git('foo.git', []).tag('1.0.0');
+
+      await d
+          .appDir(
+            dependencies: {
+              'foo': {
+                'git': {
+                  'url': p.join(d.sandbox, 'foo.git'),
+                  'tag_pattern': '{{version}}',
+                },
+              },
+            },
+            pubspec: {
+              'environment': {'sdk': '^3.9.0'},
+            },
+          )
+          .create();
+
+      await pubGet(
+        output: allOf(contains('+ foo 1.0.0')),
+        environment: {'_PUB_TEST_SDK_VERSION': '3.9.0'},
+      );
+      final lockfile = loadYaml(
+        File(p.join(d.sandbox, appPath, 'pubspec.lock')).readAsStringSync(),
+      );
+      final s = Platform.pathSeparator;
+      final foo = ((lockfile as Map)['packages'] as Map)['foo'];
+      expect(foo, {
+        'dependency': 'direct main',
+        'description': {
+          'path': '.',
+          'resolved-ref': isA<String>(),
+          'tag-pattern': '{{version}}',
+          'url': '${d.sandbox}${s}foo.git',
+        },
+        'source': 'git',
+        'version': '1.0.0',
+      });
+      await pubGet(environment: {'_PUB_TEST_SDK_VERSION': '3.9.0'});
+      final lockfile2 = loadYaml(
+        File(p.join(d.sandbox, appPath, 'pubspec.lock')).readAsStringSync(),
+      );
+      final foo2 = ((lockfile2 as Map)['packages'] as Map)['foo'];
+
+      expect(foo2, {
+        'dependency': 'direct main',
+        'description': {
+          'path': '.',
+          'resolved-ref': isA<String>(),
+          'tag-pattern': '{{version}}',
+          'url': '${d.sandbox}${s}foo.git',
+        },
+        'source': 'git',
+        'version': '1.0.0',
+      });
+    },
+  );
 }