Merge 7bd8687 into ae983e2

docs/rules/catch-error-name.md

@@ -1,12 +1,11 @@
 # Enforce a specific parameter name in catch clauses
 
-Applies to both `try/catch` clauses and `promise.catch(...)` handlers.
+Applies to both `try/catch` clauses and `promise.catch(…)` handlers.
 
 The desired name is configurable, but defaults to `error`.
 
 This rule is fixable unless the reported code was destructuring an error.
 
-
 ## Fail
 
 ```js
@@ -21,7 +20,6 @@ try {
 somePromise.catch(e => {})
 ```
 
-
 ## Pass
 
 ```js
@@ -74,7 +72,6 @@ somePromise.catch(_ => {
 });
 ```
 
-
 ## Options
 
 ### name
@@ -101,7 +98,7 @@ You can set the `name` option like this:
 ]
 ```
 
-This option lets you specify a regex pattern for matches to ignore. The default is `^_$`.
+This option lets you specify a regex pattern for matches to ignore. The default allows `_` and descriptive names like `networkError`.
 
 With `^unicorn$`, this would fail:
 
@@ -122,3 +119,7 @@ try {
 	// …
 }
 ```
+
+## Tip
+
+In order to avoid shadowing in nested catch clauses, the auto-fix rule appends underscores to the identifier name. Since this might be hard to read, the default setting for `caughtErrorsIgnorePattern` allows the use of descriptive names instead, for example, `fsError` or `authError`.

docs/rules/string-content.md

@@ -0,0 +1,78 @@
+# Enforce better string content
+
+Enforce certain things about the contents of strings. For example, you can enforce using `’` instead of `'` to avoid escaping. Or you could block some words. The possibilities are endless.
+
+This rule is fixable.
+
+*It only reports one pattern per AST node at the time.*
+
+## Fail
+
+```js
+const foo = 'Someone\'s coming!';
+```
+
+## Pass
+
+```js
+const foo = 'Someone’s coming!';
+```
+
+## Options
+
+Type: `object`
+
+### patterns
+
+Type: `object`
+
+Extend [default patterns](#default-pattern).
+
+The example below:
+
+- Disables the default `'` → `’` replacement.
+- Adds a custom `unicorn` → `🦄` replacement.
+- Adds a custom `awesome` → `😎` replacement and a custom message.
+- Adds a custom `cool` → `😎` replacement, but disables auto fix.
+
+```json
+{
+	"unicorn/string-content": [
+		"error",
+		{
+			"patterns": {
+				"'": false,
+				"unicorn": "🦄",
+				"awesome": {
+					"suggest": "😎",
+					"message": "Please use `😎` instead of `awesome`."
+				},
+				"cool": {
+					"suggest": "😎",
+					"fix": false
+				}
+			}
+		}
+	]
+}
+```
+
+The key of `patterns` is treated as a regex, so you must escape special characters.
+
+For example, if you want to enforce `...` → `…`:
+
+```json
+{
+	"patterns": {
+		"\\.\\.\\.": "…"
+	}
+}
+```
+
+## Default Pattern
+
+```json
+{
+	"'": "’"
+}
+```

index.js

@@ -64,6 +64,7 @@ module.exports = {
 				'unicorn/prefer-trim-start-end': 'error',
 				'unicorn/prefer-type-error': 'error',
 				'unicorn/prevent-abbreviations': 'error',
+				'unicorn/string-content': 'off',
 				'unicorn/throw-new-error': 'error'
 			}
 		}

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "eslint-plugin-unicorn",
-	"version": "16.1.1",
+	"version": "17.0.1",
 	"description": "Various awesome ESLint rules",
 	"license": "MIT",
 	"repository": "sindresorhus/eslint-plugin-unicorn",

readme.md

@@ -79,6 +79,7 @@ Configure it in `package.json`.
 			"unicorn/prefer-trim-start-end": "error",
 			"unicorn/prefer-type-error": "error",
 			"unicorn/prevent-abbreviations": "error",
+			"unicorn/string-content": "off",
 			"unicorn/throw-new-error": "error"
 		}
 	}
@@ -132,6 +133,7 @@ Configure it in `package.json`.
 - [prefer-trim-start-end](docs/rules/prefer-trim-start-end.md) - Prefer `String#trimStart()` / `String#trimEnd()` over `String#trimLeft()` / `String#trimRight()`. *(fixable)*
 - [prefer-type-error](docs/rules/prefer-type-error.md) - Enforce throwing `TypeError` in type checking conditions. *(fixable)*
 - [prevent-abbreviations](docs/rules/prevent-abbreviations.md) - Prevent abbreviations. *(partly fixable)*
+- [string-content](docs/rules/string-content.md) - Enforce better string content. *(fixable)*
 - [throw-new-error](docs/rules/throw-new-error.md) - Require `new` when throwing an error. *(fixable)*
 
 ## Deprecated Rules

rules/better-regex.js

@@ -62,17 +62,17 @@ const create = context => {
 			const newPattern = cleanRegexp(oldPattern, flags);
 
 			if (oldPattern !== newPattern) {
-				// Escape backslash
-				const fixed = quoteString(newPattern.replace(/\\/g, '\\\\'));
-
 				context.report({
 					node,
 					message,
 					data: {
 						original: oldPattern,
 						optimized: newPattern
 					},
-					fix: fixer => fixer.replaceText(patternNode, fixed)
+					fix: fixer => fixer.replaceText(
+						patternNode,
+						quoteString(newPattern)
+					)
 				});
 			}
 		}

rules/catch-error-name.js

@@ -34,7 +34,7 @@ const create = context => {
 
 	const options = {
 		name: 'error',
-		caughtErrorsIgnorePattern: '^_$',
+		caughtErrorsIgnorePattern: /^_$|^[\dA-Za-z]+(e|E)rror$/.source,
 		...context.options[0]
 	};
 

rules/prefer-exponentiation-operator.js

@@ -1,7 +1,10 @@
 'use strict';
 const getDocumentationUrl = require('./utils/get-documentation-url');
 
+const create = () => ({});
+
 module.exports = {
+	create,
 	meta: {
 		type: 'suggestion',
 		docs: {

rules/prefer-spread.js

@@ -14,7 +14,32 @@ const selector = [
 ].join('');
 
 const create = context => {
-	const getSource = node => context.getSourceCode().getText(node);
+	const sourceCode = context.getSourceCode();
+	const getSource = node => sourceCode.getText(node);
+
+	const needsSemicolon = node => {
+		const tokenBefore = sourceCode.getTokenBefore(node);
+
+		if (tokenBefore) {
+			const {type, value} = tokenBefore;
+			if (type === 'Punctuator') {
+				if (value === ';') {
+					return false;
+				}
+
+				if (value === ']' || value === ')') {
+					return true;
+				}
+			}
+
+			const lastBlockNode = sourceCode.getNodeByRangeIndex(tokenBefore.range[0]);
+			if (lastBlockNode && lastBlockNode.type === 'ObjectExpression') {
+				return true;
+			}
+		}
+
+		return false;
+	};
 
 	return {
 		[selector](node) {
@@ -23,7 +48,7 @@ const create = context => {
 				message: 'Prefer the spread operator over `Array.from()`.',
 				fix: fixer => {
 					const [arrayLikeArgument, mapFn, thisArgument] = node.arguments.map(getSource);
-					let replacement = `[...${arrayLikeArgument}]`;
+					let replacement = `${needsSemicolon(node) ? ';' : ''}[...${arrayLikeArgument}]`;
 
 					if (mapFn) {
 						const mapArguments = [mapFn, thisArgument].filter(Boolean);

rules/regex-shorthand.js

@@ -1,7 +1,10 @@
 'use strict';
 const getDocumentationUrl = require('./utils/get-documentation-url');
 
+const create = () => ({});
+
 module.exports = {
+	create,
 	meta: {
 		type: 'suggestion',
 		docs: {