riplex 1.0.2__tar.gz → 1.0.4__tar.gz

{riplex-1.0.2 → riplex-1.0.4}/.github/workflows/release.yml

@@ -50,6 +50,7 @@ jobs:
         run: |
           for /f "delims=" %%i in ('python -c "import flet_desktop; print(flet_desktop.ensure_client_cached())"') do set FLET_CLIENT_CACHE=%%i
           pyinstaller --name riplex-ui --onefile --windowed ^
+            --icon src/riplex_app/assets/riplex-icon.ico ^
             --hidden-import riplex_app.screens.welcome ^
             --hidden-import riplex_app.screens.disc_detection ^
             --hidden-import riplex_app.screens.metadata ^
@@ -69,6 +70,7 @@ jobs:
             --collect-all flet_desktop ^
             --collect-data certifi ^
             --collect-data dvdcompare_scraper ^
+            --add-data "src/riplex_app/assets/riplex-icon.ico;riplex_app/assets" ^
             --add-data "%FLET_CLIENT_CACHE%\flet;flet_client" ^
             src/riplex_app/main.py
         shell: cmd
@@ -128,6 +130,7 @@ jobs:
       - name: Build GUI app bundle
         run: |
           pyinstaller --name riplex-ui --onedir --windowed \
+            --icon src/riplex_app/assets/riplex-icon.icns \
             --hidden-import riplex_app.screens.welcome \
             --hidden-import riplex_app.screens.disc_detection \
             --hidden-import riplex_app.screens.metadata \
@@ -192,12 +195,6 @@ jobs:
           cp artifacts/riplex-windows/riplex-ui.exe release/riplex-ui-windows.exe
           cp artifacts/riplex-macos/riplex-macos release/
           cp artifacts/riplex-macos/riplex-ui-macos.zip release/
-          # Publish SHA-256 checksums so the app can verify a download before
-          # applying an in-place update (see riplex.updater.stage_update).
-          cd release
-          for f in riplex-windows.exe riplex-ui-windows.exe riplex-macos riplex-ui-macos.zip; do
-            sha256sum "$f" > "$f.sha256"
-          done
 
       - name: Create GitHub Release
         uses: softprops/action-gh-release@v2
@@ -210,10 +207,6 @@ jobs:
           body: ${{ steps.tag_notes.outputs.body }}
           files: |
             release/riplex-windows.exe
-            release/riplex-windows.exe.sha256
             release/riplex-ui-windows.exe
-            release/riplex-ui-windows.exe.sha256
             release/riplex-macos
-            release/riplex-macos.sha256
             release/riplex-ui-macos.zip
-            release/riplex-ui-macos.zip.sha256

{riplex-1.0.2/src/riplex.egg-info → riplex-1.0.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: riplex
-Version: 1.0.2
+Version: 1.0.4
 Summary: Automates the tedious manual work around MakeMKV: figuring out what to rip, which MKV files are actually what, and organizing everything into Plex-compatible folder structures.
 License: MIT
 Requires-Python: >=3.11
@@ -8,7 +8,7 @@ Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: certifi>=2024.2.2
 Requires-Dist: httpx>=0.27
-Requires-Dist: dvdcompare-scraper>=0.1.18
+Requires-Dist: dvdcompare-scraper>=0.1.15
 Requires-Dist: platformdirs>=4.0
 Provides-Extra: dev
 Requires-Dist: pytest>=8.0; extra == "dev"

{riplex-1.0.2 → riplex-1.0.4}/RELEASE_NOTES_v1.0.0.md

@@ -5,32 +5,12 @@ support, hardens episode matching, adds a comprehensive automated test suite so
 the flows that matter stay working as the project grows, and makes upgrades
 painless with one-click in-place updates on Windows.
 
-Until now every disc was treated as a standalone rip, and updating meant
-re-downloading the app each release. This release lets riplex understand a whole
-box set at once — detecting the *works* inside a release, grouping the discs that
-belong to each, and routing every work to its correct Plex destination in a
-single guided session — and then keeps itself up to date automatically.
-
-## Why 1.0.0?
-
-riplex has been usable for a while; three things make this the right moment to
-call it stable:
-
-- **The hard workflows are done.** Multi-work box sets and complete multi-season
-  TV series — the cases that used to mean ripping disc-by-disc and sorting by
-  hand — now work end to end, in both the desktop app and the CLI.
-- **It's protected against regressions.** A headless integration suite exercises
-  the entire GUI wizard and the CLI against real-world scenarios generated from
-  archived rips, so the flows you depend on keep working as development continues.
-- **Updates are effortless.** In-place updates mean you install once and never
-  have to re-download and re-approve the app for each new version.
-
 ## Headline features
 
 ### Multi-work box sets
 
-A release that contains several distinct works — several films, or films plus a
-TV series — is detected as *multiple works* instead of one. Each work gets its
+A release that contains several distinct works - several films, or films plus a
+TV series - is detected as *multiple works* instead of one. Each work gets its
 own title selection, its own rip grouping, and its own Plex target, so a mixed
 box set is ripped and filed correctly in one pass:
 
@@ -38,6 +18,9 @@ box set is ripped and filed correctly in one pass:
 • The Select Titles screen shows each work's rip output separately, so you can see exactly which files belong to which movie or show.
 • Full CLI parity: `riplex orchestrate` and `riplex organize` route multi-work releases the same way the GUI does, through the shared flow logic.
 
+
+![Disc Overview grouping a multi-disc box set](https://raw.githubusercontent.com/AnyCredit5518/riplex/main/screenshots/v1.0.0_Disc_Overview.png)
+
 ### Multi-season TV series
 
 Complete-series sets that span several seasons are ripped season by season:
@@ -46,12 +29,15 @@ Complete-series sets that span several seasons are ripped season by season:
 • TV rips nest automatically under `Season NN/`, and specials land in `Season 00`.
 • Season labels and chips appear throughout the Disc Overview, so a 12-disc set is easy to keep track of.
 
+
+![Season Select picker for a multi-season series](https://raw.githubusercontent.com/AnyCredit5518/riplex/main/screenshots/v1.0.0_Multi_Season_Select.png)
+
 ### Guided resume across a whole set
 
 Every session writes a `_riplex_session.json` marker capturing the full plan, so
 you can rip a few discs, come back later, and pick up exactly where you left off:
 
-• Resume from *any* disc of a multi-work or multi-season set — not just the next in sequence.
+• Resume from *any* disc of a multi-work or multi-season set - not just the next in sequence.
 • `riplex orchestrate` (CLI) resumes from the marker with GUI parity, via a shared `resume.py` adapter.
 • `riplex organize` discovers the marker and fans out across every work in the set.
 
@@ -59,13 +45,13 @@ you can rip a few discs, come back later, and pick up exactly where you left off
 
 Rip manifests record the exact TMDb and dvdcompare ids you confirmed at rip time.
 When you organize (or resume) a ripped disc, riplex reuses those ids and skips
-the metadata pickers entirely — no re-searching, no risk of picking a different
+the metadata pickers entirely - no re-searching, no risk of picking a different
 release the second time.
 
 ### Smarter, more resilient TV episode matching
 
 Matching ripped titles to episodes used to be a runtime guessing game that falls
-apart when episodes run within seconds of each other — or when a metadata
+apart when episodes run within seconds of each other - or when a metadata
 listing is simply wrong. Now:
 
 • dvdcompare's episode listing is cross-referenced against the TMDb episode list.
@@ -73,11 +59,14 @@ listing is simply wrong. Now:
 • Organize honors the rip-time season/episode classification and no longer collides identically-named files across different discs.
 • Over-length titles are labeled **Unmatched content** and stay visible instead of being forced into the wrong slot.
 
+
+![Organize preview with per-file and target durations](https://raw.githubusercontent.com/AnyCredit5518/riplex/main/screenshots/v1.0.0_Organize_Preview.png)
+
 ### Seamless in-place updates (Windows)
 
 When a new version is available, the update screen offers **Update & Restart**:
 riplex downloads the new build, **verifies its SHA-256 checksum**, swaps the
-running `.exe` in place, and relaunches — no re-downloading and no repeated
+running `.exe` in place, and relaunches - no re-downloading and no repeated
 SmartScreen approval. It only runs from a writable install folder (otherwise it
 falls back to the browser download), and every release now publishes `.sha256`
 checksums for its assets.
@@ -86,10 +75,13 @@ checksums for its assets.
 > still uses the browser download (the old build doesn't know how to self-update);
 > from v1.0.0 onward, updates are one click.
 
+
+![In-place update screen with release notes](https://raw.githubusercontent.com/AnyCredit5518/riplex/main/screenshots/v1.0.0_In_Place_Update.png)
+
 ### Auto-eject after ripping
 
-The disc ejects automatically once a rip finishes, so you can swap discs — or
-just know it's done — without reaching for the drive. On by default; set
+The disc ejects automatically once a rip finishes, so you can swap discs - or
+just know it's done - without reaching for the drive. On by default; set
 `auto_eject = false` in the config to disable.
 
 ## Reliability & testing
@@ -130,7 +122,7 @@ break the flows you rely on:
 
 Disc-group ids changed from the old `main_1` / `film_31` scheme to
 `disc_1` / `discs_1_4` / `disc_31`. A session saved by an earlier version will
-not resume — **start a fresh session after upgrading**.
+not resume - **start a fresh session after upgrading**.
 
 ## Full changelog
 

{riplex-1.0.2 → riplex-1.0.4}/docs/changelog.md

@@ -10,6 +10,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
 - **TV episodes with parenthetical titles (or filed under extras) no longer land in `Other/`.** When organizing a TV rip, the destination is now chosen from the authoritative `SxxEyy` classification recorded in each disc's rip manifest at rip time, taking priority over the re-derived dvdcompare match label. Previously an episode whose title contains a parenthetical (e.g. *Shawn and Gus in Drag (Racing)*), or one that dvdcompare files under a disc's extras (e.g. an *(Extended Version)* listing such as *Romeo and Juliet and Juliet* or *Dual Spires*), was misread as an extra and dropped into `Other/` instead of its season folder — even though orchestrate had already classified it correctly (e.g. Psych S5 E01/E05/E12).
 
+- **makemkvcon drive-listing timeout is now configurable (default raised to 120s).** `makemkvcon -r info list` previously timed out after a fixed 60 seconds; on some setups this fired even though the MakeMKV GUI worked fine. Raise `makemkv_list_timeout` in the config if a slow drive still times out. (#23)
+- **Disc and season detection for compact `SxDy` volume labels.** A label that glues the disc digit to the season token and trails a provider code — e.g. `EXPANSE_S3D1_UPB75` — now resolves to the correct title (*Expanse*), season (3) and disc (1). Previously the season was lost and disc detection fell through to duration matching, which could pick the wrong disc. (#18)
+
 ### Added
 
 - **Durations on the organize preview.** Each matched file now shows its runtime next to the matched episode/target's expected runtime; a divergence of more than a minute is highlighted so an incorrect match is easy to spot before you execute.

riplex-1.0.4/issues/thediscdb-metadata-source.md

@@ -0,0 +1,77 @@
+# Metadata source: thediscdb.com (per-disc file/extra data)
+
+Tracking: GitHub issue #19.
+
+## Problem
+
+dvdcompare.net is riplex's only structured disc source. It lists a disc's
+episodes and extras with runtimes, but it does **not** map those to the disc's
+actual title files, and its extra typing is coarse. In practice that means:
+
+- Extras frequently all collapse into `Featurettes/`, and some extras aren't
+  detected at all (reported in #19).
+- Matching relies on runtime heuristics, which is fragile when several
+  features run within seconds of each other.
+
+[thediscdb.com](https://thediscdb.com) publishes, per physical release, the
+**exact source file names** on each disc mapped to their content **titles** and
+a proper **type** (episode, deleted scene, featurette, trailer, commentary,
+etc.). Example: the Fight Club 2026 4K steelbook Blu-ray
+(<https://thediscdb.com/movie/fight-club-1999/releases/2026-steelbook-4k/discs/blu-ray>).
+
+Crucially, the data is open on GitHub — <https://github.com/TheDiscDb/data> —
+as structured files, so it can be consumed without scraping HTML.
+
+## Why this is a good fit
+
+riplex already has a clean provider seam: dvdcompare is wrapped behind
+`src/riplex/disc/provider.py` producing `PlannedDisc` / `PlannedEpisode` /
+`PlannedExtra` objects, and the organizer/matcher consume those. A second
+source that yields the same shapes can slot in behind the same seam.
+
+If TheDiscDb maps titles to *source file names*, it can also make matching
+**deterministic** (map ripped MKVs by playlist / file identity) instead of
+runtime-based, and give each extra a correct Plex folder from its declared
+type — directly addressing the "everything becomes Featurettes" complaint.
+
+## Proposed approach
+
+1. **Data access.** Prefer the GitHub `TheDiscDb/data` repo over live HTML.
+   Options: (a) fetch the specific release file on demand via the GitHub raw
+   API and cache it (mirrors the existing dvdcompare cache in `cache.py`), or
+   (b) let the user point at a local clone. Start with on-demand + cache.
+2. **New provider.** Add `src/riplex/disc/thediscdb.py` exposing the same
+   lookup surface as the dvdcompare provider, returning `PlannedDisc`/
+   `PlannedEpisode`/`PlannedExtra` (extend `PlannedExtra.feature_type` mapping
+   so TheDiscDb's types resolve to Plex extras folders via
+   `organizer._EXTRAS_FOLDER_MAP`).
+3. **Selection / precedence.** When both sources match a release, prefer
+   TheDiscDb for extras typing + file mapping, fall back to dvdcompare for
+   runtimes it lacks. Needs a release-matching step (title + year + edition).
+4. **Deterministic matching (stretch).** When TheDiscDb provides source file
+   names, match ripped files by name/playlist identity in the matcher before
+   the runtime pass.
+
+## Scope / decisions to settle
+
+- **Release identification.** dvdcompare and TheDiscDb slug releases
+  differently; we need a mapping/search step (title+year, then edition). How to
+  disambiguate multiple releases of the same film is the main design question.
+- **Coverage.** TheDiscDb is community-maintained and sparser than dvdcompare
+  for TV. Treat it as an *augmenting* source, not a replacement — keep
+  dvdcompare as the default and layer TheDiscDb when a release is found.
+- **Data schema.** Pin to the `TheDiscDb/data` file format; add a small parser
+  + fixtures. Version/format drift should degrade gracefully (fall back to
+  dvdcompare), mirroring how the dvdcompare cache invalidates on scraper
+  version change.
+- **Config.** A `thediscdb_enabled` (or source-priority) config key, off or
+  best-effort by default until coverage is proven.
+- **Testing.** Unit-test the parser against committed TheDiscDb fixtures;
+  extend the matcher/organizer tests to cover file-name-based matching and
+  richer extra typing.
+
+## Related
+
+- Provider seam: `src/riplex/disc/provider.py`, `cache.py` (caching pattern).
+- Extras typing: `organizer._EXTRAS_FOLDER_MAP` / `_extras_folder`.
+- The "everything → Featurettes" symptom this would improve is the core of #19.

{riplex-1.0.2 → riplex-1.0.4}/pyproject.toml

@@ -12,7 +12,7 @@ license = {text = "MIT"}
 dependencies = [
     "certifi>=2024.2.2",
     "httpx>=0.27",
-    "dvdcompare-scraper>=0.1.18",
+    "dvdcompare-scraper>=0.1.15",
     "platformdirs>=4.0",
 ]
 
@@ -53,12 +53,12 @@ riplex-ui = "riplex_app.main:main"
 [tool.setuptools.packages.find]
 where = ["src"]
 
+[tool.setuptools.package-data]
+riplex_app = ["assets/*"]
+
 [tool.setuptools_scm]
 version_scheme = "guess-next-dev"
 local_scheme = "no-local-version"
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
-markers = [
-    "integration: end-to-end GUI flow tests that drive the wizard headlessly",
-]

riplex-1.0.4/scripts/gui_screenshot.py

@@ -0,0 +1,296 @@
+"""Headless screen-screenshot launcher for release notes / docs.
+
+Renders any riplex-ui screen, seeded with fixture data, in Flet's *web* mode and
+captures a PNG with headless Playwright chromium -- no window ever appears.
+
+How it works:
+  * ``--_serve`` (internal) exports the Flet app as an ASGI app and serves it
+    with uvicorn on a port, with ``app.state`` seeded for one screen.
+  * The default (orchestrator) mode spawns that server as a subprocess, waits
+    for the port, screenshots ``http://127.0.0.1:<port>`` with Playwright, and
+    tears the server down.
+
+Usage:
+    python scripts/gui_screenshot.py --list
+    python scripts/gui_screenshot.py --screen organize_preview --out screenshots
+    python scripts/gui_screenshot.py --all --out screenshots
+
+Requires the dev + gui extras plus ``playwright`` and a chromium download
+(``python -m playwright install chromium``).
+"""
+
+from __future__ import annotations
+
+import argparse
+import socket
+import subprocess
+import sys
+import tempfile
+import time
+from pathlib import Path
+
+_REPO_ROOT = Path(__file__).resolve().parent.parent
+if str(_REPO_ROOT) not in sys.path:
+    sys.path.insert(0, str(_REPO_ROOT))
+
+
+# ---------------------------------------------------------------------------
+# Per-screen state overrides — the fixtures don't carry multi-work / multi-
+# season / built-plan state, so synthesize just enough for a faithful shot.
+# ---------------------------------------------------------------------------
+
+def _override_update(app, scenario, tmp) -> None:
+    app.state["update_info"] = {
+        "tag": "v1.0.2",
+        "name": "v1.0.2",
+        "url": "https://github.com/AnyCredit5518/riplex/releases/tag/v1.0.2",
+        "releases": [
+            {
+                "tag": "v1.0.2",
+                "body": (
+                    "### Fixed\n\n"
+                    "- **Episodes no longer land in `Other/`.** Organize now honors the "
+                    "rip-time `SxxEyy` classification recorded in each disc's manifest, so "
+                    "an episode whose title has a parenthetical (*Shawn and Gus in Drag "
+                    "(Racing)*) or that dvdcompare files under extras (*Dual Spires*) "
+                    "routes to its season folder.\n\n"
+                    "### Added\n\n"
+                    "- **Durations on the organize preview** highlight a mismatched match "
+                    "before you execute."
+                ),
+            },
+            {
+                "tag": "v1.0.1",
+                "body": (
+                    "### Fixed\n\n"
+                    "- **Out-of-order episode matching.** Episodes are sorted by their "
+                    "canonical `SxxEyy` number before positional alignment, so a disc "
+                    "whose titles are in broadcast order maps correctly."
+                ),
+            },
+        ],
+    }
+
+
+def _override_season_select(app, scenario, tmp) -> None:
+    """Give the show a full 8-season run so the season picker renders."""
+    from riplex.metadata.provider import EpisodeMetadata, SeasonMetadata, ShowDetail
+
+    seasons = []
+    for sn in range(1, 9):
+        eps = [
+            EpisodeMetadata(season_number=sn, episode_number=en,
+                            title=f"Episode {en}", runtime_seconds=2580)
+            for en in range(1, 17)
+        ]
+        seasons.append(SeasonMetadata(season_number=sn, episodes=eps, name=f"Season {sn}"))
+    app.state["show_detail"] = ShowDetail(
+        source_id="tv:1447", title="Psych", year=2006, seasons=seasons,
+    )
+    # Force the picker to render (not auto-skip).
+    app.state["season_number"] = None
+
+
+def _override_organize_preview(app, scenario, tmp) -> None:
+    """Seed a fully-built Psych S5 organize plan (episodes + an extra)."""
+    from riplex.organizer import OrganizePlan, FileMove
+    from riplex.models import PlannedShow, PlannedSeason, PlannedEpisode
+
+    titles = {
+        1: ("Romeo and Juliet and Juliet", 2894, 2898),
+        2: ("Feet Don't Kill Me Now", 2577, 2580),
+        3: ("Not Even Close... Encounters", 2577, 2580),
+        4: ("Chivalry Is Not Dead... But Someone Is", 2581, 2585),
+        5: ("Shawn and Gus in Drag (Racket)", 2585, 2590),
+        6: ("Viagra Falls", 2499, 2520),
+        7: ("Ferry Tale", 2530, 2540),
+        8: ("Shawn 2.0", 2578, 2585),
+    }
+    base = Path("E:/Media/TV Shows/Psych (2006)")
+    moves = []
+    for ep, (name, file_dur, tgt) in titles.items():
+        disc = 1 if ep <= 4 else 2
+        idx = (ep - 1) % 4
+        delta = abs(file_dur - tgt)
+        conf = "high" if delta <= 30 else ("medium" if delta <= 120 else "low")
+        moves.append(FileMove(
+            source=str(base.parent / f"_MakeMKV/Psych/Disc {disc}/C{idx}_t0{idx}.mkv"),
+            destination=str(base / "Season 05" / f"Psych (2006) - s05e{ep:02d} - {name}.mkv"),
+            label=f"Disc {disc}: {name}",
+            confidence=conf,
+            delta_seconds=delta,
+            file_duration_seconds=file_dur,
+            target_runtime_seconds=tgt,
+        ))
+    plan = OrganizePlan(
+        moves=moves,
+        splits=[],
+        unmatched=[],
+        missing=["Disc 1: No Comment: Behind the Scenes (featurette)"],
+    )
+    planned = PlannedShow(
+        canonical_title="Psych", year=2006,
+        seasons=[PlannedSeason(
+            season_number=5,
+            episodes=[PlannedEpisode(season_number=5, episode_number=ep,
+                                     title=name, runtime="43m")
+                      for ep, (name, _, _) in titles.items()],
+        )],
+    )
+    app.state["_organize_plan"] = (plan, planned)
+
+
+_OVERRIDES = {
+    "update": _override_update,
+    "season_select": _override_season_select,
+    "organize_preview": _override_organize_preview,
+}
+
+# Which fixture scenario backs each screen (default: a Psych season).
+_SCENARIO = {
+    "selection": "psych-season-01",
+    "season_select": "psych-season-01",
+    "disc_overview": "psych-season-01",
+    "organize_preview": "psych-season-01",
+    "update": "psych-season-01",
+    "metadata": "psych-season-01",
+    "release": "psych-season-01",
+}
+
+SHOTS = ["selection", "season_select", "disc_overview", "organize_preview", "update"]
+
+
+# ---------------------------------------------------------------------------
+# Server (internal --_serve mode)
+# ---------------------------------------------------------------------------
+
+def _build_screen(page, scenario_name: str, screen: str) -> None:
+    from riplex_app.main import RiplexApp
+    from tests.support.fixtures import load_scenario
+    from tests.support.seed import populate_state
+
+    app = RiplexApp(page)
+    scenario = load_scenario(scenario_name)
+    tmp = Path(tempfile.mkdtemp())
+    populate_state(app.state, scenario, tmp)
+    override = _OVERRIDES.get(screen)
+    if override:
+        override(app, scenario, tmp)
+    app.navigate(screen)
+    # Drop the floating bug-report button for a clean marketing shot.
+    page.floating_action_button = None
+    try:
+        page.update()
+    except Exception:
+        pass
+
+
+def _serve(scenario_name: str, screen: str, port: int) -> None:
+    import flet as ft
+    import uvicorn
+
+    def target(page):
+        _build_screen(page, scenario_name, screen)
+
+    asgi_app = ft.run(target, export_asgi_app=True)
+    uvicorn.Server(
+        uvicorn.Config(asgi_app, host="127.0.0.1", port=port, log_level="error")
+    ).run()
+
+
+# ---------------------------------------------------------------------------
+# Orchestrator (default mode)
+# ---------------------------------------------------------------------------
+
+def _free_port() -> int:
+    s = socket.socket()
+    s.bind(("127.0.0.1", 0))
+    port = s.getsockname()[1]
+    s.close()
+    return port
+
+
+def _wait_port(port: int, timeout: float = 40.0) -> bool:
+    end = time.time() + timeout
+    while time.time() < end:
+        try:
+            with socket.create_connection(("127.0.0.1", port), timeout=1):
+                return True
+        except OSError:
+            time.sleep(0.3)
+    return False
+
+
+def _capture(scenario_name: str, screen: str, out_path: Path, settle_ms: int) -> bool:
+    from playwright.sync_api import sync_playwright
+
+    port = _free_port()
+    proc = subprocess.Popen(
+        [sys.executable, __file__, "--_serve",
+         "--scenario", scenario_name, "--screen", screen, "--port", str(port)],
+    )
+    try:
+        if not _wait_port(port):
+            print(f"  ! server for {screen} did not start")
+            return False
+        with sync_playwright() as p:
+            browser = p.chromium.launch()
+            page = browser.new_page(
+                viewport={"width": 900, "height": 650}, device_scale_factor=2,
+            )
+            page.goto(f"http://127.0.0.1:{port}", wait_until="networkidle", timeout=60000)
+            page.wait_for_timeout(settle_ms)
+            out_path.parent.mkdir(parents=True, exist_ok=True)
+            page.screenshot(path=str(out_path))
+            browser.close()
+        print(f"  + {screen} -> {out_path}")
+        return True
+    finally:
+        proc.terminate()
+        try:
+            proc.wait(timeout=10)
+        except subprocess.TimeoutExpired:
+            proc.kill()
+
+
+def main() -> int:
+    parser = argparse.ArgumentParser(description=__doc__)
+    parser.add_argument("--_serve", action="store_true", help=argparse.SUPPRESS)
+    parser.add_argument("--scenario", default=None)
+    parser.add_argument("--screen", default=None)
+    parser.add_argument("--port", type=int, default=0)
+    parser.add_argument("--out", default="screenshots", help="output directory")
+    parser.add_argument("--all", action="store_true", help="capture every mapped screen")
+    parser.add_argument("--settle-ms", type=int, default=4500,
+                        help="ms to wait for Flutter to paint before capturing")
+    parser.add_argument("--list", action="store_true", help="list mapped screens")
+    args = parser.parse_args()
+
+    if args.list:
+        for s in SHOTS:
+            print(f"{s:20} <- {_SCENARIO.get(s, 'psych-season-01')}")
+        return 0
+
+    if getattr(args, "_serve"):
+        _serve(args.scenario, args.screen, args.port)
+        return 0
+
+    out_dir = Path(args.out)
+    if args.all:
+        screens = SHOTS
+    elif args.screen:
+        screens = [args.screen]
+    else:
+        parser.error("specify --screen NAME, --all, or --list")
+
+    ok = True
+    for screen in screens:
+        scenario_name = args.scenario or _SCENARIO.get(screen, "psych-season-01")
+        print(f"capturing {screen} ({scenario_name})...")
+        if not _capture(scenario_name, screen, out_dir / f"{screen}.png", args.settle_ms):
+            ok = False
+    return 0 if ok else 1
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())

{riplex-1.0.2 → riplex-1.0.4}/src/riplex/config.py

@@ -120,6 +120,22 @@ def get_auto_eject() -> bool:
     return bool(cfg.get("auto_eject", True))
 
 
+def get_makemkv_list_timeout() -> int:
+    """Seconds to wait for ``makemkvcon -r info list`` (drive enumeration).
+
+    Defaults to ``120``. Some drives are slow to spin up or enumerate and
+    time out under the old fixed 60s even though the MakeMKV GUI works;
+    raise ``makemkv_list_timeout`` in the config if you still hit
+    ``makemkvcon failed: ... timed out after N seconds``.
+    """
+    cfg = load_config()
+    try:
+        return int(cfg.get("makemkv_list_timeout", 120))
+    except (TypeError, ValueError):
+        return 120
+
+
+
 def _config_write_path() -> Path:
     """Determine the writable config path (user-scoped)."""
     appdata = os.environ.get("APPDATA")