fix: resolve CI/CD workflow issues on Windows

Author: Test User

.github/workflows/build.yml

@@ -47,6 +47,7 @@ jobs:
           go-version: '1.21'
 
       - name: Build binary
+        shell: bash
         env:
           GOOS: ${{ matrix.goos }}
           GOARCH: ${{ matrix.goarch }}
@@ -62,6 +63,7 @@ jobs:
           go build -ldflags "-X main.version=$VERSION -X main.commit=$COMMIT -X main.date=$BUILD_TIME -s -w" -trimpath -o bin/$BINARY_NAME ./cmd/doplan
 
       - name: Verify binary exists
+        shell: bash
         run: |
           if [ "${{ matrix.goos }}" = "windows" ]; then
             if [ ! -f "bin/doplan.exe" ]; then
@@ -80,6 +82,7 @@ jobs:
           fi
 
       - name: Test binary (native architecture only)
+        shell: bash
         # Only test if building for native architecture (can execute on runner)
         # Cross-compiled binaries (arm64 on amd64 runners) cannot be executed
         # Strategy: Test amd64 builds which can run on all amd64 runners
@@ -94,7 +97,9 @@ jobs:
             ./bin/doplan --version
             echo "✅ Binary test passed for ${{ matrix.goos }}/${{ matrix.goarch }}"
           fi
+
       - name: Verify cross-compiled binary
+        shell: bash
         # For cross-compiled builds (arm64 on amd64 runners), verify binary exists but skip execution
         if: matrix.goarch == 'arm64'
         run: |

.github/workflows/test.yml

@@ -49,17 +49,53 @@ jobs:
       - name: Download dependencies
         run: go mod download
 
-      - name: Clean coverage files
+      - name: Clean coverage files and artifacts
         shell: bash
         run: |
           # Remove any existing coverage files that might interfere with test discovery
           # Go test can fail if coverage files are present and cause package discovery issues
-          rm -f coverage.out coverage.html coverage_*.out 2>/dev/null || true
-          # Remove any .out files in root directory
-          find . -maxdepth 1 -name "*.out" -type f ! -name "coverage.out" -delete 2>/dev/null || true
-          
+          echo "Cleaning up coverage files and artifacts..."
+          # Remove all .out files recursively (except we'll create coverage.out later)
+          find . -name "*.out" -type f -delete 2>/dev/null || true
+          # Remove any .out directories
+          find . -name ".out" -type d -exec rm -rf {} + 2>/dev/null || true
+          # Remove any .out files in root
+          rm -f .out coverage.out coverage.html coverage_*.out 2>/dev/null || true
+          # Verify cleanup
+          echo "Verifying cleanup..."
+          if find . -maxdepth 1 -name "*.out" -type f 2>/dev/null | grep -q .; then
+            echo "Warning: Some .out files still exist"
+            find . -maxdepth 1 -name "*.out" -type f 2>/dev/null
+          else
+            echo "✅ Cleanup complete - no .out files found"
+          fi
+
       - name: Run tests
-        run: go test -v -race -coverprofile=coverage.out ./...
+        shell: bash
+        run: |
+          # Run tests - cleanup should have removed any problematic .out files
+          echo "Running tests..."
+          # Use go test directly - cleanup step should have removed any .out files
+          go test -v -race -coverprofile=coverage.out ./... 2>&1 || {
+            EXIT_CODE=$?
+            echo "Tests failed with exit code $EXIT_CODE"
+            # Check if the error is related to .out package
+            if go list ./... 2>&1 | grep -q "\.out"; then
+              echo "Error: .out package detected. Attempting to clean and retry..."
+              find . -name "*.out" -type f -delete 2>/dev/null || true
+              find . -name ".out" -type d -exec rm -rf {} + 2>/dev/null || true
+              # Retry with explicit package list
+              PACKAGES=$(go list ./... 2>&1 | grep -v "\.out" | grep -v "no required module" || true)
+              if [ -n "$PACKAGES" ]; then
+                echo "Retrying with filtered packages..."
+                go test -v -race -coverprofile=coverage.out $PACKAGES
+              else
+                exit $EXIT_CODE
+              fi
+            else
+              exit $EXIT_CODE
+            fi
+          }
 
       - name: Upload coverage reports
         uses: codecov/codecov-action@v4

internal/statistics/storage_test.go

@@ -18,17 +18,17 @@ func TestNewStorage(t *testing.T) {
 
 	assert.NotNil(t, storage)
 	assert.Equal(t, projectRoot, storage.projectRoot)
-	
+
 	// Verify path components (handle Windows path separators)
 	// Normalize path separators for comparison
 	normalizedPath := strings.ReplaceAll(storage.storagePath, "\\", "/")
 	assert.Contains(t, normalizedPath, ".doplan")
 	assert.Contains(t, normalizedPath, "stats")
 	assert.Contains(t, normalizedPath, "statistics.json")
-	
+
 	// Verify it ends with statistics.json (platform-agnostic)
 	expectedPath := filepath.Join(projectRoot, ".doplan", "stats", "statistics.json")
-	assert.Equal(t, expectedPath, storage.storagePath, 
+	assert.Equal(t, expectedPath, storage.storagePath,
 		"storage path should match expected path (normalized for platform)")
 }