riplex 0.7.3__tar.gz → 0.8.0__tar.gz

{riplex-0.7.3/src/riplex.egg-info → riplex-0.8.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: riplex
-Version: 0.7.3
+Version: 0.8.0
 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

{riplex-0.7.3 → riplex-0.8.0}/docs/changelog.md

@@ -4,6 +4,22 @@ All notable changes to the riplex documentation are recorded here.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
+## v0.8.0 — 2026-06-12
+
+### Added
+
+- **GUI: editable settings after first-run setup.** The welcome screen now exposes an **Edit Settings** button after configuration is complete, so users can update the TMDb API key, media library root, MakeMKV rip output folder, and optional archive folder without re-running setup from the command line.
+
+### Changed
+
+- **GUI: less Plex-specific library wording.** User-facing destination copy now refers to a general media library while retaining **Plex-compatible naming** where the current folder/file convention is being described.
+
+## v0.7.4 — 2026-06-10
+
+### Fixed
+
+- **GUI: empty Disc Overview after continuing without dvdcompare data.** When dvdcompare had no matching release, the orchestrate flow still navigated to the multi-disc overview, which only renders dvdcompare-provided disc rows. The no-dvdcompare fallback now treats the inserted disc as a single-disc rip and jumps directly to title selection, where TMDb runtime heuristics can pick the main feature.
+
 ## v0.7.3 — 2026-06-10
 
 ### Fixed

riplex-0.8.0/issues/manual-rip-and-organize-escape-hatch.md

@@ -0,0 +1,106 @@
+# GUI: manual rip and manual organize escape hatches
+
+## Problem
+
+When riplex can't classify a disc — no dvdcompare match, ambiguous TMDb
+result, multi-feature compilation, region-coded oddity, etc. — the GUI flow
+dead-ends at the title-selection screen. The user has to leave riplex-ui,
+launch the MakeMKV GUI to rip the disc directly, and then either keep those
+files unorganized or manually rename and move them into Plex layout.
+
+We hit this in practice with a BBC Earth Blu-ray containing two unrelated
+features (*Wild Africa* + *Tiny Giants*). The disc isn't on dvdcompare, TMDb
+has only a combined "double feature" entry, and riplex's planner assumes one
+disc → one TMDb item, so the rip flow couldn't proceed at all.
+
+## Why we don't want to model compilations directly
+
+Compilation discs that contain multiple unrelated films are rare enough that
+building first-class support (per-title TMDb matching, per-title disc
+metadata, multi-target organize) isn't justified. A generic escape hatch is
+cheaper and covers other niche scenarios as a side benefit (foreign releases
+without metadata, damaged-disc recovery, special workshop discs, etc.).
+
+## Proposed UX
+
+Two new entry points in the GUI, both intentionally generic:
+
+### 1. "Manual rip" button on the title-selection screen
+
+When riplex can't classify titles confidently (no dvdcompare match **and**
+TMDb selection is ambiguous, or user just wants raw control), surface a
+"Manual rip" button alongside the existing "Rip selected" button.
+
+Clicking it drops the user into a flat MakeMKV-style title list:
+
+- All titles, sorted by index, with duration / size / chapter count
+- Checkbox per title (default: all titles longer than 2 minutes)
+- Output folder picker (defaults to `<output_root>/_MakeMKV/<disc_label>/`)
+- "Rip" button → calls `makemkvcon mkv all` (or per-title) and shows the
+  existing progress UI
+
+No metadata lookup, no organize, no Plex naming. The user lands in the "Done"
+screen with a folder of raw MKVs, with a follow-up button: "Organize this
+folder now → opens the manual organize screen pre-pointed at that path."
+
+### 2. "Organize existing folder" entry on the welcome screen
+
+A second card on welcome (next to "Rip a disc") titled "Organize existing
+files." Picks up an arbitrary folder of MKVs and walks the user through:
+
+1. **Folder picker** → list MKVs with sizes and ffprobe runtimes.
+2. **TMDb search** (reuses existing `metadata` screen) → select the canonical
+   entry.
+3. **File assignment** → for each MKV, drop-down: `Main feature`, `Part 1
+   (-pt1)`, `Part 2 (-pt2)`, `Extra → behindthescenes`, `Extra → featurette`,
+   `Extra → trailer`, `Skip`. Auto-suggest based on duration.
+4. **Plan preview** → show source → destination paths.
+5. **Apply** → move + rename into Plex layout, with the existing dry-run
+   default and `Execute` toggle.
+
+This is essentially the existing `riplex organize` CLI behavior with a GUI in
+front of it. No new business logic — just a thin wrapper that also handles
+the "two parts of one TMDb movie" case via the assignment step.
+
+## Why this works for the BBC Earth case
+
+End-to-end flow stays inside riplex-ui:
+
+1. Insert disc → riplex tries to classify, fails to find dvdcompare match.
+2. User clicks **Manual rip**, picks the two main titles, hits Rip.
+3. Done screen → click **Organize this folder**.
+4. TMDb search → "Wild Africa Tiny Giants" → pick `tmdb-1657586`.
+5. Assign t00 → pt1, t01 → pt2.
+6. Apply → files land in `Movies\Wild Africa - Tiny Giants (2016)\`.
+
+No external tool, no manual `Move-Item`.
+
+## Tradeoffs
+
+- **Pro**: covers any disc riplex can't classify, not just compilations.
+- **Pro**: reuses existing rip + organize plumbing; mostly UI work.
+- **Con**: more GUI surface (two screens, file-assignment widget).
+- **Con**: risk of users defaulting to manual mode and bypassing the smart
+  classification we built. Mitigation: gate the manual-rip button behind a
+  "we couldn't classify this disc" trigger or an "Advanced" reveal, so it's
+  not a peer to the normal "Rip selected" button on every disc.
+
+## Out of scope
+
+- Per-title TMDb matching (full compilation support).
+- Multi-target organize where one folder of MKVs maps to N different TMDb
+  entries. Manual mode handles this by running the organize wizard once per
+  target with file selection.
+- Saving manual-mode decisions for repeat use (cache).
+
+## Files likely involved
+
+- `src/riplex_app/screens/selection.py` — add "Manual rip" button + branch.
+- `src/riplex_app/screens/manual_rip.py` (new) — flat title list + ripper.
+- `src/riplex_app/screens/welcome.py` — add "Organize existing files" card.
+- `src/riplex_app/screens/manual_organize.py` (new) — folder picker → TMDb →
+  assignment → plan preview → apply.
+- `src/riplex/organizer.py` — already has the path-building logic; may need
+  a small helper for the assignment data structure.
+- `src/riplex_cli/commands/organize.py` — sanity-check the CLI still works
+  after any planner refactor.

{riplex-0.7.3 → riplex-0.8.0/src/riplex.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: riplex
-Version: 0.7.3
+Version: 0.8.0
 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

{riplex-0.7.3 → riplex-0.8.0}/src/riplex.egg-info/SOURCES.txt

@@ -25,6 +25,7 @@ docs/getting-started/installation.md
 docs/gui-guide/gui-walkthrough.md
 docs/reference/cli.md
 issues/debug-artifacts-consolidation.md
+issues/manual-rip-and-organize-escape-hatch.md
 issues/orchestrate-dvdcompare-fallback.md
 issues/planned-features.md
 screenshots/0_Rip_Flow_BTTF.gif
@@ -114,6 +115,7 @@ tests/test_matcher.py
 tests/test_normalize.py
 tests/test_organizer.py
 tests/test_planner.py
+tests/test_release_screen.py
 tests/test_rip_guide.py
 tests/test_scanner.py
 tests/test_snapshot.py
@@ -121,6 +123,7 @@ tests/test_splitter.py
 tests/test_tagger.py
 tests/test_ui.py
 tests/test_updater.py
+tests/test_welcome_screen.py
 tests/fixtures/chernobyl_disc1.json
 tests/fixtures/makemkvcon_frozen_planet_ii_d2.txt
 tests/fixtures/makemkvcon_list.txt

{riplex-0.7.3 → riplex-0.8.0}/src/riplex_app/screens/done.py

@@ -87,7 +87,7 @@ class DoneScreen:
 
         no_metadata_hint = ft.Text(
             "Ripped without metadata — use 'Organize Rips' from the home screen "
-            "when TMDb is available to move these files into your Plex library.",
+            "when TMDb is available to move these files into your media library.",
             size=12,
             color=ft.Colors.ORANGE,
         ) if tmdb_match is None and len(successful) > 0 else ft.Container()

{riplex-0.7.3 → riplex-0.8.0}/src/riplex_app/screens/metadata.py

@@ -149,7 +149,7 @@ class MetadataScreen:
             [
                 ft.Text("Metadata Lookup", size=24, weight=ft.FontWeight.BOLD),
                 ft.Text(
-                    "TMDb is needed for canonical titles, years, and Plex folder "
+                    "TMDb is needed for canonical titles, years, and media-library folder "
                     "structure. Without it, you can still rip the disc and organize later.",
                     size=13,
                     color=ft.Colors.GREY_500,

{riplex-0.7.3 → riplex-0.8.0}/src/riplex_app/screens/orchestrate_done.py

@@ -17,7 +17,7 @@ class OrchestrateDoneScreen:
     """Shown after all discs in the orchestrate queue have been ripped.
 
     Displays a summary of all disc rips and offers to organize the
-    ripped files into Plex-compatible folder structure.
+    ripped files into a Plex-compatible folder structure.
     """
 
     def __init__(self, app):
@@ -108,7 +108,7 @@ class OrchestrateDoneScreen:
             on_click=self._open_folder,
         )
         organize_btn = ft.ElevatedButton(
-            "Organize into Plex",
+            "Organize into Library",
             icon=ft.Icons.DRIVE_FILE_MOVE,
             on_click=self._organize,
             style=ft.ButtonStyle(
@@ -136,8 +136,8 @@ class OrchestrateDoneScreen:
                 ft.Column(disc_rows, spacing=6),
                 ft.Container(height=10),
                 ft.Text(
-                    "All selected discs have been ripped. Click 'Organize into Plex' "
-                    "to match files to metadata and move them into your Plex library.",
+                    "All selected discs have been ripped. Click 'Organize into Library' "
+                    "to match files to metadata and move them into your media library.",
                     size=13,
                     color=ft.Colors.GREY_500,
                 ),
@@ -202,7 +202,7 @@ class OrchestrateDoneScreen:
                 self._organize_status.value = f"Error: {exc}"
                 self._organize_status.color = ft.Colors.RED
                 e.control.disabled = False
-                e.control.text = "Organize into Plex"
+                e.control.text = "Organize into Library"
                 self.app.page.update()
 
         threading.Thread(target=_do_organize, daemon=True).start()

{riplex-0.7.3 → riplex-0.8.0}/src/riplex_app/screens/release.py

@@ -31,6 +31,17 @@ class ReleaseScreen:
             return "disc_overview"
         return "selection"
 
+    def _skip_next_screen(self) -> str:
+        """Screen to use when continuing without dvdcompare data."""
+        workflow = self.app.state.get("workflow")
+        if workflow == "orchestrate":
+            self.app.state["_orchestrate_disc_number"] = 1
+            self.app.state["disc_queue"] = [1]
+            self.app.state["current_disc_idx"] = 0
+            self.app.state["all_rip_results"] = {}
+            return "selection"
+        return self._next_screen
+
     def _current_search_title(self) -> str:
         """Title to use for dvdcompare lookup (user override > TMDb > raw title)."""
         override = self.app.state.get("dvdcompare_title_override")
@@ -499,7 +510,7 @@ class ReleaseScreen:
     def _skip(self, e):
         """Proceed without dvdcompare data."""
         self.app.state["dvdcompare_discs"] = []
-        self.app.navigate(self._next_screen)
+        self.app.navigate(self._skip_next_screen())
 
     def _use_release(self, release):
         """Convert a dvdcompare release to PlannedDiscs and navigate."""

{riplex-0.7.3 → riplex-0.8.0}/src/riplex_app/screens/welcome.py

@@ -16,13 +16,30 @@ from riplex.updater import check_for_update, get_current_version
 log = logging.getLogger(__name__)
 
 
+def has_complete_config(config: dict) -> bool:
+    """Return True when required GUI setup values are present."""
+    return bool(
+        config
+        and config.get("tmdb_api_key")
+        and config.get("output_root")
+        and config.get("rip_output")
+    )
+
+
+def should_show_setup(config: dict, state: dict) -> bool:
+    """Return True when the welcome screen should show setup fields."""
+    return not has_complete_config(config) or bool(state.get("_show_setup"))
+
+
 class WelcomeScreen:
     def __init__(self, app):
         self.app = app
 
     def build(self) -> ft.Control:
         config = load_config()
-        has_config = bool(config and config.get("tmdb_api_key"))
+        has_api_key = bool(config and config.get("tmdb_api_key"))
+        has_config = has_complete_config(config)
+        show_setup = should_show_setup(config, self.app.state)
         has_makemkv = find_makemkvcon() is not None
         has_ffprobe = find_ffprobe() is not None
         from riplex.splitter import find_mkvmerge
@@ -31,7 +48,7 @@ class WelcomeScreen:
         # Status indicators
         checks = [
             ("Config file", has_config),
-            ("TMDb API key", has_config),
+            ("TMDb API key", has_api_key),
             ("makemkvcon", has_makemkv),
             ("ffprobe", has_ffprobe),
             ("mkvmerge", has_mkvmerge),
@@ -102,7 +119,7 @@ class WelcomeScreen:
         can_rip = all(ok for _, ok in checks)
         can_organize = has_config and has_ffprobe
 
-        # Setup fields (shown if config missing)
+        # Setup fields (shown if config missing or user chooses to edit)
         self.api_key_field = ft.TextField(
             label="TMDb API key",
             value=config.get("tmdb_api_key", ""),
@@ -111,7 +128,7 @@ class WelcomeScreen:
             expand=True,
         )
         self.output_root_field = ft.TextField(
-            label="Plex library root",
+            label="Media library root",
             value=config.get("output_root", ""),
             expand=True,
         )
@@ -155,8 +172,9 @@ class WelcomeScreen:
                 ft.Container(height=8),
                 _make_browse_row(self.output_root_field),
                 ft.Text(
-                    "Your Plex media library root. Organized files will be placed "
-                    "into Movies/ and TV Shows/ subfolders here.",
+                    "Your media library root. Organized files will be placed "
+                    "into Movies/ and TV Shows/ subfolders here using "
+                    "Plex-compatible naming.",
                     size=11,
                     color=ft.Colors.GREY_600,
                 ),
@@ -177,10 +195,20 @@ class WelcomeScreen:
                     color=ft.Colors.GREY_600,
                 ),
                 ft.Container(height=8),
-                ft.ElevatedButton("Save Config", on_click=self._save_config),
+                ft.Row(
+                    [
+                        ft.ElevatedButton("Save Config", on_click=self._save_config),
+                        ft.TextButton(
+                            "Cancel",
+                            on_click=self._hide_setup,
+                            visible=has_config,
+                        ),
+                    ],
+                    spacing=8,
+                ),
             ],
             spacing=4,
-            visible=not has_config,
+            visible=show_setup,
         )
 
         # Workflow buttons
@@ -190,7 +218,7 @@ class WelcomeScreen:
             on_click=self._start_rip,
             disabled=not can_rip,
             style=ft.ButtonStyle(padding=ft.Padding(left=30, top=15, right=30, bottom=15)),
-            tooltip="Detect a disc, look up metadata, rip selected titles, and organize into Plex.",
+            tooltip="Detect a disc, look up metadata, rip selected titles, and organize into your media library.",
         )
         organize_button = ft.ElevatedButton(
             "Organize Rips",
@@ -228,7 +256,7 @@ class WelcomeScreen:
                 ),
                 self.update_banner,
                 ft.Text(
-                    "Rip physical discs and organize into Plex-compatible libraries.",
+                    "Rip physical discs and organize them into a media library with Plex-compatible naming.",
                     size=14,
                     color=ft.Colors.GREY_400,
                 ),
@@ -244,6 +272,16 @@ class WelcomeScreen:
                 ft.Column(status_rows, spacing=4),
                 install_section,
                 ft.Container(height=10),
+                ft.Row(
+                    [
+                        ft.OutlinedButton(
+                            "Edit Settings",
+                            icon=ft.Icons.SETTINGS,
+                            on_click=self._show_setup,
+                            visible=has_config and not show_setup,
+                        ),
+                    ],
+                ),
                 setup_section,
                 ft.Container(expand=True),
                 ft.Text("What would you like to do?", size=16, weight=ft.FontWeight.BOLD),
@@ -645,6 +683,17 @@ class WelcomeScreen:
         )
 
         # Refresh the screen
+        self.app.state.pop("_show_setup", None)
+        self.app.navigate("welcome")
+
+    def _show_setup(self, e):
+        """Reveal config fields for editing after first-run setup."""
+        self.app.state["_show_setup"] = True
+        self.app.navigate("welcome")
+
+    def _hide_setup(self, e):
+        """Hide config fields without saving changes."""
+        self.app.state.pop("_show_setup", None)
         self.app.navigate("welcome")
 
     def _start_rip(self, e):

riplex-0.8.0/tests/test_release_screen.py

@@ -0,0 +1,40 @@
+from riplex_app.screens.release import ReleaseScreen
+
+
+class _App:
+    def __init__(self, state):
+        self.state = state
+        self.navigated_to = None
+
+    def navigate(self, screen):
+        self.navigated_to = screen
+
+
+class TestReleaseSkipRouting:
+    def test_rip_workflow_continues_to_selection(self):
+        app = _App({"workflow": "rip"})
+        screen = ReleaseScreen(app)
+
+        screen._skip(None)
+
+        assert app.state["dvdcompare_discs"] == []
+        assert app.navigated_to == "selection"
+
+    def test_orchestrate_with_release_uses_disc_overview(self):
+        app = _App({"workflow": "orchestrate"})
+        screen = ReleaseScreen(app)
+
+        assert screen._next_screen == "disc_overview"
+
+    def test_orchestrate_without_dvdcompare_starts_single_disc_selection(self):
+        app = _App({"workflow": "orchestrate"})
+        screen = ReleaseScreen(app)
+
+        screen._skip(None)
+
+        assert app.state["dvdcompare_discs"] == []
+        assert app.state["_orchestrate_disc_number"] == 1
+        assert app.state["disc_queue"] == [1]
+        assert app.state["current_disc_idx"] == 0
+        assert app.state["all_rip_results"] == {}
+        assert app.navigated_to == "selection"

riplex-0.8.0/tests/test_welcome_screen.py

@@ -0,0 +1,42 @@
+from riplex_app.screens.welcome import has_complete_config, should_show_setup
+
+
+class TestHasCompleteConfig:
+    def test_requires_api_key_output_root_and_rip_output(self):
+        assert has_complete_config({}) is False
+        assert has_complete_config({"tmdb_api_key": "key"}) is False
+        assert has_complete_config({"tmdb_api_key": "key", "output_root": "E:/Media"}) is False
+        assert has_complete_config({
+            "tmdb_api_key": "key",
+            "output_root": "E:/Media",
+            "rip_output": "E:/Media/_MakeMKV",
+        }) is True
+
+    def test_archive_root_is_optional(self):
+        assert has_complete_config({
+            "tmdb_api_key": "key",
+            "output_root": "E:/Media",
+            "rip_output": "E:/Media/_MakeMKV",
+            "archive_root": "",
+        }) is True
+
+
+class TestShouldShowSetup:
+    def test_shows_setup_for_incomplete_config(self):
+        assert should_show_setup({"tmdb_api_key": "key"}, {}) is True
+
+    def test_hides_setup_for_complete_config(self):
+        config = {
+            "tmdb_api_key": "key",
+            "output_root": "E:/Media",
+            "rip_output": "E:/Media/_MakeMKV",
+        }
+        assert should_show_setup(config, {}) is False
+
+    def test_can_force_show_setup_for_editing(self):
+        config = {
+            "tmdb_api_key": "key",
+            "output_root": "E:/Media",
+            "rip_output": "E:/Media/_MakeMKV",
+        }
+        assert should_show_setup(config, {"_show_setup": True}) is True