rekordbox-edit 0.13.1.dev2__tar.gz → 0.13.1.dev3__tar.gz

{rekordbox_edit-0.13.1.dev2 → rekordbox_edit-0.13.1.dev3}/CHANGELOG.md

@@ -1,6 +1,16 @@
-## v0.13.1.dev2 (2026-09-08)
-
-
+## v0.13.1.dev3 (2026-09-08)
+
+
+- fix(display): keep the new value visible in a change preview
+- The changed column rendered the old and new values into one cell that was
+no-wrap with ellipsis overflow. For FolderPath the old value alone fills
+the column at any ordinary terminal width, so the new value was truncated
+away and --dry-run showed only what the row already held. A bulk repoint
+was unverifiable from it; the new value first appeared around 600 columns.
+- The column now wraps instead of ellipsising when it carries a preview, and
+the new value renders beneath the struck-through old one.
+- The existing test missed this by asserting against a 400-column console
+with a single short column, which is not a width anyone runs.
 - chore(deps): update dependency click to v8.5.0
 - fix(edit): survive a row whose analysis directory is missing
 - A row can carry an AnalysisDataPath naming a directory that was never

{rekordbox_edit-0.13.1.dev2 → rekordbox_edit-0.13.1.dev3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: rekordbox-edit
-Version: 0.13.1.dev2
+Version: 0.13.1.dev3
 Summary: Tools for managing and modifying a RekordBox library en-masse
 Project-URL: Homepage, https://github.com/jviall/rekordbox-edit
 Project-URL: Repository, https://github.com/jviall/rekordbox-edit

{rekordbox_edit-0.13.1.dev2 → rekordbox_edit-0.13.1.dev3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "rekordbox-edit"
-version = "0.13.1.dev2"
+version = "0.13.1.dev3"
 description = "Tools for managing and modifying a RekordBox library en-masse"
 authors = [{ name = "James Viall", email= "jamesviall@pm.me"}]
 license = "MIT"

{rekordbox_edit-0.13.1.dev2 → rekordbox_edit-0.13.1.dev3}/rekordbox_edit/display.py

@@ -203,8 +203,10 @@ def print_track_info(
     """Print formatted track information.
 
     When ``changed_field`` and ``new_values`` are both provided, the matching
-    column renders each row as a before/after preview: the old value is
-    struck through and the new value is appended.
+    column renders each row as a before/after preview: the old value struck
+    through, the new value beneath it. That column wraps rather than
+    ellipsising, since both values have to stay readable at ordinary terminal
+    widths.
     """
     if (changed_field is None) != (new_values is None):
         raise ValueError("changed_field and new_values must be provided together")
@@ -232,8 +234,16 @@ def print_track_info(
     table.add_column("#", justify="right", min_width=1, no_wrap=True)
     for column in print_columns:
         cfg = _COLUMN_CONFIG.get(column, {"min_width": 5})
+        # The changed column carries both values, so it needs roughly twice the
+        # width of any other. Ellipsised at a real terminal width it shows the
+        # old value and truncates the new one away, which is the half nobody
+        # can check.
+        previewing = column is changed_field and new_values is not None
         table.add_column(
-            PRINT_HEADERS[column], no_wrap=True, overflow="ellipsis", **cfg
+            PRINT_HEADERS[column],
+            no_wrap=not previewing,
+            overflow="fold" if previewing else "ellipsis",
+            **cfg,
         )
 
     for i, track in enumerate(content_list, 1):
@@ -241,7 +251,9 @@ def print_track_info(
         for col in print_columns:
             old = _cell_value(track, col)
             if col is changed_field and new_values is not None:
-                cells.append(f"[strike]{old}[/strike] [bold]{new_values[i - 1]}[/bold]")
+                cells.append(
+                    f"[strike]{old}[/strike]\n[bold]{new_values[i - 1]}[/bold]"
+                )
             else:
                 cells.append(old)
         table.add_row(str(i), *cells)

{rekordbox_edit-0.13.1.dev2 → rekordbox_edit-0.13.1.dev3}/tests/test_display.py

@@ -86,6 +86,25 @@ def wide_console(monkeypatch):
     )
 
 
+@pytest.fixture
+def terminal_console(monkeypatch):
+    """Swap the module console for one the width of an ordinary terminal.
+
+    `wide_console` exists to keep substring assertions stable, but a preview
+    that only renders at 400 columns is not a preview anyone sees.
+    """
+    monkeypatch.setattr(
+        display,
+        "console",
+        Console(
+            record=True,
+            width=100,
+            theme=display.RBE_THEME,
+            highlighter=display.RbeHighlighter(),
+        ),
+    )
+
+
 class TestPrintTrackInfo:
     """Test print_track_info function."""
 
@@ -165,6 +184,32 @@ class TestPrintTrackInfo:
         assert "Old Name" in captured.out
         assert "New Name" in captured.out
 
+    def test_change_preview_shows_the_new_path_at_terminal_width(
+        self, capsys, terminal_console, make_track
+    ):
+        """A path rewrite is the case this preview exists for, and paths are long.
+
+        Old and new share one no-wrap ellipsis cell, so the old value alone
+        fills the column and the new value is truncated away entirely. A bulk
+        repoint is then unverifiable from --dry-run.
+        """
+        old = (
+            "A:/Music/Any Time, Any Place - Janet Jackson (1994) "
+            "{V25H-38435} [FLAC-CD]/03 Janet Jackson - Any Time, Any Place.flac"
+        )
+        new = old.replace("A:/Music/", "A:/rbe_migration_copy/Music/")
+
+        print_track_info(
+            [make_track(FolderPath=old)],
+            changed_field=PrintableField.FolderPath,
+            new_values=[new],
+        )
+
+        # The cell wraps, so the path is split across lines; join it back up.
+        rendered = "".join(capsys.readouterr().out.split())
+        assert "rbe_migration_copy/Music/" in rendered
+        assert "A:/Music/AnyTime" in rendered
+
     def test_change_preview_requires_both_args(self, make_track):
         """Providing only one of changed_field/new_values raises ValueError."""
         mock_content = make_track()

{rekordbox_edit-0.13.1.dev2 → rekordbox_edit-0.13.1.dev3}/uv.lock

@@ -1274,7 +1274,7 @@ wheels = [
 
 [[package]]
 name = "rekordbox-edit"
-version = "0.13.1.dev2"
+version = "0.13.1.dev3"
 source = { editable = "." }
 dependencies = [
     { name = "click" },