DOP-1282: Disable the postprocessor in the language server

Author: i80and

snooty/language_server.py

@@ -314,7 +314,10 @@ def m_initialize(
             root_path = Path(rootUri.replace("file://", "", 1))
             self.project = Project(root_path, self.backend, {})
             self.notify_diagnostics()
-            self.project.build()
+
+            # XXX: Disabling the postprocessor is temporary until we can test
+            # its usage in the language server more extensively
+            self.project.build(postprocess=False)
 
         if processId is not None:
 

snooty/parser.py

@@ -1048,7 +1048,9 @@ def delete(self, path: PurePath) -> None:
 
         self.backend.on_delete(self.get_fileid(path), self.build_identifiers)
 
-    def build(self, max_workers: Optional[int] = None) -> None:
+    def build(
+        self, max_workers: Optional[int] = None, postprocess: bool = True
+    ) -> None:
         all_yaml_diagnostics: Dict[PurePath, List[Diagnostic]] = {}
         pool = multiprocessing.Pool(max_workers)
         with util.PerformanceLogger.singleton().start("parse rst"):
@@ -1107,23 +1109,26 @@ def create_page(filename: str) -> Tuple[Page, EmbeddedRstParser]:
                         page, all_yaml_diagnostics.get(page.source_path, [])
                     )
 
-        post_metadata, post_diagnostics = self.pages.flush()
+        if postprocess:
+            post_metadata, post_diagnostics = self.pages.flush()
 
-        static_files = {
-            "objects.inv": self.targets.generate_inventory("").dumps(
-                self.config.name, ""
-            )
-        }
-        post_metadata["static_files"] = static_files
+            static_files = {
+                "objects.inv": self.targets.generate_inventory("").dumps(
+                    self.config.name, ""
+                )
+            }
+            post_metadata["static_files"] = static_files
 
-        for fileid, page in self.pages.items():
-            self.backend.on_update(self.prefix, self.build_identifiers, fileid, page)
-        for fileid, diagnostics in post_diagnostics.items():
-            self.backend.on_diagnostics(fileid, diagnostics)
+            for fileid, page in self.pages.items():
+                self.backend.on_update(
+                    self.prefix, self.build_identifiers, fileid, page
+                )
+            for fileid, diagnostics in post_diagnostics.items():
+                self.backend.on_diagnostics(fileid, diagnostics)
 
-        self.backend.on_update_metadata(
-            self.prefix, self.build_identifiers, post_metadata
-        )
+            self.backend.on_update_metadata(
+                self.prefix, self.build_identifiers, post_metadata
+            )
 
     def _page_updated(self, page: Page, diagnostics: List[Diagnostic]) -> None:
         """Update any state associated with a parsed page."""
@@ -1233,10 +1238,12 @@ def delete(self, path: PurePath) -> None:
         with self._lock:
             self._project.delete(path)
 
-    def build(self, max_workers: Optional[int] = None) -> None:
+    def build(
+        self, max_workers: Optional[int] = None, postprocess: bool = True
+    ) -> None:
         """Build the full project."""
         with self._lock:
-            self._project.build(max_workers)
+            self._project.build(max_workers, postprocess)
 
     def stop_monitoring(self) -> None:
         """Stop the filesystem monitoring thread associated with this project."""