fix: bd doctor no longer warns about deletions.jsonl

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 <noreply@anthropic.com>
This commit is contained in:
Steve Yegge
2025-11-26 17:48:16 -08:00
parent 30305da5fb
commit c3b4243450
2 changed files with 9 additions and 2 deletions

View File

@@ -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
}

View File

@@ -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 {