Refactor Windows collector to handle PowerShell execution.

Moved PowerShell script handling from Install() to SendSystemLogs() for improved clarity and task separation. Added execution policy configuration and error logging to handle script execution failures more gracefully.

Author: osmontero

agent/collectors/windows_arm64.go

@@ -153,16 +153,26 @@ foreach ($rawEvent in $recentLogs) {
 `
 
 func (w Windows) Install() error {
-	path := utils.GetMyPath()
-	collectorPath := filepath.Join(path, "collector.ps1")
-	err := os.WriteFile(collectorPath, []byte(PowerShellScript), 0644)
-	return err
+	return nil
 }
 
 func (w Windows) SendSystemLogs() {
 	path := utils.GetMyPath()
 	collectorPath := filepath.Join(path, "collector.ps1")
 
+	err := os.WriteFile(collectorPath, []byte(PowerShellScript), 0644)
+	if err != nil {
+		_ = utils.Logger.ErrorF("error writing powershell script: %v", err)
+		return
+	}
+
+	cmd := exec.Command("Powershell.exe", "-Command", `"Set-ExecutionPolicy -ExecutionPolicy Unrestricted -Scope CurrentUser -Force"`)
+	err = cmd.Run()
+	if err != nil {
+		_ = utils.Logger.ErrorF("error setting powershell execution policy: %v", err)
+		return
+	}
+
 	for {
 		select {
 		case <-time.After(30 * time.Second):
@@ -182,7 +192,7 @@ func (w Windows) SendSystemLogs() {
 				for _, logLine := range logLines {
 					validatedLog, _, err := validations.ValidateString(logLine, false)
 					if err != nil {
-						utils.Logger.LogF(100, "error validating log: %s: %v", logLine, err)
+						_ = utils.Logger.LogF(100, "error validating log: %s: %v", logLine, err)
 						continue
 					}