Fix ASCII encoding crash in packaged app; add bundle smoke test

The macOS .app bundle runs with LANG=C, causing Python to default to
ASCII encoding. Alembic's fileConfig() then choked on non-ASCII chars
(em dash, arrow) in alembic.ini and run.py. Fixed by:
- Replacing non-ASCII chars in runtime-read files
- Adding encoding="utf-8" to fileConfig() call

Also adds scripts/test_app_bundle.py and Makefile targets (pack,
test-bundle, pack-and-test) for local smoke-testing before releases.

Made-with: Cursor

Author: clstaudt

Makefile

@@ -81,6 +81,14 @@ dist: clean ## builds source and wheel package
 	uv build
 	ls -l dist
 
+pack: clean-build ## build the app bundle with flet pack
+	uv run python scripts/pack_app.py
+
+test-bundle: ## smoke-test the packaged app bundle (build first with `make pack`)
+	uv run python scripts/test_app_bundle.py
+
+pack-and-test: pack test-bundle ## build and smoke-test the app bundle
+
 install: ## install the package with uv
 	uv sync
 

scripts/test_app_bundle.py

@@ -0,0 +1,110 @@
+"""Smoke-test for the packaged app bundle.
+
+Launches the platform-specific executable produced by ``flet pack``
+(via ``scripts/pack_app.py``) and verifies it survives initial startup
+without crashing.
+
+Usage:
+    uv run python scripts/test_app_bundle.py          # test only
+    uv run python scripts/test_app_bundle.py --build   # build then test
+"""
+
+import os
+import signal
+import subprocess
+import sys
+import time
+from pathlib import Path
+
+import typer
+from loguru import logger
+
+APP_NAME = "Tuttle"
+DIST_DIR = Path(__file__).resolve().parent.parent / "dist"
+STARTUP_WAIT_SECONDS = 8
+
+
+def _find_executable() -> Path:
+    if sys.platform.startswith("darwin"):
+        exe = DIST_DIR / f"{APP_NAME}.app" / "Contents" / "MacOS" / APP_NAME
+    elif sys.platform.startswith("win"):
+        exe = DIST_DIR / f"{APP_NAME}.exe"
+    else:
+        exe = DIST_DIR / APP_NAME
+    return exe
+
+
+def _build():
+    logger.info("Building app bundle ...")
+    result = subprocess.run(
+        [sys.executable, "scripts/pack_app.py"],
+        cwd=DIST_DIR.parent,
+    )
+    if result.returncode != 0:
+        logger.error(f"Build failed with exit code {result.returncode}")
+        raise typer.Exit(code=1)
+    logger.info("Build succeeded")
+
+
+def _launch_and_check(exe: Path) -> bool:
+    """Return True if the app survives startup without crashing."""
+    logger.info(f"Launching {exe}")
+    env = {**os.environ, "LANG": "C"}  # simulate .app bundle locale
+
+    proc = subprocess.Popen(
+        [str(exe)],
+        stdout=subprocess.PIPE,
+        stderr=subprocess.PIPE,
+        start_new_session=True,
+        env=env,
+    )
+
+    try:
+        logger.info(f"Waiting {STARTUP_WAIT_SECONDS}s for startup ...")
+        time.sleep(STARTUP_WAIT_SECONDS)
+        exit_code = proc.poll()
+
+        if exit_code is not None:
+            _, stderr = proc.communicate(timeout=5)
+            logger.error(
+                f"App crashed during startup (exit code {exit_code})\n"
+                f"stderr:\n{stderr.decode(errors='replace')}"
+            )
+            return False
+
+        logger.info("App is still running after startup — smoke test passed")
+        return True
+
+    finally:
+        try:
+            pgid = os.getpgid(proc.pid)
+            os.killpg(pgid, signal.SIGTERM)
+            proc.wait(timeout=5)
+        except (ProcessLookupError, ChildProcessError, subprocess.TimeoutExpired):
+            try:
+                proc.kill()
+                proc.wait(timeout=3)
+            except Exception:
+                pass
+        logger.info("App process terminated")
+
+
+def main(
+    build: bool = typer.Option(False, "--build", "-b", help="Build before testing"),
+):
+    if build:
+        _build()
+
+    exe = _find_executable()
+    if not exe.exists():
+        logger.error(
+            f"Executable not found at {exe}\n" "Run with --build or `make pack` first."
+        )
+        raise typer.Exit(code=1)
+
+    ok = _launch_and_check(exe)
+    raise typer.Exit(code=0 if ok else 1)
+
+
+if __name__ == "__main__":
+    typer.run(main)

tuttle/migrations/alembic.ini

@@ -4,7 +4,7 @@
 # not from this file. See env.py for details.
 
 [alembic]
-# path to migration scripts — relative to this file
+# path to migration scripts -- relative to this file
 script_location = %(here)s
 
 # template used to generate migration file names

tuttle/migrations/env.py

@@ -37,7 +37,7 @@
 
 # Interpret the config file for Python logging, if present
 if config.config_file_name is not None:
-    fileConfig(config.config_file_name)
+    fileConfig(config.config_file_name, encoding="utf-8")
 
 # The target metadata for autogenerate support
 target_metadata = SQLModel.metadata

tuttle/migrations/run.py

@@ -83,7 +83,7 @@ def run_migrations(db_url: str) -> None:
             logger.debug(f"Database is already at head revision ({head})")
             return
 
-        logger.info(f"Running migrations: {current} → {head}")
+        logger.info(f"Running migrations: {current} -> {head}")
         command.upgrade(cfg, "head")
         logger.info("Migrations completed successfully")