Disallow spacing in object curlies or brackets

.eslintrc.js

@@ -14,7 +14,9 @@ module.exports = {
   },
   plugins: ['ava', 'import', 'node-globals', 'react', 'unicorn'],
   rules: {
+    'array-bracket-spacing': ['error', 'never'],
     'arrow-parens': ['error', 'as-needed'],
+    'computed-property-spacing': ['error', 'never'],
     'import/default': 'error',
     'import/extensions': ['error', {
       js: 'never',
@@ -47,6 +49,7 @@ module.exports = {
     }],
     'no-var': 'error',
     'no-warning-comments': 'warn',
+    'object-curly-spacing': ['error', 'never'],
     'object-shorthand': 'error',
     'prefer-const': 'error',
     'promise/no-return-in-finally': 'error',
@@ -73,6 +76,9 @@ module.exports = {
     'react/no-will-update-set-state': 'error',
     'react/require-render-return': 'error',
     'react/void-dom-elements-no-children': 'error',
+    'standard/array-bracket-even-spacing': 'off',
+    'standard/computed-property-even-spacing': 'off',
+    'standard/object-curly-even-spacing': 'off',
     'unicorn/catch-error-name': 'error',
     'unicorn/explicit-length-check': 'error',
     'unicorn/no-abusive-eslint-disable': 'error',

test/api.js

@@ -1,4 +1,4 @@
-import { resolve as resolvePath } from 'path'
+import {resolve as resolvePath} from 'path'
 
 import test from 'ava'
 
@@ -50,7 +50,7 @@ test.cb('works with invalid files', t => {
               endColumn: 16,
               nodeType: 'Literal',
               source: '  return "BAR:" + bar.toUpperCase()',
-              fix: { range: [32, 38], text: '\'BAR:\'' }
+              fix: {range: [32, 38], text: '\'BAR:\''}
             },
             {
               ruleId: 'semi',
@@ -60,7 +60,7 @@ test.cb('works with invalid files', t => {
               column: 2,
               nodeType: 'ExpressionStatement',
               source: '};',
-              fix: { range: [59, 61], text: '}' }
+              fix: {range: [59, 61], text: '}'}
             }
           ],
           errorCount: 2,

test/cli.js

@@ -1,5 +1,5 @@
-import { fork } from 'child_process'
-import { resolve as resolvePath } from 'path'
+import {fork} from 'child_process'
+import {resolve as resolvePath} from 'path'
 
 import test from 'ava'
 import getStream from 'get-stream'

test/patches/file-result-output.js

@@ -1,4 +1,4 @@
-import { join } from 'path'
+import {join} from 'path'
 import test from 'ava'
 import engine from 'standard-engine'
 import patchFileResultOutput from '../../lib/patch-file-result-output'
@@ -14,7 +14,7 @@ test.beforeEach(t => {
     stderr: ''
   }
 
-  patchFileResultOutput({ linter: Linter }, {
+  patchFileResultOutput({linter: Linter}, {
     error (output) {
       t.context.stderr += output
     },
@@ -28,7 +28,7 @@ test.beforeEach(t => {
 
 async function checkOutput (t, out, ...args) {
   const other = out === 'stderr' ? 'stdout' : 'stderr'
-  const { results } = await new Promise(resolve => {
+  const {results} = await new Promise(resolve => {
     t.context.linter.lintFiles(...args, (_, result) => resolve(result))
   })
 
@@ -56,7 +56,7 @@ test('patched lintFiles() propagates errors', async t => {
     }
   }
 
-  patchFileResultOutput({ linter: Linter })
+  patchFileResultOutput({linter: Linter})
   const linter = new Linter(Object.assign({}, options))
   const err = await t.throws(new Promise((resolve, reject) => {
     linter.lintFiles([fixture], null, reject)

test/patches/import-rule.js

@@ -7,7 +7,7 @@ import applySettings from '../../lib/settings'
 
 const NOT_PATCHED = new Error('Throw when unpatched code is run')
 for (const rule of [extensionsRule, noExtraneousDependenciesRule]) {
-  const { create } = rule
+  const {create} = rule
   rule.create = function (context) {
     const base = create.call(this, context)
     return Object.assign(base, {
@@ -19,23 +19,23 @@ for (const rule of [extensionsRule, noExtraneousDependenciesRule]) {
 
 function makeStaticRequire (value) {
   return {
-    callee: { type: 'Identifier', name: 'require' },
-    arguments: [ { type: 'Literal', value } ]
+    callee: {type: 'Identifier', name: 'require'},
+    arguments: [{type: 'Literal', value}]
   }
 }
 
 function checkPatched (t, rule) {
-  applySettings({}, { fakeDependencies: ['^fake', '^reallyFake'] }, '/')
+  applySettings({}, {fakeDependencies: ['^fake', '^reallyFake']}, '/')
 
   const instance = rule.create({
     getFilename () { return 'test.js' },
     options: []
   })
   {
-    t.notThrows(() => instance.ImportDeclaration({ source: { value: 'fake' } }))
-    t.notThrows(() => instance.ImportDeclaration({ source: { value: 'reallyFake' } }))
-    t.notThrows(() => instance.ImportDeclaration({ source: { value: 'faker' } }))
-    const err = t.throws(() => instance.ImportDeclaration({ source: { value: 'real' } }))
+    t.notThrows(() => instance.ImportDeclaration({source: {value: 'fake'}}))
+    t.notThrows(() => instance.ImportDeclaration({source: {value: 'reallyFake'}}))
+    t.notThrows(() => instance.ImportDeclaration({source: {value: 'faker'}}))
+    const err = t.throws(() => instance.ImportDeclaration({source: {value: 'real'}}))
     t.true(err === NOT_PATCHED)
   }
 

test/settings.js

@@ -1,12 +1,12 @@
-import { join } from 'path'
+import {join} from 'path'
 
 import test from 'ava'
 
 import applySettings from '../lib/settings'
 
 test('uses existing rules object', t => {
   const rules = {}
-  const eslintConfig = { rules }
+  const eslintConfig = {rules}
 
   applySettings(eslintConfig, {}, '/')
   t.true(eslintConfig.rules === rules)
@@ -34,7 +34,7 @@ test('overrides import/no-extraneous-dependencies with default devDependencies',
 
 test('supports allowDevDependencies with string value', t => {
   const eslintConfig = {}
-  applySettings(eslintConfig, { allowDevDependencies: 'foo.js' }, '/root')
+  applySettings(eslintConfig, {allowDevDependencies: 'foo.js'}, '/root')
 
   t.deepEqual(eslintConfig.rules['import/no-extraneous-dependencies'], ['error', {
     devDependencies: ['/root/foo.js']
@@ -43,7 +43,7 @@ test('supports allowDevDependencies with string value', t => {
 
 test('supports allowDevDependencies with array value', t => {
   const eslintConfig = {}
-  applySettings(eslintConfig, { allowDevDependencies: ['foo.js', 'bar.js'] }, '/root')
+  applySettings(eslintConfig, {allowDevDependencies: ['foo.js', 'bar.js']}, '/root')
 
   t.deepEqual(eslintConfig.rules['import/no-extraneous-dependencies'], ['error', {
     devDependencies: ['/root/foo.js', '/root/bar.js']
@@ -52,7 +52,7 @@ test('supports allowDevDependencies with array value', t => {
 
 test('supports fakeDependencies with string value', t => {
   const eslintConfig = {}
-  applySettings(eslintConfig, { fakeDependencies: 'foo.js' }, '/root')
+  applySettings(eslintConfig, {fakeDependencies: 'foo.js'}, '/root')
 
   t.deepEqual(eslintConfig.rules['import/no-unresolved'], ['error', {
     commonjs: true,
@@ -62,7 +62,7 @@ test('supports fakeDependencies with string value', t => {
 
 test('supports fakeDependencies with array value', t => {
   const eslintConfig = {}
-  applySettings(eslintConfig, { fakeDependencies: ['foo.js', 'bar.js'] }, '/root')
+  applySettings(eslintConfig, {fakeDependencies: ['foo.js', 'bar.js']}, '/root')
 
   t.deepEqual(eslintConfig.rules['import/no-unresolved'], ['error', {
     commonjs: true,
@@ -74,7 +74,7 @@ const RESOLVERS_DIR = join(__dirname, 'fixtures', 'resolvers')
 
 test('supports resolvers — resolves string values from the rootDir', t => {
   const eslintConfig = {}
-  const settings = { resolvers: 'foo' }
+  const settings = {resolvers: 'foo'}
   applySettings(eslintConfig, settings, RESOLVERS_DIR)
 
   t.deepEqual(eslintConfig.baseConfig, {
@@ -106,7 +106,7 @@ test('supports resolvers — resolves object keys from the rootDir', t => {
 
 test('supports resolvers — throws if values cannot be resolved', t => {
   const eslintConfig = {}
-  const settings = { resolvers: 'foo' }
+  const settings = {resolvers: 'foo'}
 
   const err = t.throws(() => applySettings(eslintConfig, settings, '/root'))
   t.true(err.message === 'Could not resolve \'foo\' import resolver')