johno/beads
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(bd-68e4): export must specify -o to write to file, not stdout

Browse Source 
When 'bd doctor --fix' detects DB has more issues than JSONL, it runs
'bd export' to sync them. However, 'bd export' without -o flag outputs
to stdout instead of writing to .beads/issues.jsonl, making the fix a
no-op.

Add -o .beads/issues.jsonl --force to the export command to ensure the
JSONL file is actually updated.
This commit is contained in:
matt wilkie
2025-12-24 09:38:13 -07:00
committed by Steve Yegge
parent 61daa59fd0
commit 0f93bef240
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
cmd/bd/doctor/fix/sync.go
View File

@@ -105,7 +105,9 @@ func DBJSONLSync(path string) error {
// Run the appropriate sync command // Run the appropriate sync command
var cmd *exec.Cmd var cmd *exec.Cmd
if syncDirection == "export" { if syncDirection == "export" {
cmd = exec.Command(bdBinary, "export") // #nosec G204 -- bdBinary from validated executable path // Export DB to JSONL file (must specify -o to write to file, not stdout)
jsonlOutputPath := filepath.Join(beadsDir, "issues.jsonl")
cmd = exec.Command(bdBinary, "export", "-o", jsonlOutputPath, "--force") // #nosec G204 -- bdBinary from validated executable path
} else { } else {
cmd = exec.Command(bdBinary, "sync", "--import-only") // #nosec G204 -- bdBinary from validated executable path cmd = exec.Command(bdBinary, "sync", "--import-only") // #nosec G204 -- bdBinary from validated executable path
} }