refactor asset collection to consistently use frozenset (#122)

Author: joshuadavidthomas

src/django_bird/components.py

@@ -117,11 +117,13 @@ def get_component(self, name: str) -> Component:
         self._components[name] = component
         return component
 
-    def get_assets(self, asset_type: AssetType) -> list[Asset]:
-        assets: list[Asset] = []
+    def get_assets(self, asset_type: AssetType) -> frozenset[Asset]:
+        assets: set[Asset] = set()
         for component in self._components.values():
-            assets.extend(a for a in component.assets if a.type == asset_type)
-        return assets
+            for asset in component.assets:
+                if asset.type == asset_type:
+                    assets.add(asset)
+        return frozenset(assets)
 
 
 components = ComponentRegistry()

src/django_bird/templatetags/tags/asset.py

@@ -36,5 +36,7 @@ def render(self, context: Context) -> str:
         assets = components.get_assets(self.asset_type)
         if not assets:
             return ""
-        rendered = {asset.render() for asset in sorted(assets, key=lambda a: a.path)}
+        rendered = frozenset(
+            asset.render() for asset in sorted(assets, key=lambda a: a.path)
+        )
         return "\n".join(rendered)