Merge main into feat/config-diff-threshold

Author: Hi7cl4w

src/Diffy.ts

@@ -628,7 +628,7 @@ class Diffy extends BaseDiffy {
       }
 
       // Get current staged diff in background
-      const diff = await this.gitService?.getDiff(repo, false).catch((_) => null);
+      const diff = await this.gitService?.getDiffAndWarnUser(repo, false, undefined, true);
       if (!diff) {
         return;
       }

src/service/GitService.ts

@@ -96,27 +96,27 @@ class GitService {
     window.showInformationMessage(`${CONSTANTS.extensionShortName}: ${msg}`);
   }
 
-  async getDiff(repo: Repository, cached = true, nameOnly?: boolean): Promise<string> {
+  async getDiffAndWarnUser(repo: Repository, cached = true, nameOnly?: boolean, silent = false) {
     const diff = await this.getGitDiff(repo, cached, nameOnly);
     if (!diff) {
       if (cached) {
         const diffUncached = await repo.diff(false);
         if (diffUncached) {
-          throw new Error("Warning: please stage your git changes");
+          if (!silent) {
+            this.showInformationMessage("warning: please stage your git changes");
+          }
         } else {
-          throw new Error("No Changes");
+          if (!silent) {
+            this.showInformationMessage("No Changes");
+          }
         }
       }
-      throw new Error("No changes");
+      if (!silent) {
+        this.showInformationMessage("No changes");
+      }
     }
     return diff;
   }
-  async getDiffAndWarnUser(repo: Repository, cached = true, nameOnly?: boolean) {
-    return this.getDiff(repo, cached, nameOnly).catch((error) => {
-      this.showInformationMessage(error.message);
-      return null;
-    });
-  }
   /**
    * Check if a file path matches any of the exclusion patterns
    * @param filePath - The file path to check
@@ -128,7 +128,11 @@ class GitService {
       // Simple glob pattern matching
       if (pattern.includes("*")) {
         const regex = new RegExp(
-          `^${pattern.replace(/\\/g, "\\\\").replace(/\./g, "\\.").replace(/\*/g, ".*").replace(/\?/g, ".")}$`,
+          `^${pattern
+            .replace(/\\/g, "\\\\")
+            .replace(/\./g, "\\.")
+            .replace(/\*/g, ".*")
+            .replace(/\?/g, ".")}$`,
         );
         if (regex.test(filePath)) {
           return true;