web_modules_dir needs to be public

the web_modules_path func already is + we need it for
idom-jupyter to be able to distribute just the modules
without the client in a non-brittle way

Author: rmorshea

src/idom/client/_private.py

@@ -16,23 +16,15 @@
     shutil.copytree(BACKUP_BUILD_DIR, IDOM_CLIENT_BUILD_DIR.get(), symlinks=True)
 
 
-def build_dir() -> Path:
-    return IDOM_CLIENT_BUILD_DIR.get()
-
-
-def web_modules_dir() -> Path:
-    return build_dir() / "_snowpack" / "pkg"
-
-
 def restore_build_dir_from_backup() -> None:
-    target = build_dir()
+    target = IDOM_CLIENT_BUILD_DIR.get()
     if target.exists():
         shutil.rmtree(target)
     shutil.copytree(BACKUP_BUILD_DIR, target, symlinks=True)
 
 
 def replace_build_dir(source: Path) -> None:
-    target = build_dir()
+    target = IDOM_CLIENT_BUILD_DIR.get()
     if target.exists():
         shutil.rmtree(target)
     shutil.copytree(source, target, symlinks=True)
@@ -58,7 +50,7 @@ def split_package_name_and_version(pkg: str) -> Tuple[str, str]:
 
 
 def build_dependencies() -> Dict[str, str]:
-    package_json = build_dir() / "package.json"
+    package_json = IDOM_CLIENT_BUILD_DIR.get() / "package.json"
     return cast(Dict[str, str], json.loads(package_json.read_text())["dependencies"])
 
 

src/idom/client/manage.py

@@ -5,14 +5,27 @@
 from tempfile import TemporaryDirectory
 from typing import Dict, Iterable, List, Sequence, Set, Union
 
-from idom.config import IDOM_CLIENT_WEB_MODULE_BASE_URL
+from idom.config import IDOM_CLIENT_BUILD_DIR, IDOM_CLIENT_WEB_MODULE_BASE_URL
 
 from . import _private
 
 
 logger = getLogger(__name__)
 
 
+def web_modules_dir() -> Path:
+    return IDOM_CLIENT_BUILD_DIR.get() / "_snowpack" / "pkg"
+
+
+def web_module_path(package_name: str, must_exist: bool = False) -> Path:
+    path = web_modules_dir().joinpath(*(package_name + ".js").split("/"))
+    if must_exist and not path.exists():
+        raise ValueError(
+            f"Web module {package_name!r} does not exist at path {str(path)!r}"
+        )
+    return path
+
+
 def web_module_exports(package_name: str) -> List[str]:
     web_module_path(package_name, must_exist=True)
     return _private.find_js_module_exports_in_source(
@@ -31,7 +44,7 @@ def web_module_exists(package_name: str) -> bool:
 
 def web_module_names() -> Set[str]:
     names = []
-    web_mod_dir = _private.web_modules_dir()
+    web_mod_dir = web_modules_dir()
     for pth in web_mod_dir.glob("**/*.js"):
         rel_pth = pth.relative_to(web_mod_dir)
         if Path("common") in rel_pth.parents:
@@ -53,15 +66,6 @@ def add_web_module(package_name: str, source: Union[Path, str]) -> str:
     return web_module_url(package_name)
 
 
-def web_module_path(package_name: str, must_exist: bool = False) -> Path:
-    path = _private.web_modules_dir().joinpath(*(package_name + ".js").split("/"))
-    if must_exist and not path.exists():
-        raise ValueError(
-            f"Web module {package_name!r} does not exist at path {str(path)!r}"
-        )
-    return path
-
-
 def restore() -> None:
     _private.restore_build_dir_from_backup()