I ran ESLint v6.3.0 locally, and ran eslint --fix on the ai/petra/ directory to automatically fix all violations of the ESLint rules in our eslintrc.json file that could be automatically fixed.
There were also a number of warnings that could not be fixed automatically, which were factored out in D2070 and D2103 and merged first.
This means there should now be 0 ESLint violations in the ai/petra directory.
In addition to the automatic fixes, this commit also two lines of commented out code that were oddly auto-formatted due to being commented out inline instead of as whole statements. Rather than fixing the indentation to adhere to ESLInt, I removed it in AttackPlan#checkTargetObstruction. It had been commented out from its first commit (r18257), and didn't seem to have an obvious purpose.