Merge pull request #7 from paleite/feature/add-ci-mode

Feature/add ci mode

Author: paleite

.husky/.gitignore

@@ -1 +1 @@
-_
+_

package.json

@@ -40,17 +40,17 @@
     "@types/eslint": "^7.2.6",
     "@types/jest": "^26.0.20",
     "@types/node": "^14.14.31",
-    "@typescript-eslint/eslint-plugin": "^4.15.1",
-    "@typescript-eslint/parser": "^4.15.1",
+    "@typescript-eslint/eslint-plugin": "^4.15.2",
+    "@typescript-eslint/parser": "^4.15.2",
     "eslint": "^7.0.0",
     "eslint-config-prettier": "^8.0.0",
     "eslint-plugin-import": "^2.22.1",
     "eslint-plugin-promise": "^4.3.1",
-    "husky": "^5.1.0",
+    "husky": "^5.1.1",
     "jest": "^26.6.3",
     "lint-staged": "^10.5.4",
     "np": "^7.4.0",
-    "pinst": "^2.1.4",
+    "pinst": "^2.1.6",
     "prettier": "^2.2.1",
     "ts-jest": "^26.5.1",
     "typescript": "^4.1.5"

src/git.ts

@@ -18,13 +18,20 @@ const diffCache = new Map<string, string>();
 const getDiffForFile = (filePath: string, staged = false): string => {
   let diff = getCachedDiff(filePath, staged);
   if (diff === undefined) {
-    const result = child_process
-      .execSync(
-        `git diff --diff-filter=ACM --unified=0 HEAD ${
-          staged ? " --staged" : ""
-        } -- ${sanitizeFilePath(filePath)}`
-      )
-      .toString();
+    const command = [
+      "git",
+      "diff",
+      "--diff-filter=ACM",
+      staged && "--staged",
+      "--unified=0",
+      JSON.stringify(process.env.ESLINT_PLUGIN_DIFF_COMMIT) ?? "HEAD",
+      "--",
+      sanitizeFilePath(filePath),
+    ]
+      .filter(Boolean)
+      .join(" ");
+
+    const result = child_process.execSync(command).toString();
     setCachedDiff(filePath, staged, result);
     diff = result;
   }
@@ -35,12 +42,19 @@ const getDiffForFile = (filePath: string, staged = false): string => {
 let diffFileListCache: string[];
 const getDiffFileList = (staged = false): string[] => {
   if (diffFileListCache === undefined) {
+    const command = [
+      "git",
+      "diff",
+      "--diff-filter=ACM",
+      "--name-only",
+      staged && "--staged",
+      JSON.stringify(process.env.ESLINT_PLUGIN_DIFF_COMMIT) ?? "HEAD",
+    ]
+      .filter(Boolean)
+      .join(" ");
+
     diffFileListCache = child_process
-      .execSync(
-        `git diff --diff-filter=ACM HEAD --name-only ${
-          staged ? "--staged" : ""
-        }`
-      )
+      .execSync(command)
       .toString()
       .trim()
       .split("\n")

src/processors.ts

@@ -1,6 +1,6 @@
-import type { Range } from "./git";
-import { getDiffForFile, getRangesForDiff, getDiffFileList } from "./git";
 import type { Linter } from "eslint";
+import type { Range } from "./git";
+import { getDiffFileList, getDiffForFile, getRangesForDiff } from "./git";
 
 const STAGED = true;
 
@@ -18,15 +18,15 @@ const diff = {
     messages: Linter.LintMessage[][],
     filename: string
   ): Linter.LintMessage[] =>
-    ([] as Linter.LintMessage[]).concat(
-      ...messages.map((message) =>
+    messages
+      .map((message) =>
         message.filter(({ line }) =>
           getRangesForDiff(getDiffForFile(filename)).some(
             isLineWithinRange(line)
           )
         )
       )
-    ),
+      .reduce((a, b) => a.concat(b), []),
 
   supportsAutofix: true,
 };
@@ -52,15 +52,16 @@ const staged = {
     messages: Linter.LintMessage[][],
     filename: string
   ): Linter.LintMessage[] =>
-    ([] as Linter.LintMessage[]).concat(
-      ...messages.map((message) =>
+    messages
+      .map((message) =>
         message.filter(({ line }) =>
           getRangesForDiff(getDiffForFile(filename, STAGED)).some(
             isLineWithinRange(line)
           )
         )
       )
-    ),
+      .reduce((a, b) => a.concat(b), []),
+
   supportsAutofix: true,
 };