fastled 1.4.3__tar.gz → 1.4.5__tar.gz

{fastled-1.4.3 → fastled-1.4.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fastled
-Version: 1.4.3
+Version: 1.4.5
 Summary: FastLED Wasm Compiler
 Home-page: https://github.com/zackees/fastled-wasm
 Maintainer: Zachary Vorhies
@@ -10,7 +10,7 @@ Classifier: Programming Language :: Python :: 3
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: fastled-wasm-server>=1.1.0
+Requires-Dist: httpx>=0.28.1
 Requires-Dist: watchdog>=6.0.0
 Requires-Dist: docker>=7.1.0
 Requires-Dist: filelock>=3.16.1

{fastled-1.4.3 → fastled-1.4.5}/compiler/run.py

@@ -4,6 +4,7 @@ import subprocess
 import sys
 import warnings
 from pathlib import Path
+from typing import Tuple
 
 from fastled_wasm_compiler import Compiler
 from fastled_wasm_compiler.paths import VOLUME_MAPPED_SRC
@@ -15,7 +16,7 @@ _CHOICES = ["compile", "server"]
 HERE = Path(__file__).parent
 
 
-def _parse_args() -> tuple[argparse.Namespace, list[str]]:
+def _parse_args() -> Tuple[argparse.Namespace, list[str]]:
     parser = argparse.ArgumentParser(
         description="Run compile.py with additional arguments"
     )

{fastled-1.4.3 → fastled-1.4.5}/install

@@ -17,7 +17,7 @@ if ! command -v uv &> /dev/null; then
 fi
 
 uv venv --python 3.11 --seed
-uv pip install -e .[full] --refresh
+uv pip install -e .[full]
 
 uv run pip install -r requirements.testing.txt
 uv run pip install -r requirements.docker.txt

{fastled-1.4.3 → fastled-1.4.5}/pyproject.toml

@@ -11,7 +11,7 @@ keywords = ["template-python-cmd"]
 license = { text = "BSD 3-Clause License" }
 classifiers = ["Programming Language :: Python :: 3"]
 dependencies = [
-    "fastled-wasm-server>=1.1.0",  # Just use the Client Code
+    "httpx>=0.28.1",
     "watchdog>=6.0.0",
     ########## Begin Docker Manager Dependencies
     "docker>=7.1.0",
@@ -43,13 +43,6 @@ version = { attr = "fastled.__version__" }
 [tool.ruff]
 line-length = 200
 
-[tool.ruff.lint]
-select = ["F", "E", "W", "UP", "PYI"]
-
-[tool.ruff.lint.per-file-ignores]
-"__init__.py" = ["F401"]
-"src/fastled/site/build.py" = ["W291", "W293"]  # Ignore whitespace in multi-line strings
-
 [tool.pylint."MESSAGES CONTROL"]
 good-names = [
     "c",

fastled-1.4.5/requirements.docker.txt

@@ -0,0 +1 @@
+fastled-wasm-server>=1.1.17

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/__init__.py

@@ -1,9 +1,9 @@
 """FastLED Wasm Compiler package."""
 
-from collections.abc import Generator
 from contextlib import contextmanager
 from multiprocessing import Process
 from pathlib import Path
+from typing import Generator
 
 from .__version__ import __version__
 from .compile_server import CompileServer
@@ -41,18 +41,29 @@ class Api:
         profile: bool = False,  # When true then profile information will be enabled and included in the zip.
         no_platformio: bool = False,
     ) -> CompileResult:
+        from fastled.sketch import looks_like_fastled_repo
         from fastled.web_compile import web_compile
 
         if isinstance(host, CompileServer):
             host = host.url()
         if isinstance(directory, str):
             directory = Path(directory)
+
+        # Guard: libfastled compilation requires volume source mapping
+        # Only allow libcompile if we're in a FastLED repository
+        allow_libcompile = looks_like_fastled_repo(Path(".").resolve())
+        if not allow_libcompile:
+            print(
+                "⚠️  libfastled compilation disabled: not running in FastLED repository"
+            )
+
         out: CompileResult = web_compile(
             directory,
             host,
             build_mode=build_mode,
             profile=profile,
             no_platformio=no_platformio,
+            allow_libcompile=allow_libcompile,
         )
         return out
 

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/__version__.py

@@ -1,6 +1,6 @@
 # IMPORTANT! There's a bug in github which will REJECT any version update
 # that has any other change in the repo. Please bump the version as the
 # ONLY change in a commit, or else the pypi update and the release will fail.
-__version__ = "1.4.3"
+__version__ = "1.4.5"
 
 __version_url_latest__ = "https://raw.githubusercontent.com/zackees/fastled-wasm/refs/heads/main/src/fastled/__version__.py"

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/client_server.py

@@ -11,6 +11,7 @@ from pathlib import Path
 from fastled.compile_server import CompileServer
 from fastled.docker_manager import DockerManager
 from fastled.filewatcher import DebouncedFileWatcherProcess, FileWatcherProcess
+from fastled.find_good_connection import ConnectionResult
 from fastled.keyboard import SpaceBarWatcher
 from fastled.open_browser import spawn_http_server
 from fastled.parse_args import Args
@@ -19,7 +20,6 @@ from fastled.sketch import looks_like_sketch_directory
 from fastled.types import BuildMode, CompileResult, CompileServerError
 from fastled.web_compile import (
     SERVER_PORT,
-    ConnectionResult,
     find_good_connection,
     web_compile,
 )
@@ -101,9 +101,17 @@ def _run_web_compiler(
     profile: bool,
     last_hash_value: str | None,
     no_platformio: bool = False,
+    allow_libcompile: bool = False,
 ) -> CompileResult:
+    # Remove the import and libcompile detection logic from here
+    # since it will now be passed as a parameter
     input_dir = Path(directory)
     output_dir = input_dir / "fastled_js"
+
+    # Guard: libfastled compilation requires volume source mapping
+    if not allow_libcompile:
+        print("⚠️  libfastled compilation disabled: not running in FastLED repository")
+
     start = time.time()
     web_result = web_compile(
         directory=input_dir,
@@ -111,6 +119,7 @@ def _run_web_compiler(
         build_mode=build_mode,
         profile=profile,
         no_platformio=no_platformio,
+        allow_libcompile=allow_libcompile,
     )
     diff = time.time() - start
     if not web_result.success:
@@ -310,6 +319,11 @@ def run_client(
         # Assume default port for www
         port = 80
 
+    # Auto-detect libcompile capability on first call
+    from fastled.sketch import looks_like_fastled_repo
+
+    allow_libcompile = looks_like_fastled_repo(Path(".").resolve())
+
     try:
 
         def compile_function(
@@ -318,6 +332,7 @@ def run_client(
             profile: bool = profile,
             last_hash_value: str | None = None,
             no_platformio: bool = no_platformio,
+            allow_libcompile: bool = allow_libcompile,
         ) -> CompileResult:
             TEST_BEFORE_COMPILE(url)
             return _run_web_compiler(
@@ -327,6 +342,7 @@ def run_client(
                 profile=profile,
                 last_hash_value=last_hash_value,
                 no_platformio=no_platformio,
+                allow_libcompile=allow_libcompile,
             )
 
         result: CompileResult = compile_function(last_hash_value=None)
@@ -454,6 +470,10 @@ def run_client(
                 if changed_files:
                     print(f"\nChanges detected in FastLED source code: {changed_files}")
                     print("Press space bar to trigger compile.")
+
+                    # Re-evaluate libcompile capability when source code changes
+                    allow_libcompile = True
+
                     while True:
                         space_bar_pressed = SpaceBarWatcher.watch_space_bar_pressed(
                             timeout=1.0
@@ -476,8 +496,9 @@ def run_client(
                                     f"Changes detected in {','.join(sketch_files_changed)}, triggering recompile..."
                                 )
                             last_compiled_result = compile_function(
-                                last_hash_value=None
+                                last_hash_value=None, allow_libcompile=allow_libcompile
                             )
+                            allow_libcompile = False
                             print("Finished recompile.")
                             # Drain the space bar queue
                             SpaceBarWatcher.watch_space_bar_pressed()

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/compile_server.py

@@ -16,6 +16,7 @@ class CompileServer:
         platform: Platform = Platform.WASM,
         remove_previous: bool = False,
         no_platformio: bool = False,
+        allow_libcompile: bool = True,
     ) -> None:
         from fastled.compile_server_impl import (  # avoid circular import
             CompileServerImpl,
@@ -31,6 +32,7 @@ class CompileServer:
             auto_start=auto_start,
             remove_previous=remove_previous,
             no_platformio=no_platformio,
+            allow_libcompile=allow_libcompile,
         )
 
     # May throw CompileServerError if server could not be started.

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/compile_server_impl.py

@@ -51,6 +51,7 @@ class CompileServerImpl:
         container_name: str | None = None,
         remove_previous: bool = False,
         no_platformio: bool = False,
+        allow_libcompile: bool = True,
     ) -> None:
         container_name = container_name or DEFAULT_CONTAINER_NAME
         if interactive and not mapped_dir:
@@ -70,6 +71,17 @@ class CompileServerImpl:
         self.auto_updates = auto_updates
         self.remove_previous = remove_previous
         self.no_platformio = no_platformio
+
+        # Guard: libfastled compilation requires volume source mapping
+        # If we don't have fastled_src_dir (not in FastLED repo), disable libcompile
+        if allow_libcompile and self.fastled_src_dir is None:
+            print(
+                "⚠️  libfastled compilation disabled: volume source mapping not available"
+            )
+            print("   (not running in FastLED repository)")
+            allow_libcompile = False
+
+        self.allow_libcompile = allow_libcompile
         self._port = 0  # 0 until compile server is started
         if auto_start:
             self.start()
@@ -112,6 +124,7 @@ class CompileServerImpl:
             build_mode=build_mode,
             profile=profile,
             no_platformio=self.no_platformio,
+            allow_libcompile=self.allow_libcompile,
         )
         return out
 

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/filewatcher.py

@@ -10,6 +10,7 @@ from contextlib import redirect_stdout
 from multiprocessing import Process, Queue
 from pathlib import Path
 from queue import Empty
+from typing import Dict, Set
 
 from watchdog.events import FileSystemEvent, FileSystemEventHandler
 from watchdog.observers import Observer
@@ -34,8 +35,8 @@ class MyEventHandler(FileSystemEventHandler):
     def __init__(
         self,
         change_queue: queue.Queue,
-        excluded_patterns: set[str],
-        file_hashes: dict[str, str],
+        excluded_patterns: Set[str],
+        file_hashes: Dict[str, str],
     ) -> None:
         super().__init__()
         self.change_queue = change_queue
@@ -93,8 +94,8 @@ class FileChangedNotifier(threading.Thread):
         )
         self.stopped = False
         self.change_queue: queue.Queue = queue.Queue()
-        self.last_notification: dict[str, float] = {}
-        self.file_hashes: dict[str, str] = {}
+        self.last_notification: Dict[str, float] = {}
+        self.file_hashes: Dict[str, str] = {}
         self.debounce_seconds = debounce_seconds
 
     def stop(self) -> None:

fastled-1.4.5/src/fastled/find_good_connection.py

@@ -0,0 +1,105 @@
+import _thread
+import time
+from concurrent.futures import Future, ThreadPoolExecutor, as_completed
+from dataclasses import dataclass
+from typing import Dict, Tuple
+
+import httpx
+
+_TIMEOUT = 30.0
+
+_EXECUTOR = ThreadPoolExecutor(max_workers=8)
+
+# In-memory cache for connection results
+# Key: (tuple of urls, filter_out_bad, use_ipv6)
+# Value: (ConnectionResult | None, timestamp)
+_CONNECTION_CACHE: Dict[
+    Tuple[tuple, bool, bool], Tuple["ConnectionResult | None", float]
+] = {}
+_CACHE_TTL = 60.0 * 60.0  # Cache results for 1 hour
+
+
+@dataclass
+class ConnectionResult:
+    host: str
+    success: bool
+    ipv4: bool
+
+
+def _sanitize_host(host: str) -> str:
+    if host.startswith("http"):
+        return host
+    is_local_host = "localhost" in host or "127.0.0.1" in host or "0.0.0.0" in host
+    use_https = not is_local_host
+    if use_https:
+        return host if host.startswith("https://") else f"https://{host}"
+    return host if host.startswith("http://") else f"http://{host}"
+
+
+def _test_connection(host: str, use_ipv4: bool) -> ConnectionResult:
+    # Function static cache
+    host = _sanitize_host(host)
+    transport = httpx.HTTPTransport(local_address="0.0.0.0") if use_ipv4 else None
+    result: ConnectionResult | None = None
+    try:
+        with httpx.Client(
+            timeout=_TIMEOUT,
+            transport=transport,
+        ) as test_client:
+            test_response = test_client.get(
+                f"{host}/healthz", timeout=3, follow_redirects=True
+            )
+            result = ConnectionResult(host, test_response.status_code == 200, use_ipv4)
+    except KeyboardInterrupt:
+        _thread.interrupt_main()
+        result = ConnectionResult(host, False, use_ipv4)
+    except TimeoutError:
+        result = ConnectionResult(host, False, use_ipv4)
+    except Exception:
+        result = ConnectionResult(host, False, use_ipv4)
+    return result
+
+
+def find_good_connection(
+    urls: list[str], filter_out_bad=True, use_ipv6: bool = True
+) -> ConnectionResult | None:
+    # Create cache key from parameters
+    cache_key = (tuple(sorted(urls)), filter_out_bad, use_ipv6)
+    current_time = time.time()
+
+    # Check if we have a cached result
+    if cache_key in _CONNECTION_CACHE:
+        cached_result, cached_time = _CONNECTION_CACHE[cache_key]
+        if current_time - cached_time < _CACHE_TTL:
+            return cached_result
+        else:
+            # Remove expired cache entry
+            del _CONNECTION_CACHE[cache_key]
+
+    # No valid cache entry, perform the actual connection test
+    futures: list[Future] = []
+    for url in urls:
+
+        f = _EXECUTOR.submit(_test_connection, url, use_ipv4=True)
+        futures.append(f)
+        if use_ipv6 and "localhost" not in url:
+            f_v6 = _EXECUTOR.submit(_test_connection, url, use_ipv4=False)
+            futures.append(f_v6)
+
+    result = None
+    try:
+        # Return first successful result
+        for future in as_completed(futures):
+            connection_result: ConnectionResult = future.result()
+            if connection_result.success or not filter_out_bad:
+                result = connection_result
+                break
+    finally:
+        # Cancel any remaining futures
+        for future in futures:
+            future.cancel()
+
+    # Cache the result (even if None)
+    _CONNECTION_CACHE[cache_key] = (result, current_time)
+
+    return result

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/playwright_browser.py

@@ -44,7 +44,7 @@ class PlaywrightBrowser:
         Args:
             headless: Whether to run the browser in headless mode
         """
-        if not PLAYWRIGHT_AVAILABLE:
+        if not is_playwright_available():
             raise ImportError(
                 "Playwright is not installed. Install with: pip install fastled[full]"
             )
@@ -56,31 +56,92 @@ class PlaywrightBrowser:
         self.playwright: Any = None
         self._should_exit = asyncio.Event()
 
+    def _detect_device_scale_factor(self) -> float | None:
+        """Detect the system's device scale factor for natural browser behavior.
+
+        Returns:
+            The detected device scale factor, or None if detection fails or
+            the value is outside reasonable bounds (0.5-4.0).
+        """
+        try:
+            import platform
+
+            if platform.system() == "Windows":
+                import winreg
+
+                try:
+                    key = winreg.OpenKey(
+                        winreg.HKEY_CURRENT_USER, r"Control Panel\Desktop\WindowMetrics"
+                    )
+                    dpi, _ = winreg.QueryValueEx(key, "AppliedDPI")
+                    winreg.CloseKey(key)
+                    device_scale_factor = dpi / 96.0
+
+                    # Validate the scale factor is within reasonable bounds
+                    if 0.5 <= device_scale_factor <= 4.0:
+                        print(
+                            f"[PYTHON] Detected Windows DPI scaling: {device_scale_factor:.2f}"
+                        )
+                        return device_scale_factor
+                    else:
+                        print(
+                            f"[PYTHON] Detected scale factor {device_scale_factor:.2f} outside reasonable bounds"
+                        )
+                        return None
+
+                except (OSError, FileNotFoundError):
+                    print(
+                        "[PYTHON] Could not detect Windows DPI, using browser default"
+                    )
+                    return None
+            else:
+                # Future: Add support for other platforms (macOS, Linux) here
+                print(
+                    f"[PYTHON] Device scale detection not implemented for {platform.system()}"
+                )
+                return None
+
+        except Exception as e:
+            print(f"[PYTHON] DPI detection failed: {e}")
+            return None
+
     async def start(self) -> None:
         """Start the Playwright browser."""
-        if self.playwright is None and async_playwright is not None:
-            self.playwright = await async_playwright().start()
+        if self.browser is None:
+            if is_playwright_available():
+                from playwright.async_api import async_playwright
 
-        if self.browser is None and self.playwright is not None:
-            # Try Chrome first, then Firefox, then WebKit
-            try:
-                self.browser = await self.playwright.chromium.launch(
+                self.playwright = async_playwright()
+                playwright = await self.playwright.start()
+                self.browser = await playwright.chromium.launch(
                     headless=self.headless,
-                    args=["--disable-web-security", "--allow-running-insecure-content"],
+                    args=[
+                        "--disable-dev-shm-usage",
+                        "--disable-web-security",
+                        "--allow-running-insecure-content",
+                    ],
                 )
-            except Exception:
-                try:
-                    self.browser = await self.playwright.firefox.launch(
-                        headless=self.headless
-                    )
-                except Exception:
-                    self.browser = await self.playwright.webkit.launch(
-                        headless=self.headless
-                    )
+            else:
+                raise RuntimeError("Playwright is not available")
 
         if self.page is None and self.browser is not None:
-            # Create a new browser context and page
-            context = await self.browser.new_context()
+            # Detect system device scale factor for natural browser behavior
+            device_scale_factor = self._detect_device_scale_factor()
+
+            # Create browser context with detected or default device scale factor
+            if device_scale_factor:
+                context = await self.browser.new_context(
+                    device_scale_factor=device_scale_factor
+                )
+                print(
+                    f"[PYTHON] Created browser context with device scale factor: {device_scale_factor:.2f}"
+                )
+            else:
+                context = await self.browser.new_context()
+                print(
+                    "[PYTHON] Created browser context with default device scale factor"
+                )
+
             self.page = await context.new_page()
 
     async def open_url(self, url: str) -> None:
@@ -99,6 +160,15 @@ class PlaywrightBrowser:
             # Wait for the page to load
             await self.page.wait_for_load_state("networkidle")
 
+            # Verify device scale factor is working correctly
+            try:
+                device_pixel_ratio = await self.page.evaluate("window.devicePixelRatio")
+                print(
+                    f"[PYTHON] Verified browser device pixel ratio: {device_pixel_ratio}"
+                )
+            except Exception as e:
+                print(f"[PYTHON] Could not verify device pixel ratio: {e}")
+
             # Set up auto-resizing functionality if enabled
             if self.auto_resize:
                 await self._setup_auto_resize()
@@ -268,7 +338,7 @@ def run_playwright_browser(url: str, headless: bool = False) -> None:
         url: The URL to open
         headless: Whether to run in headless mode
     """
-    if not PLAYWRIGHT_AVAILABLE:
+    if not is_playwright_available():
         warnings.warn(
             "Playwright is not installed. Install with: pip install fastled[full]. "
             "Falling back to default browser."
@@ -342,7 +412,7 @@ class PlaywrightBrowserProxy:
             url: The URL to open
             headless: Whether to run in headless mode
         """
-        if not PLAYWRIGHT_AVAILABLE:
+        if not is_playwright_available():
             warnings.warn(
                 "Playwright is not installed. Install with: pip install fastled[full]. "
                 "Falling back to default browser."
@@ -430,7 +500,7 @@ def run_playwright_browser_persistent(url: str, headless: bool = False) -> None:
         url: The URL to open
         headless: Whether to run in headless mode
     """
-    if not PLAYWRIGHT_AVAILABLE:
+    if not is_playwright_available():
         return
 
     async def main():

{fastled-1.4.3 → fastled-1.4.5}/src/fastled/site/build.py

@@ -265,6 +265,7 @@ INDEX_TEMPLATE = """<!DOCTYPE html>
                 checkmark.style.color = '#E0E0E0';
                 link.appendChild(checkmark);
             }});
+            
             // Now load first example and show its checkmark
             if (links.length > 0) {{
                 // Try to find SdCard example first
@@ -273,6 +274,7 @@ INDEX_TEMPLATE = """<!DOCTYPE html>
                 startLink.classList.add('active');
                 startLink.querySelector('.fa-check').style.display = 'inline-block';
             }}
+            
             // Add click handlers
             links.forEach(link => {{
                 link.addEventListener('click', function(e) {{
@@ -311,10 +313,11 @@ INDEX_TEMPLATE = """<!DOCTYPE html>
                     showNav();
                 }}
             }});
+            
             // Close menu when clicking anywhere in the document
             document.addEventListener('click', (e) => {{
-                if (navPane.classList.contains('visible') &&
-                    !navPane.contains(e.target) &&
+                if (navPane.classList.contains('visible') && 
+                    !navPane.contains(e.target) && 
                     !navTrigger.contains(e.target)) {{
                     hideNav();
                 }}