feat(bundle-jvm): Add JVM extension filtering and default excludes

Filter collected sources by JVM file extensions (java, kt, scala,
groovy, kts) and exclude common build output directories by default
(build, .gradle, .cxx, node_modules). Also respect .gitignore rules
as an additional layer of defense. Users can provide extra --exclude
glob patterns on top of the defaults.

This enables invoking bundle-jvm directly on a project root without
needing to pre-copy sources into a separate directory.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: romtsn

src/commands/debug_files/bundle_jvm.rs

@@ -7,7 +7,7 @@ use crate::utils::file_upload::SourceFile;
 use crate::utils::fs::path_as_url;
 use crate::utils::source_bundle::{self, BundleContext};
 use anyhow::{bail, Context as _, Result};
-use clap::{Arg, ArgMatches, Command};
+use clap::{Arg, ArgAction, ArgMatches, Command};
 use sentry::types::DebugId;
 use std::collections::BTreeMap;
 use std::fs;
@@ -16,6 +16,12 @@ use std::str::FromStr as _;
 use std::sync::Arc;
 use symbolic::debuginfo::sourcebundle::SourceFileType;
 
+/// File extensions for JVM-based languages.
+const JVM_EXTENSIONS: &[&str] = &["java", "kt", "scala", "groovy", "kts"];
+
+/// Default directory patterns to exclude from source collection.
+const DEFAULT_EXCLUDES: &[&str] = &["!build", "!.gradle", "!.cxx", "!node_modules"];
+
 pub fn make_command(command: Command) -> Command {
     command
         .hide(true) // experimental for now
@@ -47,6 +53,16 @@ pub fn make_command(command: Command) -> Command {
                 .value_parser(DebugId::from_str)
                 .help("Debug ID (UUID) to use for the source bundle."),
         )
+        .arg(
+            Arg::new("exclude")
+                .long("exclude")
+                .value_name("PATTERN")
+                .action(ArgAction::Append)
+                .help(
+                    "Glob pattern to exclude files/directories. Can be repeated. \
+                     By default, 'build', '.gradle', and '.cxx' directories are excluded.",
+                ),
+        )
 }
 
 pub fn execute(matches: &ArgMatches) -> Result<()> {
@@ -75,7 +91,22 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
         ))?;
     }
 
-    let sources = ReleaseFileSearch::new(path.clone()).collect_files()?;
+    let user_excludes: Vec<String> = matches
+        .get_many::<String>("exclude")
+        .map(|vals| vals.map(|v| format!("!{v}")).collect())
+        .unwrap_or_default();
+
+    let all_excludes: Vec<&str> = DEFAULT_EXCLUDES
+        .iter()
+        .copied()
+        .chain(user_excludes.iter().map(|s| s.as_str()))
+        .collect();
+
+    let sources = ReleaseFileSearch::new(path.clone())
+        .extensions(JVM_EXTENSIONS.iter().copied())
+        .ignores(all_excludes)
+        .respect_ignores(true)
+        .collect_files()?;
     let files = sources.iter().map(|source| {
         let local_path = source.path.strip_prefix(&source.base_path).unwrap();
         let local_path_jvm_ext = local_path.with_extension("jvm");

src/utils/file_search.rs

@@ -20,6 +20,7 @@ pub struct ReleaseFileSearch {
     ignores: BTreeSet<String>,
     ignore_file: Option<String>,
     decompress: bool,
+    respect_ignores: bool,
 }
 
 #[derive(Eq, PartialEq, Hash)]
@@ -37,6 +38,7 @@ impl ReleaseFileSearch {
             ignore_file: None,
             ignores: BTreeSet::new(),
             decompress: false,
+            respect_ignores: false,
         }
     }
 
@@ -78,6 +80,11 @@ impl ReleaseFileSearch {
         self
     }
 
+    pub fn respect_ignores(&mut self, respect: bool) -> &mut Self {
+        self.respect_ignores = respect;
+        self
+    }
+
     pub fn collect_file(path: PathBuf) -> Result<ReleaseFileMatch> {
         // NOTE: `collect_file` currently do not handle gzip decompression,
         // as its mostly used for 3rd tools like xcode or gradle.
@@ -105,11 +112,11 @@ impl ReleaseFileSearch {
         let mut collected = Vec::new();
 
         let mut builder = WalkBuilder::new(&self.path);
-        builder
-            .follow_links(true)
-            .git_exclude(false)
-            .git_ignore(false)
-            .ignore(false);
+        builder.follow_links(true);
+
+        if !self.respect_ignores {
+            builder.git_exclude(false).git_ignore(false).ignore(false);
+        }
 
         if !&self.extensions.is_empty() {
             let mut types_builder = TypesBuilder::new();