eslint-plugin-lube 0.1.6 → 0.2.0

package/README.md

@@ -2,4 +2,5 @@
 
 | Rule | Code | Test |
 |---|---|---|
-| svelte-naming-convention | [code](https://github.com/Artxe2/lube-series/blob/master/packages/eslint-plugin-lube/rules/svelte-naming-convention.js) | [test](https://github.com/Artxe2/lube-series/blob/master/packages/eslint-plugin-lube/test/svelte-naming-convention.js) |
+| svelte-naming-convention | [code](https://github.com/Artxe2/lube-series/blob/master/packages/eslint-plugin-lube/rules/svelte-naming-convention.js) | [test](https://github.com/Artxe2/lube-series/blob/master/packages/eslint-plugin-lube/test/svelte-naming-convention) |
+| pretty-imports | [code](https://github.com/Artxe2/lube-series/blob/master/packages/eslint-plugin-lube/rules/pretty-imports.js) | [test](https://github.com/Artxe2/lube-series/blob/master/packages/eslint-plugin-lube/test/pretty-imports) |

package/index.js

@@ -1 +1,53 @@
-module.exports = { rules: { "svelte-naming-convention": require("./rules/svelte-naming-convention") } }
+/**
+ * @type {import("eslint").ESLint.Plugin & {
+ *   configs: Record<string, import("eslint").ESLint.ConfigData|{}>
+ * }}
+ */
+module.exports = {
+	configs: {
+		recommended: {
+			extends: ["eslint:recommended"],
+			plugins: ["lube"],
+			rules: {
+				"arrow-spacing": "error",
+				"comma-spacing": "error",
+				"dot-location": ["error", "property"],
+				"for-direction": "error",
+				"getter-return": "error",
+				"indent": ["error", "tab"],
+				"keyword-spacing": ["error", {
+					"after": true,
+					"before": true
+				}],
+				"lube/svelte-naming-convention": "error",
+				"lube/pretty-imports": "error",
+				"max-len": ["warn", 120],
+				"max-nested-callbacks": ["error", 3],
+				"no-console": "warn",
+				"no-duplicate-imports": "error",
+				"no-empty-pattern": "error",
+				"no-multi-spaces": "error",
+				"object-curly-spacing": ["error", "always"],
+				"prefer-const": "warn",
+				"quotes": ["error", "double"],
+				"semi": ["error", "never"],
+				"semi-spacing": ["error", {
+					"after": true,
+					"before": false
+				}],
+				"space-before-function-paren": ["error", {
+					"anonymous": "never",
+					"asyncArrow": "always",
+					"named": "never"
+				}],
+				"space-in-parens": ["error", "never"],
+				"space-infix-ops": "error",
+				"space-unary-ops": "error"
+			}
+		}
+	},
+	rules: {
+		"pretty-imports": require("./rules/pretty-imports"),
+		"svelte-naming-convention": require("./rules/svelte-naming-convention")
+	}
+}

package/node-type.text

@@ -0,0 +1,71 @@
+/* ArrayExpression */
+/* ArrayPattern */
+/* ArrowFunctionExpression */
+/* AssignmentExpression */
+/* AssignmentPattern */
+/* AwaitExpression */
+/* BinaryExpression */
+/* BlockStatement */
+/* BreakStatement */
+/* CallExpression */
+/* CatchClause */
+/* ChainExpression */
+/* ClassBody */
+/* ClassDeclaration */
+/* ClassExpression */
+/* ConditionalExpression */
+/* ContinueStatement */
+/* DebuggerStatement */
+/* DoWhileStatement */
+/* EmptyStatement */
+/* ExportAllDeclaration */
+/* ExportDefaultDeclaration */
+/* ExportNamedDeclaration */
+/* ExportSpecifier */
+/* ExpressionStatement */
+/* ForInStatement */
+/* ForOfStatement */
+/* ForStatement */
+/* FunctionDeclaration */
+/* FunctionExpression */
+/* Identifier */
+/* IfStatement */
+/* ImportDeclaration */
+/* ImportDefaultSpecifier */
+/* ImportExpression */
+/* ImportNamespaceSpecifier */
+/* ImportSpecifier */
+/* LabeledStatement */
+/* Literal */
+/* LogicalExpression */
+/* MemberExpression */
+/* MetaProperty */
+/* MethodDefinition */
+/* NewExpression */
+/* ObjectExpression */
+/* ObjectPattern */
+/* PrivateIdentifier */
+/* Program */
+/* Property */
+/* PropertyDefinition */
+/* RestElement */
+/* ReturnStatement */
+/* SequenceExpression */
+/* SpreadElement */
+/* StaticBlock */
+/* Super */
+/* SwitchCase */
+/* SwitchStatement */
+/* TaggedTemplateExpression */
+/* TemplateElement */
+/* TemplateLiteral */
+/* ThisExpression */
+/* ThrowStatement */
+/* TryStatement */
+/* UnaryExpression */
+/* UpdateExpression */
+/* VariableDeclaration */
+/* VariableDeclarator */
+/* WhileStatement */
+/* WithStatement */
+/* YieldExpression */

package/package.json

@@ -1,27 +1,26 @@
-{
-	"author": "artxe2",
-	"bugs": {
-		"url": "https://github.com/Artxe2/lube-series/issues"
-	},
-	"dependencies": {
-		"@types/estree": "^1.0.1"
-	},
-	"description": "Eslint rules expand options",
-	"devDependencies": {
-		"@types/eslint": "^8.44.2",
-		"@typescript-eslint/parser": "^6.3.0",
-		"eslint": "^8.47.0",
-		"mocha": "^10.2.0"
-	},
-	"license": "MIT",
-	"name": "eslint-plugin-lube",
-	"repository": {
-		"type": "git",
-		"url": "https://github.com/Artxe2/lube-series"
-	},
-	"scripts": {
-		"test": "mocha"
-	},
-	"type": "commonjs",
-	"version": "0.1.6"
-}
+{
+	"author": "artxe2",
+	"bugs": {
+		"url": "https://github.com/Artxe2/lube-series/issues"
+	},
+	"description": "Eslint rules expand options",
+	"devDependencies": {
+		"@types/eslint": "^8.44.2",
+		"@types/estree": "^1.0.1",
+		"@typescript-eslint/parser": "^6.3.0",
+		"eslint": "^8.47.0",
+		"mocha": "^10.2.0"
+	},
+	"license": "MIT",
+	"name": "eslint-plugin-lube",
+	"repository": {
+		"type": "git",
+		"url": "https://github.com/Artxe2/lube-series",
+		"directory": "packages/eslint-plugin-lube"
+	},
+	"scripts": {
+		"test": "mocha"
+	},
+	"type": "commonjs",
+	"version": "0.2.0"
+}

package/private.d.ts

@@ -1,22 +1,109 @@
-import {
-	Node,
-	Pattern
-} from "estree"
+import * as estree from "estree"
 
-interface ASTNode extends Node, Pattern {
-	computed: boolean
-	id: ASTNode
-	imported: ASTNode
-	key: ASTNode
-	left: ASTNode
-	local: ASTNode
-	name: string
-	object: ASTNode
+export type ASTNode = {
 	parent: ASTNode
-	property: ASTNode
 	range: [number, number]
-	right: ASTNode
-	shorthand: boolean
-	type: string
-	value: ASTNode
-}
+} & (
+	estree.ArrayExpression & {
+		elements: ASTNode[]
+	}
+	| estree.ArrayPattern & {
+		elements: ASTNode[]
+	}
+	| estree.ArrowFunctionExpression & {
+		body: ASTNode
+		params: ASTNode[]
+	}
+	| estree.AssignmentExpression
+	| estree.AssignmentPattern
+	| estree.AwaitExpression
+	| estree.BinaryExpression
+	| estree.BlockStatement
+	| estree.BreakStatement
+	| estree.CallExpression
+	| estree.CatchClause
+	| estree.ChainExpression
+	| estree.ClassBody
+	| estree.ClassDeclaration
+	| estree.ClassExpression
+	| estree.ConditionalExpression
+	| estree.ContinueStatement
+	| estree.DebuggerStatement
+	| estree.DoWhileStatement
+	| estree.EmptyStatement
+	| estree.ExportAllDeclaration
+	| estree.ExportDefaultDeclaration
+	| estree.ExportNamedDeclaration & {
+		source: estree.SimpleLiteral & {
+			range: [number, number]
+			raw: string
+			value: string
+		}
+		specifiers: ASTNode[]
+	}
+	| estree.ExportSpecifier
+	| estree.ExpressionStatement
+	| estree.ForInStatement
+	| estree.ForOfStatement
+	| estree.ForStatement
+	| estree.FunctionDeclaration & {
+		body: ASTNode
+		params: ASTNode[]
+	}
+	| estree.FunctionExpression & {
+		body: ASTNode
+		params: ASTNode[]
+	}
+	| estree.Identifier
+	| estree.IfStatement
+	| estree.ImportDeclaration & {
+		source: estree.SimpleLiteral & {
+			range: [number, number]
+			raw: string
+			value: string
+		}
+		specifiers: ASTNode[]
+	}
+	| estree.ImportDefaultSpecifier
+	| estree.ImportExpression
+	| estree.ImportNamespaceSpecifier
+	| estree.ImportSpecifier
+	| estree.LabeledStatement
+	| estree.Literal
+	| estree.LogicalExpression
+	| estree.MemberExpression
+	| estree.MetaProperty
+	| estree.MethodDefinition
+	| estree.NewExpression
+	| estree.ObjectExpression & {
+		properties: ASTNode[]
+	}
+	| estree.ObjectPattern & {
+		properties: ASTNode[]
+	}
+	| estree.PrivateIdentifier
+	| estree.Program
+	| estree.Property & { key: estree.Identifier }
+	| estree.PropertyDefinition
+	| estree.RestElement
+	| estree.ReturnStatement
+	| estree.SequenceExpression
+	| estree.SpreadElement
+	| estree.StaticBlock
+	| estree.Super
+	| estree.SwitchCase
+	| estree.SwitchStatement
+	| estree.TaggedTemplateExpression
+	| estree.TemplateElement
+	| estree.TemplateLiteral
+	| estree.ThisExpression
+	| estree.ThrowStatement
+	| estree.TryStatement
+	| estree.UnaryExpression
+	| estree.UpdateExpression
+	| estree.VariableDeclaration
+	| estree.VariableDeclarator
+	| estree.WhileStatement
+	| estree.WithStatement
+	| estree.YieldExpression
+)

package/rules/pretty-imports.js

@@ -0,0 +1,110 @@
+"use strict"
+/** @type {import("eslint").Rule.RuleModule} */
+module.exports = {
+	meta: {
+		docs: {
+			description: "Enforces pretty imports.",
+			recommended: true,
+			url: "https://github.com/Artxe2/lube-series/blob/master/packages/eslint-plugin-lube/test/pretty-imports"
+		},
+		fixable: "code",
+		messages: {
+			not_match: "This imports is not pretty"
+		},
+		schema: [{
+			properties: {
+				checkExports: {
+					type: "boolean",
+					default: true
+				},
+				checkImports: {
+					type: "boolean",
+					default: true
+				},
+				indent: {
+					type: "string",
+					default: "\t"
+				},
+				maxLength: {
+					type: "number",
+					default: 30
+				}
+			},
+			type: "object"
+		}],
+		type: "suggestion"
+	},
+	create(context) {
+		const check_exports = /** @type {boolean} */ (context.options[0]?.checkExports) ?? true
+		const check_imports = /** @type {boolean} */ (context.options[0]?.checkImports) ?? true
+		const indent = /** @type {string} */ (context.options[0]?.indent) ?? "\t"
+		const max_length = /** @type {number} */ (context.options[0]?.maxLength) ?? 30
+		const source_code = context.sourceCode
+		const text = source_code.text
+
+		const indent_regex = /[ \t]*(?=[^\n]*$)/
+		/** @param {number} index */
+		function get_indent(index) {
+			return text.slice(0, index).match(indent_regex)?.[0] || ""
+		}
+
+		/**
+		 * @param {import("../private").ASTNode} node
+		 * @param {string} corrected_text
+		 */
+		function report(node, corrected_text) {
+			context.report({
+				fix(fixer) {
+					return fixer.replaceText(node, corrected_text)
+				},
+				messageId: "not_match",
+				node: /** @type {import("estree").Node} */(node)
+			})
+		}
+
+		return {
+			/** @param {import("../private").ASTNode & import("estree").ExportNamedDeclaration} node */
+			ExportNamedDeclaration(node) {
+				if (check_exports && node.specifiers[0]?.type == "ExportSpecifier") {
+					const line_indent = get_indent(node.range[0])
+					let length = 0
+					for (const s of node.specifiers) {
+						length += s.range[1] - s.range[0]
+					}
+					const corrected_text = length > max_length
+						? `export {\n${line_indent + indent}${
+							/** @type {import("../private").ASTNode[]} v */
+							(node.specifiers).map(v => text.slice(v.range[0], v.range[1]))
+								.join(",\n" + line_indent + indent)
+						}\n${line_indent}} from ${text.slice(node.source.range[0], node.range[1])}`
+						: `export { ${
+							/** @type {import("../private").ASTNode[]} v */
+							(node.specifiers).map(v => text.slice(v.range[0], v.range[1])).join(", ")
+						} } from ${text.slice(node.source.range[0], node.range[1])}`
+					if (corrected_text != text.slice(node.range[0], node.range[1])) report(node, corrected_text)
+				}
+			},
+			/** @param {import("../private").ASTNode & import("estree").ImportDeclaration} node */
+			ImportDeclaration(node) {
+				if (check_imports && node.specifiers[0]?.type == "ImportSpecifier") {
+					const line_indent = get_indent(node.range[0])
+					let length = 0
+					for (const s of node.specifiers) {
+						length += s.range[1] - s.range[0]
+					}
+					const corrected_text = length > max_length
+						? `import {\n${line_indent + indent}${
+							/** @type {import("../private").ASTNode[]} v */
+							(node.specifiers).map(v => text.slice(v.range[0], v.range[1]))
+								.join(",\n" + line_indent + indent)
+						}\n${line_indent}} from ${text.slice(node.source.range[0], node.range[1])}`
+						: `import { ${
+							/** @type {import("../private").ASTNode[]} v */
+							(node.specifiers).map(v => text.slice(v.range[0], v.range[1])).join(", ")
+						} } from ${text.slice(node.source.range[0], node.range[1])}`
+					if (corrected_text != text.slice(node.range[0], node.range[1])) report(node, corrected_text)
+				}
+			}
+		}
+	}
+}

package/rules/svelte-naming-convention.js

@@ -15,7 +15,7 @@ module.exports = {
 			properties: {
 				fixSameNames: {
 					type: "boolean",
-					default: false
+					default: true
 				},
 			},
 			type: "object"
@@ -23,13 +23,13 @@ module.exports = {
 		type: "suggestion"
 	},
 	create(context) {
-		const fix_same_names = /** @type {boolean} */ (context.options[0]?.fixSameNames)
+		const fix_same_names = /** @type {boolean} */ (context.options[0]?.fixSameNames) ?? true
 
 		// let name | const name | class name
-		const allow_regex = /^[_$]?[_$]?(?:[0-9a-z]+(?:_[0-9a-z]+)*\$?\$?)?$|^[A-Z][0-9A-Z_]*$|^(?:[0-9A-Z][0-9a-z]*)+$/
-		const fixable_name_regex = /^[_$]?[_$]?(?:[0-9A-Za-z]+(?:_[0-9A-Za-z]+)*\$?\$?)?$/
-		const fix_regex = /([0-9a-z]?)([A-Z][0-9A-Z]*)/g
-		const camel_case_regex = /^[0-9a-z]+([A-Z][0-9a-z]*)*$/
+		const allow_regex = /^[_$]?[_$]?(?:[\da-z]+(?:_[\da-z]+)*\$?\$?)?$|^[A-Z][\dA-Z_]*$|^(?:[\dA-Z][\da-z]*)+$/
+		const fixable_name_regex = /^[_$]?[_$]?(?:[\dA-Za-z]+(?:_[\dA-Za-z]+)*\$?\$?)?$/
+		const fix_regex = /([\da-z]?)([A-Z][\dA-Z]*)/g
+		const camel_case_regex = /^[\da-z]+([A-Z][\da-z]*)*$/
 
 		/**
 		 * @param {string} _
@@ -47,7 +47,7 @@ module.exports = {
 		/** @type {Set<import("../private").ASTNode>} */
 		const shortand_properties = new Set()
 
-		/** @param {import("../private").ASTNode} node */
+		/** @param {import("../private").ASTNode & import("estree").Identifier} node */
 		function defer(node) {
 			if (fix_same_names) {
 				const name = node.name
@@ -63,7 +63,7 @@ module.exports = {
 			}
 		}
 
-		/** @param {import("../private").ASTNode} node */
+		/** @param {import("../private").ASTNode & import("estree").Identifier} node */
 		function report(node) {
 			const name = node.name
 			context.report({
@@ -78,15 +78,13 @@ module.exports = {
 					}
 					return null
 				},
-				messageId: node.type == "PrivateIdentifier"
-					? "not_match_private"
-					: "not_match",
+				messageId: "not_match",
 				node: /** @type {import("estree").Node} */(node)
 			})
 			if (fix_same_names) {
 				const usages = pending_usages.get(name)
 				while (usages?.length) {
-					report(/** @type {import("../private").ASTNode} */(
+					report(/** @type {import("../private").ASTNode & import("estree").Identifier} */(
 						usages.pop()
 					))
 				}
@@ -95,7 +93,7 @@ module.exports = {
 		}
 
 		return {
-			/** @param {import("../private").ASTNode} node */
+			/** @param {import("../private").ASTNode & import("estree").Identifier} node */
 			Identifier(node) {
 				const name = node.name
 				if (allow_regex.test(name)) return
@@ -124,7 +122,10 @@ module.exports = {
 				// (left = right) => {}
 				if ("AssignmentPattern" == type) {
 					if (parent.left == node) {
-						report(node)
+						// function func({ camelCase = true }) {}
+						if (parent.parent?.type != "Property") {
+							report(node)
+						}
 					} else {
 						defer(node)
 					}

package/test/pretty-imports/check_exports.js

@@ -0,0 +1,85 @@
+/**
+ * @param {import("eslint").RuleTester.ValidTestCase[]} valid 
+ * @param {import("eslint").RuleTester.InvalidTestCase[]} invalid
+ */
+exports.module = (valid, invalid) => {
+	valid.push({
+		code: `
+
+		export { a, b as c, d } from 'module'  ;
+		export { fff, ggg, hhh, iii, jjj, kkk, lll } from 'module/2'
+		`,
+		options: [{ maxLength: 30 }],
+		parserOptions: {
+			ecmaVersion: "latest",
+			sourceType: "module"
+		}
+	})
+	invalid.push(
+		{
+			code: `
+
+			export { a, b as c, d } from 'module'  ;
+		    export { fff, ggg, hhh, iii, jjj, kkk, lll } from 'module/2'
+			`,
+			errors: [{
+				messageId: "not_match",
+				type: "ExportNamedDeclaration",
+				line: 4,
+				column: 7
+			}],
+			options: [{ maxLength: 10 }],
+			output: `
+
+			export { a, b as c, d } from 'module'  ;
+		    export {
+		    	fff,
+		    	ggg,
+		    	hhh,
+		    	iii,
+		    	jjj,
+		    	kkk,
+		    	lll
+		    } from 'module/2'
+			`,
+			parserOptions: {
+				ecmaVersion: "latest",
+				sourceType: "module"
+			}
+		},
+		{
+			code: `
+
+			export { a, b as c, d } from 'module'  ;
+		    export { fff, ggg, hhh, iii, jjj, kkk, lll } from 'module/2'
+			`,
+			errors: [{
+				messageId: "not_match",
+				type: "ExportNamedDeclaration",
+				line: 4,
+				column: 7
+			}],
+			options: [{
+				indent: "  ",
+				maxLength: 10
+			}],
+			output: `
+
+			export { a, b as c, d } from 'module'  ;
+		    export {
+		      fff,
+		      ggg,
+		      hhh,
+		      iii,
+		      jjj,
+		      kkk,
+		      lll
+		    } from 'module/2'
+			`,
+			parserOptions: {
+				ecmaVersion: "latest",
+				sourceType: "module"
+			}
+		}
+	)
+}

package/test/pretty-imports/check_imports.js

@@ -0,0 +1,85 @@
+/**
+ * @param {import("eslint").RuleTester.ValidTestCase[]} valid 
+ * @param {import("eslint").RuleTester.InvalidTestCase[]} invalid
+ */
+exports.module = (valid, invalid) => {
+	valid.push({
+		code: `
+
+		import { a, b as c, d } from 'module'  ;
+		import { fff, ggg, hhh, iii, jjj, kkk, lll } from 'module/2'
+		`,
+		options: [{ maxLength: 30 }],
+		parserOptions: {
+			ecmaVersion: "latest",
+			sourceType: "module"
+		}
+	})
+	invalid.push(
+		{
+			code: `
+
+			import { a, b as c, d } from 'module'  ;
+		    import { fff, ggg, hhh, iii, jjj, kkk, lll } from 'module/2'
+			`,
+			errors: [{
+				messageId: "not_match",
+				type: "ImportDeclaration",
+				line: 4,
+				column: 7
+			}],
+			options: [{ maxLength: 10 }],
+			output: `
+
+			import { a, b as c, d } from 'module'  ;
+		    import {
+		    	fff,
+		    	ggg,
+		    	hhh,
+		    	iii,
+		    	jjj,
+		    	kkk,
+		    	lll
+		    } from 'module/2'
+			`,
+			parserOptions: {
+				ecmaVersion: "latest",
+				sourceType: "module"
+			}
+		},
+		{
+			code: `
+
+			import { a, b as c, d } from 'module'  ;
+		    import { fff, ggg, hhh, iii, jjj, kkk, lll } from 'module/2'
+			`,
+			errors: [{
+				messageId: "not_match",
+				type: "ImportDeclaration",
+				line: 4,
+				column: 7
+			}],
+			options: [{
+				indent: "  ",
+				maxLength: 10
+			}],
+			output: `
+
+			import { a, b as c, d } from 'module'  ;
+		    import {
+		      fff,
+		      ggg,
+		      hhh,
+		      iii,
+		      jjj,
+		      kkk,
+		      lll
+		    } from 'module/2'
+			`,
+			parserOptions: {
+				ecmaVersion: "latest",
+				sourceType: "module"
+			}
+		}
+	)
+}

package/test/pretty-imports.js

@@ -0,0 +1,19 @@
+"use strict"
+const { RuleTester } = require("eslint")
+
+/** @type {RuleTester.ValidTestCase[]} */
+const valid = []
+/** @type {RuleTester.InvalidTestCase[]} */
+const invalid = []
+
+require("./pretty-imports/check_exports").module(valid, invalid)
+require("./pretty-imports/check_imports").module(valid, invalid)
+
+new RuleTester().run(
+	"pretty-imports",
+	require("../rules/pretty-imports"),
+	{ 
+		valid,
+		invalid
+	}
+)

package/test/svelte-naming-convention/case_by_identifier.js

@@ -33,7 +33,6 @@ exports.module = (valid, invalid) => {
 					column: 18
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			var value = [ deferA ]
 			var value = [ pending_a ]
@@ -70,6 +69,7 @@ exports.module = (valid, invalid) => {
 			pendingA = pendingB
 			var pendingA, pendingB
 			pendingA = pendingB
+			function func({ pendingA = 123 }) {}
 			`,
 			errors: [
 				{
@@ -103,13 +103,14 @@ exports.module = (valid, invalid) => {
 					column: 15
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			deferA = deferB
 			pending_a = pending_b
 			var pending_a, pending_b
 			pending_a = pending_b
-			`
+			function func({ pendingA = 123 }) {}
+			`,
+			parserOptions: { ecmaVersion: "latest" }
 		},
 		// AssignmentPattern
 		{
@@ -141,7 +142,6 @@ exports.module = (valid, invalid) => {
 					column: 18
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			(camel_case = deferA) => {}
 			(snake_case = pending_a) => {}
@@ -175,7 +175,6 @@ exports.module = (valid, invalid) => {
 					column: 10
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			await deferA
 			await pending_a
@@ -227,7 +226,6 @@ exports.module = (valid, invalid) => {
 					column: 15
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			deferA = deferB
 			pending_a - pending_b
@@ -260,7 +258,6 @@ exports.module = (valid, invalid) => {
 					column: 4
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			deferA()
 			pending_a()
@@ -319,7 +316,6 @@ exports.module = (valid, invalid) => {
 					column: 22
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			test ? deferA : deferB
 			test ? pending_a : pending_b
@@ -338,7 +334,6 @@ exports.module = (valid, invalid) => {
 				line: 3,
 				column: 8
 			}],
-			options: [{ fixSameNames: true }],
 			output: `
 			export * as camelCase from "module"
 			var camel_case
@@ -359,6 +354,7 @@ exports.module = (valid, invalid) => {
 				line: 3,
 				column: 8
 			}],
+			options: [{ fixSameNames: false }],
 			output: `
 			export default camelCase
 			var camel_case
@@ -385,7 +381,6 @@ exports.module = (valid, invalid) => {
 					column: 8
 				},
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			export default camel_case
 			var camel_case
@@ -413,7 +408,6 @@ exports.module = (valid, invalid) => {
 					column: 19
 				},
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			export { default as camelCase, camelCase2 } from 'module'
 			var camel_case, camel_case2
@@ -448,7 +442,6 @@ exports.module = (valid, invalid) => {
 					column: 4
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			deferA
 			pending_a
@@ -486,7 +479,6 @@ exports.module = (valid, invalid) => {
 					column: 23
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			for (camel_case in deferA) {}
 			for (snake_case in pending_a) {}
@@ -550,7 +542,6 @@ exports.module = (valid, invalid) => {
 					column: 29
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			for (deferA; deferB; deferC) {}
 			for (pending_a; pending_b; pending_c) {}
@@ -613,7 +604,6 @@ exports.module = (valid, invalid) => {
 					column: 8
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			if (deferA) {}
 			if (pending_a) {}
@@ -668,7 +658,6 @@ exports.module = (valid, invalid) => {
 					column: 4
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			import { imported as camel_case, camelCase2, camelCase3 } from "module"
 			camel_case
@@ -719,7 +708,6 @@ exports.module = (valid, invalid) => {
 					column: 16
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			deferA && deferB
 			pending_a || pending_b
@@ -772,7 +760,6 @@ exports.module = (valid, invalid) => {
 					column: 11
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			deferA.camelCase
 			pending_a.camelCase
@@ -826,7 +813,6 @@ exports.module = (valid, invalid) => {
 					column: 8
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			new deferA
 			new pending_a
@@ -865,7 +851,6 @@ exports.module = (valid, invalid) => {
 					column: 18
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			var value = { camelCase: deferA }
 			var value = { key: pending_a }
@@ -893,7 +878,6 @@ exports.module = (valid, invalid) => {
 					column: 19
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			class ObjectB { camelCase = camelCase2 }
 			var camel_case, camel_case2
@@ -918,7 +902,6 @@ exports.module = (valid, invalid) => {
 					column: 13
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			var [ ...camel_case_a ] = value
 			var { ...camel_case_b } = value
@@ -950,7 +933,6 @@ exports.module = (valid, invalid) => {
 					column: 31
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			function func_a() { return deferA }
 			function func_b() { return pending_a }
@@ -983,7 +965,6 @@ exports.module = (valid, invalid) => {
 					column: 10
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			a, b, deferA
 			a, b, pending_a
@@ -1031,7 +1012,6 @@ exports.module = (valid, invalid) => {
 					column: 31
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			var value = [ ...deferA, deferB ]
 			var value = [ ...pending_a, pending_b ]
@@ -1065,7 +1045,6 @@ exports.module = (valid, invalid) => {
 					column: 12
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			switch (deferA) {}
 			switch (pending_a) {}
@@ -1098,7 +1077,6 @@ exports.module = (valid, invalid) => {
 					column: 25
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			switch (cond) { case deferA: break }
 			switch (cond) { case pending_a: break }
@@ -1131,7 +1109,6 @@ exports.module = (valid, invalid) => {
 					column: 4
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			deferA\`quasis\${expression}quasis\`
 			pending_a\`quasis\${expression}quasis\`
@@ -1165,7 +1142,6 @@ exports.module = (valid, invalid) => {
 					column: 10
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			throw deferA
 			throw pending_a
@@ -1198,7 +1174,6 @@ exports.module = (valid, invalid) => {
 					column: 5
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			+deferA
 			-pending_a
@@ -1231,7 +1206,6 @@ exports.module = (valid, invalid) => {
 					column: 4
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			++deferA
 			--pending_a
@@ -1276,7 +1250,6 @@ exports.module = (valid, invalid) => {
 					column: 15
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			var camel_case = deferA
 			var snake_case = pending_a
@@ -1315,7 +1288,6 @@ exports.module = (valid, invalid) => {
 					column: 11
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			while (deferA) {}
 			while (pending_a) {}
@@ -1348,7 +1320,6 @@ exports.module = (valid, invalid) => {
 					column: 10
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			with (deferA) {}
 			with (pending_a) {}
@@ -1381,7 +1352,6 @@ exports.module = (valid, invalid) => {
 					column: 31
 				}
 			],
-			options: [{ fixSameNames: true }],
 			output: `
 			function* func_a() { yield deferA }
 			function* func_b() { yield pending_a }