- Create vcs/util.go with GetAllFilesInPath and BuildLineToPositionMap
- Create vcs/util_test.go with comprehensive tests for both functions
- Remove review.ContentEntry type, replace with vcs.ContentEntry
- Remove review.GiteaClient interface, replace with vcs.FileReader
- Update review/repo_persona.go to use vcs.FileReader
- Update review/repo_persona_test.go to use vcs.ContentEntry
- Update cmd/review-bot/main.go adapter to implement vcs.FileReader
- Add Number and Base fields to vcs.PullRequest
- Add CommitStatus type to vcs/types.go
- Add GetFileContentAtRef to vcs.PRReader interface
- Add GetCommitStatuses to vcs.PRReader interface
- Add DismissReview to vcs.Reviewer interface
- Add stub implementations on gitea.Client for new interface methods

Closes #84, Closes #85, Closes #86

@@ -0,0 +54,4 @@
+// Returns: map[filename]map[newFileLine]diffPosition
+func BuildLineToPositionMap(diff string) map[string]map[int]int {
+	result := make(map[string]map[int]int)
+

@@ -0,0 +85,4 @@
+			strings.HasPrefix(line, "\\") {
+			continue
+		}
+

@@ -0,0 +105,4 @@
+			result[currentFile][newLine] = position
+			newLine++
+		case strings.HasPrefix(line, "-"):
+			// Deletion: occupies a position but has no new-file line number.

@@ -0,0 +192,4 @@
+-	fmt.Println("old")
++	fmt.Println("new")
++	fmt.Println("added")
+ 	return

@@ -0,0 +8,4 @@
+)
+
+// GetAllFilesInPath recursively fetches all file contents under a path using the
+// provided FileReader. Returns a map of filepath -> content for all files found.

@@ -0,0 +66,4 @@
+			currentFile = strings.TrimPrefix(line, "+++ b/")
+			position = 0
+			newLine = 0
+			if result[currentFile] == nil {

@@ -0,0 +19,4 @@
+	}
+
+	for _, entry := range entries {
+		switch entry.Type {

@@ -0,0 +78,4 @@
+			continue
+		}
+
+		// Skip metadata lines that are not part of position counting.

@@ -831,3 +831,15 @@ func (c *Client) ResolveComment(ctx context.Context, owner, repo string, comment
 	}
 	return nil

@@ -0,0 +24,4 @@
+			if err != nil {
+				return nil, fmt.Errorf("get file %q: %w", entry.Path, err)
+			}
+			results[entry.Path] = content

@@ -0,0 +94,4 @@
+
+		// Process diff content lines
+		if strings.HasPrefix(line, "+") {
+			position++

@@ -0,0 +115,4 @@
+
+// parseHunkNewStart extracts the new-file starting line number from a hunk header.
+// Format: @@ -old_start[,old_count] +new_start[,new_count] @@
+func parseHunkNewStart(hunkLine string) int {

@@ -0,0 +1,250 @@
+package vcs_test

@@ -0,0 +9,4 @@
+// GetAllFilesInPath recursively fetches all file contents under a path using the
+// provided FileReader. Returns a map of filepath -> content for all files found.
+// If the path points to an empty directory, returns an empty map.
+func GetAllFilesInPath(ctx context.Context, client FileReader, owner, repo, path string) (map[string]string, error) {

@@ -0,0 +47,4 @@
+func BuildLineToPositionMap(diff string) map[string]map[int]int {
+	result := make(map[string]map[int]int)
+
+	lines := strings.Split(diff, "\n")

@@ -0,0 +21,4 @@
+		switch entry.Type {
+		case "file":
+			content, err := client.GetFileContent(ctx, owner, repo, entry.Path, "")
+			if err != nil {

@@ -0,0 +73,4 @@
+		if strings.HasPrefix(line, "diff --git") {
+			continue
+		}
+

@@ -837,6 +838,9 @@ func (a *giteaClientAdapter) ListContents(ctx context.Context, owner, repo, path
 	return result, nil

@@ -831,3 +831,15 @@ func (c *Client) ResolveComment(ctx context.Context, owner, repo string, comment
 	}
 	return nil
 }
+

@@ -0,0 +43,4 @@
+		entries, err := client.ListContents(ctx, owner, repo, dir)
+		if err != nil {
+			return fmt.Errorf("list contents %q: %w", dir, err)
+		}

@@ -0,0 +63,4 @@
+
+				totalBytes += len(content)
+				if totalBytes > maxTotalBytesInPath {
+					return fmt.Errorf("exceeded max total bytes (%d) in path %q", maxTotalBytesInPath, path)

@@ -0,0 +2,4 @@
+
+import (
+	"context"
+	"strings"

@@ -11,2 +11,4 @@
 	GetPullRequestDiff(ctx context.Context, owner, repo string, number int) (string, error)
 	GetPullRequestFiles(ctx context.Context, owner, repo string, number int) ([]ChangedFile, error)
+	GetFileContentAtRef(ctx context.Context, owner, repo, path, ref string) (string, error)
+	GetCommitStatuses(ctx context.Context, owner, repo, sha string) ([]CommitStatus, error)

@@ -0,0 +31,4 @@
+// maxTotalBytesInPath (total accumulated bytes). The context is checked before each
+// recursive call and file fetch to respect cancellation.
+func GetAllFilesInPath(ctx context.Context, client FileReader, owner, repo, path string) (map[string]string, error) {
+	results := make(map[string]string)

@@ -0,0 +41,4 @@
+		}
+
+		entries, err := client.ListContents(ctx, owner, repo, dir)
+		if err != nil {

@@ -834,0 +837,4 @@
+func (c *Client) DismissReview(ctx context.Context, owner, repo string, number int, reviewID int64, message string) error {
+	return fmt.Errorf("dismiss review %d on %s/%s#%d: %w", reviewID, owner, repo, number, errors.ErrUnsupported)
+}
+

@@ -0,0 +18,4 @@
+//
+// Remove the phase2 build tag once the adapter is complete.
+var (
+	_ vcs.PRReader   = (*gitea.Client)(nil)

@@ -0,0 +40,4 @@
+			return fmt.Errorf("context canceled during traversal: %w", err)
+		}
+
+		entries, err := client.ListContents(ctx, owner, repo, dir)