From 27de61fbd08e00a13b9717c1a041b5c055fe9c54 Mon Sep 17 00:00:00 2001 From: Steve Yegge Date: Sun, 28 Dec 2025 16:34:24 -0800 Subject: [PATCH] chore: Convert scattered TODOs to tracked issues (bd-lrj8) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Created 7 new beads to track TODOs that were previously inline comments: - bd-7l27: Integrate migration detection into bd doctor (5 files) - bd-6x6g: Add multi-repo target repo switching in bd create - bd-ag35: Add daemon RPC endpoints for config and stale check - bd-2dwo: Remove deprecated daemon logger function - bd-0qx5: Implement Jira issue timestamp comparison for sync - bd-7zka: Implement formula features (repeat, for_each, branch, gate, split) - bd-h048: Refactor sync_test to use direct import logic Updated 16 TODO comments to include bead ID references for tracking. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- cmd/bd/create.go | 4 ++-- cmd/bd/daemon_logger.go | 2 +- cmd/bd/jira.go | 2 +- cmd/bd/migrate.go | 2 +- cmd/bd/migrate_hash_ids.go | 2 +- cmd/bd/migrate_issues.go | 2 +- cmd/bd/migrate_sync.go | 2 +- cmd/bd/migrate_tombstones.go | 2 +- cmd/bd/mol_stale.go | 2 +- cmd/bd/sync_test.go | 2 +- internal/formula/types.go | 10 +++++----- 11 files changed, 16 insertions(+), 16 deletions(-) diff --git a/cmd/bd/create.go b/cmd/bd/create.go index 0a6de264..2e0b3894 100644 --- a/cmd/bd/create.go +++ b/cmd/bd/create.go @@ -161,7 +161,7 @@ var createCmd = &cobra.Command{ repoPath = routing.DetermineTargetRepo(routingConfig, userRole, ".") } - // TODO: Switch to target repo for multi-repo support + // TODO(bd-6x6g): Switch to target repo for multi-repo support // For now, we just log the target repo in debug mode if repoPath != "." { debug.Logf("DEBUG: Target repo: %s\n", repoPath) @@ -205,7 +205,7 @@ var createCmd = &cobra.Command{ // Get database prefix from config var dbPrefix string if daemonClient != nil { - // TODO: Add RPC method to get config in daemon mode + // TODO(bd-ag35): Add RPC method to get config in daemon mode // For now, skip validation in daemon mode (needs RPC enhancement) } else { // Direct mode - check config diff --git a/cmd/bd/daemon_logger.go b/cmd/bd/daemon_logger.go index bf085871..17da2207 100644 --- a/cmd/bd/daemon_logger.go +++ b/cmd/bd/daemon_logger.go @@ -128,7 +128,7 @@ func setupDaemonLogger(logPath string, jsonFormat bool, level slog.Level) (*lumb } // setupDaemonLoggerLegacy is the old signature for backward compatibility during migration. -// TODO: Remove this once all callers are updated to use the new signature. +// TODO(bd-2dwo): Remove this once all callers are updated to use the new signature. func setupDaemonLoggerLegacy(logPath string) (*lumberjack.Logger, daemonLogger) { return setupDaemonLogger(logPath, false, slog.LevelInfo) } diff --git a/cmd/bd/jira.go b/cmd/bd/jira.go index 88141549..2571d4a0 100644 --- a/cmd/bd/jira.go +++ b/cmd/bd/jira.go @@ -630,7 +630,7 @@ func detectJiraConflicts(ctx context.Context) ([]JiraConflict, error) { // Check if updated since last sync if issue.UpdatedAt.After(lastSync) { // This is a potential conflict - for now, mark as conflict - // TODO: In a full implementation, we'd fetch the Jira issue and compare timestamps + // TODO(bd-0qx5): In a full implementation, we'd fetch the Jira issue and compare timestamps conflicts = append(conflicts, JiraConflict{ IssueID: issue.ID, LocalUpdated: issue.UpdatedAt, diff --git a/cmd/bd/migrate.go b/cmd/bd/migrate.go index 14d15bbd..5e8ef980 100644 --- a/cmd/bd/migrate.go +++ b/cmd/bd/migrate.go @@ -19,7 +19,7 @@ import ( _ "github.com/ncruces/go-sqlite3/embed" ) -// TODO: Consider integrating into 'bd doctor' migration detection +// TODO(bd-7l27): Consider integrating into 'bd doctor' migration detection var migrateCmd = &cobra.Command{ Use: "migrate", GroupID: "maint", diff --git a/cmd/bd/migrate_hash_ids.go b/cmd/bd/migrate_hash_ids.go index e9139769..f1e743fd 100644 --- a/cmd/bd/migrate_hash_ids.go +++ b/cmd/bd/migrate_hash_ids.go @@ -21,7 +21,7 @@ import ( "github.com/steveyegge/beads/internal/ui" ) -// TODO: Consider integrating into 'bd doctor' migration detection +// TODO(bd-7l27): Consider integrating into 'bd doctor' migration detection var migrateHashIDsCmd = &cobra.Command{ Use: "hash-ids", Short: "Migrate sequential IDs to hash-based IDs (legacy)", diff --git a/cmd/bd/migrate_issues.go b/cmd/bd/migrate_issues.go index eaad3f88..b6c05d17 100644 --- a/cmd/bd/migrate_issues.go +++ b/cmd/bd/migrate_issues.go @@ -12,7 +12,7 @@ import ( "github.com/steveyegge/beads/internal/storage/sqlite" ) -// TODO: Consider integrating into 'bd doctor' migration detection +// TODO(bd-7l27): Consider integrating into 'bd doctor' migration detection var migrateIssuesCmd = &cobra.Command{ Use: "issues", Short: "Move issues between repositories", diff --git a/cmd/bd/migrate_sync.go b/cmd/bd/migrate_sync.go index ba37dfc9..350315cd 100644 --- a/cmd/bd/migrate_sync.go +++ b/cmd/bd/migrate_sync.go @@ -13,7 +13,7 @@ import ( "github.com/steveyegge/beads/internal/syncbranch" ) -// TODO: Consider integrating into 'bd doctor' migration detection +// TODO(bd-7l27): Consider integrating into 'bd doctor' migration detection var migrateSyncCmd = &cobra.Command{ Use: "sync ", Short: "Migrate to sync.branch workflow for multi-clone setups", diff --git a/cmd/bd/migrate_tombstones.go b/cmd/bd/migrate_tombstones.go index cf094bd7..2d1357c4 100644 --- a/cmd/bd/migrate_tombstones.go +++ b/cmd/bd/migrate_tombstones.go @@ -69,7 +69,7 @@ func loadLegacyDeletionsCmd(path string) (map[string]legacyDeletionRecordCmd, [] return records, warnings, nil } -// TODO: Consider integrating into 'bd doctor' migration detection +// TODO(bd-7l27): Consider integrating into 'bd doctor' migration detection var migrateTombstonesCmd = &cobra.Command{ Use: "tombstones", Short: "Convert deletions.jsonl entries to inline tombstones", diff --git a/cmd/bd/mol_stale.go b/cmd/bd/mol_stale.go index 0460f0ee..d510a6fc 100644 --- a/cmd/bd/mol_stale.go +++ b/cmd/bd/mol_stale.go @@ -64,7 +64,7 @@ func runMolStale(cmd *cobra.Command, args []string) { if daemonClient != nil { // For now, stale check requires direct store access - // TODO: Add RPC endpoint for stale check + // TODO(bd-ag35): Add RPC endpoint for stale check fmt.Fprintf(os.Stderr, "Error: mol stale requires direct database access\n") fmt.Fprintf(os.Stderr, "Hint: use --no-daemon flag: bd --no-daemon mol stale\n") os.Exit(1) diff --git a/cmd/bd/sync_test.go b/cmd/bd/sync_test.go index df6ec510..b3f2302e 100644 --- a/cmd/bd/sync_test.go +++ b/cmd/bd/sync_test.go @@ -441,7 +441,7 @@ func TestHasJSONLConflict_MultipleConflicts(t *testing.T) { func TestZFCSkipsExportAfterImport(t *testing.T) { // Skip this test - it calls importFromJSONL which spawns bd import as subprocess, // but os.Executable() returns the test binary during tests, not the bd binary. - // TODO: Refactor to use direct import logic instead of subprocess. + // TODO(bd-h048): Refactor to use direct import logic instead of subprocess. t.Skip("Test requires subprocess spawning which doesn't work in test environment") if testing.Short() { t.Skip("Skipping test that spawns subprocess in short mode") diff --git a/internal/formula/types.go b/internal/formula/types.go index 4b253e69..8c79fcb8 100644 --- a/internal/formula/types.go +++ b/internal/formula/types.go @@ -167,23 +167,23 @@ type Step struct { // Expand references an expansion formula to inline here. // When set, this step is replaced by the expansion's steps. - // TODO(future): Not yet implemented in bd cook. Filed as future work. + // TODO(bd-7zka): Not yet implemented in bd cook. Filed as future work. Expand string `json:"expand,omitempty"` // ExpandVars are variable overrides for the expansion. - // TODO(future): Not yet implemented in bd cook. Filed as future work. + // TODO(bd-7zka): Not yet implemented in bd cook. Filed as future work. ExpandVars map[string]string `json:"expand_vars,omitempty"` // Condition makes this step optional based on a variable. // Format: "{{var}}" (truthy) or "{{var}} == value". - // TODO(future): Not yet implemented in bd cook. Filed as future work. + // TODO(bd-7zka): Not yet implemented in bd cook. Filed as future work. Condition string `json:"condition,omitempty"` // Children are nested steps (for creating epic hierarchies). Children []*Step `json:"children,omitempty"` // Gate defines an async wait condition for this step. - // TODO(future): Not yet implemented in bd cook. Will integrate with bd-udsi gates. + // TODO(bd-7zka): Not yet implemented in bd cook. Will integrate with bd-udsi gates. Gate *Gate `json:"gate,omitempty"` // Loop defines iteration for this step. @@ -207,7 +207,7 @@ type Step struct { } // Gate defines an async wait condition (integrates with bd-udsi). -// TODO(future): Not yet implemented in bd cook. Schema defined for future use. +// TODO(bd-7zka): Not yet implemented in bd cook. Schema defined for future use. type Gate struct { // Type is the condition type: gh:run, gh:pr, timer, human, mail. Type string `json:"type"`