fastled 1.3.32__tar.gz → 1.3.34__tar.gz

{fastled-1.3.32 → fastled-1.3.34}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fastled
-Version: 1.3.32
+Version: 1.3.34
 Summary: FastLED Wasm Compiler
 Home-page: https://github.com/zackees/fastled-wasm
 Maintainer: Zachary Vorhies

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/__init__.py

@@ -64,6 +64,7 @@ class Api:
         http_port: (
             int | None
         ) = None,  # None means auto select a free port. -1 means no server.
+        no_platformio: bool = False,
     ) -> LiveClient:
         return LiveClient(
             sketch_directory=sketch_directory,
@@ -75,6 +76,7 @@ class Api:
             build_mode=build_mode,
             profile=profile,
             http_port=http_port,
+            no_platformio=no_platformio,
         )
 
     @staticmethod
@@ -85,6 +87,7 @@ class Api:
         mapped_dir: Path | None = None,  # Sketch directory.
         container_name: str | None = None,  # Specific docker container name.
         remove_previous: bool = False,
+        no_platformio: bool = False,
     ) -> CompileServer:
         """Uses docker to spawn a compile server from the given name."""
         from fastled.compile_server import CompileServer
@@ -96,6 +99,7 @@ class Api:
             mapped_dir=mapped_dir,
             auto_start=auto_start,
             remove_previous=remove_previous,
+            no_platformio=no_platformio,
         )
         return out
 
@@ -108,6 +112,7 @@ class Api:
         mapped_dir: Path | None = None,  # Sketch directory.
         container_name: str | None = None,  # Specific docker container name.
         remove_previous=False,
+        no_platformio: bool = False,
     ) -> Generator[CompileServer, None, None]:
         server = Api.spawn_server(
             interactive=interactive,
@@ -116,6 +121,7 @@ class Api:
             mapped_dir=mapped_dir,
             container_name=container_name,
             remove_previous=remove_previous,
+            no_platformio=no_platformio,
         )
         try:
             yield server

{fastled-1.3.32 → fastled-1.3.34}/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.3.32"
+__version__ = "1.3.34"
 
 __version_url_latest__ = "https://raw.githubusercontent.com/zackees/fastled-wasm/refs/heads/main/src/fastled/__version__.py"

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/app.py

@@ -27,6 +27,7 @@ def run_server(args: Args) -> int:
         mapped_dir=mapped_dir,
         auto_start=True,
         remove_previous=args.clear,
+        no_platformio=args.no_platformio,
     )
 
     if not interactive:
@@ -87,7 +88,9 @@ def main() -> int:
 
     if update:
         # Force auto_update to ensure update check happens
-        compile_server = CompileServer(interactive=False, auto_updates=True)
+        compile_server = CompileServer(
+            interactive=False, auto_updates=True, no_platformio=args.no_platformio
+        )
         compile_server.stop()
         print("Finished updating.")
         return 0
@@ -109,6 +112,7 @@ def main() -> int:
             mapped_dir=directory,
             auto_start=False,
             remove_previous=args.clear,
+            no_platformio=args.no_platformio,
         )
 
         server.start(wait_for_startup=False)

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/args.py

@@ -12,6 +12,7 @@ class Args:
     interactive: bool
     profile: bool
     force_compile: bool
+    no_platformio: bool
     auto_update: bool | None
     update: bool
     localhost: bool
@@ -47,6 +48,9 @@ class Args:
         assert isinstance(
             args.force_compile, bool
         ), f"expected bool, got {type(args.force_compile)}"
+        assert isinstance(
+            args.no_platformio, bool
+        ), f"expected bool, got {type(args.no_platformio)}"
         assert isinstance(
             args.no_auto_updates, bool | None
         ), f"expected bool | None, got {type(args.no_auto_updates)}"
@@ -78,6 +82,7 @@ class Args:
             interactive=args.interactive,
             profile=args.profile,
             force_compile=args.force_compile,
+            no_platformio=args.no_platformio,
             auto_update=not args.no_auto_updates,
             update=args.update,
             localhost=args.localhost,

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/client_server.py

@@ -130,7 +130,11 @@ def _run_web_compiler(
 
 
 def _try_start_server_or_get_url(
-    auto_update: bool, args_web: str | bool, localhost: bool, clear: bool
+    auto_update: bool,
+    args_web: str | bool,
+    localhost: bool,
+    clear: bool,
+    no_platformio: bool = False,
 ) -> tuple[str, CompileServer | None]:
     is_local_host = localhost or (
         isinstance(args_web, str)
@@ -161,7 +165,9 @@ def _try_start_server_or_get_url(
         try:
             print("No local server found, starting one...")
             compile_server = CompileServer(
-                auto_updates=auto_update, remove_previous=clear
+                auto_updates=auto_update,
+                remove_previous=clear,
+                no_platformio=no_platformio,
             )
             print("Waiting for the local compiler to start...")
             if not compile_server.ping():
@@ -177,7 +183,9 @@ def _try_start_server_or_get_url(
             return (DEFAULT_URL, None)
 
 
-def _try_make_compile_server(clear: bool = False) -> CompileServer | None:
+def _try_make_compile_server(
+    clear: bool = False, no_platformio: bool = False
+) -> CompileServer | None:
     if not DockerManager.is_docker_installed():
         return None
     try:
@@ -189,7 +197,9 @@ def _try_make_compile_server(clear: bool = False) -> CompileServer | None:
         free_port = find_free_port(start_port=9723, end_port=9743)
         if free_port is None:
             return None
-        compile_server = CompileServer(auto_updates=False, remove_previous=clear)
+        compile_server = CompileServer(
+            auto_updates=False, remove_previous=clear, no_platformio=no_platformio
+        )
         print("Waiting for the local compiler to start...")
         if not compile_server.ping():
             print("Failed to start local compiler.")
@@ -228,13 +238,16 @@ def run_client(
         int | None
     ) = None,  # None means auto select a free port, http_port < 0 means no server.
     clear: bool = False,
+    no_platformio: bool = False,
 ) -> int:
     has_checked_newer_version_yet = False
     compile_server: CompileServer | None = None
 
     if host is None:
         # attempt to start a compile server if docker is installed.
-        compile_server = _try_make_compile_server(clear=clear)
+        compile_server = _try_make_compile_server(
+            clear=clear, no_platformio=no_platformio
+        )
         if compile_server is None:
             host = DEFAULT_URL
     elif isinstance(host, CompileServer):
@@ -463,6 +476,7 @@ def run_client_server(args: Args) -> int:
     force_compile = bool(args.force_compile)
     open_web_browser = not just_compile and not interactive
     build_mode: BuildMode = BuildMode.from_args(args)
+    no_platformio = bool(args.no_platformio)
 
     if not force_compile and not looks_like_sketch_directory(directory):
         # if there is only one directory in the sketch directory, use that
@@ -492,7 +506,7 @@ def run_client_server(args: Args) -> int:
     compile_server: CompileServer | None = None
     try:
         url, compile_server = _try_start_server_or_get_url(
-            auto_update, web, localhost, args.clear
+            auto_update, web, localhost, args.clear, no_platformio
         )
     except KeyboardInterrupt:
         print("\nExiting from first try...")
@@ -524,6 +538,7 @@ def run_client_server(args: Args) -> int:
             build_mode=build_mode,
             profile=profile,
             clear=args.clear,
+            no_platformio=no_platformio,
         )
     except KeyboardInterrupt:
         return 1

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/compile_server.py

@@ -15,6 +15,7 @@ class CompileServer:
         container_name: str | None = None,
         platform: Platform = Platform.WASM,
         remove_previous: bool = False,
+        no_platformio: bool = False,
     ) -> None:
         from fastled.compile_server_impl import (  # avoid circular import
             CompileServerImpl,
@@ -29,6 +30,7 @@ class CompileServer:
             mapped_dir=mapped_dir,
             auto_start=auto_start,
             remove_previous=remove_previous,
+            no_platformio=no_platformio,
         )
 
     # May throw CompileServerError if server could not be started.

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/compile_server_impl.py

@@ -50,6 +50,7 @@ class CompileServerImpl:
         auto_start: bool = True,
         container_name: str | None = None,
         remove_previous: bool = False,
+        no_platformio: bool = False,
     ) -> None:
         container_name = container_name or DEFAULT_CONTAINER_NAME
         if interactive and not mapped_dir:
@@ -68,6 +69,7 @@ class CompileServerImpl:
         self.running_container: RunningContainer | None = None
         self.auto_updates = auto_updates
         self.remove_previous = remove_previous
+        self.no_platformio = no_platformio
         self._port = 0  # 0 until compile server is started
         if auto_start:
             self.start()
@@ -217,6 +219,8 @@ class CompileServerImpl:
             server_command = ["/bin/bash"]
         else:
             server_command = ["python", "/js/run.py", "server"] + SERVER_OPTIONS
+            if self.no_platformio:
+                server_command.append("--no-platformio")
         if self.interactive:
             print("Disabling port forwarding in interactive mode")
             ports = {}

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/live_client.py

@@ -22,6 +22,7 @@ class LiveClient:
         keep_running: bool = True,
         build_mode: BuildMode = BuildMode.QUICK,
         profile: bool = False,
+        no_platformio: bool = False,
     ) -> None:
         self.sketch_directory = sketch_directory
         self.host = host
@@ -34,6 +35,7 @@ class LiveClient:
         self.shutdown = threading.Event()
         self.thread: threading.Thread | None = None
         self.auto_updates = auto_updates
+        self.no_platformio = no_platformio
         if auto_start:
             self.start()
         if self.auto_updates is False:
@@ -52,6 +54,7 @@ class LiveClient:
             profile=self.profile,
             shutdown=self.shutdown,
             http_port=self.http_port,
+            no_platformio=self.no_platformio,
         )
         return rtn
 

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/parse_args.py

@@ -30,6 +30,7 @@ FastLED WASM Compiler - Useful options:
   --init [example]      Initialize one of the top tier WASM examples
   --web [url]           Use web compiler
   --server              Run the compiler server
+  --no-platformio       Bypass PlatformIO constraints using local Docker compilation
   --quick               Build in quick mode (default)
   --profile             Enable profiling the C++ build system
   --update              Update the docker image for the wasm compiler
@@ -114,6 +115,11 @@ def parse_args() -> Args:
         action="store_true",
         help="Disable automatic updates of the wasm compiler image when using docker.",
     )
+    parser.add_argument(
+        "--no-platformio",
+        action="store_true",
+        help="Bypass PlatformIO constraints by using local Docker compilation with custom build environment",
+    )
     parser.add_argument(
         "-u",
         "--update",
@@ -268,6 +274,12 @@ def parse_args() -> Args:
         if cwd_is_fastled and not args.web and not args.server:
             print("Forcing --local mode because we are in the FastLED repo")
             args.localhost = True
+        if args.no_platformio:
+            print(
+                "--no-platformio mode enabled: forcing local Docker compilation to bypass PlatformIO constraints"
+            )
+            args.localhost = True
+            args.web = None  # Clear web flag to ensure local compilation
         if args.localhost:
             args.web = "localhost"
         if args.interactive and not args.server:

{fastled-1.3.32 → fastled-1.3.34}/src/fastled/server_flask.py

@@ -85,6 +85,14 @@ def _run_flask_server(
 
         # logger.error(f"Server error: {e}")
 
+        @app.after_request
+        def add_security_headers(response):
+            """Add security headers required for cross-origin isolation and audio worklets"""
+            # Required for SharedArrayBuffer and audio worklets
+            response.headers["Cross-Origin-Embedder-Policy"] = "credentialless"
+            response.headers["Cross-Origin-Opener-Policy"] = "same-origin"
+            return response
+
         @app.before_request
         def log_request_info():
             """Log details of each request before processing"""

{fastled-1.3.32 → fastled-1.3.34}/src/fastled.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fastled
-Version: 1.3.32
+Version: 1.3.34
 Summary: FastLED Wasm Compiler
 Home-page: https://github.com/zackees/fastled-wasm
 Maintainer: Zachary Vorhies

{fastled-1.3.32 → fastled-1.3.34}/src/fastled.egg-info/SOURCES.txt

@@ -95,11 +95,13 @@ tests/unit/test_api.py
 tests/unit/test_bad_ino.py
 tests/unit/test_banner_string.py
 tests/unit/test_cli.py
+tests/unit/test_cli_no_platformio.py
 tests/unit/test_compile_server.py
 tests/unit/test_debug_fetch_source_files.py
 tests/unit/test_docker_linux_on_windows.py
 tests/unit/test_embedded_data.py
 tests/unit/test_filechanger.py
+tests/unit/test_flask_headers.py
 tests/unit/test_http_server.py
 tests/unit/test_manual_api_invocation.py
 tests/unit/test_no_platformio_compile.py

fastled-1.3.34/tests/unit/test_cli_no_platformio.py

@@ -0,0 +1,225 @@
+"""
+Unit test for FastLED CLI --no-platformio flag functionality.
+Tests that the CLI correctly handles the --no-platformio argument and compiles successfully.
+"""
+
+import os
+import platform
+import subprocess
+import unittest
+from pathlib import Path
+
+HERE = Path(__file__).parent
+TEST_SKETCH_DIR = HERE / "test_ino" / "wasm"
+WORKSPACE_ROOT = HERE.parent.parent
+
+
+def _enabled() -> bool:
+    """Check if this system can run the tests."""
+    is_github_runner = "GITHUB_ACTIONS" in os.environ
+    if not is_github_runner:
+        return True
+    # This only works in ubuntu at the moment
+    return platform.system() == "Linux"
+
+
+def _docker_available() -> bool:
+    """Check if Docker is available for no-platformio compilation."""
+    try:
+        from fastled.docker_manager import DockerManager
+
+        return DockerManager.is_docker_installed()
+    except Exception as e:
+        print(f"Docker is not available: {e}")
+        return False
+
+
+class CLINoPlatformIOTest(unittest.TestCase):
+    """Test FastLED CLI --no-platformio flag functionality."""
+
+    def test_no_platformio_flag_recognized(self) -> None:
+        """Test that --no-platformio flag is recognized by the CLI without errors."""
+
+        # Test that the flag is recognized in help
+        result = subprocess.run(
+            ["uv", "run", "fastled", "--help"],
+            cwd=WORKSPACE_ROOT,
+            capture_output=True,
+            text=True,
+            timeout=30,
+        )
+
+        self.assertEqual(result.returncode, 0, "Help command should succeed")
+        self.assertIn(
+            "--no-platformio", result.stdout, "--no-platformio should appear in help"
+        )
+        self.assertIn(
+            "Bypass PlatformIO constraints",
+            result.stdout,
+            "Help text should be present",
+        )
+
+    def test_no_platformio_flag_parsing(self) -> None:
+        """Test that --no-platformio flag is parsed correctly without compilation."""
+
+        # Ensure test sketch directory exists
+        self.assertTrue(
+            TEST_SKETCH_DIR.exists(),
+            f"Test sketch directory not found: {TEST_SKETCH_DIR}",
+        )
+
+        # Test with --help to verify flag parsing without triggering compilation
+        result = subprocess.run(
+            ["uv", "run", "fastled", str(TEST_SKETCH_DIR), "--no-platformio", "--help"],
+            cwd=WORKSPACE_ROOT,
+            capture_output=True,
+            text=True,
+            timeout=30,
+        )
+
+        self.assertEqual(result.returncode, 0, "CLI should accept --no-platformio flag")
+        self.assertIn(
+            "--no-platformio", result.stdout, "--no-platformio should be in help output"
+        )
+
+    @unittest.skipUnless(
+        _enabled(),
+        "Test requires Linux environment",
+    )
+    def test_no_platformio_flag_forces_local_mode(self) -> None:
+        """Test that --no-platformio flag correctly forces local Docker compilation mode."""
+
+        # Ensure test sketch directory exists
+        self.assertTrue(
+            TEST_SKETCH_DIR.exists(),
+            f"Test sketch directory not found: {TEST_SKETCH_DIR}",
+        )
+
+        # Run with --no-platformio and --just-compile to avoid browser opening
+        # Use a short timeout to avoid long waits if compilation hangs
+        try:
+            result = subprocess.run(
+                [
+                    "uv",
+                    "run",
+                    "fastled",
+                    str(TEST_SKETCH_DIR),
+                    "--no-platformio",
+                    "--just-compile",
+                ],
+                cwd=WORKSPACE_ROOT,
+                capture_output=True,
+                text=True,
+                timeout=180,  # 3 minutes timeout
+            )
+
+            # Check that the --no-platformio message appears
+            output = result.stdout + result.stderr
+            self.assertIn(
+                "--no-platformio mode enabled: forcing local Docker compilation to bypass PlatformIO constraints",
+                output,
+                "Should display --no-platformio mode message",
+            )
+
+            # Check that it attempts local compilation (even if Docker isn't available)
+            self.assertTrue(
+                "localhost" in output.lower()
+                or "local" in output.lower()
+                or "docker" in output.lower(),
+                "Should attempt local compilation mode",
+            )
+
+            # If compilation succeeded, verify success
+            if result.returncode == 0:
+                self.assertIn(
+                    "compilation success",
+                    output.lower(),
+                    "Should indicate compilation success",
+                )
+                print("✅ --no-platformio CLI compilation succeeded!")
+            else:
+                # If it failed, it should be due to Docker not being available or similar infrastructure issue
+                # The important thing is that the flag was recognized and processed
+                self.assertNotIn(
+                    "unrecognized arguments", output, "Flag should be recognized"
+                )
+                print(
+                    f"ℹ️  --no-platformio flag processed correctly (exit code: {result.returncode})"
+                )
+                print(
+                    f"Output: {output[:500]}..."
+                )  # Print first 500 chars for debugging
+
+        except subprocess.TimeoutExpired:
+            self.fail(
+                "Command timed out - this suggests the flag was processed but compilation took too long"
+            )
+
+    def test_no_platformio_cli_argument_structure(self) -> None:
+        """Test the CLI argument structure for --no-platformio flag."""
+
+        # Test that the flag doesn't conflict with other flags
+        conflicting_flags = [
+            ["--web"],
+            ["--server"],
+            ["--debug"],
+            ["--quick"],
+            ["--release"],
+        ]
+
+        for flags in conflicting_flags:
+            with self.subTest(flags=flags):
+                # Test help output to ensure no argument conflicts
+                cmd = ["uv", "run", "fastled", "--no-platformio"] + flags + ["--help"]
+                result = subprocess.run(
+                    cmd, cwd=WORKSPACE_ROOT, capture_output=True, text=True, timeout=30
+                )
+
+                self.assertEqual(
+                    result.returncode, 0, f"Should accept --no-platformio with {flags}"
+                )
+                self.assertNotIn(
+                    "error:",
+                    result.stderr.lower(),
+                    f"No errors with --no-platformio + {flags}",
+                )
+
+    def test_no_platformio_with_different_sketch_directories(self) -> None:
+        """Test --no-platformio flag with different test sketch directories."""
+
+        test_sketches = [
+            TEST_SKETCH_DIR,  # wasm sketch
+            HERE / "test_ino" / "embedded",  # embedded sketch
+        ]
+
+        for sketch_dir in test_sketches:
+            if sketch_dir.exists():
+                with self.subTest(sketch_dir=sketch_dir):
+                    # Test with --help to verify flag parsing
+                    result = subprocess.run(
+                        [
+                            "uv",
+                            "run",
+                            "fastled",
+                            str(sketch_dir),
+                            "--no-platformio",
+                            "--help",
+                        ],
+                        cwd=WORKSPACE_ROOT,
+                        capture_output=True,
+                        text=True,
+                        timeout=30,
+                    )
+
+                    self.assertEqual(
+                        result.returncode, 0, f"Should work with sketch at {sketch_dir}"
+                    )
+                    self.assertIn(
+                        "--no-platformio",
+                        result.stdout,
+                        "Flag should be present in help",
+                    )
+
+
+if __name__ == "__main__":
+    unittest.main()