From be8b98c9ebde81ddfde4e5767c253d99ae173614 Mon Sep 17 00:00:00 2001 From: Steve Yegge Date: Sat, 27 Dec 2025 16:02:23 -0800 Subject: [PATCH] refactor: remove unused Beads.Pin/Unpin methods (bd-x0zl) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit These methods called `bd pin` and `bd unpin` which have been removed. Neither method was ever called - gt uses `bd update --status=pinned` instead. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- internal/beads/beads.go | 23 ----------------------- 1 file changed, 23 deletions(-) diff --git a/internal/beads/beads.go b/internal/beads/beads.go index 7cee91bc..cc49f6b1 100644 --- a/internal/beads/beads.go +++ b/internal/beads/beads.go @@ -394,29 +394,6 @@ func (b *Beads) CloseWithReason(reason string, ids ...string) error { return err } -// Pin pins an issue to an agent's hook. -// This sets the pinned boolean field and optionally the assignee. -// If agent is empty, only sets the pinned field. -// NOTE: There's a known issue (gt-o3is) where the pinned field gets overwritten -// by subsequent bd commands due to a bug in beads auto-import logic. -// The handoff bead attachment mechanism works correctly and is the primary -// work assignment mechanism. The pinned field is cosmetic for bd hook visibility. -func (b *Beads) Pin(id string, agent string) error { - args := []string{"--no-daemon", "pin", id} - if agent != "" { - args = append(args, "--for="+agent) - } - _, err := b.run(args...) - return err -} - -// Unpin removes the pinned flag from an issue. -func (b *Beads) Unpin(id string) error { - args := []string{"unpin", id, "--json"} - _, err := b.run(args...) - return err -} - // Release moves an in_progress issue back to open status. // This is used to recover stuck steps when a worker dies mid-task. // It clears the assignee so the step can be claimed by another worker.