dprint codebase

.c8rc.json

@@ -4,4 +4,4 @@
     "include": ["src/**", "built/local/**"],
     "exclude": ["**/node_modules/**"],
     "mergeAsync": true
-}
+}

.devcontainer/devcontainer.json

@@ -11,9 +11,9 @@
         "terminal.integrated.profiles.linux": {
             "bash": {
                 "path": "/bin/bash",
-                "icon": "terminal-bash",
-            },
-        },
+                "icon": "terminal-bash"
+            }
+        }
     },
     "extensions": [
         "dbaeumer.vscode-eslint"

.eslintplugin.js

@@ -3,10 +3,10 @@ const path = require("path");
 
 const rulesDir = path.join(__dirname, "scripts", "eslint", "rules");
 const ext = ".cjs";
-const ruleFiles = fs.readdirSync(rulesDir).filter((p) => p.endsWith(ext));
+const ruleFiles = fs.readdirSync(rulesDir).filter(p => p.endsWith(ext));
 
 module.exports = {
-    rules: Object.fromEntries(ruleFiles.map((p) => {
+    rules: Object.fromEntries(ruleFiles.map(p => {
         return [p.slice(0, -ext.length), require(path.join(rulesDir, p))];
     })),
-}
+};

.eslintrc.json

@@ -11,7 +11,11 @@
         "es6": true
     },
     "plugins": [
-        "@typescript-eslint", "no-null", "import", "eslint-plugin-local", "simple-import-sort"
+        "@typescript-eslint",
+        "no-null",
+        "import",
+        "eslint-plugin-local",
+        "simple-import-sort"
     ],
     "ignorePatterns": [
         "**/node_modules/**",
@@ -32,9 +36,6 @@
         "@typescript-eslint/array-type": "error",
         "@typescript-eslint/no-array-constructor": "error",
 
-        "brace-style": "off",
-        "@typescript-eslint/brace-style": ["error", "stroustrup", { "allowSingleLine": true }],
-
         "@typescript-eslint/naming-convention": [
             "error",
             { "selector": "typeLike", "format": ["PascalCase"], "filter": { "regex": "^(__String|[A-Za-z]+_[A-Za-z]+)$", "match": false } },
@@ -51,55 +52,33 @@
         "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
         "@typescript-eslint/consistent-type-assertions": ["error", { "assertionStyle": "as" }],
 
-        "max-statements-per-line": ["error", { "max": 1 }],
-
-        "no-duplicate-imports": "off",
         "@typescript-eslint/no-duplicate-imports": "error",
 
         "@typescript-eslint/no-inferrable-types": "error",
         "@typescript-eslint/no-misused-new": "error",
         "@typescript-eslint/no-this-alias": "error",
 
-        "no-unused-expressions": "off",
         "@typescript-eslint/no-unused-expressions": ["error", { "allowTernary": true }],
 
         "@typescript-eslint/prefer-for-of": "error",
         "@typescript-eslint/prefer-function-type": "error",
         "@typescript-eslint/prefer-namespace-keyword": "error",
         "@typescript-eslint/prefer-as-const": "error",
 
-        "quotes": "off",
-        "@typescript-eslint/quotes": ["error", "double", { "avoidEscape": true, "allowTemplateLiterals": true }],
-
-        "semi": "off",
-        "@typescript-eslint/semi": "error",
-        "@typescript-eslint/no-extra-semi": "error",
-
-        "space-before-function-paren": "off",
-        "@typescript-eslint/space-before-function-paren": ["error", {
-            "asyncArrow": "always",
-            "anonymous": "always",
-            "named": "never"
-        }],
-
         "@typescript-eslint/triple-slash-reference": "error",
-        "@typescript-eslint/type-annotation-spacing": "error",
         "@typescript-eslint/unified-signatures": "error",
 
         "@typescript-eslint/no-extra-non-null-assertion": "error",
 
         // scripts/eslint/rules
-        "local/object-literal-surrounding-space": "error",
-        "local/no-type-assertion-whitespace": "error",
-        "local/type-operator-spacing": "error",
+        // TODO(jakebailey): no-double-space covers formatting of code that dprint handles, but also reads the text in comments.
+        "local/no-double-space": "error",
         "local/only-arrow-functions": ["error", {
-            "allowNamedFunctions": true ,
+            "allowNamedFunctions": true,
             "allowDeclarations": true
         }],
-        "local/no-double-space": "error",
         "local/argument-trivia": "error",
         "local/no-in-operator": "error",
-        "local/simple-indent": "error",
         "local/debug-assert": "error",
         "local/no-keywords": "error",
         "local/jsdoc-format": "error",
@@ -112,11 +91,9 @@
 
         // eslint
         "constructor-super": "error",
-        "curly": ["error", "multi-line"],
         "dot-notation": "error",
         "eqeqeq": "error",
         "linebreak-style": ["error", "windows"],
-        "new-parens": "error",
         "no-caller": "error",
         "no-duplicate-case": "error",
         "no-empty": "error",
@@ -126,18 +103,10 @@
         "no-new-func": "error",
         "no-new-wrappers": "error",
         "no-return-await": "error",
-        "no-restricted-globals": ["error",
-            { "name": "setTimeout" },
-            { "name": "clearTimeout" },
-            { "name": "setInterval" },
-            { "name": "clearInterval" },
-            { "name": "setImmediate" },
-            { "name": "clearImmediate" }
-        ],
+        "no-restricted-globals": ["error", { "name": "setTimeout" }, { "name": "clearTimeout" }, { "name": "setInterval" }, { "name": "clearInterval" }, { "name": "setImmediate" }, { "name": "clearImmediate" }],
         "no-sparse-arrays": "error",
         "no-template-curly-in-string": "error",
         "no-throw-literal": "error",
-        "no-trailing-spaces": "error",
         "no-undef-init": "error",
         "no-unsafe-finally": "error",
         "no-unused-labels": "error",
@@ -146,7 +115,6 @@
         "prefer-const": "error",
         "prefer-object-spread": "error",
         "quote-props": ["error", "consistent-as-needed"],
-        "space-in-parens": "error",
         "unicode-bom": ["error", "never"],
         "use-isnan": "error",
         "no-prototype-builtins": "error",
@@ -156,7 +124,7 @@
     "overrides": [
         // By default, the ESLint CLI only looks at .js files. But, it will also look at
         // any files which are referenced in an override config. Most users of typescript-eslint
-        // get this behavior by default by extending a recommended typescript-eslint config, which 
+        // get this behavior by default by extending a recommended typescript-eslint config, which
         // just so happens to override some core ESLint rules. We don't extend from any config, so
         // explicitly reference TS files here so the CLI picks them up.
         //
@@ -169,13 +137,7 @@
             "files": ["*.mjs", "*.mts"],
             "rules": {
                 // These globals don't exist outside of CJS files.
-                "no-restricted-globals": ["error",
-                    { "name": "__filename" },
-                    { "name": "__dirname" },
-                    { "name": "require" },
-                    { "name": "module" },
-                    { "name": "exports" }
-                ]
+                "no-restricted-globals": ["error", { "name": "__filename" }, { "name": "__dirname" }, { "name": "require" }, { "name": "module" }, { "name": "exports" }]
             }
         },
         {

.github/workflow-resources/.lsifrc.json

@@ -1,6 +1,6 @@
 {
-	"project": "../../src/tsconfig.json",
-	"source": "../../package.json",
-	"package": "../../package.json",
-	"out": "../../typescript.lsif"
-}
+    "project": "../../src/tsconfig.json",
+    "source": "../../package.json",
+    "package": "../../package.json",
+    "out": "../../typescript.lsif"
+}

.vscode/extensions.json

@@ -1,10 +1,11 @@
 {
     "recommendations": [
         "dbaeumer.vscode-eslint",
-        "rbuckton.tsserver-live-reload"
+        "rbuckton.tsserver-live-reload",
+        "dprint.dprint"
     ],
 
     "unwantedRecommendations": [
         "ms-vscode.vscode-typescript-tslint-plugin"
     ]
-}
+}

.vscode/launch.template.json

@@ -1,4 +1,4 @@
-/* 
+/*
 
   Copy this file into '.vscode/launch.json' or merge its
   contents into your existing configurations.
@@ -13,52 +13,52 @@
 */
 
 {
-  // Use IntelliSense to learn about possible attributes.
-  // Hover to view descriptions of existing attributes.
-  // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
-  "version": "0.2.0",
-  "configurations": [
-    {
-      "type": "node",
-      "request": "launch",
-      "name": "Mocha Tests (currently opened test)",
-      "runtimeArgs": ["--nolazy"],
-      "program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
-      "args": [
-        "-u",
-        "bdd",
-        "--no-timeouts",
-        "--colors",
-        "built/local/run.js",
-        "-f",
-        // You can change this to be the name of a specific test file (without the file extension)
-        // to consistently launch the same test
-        "${fileBasenameNoExtension}",
-        "--skip-percent",
-        "0"
-      ],
-      "env": {
-        "NODE_ENV": "testing"
-      },
-      "outFiles": [
-        "${workspaceFolder}/built/**/*.js",
-        "${workspaceFolder}/built/**/*.mjs",
-        "${workspaceFolder}/built/**/*.cjs",
-        "!**/node_modules/**"
-      ],
-      "sourceMaps": true,
-      "smartStep": true,
-      "preLaunchTask": "npm: build:tests",
-      "console": "integratedTerminal",
-      "customDescriptionGenerator": "'__tsDebuggerDisplay' in this ? this.__tsDebuggerDisplay(defaultValue) : defaultValue"
-    },
-    {
-      // See: https://github.com/microsoft/TypeScript/wiki/Debugging-Language-Service-in-VS-Code
-      "type": "node",
-      "request": "attach",
-      "name": "Attach to VS Code TS Server via Port",
-      "processId": "${command:PickProcess}",
-      "customDescriptionGenerator": "'__tsDebuggerDisplay' in this ? this.__tsDebuggerDisplay(defaultValue) : defaultValue"
-    }
-  ]
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "type": "node",
+            "request": "launch",
+            "name": "Mocha Tests (currently opened test)",
+            "runtimeArgs": ["--nolazy"],
+            "program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
+            "args": [
+                "-u",
+                "bdd",
+                "--no-timeouts",
+                "--colors",
+                "built/local/run.js",
+                "-f",
+                // You can change this to be the name of a specific test file (without the file extension)
+                // to consistently launch the same test
+                "${fileBasenameNoExtension}",
+                "--skip-percent",
+                "0"
+            ],
+            "env": {
+                "NODE_ENV": "testing"
+            },
+            "outFiles": [
+                "${workspaceFolder}/built/**/*.js",
+                "${workspaceFolder}/built/**/*.mjs",
+                "${workspaceFolder}/built/**/*.cjs",
+                "!**/node_modules/**"
+            ],
+            "sourceMaps": true,
+            "smartStep": true,
+            "preLaunchTask": "npm: build:tests",
+            "console": "integratedTerminal",
+            "customDescriptionGenerator": "'__tsDebuggerDisplay' in this ? this.__tsDebuggerDisplay(defaultValue) : defaultValue"
+        },
+        {
+            // See: https://github.com/microsoft/TypeScript/wiki/Debugging-Language-Service-in-VS-Code
+            "type": "node",
+            "request": "attach",
+            "name": "Attach to VS Code TS Server via Port",
+            "processId": "${command:PickProcess}",
+            "customDescriptionGenerator": "'__tsDebuggerDisplay' in this ? this.__tsDebuggerDisplay(defaultValue) : defaultValue"
+        }
+    ]
 }

.vscode/settings.template.json

@@ -4,6 +4,11 @@
     // To use the locally built compiler, after 'npm run build':
     // "typescript.tsdk": "built/local"
 
+    "[typescript][javascript][json][jsonc]": {
+        // "editor.formatOnSave": true,
+        "editor.defaultFormatter": "dprint.dprint"
+    },
+
     // To ignore commits listed in .git-blame-ignore-revs in GitLens:
     "gitlens.advanced.blame.customArguments": [
         "--ignore-revs-file",

.vscode/tasks.json

@@ -49,6 +49,6 @@
             "problemMatcher": [
                 "$tsc"
             ]
-        },
+        }
     ]
 }