From bde246f52ed861b60b9c46fdbcd476d0e1eef616 Mon Sep 17 00:00:00 2001
From: Vasista Vovveti <vasistavovveti@gmail.com>
Date: Mon, 4 Oct 2021 16:03:11 -0500
Subject: [PATCH] Cache cloned repos per build (#10)

* Cache cloned repos per build

* Update toptranslators.py
---
 sphinxext/toptranslators.py | 39 ++++++++++++++++++++++++-------------
 1 file changed, 25 insertions(+), 14 deletions(-)

diff --git a/sphinxext/toptranslators.py b/sphinxext/toptranslators.py
index e958cbc..8c18c38 100644
--- a/sphinxext/toptranslators.py
+++ b/sphinxext/toptranslators.py
@@ -20,6 +20,8 @@
 
 TRANSLATORS_MARKER_NAME = "# Translators:"
 
+CACHE = {}
+
 # https://stackoverflow.com/questions/517923/what-is-the-best-way-to-remove-accents-normalize-in-a-python-unicode-string/518232#518232
 def strip_accents(s):
     return "".join(
@@ -137,32 +139,41 @@ def run(self) -> List[nodes.Node]:
 
         repo_url = f"https://github.com/{self.arguments[0]}.git"
 
-        with TempDir() as temp_dir:
-
+        if repo_url in CACHE:
+            temp_dir = CACHE[repo_url]
+        else:
+            temp_dir = mkdtemp()
             # Clone repo
             try:
                 Repo.clone_from(repo_url, temp_dir, multi_options=["--depth 1"])
             except Exception as e:
+                del_directory_exists(temp_dir)
                 raise ExtensionError("Invalid git repository given!", e)
+            CACHE[repo_url] = temp_dir
+
+        top_contributors = get_top_translators(temp_dir, locale).most_common(limit)
 
-            top_contributors = get_top_translators(temp_dir, locale).most_common(limit)
+        if "alphabetical" in order:
+            top_contributors.sort(key=lambda tup: strip_accents(tup[0]))
 
-            if "alphabetical" in order:
-                top_contributors.sort(key=lambda tup: strip_accents(tup[0]))
-            
-            return [
-                ContributorSource(
-                    [
-                        Contributor(name, hide_contributions, contributions)
-                        for name, contributions in top_contributors
-                    ]
-                ).build()
-            ]
+        return [
+            ContributorSource(
+                [
+                    Contributor(name, hide_contributions, contributions)
+                    for name, contributions in top_contributors
+                ]
+            ).build()
+        ]
 
+def cleanup_tempdirs(*args, **kwargs):
+    for url, temp_dir in CACHE.items():
+        del_directory_exists(temp_dir)
+    CACHE.clear()
 
 
 def setup(app: Sphinx) -> Dict[str, Any]:
     directives.register_directive("toptranslators", TopTranslators)
+    app.connect("build-finished", cleanup_tempdirs)
 
     return {
         "parallel_read_safe": True,