feat: add object-curly-newline rule

index.js

@@ -204,7 +204,7 @@
 		radix: 'error',
 
 		// Disabled for now as it causes too much churn
 		// TODO: Enable it in the future when I have time to deal with
 		// the churn and the rule is stable and has an autofixer.
 		// Still doesn't have a fixer as of ESLint 7.24.0.
 		// 'require-unicode-regexp': 'error',
@@ -366,12 +366,12 @@
 			'always',
 			{
 				// Workaround to allow class fields to not have lines between them.
 				// TODO: Get ESLint to add an option to ignore class fields.
 				exceptAfterSingleLine: true,
 			},
 		],
 
 		// TODO: Enable this again when targeting Node.js 16.
 		// 'logical-assignment-operators': [
 		// 	'error',
 		// 	'always',
@@ -420,8 +420,30 @@
 			'error',
 			'never',
 		],
-		// Disabled because of https://github.com/xojs/eslint-config-xo/issues/27
-		// 'object-property-newline': 'error',
+		'object-curly-newline': [
+			'error',
+			{
+				ObjectExpression: {
+					multiline: true,
+					minProperties: 4,
+					consistent: true,
+				},
+				ObjectPattern: {
+					multiline: true,
+					consistent: true,
+				},
+				ImportDeclaration: {
+					multiline: true,
+					minProperties: 4,
+					consistent: true,
+				},
+				ExportDeclaration: {
+					multiline: true,
+					minProperties: 4,
+					consistent: true,
+				},
+			},
+		],
 		'one-var': [
 			'error',
 			'never',
@@ -594,7 +616,7 @@
 			{
 				// `array` is disabled because it forces destructuring on
 				// stupid stuff like `foo.bar = process.argv[2];`
 				// TODO: Open ESLint issue about this
 				VariableDeclarator: {
 					array: false,
 					object: true,
@@ -613,7 +635,7 @@
 		],
 		'prefer-numeric-literals': 'error',
 
 		// TODO: Enable when targeting Node.js 16.
 		// 'prefer-object-has-own': 'error',
 
 		'prefer-rest-params': 'error',