Simplify implementation and inject node_modules directly

Author: codingjoe

README.md

@@ -43,6 +43,23 @@ STATICFILES_FINDERS = [
 ]
 ```
 
+You will also need to expose your `node_modules` directory to Django's
+staticfiles finder. You may run `npm ci --omit=dev` prior to running
+`collectstatic` to avoid exposing your `devDependencies` publicly.
+
+```python
+# settings.py
+from pathlib import Path
+
+# add BASE_DIR (if not already present)
+BASE_DIR = Path(__file__).resolve().parent.parent
+
+STATICFILES_DIRS = [
+    # …
+    BASE_DIR / "node_modules",
+]
+```
+
 Finally, add the import map to your base template:
 
 ```html

django_esm/finders.py

@@ -42,15 +42,10 @@ def list(self, ignore_patterns):
     def _list(*ignore_patterns):
         with (settings.BASE_DIR / "package.json").open() as f:
             package_json = json.load(f)
-
         return [
             (path, storages.root_storage)
             for mod, path in utils.parse_root_package(package_json)
             if not matches_patterns(path, ignore_patterns)
-        ] + [
-            (path, storages.node_modules_storage)
-            for mod, path in utils.parse_dependencies(package_json)
-            if not matches_patterns(path, ignore_patterns)
         ]
 
     @functools.cached_property

tests/test_finders.py

@@ -17,9 +17,7 @@ def test_find(self):
             self.finder.find("testapp/static/js/components/index.js")
             == "testapp/static/js/components/index.js"
         )
-        assert self.finder.find("lit-html/lit-html.js", all=True) == [
-            "lit-html/lit-html.js"
-        ]
+        assert self.finder.find("lit-html/lit-html.js", all=True) == []
         assert self.finder.find("foo/bar.js") == []
 
     def test_list(self):
@@ -29,8 +27,6 @@ def test_list(self):
             "testapp/static/js/components/index.js",
             storages.root_storage,
         ) in all_files
-        assert ("lit-html/lit-html.js", storages.node_modules_storage) in all_files
-        assert ("htmx.org/dist/htmx.min.js", storages.node_modules_storage) in all_files
 
     def test_check(self, settings):
         assert not self.finder.check()