.jscsrc

{
	"requireCurlyBraces": [
		"if", "else", "for", "while", "do", "try",
		"catch"
	],
	"requireSpaceAfterKeywords": [
		"if", "else", "for", "while", "do", "switch",
		"return", "try", "catch"
	],
	"requireSpaceBeforeBlockStatements": true,
	"requireParenthesesAroundIIFE": true,
	"requireSpacesInConditionalExpression": true,
	"requireSpacesInAnonymousFunctionExpression": {
		"beforeOpeningRoundBrace": true,
		"beforeOpeningCurlyBrace": true
	},
	"requireSpacesInNamedFunctionExpression": {
		"beforeOpeningCurlyBrace": true
	},
	"disallowSpacesInNamedFunctionExpression": {
		"beforeOpeningRoundBrace": true
	},
	"requireSpacesInFunctionDeclaration": {
		"beforeOpeningCurlyBrace": true
	},
	"disallowSpacesInFunctionDeclaration": {
		"beforeOpeningRoundBrace": true
	},
	"disallowMultipleVarDecl": true,
	"requireBlocksOnNewline": true,
	"disallowEmptyBlocks": true,
	"requireSpacesInsideObjectBrackets": "allButNested",
	"requireSpacesInsideArrayBrackets": "allButNested",
	"disallowQuotedKeysInObjects": "allButReserved",
	"disallowSpaceAfterObjectKeys": true,
	"requireCommaBeforeLineBreak": true,
	"requireOperatorBeforeLineBreak": true,
	"disallowSpaceAfterPrefixUnaryOperators": true,
	"disallowSpaceBeforePostfixUnaryOperators": true,
	"requireSpaceBeforeBinaryOperators": true,
	"requireSpaceAfterBinaryOperators": true,
	"disallowImplicitTypeConversion": [
		"binary"
	],
	"requireCamelCaseOrUpperCaseIdentifiers": true,
	"disallowKeywords": [
		"with"
	],
	"disallowMultipleLineStrings": true,
	"disallowMultipleLineBreaks": true,
	"validateQuoteMarks": {
		"mark": "'",
		"escape": true
	},
	"validateIndentation": "\t",
	"disallowMixedSpacesAndTabs": "smart",
	"disallowTrailingWhitespace": true,
	"disallowTrailingComma": true,
	"requireKeywordsOnNewLine": [
		"else"
	],
	"requireLineFeedAtFileEnd": true,
	"maximumLineLength": 120,
	"requireCapitalizedConstructors": true,
	"requireDotNotation": true,
	"disallowYodaConditions": true,
	"requireSpaceAfterLineComment": true
}

• After disallowMultipleVarDecl, I'd suggest adding:

"requireBlocksOnNewline": true,
"disallowEmptyBlocks": true,

• I'm on the fence RE allButNested vs. just plain all for requiring spaces inside object/array brackets.
• What is "disallowSpaceBeforeBinaryOperators": [ ":" ] intending to accomplish? (There's already a rule about colons between object keys/values...is this affecting that?)
• You're gonna hate me for this, but after requireSpaceAfterBinaryOperators, I'd suggest adding "disallowImplicitTypeConversion": ["binary"]
• I might suggest removing "disallowMultipleLineBreaks": true. Although it's rare, it can help make things more readable, and it seems like an odd thing to disallow.
• In place of the previous, would we want to consider "validateLineBreaks": "LF" ?
• I'm curious if others agree with escape: true for validateQuoteMarks. I think I'd be willing to agree with it - e.g. single quotes get really annoying really fast for locale strings.
• Based on jscs' examples, I get the impression "disallowMixedSpacesAndTabs": "smart" is somewhat more lenient than we'd like, but I guess there's no better alternative, if true is going to get in the way of jsdoc / dojodoc comments.
• After requireKeywordsOnNewLine, should we add "requireLineFeedAtFileEnd": true? This would match our style guide.
• At end, add "disallowNewlineBeforeBlockStatements": true

A couple more comments:

• Scratch what I said above about allButNested, I see why it's useful now (e.g. [{ ... }]... mysteriously though, it's not working for me locally :/
  • Oh, whoops, allButNested only covers arrays-in-arrays and objects-in-objects. See jscs-dev/node-jscs#111
• Seems to me that requireSpaceAfterLineComment gripes at dojo-doc style comments because apparently it considers tab to not be spacey enough. I had to remove it when testing this against dgrid.
  • Yup... jscs-dev/node-jscs#592

Thanks for the suggestions, Ken. I have updated them all except for the following:

• I'd like more input on "disallowMultipleLineBreaks": true before removing it.
• Since git takes care of line break normalization, I'd rather not add "validateLineBreaks": "LF".

Some other comments:

• The "disallowSpaceBeforeBinaryOperators": [ ":" ] rule was to get around a bug in jscs which has since been fixed.
• The exact reason I chose escape: true for validateQuoteMarks was for locale strings.

Once the two issues you noted get resolved, I think this config file is good to go.