Dataset: lazy-load git-repo

Author: Gerit Wagner

colrev/dataset.py

@@ -5,6 +5,7 @@
 import os
 import tempfile
 import typing
+from functools import cached_property
 from pathlib import Path
 
 import colrev.exceptions as colrev_exceptions
@@ -27,7 +28,10 @@ class Dataset:
 
     def __init__(self, *, review_manager: colrev.review_manager.ReviewManager) -> None:
         self.review_manager = review_manager
-        self.git_repo = GitRepo(path=review_manager.path)
+
+    @cached_property
+    def git_repo(self) -> GitRepo:
+        return GitRepo(path=self.review_manager.path)
 
     def get_origin_state_dict(self, records_string: str = "") -> dict:
         """Get the origin_state_dict (to determine state transitions efficiently)

colrev/packages/files_dir/src/files_dir.py

@@ -87,7 +87,6 @@ def __init__(
         import colrev.review_manager as colrev_review_manager
 
         self.review_manager = colrev_review_manager.ReviewManager()
-        colrev.ops.check.CheckOperation(self.review_manager)
 
     def _update_if_pdf_renamed(
         self,
@@ -656,6 +655,7 @@ def search(self, rerun: bool) -> None:
         if self.review_manager.force_mode:  # i.e., reindex all
             self.logger.info("Reindex all")
 
+        colrev.ops.check.CheckOperation(self.review_manager)
         # Removing records/origins for which PDFs were removed makes sense for curated repositories
         # In regular repositories, it may be confusing (e.g., if PDFs are renamed)
         # In these cases, we may simply print a warning instead of modifying/removing records?