Update vbscript-syntax-check.yml

Signed-off-by: LUIZ HAMILTON ROBERTO DA SILVA <[email protected]>

Author: brazilianscriptguy

.github/workflows/vbscript-syntax-check.yml

@@ -6,111 +6,77 @@ on:
     paths:
       - '**/*.vbs'
       - '**/*.hta'
-
   pull_request:
     branches: [main, develop]
     paths:
       - '**/*.vbs'
       - '**/*.hta'
-
   workflow_dispatch:
 
 jobs:
   vbscript-lint:
-    name: VBScript Syntax Validation
-    runs-on: windows-latest
+    name: 🧪 VBScript Syntax Validation (Wine + Ubuntu)
+    runs-on: ubuntu-latest
 
     steps:
-      - name: 📦 Checkout Repository
+      - name: 📦 Checkout Repository (short path to prevent long file issues)
         uses: actions/checkout@v4
         with:
-          path: repo
+          path: src
 
-      - name: 🔍 Locate VBS and HTA Files (with filtering)
-        working-directory: ./repo
-        shell: pwsh
+      - name: 🍷 Install Wine and VBScript Runtime
         run: |
-          Get-ChildItem -Recurse -Include *.vbs,*.hta | ForEach-Object {
-            if ($_.Extension -eq ".hta") {
-              $content = Get-Content $_.FullName -Raw
-              if ($content -match '<script\s+language=["'']?VBScript["'']?' -or $content -match '<script\s+type=["'']?text/vbscript["'']?') {
-                $_.FullName
-              }
-            }
-            elseif ($_.Extension -eq ".vbs") {
-              $_.FullName
-            }
-          } > vbscript-files.txt
-
-      - name: ✅ Run cscript Syntax Validation and Generate SARIF
-        working-directory: ./repo
-        shell: pwsh
+          sudo dpkg --add-architecture i386
+          sudo apt update
+          sudo apt install -y wine64 wine32 cabextract unzip
+          mkdir -p ~/.wine/drive_c/windows/system32
+          echo "[✔️] Wine installed successfully"
+
+      - name: 📂 Find .vbs and .hta Files
+        working-directory: ./src
         run: |
-          $sarifPath = "sarif-output"
-          New-Item -ItemType Directory -Path $sarifPath -Force | Out-Null
-          $sarif = @{
-            version = "2.1.0"
-            runs = @(@{
-              tool = @{
-                driver = @{
-                  name = "cscript.exe VBScript Syntax Checker"
-                  informationUri = "https://learn.microsoft.com/en-us/previous-versions//d1wf56tt(v=vs.85)"
-                  rules = @()
-                }
-              }
-              results = @()
-            })
-          }
-
-          $files = Get-Content vbscript-files.txt | Where-Object { Test-Path $_ }
-
-          foreach ($file in $files) {
-            Write-Host "🔍 Checking: $file"
-
-            $output = cmd /c "cscript.exe //nologo `"$file`"" 2>&1
-            $exitCode = $LASTEXITCODE
+          find . -type f \( -iname "*.vbs" -o -iname "*.hta" \) > vbscript-files.txt
+          cat vbscript-files.txt || echo "No VBS or HTA files found"
 
-            if ($exitCode -ne 0) {
-              Write-Warning "❌ Syntax Error in $file"
-
-              # Attempt to extract line number
-              $lineNum = 1
-              if ($output -match "line (\d+)" -or $output -match "Line:(\d+)") {
-                $lineNum = [int]($matches[1])
-              }
-
-              # Heuristic severity
-              $severity = if ($output -match "Expected|Syntax error|Invalid") {
-                "error"
-              } elseif ($output -match "unterminated|not defined") {
-                "warning"
-              } else {
-                "note"
-              }
-
-              $sarif.runs[0].results += @{
-                ruleId = "vbscript-syntax-error"
-                level = $severity
-                message = @{ text = $output.Trim() }
-                locations = @(@{
-                  physicalLocation = @{
-                    artifactLocation = @{ uri = $file.Replace('\','/') }
-                    region = @{ startLine = $lineNum }
-                  }
-                })
-              }
-            }
-          }
-
-          $sarif | ConvertTo-Json -Depth 10 | Set-Content "$sarifPath/vbscript-results.sarif" -Encoding UTF8
-
-      - name: 📤 Upload SARIF Artifact
+      - name: 🧠 Analyze and Validate VBScript Files
+        working-directory: ./src
+        run: |
+          mkdir -p sarif
+          echo '{"version":"2.1.0","runs":[{"tool":{"driver":{"name":"VBScript Syntax Check","informationUri":"https://learn.microsoft.com/en-us/previous-versions//d1wf56tt(v=vs.85)","rules":[]}},"results":[' > sarif/results.sarif
+
+          while IFS= read -r file; do
+            echo "Checking: $file"
+
+            if [[ "$file" == *.hta ]]; then
+              if ! grep -qi '<script[^>]*language="VBScript"' "$file"; then
+                echo "Skipping non-VBScript HTA: $file"
+                continue
+              fi
+            fi
+
+            if file "$file" | grep -qi text; then
+              wine cscript.exe //nologo "$file" 2> stderr.txt
+              exitCode=$?
+
+              if [ $exitCode -ne 0 ]; then
+                msg=$(<stderr.txt)
+                msg=${msg//\"/\\\"}
+                echo "{\"level\": \"error\", \"message\": {\"text\": \"$msg\"}, \"locations\": [{\"physicalLocation\": {\"artifactLocation\": {\"uri\": \"$file\"}}}]}," >> sarif/results.sarif
+              fi
+            fi
+          done < vbscript-files.txt
+
+          sed -i '$ s/},$/}]/' sarif/results.sarif
+          echo '}]}]' >> sarif/results.sarif
+
+      - name: 📤 Upload SARIF Results
         uses: actions/upload-artifact@v4
         with:
-          name: vbscript-sarif-results
-          path: repo/sarif-output/vbscript-results.sarif
+          name: vbscript-syntax-results
+          path: src/sarif/results.sarif
+          if-no-files-found: warn
 
-      - name: 📡 Publish SARIF to GitHub Code Scanning
+      - name: 🚨 Upload to GitHub Code Scanning (optional)
         uses: github/codeql-action/upload-sarif@v3
         with:
-          sarif_file: repo/sarif-output/vbscript-results.sarif
+          sarif_file: src/sarif/results.sarif