Only check WF fork when needed (#268)

puerco · web-flow · commit 56da18ba9667 · 2025-08-01T22:30:25.000-06:00
Signed-off-by: Adolfo Garcia Veytia (puerco) &lt;puerco@carabiner.dev&gt;
diff --git a/pkg/sourcetool/backends/vcs/github/manage.go b/pkg/sourcetool/backends/vcs/github/manage.go
@@ -96,8 +96,10 @@ func (b *Backend) CreateWorkflowPR(r *models.Repository, branches []*models.Bran
 	}
 
 	// If user does not have push access, use a fork
-	if err := b.CheckWorkflowFork(r); err != nil {
-		return nil, fmt.Errorf("checking for required repository fork: %w", err)
+	if !hasPush {
+		if err := b.CheckWorkflowFork(r); err != nil {
+			return nil, fmt.Errorf("checking for required repository fork: %w", err)
+		}
 	}
 
 	// Create a PR manager

Original file line number	Diff line number	Diff line change
`@@ -96,8 +96,10 @@ func (b Backend) CreateWorkflowPR(r models.Repository, branches []*models.Bran`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`// If user does not have push access, use a fork`
`99`		`- if err := b.CheckWorkflowFork(r); err != nil {`
`100`		`- return nil, fmt.Errorf("checking for required repository fork: %w", err)`
	`99`	`+ if !hasPush {`
	`100`	`+ if err := b.CheckWorkflowFork(r); err != nil {`
	`101`	`+ return nil, fmt.Errorf("checking for required repository fork: %w", err)`
	`102`	`+ }`
`101`	`103`	`}`
`102`	`104`
`103`	`105`	`// Create a PR manager`