@momsfriendlydevco/eslint-config 2.2.0 → 2.3.0

package/eslint.config.js

@@ -13,8 +13,7 @@ export default [
 		// Generic globals
 		languageOptions: {
 			globals: {
-				console: 'readonly',
-				fetch: 'readonly',
+				// Put custom globals here
 			},
 		},
 	},

package/index.js

@@ -12,15 +12,17 @@ export let JSCommon = {
 	// Generic rules
 	'html-closing-bracket-spacing': ['off'], // Annoying doesn't allow <this-kind-of-thing/>
 	'no-debugger': ['warn'], // Debuggers are fine, just warn
+	'no-extra-boolean-cast': ['off'], // ESlint being a mama bear over boolean casting shorthand (`!!thing`)
 	'no-useless-escape': ['off'], // ESlint frequently gets what should and shouldn't be escaped wrong
 	'no-unused-vars': ['warn'], // Dont make unused vars the end of the world
 
 	// JSDoc
 	'jsdoc/check-alignment': ['off'],  // Disable the JSDoc parser insisting on correct indents
 	'jsdoc/check-tag-names': ['warn', { // Extend JSDoc allowed tags
-		definedTags: ['date', 'fixme', 'note', 'slot', 'url'],
+		definedTags: ['date', 'env', 'fixme', 'note', 'slot', 'url'],
 	}],
 	'jsdoc/check-types': ['off'], // Disable the JSDoc parser being fussy about `@param {String}` vs `@param {string}`
+	'jsdoc/empty-tags': ['off'], // Disable JSDoc being fussy about using stuff like `@override` with any other description parts
 	'jsdoc/no-defaults': ['off'], // Disable the JSDoc parser that complains about optional params as its just silly
 	'jsdoc/no-undefined-types': ['warn', {disableReporting: true}], // Disable the JSDoc parser for "undefined" types which it usually gets wrong. Ask eslint to still mark these as used though
 	'jsdoc/require-returns-description': ['off'], // Disable the JSDoc parse being fussy about return descriptions
@@ -28,7 +30,41 @@ export let JSCommon = {
 };
 
 export default [
-	// eslint-plugin-jsdoc (with custom settings)
+
+	// Generic globals {{{
+	{
+		languageOptions: {
+			globals: {
+				// Implied global functions
+				atob: 'readable',
+				btoa: 'readable',
+				clearInterval: 'readonly',
+				clearTimeout: 'readonly',
+				console: 'readonly',
+				fetch: 'readonly',
+				setInterval: 'readonly',
+				setTimeout: 'readonly',
+
+				// Implied Global singletons
+				console: 'readable',
+
+				// Implied Global Classes
+				Array: 'readable',
+				Blob: 'readable',
+				Buffer: 'readable',
+				File: 'readable',
+				FormData: 'readable',
+				JSON: 'readable',
+				Promise: 'readable',
+				RegExp: 'readable',
+				URL: 'readable',
+				URLSearchParams: 'readable',
+			},
+		},
+	},
+	// }}}
+
+	// eslint-plugin-jsdoc (with custom settings) {{{
 	{
 		...pluginJSDoc.configs['flat/recommended'],
 		settings: {
@@ -44,8 +80,9 @@ export default [
 			},
 		},
 	},
+	// }}}
 
-	// eslint-plugin-unicorn
+	// eslint-plugin-unicorn {{{
 	{
 		...pluginUnicorn.configs['flat/recommended'],
 		rules: {
@@ -79,9 +116,32 @@ export default [
 			'unicorn/switch-case-braces': ['warn', 'avoid'],
 		},
 	},
+	// }}}
 
-	// eslint-plugin-vue
+	// eslint-plugin-vue {{{
 	...pluginVue.configs['flat/recommended'],
+	// }}}
+
+	// Cloudflare workers {{{
+	{
+		files: ['**/workers/**/*.js'],
+
+		// Wrangler ignore rules
+		languageOptions: {
+			globals: {
+				// Implied Global Cloudflare Worker Classes
+				crypto: 'readable',
+				Headers: 'readable',
+				TextEncoder: 'readable',
+				Request: 'readable',
+				Response: 'readable',
+			},
+		},
+		rules: {
+			...JSCommon,
+		},
+	},
+	// }}}
 
 	// .doop backend Files {{{
 	{
@@ -103,20 +163,48 @@ export default [
 	// .mjs / .js regular files {{{
 	{
 		files: ['**/*.js', '**/*.mjs'],
+		languageOptions: {
+			globals: {
+				// Implied Global singletons
+				process: 'readable',
+			},
+		},
 		rules: {
 			...JSCommon,
 		},
 	},
 	// }}}
 
+	// Mocha test files {{{
+	{
+		files: ['**/test/**/*.js'],
+		languageOptions: {
+			globals: {
+				afterAll: 'readonly',
+				after: 'readonly',
+				beforeAll: 'readonly',
+				before: 'readonly',
+				describe: 'readonly',
+				it: 'readonly',
+			},
+		},
+	},
+	// }}}
+
 	// .vue frontend Files {{{
 	{
 		files: ['**/*.vue'],
 		languageOptions: {
 			globals: {
-				$: 'readable', // jQuery
-				_: 'readable', // Lodash
-				app: 'readable', // Global frontend app object
+				// See also: "Generic globals" > languageOptions > globals
+
+				// Not-really-a-thing-but-I'll-allow-it globals
+				app: 'readable', // Generally the Vue.$root frontend app object
+
+				// Implied meta-globals
+				document: 'readable',
+				navigator: 'readable',
+				window: 'readable',
 			},
 			parser: eslintParser,
 			parserOptions: {
@@ -241,4 +329,5 @@ export default [
 		},
 	},
 	// }}}
+
 ]

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@momsfriendlydevco/eslint-config",
   "description": "ESLint plugin for @MomsFriendlyDevCo projects",
-  "version": "2.2.0",
+  "version": "2.3.0",
   "type": "module",
   "scripts": {
     "test": "eslint --config index.js test/data"
@@ -24,10 +24,10 @@
   },
   "homepage": "https://github.com/MomsFriendlyDevCo/eslint#readme",
   "dependencies": {
-    "eslint-plugin-jsdoc": "^51.0.1",
+    "eslint-plugin-jsdoc": "^51.3.4",
     "eslint-plugin-unicorn": "^59.0.1",
-    "eslint-plugin-vue": "^10.2.0",
-    "vue-eslint-parser": "^10.1.3"
+    "eslint-plugin-vue": "^10.3.0",
+    "vue-eslint-parser": "^10.2.0"
   },
   "peerDependencies": {
     "eslint": "^9.26.0"