40b07045c7
Extract shared JSONL file discovery logic to internal/utils/path.go. Both autoimport and beads packages now use this shared implementation. Changes: - Add utils.FindJSONLInDir with common logic - Update autoimport.go to use utils.FindJSONLInDir - Update beads.go to delegate to utils.FindJSONLInDir - Update server_export_import_auto.go to use utils.FindJSONLInDir - Move FindJSONLInDir test to utils/path_test.go - Fix pre-existing duplicate countIssuesInJSONLFile in init.go 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
85 lines
2.5 KiB
Go
85 lines
2.5 KiB
Go
// Package utils provides utility functions for issue ID parsing and path handling.
|
|
package utils
|
|
|
|
import (
|
|
"path/filepath"
|
|
)
|
|
|
|
// FindJSONLInDir finds the JSONL file in the given .beads directory.
|
|
// It prefers issues.jsonl over other .jsonl files to prevent accidentally
|
|
// reading/writing to deletions.jsonl or merge artifacts (bd-tqo fix).
|
|
// Always returns a path (defaults to issues.jsonl if nothing suitable found).
|
|
//
|
|
// Search order:
|
|
// 1. issues.jsonl (canonical name)
|
|
// 2. beads.jsonl (legacy support)
|
|
// 3. Any other .jsonl file except deletions/merge artifacts
|
|
// 4. Default to issues.jsonl
|
|
func FindJSONLInDir(dbDir string) string {
|
|
pattern := filepath.Join(dbDir, "*.jsonl")
|
|
matches, err := filepath.Glob(pattern)
|
|
if err != nil || len(matches) == 0 {
|
|
// Default to issues.jsonl if glob fails or no matches
|
|
return filepath.Join(dbDir, "issues.jsonl")
|
|
}
|
|
|
|
// Prefer issues.jsonl over other .jsonl files (bd-tqo fix)
|
|
// This prevents accidentally using deletions.jsonl or merge artifacts
|
|
for _, match := range matches {
|
|
if filepath.Base(match) == "issues.jsonl" {
|
|
return match
|
|
}
|
|
}
|
|
|
|
// Fall back to beads.jsonl for legacy support
|
|
for _, match := range matches {
|
|
if filepath.Base(match) == "beads.jsonl" {
|
|
return match
|
|
}
|
|
}
|
|
|
|
// Last resort: use first match (but skip deletions.jsonl and merge artifacts)
|
|
for _, match := range matches {
|
|
base := filepath.Base(match)
|
|
// Skip deletions manifest and merge artifacts
|
|
if base == "deletions.jsonl" ||
|
|
base == "beads.base.jsonl" ||
|
|
base == "beads.left.jsonl" ||
|
|
base == "beads.right.jsonl" {
|
|
continue
|
|
}
|
|
return match
|
|
}
|
|
|
|
// If only deletions/merge files exist, default to issues.jsonl
|
|
return filepath.Join(dbDir, "issues.jsonl")
|
|
}
|
|
|
|
// CanonicalizePath converts a path to its canonical form by:
|
|
// 1. Converting to absolute path
|
|
// 2. Resolving symlinks
|
|
//
|
|
// If either step fails, it falls back to the best available form:
|
|
// - If symlink resolution fails, returns absolute path
|
|
// - If absolute path conversion fails, returns original path
|
|
//
|
|
// This function is used to ensure consistent path handling across the codebase,
|
|
// particularly for BEADS_DIR environment variable processing.
|
|
func CanonicalizePath(path string) string {
|
|
// Try to get absolute path
|
|
absPath, err := filepath.Abs(path)
|
|
if err != nil {
|
|
// If we can't get absolute path, return original
|
|
return path
|
|
}
|
|
|
|
// Try to resolve symlinks
|
|
canonical, err := filepath.EvalSymlinks(absPath)
|
|
if err != nil {
|
|
// If we can't resolve symlinks, return absolute path
|
|
return absPath
|
|
}
|
|
|
|
return canonical
|
|
}
|