Merge pull request #27 from IsLand-x/main

fix: Refresh untracked file list once a new file is created.

Author: paleite

.gitignore

@@ -1,3 +1,4 @@
 *.tsbuildinfo
 /coverage/
 /dist/
+node_modules

src/git.ts

@@ -77,19 +77,25 @@ const fetchFromOrigin = (branch: string) => {
   child_process.execFileSync(COMMAND, args, OPTIONS);
 };
 
-const getUntrackedFileList = (staged = false): string[] => {
+let untrackedFileListCache: string[] | undefined;
+const getUntrackedFileList = (
+  staged = false,
+  shouldRefresh = false
+): string[] => {
   if (staged) {
     return [];
   }
 
-  const args = ["ls-files", "--exclude-standard", "--others"];
+  if (untrackedFileListCache === undefined || shouldRefresh) {
+    const args = ["ls-files", "--exclude-standard", "--others"];
 
-  const untrackedFileListCache = child_process
-    .execFileSync(COMMAND, args, OPTIONS)
-    .toString()
-    .trim()
-    .split("\n")
-    .map((filePath) => resolve(filePath));
+    untrackedFileListCache = child_process
+      .execFileSync(COMMAND, args, OPTIONS)
+      .toString()
+      .trim()
+      .split("\n")
+      .map((filePath) => resolve(filePath));
+  }
 
   return untrackedFileListCache;
 };

src/processors.ts

@@ -28,8 +28,14 @@ if (process.env.CI !== undefined) {
  * This is increasingly useful the more files there are in the repository.
  */
 const getPreProcessor =
-  (untrackedFileList: string[], diffFileList: string[]) =>
+  (diffFileList: string[], staged: boolean) =>
   (text: string, filename: string) => {
+    let untrackedFileList = getUntrackedFileList(staged);
+    const shouldRefresh =
+      !diffFileList.includes(filename) && !untrackedFileList.includes(filename);
+    if (shouldRefresh) {
+      untrackedFileList = getUntrackedFileList(staged, true);
+    }
     const shouldBeProcessed =
       process.env.VSCODE_CLI !== undefined ||
       diffFileList.includes(filename) ||
@@ -66,7 +72,7 @@ const getUnstagedChangesError = (filename: string): [Linter.LintMessage] => {
 };
 
 const getPostProcessor =
-  (untrackedFileList: string[], staged = false) =>
+  (staged = false) =>
   (
     messages: Linter.LintMessage[][],
     filename: string
@@ -75,7 +81,7 @@ const getPostProcessor =
       // No need to filter, just return
       return [];
     }
-
+    const untrackedFileList = getUntrackedFileList(staged);
     if (untrackedFileList.includes(filename)) {
       // We don't need to filter the messages of untracked files because they
       // would all be kept anyway, so we return them as-is.
@@ -111,12 +117,11 @@ const getProcessors = (
   processorType: ProcessorType
 ): Required<Linter.Processor> => {
   const staged = processorType === "staged";
-  const untrackedFileList = getUntrackedFileList(staged);
   const diffFileList = getDiffFileList(staged);
 
   return {
-    preprocess: getPreProcessor(untrackedFileList, diffFileList),
-    postprocess: getPostProcessor(untrackedFileList, staged),
+    preprocess: getPreProcessor(diffFileList, staged),
+    postprocess: getPostProcessor(staged),
     supportsAutofix: true,
   };
 };