From b49f130759f007a2a7fbfb2b80205b8111d25e67 Mon Sep 17 00:00:00 2001
From: utnim2 <mintugogoi567@gmail.com>
Date: Mon, 29 Jul 2024 06:21:58 +0530
Subject: [PATCH] Revert "added the version field in `kcl.mod`"

This reverts commit 9b57736b7fb7baebcc1f8aacfbcfdee3476242ce.
---
 pkg/client/client.go   |  9 ---------
 pkg/package/modfile.go |  2 +-
 pkg/package/toml.go    | 25 +------------------------
 3 files changed, 2 insertions(+), 34 deletions(-)

diff --git a/pkg/client/client.go b/pkg/client/client.go
index 80c23c56..631ffb2d 100644
--- a/pkg/client/client.go
+++ b/pkg/client/client.go
@@ -829,15 +829,6 @@ func (c *KpmClient) AddDepToPkg(kclPkg *pkg.KclPkg, d *pkg.Dependency) error {
 		return err
 	}
 
-	// After downloading, extract the version from the kcl.mod file
-    downloadedPkg, err := c.LoadPkgFromPath(c.getDepStorePath(c.homePath, d, false))
-    if err != nil {
-        return fmt.Errorf("failed to load downloaded package: %w", err)
-    }
-    d.Version = downloadedPkg.GetPkgVersion()
-    kclPkg.ModFile.Dependencies.Deps.Set(d.Name, *d)
-
-
 	return err
 }
 
diff --git a/pkg/package/modfile.go b/pkg/package/modfile.go
index a73c12ee..b4fb3b49 100644
--- a/pkg/package/modfile.go
+++ b/pkg/package/modfile.go
@@ -191,13 +191,13 @@ func (deps *Dependencies) CheckForLocalDeps() bool {
 type Dependency struct {
 	Name     string `json:"name" toml:"name,omitempty"`
 	FullName string `json:"-" toml:"full_name,omitempty"`
+	Version  string `json:"-" toml:"version,omitempty"`
 	Sum      string `json:"-" toml:"sum,omitempty"`
 	// The actual local path of the package.
 	// In vendor mode is "current_kcl_package/vendor"
 	// In non-vendor mode is "$KCL_PKG_PATH"
 	LocalFullPath     string `json:"manifest_path" toml:"-"`
 	downloader.Source `json:"-"`
-	Version string `json:"version,omitempty" toml:"version,omitempty"`
 }
 
 func (d *Dependency) FromKclPkg(pkg *KclPkg) {
diff --git a/pkg/package/toml.go b/pkg/package/toml.go
index 7de7abcd..f0c7eb6c 100644
--- a/pkg/package/toml.go
+++ b/pkg/package/toml.go
@@ -85,17 +85,7 @@ func (dep *Dependency) MarshalTOML() string {
 	source := dep.Source.MarshalTOML()
 	var sb strings.Builder
 	if len(source) != 0 {
-		sourceWithoutBrace := strings.TrimSuffix(source, "}")
-
-		// Add the version if it exists
-        if dep.Version != "" {
-            sourceWithoutBrace += fmt.Sprintf(`, version = "%s"`, dep.Version)
-        }
-
-		// Add the closing brace back
-        sourceWithoutBrace += "}"
-
-		sb.WriteString(fmt.Sprintf(DEP_PATTERN, dep.Name, sourceWithoutBrace))
+		sb.WriteString(fmt.Sprintf(DEP_PATTERN, dep.Name, source))
 	}
 	return sb.String()
 }
@@ -245,14 +235,6 @@ func (dep *Dependency) UnmarshalModTOML(data interface{}) error {
 	}
 
 	dep.Source = source
-
-	// Try to extract version from the data
-    if meta, ok := data.(map[string]interface{}); ok {
-        if v, ok := meta["version"].(string); ok {
-            dep.Version = v
-        }
-    }
-
 	var version string
 	if source.Git != nil {
 		version, err = source.Git.GetValidGitReference()
@@ -267,11 +249,6 @@ func (dep *Dependency) UnmarshalModTOML(data interface{}) error {
 		version = source.Registry.Version
 	}
 
-	// If version is not set explicitly, use the one from the source
-    if dep.Version == "" {
-        dep.Version = version
-    }
-
 	dep.FullName = fmt.Sprintf(PKG_NAME_PATTERN, dep.Name, version)
 	dep.Version = version
 	return nil