Add CI workflow and improve file handling

Introduces GitHub Actions CI workflow and golangci-lint configuration. Refactors file and writer closing in parser.go to use explicit error handling. Updates convert.go to use switch for file extension checks. Removes unnecessary random seed in utils.go.

Author: YoungMayor

.github/workflows/ci.yml

@@ -0,0 +1,32 @@
+name: Go CI
+
+on:
+  push:
+    branches: [ main, develop ]
+  pull_request:
+    branches: [ main, develop ]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v5
+
+    - name: Set up Go
+      uses: actions/setup-go@v6
+      with:
+        go-version: stable
+
+    - name: Install dependencies
+      run: go mod download
+
+    - name: Run tests
+      run: go test -v ./...
+
+    - name: Build
+      run: go build -v ./...
+
+    - name: Run linter
+      uses: golangci/golangci-lint-action@v8
+      with:
+        version: v2.1

.golangci.yml

@@ -0,0 +1,26 @@
+version: "2"
+
+linters:
+  enable:
+    - errcheck
+    - govet
+    - ineffassign
+    - staticcheck
+    - unused
+    - misspell
+    - gocyclo
+    - unconvert
+    - dupl
+
+issues:
+  max-issues-per-linter: 50
+  max-same-issues: 3
+  new: false
+  fix: false
+  uniq-by-line: true
+  whole-files: false
+
+run:
+  timeout: 5m
+  tests: true
+

internal/commands/convert.go

@@ -150,17 +150,18 @@ func NewFromCmd() *cobra.Command {
 			ext := strings.ToLower(filepath.Ext(inputFile))
 			envMap := make(map[string]string)
 
-			if ext == ".json" {
+			switch ext {
+			case ".json":
 				if err := json.Unmarshal([]byte(data), &envMap); err != nil {
 					fmt.Printf("Error parsing JSON: %v\n", err)
 					os.Exit(1)
 				}
-			} else if ext == ".yaml" || ext == ".yml" {
+			case ".yaml", ".yml":
 				if err := yaml.Unmarshal([]byte(data), &envMap); err != nil {
 					fmt.Printf("Error parsing YAML: %v\n", err)
 					os.Exit(1)
 				}
-			} else {
+			default:
 				fmt.Printf("Error: Unsupported file format '%s'. Must be .json, .yaml, or .yml\n", ext)
 				os.Exit(1)
 			}

internal/parser/parser.go

@@ -21,7 +21,12 @@ func ParseEnvFile(filename string) ([]EnvVar, error) {
 	if err != nil {
 		return nil, fmt.Errorf("failed to open file: %w", err)
 	}
-	defer file.Close()
+	defer func(file *os.File) {
+		err := file.Close()
+		if err != nil {
+			return
+		}
+	}(file)
 
 	var envVars []EnvVar
 	scanner := bufio.NewScanner(file)
@@ -69,16 +74,34 @@ func WriteEnvFile(filename string, envVars []EnvVar) error {
 	if err != nil {
 		return fmt.Errorf("failed to create file: %w", err)
 	}
-	defer file.Close()
+	defer func(file *os.File) {
+		err := file.Close()
+		if err != nil {
+			return
+		}
+	}(file)
 
 	writer := bufio.NewWriter(file)
-	defer writer.Flush()
+	defer func(writer *bufio.Writer) {
+		err := writer.Flush()
+		if err != nil {
+			return
+		}
+	}(writer)
 
 	for _, envVar := range envVars {
 		if envVar.Comment != "" {
-			fmt.Fprintln(writer, envVar.Comment)
+			_, err := fmt.Fprintln(writer, envVar.Comment)
+
+			if err != nil {
+				return err
+			}
+		}
+
+		_, err := fmt.Fprintf(writer, "%s=%s\n", envVar.Key, envVar.Value)
+		if err != nil {
+			return err
 		}
-		fmt.Fprintf(writer, "%s=%s\n", envVar.Key, envVar.Value)
 	}
 
 	return nil

internal/utils/utils.go

@@ -30,7 +30,6 @@ func ConfirmWithPin(message string) (bool, error) {
 
 // generatePin generates a random 6-digit PIN
 func generatePin() string {
-	rand.Seed(time.Now().UnixNano())
 	return fmt.Sprintf("%06d", rand.Intn(1000000))
 }