ignore binary extensions, but not folders

We generate a glob to ignore binary extensions that looks like this:
```
*.{mp4,pages,exe,…}
```

However, if you have a folder that happens to end in `.pages` for
example, then this folder will be ignored in its entirety.

To solve this, we will instead generate the following globs:
```
*.{mp4,pages,exe,…}
!*.{mp4,pages,exe,…}/
```

This way, the `/` indicates that we are dealing with a folder and that
these should _not_ be ignored.

Author: RobinMalfait

crates/oxide/src/scanner/auto_source_detection.rs

@@ -15,7 +15,9 @@ pub static RULES: sync::LazyLock<Gitignore> = sync::LazyLock::new(|| {
 
     builder.add_line(None, &IGNORED_CONTENT_DIRS_GLOB).unwrap();
     builder.add_line(None, &IGNORED_EXTENSIONS_GLOB).unwrap();
-    builder.add_line(None, &BINARY_EXTENSIONS_GLOB).unwrap();
+    for glob in BINARY_EXTENSIONS_GLOB.clone() {
+        builder.add_line(None, &glob).unwrap();
+    }
     builder.add_line(None, &IGNORED_FILES_GLOB).unwrap();
 
     builder.build().unwrap()
@@ -42,15 +44,27 @@ static IGNORED_EXTENSIONS_GLOB: sync::LazyLock<String> = sync::LazyLock::new(||
     )
 });
 
-pub static BINARY_EXTENSIONS_GLOB: sync::LazyLock<String> = sync::LazyLock::new(|| {
-    format!(
-        "*.{{{}}}",
-        include_str!("fixtures/binary-extensions.txt")
-            .trim()
-            .lines()
-            .collect::<Vec<&str>>()
-            .join(",")
-    )
+pub static BINARY_EXTENSIONS_GLOB: sync::LazyLock<Vec<String>> = sync::LazyLock::new(|| {
+    vec![
+        // Ignore the extensions
+        format!(
+            "*.{{{}}}",
+            include_str!("fixtures/binary-extensions.txt")
+                .trim()
+                .lines()
+                .collect::<Vec<&str>>()
+                .join(","),
+        ),
+        // Do not ignore folders that happen to end with a binary extension
+        format!(
+            "!*.{{{}}}/",
+            include_str!("fixtures/binary-extensions.txt")
+                .trim()
+                .lines()
+                .collect::<Vec<&str>>()
+                .join(","),
+        ),
+    ]
 });
 
 static IGNORED_FILES_GLOB: sync::LazyLock<String> = sync::LazyLock::new(|| {

crates/oxide/src/scanner/mod.rs

@@ -564,10 +564,9 @@ fn create_walker(sources: Sources) -> Option<WalkBuilder> {
                     .insert(format!("!{}", "/**/*"));
 
                 // External sources should still disallow binary extensions:
-                ignores
-                    .entry(base)
-                    .or_default()
-                    .insert(BINARY_EXTENSIONS_GLOB.clone());
+                for glob in BINARY_EXTENSIONS_GLOB.clone() {
+                    ignores.entry(base).or_default().insert(glob);
+                }
             }
         }
     }