rekordbox-edit 0.4.0.dev20__tar.gz → 0.4.0.dev22__tar.gz

{rekordbox_edit-0.4.0.dev20 → rekordbox_edit-0.4.0.dev22}/AGENTS.md

@@ -60,11 +60,23 @@ Full directive text, BAD/GOOD example pairs, and rationale per rule: see `.agent
 ## Code style
 
 - Only comment non-obvious code — hidden constraints, subtle invariants, workarounds. If a future reader would not be confused by the comment's absence, leave it out.
-- Do not record conversation history, decisions, or task context in comments. That belongs in the commit message or PR description.
+- Do not record conversation history, decisions, or task context in comments.
 - Keep function and class header comments terse. No more than a paragraph.
 - Name identifiers so the code explains itself; rename rather than annotate.
+- Follow the "Rule of Three" as a guidepost for factoring out shared logic.
+- Break out long functions in to multiple smaller testable units once they exceed a few hundred lines of code. Command entry points can be a bit longer than other functions.
 - Emphasis on consistency of patterns across the codebase, except where clarity for other devs would be improved by diverging from a pattern.
 
+### Logging and Exceptions
+
+Every module should get and use its own `logger` for all logging purposes and follow semantic best practices of log levels:
+
+- `debug` logs should be used to record relevant application state at significant logic branches and points of execution. These are not printed to the user unless the `--print debug` option is provided.
+- `info` logs are the primary means of showing output to the user, and should be used sparingly/as necessary.
+- `warning` logs communicate potential issues with application state or dangerous/unexpected circumstances that do not prevent execution from continuing.
+- `error` logs communicate that invalid application or system state was encountered and needs attention, or invalid input was given, and execution cannot continue.
+- `critical` logs communicate that an unexpected/unstable state was reached and execution was interrupted.
+
 ## Testing
 
 - Cover user-visible behavior and code paths that pose risk to the database or user file system. Skip tests that exist only to raise coverage.
@@ -76,7 +88,9 @@ Full directive text, BAD/GOOD example pairs, and rationale per rule: see `.agent
 
 - Implement the smallest functional slice first, then layer features in follow-up commits. Each commit should be "green" and independently deployable.
 - Follow Conventional Commits (see `CONTRIBUTING.md` and the schema in `pyproject.toml`). No `Co-Authored-By` trailer.
-- Claude commits. The user pushes, opens PRs, and rebases after merges.
+- Commit descriptions should be terse and use active tenses (e.g. "add feature")
+- Commit bodies should describe changes, but should not include agent conversation context, decisions, or plan notes.
+- The agent may commit. But the user always handles pushes, PR creation, and rebases after merges.
 - Do not commit spec, brainstorming, or design documents (e.g. anything under `docs/superpowers/specs/`).
 
 ## Tooling

{rekordbox_edit-0.4.0.dev20 → rekordbox_edit-0.4.0.dev22}/CHANGELOG.md

@@ -1,6 +1,23 @@
-## v0.4.0.dev20 (2026-05-16)
-
-
+## v0.4.0.dev22 (2026-05-17)
+
+
+- feat(edit): add --match for literal find/replace within field value
+- Introduces a _compute_new_value helper and --match option so that
+rbe edit can replace a substring of the current field value instead of
+performing a wholesale replacement; None current values and non-matching
+patterns are treated as no-ops.
+- chore: add max-complexity lint rule
+- docs: update AGENTS.md
+- test: add coverage for --interactive + --yes skipping all confirms
+- feat: adds edit command
+- Adds an `edit` subcommand to the CLI with:
+- Title field support via a required FIELD argument and --replace option
+- Single-track safety guard that aborts when >1 track would be modified
+- --dry-run mode that previews changes without writing to the database
+- --yes flag to skip confirmation, --interactive to confirm per-track
+- Scripting mode (--print=ids) requiring --yes or --dry-run
+- Piped stdin rejection without --yes or --dry-run
+- All global filter flags forwarded to get_filtered_content
 - docs: add AGENTS.md with project conventions and CLAUDE.md symlink
 - chore: add renovate config
 - feat: Add --path and --exact-path search filters

{rekordbox_edit-0.4.0.dev20 → rekordbox_edit-0.4.0.dev22}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rekordbox-edit
-Version: 0.4.0.dev20
+Version: 0.4.0.dev22
 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.4.0.dev20 → rekordbox_edit-0.4.0.dev22}/pyproject.toml

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

{rekordbox_edit-0.4.0.dev20 → rekordbox_edit-0.4.0.dev22}/rekordbox_edit/cli.py

@@ -7,6 +7,7 @@ import sys
 import click
 
 from rekordbox_edit.commands.convert import convert_command
+from rekordbox_edit.commands.edit import edit_command
 from rekordbox_edit.commands.search import search_command
 from rekordbox_edit.logger import get_debug_file_path, setup_logging
 
@@ -23,9 +24,8 @@ def cli():
 
 
 cli.add_command(search_command)
-cli.add_command(
-    convert_command,
-)
+cli.add_command(convert_command)
+cli.add_command(edit_command)
 
 
 def main():

rekordbox_edit-0.4.0.dev22/rekordbox_edit/commands/edit.py

@@ -0,0 +1,195 @@
+"""Edit command for rekordbox-edit."""
+
+import logging
+import sys
+from typing import List
+
+import click
+from pyrekordbox import Rekordbox6Database
+
+from rekordbox_edit._click import (
+    PrintChoice,
+    add_click_options,
+    global_click_filters,
+    print_option,
+    track_ids_argument,
+)
+from rekordbox_edit.logger import get_debug_file_path, set_level
+from rekordbox_edit.query import get_filtered_content
+from rekordbox_edit.utils import UserQuit, confirm, print_track_info
+
+logger = logging.getLogger(__name__)
+
+# Maps CLI field names to DjmdContent column attribute names.
+FIELD_COLUMNS = {
+    "Title": "Title",
+}
+
+
+def _compute_new_value(
+    current: str | int | None,
+    match_pattern: str | None,
+    replace_value: str | int,
+) -> str | int | None:
+    """Derive the new field value."""
+    if current is None:
+        return None
+    if match_pattern is not None:
+        return str(current).replace(match_pattern, str(replace_value))
+    return replace_value
+
+
+@click.command(
+    epilog=f"Debug logs for each run can be found at:\n{get_debug_file_path().parent}"
+)
+@add_click_options([*global_click_filters, print_option])
+@click.option(
+    "--interactive",
+    "-i",
+    is_flag=True,
+    help="Confirm each track individually before editing",
+)
+@click.option(
+    "--yes",
+    "-y",
+    is_flag=True,
+    help="Skip confirmation prompt",
+)
+@click.option(
+    "--dry-run",
+    is_flag=True,
+    help="Show what would change without writing to the database",
+)
+@click.option(
+    "--replace",
+    "replace_value",
+    required=True,
+    help="The new value to write to the field",
+)
+@click.option(
+    "--match",
+    "match_pattern",
+    default=None,
+    metavar="PATTERN",
+    help="Find this literal string within the field value and replace only that portion",
+)
+@track_ids_argument
+@click.argument(
+    "field",
+    type=click.Choice(list(FIELD_COLUMNS.keys()), case_sensitive=False),
+)
+def edit_command(
+    field: str,
+    replace_value: str,
+    match_pattern: str | None,
+    dry_run: bool,
+    yes: bool,
+    interactive: bool,
+    track_ids: List[str] | None,
+    track_id: List[str] | None,
+    playlist: List[str] | None,
+    exact_playlist: List[str] | None,
+    album: List[str] | None,
+    exact_album: List[str] | None,
+    artist: List[str] | None,
+    exact_artist: List[str] | None,
+    title: List[str] | None,
+    exact_title: List[str] | None,
+    path: List[str] | None,
+    exact_path: List[str] | None,
+    format: List[str] | None,
+    match_all: bool,
+    print_opt: PrintChoice | None,
+):
+    """Edit a metadata field on tracks in the RekordBox database."""
+
+    set_level(print_opt)
+
+    piped_stdin = False
+    if not sys.stdin.isatty():
+        stdin_data = sys.stdin.read().strip()
+        if stdin_data:
+            piped_stdin = True
+            track_ids = list(track_ids or []) + stdin_data.split()
+
+    scripting_mode = print_opt in (PrintChoice.IDS, PrintChoice.SILENT)
+    if scripting_mode and not (dry_run or yes):
+        raise click.UsageError(
+            "--print=ids or --print=silent requires --dry-run or --yes to skip confirmation"
+        )
+
+    if piped_stdin and not (dry_run or yes):
+        raise click.UsageError("Piping track IDs into edit requires --dry-run or --yes")
+
+    db = Rekordbox6Database()
+    if not db.session:
+        raise RuntimeError("Failed to connect to Rekordbox Database: No Session.")
+
+    result = get_filtered_content(
+        db,
+        track_id_args=track_ids,
+        track_ids=track_id,
+        playlists=playlist,
+        exact_playlists=exact_playlist,
+        artists=artist,
+        exact_artists=exact_artist,
+        albums=album,
+        exact_albums=exact_album,
+        titles=title,
+        exact_titles=exact_title,
+        paths=path,
+        exact_paths=exact_path,
+        formats=format,
+        match_all=match_all,
+    )
+    tracks = result.scalars().all()
+
+    col_name = FIELD_COLUMNS[field]
+    edits = []
+    for track in tracks:
+        current = getattr(track, col_name)
+        new_value = _compute_new_value(current, match_pattern, replace_value)
+        if new_value is None or new_value == current:
+            continue
+        edits.append((track, new_value))
+
+    if not edits:
+        logger.info("No changes to make.")
+        return
+
+    if len(edits) > 1:
+        raise click.UsageError(
+            f"Found {len(edits)} tracks that would be edited. "
+            "Refine your filters, or use --dry-run to inspect."
+        )
+
+    print_track_info([t for t, _ in edits])
+
+    if dry_run:
+        if print_opt is PrintChoice.IDS:
+            print(" ".join(str(t.ID) for t, _ in edits))
+        return
+
+    if not yes and not interactive:
+        try:
+            if not confirm(f"Apply {len(edits)} edit(s)?", default=True):
+                logger.info("Cancelled.")
+                return
+        except UserQuit:
+            return
+
+    for track, new_value in edits:
+        if interactive and not yes:
+            try:
+                if not confirm(f"  Edit {track.ID}?", default=True):
+                    continue
+            except UserQuit:
+                logger.info("Cancelled.")
+                return
+        setattr(track, col_name, new_value)
+
+    db.session.commit()
+    logger.info(f"Applied {len(edits)} edit(s).")
+
+    if print_opt is PrintChoice.IDS:
+        print(" ".join(str(t.ID) for t, _ in edits))

{rekordbox_edit-0.4.0.dev20 → rekordbox_edit-0.4.0.dev22}/ruff.toml

@@ -44,9 +44,10 @@ select = [
   "E7",
   "E9",
   "F",
-  "W" ,
-  # "C901"
+  "W",
+  "C90",
 ]
+
 ignore = []
 
 # Allow fix for all enabled rules (when `--fix`) is provided.
@@ -56,6 +57,9 @@ unfixable = []
 # Allow unused variables when underscore-prefixed.
 dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
 
+[lint.mccabe]
+max-complexity = 50
+
 [format]
 # Like Black, use double quotes for strings.
 quote-style = "double"

rekordbox_edit-0.4.0.dev22/tests/commands/test_edit.py

@@ -0,0 +1,429 @@
+"""Unit tests for edit command."""
+
+from unittest.mock import Mock, patch
+
+import pytest
+
+from rekordbox_edit.commands.edit import edit_command
+
+
+@pytest.fixture(autouse=True)
+def mock_logger():
+    with patch("rekordbox_edit.commands.edit.logger") as mock_log:
+        yield mock_log
+
+
+def _make_db_and_result(tracks):
+    """Return (mock_db_class, mock_get_filtered_content) with given tracks."""
+    mock_db = Mock()
+    mock_db.session = Mock()
+    mock_result = Mock()
+    mock_result.scalars.return_value.all.return_value = tracks
+    return mock_db, mock_result
+
+
+class TestEditCommandPhase1:
+    """Phase 1: Title field, wholesale replace, single-track guard, confirm flow."""
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_sets_title_and_commits(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--replace sets the field on the matched track and commits the session."""
+        track = make_djmd_content_item(Title="Old Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+        mock_confirm.return_value = True
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(edit_command, ["Title", "--replace", "New Title"])
+
+        assert result.exit_code == 0
+        assert track.Title == "New Title"
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_noop_tracks_are_skipped(
+        self, mock_db_class, mock_gfc, make_djmd_content_item
+    ):
+        """Tracks whose current value already equals --replace are not committed."""
+        track = make_djmd_content_item(Title="Same Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(edit_command, ["Title", "--replace", "Same Title", "--yes"])
+
+        assert result.exit_code == 0
+        mock_db.session.commit.assert_not_called()
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_single_track_guard_aborts_on_multiple_matches(
+        self, mock_db_class, mock_gfc, make_djmd_content_item
+    ):
+        """When >1 track would change and --multi is absent, exit with non-zero code."""
+        tracks = [
+            make_djmd_content_item(Title="Track A"),
+            make_djmd_content_item(Title="Track B"),
+        ]
+        mock_db, mock_result = _make_db_and_result(tracks)
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(edit_command, ["Title", "--replace", "New", "--yes"])
+
+        assert result.exit_code != 0
+        mock_db.session.commit.assert_not_called()
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_dry_run_does_not_commit(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--dry-run shows preview but does not write to the database."""
+        track = make_djmd_content_item(Title="Old Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(edit_command, ["Title", "--replace", "New Title", "--dry-run"])
+
+        assert result.exit_code == 0
+        mock_db.session.commit.assert_not_called()
+        mock_confirm.assert_not_called()
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_yes_skips_confirm(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--yes applies changes without prompting."""
+        track = make_djmd_content_item(Title="Old Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(edit_command, ["Title", "--replace", "New Title", "--yes"])
+
+        assert result.exit_code == 0
+        mock_confirm.assert_not_called()
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_interactive_confirms_each_track(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--interactive prompts per track (skipping the batch prompt)."""
+        track = make_djmd_content_item(Title="Old Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+        mock_confirm.return_value = True
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command, ["Title", "--replace", "New Title", "--interactive"]
+        )
+
+        assert result.exit_code == 0
+        mock_confirm.assert_called_once()
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_piped_stdin_requires_yes_or_dry_run(self, mock_db_class, mock_gfc):
+        """Piping track IDs into edit without --yes or --dry-run is rejected."""
+        mock_db = Mock()
+        mock_db.session = Mock()
+        mock_db_class.return_value = mock_db
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command, ["Title", "--replace", "New Title"], input="12345"
+        )
+
+        assert result.exit_code != 0
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_scripting_mode_requires_yes_or_dry_run(self, mock_db_class, mock_gfc):
+        """--print=ids without --yes or --dry-run is rejected."""
+        mock_db = Mock()
+        mock_db.session = Mock()
+        mock_db_class.return_value = mock_db
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command, ["Title", "--replace", "New Title", "--print", "ids"]
+        )
+
+        assert result.exit_code != 0
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_filters_forwarded_to_get_filtered_content(
+        self, mock_db_class, mock_gfc, mock_confirm
+    ):
+        """All global filter flags are forwarded to get_filtered_content."""
+        mock_db = Mock()
+        mock_db.session = Mock()
+        mock_db_class.return_value = mock_db
+        mock_result = Mock()
+        mock_result.scalars.return_value.all.return_value = []
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        CliRunner().invoke(
+            edit_command,
+            [
+                "Title",
+                "--replace", "New Title",
+                "--artist", "Bicep",
+                "--format", "flac",
+                "--match-all",
+                "--yes",
+            ],
+        )
+
+        kwargs = mock_gfc.call_args.kwargs
+        assert kwargs["artists"] == ("Bicep",)
+        assert kwargs["formats"] == ("flac",)
+        assert kwargs["match_all"] is True
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_interactive_and_yes_skips_all_confirms(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--interactive + --yes skips both batch and per-track confirms."""
+        track = make_djmd_content_item(Title="Old Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--replace", "New Title", "--interactive", "--yes"],
+        )
+
+        assert result.exit_code == 0
+        mock_confirm.assert_not_called()
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_print_ids_outputs_edited_track_ids(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--print=ids outputs the IDs of edited tracks after committing."""
+        track = make_djmd_content_item(ID="99999", Title="Old Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--replace", "New Title", "--print", "ids", "--yes"],
+        )
+
+        assert result.exit_code == 0
+        assert "99999" in result.output
+
+
+class TestEditCommandPhase3:
+    """Phase 3: --match flag for literal find/replace within field value."""
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_match_replaces_substring(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--match substitutes the pattern within the current value."""
+        track = make_djmd_content_item(Title="Dark Matter (feat. Jane)")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+        mock_confirm.return_value = True
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--match", "feat.", "--replace", "ft."],
+        )
+
+        assert result.exit_code == 0
+        assert track.Title == "Dark Matter (ft. Jane)"
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_match_no_match_is_noop(
+        self, mock_db_class, mock_gfc, make_djmd_content_item
+    ):
+        """When --match pattern is not found in current value, track is a no-op."""
+        track = make_djmd_content_item(Title="Dark Matter")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--match", "feat.", "--replace", "ft.", "--yes"],
+        )
+
+        assert result.exit_code == 0
+        mock_db.session.commit.assert_not_called()
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_match_with_none_current_value_is_noop(
+        self, mock_db_class, mock_gfc, make_djmd_content_item
+    ):
+        """--match on a track where the field is None is silently skipped."""
+        track = make_djmd_content_item(Title=None)
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--match", "feat.", "--replace", "ft.", "--yes"],
+        )
+
+        assert result.exit_code == 0
+        mock_db.session.commit.assert_not_called()
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_match_treats_pattern_as_literal(
+        self, mock_db_class, mock_gfc, make_djmd_content_item
+    ):
+        """--match treats the pattern as a literal string, not a regex."""
+        # "1.0" as a regex would also match "1X0"; as a literal it should not
+        track = make_djmd_content_item(Title="Version 1X0")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--match", "1.0", "--replace", "2.0", "--yes"],
+        )
+
+        assert result.exit_code == 0
+        mock_db.session.commit.assert_not_called()
+
+
+class TestEditCommandUnicode:
+    """Unicode and multibyte character handling in edit command fields."""
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_wholesale_replace_with_unicode_value(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--replace accepts and writes multibyte unicode titles correctly."""
+        track = make_djmd_content_item(Title="Original Title")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+        mock_confirm.return_value = True
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--replace", "日本語タイトル"],
+        )
+
+        assert result.exit_code == 0
+        assert track.Title == "日本語タイトル"
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_wholesale_replace_of_unicode_current_value(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """A track with a unicode title can be replaced wholesale."""
+        track = make_djmd_content_item(Title="Ü-Bahn Nights (feat. Ångström)")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+        mock_confirm.return_value = True
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--replace", "U-Bahn Nights (feat. Angstrom)"],
+        )
+
+        assert result.exit_code == 0
+        assert track.Title == "U-Bahn Nights (feat. Angstrom)"
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.confirm")
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_match_replace_within_unicode_title(
+        self, mock_db_class, mock_gfc, mock_confirm, make_djmd_content_item
+    ):
+        """--match correctly finds and replaces a substring within a unicode title."""
+        track = make_djmd_content_item(Title="夜 feat. 山田太郎")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+        mock_confirm.return_value = True
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--match", "feat.", "--replace", "ft."],
+        )
+
+        assert result.exit_code == 0
+        assert track.Title == "夜 ft. 山田太郎"
+        mock_db.session.commit.assert_called_once()
+
+    @patch("rekordbox_edit.commands.edit.get_filtered_content")
+    @patch("rekordbox_edit.commands.edit.Rekordbox6Database")
+    def test_noop_when_unicode_values_are_equal(
+        self, mock_db_class, mock_gfc, make_djmd_content_item
+    ):
+        """No edit is made when the current unicode value already equals --replace."""
+        track = make_djmd_content_item(Title="Ø (Disambiguation)")
+        mock_db, mock_result = _make_db_and_result([track])
+        mock_db_class.return_value = mock_db
+        mock_gfc.return_value = mock_result
+
+        from click.testing import CliRunner
+        result = CliRunner().invoke(
+            edit_command,
+            ["Title", "--replace", "Ø (Disambiguation)", "--yes"],
+        )
+
+        assert result.exit_code == 0
+        mock_db.session.commit.assert_not_called()

{rekordbox_edit-0.4.0.dev20 → rekordbox_edit-0.4.0.dev22}/uv.lock

@@ -985,7 +985,7 @@ wheels = [
 
 [[package]]
 name = "rekordbox-edit"
-version = "0.4.0.dev20"
+version = "0.4.0.dev22"
 source = { editable = "." }
 dependencies = [
     { name = "click" },