skip outputting lsif snapshot for files located outside the sourceroot (

#195) Co-authored-by: Ólafur Páll Geirsson <[email protected]>
sourcegraph · May 12, 2021 · e53a450 · e53a450
1 parent 548e179
commit e53a450
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/lsif-java/src/main/scala/com/sourcegraph/lsif_java/commands/SnapshotLsifCommand.scala b/lsif-java/src/main/scala/com/sourcegraph/lsif_java/commands/SnapshotLsifCommand.scala
@@ -49,8 +49,8 @@ case class SnapshotLsifCommand(
     @PositionalArguments() input: List[Path] = List(Paths.get("dump.lsif"))
 ) extends Command {
 
-  private val finalOutput = AbsolutePath.of(output, sourceroot)
   def sourceroot: Path = AbsolutePath.of(app.env.workingDirectory)
+  private val finalOutput = AbsolutePath.of(output, sourceroot)
 
   def run(): Int = {
     Files.walkFileTree(finalOutput, new DeleteVisitor())
@@ -59,7 +59,12 @@ case class SnapshotLsifCommand(
       in = AbsolutePath.of(inputPath, sourceroot)
       doc <- SnapshotLsifCommand.parseTextDocument(in, sourceroot)
     } {
-      SnapshotCommand.writeSnapshot(doc, finalOutput)
+      val docPath = AbsolutePath
+        .of(Paths.get(doc.getUri), sourceroot)
+        .toRealPath()
+      if (docPath.toAbsolutePath.startsWith(sourceroot)) {
+        SnapshotCommand.writeSnapshot(doc, finalOutput)
+      }
     }
     0
   }