fix: add LatestVersion for gitdownloader

Signed-off-by: zongz <zongzhe1024@163.com>

pkg/client/add_test.go

@@ -11,7 +11,7 @@ import (
 	"kcl-lang.io/kpm/pkg/utils"
 )
 
-func testAddWithModSpec(t *testing.T) {
+func TestAddWithModSpec(t *testing.T) {
 	tests := []struct {
 		name       string
 		pkgSubPath string
@@ -44,9 +44,14 @@ func testAddWithModSpec(t *testing.T) {
 		},
 		{
 			name:       "TestAddOciWithNoTag",
-			pkgSubPath: "no_tag",
+			pkgSubPath: "no_oci_ref",
 			sourceUrl:  "oci://ghcr.io/kcl-lang/helloworld",
 		},
+		{
+			name:       "TestAddGitWithNoTag",
+			pkgSubPath: "no_git_ref",
+			sourceUrl:  "git://github.com/kcl-lang/flask-demo-kcl-manifests.git",
+		},
 	}
 
 	for _, tt := range tests {

pkg/client/client_test.go

@@ -82,7 +82,7 @@ func TestWithGlobalLock(t *testing.T) {
 	test.RunTestWithGlobalLock(t, "TestDownloadGitWithPackage", testDownloadGitWithPackage)
 	test.RunTestWithGlobalLock(t, "TestModandLockFilesWithGitPackageDownload", testModandLockFilesWithGitPackageDownload)
 	test.RunTestWithGlobalLock(t, "TestDependencyGraph", testDependencyGraph)
-	test.RunTestWithGlobalLock(t, "testAddWithModSpec", testAddWithModSpec)
+	test.RunTestWithGlobalLock(t, "testAddWithModSpec", TestAddWithModSpec)
 }
 
 // TestDownloadOci test download from oci registry.

pkg/client/test_data/add_with_mod_spec/no_git_ref/kcl.mod.bk

@@ -0,0 +1,4 @@
+[package]
+name = "git"
+edition = "v0.10.0"
+version = "0.0.1"

pkg/client/test_data/add_with_mod_spec/no_git_ref/kcl.mod.expect

@@ -0,0 +1,7 @@
+[package]
+name = "git"
+edition = "v0.10.0"
+version = "0.0.1"
+
+[dependencies]
+flask_manifests = { git = "https://github.com/kcl-lang/flask-demo-kcl-manifests.git", commit = "ade147b", version = "0.0.1" }

pkg/client/test_data/add_with_mod_spec/no_git_ref/kcl.mod.lock.expect

@@ -0,0 +1,7 @@
+[dependencies]
+  [dependencies.flask_manifests]
+    name = "flask_manifests"
+    full_name = "flask_manifests_0.0.1"
+    version = "0.0.1"
+    url = "https://github.com/kcl-lang/flask-demo-kcl-manifests.git"
+    commit = "ade147b"

pkg/client/test_data/add_with_mod_spec/no_git_ref/main.k

@@ -0,0 +1 @@
+The_first_kcl_program = 'Hello World!'

pkg/downloader/downloader.go

@@ -138,7 +138,41 @@ type GitDownloader struct{}
 func (d *GitDownloader) LatestVersion(opts *DownloadOptions) (string, error) {
 	// TODO：supports fetch the latest commit from the git bare repo,
 	// after totally transfer to the new storage.
-	return "main", nil
+	// refer to cargo: https://github.com/rust-lang/cargo/blob/3dedb85a25604bdbbb8d3bf4b03162961a4facd0/crates/cargo-util-schemas/src/core/source_kind.rs#L133
+	var err error
+	tmp, err := os.MkdirTemp("", "")
+	if err != nil {
+		return "", err
+	}
+	tmp = filepath.Join(tmp, constants.GitScheme)
+
+	defer func() {
+		err = os.RemoveAll(tmp)
+	}()
+
+	repo, err := git.CloneWithOpts(
+		git.WithCommit(opts.Source.Commit),
+		git.WithBranch(opts.Source.Branch),
+		git.WithTag(opts.Source.Git.Tag),
+		git.WithRepoURL(opts.Source.Git.Url),
+		git.WithLocalPath(tmp),
+	)
+
+	if err != nil {
+		return "", err
+	}
+
+	ref, err := repo.Head()
+	if err != nil {
+		return "", err
+	}
+
+	commit, err := repo.CommitObject(ref.Hash())
+	if err != nil {
+		return "", err
+	}
+
+	return commit.Hash.String()[:7], nil
 }
 
 // OciDownloader is the downloader for the OCI source.

pkg/visitor/visitor.go

@@ -154,7 +154,7 @@ func (rv *RemoteVisitor) Visit(s *downloader.Source, v visitFunc) error {
 			s.Oci.Tag = latest
 		}
 		if s.Git != nil {
-			s.Git.Branch = latest
+			s.Git.Commit = latest
 		}
 	}
 	var modPath string