Added configurability to the bazel query buildfiles parameter (#19)

Author: runchen0919

bundles/com.salesforce.bazel.eclipse.core/src/com/salesforce/bazel/eclipse/core/model/discovery/BazelQueryTargetDiscovery.java

@@ -43,12 +43,18 @@ public Collection<WorkspacePath> discoverPackages(BazelWorkspace bazelWorkspace,
             throws CoreException {
         var monitor = SubMonitor.convert(progress, 100);
 
+        // get buildfiles argument from projevt view
+        var buildfile_query = bazelWorkspace.getBazelProjectView().targetDiscoverySettings().get("buildfile_query");
+        if (buildfile_query == null || buildfile_query.isBlank()) {
+            buildfile_query = "//...";
+        }
+
         // bazel query 'buildfiles(//...)' --output package
         Collection<String> labels = bazelWorkspace.getCommandExecutor()
                 .runQueryWithoutLock(
                     new BazelQueryForPackagesCommand(
                             bazelWorkspace.getLocation().toPath(),
-                            "buildfiles(//...)",
+                            "buildfiles(" + buildfile_query + ")",
                             true,
                             "Querying for all available packages in workspace"));
 

bundles/com.salesforce.bazel.eclipse.core/src/com/salesforce/bazel/eclipse/core/projectview/BazelProjectFileReader.java

@@ -66,6 +66,7 @@ public void close() {
         final LinkedHashSet<Path> importingFiles = new LinkedHashSet<>();
         IPath bazelBinary;
         final LinkedHashMap<String, String> targetProvisioningSettings = new LinkedHashMap<>();
+        final LinkedHashMap<String, String> targetDiscoverySettings = new LinkedHashMap<>();
         final LinkedHashMap<String, String> projectMappings = new LinkedHashMap<>();
         final LinkedHashSet<String> importPreferences = new LinkedHashSet<>();
         final LinkedHashSet<String> projectSettings = new LinkedHashSet<>();
@@ -136,6 +137,7 @@ public BazelProjectView build() throws IllegalStateException {
                     bazelBinary,
                     targetDiscoveryStrategy,
                     targetProvisioningStrategy,
+                    targetDiscoverySettings,
                     targetProvisioningSettings,
                     projectMappings,
                     preferencesToImport,
@@ -342,6 +344,18 @@ private void parseProjectFile(Path bazelProjectFile, BazelProjectViewBuilder bui
                         }
                         break;
                     }
+                    case "target_discovery_settings": {
+                        // extension for BEF
+                        parseSectionBodyIntoList(rawSection).forEach(s -> {
+                            var separator = s.indexOf('=');
+                            if (separator != -1) {
+                                var key = s.substring(0, separator).trim();
+                                var value = s.substring(separator + 1).trim();
+                                builder.targetDiscoverySettings.put(key, value);
+                            }
+                        });
+                        break;
+                    }
                     case "target_provisioning_settings": {
                         // extension for BEF
                         parseSectionBodyIntoList(rawSection).forEach(s -> {

bundles/com.salesforce.bazel.eclipse.core/src/com/salesforce/bazel/eclipse/core/projectview/BazelProjectView.java

@@ -40,6 +40,7 @@ public record BazelProjectView(
         IPath bazelBinary,
         String targetDiscoveryStrategy,
         String targetProvisioningStrategy,
+        Map<String, String> targetDiscoverySettings,
         Map<String, String> targetProvisioningSettings,
         Map<String, String> projectMappings,
         Collection<WorkspacePath> importPreferences,
@@ -59,6 +60,7 @@ public record BazelProjectView(
         targets = unmodifiableCollection(targets);
         additionalLanguages = unmodifiableCollection(additionalLanguages);
         tsConfigRules = unmodifiableCollection(tsConfigRules);
+        targetDiscoverySettings = Collections.unmodifiableMap(targetDiscoverySettings);
         targetProvisioningSettings = Collections.unmodifiableMap(targetProvisioningSettings);
         projectMappings = Collections.unmodifiableMap(projectMappings);
         importPreferences = unmodifiableCollection(importPreferences);

docs/common/projectviews.md

@@ -159,6 +159,14 @@ The syntax of each entry is `key=value`, where `key` and `value` are expected st
 	* Providing a value greater then 0 will filter transitive dependencies and provide a partial classpath to the project model.
 	* Note: While all required compile-time dependencies are added regardless, there may be edge cases with the Eclipse compiler which may result in compile errors.
 
+### `target_discovery_settings`
+
+* `buildfile_query` Specifies the Bazel query expression used to discover packages when `target_discovery_strategy` is set to `bazel-query` (default is `//...`).
+	* This parameter is wrapped in a `buildfiles()` function during package discovery, e.g., `buildfiles(//...)`.
+	* Example: Set to `//some/path/...` to limit package discovery to a specific directory tree.
+	* Example: Set to `//module1/... + //module2/...` to discover packages from multiple specific paths. 
+
+
 ### `project_mappings`
 
 A list of mappings from targets (typically from external repositories) to projects in the IDE.