fix: address CI security and lint warnings

- Change directory permissions from 0755 to 0750
- Rename unused request parameters to _

Author: admSla99

internal/cmd/issue/attachment/download/download.go

@@ -72,7 +72,7 @@ func download(cmd *cobra.Command, args []string) {
 	}
 
 	// Create output directory
-	if err := os.MkdirAll(outputDir, 0755); err != nil {
+	if err := os.MkdirAll(outputDir, 0o750); err != nil {
 		cmdutil.ExitIfError(fmt.Errorf("failed to create directory %s: %w", outputDir, err))
 	}
 

pkg/jira/attachment.go

@@ -72,7 +72,7 @@ func (c *Client) DownloadAttachment(contentURL, targetPath string) error {
 
 	// Ensure directory exists
 	dir := filepath.Dir(targetPath)
-	if err := os.MkdirAll(dir, 0755); err != nil {
+	if err := os.MkdirAll(dir, 0o750); err != nil {
 		return fmt.Errorf("failed to create directory: %w", err)
 	}
 

pkg/jira/attachment_test.go

@@ -57,7 +57,7 @@ func TestGetIssueAttachmentsV2(t *testing.T) {
 }
 
 func TestGetIssueAttachments_NoAttachments(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 		w.Header().Set("Content-Type", "application/json")
 		w.WriteHeader(200)
 		_, _ = w.Write([]byte(`{"key": "TEST-1", "fields": {"attachment": []}}`))
@@ -74,7 +74,7 @@ func TestGetIssueAttachments_NoAttachments(t *testing.T) {
 func TestDownloadAttachment(t *testing.T) {
 	expectedContent := []byte("test file content")
 
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 		w.Header().Set("Content-Type", "application/octet-stream")
 		w.WriteHeader(200)
 		_, _ = w.Write(expectedContent)
@@ -97,7 +97,7 @@ func TestDownloadAttachment(t *testing.T) {
 }
 
 func TestDownloadAttachment_HTTPError(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 		w.WriteHeader(404)
 	}))
 	defer server.Close()