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

fix(lint): address G304 and unparam warnings

Browse Source 
- Add #nosec G304 comment to parseJSONLFile in rename_prefix.go
- Mark unused noGitHistory param in importFromJSONLInline (mirrors import.go pattern)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
beads/crew/wolf
2026-01-06 19:32:20 -08:00
committed by Steve Yegge
parent 520518c68b
commit 6ce36ab1f3
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
cmd/bd/rename_prefix.go
View File

@@ -572,6 +572,7 @@ func generateRepairHashID(ctx context.Context, conn *sql.Conn, prefix string, is
// parseJSONLFile reads and parses a JSONL file into a slice of issues // parseJSONLFile reads and parses a JSONL file into a slice of issues
func parseJSONLFile(jsonlPath string) ([]*types.Issue, error) { func parseJSONLFile(jsonlPath string) ([]*types.Issue, error) {
// #nosec G304 - jsonlPath is from findJSONLPath() which uses trusted paths
f, err := os.Open(jsonlPath) f, err := os.Open(jsonlPath)
if err != nil { if err != nil {
return nil, fmt.Errorf("failed to open JSONL file: %w", err) return nil, fmt.Errorf("failed to open JSONL file: %w", err)

2
cmd/bd/sync_import.go
View File

@@ -65,7 +65,7 @@ func importFromJSONL(ctx context.Context, jsonlPath string, renameOnImport bool,
// This avoids path resolution issues when running from directories with .beads/redirect. // This avoids path resolution issues when running from directories with .beads/redirect.
// The parent process's store and dbPath are used, ensuring consistent path resolution. // The parent process's store and dbPath are used, ensuring consistent path resolution.
// (bd-ysal fix) // (bd-ysal fix)
func importFromJSONLInline(ctx context.Context, jsonlPath string, renameOnImport bool, noGitHistory bool) error { func importFromJSONLInline(ctx context.Context, jsonlPath string, renameOnImport bool, _ /* noGitHistory */ bool) error {
// Verify we have an active store // Verify we have an active store
if store == nil { if store == nil {
return fmt.Errorf("no database store available for inline import") return fmt.Errorf("no database store available for inline import")