optimize code: #218 (comment)

package.json

@@ -2,7 +2,7 @@
   "name": "php-cs-fixer",
   "displayName": "php cs fixer",
   "description": "PHP CS Fixer extension for VS Code, php formatter, php code beautify tool, format html",
-  "version": "0.3.18",
+  "version": "0.3.19",
   "publisher": "junstyle",
   "author": "junstyle",
   "license": "ISC",
@@ -168,6 +168,7 @@
   },
   "devDependencies": {
     "@types/node": "^16.11.43",
-    "@types/vscode": "1.56.0"
+    "@types/vscode": "1.56.0",
+    "esbuild": "^0.21.4"
   }
 }

src/index.ts

@@ -152,14 +152,7 @@ class PHPCSFixer extends PHPCSFixerConfig {
       }
     }
     if (!useConfig && this.rules) {
-      let rules = this.rules as string
-      if (process.platform === 'win32') {
-        rules = '"' + rules.replace(/"/g, "\\\"") + '"'
-      } else {
-        rules = rules.replace(/\s+/g, '')
-        statusInfo("rules can't contain whitespaces on linux os!")
-      }
-      args.push('--rules=' + rules)
+      args.push('--rules="' + (this.rules as string).replace(/"/g, "\\\"") + '"')
     }
     if (this.allowRisky) {
       args.push('--allow-risky=yes')

src/runAsync.ts

@@ -2,13 +2,11 @@ import { spawn, SpawnOptionsWithoutStdio } from 'child_process';
 import { output } from './output';
 
 export function runAsync(command: string, args: string[], options: SpawnOptionsWithoutStdio, onData: (data: Buffer) => void = null) {
-  const cpOptions = Object.assign({}, options, { shell: process.platform == 'win32', })
+  const cpOptions = Object.assign({}, options, { shell: true })
   let cp;
   try {
-    if (process.platform == 'win32') {
-      if (command.includes(" ") && command[0] != '"') {
-        command = '"' + command + '"'
-      }
+    if (command.includes(" ") && command[0] != '"') {
+      command = '"' + command + '"'
     }
 
     output('runAsync: spawn ' + command);