Fix/commit handling

.changes/unreleased/Fixed-20250815-184431.yaml

@@ -0,0 +1,3 @@
+kind: Fixed
+body: Improved check for LatestVersion when component has no previous versions
+time: 2025-08-15T18:44:31.55796646+02:00

.changes/unreleased/Fixed-20250815-184538.yaml

@@ -0,0 +1,3 @@
+kind: Fixed
+body: Improve path filter when comparing paths with from different roots
+time: 2025-08-15T18:45:38.055338092+02:00

internal/cloud/client_wrapper.go

@@ -3,6 +3,7 @@
 import (
 	"context"
 	"github.com/mach-composer/mcc-sdk-go/mccsdk"
+	"io"
 )
 
 var _ ClientWrapper = (*MccSdkClientWrapper)(nil)
@@ -31,7 +32,7 @@
 		ComponentCommitCreateDraft(mccsdk.ComponentCommitCreateDraft{
 			Commits: commits,
 		}).
 		Execute()
 
 	return err
 }
@@ -43,11 +44,19 @@
 		ComponentsApi.
 		ComponentLatestVersion(ctx, organization, project, componentKey).
 		Branch(branch).
 		Execute()
 
 	// Small hack because the mccsdk cannot deserialize an empty response body
 	if res != nil && res.StatusCode == 200 {
-		return nil, nil
+		b, err := io.ReadAll(res.Body)
+		if err != nil {
+			return nil, err
+		}
+
+		if string(b) == "{}" {
+			// No version found, return nil without an error
+			return nil, nil
+		}
 	}
 
 	return r, err
@@ -61,7 +70,7 @@
 			Version: version,
 			Branch:  &branch,
 		}).
 		Execute()
 	return r, err
 }
 

internal/gitutils/diff.go

@@ -3,6 +3,7 @@ package gitutils
 import (
 	"context"
 	"fmt"
+	"path/filepath"
 	"strings"
 
 	"github.com/go-git/go-git/v5"
@@ -19,7 +20,9 @@ func pathFilter(paths []string) func(path string) bool {
 		}
 
 		for _, p := range paths {
-			if strings.HasPrefix(path, p) {
+			dirPath := filepath.Dir(path)
+
+			if strings.Contains(p, dirPath) {
 				return true
 			}
 		}
@@ -67,6 +70,8 @@ func commitsBetween(ctx context.Context, repository *git.Repository, first, last
 		lastHash = val
 	}
 
+	log.Debug().Msgf("First hash: %s, Last hash: %s", firstHash, lastHash)
+
 	cIter, err := repository.Log(&git.LogOptions{
 		Order:      git.LogOrderCommitterTime,
 		PathFilter: pathFilter(paths),

internal/gitutils/diff_test.go

@@ -11,6 +11,7 @@ import (
 func TestPathFilter(t *testing.T) {
 	assert.True(t, pathFilter([]string{})("test/test"))
 	assert.True(t, pathFilter([]string{"test"})("test/test"))
+	assert.True(t, pathFilter([]string{"foo/bar/baz/test"})("test/file.txt"))
 	assert.False(t, pathFilter([]string{"foo"})("test/test"))
 }