fix(gitcli): wrap the tree id error and do not retain ls-tree entries

ListPaths wraps a ParseObjectID error like its other returns, and copies
each path so the list does not keep the whole "<mode> <type> <id>\t"
entry alive.

commit: aff5730aa55c596e2e9f2755193b1367ce5b7269
author: brice · 10d ago · 2026-10-11 19:15 UTC
parent: e2ac0a6
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +3 -2

M internal/adapters/gitcli/read.go +3 -2


@@ -303,7 +303,7 @@ func (g *Git) ListPaths(ctx context.Context, repo domain.RepoName, rev domain.Re
303	303		}
304	304		treeID, err := domain.ParseObjectID(obj.info.ID)
305	305		if err != nil {
306		-	return domain.PathList{}, err
	306	+	return domain.PathList{}, fmt.Errorf("paths of %q: %w", repo, err)
307	307		}
308	308		list := domain.PathList{CommitID: obj.commit, TreeID: treeID}
309	309		size := 0

@@ -330,7 +330,8 @@ func (g *Git) ListPaths(ctx context.Context, repo domain.RepoName, rev domain.Re
330	330		return errStop
331	331		}
332	332		size += len(path)
333		-	list.Paths = append(list.Paths, path)
	333	+	// A copy: path would keep the whole "<mode> <type> <id>\t" entry alive.
	334	+	list.Paths = append(list.Paths, strings.Clone(path))
334	335		}
335	336		})
336	337		if err != nil {

 	}
 	treeID, err := domain.ParseObjectID(obj.info.ID)
 	if err != nil {
-		return domain.PathList{}, err
+		return domain.PathList{}, fmt.Errorf("paths of %q: %w", repo, err)
 	}
 	list := domain.PathList{CommitID: obj.commit, TreeID: treeID}
 	size := 0
 				return errStop
 			}
 			size += len(path)
-			list.Paths = append(list.Paths, path)
+			// A copy: path would keep the whole "<mode> <type> <id>\t" entry alive.
+			list.Paths = append(list.Paths, strings.Clone(path))
 		}
 	})
 	if err != nil {