eslint-plugin: Enforce object-shorthand

docs/tool/manifest.js

@@ -91,10 +91,10 @@ function generateRootManifestFromTOCItems( items, parent = null ) {
 		}
 
 		pageItems.push( {
-			title: title,
-			slug: slug,
+			title,
+			slug,
 			markdown_source: `${ baseRepoUrl }\/${ fileName }`,
-			parent: parent,
+			parent,
 		} );
 		if ( Array.isArray( children ) && children.length ) {
 			pageItems = pageItems.concat( generateRootManifestFromTOCItems( children, slug ) );

packages/block-library/src/classic/edit.js

@@ -128,7 +128,7 @@ export default class ClassicEdit extends Component {
 		editor.addButton( 'kitchensink', {
 			tooltip: _x( 'More', 'button to expand options' ),
 			icon: 'dashicon dashicons-editor-kitchensink',
-			onClick: function() {
+			onClick() {
 				const button = this;
 				const active = ! button.active();
 

packages/block-library/src/html/index.js

@@ -59,7 +59,7 @@ export const settings = {
 		],
 	},
 
-	edit: edit,
+	edit,
 
 	save( { attributes } ) {
 		return <RawHTML>{ attributes.content }</RawHTML>;

packages/block-library/src/paragraph/index.js

@@ -174,9 +174,9 @@ export const settings = {
 					'has-drop-cap': dropCap,
 				} );
 				const styles = {
-					backgroundColor: backgroundColor,
+					backgroundColor,
 					color: textColor,
-					fontSize: fontSize,
+					fontSize,
 					textAlign: align,
 				};
 

packages/block-library/src/verse/index.js

@@ -76,7 +76,7 @@ export const settings = {
 							content: nextContent,
 						} );
 					} }
-					style={ { textAlign: textAlign } }
+					style={ { textAlign } }
 					placeholder={ __( 'Write…' ) }
 					wrapperClassName={ className }
 					onMerge={ mergeBlocks }
@@ -91,7 +91,7 @@ export const settings = {
 		return (
 			<RichText.Content
 				tagName="pre"
-				style={ { textAlign: textAlign } }
+				style={ { textAlign } }
 				value={ content }
 			/>
 		);

packages/components/src/button/index.native.js

@@ -37,16 +37,16 @@ const styles = StyleSheet.create( {
 		fontWeight: 'bold',
 		fontSize: 13,
 		alignSelf: 'flex-end',
-		marginLeft: marginLeft,
-		marginBottom: marginBottom,
+		marginLeft,
+		marginBottom,
 	},
 	subscriptActive: {
 		color: 'white',
 		fontWeight: 'bold',
 		fontSize: 13,
 		alignSelf: 'flex-end',
-		marginLeft: marginLeft,
-		marginBottom: marginBottom,
+		marginLeft,
+		marginBottom,
 	},
 } );
 

packages/components/src/color-palette/index.js

@@ -27,7 +27,7 @@ export default function ColorPalette( { colors, disableCustomColors = false, val
 	return (
 		<div className={ classes }>
 			{ map( colors, ( { color, name } ) => {
-				const style = { color: color };
+				const style = { color };
 				const itemClasses = classnames( 'components-color-palette__item', { 'is-active': value === color } );
 
 				return (

packages/components/src/form-token-field/test/index.js

@@ -49,7 +49,7 @@ describe( 'FormTokenField', function() {
 		TestUtils.Simulate.keyDown(
 			wrapperElement(),
 			{
-				keyCode: keyCode,
+				keyCode,
 				shiftKey: ! ! shiftKey,
 			}
 		);
@@ -58,7 +58,7 @@ describe( 'FormTokenField', function() {
 	function sendKeyPress( charCode ) {
 		TestUtils.Simulate.keyPress(
 			wrapperElement(),
-			{ charCode: charCode }
+			{ charCode }
 		);
 	}
 

packages/edit-post/src/components/header/index.js

@@ -92,7 +92,7 @@ export default compose(
 
 		return {
 			openGeneralSidebar: () => openGeneralSidebar( getBlockSelectionStart() ? 'edit-post/block' : 'edit-post/document' ),
-			closeGeneralSidebar: closeGeneralSidebar,
+			closeGeneralSidebar,
 		};
 	} ),
 )( Header );

packages/editor/src/components/block-draggable/index.js

@@ -22,8 +22,8 @@ const BlockDraggable = ( { children, clientId, rootClientId, blockElementId, ind
 			{
 				( { onDraggableStart, onDraggableEnd } ) => {
 					return children( {
-						onDraggableStart: onDraggableStart,
-						onDraggableEnd: onDraggableEnd,
+						onDraggableStart,
+						onDraggableEnd,
 					} );
 				}
 			}

packages/editor/src/components/inserter/test/menu.js

@@ -93,7 +93,7 @@ const items = [
 
 const DEFAULT_PROPS = {
 	position: 'top center',
-	items: items,
+	items,
 	debouncedSpeak: noop,
 	fetchReusableBlocks: noop,
 	setTimeout: noop,

packages/editor/src/editor-styles/ast/parse.js

@@ -36,7 +36,7 @@ export default function( css, options ) {
    */
 
 	function position() {
-		const start = { line: lineno, column: column };
+		const start = { line: lineno, column };
 		return function( node ) {
 			node.position = new Position( start );
 			whitespace();
@@ -50,7 +50,7 @@ export default function( css, options ) {
 
 	function Position( start ) {
 		this.start = start;
-		this.end = { line: lineno, column: column };
+		this.end = { line: lineno, column };
 		this.source = options.source;
 	}
 
@@ -351,8 +351,8 @@ export default function( css, options ) {
 
 		return pos( {
 			type: 'keyframes',
-			name: name,
-			vendor: vendor,
+			name,
+			vendor,
 			keyframes: frames,
 		} );
 	}
@@ -382,7 +382,7 @@ export default function( css, options ) {
 
 		return pos( {
 			type: 'supports',
-			supports: supports,
+			supports,
 			rules: style,
 		} );
 	}
@@ -440,7 +440,7 @@ export default function( css, options ) {
 
 		return pos( {
 			type: 'media',
-			media: media,
+			media,
 			rules: style,
 		} );
 	}
@@ -527,7 +527,7 @@ export default function( css, options ) {
 		return pos( {
 			type: 'document',
 			document: doc,
-			vendor: vendor,
+			vendor,
 			rules: style,
 		} );
 	}

packages/eslint-plugin/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 2.0.0 (Unreleased)
+
+### Breaking Changes
+
+- The `esnext` and `recommended` rulesets now enforce [`object-shorthand`](https://eslint.org/docs/rules/object-shorthand)
+
 ## 1.0.0 (2018-12-12)
 
 ### New Features

packages/eslint-plugin/configs/esnext.js

@@ -23,6 +23,7 @@ module.exports = {
 		'no-useless-computed-key': 'error',
 		'no-useless-constructor': 'error',
 		'no-var': 'error',
+		'object-shorthand': 'error',
 		'prefer-const': 'error',
 		quotes: [ 'error', 'single', { allowTemplateLiterals: true, avoidEscape: true } ],
 		'space-unary-ops': [ 'error', {

packages/hooks/src/createHooks.js

@@ -34,8 +34,8 @@ function createHooks() {
 		doingFilter: createDoingHook( filters ),
 		didAction: createDidHook( actions ),
 		didFilter: createDidHook( filters ),
-		actions: actions,
-		filters: filters,
+		actions,
+		filters,
 	};
 }
 

packages/rich-text/src/create.js

@@ -124,7 +124,7 @@ export function create( {
 	if ( typeof text === 'string' && text.length > 0 ) {
 		return {
 			formats: Array( text.length ),
-			text: text,
+			text,
 		};
 	}