coder-music-cli 0.9.0__tar.gz → 0.9.1__tar.gz

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: coder-music-cli
-Version: 0.9.0
+Version: 0.9.1
 Summary: A command-line music application for coders with daemon support, radio streaming, and AI-generated music
 Author-email: Luong Nguyen <luongnv89@gmail.com>
 Maintainer-email: Luong Nguyen <luongnv89@gmail.com>
@@ -203,7 +203,7 @@ The base install is tiny. The `[ai]` extra downloads ~5GB (PyTorch + HuggingFace
 music-cli is not a replacement for your music library. It's a lightweight, terminal-native player for background music while coding. No browser tabs, no electron apps, no accounts.
 
 **Is it actively maintained?**
-Yes. The latest release is v0.9.0. Check the [changelog](CHANGELOG.md) for recent updates.
+Yes. The latest release is v0.9.1. Check the [changelog](CHANGELOG.md) for recent updates.
 
 **Can I add my own radio stations?**
 Absolutely. Run `mc radio add` or edit `~/.config/music-cli/radios.txt` directly. Format: `Station Name|stream-url`.
@@ -552,7 +552,7 @@ Context: morning / weekday
 
 "Music gives a soul to the universe, wings to the mind, flight to the imagination." - Plato
 
-Version: 0.9.0
+Version: 0.9.1
 GitHub: https://github.com/luongnv89/music-cli
 ```
 

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/README.md

@@ -150,7 +150,7 @@ The base install is tiny. The `[ai]` extra downloads ~5GB (PyTorch + HuggingFace
 music-cli is not a replacement for your music library. It's a lightweight, terminal-native player for background music while coding. No browser tabs, no electron apps, no accounts.
 
 **Is it actively maintained?**
-Yes. The latest release is v0.9.0. Check the [changelog](CHANGELOG.md) for recent updates.
+Yes. The latest release is v0.9.1. Check the [changelog](CHANGELOG.md) for recent updates.
 
 **Can I add my own radio stations?**
 Absolutely. Run `mc radio add` or edit `~/.config/music-cli/radios.txt` directly. Format: `Station Name|stream-url`.
@@ -499,7 +499,7 @@ Context: morning / weekday
 
 "Music gives a soul to the universe, wings to the mind, flight to the imagination." - Plato
 
-Version: 0.9.0
+Version: 0.9.1
 GitHub: https://github.com/luongnv89/music-cli
 ```
 

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/coder_music_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: coder-music-cli
-Version: 0.9.0
+Version: 0.9.1
 Summary: A command-line music application for coders with daemon support, radio streaming, and AI-generated music
 Author-email: Luong Nguyen <luongnv89@gmail.com>
 Maintainer-email: Luong Nguyen <luongnv89@gmail.com>
@@ -203,7 +203,7 @@ The base install is tiny. The `[ai]` extra downloads ~5GB (PyTorch + HuggingFace
 music-cli is not a replacement for your music library. It's a lightweight, terminal-native player for background music while coding. No browser tabs, no electron apps, no accounts.
 
 **Is it actively maintained?**
-Yes. The latest release is v0.9.0. Check the [changelog](CHANGELOG.md) for recent updates.
+Yes. The latest release is v0.9.1. Check the [changelog](CHANGELOG.md) for recent updates.
 
 **Can I add my own radio stations?**
 Absolutely. Run `mc radio add` or edit `~/.config/music-cli/radios.txt` directly. Format: `Station Name|stream-url`.
@@ -552,7 +552,7 @@ Context: morning / weekday
 
 "Music gives a soul to the universe, wings to the mind, flight to the imagination." - Plato
 
-Version: 0.9.0
+Version: 0.9.1
 GitHub: https://github.com/luongnv89/music-cli
 ```
 

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/coder_music_cli.egg-info/SOURCES.txt

@@ -47,4 +47,5 @@ tests/test_cli.py
 tests/test_config.py
 tests/test_context.py
 tests/test_e2e.py
-tests/test_history.py
+tests/test_history.py
+tests/test_local_source.py

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/music_cli/__init__.py

@@ -1,4 +1,4 @@
 """music-cli: A command-line music application for coders."""
 
-__version__ = "0.9.0"
+__version__ = "0.9.1"
 __github_url__ = "https://github.com/luongnv89/music-cli"

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/music_cli/cli.py

@@ -289,6 +289,25 @@ def _detect_play_mode(source_arg):
     return "radio", source_arg
 
 
+def _resolve_local_path(source: str) -> str:
+    """Resolve a local playback source against the CLI's cwd.
+
+    The daemon is a long-running background process started once with its
+    own cwd (see start_daemon_background) — it is generally not the cwd of
+    the terminal a later `play` command runs in. A still-relative path
+    handed to the daemon therefore gets checked against the *daemon's* cwd,
+    not the user's, and silently misses even when the file is right there.
+    Resolve it to an absolute path here, while we still have the correct
+    cwd, so it survives the trip over IPC unchanged. Leave anything that
+    doesn't exist relative to cwd as-is so the daemon's fallback to the
+    configured music directory keeps working.
+    """
+    path = Path(source)
+    if not path.is_absolute() and path.exists():
+        return str(path.resolve())
+    return source
+
+
 @main.command()
 @click.argument("source", required=False, default=None)
 @click.option(
@@ -366,6 +385,11 @@ def play(source, mode, source_flag, mood, auto, duration, index):
     # If mode was explicitly given, keep the old behaviour
     # (effective_source from flag/positional is used as-is)
 
+    # Issue #18: a relative local path must be resolved before it crosses
+    # the IPC boundary to the daemon (see _resolve_local_path).
+    if mode == "local" and effective_source:
+        effective_source = _resolve_local_path(effective_source)
+
     client = ensure_daemon()
 
     # Show animation for AI generation

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/music_cli/sources/local.py

@@ -23,8 +23,14 @@ class LocalSource:
         file_path = Path(path)
 
         if not file_path.is_absolute():
-            # Try relative to music dir
-            file_path = self.music_dir / file_path
+            # Prefer a match relative to the current working directory;
+            # resolve it now so the result doesn't depend on cwd staying
+            # the same for the rest of the call chain (e.g. across the
+            # daemon's IPC boundary). Fall back to the configured music dir.
+            if file_path.exists():
+                file_path = file_path.resolve()
+            else:
+                file_path = self.music_dir / file_path
 
         if not file_path.exists():
             return None

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "coder-music-cli"
-version = "0.9.0"
+version = "0.9.1"
 description = "A command-line music application for coders with daemon support, radio streaming, and AI-generated music"
 readme = "README.md"
 requires-python = ">=3.10"

{coder_music_cli-0.9.0 → coder_music_cli-0.9.1}/tests/test_cli.py

@@ -1,12 +1,13 @@
 """Tests for CLI v2 Phase 1, Phase 2 & Phase 3."""
 
 import os
+from pathlib import Path
 from unittest.mock import MagicMock, patch
 
 import pytest
 from click.testing import CliRunner
 
-from music_cli.cli import _detect_play_mode, icon, main
+from music_cli.cli import _detect_play_mode, _resolve_local_path, icon, main
 
 
 @pytest.fixture
@@ -381,6 +382,90 @@ class TestSmartPlayDetection:
         assert call_kwargs[1]["mode"] == "context" or call_kwargs.kwargs["mode"] == "context"
 
 
+class TestResolveLocalPath:
+    """Unit tests for _resolve_local_path (issue #18)."""
+
+    def test_relative_existing_path_resolved_to_absolute(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        f = Path("song.mp3")
+        f.touch()
+
+        resolved = _resolve_local_path("song.mp3")
+
+        assert resolved == str((tmp_path / "song.mp3").resolve())
+
+    def test_relative_missing_path_returned_unchanged(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+
+        resolved = _resolve_local_path("missing.mp3")
+
+        assert resolved == "missing.mp3"
+
+    def test_absolute_path_returned_unchanged(self, tmp_path):
+        f = tmp_path / "song.mp3"
+        f.touch()
+
+        resolved = _resolve_local_path(str(f))
+
+        assert resolved == str(f)
+
+
+class TestPlayRelativeLocalPath:
+    """`mc play <relative-file>` must send the daemon an absolute path.
+
+    The daemon is a separate long-running process — a relative path sent
+    as-is would be checked against the *daemon's* cwd, not the terminal's,
+    and would silently fail to be found (issue #18).
+    """
+
+    @patch("music_cli.cli.ensure_daemon")
+    @patch("music_cli.cli.check_ffplay_available", return_value=True)
+    def test_play_relative_file_sends_absolute_source(
+        self, mock_ffplay, mock_daemon, runner, mock_daemon_client, tmp_path, monkeypatch
+    ):
+        monkeypatch.chdir(tmp_path)
+        Path("song.mp3").touch()
+        mock_daemon.return_value = mock_daemon_client
+
+        result = runner.invoke(main, ["play", "song.mp3"])
+
+        assert result.exit_code == 0
+        call_kwargs = mock_daemon_client.play.call_args.kwargs
+        assert call_kwargs["mode"] == "local"
+        assert call_kwargs["source"] == str((tmp_path / "song.mp3").resolve())
+
+    @patch("music_cli.cli.ensure_daemon")
+    @patch("music_cli.cli.check_ffplay_available", return_value=True)
+    def test_play_explicit_mode_local_relative_file_sends_absolute_source(
+        self, mock_ffplay, mock_daemon, runner, mock_daemon_client, tmp_path, monkeypatch
+    ):
+        monkeypatch.chdir(tmp_path)
+        Path("song.mp3").touch()
+        mock_daemon.return_value = mock_daemon_client
+
+        result = runner.invoke(main, ["play", "song.mp3", "--mode", "local"])
+
+        assert result.exit_code == 0
+        call_kwargs = mock_daemon_client.play.call_args.kwargs
+        assert call_kwargs["source"] == str((tmp_path / "song.mp3").resolve())
+
+    @patch("music_cli.cli.ensure_daemon")
+    @patch("music_cli.cli.check_ffplay_available", return_value=True)
+    def test_play_explicit_mode_local_missing_file_kept_relative(
+        self, mock_ffplay, mock_daemon, runner, mock_daemon_client, tmp_path, monkeypatch
+    ):
+        # Not found in cwd -> left unchanged so the daemon still falls back
+        # to the configured music directory (today's behavior, preserved).
+        monkeypatch.chdir(tmp_path)
+        mock_daemon.return_value = mock_daemon_client
+
+        result = runner.invoke(main, ["play", "not-in-cwd.mp3", "--mode", "local"])
+
+        assert result.exit_code == 0
+        call_kwargs = mock_daemon_client.play.call_args.kwargs
+        assert call_kwargs["source"] == "not-in-cwd.mp3"
+
+
 # -------------------------------------------------------------------------
 # 2.2 — Deprecate play -m ai
 # -------------------------------------------------------------------------

coder_music_cli-0.9.1/tests/test_local_source.py

@@ -0,0 +1,88 @@
+"""Tests for LocalSource relative-path resolution (issue #18)."""
+
+from music_cli.sources.local import LocalSource
+
+
+class TestGetTrackRelativePaths:
+    """get_track() should check the current working directory before
+    falling back to the configured music directory."""
+
+    def test_relative_path_found_in_cwd(self, tmp_path, monkeypatch):
+        cwd = tmp_path / "cwd"
+        music_dir = tmp_path / "music"
+        cwd.mkdir()
+        music_dir.mkdir()
+        (cwd / "song.mp3").touch()
+        monkeypatch.chdir(cwd)
+
+        source = LocalSource(music_dir=music_dir)
+        track = source.get_track("song.mp3")
+
+        assert track is not None
+        assert track.title == "song"
+        assert track.source == str((cwd / "song.mp3").resolve())
+
+    def test_relative_path_falls_back_to_music_dir(self, tmp_path, monkeypatch):
+        cwd = tmp_path / "cwd"
+        music_dir = tmp_path / "music"
+        cwd.mkdir()
+        music_dir.mkdir()
+        (music_dir / "song.mp3").touch()
+        monkeypatch.chdir(cwd)
+
+        source = LocalSource(music_dir=music_dir)
+        track = source.get_track("song.mp3")
+
+        assert track is not None
+        assert track.source == str(music_dir / "song.mp3")
+
+    def test_relative_path_not_found_anywhere_returns_none(self, tmp_path, monkeypatch):
+        cwd = tmp_path / "cwd"
+        music_dir = tmp_path / "music"
+        cwd.mkdir()
+        music_dir.mkdir()
+        monkeypatch.chdir(cwd)
+
+        source = LocalSource(music_dir=music_dir)
+        track = source.get_track("missing.mp3")
+
+        assert track is None
+
+    def test_cwd_match_takes_priority_over_music_dir(self, tmp_path, monkeypatch):
+        cwd = tmp_path / "cwd"
+        music_dir = tmp_path / "music"
+        cwd.mkdir()
+        music_dir.mkdir()
+        (cwd / "song.mp3").touch()
+        (music_dir / "song.mp3").touch()
+        monkeypatch.chdir(cwd)
+
+        source = LocalSource(music_dir=music_dir)
+        track = source.get_track("song.mp3")
+
+        assert track.source == str((cwd / "song.mp3").resolve())
+
+    def test_absolute_path_unaffected(self, tmp_path):
+        music_dir = tmp_path / "music"
+        music_dir.mkdir()
+        f = music_dir / "song.mp3"
+        f.touch()
+
+        source = LocalSource(music_dir=tmp_path / "other")
+        track = source.get_track(str(f))
+
+        assert track is not None
+        assert track.source == str(f)
+
+    def test_unsupported_extension_in_cwd_returns_none(self, tmp_path, monkeypatch):
+        cwd = tmp_path / "cwd"
+        music_dir = tmp_path / "music"
+        cwd.mkdir()
+        music_dir.mkdir()
+        (cwd / "notes.txt").touch()
+        monkeypatch.chdir(cwd)
+
+        source = LocalSource(music_dir=music_dir)
+        track = source.get_track("notes.txt")
+
+        assert track is None