Fix DeepSource issues in monorepo PR

- Fix retry loop skipping base-repo fallback when all path segments exhausted
- Guard against filepath.Rel returning "." which creates invalid repo suffix
- Rename blank receiver `_` to `lo` in PAT login flow
- Extract loadGoldenFile helper to reduce test complexity

Author: jai-deepsource

command/auth/login/pat_login_flow.go

@@ -9,7 +9,7 @@ import (
 	"github.com/fatih/color"
 )
 
-func (_ *LoginOptions) startPATLoginFlow(svc *authsvc.Service, cfg *config.CLIConfig, token string) error {
+func (lo *LoginOptions) startPATLoginFlow(svc *authsvc.Service, cfg *config.CLIConfig, token string) error {
 	cfg.Token = token
 
 	viewer, err := svc.GetViewer(context.Background(), cfg)

command/issues/issues.go

@@ -300,12 +300,16 @@ func (opts *IssuesOptions) resolveIssuesWithRetry(ctx context.Context, client *d
 		return nil, err
 	}
 
+	baseName := strings.SplitN(remote.RepoName, ":", 2)[0]
 	parts := strings.Split(remote.SubRepoSuffix, ":")
-	for len(parts) > 1 {
+	for len(parts) > 0 {
 		parts = parts[:len(parts)-1]
 		remote.SubRepoSuffix = strings.Join(parts, ":")
-		baseName := strings.SplitN(remote.RepoName, ":", 2)[0]
-		remote.RepoName = baseName + ":" + remote.SubRepoSuffix
+		if remote.SubRepoSuffix == "" {
+			remote.RepoName = baseName
+		} else {
+			remote.RepoName = baseName + ":" + remote.SubRepoSuffix
+		}
 
 		issuesList, err = opts.resolveIssues(ctx, client, remote)
 		if err == nil {

command/issues/tests/issues_test.go

@@ -420,30 +420,27 @@ func TestIssuesMultipleFilters(t *testing.T) {
 	}
 }
 
+// loadGoldenFile reads a golden file and fails the test if it cannot be read.
+func loadGoldenFile(t *testing.T, name string) []byte {
+	t.Helper()
+	data, err := os.ReadFile(goldenPath(name))
+	if err != nil {
+		t.Fatalf("failed to read golden file %s: %v", name, err)
+	}
+	return data
+}
+
 // TestIssuesPRFallbackUsesRunIssues verifies that when auto-branch resolution
 // detects a PR AND the current run is in-progress (fallback), the code fetches
 // issues via GetRunIssuesFlat (commit-scoped) instead of GetPRIssues.
 // Regression test for ticket #6884175.
 func TestIssuesPRFallbackUsesRunIssues(t *testing.T) {
 	cfgMgr := testutil.CreateTestConfigManager(t, "test-token", "deepsource.com", "test@example.com")
 
-	// Load golden files for the multi-step mock.
-	prFoundData, err := os.ReadFile(goldenPath("get_pr_by_branch_found_response.json"))
-	if err != nil {
-		t.Fatalf("failed to read PR golden file: %v", err)
-	}
-	runsFirstData, err := os.ReadFile(goldenPath("get_analysis_runs_pr_fallback_first_response.json"))
-	if err != nil {
-		t.Fatalf("failed to read first runs golden file: %v", err)
-	}
-	runsCompletedData, err := os.ReadFile(goldenPath("get_analysis_runs_pr_fallback_completed_response.json"))
-	if err != nil {
-		t.Fatalf("failed to read completed runs golden file: %v", err)
-	}
-	commitScopeData, err := os.ReadFile(goldenPath("commit_scope_response.json"))
-	if err != nil {
-		t.Fatalf("failed to read commit scope golden file: %v", err)
-	}
+	prFoundData := loadGoldenFile(t, "get_pr_by_branch_found_response.json")
+	runsFirstData := loadGoldenFile(t, "get_analysis_runs_pr_fallback_first_response.json")
+	runsCompletedData := loadGoldenFile(t, "get_analysis_runs_pr_fallback_completed_response.json")
+	commitScopeData := loadGoldenFile(t, "commit_scope_response.json")
 
 	mock := graphqlclient.NewMockClient()
 	analysisRunsCalls := 0

internal/vcs/remotes.go

@@ -156,6 +156,10 @@ func detectSubRepoPath() string {
 		return ""
 	}
 
+	if rel == "." {
+		return ""
+	}
+
 	debug.Log("git: sub-repo relative path %q", rel)
 	return strings.ReplaceAll(rel, string(os.PathSeparator), ":")
 }