fix(qsync): NPE when trying to enable analysis for a directory or for…

… a BUILD file
bazelbuild · Nov 14, 2024 · 6286c1a · 6286c1a
1 parent fc3990b
commit 6286c1a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/querysync/java/com/google/idea/blaze/qsync/project/BuildGraphData.java b/querysync/java/com/google/idea/blaze/qsync/project/BuildGraphData.java
@@ -407,7 +407,7 @@ public ImmutableSet<DependencyTrackingBehavior> getDependencyTrackingBehaviors(L
    *     the set of all targets defined in all build packages within the directory (recursively).
    */
   public TargetsToBuild getProjectTargets(Context<?> context, Path workspaceRelativePath) {
-    if (workspaceRelativePath.endsWith("BUILD")) {
+    if (workspaceRelativePath.endsWith("BUILD") || workspaceRelativePath.endsWith("BUILD.bazel")) {
       Path packagePath = workspaceRelativePath.getParent();
       return TargetsToBuild.targetGroup(allTargets().get(packagePath));
     } else {

diff --git a/shared/java/com/google/idea/blaze/common/Label.java b/shared/java/com/google/idea/blaze/common/Label.java
@@ -47,7 +47,7 @@ public static Label of(String label) {
   }
 
   public static Label fromPackageAndName(Path packagePath, Path name) {
-    return of(String.format("//%s:%s", packagePath, name));
+    return of(String.format("@@//%s:%s", packagePath, name));
   }
 
   public static Label fromPackageAndName(Path packagePath, String name) {