ESLint Error 'Invalid count value' when run with --fix

I couldn’t run the lint test with --fix and I am getting the following error.

Run command

> esw -c .eslintrc ./src '.js, .jsx' --color --fix

Error

Invalid count value
RangeError: Invalid count value
    at String.repeat (native)
    at report (/Users/Projects/uni-sh/fr-webapp/node_modules/eslint/lib/rules/indent.js:267:73)
    at checkLastReturnStatementLineIndent (/Users/Projects/uni-sh/fr-webapp/node_modules/eslint/lib/rules/indent.js:430:17)
    at EventEmitter.ReturnStatement (/Users/Projects/uni-sh/fr-webapp/node_modules/eslint/lib/rules/indent.js:1050:21)
    at emitOne (events.js:101:20)
    at EventEmitter.emit (events.js:188:7)
    at NodeEventGenerator.enterNode (/Users/Projects/uni-sh/fr-webapp/node_modules/eslint/lib/util/node-event-generator.js:40:22)
    at CodePathAnalyzer.enterNode (/Users/Projects/uni-sh/fr-webapp/node_modules/eslint/lib/code-path-analysis/code-path-analyzer.js:608:23)
    at CommentEventGenerator.enterNode (/Users/Projects/uni-sh/fr-webapp/node_modules/eslint/lib/util/comment-event-generator.js:97:23)
    at Controller.enter (/Users/Projects/uniqlo-sh/fr-webapp/node_modules/eslint/lib/eslint.js:927:36)

I have resolved the issue. If there is no blank line before the return statement in anywhere in the code we will get this error message.

I have just put a blank line before the return statement then lint command with --fix is working without any issue

[btmills]

I was able to narrow down a repro:

// .eslintrc.js
module.exports = {
    "parserOptions": {
        "ecmaFeatures": {
            "globalReturn": true
        }
    },
    "rules": {
        "indent": ["error", 2]
    }
};

// test.js
return (
  42
  );

$ npm ls --depth 0
~/Desktop/7573
└── eslint@3.10.0

$ node_modules/.bin/eslint test.js
Invalid count value
RangeError: Invalid count value
    at String.repeat (native)
    at report (~/Desktop/7573/node_modules/eslint/lib/rules/indent.js:267:73)
    at checkLastReturnStatementLineIndent (~/Desktop/7573/node_modules/eslint/lib/rules/indent.js:430:17)
    at EventEmitter.ReturnStatement (~/Desktop/7573/node_modules/eslint/lib/rules/indent.js:1048:21)
    at emitOne (events.js:96:13)
    at EventEmitter.emit (events.js:188:7)
    at NodeEventGenerator.enterNode (~/Desktop/7573/node_modules/eslint/lib/util/node-event-generator.js:40:22)
    at CodePathAnalyzer.enterNode (~/Desktop/7573/node_modules/eslint/lib/code-path-analysis/code-path-analyzer.js:608:23)
    at CommentEventGenerator.enterNode (~/Desktop/7573/node_modules/eslint/lib/util/comment-event-generator.js:97:23)
    at Controller.enter (~/Desktop/7573/node_modules/eslint/lib/eslint.js:925:36)

[not-an-aardvark]

The code is able to remove indentations as a fix. The issue is that the desired indent is coming out as a negative number for some reason. The correct behavior would be for the desired indent size to be 0, which would cause desiredIndent to be an empty string.

For what it’s worth, I think this code has the correct behavior:

return (
  42
  ) // <-- same as above example, except no semicolon

So I’m guessing there is an off-by-one error somewhere caused by the semicolon after the closing paren.