fix: refinery PostMerge uses ForceCloseWithReason for source issue (GH #2321)

The refinery PostMerge and HandleMRInfoSuccess methods used plain
Close/CloseWithReason to close the source task bead after merge.
This failed when the source issue had an attached molecule (wisp)
with open steps, since bd close respects dependency checks.

Fix: use ForceCloseWithReason (matching how gt done handles closures)
to bypass dependency checks, and add a close reason for audit trail.
Also handle the already-closed case gracefully since the polecat gt done
may have already closed the issue.

Additionally fix pre-existing test compilation errors where CreateOptions
used the deprecated Label field instead of Labels.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: steveyegge

internal/refinery/engineer.go

@@ -939,11 +939,19 @@ func (e *Engineer) HandleMRInfoSuccess(mr *MRInfo, result ProcessResult) {
 		}
 	}
 
-	// 1. Close source issue with reference to MR
+	// 1. Close source issue with reference to MR.
+	// Use ForceCloseWithReason to bypass dependency checks — the source issue
+	// may have an attached molecule (wisp) whose open steps would block a
+	// normal close. This matches how gt done handles closures.
 	if mr.SourceIssue != "" {
 		closeReason := fmt.Sprintf("Merged in %s", mr.ID)
-		if err := e.beads.CloseWithReason(closeReason, mr.SourceIssue); err != nil {
-			_, _ = fmt.Fprintf(e.output, "[Engineer] Warning: failed to close source issue %s: %v\n", mr.SourceIssue, err)
+		if err := e.beads.ForceCloseWithReason(closeReason, mr.SourceIssue); err != nil {
+			// Check if already closed (by polecat's gt done) — that's fine
+			if issue, showErr := e.beads.Show(mr.SourceIssue); showErr == nil && beads.IssueStatus(issue.Status).IsTerminal() {
+				_, _ = fmt.Fprintf(e.output, "[Engineer] Source issue already closed: %s\n", mr.SourceIssue)
+			} else {
+				_, _ = fmt.Fprintf(e.output, "[Engineer] Warning: failed to close source issue %s: %v\n", mr.SourceIssue, err)
+			}
 		} else {
 			_, _ = fmt.Fprintf(e.output, "[Engineer] Closed source issue: %s\n", mr.SourceIssue)
 		}

internal/refinery/manager.go

@@ -540,12 +540,21 @@ func (m *Manager) PostMerge(idOrBranch string) (*PostMergeResult, error) {
 		result.MRClosed = true
 	}
 
-	// Close the source issue
+	// Close the source issue with reason and --force to bypass dependency checks.
+	// The source issue may have an attached molecule (wisp) whose open steps
+	// would block a normal bd close. ForceCloseWithReason bypasses this,
+	// matching how gt done handles closures for the no-MR path.
 	if mr.IssueID != "" {
-		if err := b.Close(mr.IssueID); err != nil {
-			// Source issue may already be closed or not exist
-			_, _ = fmt.Fprintf(m.output, "  %s source issue close: %v\n", style.Dim.Render("○"), err)
-			result.SourceIssueNotFound = true
+		closeReason := fmt.Sprintf("Merged in %s", mr.ID)
+		if err := b.ForceCloseWithReason(closeReason, mr.IssueID); err != nil {
+			// Check if already closed (by polecat's gt done) — that's fine
+			if issue, showErr := b.Show(mr.IssueID); showErr == nil && beads.IssueStatus(issue.Status).IsTerminal() {
+				_, _ = fmt.Fprintf(m.output, "  %s source issue already closed: %s\n", style.Dim.Render("○"), mr.IssueID)
+				result.SourceIssueClosed = true
+			} else {
+				_, _ = fmt.Fprintf(m.output, "  %s source issue close: %v\n", style.Dim.Render("○"), err)
+				result.SourceIssueNotFound = true
+			}
 		} else {
 			result.SourceIssueClosed = true
 		}

internal/refinery/manager_test.go

@@ -111,14 +111,14 @@ func TestManager_Queue_FiltersClosedMergeRequests(t *testing.T) {
 
 	openIssue, err := b.Create(beads.CreateOptions{
 		Title: "Open MR",
-		Label: "gt:merge-request",
+		Labels: []string{"gt:merge-request"},
 	})
 	if err != nil {
 		t.Fatalf("create open merge-request issue: %v", err)
 	}
 	closedIssue, err := b.Create(beads.CreateOptions{
 		Title: "Closed MR",
-		Label: "gt:merge-request",
+		Labels: []string{"gt:merge-request"},
 	})
 	if err != nil {
 		t.Fatalf("create closed merge-request issue: %v", err)
@@ -227,7 +227,7 @@ func TestManager_PostMerge_ClosesMRAndSourceIssue(t *testing.T) {
 	// Create a source issue
 	srcIssue, err := b.Create(beads.CreateOptions{
 		Title: "Implement feature X",
-		Label: "gt:task",
+		Labels: []string{"gt:task"},
 	})
 	if err != nil {
 		t.Fatalf("create source issue: %v", err)
@@ -237,7 +237,7 @@ func TestManager_PostMerge_ClosesMRAndSourceIssue(t *testing.T) {
 	mrDesc := "branch: polecat/test/gt-xyz\nsource_issue: " + srcIssue.ID + "\nworker: test\ntarget: main"
 	mrIssue, err := b.Create(beads.CreateOptions{
 		Title:       "MR for feature X",
-		Label:       "gt:merge-request",
+		Labels:      []string{"gt:merge-request"},
 		Description: mrDesc,
 	})
 	if err != nil {
@@ -277,7 +277,7 @@ func TestManager_PostMerge_AlreadyClosedMR(t *testing.T) {
 	// Create and close an MR bead
 	mrIssue, err := b.Create(beads.CreateOptions{
 		Title:       "Already merged MR",
-		Label:       "gt:merge-request",
+		Labels:      []string{"gt:merge-request"},
 		Description: "branch: polecat/old/gt-old\ntarget: main",
 	})
 	if err != nil {