List untracked files

Author: paleite

src/git.test.ts

@@ -6,6 +6,7 @@ import {
   getDiffForFile,
   getGitFileList,
   getRangesForDiff,
+  getUntrackedFileList,
   hasCleanIndex,
 } from "./git";
 import {
@@ -118,6 +119,28 @@ describe("getDiffFileList", () => {
   });
 });
 
+describe("getUntrackedFileList", () => {
+  it("should get the list of untracked files", () => {
+    jest.mock("child_process").resetAllMocks();
+    mockedChildProcess.execSync.mockReturnValueOnce(Buffer.from(diffFileList));
+    expect(mockedChildProcess.execSync).toHaveBeenCalledTimes(0);
+    const fileListA = getUntrackedFileList();
+    const staged = false;
+    const fileListB = getUntrackedFileList(staged);
+
+    expect(mockedChildProcess.execSync).toHaveBeenCalledTimes(1);
+    expect(fileListA).toEqual(
+      ["file1", "file2", "file3"].map((p) => path.resolve(p))
+    );
+    expect(fileListA).toEqual(fileListB);
+  });
+
+  it("should not get a list when looking when using staged", () => {
+    const staged = true;
+    expect(getUntrackedFileList(staged)).toEqual([]);
+  });
+});
+
 describe("getGitFileList", () => {
   it("should get the list of committed files", () => {
     mockedChildProcess.execSync.mockReturnValueOnce(Buffer.from(diffFileList));

src/git.ts

@@ -104,21 +104,19 @@ const hasCleanIndex = (filePath: string): boolean => {
 
 let untrackedFileListCache: string[] | undefined;
 const getUntrackedFileList = (staged = false): string[] => {
-  if (untrackedFileListCache === undefined) {
+  if (staged) {
+    untrackedFileListCache = [];
+  } else if (untrackedFileListCache === undefined) {
     const command = ["git", "ls-files", "--exclude-standard", "--others"]
       .filter(Boolean)
       .join(" ");
 
-    if (staged === false) {
-      untrackedFileListCache = child_process
-        .execSync(command)
-        .toString()
-        .trim()
-        .split("\n")
-        .map((filePath) => path.resolve(filePath));
-    } else {
-      untrackedFileListCache = [];
-    }
+    untrackedFileListCache = child_process
+      .execSync(command)
+      .toString()
+      .trim()
+      .split("\n")
+      .map((filePath) => path.resolve(filePath));
   }
   return untrackedFileListCache;
 };

src/processors.ts

@@ -1,6 +1,7 @@
 import type { Linter } from "eslint";
 import type { Range } from "./git";
 import {
+  getUntrackedFileList,
   getDiffFileList,
   getDiffForFile,
   getRangesForDiff,
@@ -19,7 +20,9 @@ const STAGED = true;
 const getPreProcessor =
   (staged = false) =>
   (text: string, filename: string) => {
-    const shouldBeProcessed = getDiffFileList(staged).includes(filename);
+    const shouldBeProcessed =
+      getDiffFileList(staged).includes(filename) ||
+      getUntrackedFileList(staged).includes(filename);
 
     return shouldBeProcessed ? [text] : [];
   };