Exclude .git directory content for package file count

clearlydefined · Oct 31, 2023 · 6eead81 · 6eead81
1 parent a793608
commit 6eead81
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/providers/process/abstractClearlyDefinedProcessor.js b/providers/process/abstractClearlyDefinedProcessor.js
@@ -75,7 +75,7 @@ class AbstractClearlyDefinedProcessor extends AbstractProcessor {
     let count = 0
     const bytes = await du(location, {
       filter: file => {
-        if (path.basename(file) === '.git') return false
+        if (!this.isValidExcludingGit(file)) return false
         count++
         return true
       }

diff --git a/providers/process/abstractProcessor.js b/providers/process/abstractProcessor.js
@@ -150,15 +150,21 @@ class AbstractProcessor extends BaseHandler {
    */
   async filterFiles(location) {
     const fullList = await this.getFiles(location)
-    const exclusions = ['.git']
-    const filteredList = fullList.filter(file => {
-      if (!file) return false
-      const segments = file.split(/[\\/]/g)
-      return !intersection(segments, exclusions).length
-    })
+    const filteredList = fullList.filter(file => this.isValidExcludingGit(file))
     return trimAllParents(filteredList, location).filter(x => x)
   }
 
+  _isValid(file, exclusions) {
+    if (!file) return false
+    const segments = file.split(/[\\/]/g)
+    return !intersection(segments, exclusions).length
+  }
+
+  isValidExcludingGit(file) {
+    const exclusions = ['.git']
+    return this._isValid(file, exclusions)
+  }
+
   shouldFetch() {
     return true
   }