From 44c17012e50a1c46bad1f8f39f4af5266fb99b31 Mon Sep 17 00:00:00 2001 From: beads/crew/emma Date: Sun, 11 Jan 2026 20:44:16 -0800 Subject: [PATCH] fix(doctor): exclude sync_base.jsonl from multiple JSONL files check (#1021) sync_base.jsonl is an internal sync mechanism file used for 3-way merge, not a competing issue database. The doctor check now correctly ignores it. Co-Authored-By: Claude Opus 4.5 --- cmd/bd/doctor/legacy.go | 1 + cmd/bd/doctor/legacy_test.go | 8 +++++++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/cmd/bd/doctor/legacy.go b/cmd/bd/doctor/legacy.go index 77a5f4f2..d1494d82 100644 --- a/cmd/bd/doctor/legacy.go +++ b/cmd/bd/doctor/legacy.go @@ -156,6 +156,7 @@ func CheckLegacyJSONLFilename(repoPath string) DoctorCheck { name == "deletions.jsonl" || name == "interactions.jsonl" || name == "molecules.jsonl" || + name == "sync_base.jsonl" || // Git merge conflict artifacts (e.g., issues.base.jsonl, issues.left.jsonl) strings.Contains(lowerName, ".base.jsonl") || strings.Contains(lowerName, ".left.jsonl") || diff --git a/cmd/bd/doctor/legacy_test.go b/cmd/bd/doctor/legacy_test.go index 9c5fb49d..543f44dc 100644 --- a/cmd/bd/doctor/legacy_test.go +++ b/cmd/bd/doctor/legacy_test.go @@ -290,9 +290,15 @@ func TestCheckLegacyJSONLFilename(t *testing.T) { expectedStatus: "ok", expectWarning: false, }, + { + name: "sync_base.jsonl ignored as system file (GH#1021)", + files: []string{"issues.jsonl", "sync_base.jsonl"}, + expectedStatus: "ok", + expectWarning: false, + }, { name: "all system files ignored together", - files: []string{"issues.jsonl", "deletions.jsonl", "interactions.jsonl", "molecules.jsonl"}, + files: []string{"issues.jsonl", "deletions.jsonl", "interactions.jsonl", "molecules.jsonl", "sync_base.jsonl"}, expectedStatus: "ok", expectWarning: false, },