Skip to content

fix: remove --flat flag from all bd list invocations#2701

Closed
pae23 wants to merge 1 commit intosteveyegge:mainfrom
pae23:fix/remove-flat-flag
Closed

fix: remove --flat flag from all bd list invocations#2701
pae23 wants to merge 1 commit intosteveyegge:mainfrom
pae23:fix/remove-flat-flag

Conversation

@pae23
Copy link
Contributor

@pae23 pae23 commented Mar 12, 2026

Summary

  • bd list --flat is no longer a valid flag, breaking gt mq list and gt patrol report
  • Removes --flat from all 11 call sites across the codebase
  • Both gitalb and gitalb_ee_insight refineries reported this as HIGH severity

Changes

  • internal/beads/beads.go: Remove --flat usage and simplify list parsing
  • 10 other files: Remove --flat from bd list invocations

Test plan

  • Verified all --flat references removed
  • Manual: run gt mq list and gt patrol report — should no longer error

🤖 Generated with Claude Code

Co-Authored-By: furiosa paentraygues@gmail.com
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

bd list --json now produces JSON output directly without requiring --flat.
Remove --flat from all direct exec.Command calls, the InjectFlatForListJSON
helper (now a no-op), and the --flat retry logic in beads.run(). This fixes
gt mq list and gt patrol report failures on bd versions where --flat is
no longer supported.

Closes: gt-50i

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the status/needs-triage Inbox — we haven't looked at it yet label Mar 12, 2026
@pae23 pae23 closed this Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/needs-triage Inbox — we haven't looked at it yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant