eslint-plugin-brackets

Opinionated ESLint style rules dealing with line breaks after opening brackets and before closing brackets of objects, arrays, conditionals, and functions.

Overview

This project is opinionated, meaning there are no options for any of the rules. However, this may change in the future.

The general goal of these rules is to enforce a coding style that minimizes diffs when items are added to lists (be it arrays, objects, parameter lists, etc.)

For example, we want to avoid this:

// Bad
 
array = ['a',
         'b',
         'c'];
 
array2 = ["this is a long string" +
          "spanning multiple lines"];
 
object = { a: 'a',
           b: 'b' };
 
function foo(bar,
             baz) {
}
 
call('a',
     'b');
 
if (test &&
    test2 &&
    test3) {
}

And instead prefer this:

// Good
 
array = [
  'a',
  'b',
  'c',
];
 
array2 = [
  "this is a long string" +
  "spanning multiple lines"
];
 
object = {
  a: 'a',
  b: 'b'
};
 
function foo(
  bar,
  baz
) {
}
 
call(
  'a',
  'b'
);
 
if (
  test &&
  test2 &&
  test3
) {
}

Why?

• Adding an item to the top of the list or to the bottom of the list will afford smaller diffs.
• Changing the name of the assigned variable/callee won't require shifting every other item in the list to fix indentation.
• There's an argument for symmetry somewhere.

In other words, there shouldn't be items on the same line as the opening bracket or closing bracket enclosing the list if the list spans multiple line.

However, there is an exception to this: if the last token of the line containing the opening bracket is another opening bracket, or if the first token of the line containing the closing bracket is another closing bracket, then that is allowed:

// Fine
 
array = ['a', function() { // ends with an opening bracket
}];
 
call('a', {
  b: 'c',
});
 
if (confirm(
  'are you sure?'
)) {
}

The exception to that is it's not allowed for objects:

// Bad
 
obj = { a: 'b', c: {
  d: 'e',
} };

Preferring this:

// Good
 
obj = {
  a: 'b',
  c: {
    d: 'e',
  },
};

Note on auto-fixing

All rules can be auto-fixed (running eslint with --fix). The fix will add appropriate line breaks after the opening bracket and before the closing bracket at violation sites. This may produce code with bad indentation. We rely on eslint's indent rule to fix the indentation in a second pass.

Currently (as of ESLint 3.10) the indent rule is broken for params inside call expressions and expressions inside conditionals, but this should be fixed soon.

List of rules

• brackets/array-bracket-newline
• brackets/call-parens-newline
• brackets/conditional-parens-newline
• brackets/func-parens-newline
• brackets/object-curly-newline

Take a look at the rule tests to see valid and invalid patterns for each rule.

Installation

You'll first need to install ESLint:

$ npm install eslint

Next, install eslint-plugin-brackets:

$ npm install eslint-plugin-brackets

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

Usage

Add brackets to the plugins section of your .eslintrc configuration file. You can omit the eslint-plugin- prefix:

{
  "plugins": [
    "brackets"
  ]
}

Then configure the rules you want to use under the rules section.

{
  "rules": {
    "brackets/array-bracket-newline": 2,
    "brackets/object-curly-newline": 2,
  }
}

Development

• Install packages with npm install or yarn.
• Run npm run test:watch to continually run the tests

License

MIT