From c3b42434503b2dcea868fd999cfe9c1e9927f5de Mon Sep 17 00:00:00 2001 From: Steve Yegge Date: Wed, 26 Nov 2025 17:48:16 -0800 Subject: [PATCH] fix: bd doctor no longer warns about deletions.jsonl MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The multiple JSONL files check was incorrectly flagging deletions.jsonl as a problem, even though it is a valid system file for tracking deleted issues. Added deletions.jsonl to the skip list alongside backups and merge artifacts. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- cmd/bd/doctor/legacy.go | 5 +++-- cmd/bd/doctor/legacy_test.go | 6 ++++++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/cmd/bd/doctor/legacy.go b/cmd/bd/doctor/legacy.go index dafec81f..656e10ee 100644 --- a/cmd/bd/doctor/legacy.go +++ b/cmd/bd/doctor/legacy.go @@ -132,13 +132,14 @@ func CheckLegacyJSONLFilename(repoPath string) DoctorCheck { continue } - // Skip merge artifacts and backups + // Skip merge artifacts, backups, and system files lowerName := strings.ToLower(name) if strings.Contains(lowerName, "backup") || strings.Contains(lowerName, ".orig") || strings.Contains(lowerName, ".bak") || strings.Contains(lowerName, "~") || - strings.HasPrefix(lowerName, "backup_") { + strings.HasPrefix(lowerName, "backup_") || + name == "deletions.jsonl" { continue } diff --git a/cmd/bd/doctor/legacy_test.go b/cmd/bd/doctor/legacy_test.go index 6b6d63ab..9469bb57 100644 --- a/cmd/bd/doctor/legacy_test.go +++ b/cmd/bd/doctor/legacy_test.go @@ -225,6 +225,12 @@ func TestCheckLegacyJSONLFilename(t *testing.T) { expectedStatus: "warning", expectWarning: true, }, + { + name: "deletions.jsonl ignored as system file", + files: []string{"beads.jsonl", "deletions.jsonl"}, + expectedStatus: "ok", + expectWarning: false, + }, } for _, tt := range tests {