eslint-plugin-crisp 1.7.16 → 1.7.17

package/README.md

@@ -56,6 +56,13 @@ Each item has emojis denoting:
 
 ### Rules
 
+#### General TS rules
+
+| Name | Description | 🟠 | 🟢 | 🟣 |
+| :- | :- | :- | :- | :- |
+| [@typescript-eslint/await-thenable](https://typescript-eslint.io/rules/await-thenable) | Disallows awaiting a value that is not thenable |  |  | 🟣 |
+| [@typescript-eslint/no-unused-vars](https://typescript-eslint.io/rules/no-unused-vars) | Disallows unused variables |  |  | 🟣 |
+
 #### General JS rules
 
 | Name | Description | 🟠 | 🟢 | 🟣 |
@@ -76,10 +83,10 @@ Each item has emojis denoting:
 | [eol-last](https://eslint.org/docs/latest/rules/eol-last) | Requires newline at the end of files | 🟠 | 🟢 | 🟣 |
 | [for-direction](https://eslint.org/docs/latest/rules/for-direction) | Prevents for-loops to have stop condition that can never be reached | 🟠 | 🟢 | 🟣 |
 | [indent](https://eslint.org/docs/latest/rules/indent) | Enforces 2-space indentation, and specific indentation levels for some nodes | 🟠 | 🟢 | 🟣 |
-| [key-spacing](https://eslint.org/docs/latest/rules/key-spacing) | Enforces consistent spacing in object literal properties (no space between the key and the colon, one space between the colon and the value) |  | 🟢 | 🟣 |
+| [key-spacing](https://eslint.org/docs/latest/rules/key-spacing) | Enforces consistent spacing in object literal properties (no space between the key and the colon, one space between the colon and the value) | 🟠 | 🟢 | 🟣 |
 | [keyword-spacing](https://eslint.org/docs/latest/rules/keyword-spacing) | Enforces consistent spacing before and after keywords | 🟠 | 🟢 | 🟣 |
 | [linebreak-style](https://eslint.org/docs/latest/rules/linebreak-style) | Enforces Unix-style line endings | 🟠 | 🟢 | 🟣 |
-| [max-len](https://eslint.org/docs/latest/rules/max-len) | Enforces a maximum line length of 80 / 100 characters (`@import`, `@typedef` and `@property` lines are ignored) | 🟠 | 🟢 | 🟣 |
+| [max-len](https://eslint.org/docs/latest/rules/max-len) | Enforces a maximum line length of 100 characters (`@import`, `@typedef` and `@property` lines are ignored) | 🟠 |  | 🟣 |
 | [newline-per-chained-call](https://eslint.org/docs/latest/rules/newline-per-chained-call) | Requires a newline after each call in a method chain |  | 🟢 | 🟣 |
 | [no-console](https://eslint.org/docs/latest/rules/no-console) | Disallows the use of `console` | 🟠 | 🟢 | 🟣 |
 | [no-debugger](https://eslint.org/docs/latest/rules/no-debugger) | Disallows the use of `debugger` | 🟠 | 🟢 | 🟣 |
@@ -87,14 +94,14 @@ Each item has emojis denoting:
 | [no-implicit-coercion](https://eslint.org/docs/latest/rules/no-implicit-coercion) | Disallows short notations for type conversion |  | 🟢 | 🟣 |
 | [no-magic-numbers](https://eslint.org/docs/latest/rules/no-magic-numbers) | Disallows magic numbers (except the ones in the context of array indexes and in default value assignments) | 🟠 |
 | [no-multi-str](https://eslint.org/docs/latest/rules/no-multi-str) | Disallows multiline strings | 🟠 | 🟢 | 🟣 |
+| [no-multi-spaces](https://eslint.org/docs/latest/rules/no-multi-spaces) | Disallows multiple consecutive spaces except for indentation | 🟠 |  |  |
 | [no-multiple-empty-lines](https://eslint.org/docs/latest/rules/no-multiple-empty-lines) | Allows a maximum of 1 consecutive empty lines |  | 🟢 | 🟣 |
 | [no-mixed-spaces-and-tabs](https://eslint.org/docs/latest/rules/no-mixed-spaces-and-tabs) | Disallows mixed spaces and tabs for indentation | 🟠 | 🟢 | 🟣 |
 | [no-restricted-syntax](https://eslint.org/docs/latest/rules/no-restricted-syntax) | Enforces `switch` `case`'s content to be enclosed in braces |  | 🟢 | 🟣 |
-| [no-restricted-syntax](https://eslint.org/docs/latest/rules/no-restricted-syntax) | Enforces `watch` watchers to use the object form with a `handler` function |  | 🟢 |  |
 | [no-tabs](https://eslint.org/docs/latest/rules/no-tabs) | Disallows tabs | 🟠 | 🟢 | 🟣 |
 | [no-trailing-spaces](https://eslint.org/docs/latest/rules/no-trailing-spaces) | Disallows trailing whitespace at the end of lines | 🟠 | 🟢 | 🟣 |
 | [no-undef](https://eslint.org/docs/latest/rules/no-undef) | Disallows use of undeclared variables | 🟠 | 🟢 | 🟣 |
-| [no-unused-vars](https://eslint.org/docs/latest/rules/no-unused-vars) | Disallows unused variables | 🟠 | 🟢 | 🟣 |
+| [no-unused-vars](https://eslint.org/docs/latest/rules/no-unused-vars) | Disallows unused variables | 🟠 | 🟢 |  |
 | [no-unsafe-optional-chaining](https://eslint.org/docs/latest/rules/no-unsafe-optional-chaining) | Disallows use of optional chaining in contexts where the `undefined` value is not allowed | 🟠 | 🟢 | 🟣 |
 | [object-curly-newline](https://eslint.org/docs/latest/rules/object-curly-newline) | Requires line breaks after opening and before closing braces | 🟠 | 🟢 | 🟣 |
 | [object-curly-spacing](https://eslint.org/docs/latest/rules/object-curly-spacing) | Requires spacing inside of braces | 🟠 | 🟢 | 🟣 |
@@ -117,25 +124,24 @@ Each item has emojis denoting:
 | :- | :- | :- | :- | :- |
 | [crisp/align-comments](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/align-comments.js) | Enforces alignment of comments compared to the previous line (the `indent` rule doesn't check this case) | 🟠 | 🟢 | 🟣 |
 | [crisp/consecutive-line-comments](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/consecutive-line-comments.js) | Enforces consecutive single-line comments to form a single multi-line continuation block | 🟠 | 🟢 | 🟣 |
-| [crisp/align-consecutive-class-assignements](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/align-consecutive-class-assignements.js) | Enforces alignment of consecutive assignment statements in a class constructor | 🟠 |
-| [crisp/align-one-var](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/align-one-var.js) | Enforces alignment of variables in 'one-var' statements | 🟠 |
-| [crisp/align-requires](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/align-requires.js) | Enforces alignment of require statements | 🟠 |
 | [crisp/arrow-block-newline](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/arrow-block-newline.js) | Requires a newline after `{` in arrow function blocks | 🟠 | 🟢 | 🟣 |
-| [crisp/const](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/const.js) | Enforces that `const` variables start with `__` or are all uppercase | 🟠 |
+| [crisp/const](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/const.js) | Enforces that `const` variables start with `__` or are all uppercase | 🟠 | 🟢 |  |
 | [crisp/constructor-variables](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/constructor-variables.js) | Ensures all class properties in the constructor start with `_` | 🟠 |
 | [crisp/enforce-optional](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/enforce-optional.js) | Enforces use of optional chaining | 🟠 | 🟢 | 🟣 |
 | [crisp/header-check](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/header-check.js) | Enforces files to start with Crisp header | 🟠 | 🟢 | 🟣 |
 | [crisp/header-comments-check](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/header-comments-check.js) | Enforces different comment blocks before different groups (imports, constants and instances) | 🟠 | 🟢 | 🟣 |
 | [crisp/import-group-comment](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/import-group-comment.js) | Ensures `import` statements are preceded by a comment stating their type |  | 🟢 | 🟣 |
 | [crisp/import-group-order](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/import-group-order.js) | Ensures `import` statements are correctly ordered |  | 🟢 | 🟣 |
-| [crisp/logger-arguments](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/logger-arguments.js) | Requires static logger messages and reference-only interpolation arguments | 🟠 | 🟢 | 🟣 |
+| [crisp/logger-arguments](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/logger-arguments.js) | Requires static logger messages and reference-only interpolation arguments | 🟠 |  | 🟣 |
 | [crisp/methods-naming](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/methods-naming.js) | Ensures methods are named according to their access (`public`, `private`, `protected`) | 🟠 | 🟢 | 🟣 |
 | [crisp/methods-ordering](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/methods-ordering.js) | Ensures methods order according to their access: `public` then `protected` then `private` | 🟠 | 🟢 | 🟣 |
 | [crisp/multiline-comment-end-backslash](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/multiline-comment-end-backslash.js) | Enforces multiline comments to end with a backslash | 🟠 | 🟢 | 🟣 |
 | [crisp/newline-after-switch-case](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/newline-after-switch-case.js) | Requires newline between switch cases | 🟠 | 🟢 | 🟣 |
 | [crisp/no-async](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-async.js) | Disallows the use of `async/syntax`, in favor of `Promise` | 🟠 | 🟢 |
+| [crisp/no-extra-line-within-function](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-extra-line-within-function.js) | Allows at most one empty line between declarations within functions | 🟠 | 🟢 | 🟣 |
+| [crisp/no-inline-function-in-ternary](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-inline-function-in-ternary.js) | Disallows inline functions in ternary branches | 🟠 | 🟢 | 🟣 |
 | [crisp/no-short-parameters](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-short-parameters.js) | Disallows one-character parameter names | 🟠 | 🟢 | 🟣 |
-| [crisp/no-space-in-optional-arguments](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-space-in-optional-arguments.js) | Disallows space before or after `=` in optional parameters | 🟠 |
+| [crisp/no-space-in-default-parameters](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-space-in-default-parameters.js) | Disallows space before or after `=` in default parameters | 🟠 |
 | [crisp/no-snake-case](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-snake-case.js) | Disallows snake_case |  | 🟢 | 🟣 |
 | [crisp/no-useless-template-literals](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-useless-template-literals.js) | Disallows unnecessary use of template literals |  | 🟢 | 🟣 |
 | [crisp/no-var-in-blocks](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/no-var-in-blocks.js) | Disallows `var` declarations inside function, method, or class block | 🟠 | 🟢 | 🟣 |
@@ -151,6 +157,8 @@ Each item has emojis denoting:
 | Name | Description | 🟠 | 🟢 | 🟣 |
 | :- | :- | :- | :- | :- |
 | [jsdoc/no-undefined-types](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/no-undefined-types.md) | Rule is **disabled** to allow some undefined types | 🟠 |
+| [jsdoc/reject-any-type](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/reject-any-type.md) | Rule is **disabled** to allow the `any` type | 🟠 | 🟢 |  |
+| [jsdoc/reject-function-type](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/reject-function-type.md) | Rule is **disabled** to allow the `Function` type | 🟠 | 🟢 |  |
 | [jsdoc/no-types](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/no-types.md) | Disallows types in JSDoc tags, relying on the TypeScript signature instead |  |  | 🟣 |
 | [jsdoc/require-description](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-description.md) | Requires all functions to have a description in their JSDoc |  | 🟢 | 🟣 |
 | [jsdoc/require-param](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-param.md) | Rule is **disabled** as TS functions are self documented |  |  | 🟣 |
@@ -158,13 +166,13 @@ Each item has emojis denoting:
 | [jsdoc/require-param-description](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-param-description.md) | Rule is **disabled** as we don't write any description for `@param` tags | 🟠 | 🟢 | 🟣 |
 | [jsdoc/check-param-names](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/check-param-names.md) | Rule is **disabled** as TS functions are self documented |  |  | 🟣 |
 | [jsdoc/require-property-description](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-property-description.md) | Rule is **disabled** as we don't write any description for `@property` tags | 🟠 | 🟢 | 🟣 |
-| [jsdoc/require-returns](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-returns.md) | Rule is **disabled** as TS functions are self documented |  |  | 🟣 |
+| [jsdoc/require-returns](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-returns.md) | Rule is **disabled** to allow functions without a `@return` tag | 🟠 | 🟢 | 🟣 |
 | [jsdoc/require-returns-type](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-returns-type.md) | Rule is **disabled** as types come from the TypeScript signature |  |  | 🟣 |
 | [jsdoc/require-returns-check](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-returns-check.md) | Rule is **disabled** as we don't write `@returns` tags |  |  | 🟣 |
 | [jsdoc/require-yields](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-yields.md) | Rule is **disabled** as TS functions are self documented |  |  | 🟣 |
 | [jsdoc/require-yields-check](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-yields-check.md) | Rule is **disabled** as we don't write `@yields` tags |  |  | 🟣 |
 | [jsdoc/require-jsdoc](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/require-jsdoc.md) | Enforces JSDoc comments on functions and classes | 🟠 | 🟢 | 🟣 |
-| [jsdoc/sort-tags](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/sort-tags.md) | Enforces specific order for tags | 🟠 | 🟢 | 🟣 |
+| [jsdoc/sort-tags](https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/rules/sort-tags.md) | Enforces specific order for tags | 🟠 | 🟢 |  |
 
 #### Crisp JSDoc rules
 
@@ -173,6 +181,7 @@ Each item has emojis denoting:
 | [crisp/jsdoc-align-params](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/jsdoc-align-params.js) | Enforces various alignments | 🟠 | 🟢 |
 | [crisp/jsdoc-check-indentation](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/jsdoc-check-indentation.js) | Enforces consistent indentation | 🟠 | 🟢 |
 | [crisp/jsdoc-check-optional-params](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/jsdoc-check-optional-params.js) | Requires optional parameters to be surrounded by brackets |  | 🟢 |
+| [crisp/jsdoc-description-multiline](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/jsdoc-description-multiline.js) | Enforces multiline formatting for function JSDoc descriptions |  |  | 🟣 |
 | [crisp/jsdoc-enforce-access](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/jsdoc-enforce-access.js) | Requires one of `@public`, `@private`, or `@protected` for functions |  | 🟢 |
 | [crisp/jsdoc-enforce-classdesc](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/jsdoc-enforce-classdesc.js) | Ensures JSDoc for class headers to include a non-empty `@classdesc` | 🟠 | 🟢 |
 | [crisp/jsdoc-require-description-uppercase](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/jsdoc-require-description-uppercase.js) | Requires descriptions to start with an uppercase character | 🟠 | 🟢 |
@@ -225,6 +234,7 @@ Each item has emojis denoting:
 | [crisp/vue-class-attribute-last](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-class-attribute-last.js) | Enforces `class` to be the last attribute holding a value, and `:class` to be the last bound attribute |  | 🟢 |
 | [crisp/vue-class-shorthand-max-len](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-class-shorthand-max-len.js) | Requires a multiline `:class` array when Pug class shorthand exceeds 100 characters |  | 🟢 |
 | [crisp/vue-computed-order](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-computed-order.js) | Ensures computed properties are alphabetically ordered |  | 🟢 |
+| [crisp/vue-data-comment](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-data-comment.js) | Requires a comment as the first line of a Vue `data()` return object |  | 🟢 |  |
 | [crisp/vue-define-component](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-define-component.js) | Enforces mixins and components to use the `defineComponent` wrapper |  | 🟢 |
 | [crisp/vue-emits-order](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-emits-order.js) | Ensures emits properties are alphabetically ordered |  | 🟢 |
 | [crisp/vue-event-listener-jsdoc](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-event-listener-jsdoc.js) | Requires event listener method JSDoc descriptions to start with `Triggers on` |  | 🟢 |
@@ -233,14 +243,17 @@ Each item has emojis denoting:
 | [crisp/vue-html-indent](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-html-indent.js) | Enforces consistent indentation in `template` (supports for Pug) |  | 🟢 |
 | [crisp/vue-html-quotes](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-html-quotes.js) | Enforces HTML attributes to be enclosed with double quotes |  | 🟢 |
 | [crisp/vue-methods-separator](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-methods-separator.js) | Requires method section comments to be ordered as actions, helpers, then event listeners |  | 🟢 |
+| [crisp/vue-name-prop](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-name-prop.js) | Requires form component `name` props to be lowercase snake case |  | 🟢 |  |
 | [crisp/vue-no-component-class-shorthand](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-no-component-class-shorthand.js) | Disallows Pug class shorthand on Vue components |  | 🟢 |
 | [crisp/vue-no-mixed-class](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-no-mixed-class.js) | Disallows using both `class` and `:class` on the same element |  | 🟢 |
+| [crisp/vue-no-mixed-class-shorthand](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-no-mixed-class-shorthand.js) | Disallows mixing Pug class shorthand and class attributes |  | 🟢 |  |
 | [crisp/vue-no-name-option](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-no-name-option.js) | Disallows explicit component names in Vue single-file components |  | 🟢 |
 | [crisp/vue-no-regex-data](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-no-regex-data.js) | Disallows regular expressions to be declared in Vue data object |  | 🟢 |
 | [crisp/vue-props-declaration-line-break](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-props-declaration-line-break.js) | Enforces line break between type and default function in prop definition |  | 🟢 |
 | [crisp/vue-props-declaration-multiline](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-props-declaration-multiline.js) | Enforces props declarations to be multiline |  | 🟢 |
 | [crisp/vue-props-declaration-order](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-props-declaration-order.js) | Ensures props declarations are alphabetically ordered |  | 🟢 |
 | [crisp/vue-ref-case](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-ref-case.js) | Enforces `ref` attributes to snake case |  | 🟢 |
+| [crisp/vue-watch-handler](https://github.com/crisp-oss/eslint-plugin-crisp/blob/master/rules/vue-watch-handler.js) | Requires Vue watchers to use the object form with a valid `handler` function |  | 🟢 |  |
 
 ## License
 

package/index.js

@@ -3,9 +3,6 @@ import configRecommendedVue from "./recommended-vue.js";
 import configRecommendedTS from "./recommended-ts.js";
 
 import ruleAlignComments from "./rules/align-comments.js";
-import ruleAlignConsecutiveClassAssignements from "./rules/align-consecutive-class-assignements.js";
-import ruleAlignOneVar from "./rules/align-one-var.js";
-import ruleAlignRequires from "./rules/align-requires.js";
 import ruleArrowBlockNewline from "./rules/arrow-block-newline.js";
 import ruleConsecutiveLineComments from "./rules/consecutive-line-comments.js";
 import ruleConst from "./rules/const.js";
@@ -32,7 +29,7 @@ import ruleNewlineAfterSwitchCase from "./rules/newline-after-switch-case.js";
 import ruleNoAsync from "./rules/no-async.js";
 import ruleNoExtraLineWithinFunction from "./rules/no-extra-line-within-function.js";
 import ruleNoInlineFunctionInTernary from "./rules/no-inline-function-in-ternary.js";
-import ruleNoSpaceInOptionalArguments from "./rules/no-space-in-optional-arguments.js";
+import ruleNoSpaceInDefaultParameters from "./rules/no-space-in-default-parameters.js";
 import ruleNoUselessTemplateLiterals from "./rules/no-useless-template-literals.js";
 import ruleNoVarInBlocks from "./rules/no-var-in-blocks.js";
 import ruleNoShortParameters from "./rules/no-short-parameters.js";
@@ -84,9 +81,6 @@ const plugin = {
 
   rules: {
     "align-comments": ruleAlignComments,
-    "align-consecutive-class-assignements": ruleAlignConsecutiveClassAssignements,
-    "align-one-var": ruleAlignOneVar,
-    "align-requires": ruleAlignRequires,
     "arrow-block-newline": ruleArrowBlockNewline,
     "consecutive-line-comments": ruleConsecutiveLineComments,
     "const": ruleConst,
@@ -113,7 +107,7 @@ const plugin = {
     "no-async": ruleNoAsync,
     "no-extra-line-within-function": ruleNoExtraLineWithinFunction,
     "no-inline-function-in-ternary": ruleNoInlineFunctionInTernary,
-    "no-space-in-optional-arguments": ruleNoSpaceInOptionalArguments,
+    "no-space-in-default-parameters": ruleNoSpaceInDefaultParameters,
     "no-var-in-blocks": ruleNoVarInBlocks,
     "no-useless-template-literals": ruleNoUselessTemplateLiterals,
     "no-short-parameters": ruleNoShortParameters,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-crisp",
-  "version": "1.7.16",
+  "version": "1.7.17",
   "description": "Custom ESLint Rules for Crisp",
   "author": "Crisp IM SAS",
   "main": "index.js",

package/recommended-ts.js

@@ -96,6 +96,7 @@ export default function configRecommendedTS(pluginCrisp) {
         "no-eval": "error",
         "no-implicit-coercion": "error",
         "no-multi-str": "error",
+        "no-multi-spaces": "error",
         "no-multiple-empty-lines": ["error", { max: 1 }],
         "no-mixed-spaces-and-tabs": "error",
         "no-restricted-syntax": [

package/recommended-vue.js

@@ -102,6 +102,7 @@ export default function configRecommendedVue(pluginCrisp) {
         "no-eval": "error",
         "no-implicit-coercion": "error",
         "no-multi-str": "error",
+        "no-multi-spaces": "error",
         "no-multiple-empty-lines": ["error", { max: 1 }],
         "no-mixed-spaces-and-tabs": "error",
         "no-restricted-syntax": [

package/recommended.js

@@ -89,6 +89,7 @@ export default function configRecommended(pluginCrisp) {
             ]
           }
         ],
+        "key-spacing": ["error", { "beforeColon": false, "afterColon": true }],
         "keyword-spacing": "error",
         "linebreak-style": ["error", "unix"],
         "max-len": ["error", {
@@ -108,6 +109,7 @@ export default function configRecommended(pluginCrisp) {
           }
         ],
         "no-multi-str": "error",
+        "no-multi-spaces": "error",
         "no-mixed-spaces-and-tabs": "error",
         "no-tabs": "error",
         "no-trailing-spaces": "error",
@@ -156,9 +158,6 @@ export default function configRecommended(pluginCrisp) {
         // Crisp JS rules
         "crisp/align-comments": "error",
         "crisp/consecutive-line-comments": "error",
-        "crisp/align-consecutive-class-assignements": "error",
-        "crisp/align-one-var": "error",
-        "crisp/align-requires": "error",
         "crisp/arrow-block-newline": "error",
         "crisp/const": "error",
         "crisp/constructor-variables": "error",
@@ -181,8 +180,14 @@ export default function configRecommended(pluginCrisp) {
             "exceptions": ["_", "$"]
           }
         ],
-        "crisp/no-space-in-optional-arguments": "error",
-        "crisp/one-space-after-operator": "error",
+        "crisp/no-space-in-default-parameters": "error",
+        "crisp/one-space-after-operator": [
+          "error",
+
+          {
+            "checkColon": false
+          }
+        ],
         "crisp/regex-in-constructor": "error",
         "crisp/ternary-parenthesis": "error",
         "crisp/two-lines-between-class-members": "error",

package/rules/{no-space-in-optional-arguments.js → no-space-in-default-parameters.js}

@@ -13,7 +13,7 @@ export default {
             ) {
               context.report({
                 node,
-                message: "There should be no space before or after = in optional parameters"
+                message: "There should be no space before or after = in default parameters"
               });
             }
           }

package/tests/rules/{no-space-in-optional-arguments.test.js → no-space-in-default-parameters.test.js}

@@ -1,9 +1,9 @@
-import rule from "../../rules/no-space-in-optional-arguments.js";
+import rule from "../../rules/no-space-in-default-parameters.js";
 import createRuleTester from "../rule-tester.js";
 
 const ruleTester = createRuleTester();
 
-ruleTester.run("no-space-in-optional-arguments", rule, {
+ruleTester.run("no-space-in-default-parameters", rule, {
   valid: [
     "function greet(name=\"World\") {}",
     "const greet = (name=\"World\") => name;",
@@ -16,7 +16,7 @@ ruleTester.run("no-space-in-optional-arguments", rule, {
       code: "function greet(name =\"World\") {}",
       errors: [
         {
-          message: "There should be no space before or after = in optional parameters"
+          message: "There should be no space before or after = in default parameters"
         }
       ]
     },
@@ -24,7 +24,7 @@ ruleTester.run("no-space-in-optional-arguments", rule, {
       code: "const greet = (name= \"World\") => name;",
       errors: [
         {
-          message: "There should be no space before or after = in optional parameters"
+          message: "There should be no space before or after = in default parameters"
         }
       ]
     },
@@ -32,7 +32,7 @@ ruleTester.run("no-space-in-optional-arguments", rule, {
       code: "class Greeter { greet(name = \"World\") {} }",
       errors: [
         {
-          message: "There should be no space before or after = in optional parameters"
+          message: "There should be no space before or after = in default parameters"
         }
       ]
     }

package/rules/align-consecutive-class-assignements.js

@@ -1,76 +0,0 @@
-export default {
-  meta: {
-    type: "problem",
-    docs: {
-      description: "align consecutive assignment statements in a class",
-      category: "Stylistic Issues",
-      recommended: false,
-    },
-    fixable: null,
-    schema: [],
-  },
-
-  create: function (context) {
-    let sourceCode = context.sourceCode;
-    let currentBlock = [];
-    let maxColumn = 0;
-
-    function reportMisalignedAssignments(assignments, maxColumn) {
-      assignments.forEach(node => {
-        let alignmentColumn = sourceCode.getText(node).indexOf("=") + node.loc.start.column;
-        if (alignmentColumn < maxColumn) {
-          context.report({
-            node: node,
-            message: `Assignment of '${sourceCode.getText(node.left).trim()}' at line ${node.loc.start.line} column ${alignmentColumn + 1} should be aligned at column ${maxColumn + 1}`,
-          });
-        }
-      });
-    }
-
-    function handleAssignmentExpression(node) {
-      // Ensure we're only dealing with assignments directly in constructors
-      if (!node.parent || node.parent.type !== "ExpressionStatement" ||
-          !node.parent.parent || node.parent.parent.type !== "BlockStatement" ||
-          !node.parent.parent.parent || node.parent.parent.parent.type !== "FunctionExpression" ||
-          !node.parent.parent.parent.parent || node.parent.parent.parent.parent.type !== "MethodDefinition" ||
-          node.parent.parent.parent.parent.kind !== "constructor") {
-        return;
-      }
-
-      if (currentBlock.length > 0) {
-        const prevNode = currentBlock[currentBlock.length - 1];
-
-        if (prevNode.loc.end.line !== node.loc.start.line - 1) {
-          if (currentBlock.length > 1) {
-            reportMisalignedAssignments(currentBlock, maxColumn);
-          }
-
-          currentBlock = [];
-          maxColumn = 0;
-        }
-      }
-
-      currentBlock.push(node);
-      let currentAssignmentColumn = sourceCode.getText(node).indexOf("=") + node.loc.start.column;
-      maxColumn = Math.max(maxColumn, currentAssignmentColumn);
-    }
-
-
-    return {
-      AssignmentExpression: handleAssignmentExpression,
-      "MethodDefinition:exit": function () {
-        if (currentBlock.length > 1) {
-          reportMisalignedAssignments(currentBlock, maxColumn);
-        }
-
-        currentBlock = [];
-        maxColumn = 0;
-      },
-      "Program:exit": function () {
-        if (currentBlock.length > 1) {
-          reportMisalignedAssignments(currentBlock, maxColumn);
-        }
-      }
-    };
-  },
-};

package/rules/align-one-var.js

@@ -1,70 +0,0 @@
-export default {
-  meta: {
-    type: "layout",
-    docs: {
-      description: "enforce alignment of variables in consecutive 'one-var' statements",
-      category: "Stylistic Issues",
-      recommended: false,
-    },
-  },
-  create(context) {
-    const sourceCode = context.sourceCode;
-
-    return {
-      VariableDeclaration(node) {
-        if (node.declarations.length <= 1) {
-          return;
-        }
-
-        let assignmentOperatorColumn = null;
-        let variableNameColumn = null;
-        let lastDeclarationLine = null;
-
-        for (let i = 0; i < node.declarations.length; i++) {
-          const declaration = node.declarations[i];
-
-          // If the current declaration is on the same line as the last one, \
-          //   skip it
-          if (lastDeclarationLine === declaration.loc.start.line) {
-            continue;
-          }
-
-          lastDeclarationLine = declaration.loc.start.line;
-
-          const variableNameToken = sourceCode.getFirstToken(declaration);
-          const variableNameColumnCurrent = variableNameToken.loc.start.column;
-
-          if (variableNameColumn === null) {
-            variableNameColumn = variableNameColumnCurrent;
-          } else if (variableNameColumnCurrent !== variableNameColumn) {
-            context.report({
-              node: declaration,
-              message: "Misaligned variable declaration."
-            });
-          }
-
-          if (!declaration.init) {
-            continue; // Skip if there's no assignment
-          }
-
-          const equalsSignToken = sourceCode.getTokenBefore(declaration.init);
-
-          if (equalsSignToken.value !== "=") {
-            continue; // Skip if it's not an assignment
-          }
-
-          const equalsSignColumn = equalsSignToken.loc.start.column;
-
-          if (assignmentOperatorColumn === null) {
-            assignmentOperatorColumn = equalsSignColumn;
-          } else if (equalsSignColumn !== assignmentOperatorColumn) {
-            context.report({
-              node: declaration,
-              message: "Misaligned assignment operator."
-            });
-          }
-        }
-      },
-    };
-  },
-};

package/rules/align-requires.js

@@ -1,36 +0,0 @@
-export default {
-  meta: {
-    type: "layout",
-    docs: {
-      description: "enforce alignment of require statements",
-      category: "Stylistic Issues",
-      recommended: false,
-    },
-    fixable: null,  // not auto-fixable
-  },
-  create(context) {
-    return {
-      VariableDeclaration(node) {
-        const declarations = node.declarations;
-
-        // We only check consecutive VariableDeclarator nodes with 'require' init
-        let lastColumn = null;
-
-        for (const declaration of declarations) {
-          if (declaration.init && declaration.init.callee && declaration.init.callee.name === "require") {
-            if (lastColumn === null) {
-              lastColumn = declaration.loc.start.column;
-            } else if (declaration.loc.start.column !== lastColumn) {
-              context.report({
-                node: declaration,
-                message: "Misaligned require statement.",
-              });
-            }
-          } else {
-            lastColumn = null;  // reset for non-require declarations
-          }
-        }
-      },
-    };
-  },
-};

package/tests/rules/align-consecutive-class-assignements.test.js

@@ -1,24 +0,0 @@
-import rule from "../../rules/align-consecutive-class-assignements.js";
-import createRuleTester from "../rule-tester.js";
-
-const ruleTester = createRuleTester();
-
-ruleTester.run("align-consecutive-class-assignements", rule, {
-  valid: [
-    "class Example {\n  constructor() {\n    this.a    = 1;\n    this.long = 2;\n  }\n}",
-    "class Example {\n  constructor() {\n    this.a = 1;\n\n    this.long = 2;\n  }\n}",
-    "class Example {\n  method() {\n    this.a = 1;\n    this.long = 2;\n  }\n}"
-  ],
-
-  invalid: [
-    {
-      code: "class Example {\n  constructor() {\n    this.a = 1;\n    this.long = 2;\n  }\n}",
-      errors: [
-        {
-          message:
-            "Assignment of 'this.a' at line 3 column 12 should be aligned at column 15"
-        }
-      ]
-    }
-  ]
-});

package/tests/rules/align-one-var.test.js

@@ -1,31 +0,0 @@
-import rule from "../../rules/align-one-var.js";
-import createRuleTester from "../rule-tester.js";
-
-const ruleTester = createRuleTester();
-
-ruleTester.run("align-one-var", rule, {
-  valid: [
-    "const first = 1,\n      next  = 2;",
-    "const first = 1, next = 2;",
-    "let first,\n    next;"
-  ],
-
-  invalid: [
-    {
-      code: "const first = 1,\n       next = 2;",
-      errors: [
-        {
-          message: "Misaligned variable declaration."
-        }
-      ]
-    },
-    {
-      code: "const first = 1,\n      next = 2;",
-      errors: [
-        {
-          message: "Misaligned assignment operator."
-        }
-      ]
-    }
-  ]
-});

package/tests/rules/align-requires.test.js

@@ -1,27 +0,0 @@
-import rule from "../../rules/align-requires.js";
-import createRuleTester from "../rule-tester.js";
-
-const ruleTester = createRuleTester({
-  languageOptions: {
-    sourceType: "commonjs"
-  }
-});
-
-ruleTester.run("align-requires", rule, {
-  valid: [
-    "const alpha = require(\"alpha\"),\n      beta = require(\"beta\");",
-    "const alpha = require(\"alpha\");\nconst beta = require(\"beta\");",
-    "const alpha = require(\"alpha\"), value = 1,\n        beta = require(\"beta\");"
-  ],
-
-  invalid: [
-    {
-      code: "const alpha = require(\"alpha\"),\n       beta = require(\"beta\");",
-      errors: [
-        {
-          message: "Misaligned require statement."
-        }
-      ]
-    }
-  ]
-});