Fix VS Code formatting inconsistencies (#1133)

Author: matthewbastien

.eslintrc.json

@@ -14,10 +14,6 @@
     "@typescript-eslint/no-non-null-assertion": "off",
     "@typescript-eslint/semi": "error"
   },
-  "extends": [
-      "eslint:recommended",
-      "plugin:@typescript-eslint/recommended",
-      "prettier"
-  ],
+  "extends": ["eslint:recommended", "plugin:@typescript-eslint/recommended", "prettier"],
   "ignorePatterns": ["out", "dist", "**/*.d.ts"]
 }

.prettierignore

@@ -1 +1,17 @@
-.vscode-test
+# Only lint certain file types
+*.*
+!*.js
+!*.json
+!*.ts
+
+# NodeJS
+node_modules/
+
+# vscode-test
+.vscode-test/
+
+# Extension directories that don't need formatting
+/assets/
+/coverage/
+/dist/
+/snippets/

.prettierrc.json

@@ -11,4 +11,4 @@
       }
     }
   ]
-}
+}

.vscode-test.js

@@ -21,7 +21,7 @@ const isFastTestRun = process.env["FAST_TEST_RUN"] === "1";
 const isDebugRun = !(process.env["_"] ?? "").endsWith("node_modules/.bin/vscode-test");
 
 // so tests don't timeout when a breakpoint is hit
-const timeout = isDebugRun ? Number.MAX_SAFE_INTEGER : 3000
+const timeout = isDebugRun ? Number.MAX_SAFE_INTEGER : 3000;
 
 module.exports = defineConfig({
     tests: [
@@ -34,7 +34,7 @@ module.exports = defineConfig({
                 "--disable-updates",
                 "--disable-crash-reporter",
                 "--disable-workspace-trust",
-                "--disable-telemetry"
+                "--disable-telemetry",
             ],
             mocha: {
                 ui: "tdd",
@@ -43,7 +43,7 @@ module.exports = defineConfig({
                 forbidOnly: isCIBuild,
                 grep: isFastTestRun ? "@slow" : undefined,
                 invert: isFastTestRun,
-                slow: 10000
+                slow: 10000,
             },
             installExtensions: ["vadimcn.vscode-lldb"],
             reuseMachineInstall: !isCIBuild,
@@ -57,14 +57,14 @@ module.exports = defineConfig({
                 "--disable-updates",
                 "--disable-crash-reporter",
                 "--disable-workspace-trust",
-                "--disable-telemetry"
+                "--disable-telemetry",
             ],
             mocha: {
                 ui: "tdd",
                 color: true,
                 timeout,
                 forbidOnly: isCIBuild,
-                slow: 100
+                slow: 100,
             },
             reuseMachineInstall: !isCIBuild,
         },

.vscode/settings.json

@@ -10,11 +10,6 @@
     "typescript.tsc.autoDetect": "off",
 
     // Configure Prettier
+    "editor.formatOnSave": true,
     "editor.defaultFormatter": "esbenp.prettier-vscode",
-    "[json]": {
-        "editor.formatOnSave": true
-    },
-    "[typescript]": {
-        "editor.formatOnSave": true
-    }
 }

package.json

@@ -1274,7 +1274,7 @@
     "compile": "tsc",
     "watch": "tsc --watch",
     "lint": "eslint ./ --ext ts && tsc --noEmit",
-    "format": "prettier --check *.json src test",
+    "format": "prettier --check .",
     "pretest": "npm run compile-tests",
     "soundness": "docker compose -f docker/docker-compose.yaml -p swift-vscode-soundness-prb run --rm soundness",
     "test": "vscode-test",