Fix CRS 4.0.0 rc2 adding a cleanup phase

example/main.go

@@ -3,7 +3,6 @@ package main
 import (
 	"fmt"
 
-	"github.com/corazawaf/coraza-coreruleset"
 	coreruleset "github.com/corazawaf/coraza-coreruleset"
 	"github.com/corazawaf/coraza/v3"
 )

magefile.go

@@ -23,11 +23,18 @@ import (
 func DownloadCRS() error {
 	rulesDir := "rules"
 	rulesDstDir := rulesDir + "/@owasp_crs"
-	if err := os.MkdirAll(rulesDstDir, os.ModePerm); err != nil {
+	testsDir := "tests"
+
+	// Before downloading, we need to remove:
+	// - old rules under rules/@owasp_crs
+	// - all the related tests
+	if err := cleanupOldCRS(rulesDstDir, testsDir); err != nil {
 		return err
 	}
 
-	testsDir := "tests"
+	if err := os.MkdirAll(rulesDstDir, os.ModePerm); err != nil {
+		return err
+	}
 
 	uri := fmt.Sprintf("https://github.com/coreruleset/coreruleset/archive/%s.zip", crsVersion)
 
@@ -158,6 +165,27 @@ func copyFile(f *zip.File, dstPath string) error {
 	return nil
 }
 
+func cleanupOldCRS(rulesDstDir, testsDir string) error {
+	if err := os.RemoveAll(rulesDstDir); err != nil {
+		return err
+	}
+	if err := filepath.WalkDir(testsDir, func(path string, d os.DirEntry, err error) error {
+		if err != nil {
+			return err
+		}
+		if strings.HasPrefix(d.Name(), "REQUEST-") || strings.HasPrefix(d.Name(), "RESPONSE-") {
+			if err := os.RemoveAll(path); err != nil {
+				return err
+			}
+			return filepath.SkipDir
+		}
+		return nil
+	}); err != nil {
+		return err
+	}
+	return nil
+}
+
 func Test() error {
 	return sh.RunV("go", "test", "./...")
 }