Fix asset URL handling in development mode (#222)

* Fix asset URL handling in development mode

Fix asset URL handling to exclude the django_bird prefix in development mode,
while maintaining it in production mode. This ensures that static assets are
correctly referenced in both environments:

- In development: static/bird/modal.js
- In production: static/django_bird/bird/modal.js

* update comment

* update tests

Author: joshuadavidthomas

CHANGELOG.md

@@ -22,6 +22,10 @@ and this project attempts to adhere to [Semantic Versioning](https://semver.org/
 
 - Normalize paths in asset manifest to prevent leaking system-specific information like Python version and installation paths
 
+### Fixed
+
+- Fixed asset URL handling in development mode to exclude the "django_bird/" prefix, matching the actual file paths in development environments. Production mode (DEBUG=False) still uses the prefix to maintain compatibility with collected static files.
+
 ## [0.17.1]
 
 ### Fixed

src/django_bird/staticfiles.py

@@ -12,6 +12,7 @@
 from typing import final
 from typing import overload
 
+from django.conf import settings
 from django.contrib.staticfiles import finders
 from django.contrib.staticfiles.finders import BaseFinder
 from django.contrib.staticfiles.storage import StaticFilesStorage
@@ -178,7 +179,11 @@ def __init__(self, *args: Any, prefix: str, **kwargs: Any):
     def url(self, name: str | None) -> str:
         if name is None:
             return super().url(name)
-        if not name.startswith(f"{self.prefix}/"):
+        # Only add prefix in production (when DEBUG is False)
+        # In development, asset paths don't include the app label prefix
+        # because they come directly from source directories and in production,
+        # assets are collected to STATIC_ROOT/django_bird/
+        if not settings.DEBUG and not name.startswith(f"{self.prefix}/"):
             name = f"{self.prefix}/{name}"
         return super().url(name)
 

tests/templatetags/test_asset.py

@@ -480,7 +480,7 @@ def test_asset_tag_in_debug_mode(
             rendered = template.render({})
 
             assert (
-                f'<link rel="stylesheet" href="/static/django_bird/bird/{button_css.file.name}">'
+                f'<link rel="stylesheet" href="/static/bird/{button_css.file.name}">'
                 in rendered
             )
 

tests/test_staticfiles.py

@@ -198,7 +198,14 @@ def test_relative_path_nested(self, templates_dir):
 
         assert asset.relative_path == Path("bird/nested/button.css")
 
-    def test_url(self, templates_dir):
+    @pytest.mark.parametrize(
+        "DEBUG,expected_prefix",
+        [
+            (False, "/static/django_bird"),
+            (True, "/static"),
+        ],
+    )
+    def test_url(self, DEBUG, expected_prefix, templates_dir):
         button = TestComponent(
             name="button",
             content="<button>Click me</button>",
@@ -212,10 +219,11 @@ def test_url(self, templates_dir):
         component = Component.from_name(button.name)
         asset = component.get_asset(button_css.file.name)
 
-        assert (
-            asset.url
-            == f"/static/django_bird/{button_css.file.parent.name}/{button_css.file.name}"
-        )
+        with override_settings(DEBUG=DEBUG):
+            assert (
+                asset.url
+                == f"{expected_prefix}/{button_css.file.parent.name}/{button_css.file.name}"
+            )
 
     def test_url_nonexistent(self, templates_dir):
         button = TestComponent(