Skip to content

Latest commit

 

History

History
174 lines (136 loc) · 8.32 KB

File metadata and controls

174 lines (136 loc) · 8.32 KB

build npm

eslint-plugin-sort-class-members

ESLint rule for enforcing consistent ES6 class member order.

Installation

Install ESLint and eslint-plugin-sort-class-members:

$ npm install eslint eslint-plugin-sort-class-members --save-dev

Note: If you installed ESLint globally (using the -g flag) then you must also install eslint-plugin-sort-class-members globally.

Usage

Add sort-class-members to the plugins section of your .eslintrc configuration file, and configure the rule under the rules section.

{
	"plugins": ["sort-class-members"],
	"rules": {
		"sort-class-members/sort-class-members": [
			2,
			{
				"order": [
					"[static-properties]",
					"[static-methods]",
					"[properties]",
					"[conventional-private-properties]",
					"constructor",
					"[methods]",
					"[conventional-private-methods]"
				],
				"accessorPairPositioning": "getThenSet"
			}
		]
	}
}

Or, using ESLint's Flat Config (eslint.config.js):

import sortClassMembers from "eslint-plugin-sort-class-members";

export default [
	// Use the default configuration shown above.
	sortClassMembers.configs["flat/recommended"],

	rules: {
		// Customize specific configuration properties
		"sort-class-members/sort-class-members": [
			2,
			{
				"accessorPairPositioning": "together",
				"stopAfterFirstProblem": true
			}
		]
  }
];

When using the default configuration (shown above), the following patterns are considered problems:

class Foo {
	b = 'bar';

	c() {}

	constructor() {} // error Expected constructor to come before method c

	static a() {} // error Expected static method a to come before property b
}

When using the default configuration (shown above), the following patterns are not considered problems:

class Foo {
	static a() {}

	b = 'bar';

	constructor() {}

	c() {}
}

Configuration

The rule accepts the following configuration properties:

  • order: Used to specify the expected sort order of class members.
  • groups: May optionally be used to created customized named groups of members so that order can be more easily maintained. Groups can be referenced by name by using square brackets. E.g., "[group-name]".
  • accessorPairPositioning: Used to specify the required positioning of get/set pairs. Available values: getThenSet, setThenGet, together, any.
  • stopAfterFirstProblem: Only report the first sort problem in each class (plus the number of problems found). Useful if you only want to know that the class has sort problems without spamming error messages. The default is false.
  • locale: Used to specify the locale for the name comparison method. The default is en-US.
  • sortInterfaces: Used to specify whether to sort TypeScript interfaces as well. The default is false.
{
  "order": [
    "constructor",
    "[event-handlers]", // reference the custom group defined in the "groups" property
    "[everything-else]" // a few groups are provided by default (see list below)
  ],
  "groups": {
    "event-handlers": [{ "name": "/on.+/", "type": "method" }]
  },
  "accessorPairPositioning": "getThenSet",
  "stopAfterFirstProblem": false
}

Members can be matched to positional slots using several criteria, including name (exact match or regexp), member type (method or property), and whether or not the member is static. Each match slot is described by an object with six properties, all of which are optional.

  • name: a string matching the name of the member. If the string starts and ends with / it will be interpreted as a regular expression. E.g., "/_.+/" will match members whose name starts with an underscore.
  • type: "method"|"property". Note: Class properties currently require a custom parser like babel-eslint.
  • kind: "get"|"set"|"accessor"|"nonAccessor". A subtype of type: "method" that can match getter or setter methods. "accessor" matches both getters and setters, while "nonAccessor" matches methods that are neither getters or setters.
  • propertyType: A subtype of type: "property" that can match the type of the property value. e.g., propertyType: "ArrowFunctionExpression" to match properties whose value is initialized to an arrow function.
  • accessorPair: true|false. True to match only getters and setters that are part of a pair. i.e., only those that have both get and set methods defined.
  • static: true|false to restrict the match to static or instance members.
  • private: true|false to restrict the match to private members. Note: Private members currently require a custom parser like babel-eslint.
  • accessibility: "public"|"private"|"protected" to restrict the match to members with the specified typescript accessibility modifier. Note: Requires @typescript-eslint/parser.
  • abstract: true|false to restrict the match to members with typescript abstract keyword.
  • override: true|false to restrict the match to members with typescript override keyword.
  • readonly: true|false to restrict the match to members with typescript readonly keyword.
  • async: true|false to restrict the match to async members.
  • sort: "alphabetical"|"none". Used to require a specific sorting within the slot for matched members. Defaults to "none".
  • groupByDecorator: string|true|false. A boolean used to indicate whether a property/method should have a decorator or a string used to group properties/methods with the same decorator name (e.g. observable for @observable). If the string starts and ends with / it will be interpreted as a regular expression. E.g., "/_.+/" will match members that have a decorator that starts with an underscore. Can be used together with sort. Note: Decorators are a Stage 2 proposal and require a custom parser like babel-eslint.

A few examples:

  • { "name": "create", "type": "method", "static": true } would match a static method named create.
  • { "static": true } would match all static methods and properties.
  • { "type": "method", "private": true } would match all private methods.
  • { "async": true } would match all async methods.
  • { "name": "/on.+/", "type": "method" } would match both static and instance methods whose names start with "on".
  • "/on.+/" is shorthand for { "name": "/on.+/" }, and would match all static and instance methods and properties whose names start with "on".
  • { "type": "method", "sort": "alphabetical" } would match all methods, and enforce an alphabetical sort.

Note: You can simply use a string if you only want to match on the name.

The following groups are provided by default:

  • [properties]: matches all properties
  • [getters]: matches all getter methods
  • [setters]: matches all setter methods
  • [accessor-pairs]: matches getters and setters that are part of a pair (where both get and set methods are defined)
  • [static-properties]: matches all static properties
  • [conventional-private-properties]: matches properties whose name starts with an underscore
  • [arrow-function-properties]: matches properties whose value is initialized to an arrow function
  • [methods]: matches all methods
  • [static-methods]: matches all static methods
  • [async-methods]: matches all async methods
  • [conventional-private-methods]: matches methods whose name starts with an underscore
  • [everything-else]: matches all class members not matched by any other rule

NOTE: Currently only class properties using the proposed syntax are matched by "type": "property". Properties added via assignment are not considered by this rule.

Acknowledgements

Inspired by the sort-comp rule from eslint-plugin-react.