From 6ce36ab1f38d132ae2c1f860231bd27da19c29db Mon Sep 17 00:00:00 2001 From: beads/crew/wolf Date: Tue, 6 Jan 2026 19:32:20 -0800 Subject: [PATCH] fix(lint): address G304 and unparam warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 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 --- cmd/bd/rename_prefix.go | 1 + cmd/bd/sync_import.go | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/cmd/bd/rename_prefix.go b/cmd/bd/rename_prefix.go index af213448..d6dda7f3 100644 --- a/cmd/bd/rename_prefix.go +++ b/cmd/bd/rename_prefix.go @@ -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 func parseJSONLFile(jsonlPath string) ([]*types.Issue, error) { + // #nosec G304 - jsonlPath is from findJSONLPath() which uses trusted paths f, err := os.Open(jsonlPath) if err != nil { return nil, fmt.Errorf("failed to open JSONL file: %w", err) diff --git a/cmd/bd/sync_import.go b/cmd/bd/sync_import.go index e76690e0..859a7b3d 100644 --- a/cmd/bd/sync_import.go +++ b/cmd/bd/sync_import.go @@ -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. // The parent process's store and dbPath are used, ensuring consistent path resolution. // (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 if store == nil { return fmt.Errorf("no database store available for inline import")