@firefoxic/eslint-config 6.0.0 → 6.0.1

package/lib/enough-is-enough/index.js

@@ -0,0 +1,16 @@
+import noMultilineNamedImports from "./no-multiline-named-imports/index.js"
+import noSingleQuotesInImportsAndObjectKeys from "./no-single-quotes-in-imports-and-object-keys/index.js"
+
+const plugin = {
+	// preferred location of name and version
+	meta: {
+		name: `eslint-plugin-enough-is-enough`,
+		namespace: `enough-is-enough`,
+	},
+	rules: {
+		"no-multiline-named-imports": noMultilineNamedImports,
+		"no-single-quotes-in-imports-and-object-keys": noSingleQuotesInImportsAndObjectKeys,
+	},
+}
+
+export default plugin

package/lib/enough-is-enough/no-multiline-named-imports/index.js

@@ -0,0 +1,53 @@
+export default {
+	meta: {
+		type: `layout`,
+		fixable: `code`,
+		messages: {
+			multiline: `Multi-line named imports are disallowed.`,
+		},
+	},
+	create (context) {
+		let sourceCode = context.sourceCode
+
+		return {
+			ImportDeclaration (node) {
+				let namedSpecifiers = node.specifiers.filter((s) => s.type === `ImportSpecifier`)
+
+				if (namedSpecifiers.length === 0) return
+
+				let tokens = sourceCode.getTokens(node)
+				let openBrace = tokens.find((t) => t.value === `{`)
+				let closeBrace = tokens.reverse().find((t) => t.value === `}`)
+
+				if (!openBrace || !closeBrace) return
+
+				if (openBrace.loc.start.line !== closeBrace.loc.end.line) {
+					context.report({
+						node,
+						messageId: `multiline`,
+						fix (fixer) {
+							let imports = namedSpecifiers.map((s) => sourceCode.getText(s)).join(`, `)
+							let defaultImport = node.specifiers.find((s) => s.type === `ImportDefaultSpecifier`)
+							let namespaceImport = node.specifiers.find((s) => s.type === `ImportNamespaceSpecifier`)
+							let source = sourceCode.getText(node.source)
+
+							let newImport = `import `
+
+							if (defaultImport) newImport += `${sourceCode.getText(defaultImport)}, `
+							if (namespaceImport) newImport += `${sourceCode.getText(namespaceImport)}, `
+							newImport += `{ ${imports} } from ${source}`
+
+							let lastToken = sourceCode.getLastToken(node)
+							let hasSemicolon = lastToken && lastToken.value === `;`
+
+							if (hasSemicolon) newImport += `;`
+
+							return fixer.replaceText(node, newImport)
+						},
+					})
+				}
+			},
+
+		}
+	},
+}

package/lib/enough-is-enough/no-single-quotes-in-imports-and-object-keys/index.js

@@ -0,0 +1,79 @@
+export default {
+	meta: {
+		type: `suggestion`,
+		docs: { description: `Disallow single quotes in import/export sources and object keys when quoted` },
+		fixable: `code`,
+		schema: [],
+	},
+
+	create (context) {
+		let sourceCode = context.sourceCode
+
+		function isSingleQuoted (node) {
+			if (typeof node.value !== `string`) return false
+
+			let text = sourceCode.getText(node)
+
+			return text.startsWith(`'`) && text.endsWith(`'`)
+		}
+
+		function checkLiteralInImportOrExport (node) {
+			if (!node.source) return
+
+			let literal = node.source
+
+			if (isSingleQuoted(literal)) {
+				context.report({
+					node: literal,
+					message: `Use double quotes for import/export source.`,
+					fix (fixer) {
+						let newText = `"${literal.value.replace(/"/g, `\\"`)}"`
+
+						return fixer.replaceText(literal, newText)
+					},
+				})
+			}
+		}
+
+		function checkObjectExpression (node) {
+			let properties = node.properties
+
+			let hasQuotedKey = properties.some(
+				(prop) => prop.type === `Property`
+					&& prop.key.type === `Literal`
+					&& typeof prop.key.value === `string`,
+			)
+
+			if (!hasQuotedKey) return
+
+			for (let prop of properties) {
+				if (
+					prop.type !== `Property`
+					|| prop.key.type !== `Literal`
+					|| typeof prop.key.value !== `string`
+				) continue
+
+				let key = prop.key
+
+				if (isSingleQuoted(key)) {
+					context.report({
+						node: key,
+						message: `Use double quotes for quoted object keys.`,
+						fix (fixer) {
+							let newText = `"${key.value.replace(/"/g, `\\"`)}"`
+
+							return fixer.replaceText(key, newText)
+						},
+					})
+				}
+			}
+		}
+
+		return {
+			ImportDeclaration: checkLiteralInImportOrExport,
+			ExportNamedDeclaration: checkLiteralInImportOrExport,
+			ExportAllDeclaration: checkLiteralInImportOrExport,
+			ObjectExpression: checkObjectExpression,
+		}
+	},
+}

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@firefoxic/eslint-config",
 	"description": "Shared config for eslint by firefoxic.",
-	"version": "6.0.0",
+	"version": "6.0.1",
 	"license": "MIT",
 	"author": {
 		"name": "Sergey Artemov",
@@ -24,8 +24,8 @@
 		"types": "./lib/index.d.ts"
 	},
 	"files": [
-		"./lib/index.d.ts",
-		"./lib/index.js"
+		"./lib/**/*.d.ts",
+		"./lib/**/*.js"
 	],
 	"peerDependencies": {
 		"eslint": "^9.39.2"