[paldepind]

The index-files.sh file in the extractors for Ruby, Ql, and Unified all pass the filename with the list of files to extract like this --file-list "$1".

However, in index-files.sh in the Rust extractor we pass it like @"$1". The @ is syntax to pass arguments by filename (I'm not sure what the origin of this notation is, but it's a somewhat well-known notation). We handle the @ by a dependency argfile.

The only thing we gain from introducing this dependency, using @ (that I was not familiar with), deviating from the other extractors, and special casing in the extractor_cli_config macro, is that we don't have to read the named file ourselves.

I don't think that's worth it, so this PR removes the argfile dependency and reads and parses the file manually (like the other extractors do). This gives closer alignment with the other extractors and less complexity. Furthermore, it turns out that, all things considered, this is in fact fewer lines of code than what we had with the dependency.

The dependency removal and generated Cargo/Bazel metadata are in the first commit. The code changes are in the second commit.

[Copilot]

🟡 Changes recommended

An unresolved minor finding notes that the configuration log omits the input-file count, making empty or partial extractions harder to diagnose.

1 open issue

• Optional: the configuration log no longer reports how many input files were supplied, making empty… · New

What changed in this PR

This PR aligns the Rust extractor with other CodeQL extractors by replacing @argfile expansion with an explicit --file-list option.

Changes:

• Update both launchers and read input paths directly from a newline-separated file.
• Return QL test sources separately from configuration and remove input-specific macro handling.
• Remove argfile and its unused transitive dependencies from Cargo and Bazel metadata.

File	Description
rust/​tools/​index-files.sh	Pass the file list through --file-list.
rust/​tools/​index-files.cmd	Use --file-list in the Windows launcher.
rust/​extractor/​src/​qltest.rs	Return source paths instead of mutating configuration.
rust/​extractor/​src/​main.rs	Read input paths from the configured file list.
rust/​extractor/​src/​config.rs	Replace positional inputs with a file-list option.
rust/​extractor/​macros/​src/​lib.rs	Remove special handling for input paths.
rust/​extractor/​Cargo.toml	Remove the argfile dependency.
MODULE.bazel	Remove the argfile repository import.
misc/​bazel/​3rdparty/​tree_sitter_extractors_deps/​crates.bzl	Remove unused dependency and repository definitions.
misc/​bazel/​3rdparty/​tree_sitter_extractors_deps/​BUILD.os_str_bytes-7.2.0.bazel	Delete unused transitive-dependency build metadata.
misc/​bazel/​3rdparty/​tree_sitter_extractors_deps/​BUILD.fs-err-3.3.0.bazel	Delete unused transitive-dependency build metadata.
misc/​bazel/​3rdparty/​tree_sitter_extractors_deps/​BUILD.bazel	Remove argfile aliases.
misc/​bazel/​3rdparty/​tree_sitter_extractors_deps/​BUILD.argfile-1.0.0.bazel	Delete generated argfile build metadata.
misc/​bazel/​3rdparty/​tree_sitter_extractors_deps/​argfile/​BUILD.bazel	Delete the unversioned dependency alias.
misc/​bazel/​3rdparty/​tree_sitter_extractors_deps/​argfile-1.0.0/​BUILD.bazel	Delete the versioned dependency alias.
Cargo.lock	Remove argfile and its unused transitive dependencies.

_{Review effort: Max}