Sort failures by failure's line and character

src/linter.ts

@@ -131,7 +131,9 @@ class Linter {
             throw new Error(`formatter '${formatterName}' not found`);
         }
 
-        const output = formatter.format(this.failures, this.fixes);
+        // Sort failures by failure's line and character.
+        const failures = this.sortFailures(this.failures);
+        const output = formatter.format(failures, this.fixes);
 
         const errorCount = this.failures.filter((failure) => failure.getRuleSeverity() === "error").length;
         return {
@@ -144,6 +146,24 @@ class Linter {
         };
     }
 
+    public sortFailures(failures: RuleFailure[]): RuleFailure[] {
+        let sortedFailures: RuleFailure[] = [];
+
+        const fileFailuresMap = createMultiMap(failures, (failure) => [failure.getFileName(), failure]);
+        fileFailuresMap.forEach((fileFailures) => {
+            sortedFailures = sortedFailures.concat(fileFailures.sort((failureA, failureB) => {
+                const failureALineAndCharacter = failureA.getStartPosition().getLineAndCharacter();
+                const failureBLineAndCharacter = failureB.getStartPosition().getLineAndCharacter();
+                if (failureALineAndCharacter.line === failureBLineAndCharacter.line) {
+                    return failureALineAndCharacter.character - failureBLineAndCharacter.character;
+                }
+                return failureALineAndCharacter.line - failureBLineAndCharacter.line;
+            }));
+        });
+
+        return sortedFailures;
+    }
+
     private getAllFailures(sourceFile: ts.SourceFile, enabledRules: IRule[]): RuleFailure[] {
         const failures = flatMap(enabledRules, (rule) => this.applyRule(rule, sourceFile));
         return removeDisabledFailures(sourceFile, failures);