lemonpie-cli 0.2.0a3__tar.gz → 0.2.0b1__tar.gz

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/.github/workflows/ci.yml

@@ -28,5 +28,5 @@ jobs:
       - name: Verify build artifact
         run: uv build
 
-      # - name: Run tests (Uncomment in Phase 3)
-      #   run: uv run pytest
+      - name: Run tests
+        run: uv run pytest

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/.github/workflows/release.yml

@@ -21,6 +21,7 @@ jobs:
       - uses: googleapis/release-please-action@v4
         id: release
         with:
+          token: ${{ secrets.RELEASE_PLEASE_TOKEN }}
           config-file: .release-please/config.json
           manifest-file: .release-please/manifest.json
 

lemonpie_cli-0.2.0b1/.release-please/manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.2.0-beta.1"
+}

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.2.0-beta.1](https://github.com/famousStratum/LemonPie/compare/v0.2.0-alpha.3...v0.2.0-beta.1) (2026-09-22)
+
+
+### Bug Fixes
+
+* eliminate now_ts() double-calls and session ID collisions in session creation ([#27](https://github.com/famousStratum/LemonPie/issues/27)) ([7864136](https://github.com/famousStratum/LemonPie/commit/7864136b6dc200e1e682b61252eadc74e99ac66a))
+
 ## [0.2.0-alpha.3](https://github.com/famousStratum/LemonPie/compare/v0.2.0-alpha.2...v0.2.0-alpha.3) (2026-09-21)
 
 

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: lemonpie-cli
-Version: 0.2.0a3
+Version: 0.2.0b1
 Summary: LemonPie — A lightweight Ollama client for terminal users
 Project-URL: homepage, https://versicode.com/lemonpie
 Project-URL: repository, https://github.com/famousStratum/LemonPie

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/pyproject.toml

@@ -37,3 +37,10 @@ version-file = "src/lemonpie/_version.py"
 
 [tool.hatch.build.targets.wheel]
 packages = ["src/lemonpie"]
+
+[dependency-groups]
+dev = ["pytest>=8.0"]
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+addopts = "-ra"

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/src/lemonpie/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.2.0a3'
-__version_tuple__ = version_tuple = (0, 2, 0, 'a3')
+__version__ = version = '0.2.0b1'
+__version_tuple__ = version_tuple = (0, 2, 0, 'b1')
 
 __commit_id__ = commit_id = None

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/src/lemonpie/cli/config_cmd.py

@@ -196,7 +196,7 @@ def cmd_show(cfg):
     return 0
 
 
-def main():
+def build_parser():
     parser = argparse.ArgumentParser(prog="lmConfig", description="Manage LemonPie configuration (config.json)")
     parser.add_argument("--host", help="Set Ollama host URL (http(s)://host:port)")
     parser.add_argument("--default-model", help="Set default model using alias or full name (must exist in models)")
@@ -205,6 +205,11 @@ def main():
     parser.add_argument("--list-models", action="store_true", help="List configured model aliases and names")
     parser.add_argument("--show", action="store_true", help="Print full config.json")
     parser.add_argument("--force", action="store_true", help="Force overwrite when adding a model")
+    return parser
+
+
+def main():
+    parser = build_parser()
     args = parser.parse_args()
 
     cfg = load_config()

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/src/lemonpie/main.py

@@ -47,7 +47,6 @@ from lemonpie.engine.ollama import (
 )
 
 from lemonpie.sessions.storage import (
-    create_session,
     delete_session_by_id,
     delete_all_sessions,
     resolve_session,
@@ -134,13 +133,9 @@ def main():
     if args.prompt:
         user_prompt = " ".join(args.prompt)
         turn_ts = now_ts()
-        if session:
-            session["history"].append({"role": "user", "content": user_prompt, "ts": turn_ts})
-            session["updated_at"] = turn_ts
-            save_session(session_id, session)
-        else:
-            session = create_session(user_prompt, model)
-            session_id = session["id"]
+        session["history"].append({"role": "user", "content": user_prompt, "ts": turn_ts})
+        session["updated_at"] = turn_ts
+        save_session(session_id, session)
 
         if not find_by_name(cfg, model):
             print(f'Configured models do not include "{model}". Use lmConfig to add it or pass a known alias.')

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/src/lemonpie/sessions/storage.py

@@ -1,4 +1,4 @@
-import glob, json, os, sys
+import glob, json, os, sys, uuid
 from datetime import datetime, timezone
 
 from lemonpie.paths import CURRENT_FILE, SESSIONS_DIR
@@ -99,10 +99,12 @@ def set_title(session, new_title):
     save_session(session["id"], session)
     return True
 
-def create_session(prompt, model):
-    session_id = datetime.now().strftime("%Y%m%d-%H%M%S")
-    title = default_title(" ".join(prompt)) if prompt else "<no title>"
-    session = {"id": session_id, "model": model, "title": title, "history": [], "created_at": now_ts(), "updated_at": now_ts()}
+def create_session(prompt, model, title=None):
+    session_id = datetime.now().strftime("%Y%m%d-%H%M%S") + "-" + uuid.uuid4().hex[:6]
+    if title is None:
+        title = default_title(" ".join(prompt)) if prompt else "<no title>"
+    ts = now_ts()
+    session = {"id": session_id, "model": model, "title": title, "history": [], "created_at": ts, "updated_at": ts}
     save_session(session_id, session)
     write_current(session_id)
     return session
@@ -138,18 +140,8 @@ def update_title(session, model, new_title, prompt=None):
     if not session:
         confirm = input(f'No current session. Create a new session with title "{new_title}"? (Y/n): ')
         if confirm.lower() in ("y", "yes", ""):
-            session_id = datetime.now().strftime("%Y%m%d-%H%M%S")
-            session = {
-                "id": session_id,
-                "model": model,
-                "title": new_title,
-                "history": [],
-                "created_at": now_ts(),
-                "updated_at": now_ts()
-            }
-            save_session(session_id, session)
-            write_current(session_id)
-            print(f'Created new session {session_id} with title "{new_title}".')
+            session = create_session(prompt or [], model, title=new_title)
+            print(f'Created new session {session["id"]} with title "{new_title}".')
         else:
             print("Aborted creating new session.")
             sys.exit(0)

lemonpie_cli-0.2.0b1/tests/conftest.py

@@ -0,0 +1,84 @@
+"""
+Shared pytest fixtures for LemonPie's test suite.
+
+The tricky part here is that `lemonpie.paths` computes CONFIG_FILE / SESSIONS_DIR /
+CURRENT_FILE exactly once, at import time, from $LEMONPIE_CONFIG_DIR and
+$LEMONPIE_SESSIONS_DIR. By the time any test runs, that module has already been
+imported and those names are already bound to real strings. Setting the env vars
+inside a fixture has no effect on values that were already computed.
+
+Worse, `lemonpie.config` and `lemonpie.sessions.storage` each did
+`from lemonpie.paths import CONFIG_FILE` / `from lemonpie.paths import CURRENT_FILE, SESSIONS_DIR`
+— a value copy into their own module namespace, not a live reference back to
+`lemonpie.paths`. So patching `lemonpie.paths.CONFIG_FILE` alone would not affect
+`lemonpie.config.CONFIG_FILE`.
+
+The `isolated_dirs` fixture below patches every one of those copies directly with
+monkeypatch (which auto-reverts after each test), so each test gets its own throwaway
+config file and sessions directory with no risk of touching the developer's real
+~/.config/lemonpie or ~/.local/share/lemonpie.
+"""
+
+import json
+
+import pytest
+
+import lemonpie.config as config
+import lemonpie.paths as paths
+import lemonpie.sessions.storage as storage
+
+
+@pytest.fixture
+def isolated_dirs(tmp_path, monkeypatch):
+    """Redirect all on-disk LemonPie state into a temp directory for one test."""
+    config_dir = tmp_path / "config"
+    data_dir = tmp_path / "data"
+    sessions_dir = data_dir / "sessions"
+    config_dir.mkdir()
+    sessions_dir.mkdir(parents=True)
+
+    config_file = config_dir / "config.json"
+    current_file = sessions_dir / ".current"
+
+    # Keep the env vars themselves in sync too, for any code path that reads
+    # them directly (e.g. a future subprocess-based test).
+    monkeypatch.setenv("LEMONPIE_CONFIG_DIR", str(config_dir))
+    monkeypatch.setenv("LEMONPIE_SESSIONS_DIR", str(sessions_dir))
+
+    # lemonpie.paths itself
+    monkeypatch.setattr(paths, "CONFIG_DIR", str(config_dir))
+    monkeypatch.setattr(paths, "CONFIG_FILE", str(config_file))
+    monkeypatch.setattr(paths, "DATA_DIR", str(data_dir))
+    monkeypatch.setattr(paths, "SESSIONS_DIR", str(sessions_dir))
+    monkeypatch.setattr(paths, "CURRENT_FILE", str(current_file))
+
+    # Every module that imported these names directly (own copies, not live refs)
+    monkeypatch.setattr(config, "CONFIG_FILE", str(config_file))
+    monkeypatch.setattr(storage, "SESSIONS_DIR", str(sessions_dir))
+    monkeypatch.setattr(storage, "CURRENT_FILE", str(current_file))
+
+    return {
+        "config_dir": config_dir,
+        "sessions_dir": sessions_dir,
+        "config_file": config_file,
+        "current_file": current_file,
+    }
+
+
+@pytest.fixture
+def sample_cfg():
+    """A minimal, valid config dict: one aliased model, set as default."""
+    return {
+        "server": "http://127.0.0.1:11434",
+        "default": "qwen2.5-coder:3b",
+        "models": [
+            {"alias": "qwen", "name": "qwen2.5-coder:3b"},
+        ],
+    }
+
+
+@pytest.fixture
+def write_cfg(isolated_dirs, sample_cfg):
+    """Write sample_cfg to the isolated config file and hand back the dict."""
+    isolated_dirs["config_file"].write_text(json.dumps(sample_cfg), encoding="utf-8")
+    return sample_cfg

lemonpie_cli-0.2.0b1/tests/test_cli.py

@@ -0,0 +1,110 @@
+"""Argument-parsing tests for `lm` (lemonpie.cli.parser) and `lmConfig`
+(lemonpie.cli.config_cmd). These only exercise parser.parse_args() — no config
+files, no sessions, no network — so they don't need the isolated_dirs fixture.
+"""
+
+import pytest
+
+from lemonpie.cli.config_cmd import build_parser as build_lmconfig_parser
+from lemonpie.cli.parser import build_parser as build_lm_parser
+
+
+class TestLmParser:
+    def test_default_no_args(self):
+        args = build_lm_parser().parse_args([])
+        assert args.model is None
+        assert args.new is False
+        assert args.session is None
+        assert args.list is False
+        assert args.delete is None
+        assert args.title is None
+        assert args.close is False
+        assert args.print is False
+        assert args.delete_all is False
+        assert args.prompt == []
+
+    def test_prompt_is_collected_as_list(self):
+        args = build_lm_parser().parse_args(["tell", "me", "a", "joke"])
+        assert args.prompt == ["tell", "me", "a", "joke"]
+
+    def test_model_flag_short_and_long_forms_agree(self):
+        assert build_lm_parser().parse_args(["-m", "qwen"]).model == "qwen"
+        assert build_lm_parser().parse_args(["--model", "qwen"]).model == "qwen"
+
+    def test_new_session_flag_with_prompt(self):
+        args = build_lm_parser().parse_args(["-n", "hello"])
+        assert args.new is True
+        assert args.prompt == ["hello"]
+
+    def test_list_flag(self):
+        assert build_lm_parser().parse_args(["-l"]).list is True
+
+    def test_delete_takes_a_session_id(self):
+        args = build_lm_parser().parse_args(["-d", "20260101-000000"])
+        assert args.delete == "20260101-000000"
+
+    def test_delete_all_is_distinct_from_delete(self):
+        args = build_lm_parser().parse_args(["--delete-all"])
+        assert args.delete_all is True
+        assert args.delete is None
+
+    def test_title_combined_with_prompt(self):
+        args = build_lm_parser().parse_args(["-t", "My title", "hi", "there"])
+        assert args.title == "My title"
+        assert args.prompt == ["hi", "there"]
+
+    def test_session_flag_takes_an_id(self):
+        args = build_lm_parser().parse_args(["-s", "20260101-000000"])
+        assert args.session == "20260101-000000"
+
+    def test_close_flag(self):
+        assert build_lm_parser().parse_args(["-c"]).close is True
+
+    def test_print_flag(self):
+        assert build_lm_parser().parse_args(["-p"]).print is True
+
+    def test_version_flag_exits_zero_and_prints(self, capsys):
+        with pytest.raises(SystemExit) as exc_info:
+            build_lm_parser().parse_args(["-v"])
+        assert exc_info.value.code == 0
+        assert capsys.readouterr().out.startswith("lm ")
+
+
+class TestLmConfigParser:
+    def test_no_args_all_defaults(self):
+        args = build_lmconfig_parser().parse_args([])
+        assert args.host is None
+        assert args.default_model is None
+        assert args.add_model is None
+        assert args.remove_model is None
+        assert args.list_models is False
+        assert args.show is False
+        assert args.force is False
+
+    def test_host(self):
+        args = build_lmconfig_parser().parse_args(["--host", "http://127.0.0.1:11434"])
+        assert args.host == "http://127.0.0.1:11434"
+
+    def test_add_model_takes_alias_and_name(self):
+        args = build_lmconfig_parser().parse_args(["--add-model", "qwen", "qwen2.5-coder:3b"])
+        assert args.add_model == ["qwen", "qwen2.5-coder:3b"]
+
+    def test_add_model_with_force(self):
+        args = build_lmconfig_parser().parse_args(
+            ["--add-model", "qwen", "qwen2.5-coder:3b", "--force"]
+        )
+        assert args.force is True
+
+    def test_remove_model_takes_an_identifier(self):
+        args = build_lmconfig_parser().parse_args(["--remove-model", "qwen"])
+        assert args.remove_model == "qwen"
+
+    def test_list_models_flag(self):
+        assert build_lmconfig_parser().parse_args(["--list-models"]).list_models is True
+
+    def test_default_model_takes_an_identifier(self):
+        args = build_lmconfig_parser().parse_args(["--default-model", "qwen"])
+        assert args.default_model == "qwen"
+
+    def test_show_flag(self):
+        assert build_lmconfig_parser().parse_args(["--show"]).show is True

lemonpie_cli-0.2.0b1/tests/test_sessions.py

@@ -0,0 +1,294 @@
+"""Tests for session creation, retrieval, and saving logic in
+lemonpie.sessions.storage. Every test uses the isolated_dirs fixture from
+conftest.py so nothing here touches a real ~/.local/share/lemonpie/sessions.
+"""
+
+import json
+
+import pytest
+
+from lemonpie.sessions import storage
+
+
+# ---------------------------------------------------------------------------
+# create_session / default_title
+# ---------------------------------------------------------------------------
+
+def test_create_session_writes_file_and_sets_current(isolated_dirs):
+    session = storage.create_session(["hello", "world"], "qwen2.5-coder:3b")
+
+    assert session["model"] == "qwen2.5-coder:3b"
+    assert session["title"] == "hello world"
+    assert session["history"] == []
+    assert session["created_at"] == session["updated_at"]
+
+    saved_path = isolated_dirs["sessions_dir"] / f"{session['id']}.json"
+    assert saved_path.exists()
+    on_disk = json.loads(saved_path.read_text(encoding="utf-8"))
+    assert on_disk["id"] == session["id"]
+
+    assert storage.read_current() == session["id"]
+
+
+def test_create_session_without_prompt_uses_placeholder_title(isolated_dirs):
+    session = storage.create_session([], "qwen2.5-coder:3b")
+    assert session["title"] == "<no title>"
+
+
+def test_default_title_truncates_long_prompts(isolated_dirs):
+    long_prompt = "x" * 80
+    title = storage.default_title(long_prompt)
+    assert len(title) <= storage.MAX_TITLE_LEN + 1  # +1 for the trailing "…"
+    assert title.endswith("…")
+
+
+def test_default_title_leaves_short_prompts_untouched(isolated_dirs):
+    assert storage.default_title("short prompt") == "short prompt"
+
+
+# ---------------------------------------------------------------------------
+# save_session / load_session
+# ---------------------------------------------------------------------------
+
+def test_save_and_load_session_round_trip(isolated_dirs):
+    session = storage.create_session(["ping"], "qwen")
+    session["history"].append(
+        {"role": "user", "content": "ping", "ts": "2026-09-22T00:00:00+00:00"}
+    )
+    storage.save_session(session["id"], session)
+
+    loaded = storage.load_session(session["id"])
+    assert loaded["history"] == session["history"]
+    assert loaded["created_at"] == session["created_at"]
+
+
+def test_load_session_missing_returns_empty_shell(isolated_dirs):
+    loaded = storage.load_session("does-not-exist")
+    assert loaded == {
+        "id": "does-not-exist",
+        "title": None,
+        "history": [],
+        "created_at": None,
+        "updated_at": None,
+    }
+
+
+def test_load_session_backfills_missing_timestamps_from_history(isolated_dirs):
+    # Simulate an older/hand-edited session file with no created_at/updated_at,
+    # written directly rather than via create_session().
+    session_id = "20260101-000000"
+    raw = {
+        "id": session_id,
+        "model": "qwen",
+        "title": "old session",
+        "history": [
+            {"role": "user", "content": "hi", "ts": "2026-01-01T00:00:00+00:00"},
+            {"role": "assistant", "content": "hello", "ts": "2026-01-01T00:00:05+00:00"},
+        ],
+    }
+    (isolated_dirs["sessions_dir"] / f"{session_id}.json").write_text(
+        json.dumps(raw), encoding="utf-8"
+    )
+
+    loaded = storage.load_session(session_id)
+    assert loaded["created_at"] == "2026-01-01T00:00:00+00:00"
+    assert loaded["updated_at"] == "2026-01-01T00:00:05+00:00"
+
+
+def test_load_session_backfills_from_mtime_when_history_empty(isolated_dirs):
+    session_id = "20260101-000001"
+    raw = {"id": session_id, "model": "qwen", "title": "empty", "history": []}
+    (isolated_dirs["sessions_dir"] / f"{session_id}.json").write_text(
+        json.dumps(raw), encoding="utf-8"
+    )
+
+    loaded = storage.load_session(session_id)
+    assert loaded["created_at"] is not None
+    assert loaded["updated_at"] == loaded["created_at"]
+
+
+# ---------------------------------------------------------------------------
+# delete_session / delete_all_sessions
+# ---------------------------------------------------------------------------
+
+def test_delete_session_removes_file_and_reports_result(isolated_dirs):
+    session = storage.create_session(["bye"], "qwen")
+    assert storage.delete_session(session["id"]) is True
+    assert not (isolated_dirs["sessions_dir"] / f"{session['id']}.json").exists()
+    assert storage.delete_session(session["id"]) is False  # already gone
+
+
+def test_create_session_ids_are_unique_even_within_the_same_second(isolated_dirs):
+    ids = {storage.create_session([], "qwen")["id"] for _ in range(20)}
+    assert len(ids) == 20
+
+
+def test_delete_all_sessions_clears_dir_and_current(isolated_dirs):
+    storage.create_session(["one"], "qwen")
+    storage.create_session(["two"], "qwen")
+    assert storage.read_current() is not None
+
+    count = storage.delete_all_sessions()
+
+    assert count == 2
+    assert storage.read_current() is None
+    assert list(isolated_dirs["sessions_dir"].glob("*.json")) == []
+
+
+# ---------------------------------------------------------------------------
+# list_sessions
+# ---------------------------------------------------------------------------
+
+def test_list_sessions_marks_current_and_ignores_non_json(isolated_dirs):
+    s1 = storage.create_session(["first"], "qwen")
+    s2 = storage.create_session(["second"], "qwen")  # becomes current
+    (isolated_dirs["sessions_dir"] / "notes.txt").write_text("ignore me", encoding="utf-8")
+
+    sessions = storage.list_sessions()
+    ids = {row[0] for row in sessions}
+    assert ids == {s1["id"], s2["id"]}
+
+    current_rows = [row for row in sessions if row[4] == "*"]
+    assert len(current_rows) == 1
+    assert current_rows[0][0] == s2["id"]
+
+
+def test_list_sessions_skips_invalid_json_without_crashing(isolated_dirs):
+    storage.create_session(["ok"], "qwen")
+    (isolated_dirs["sessions_dir"] / "corrupt.json").write_text(
+        "{not valid json", encoding="utf-8"
+    )
+
+    sessions = storage.list_sessions()
+    assert len(sessions) == 1
+
+
+# ---------------------------------------------------------------------------
+# current-session pointer
+# ---------------------------------------------------------------------------
+
+def test_write_read_clear_current(isolated_dirs):
+    assert storage.read_current() is None
+    storage.write_current("abc")
+    assert storage.read_current() == "abc"
+    storage.clear_current()
+    assert storage.read_current() is None
+
+
+def test_close_session_clears_current(isolated_dirs):
+    storage.write_current("abc")
+    storage.close_session()
+    assert storage.read_current() is None
+
+
+# ---------------------------------------------------------------------------
+# set_title
+# ---------------------------------------------------------------------------
+
+def test_set_title_rejects_overlong_title(isolated_dirs):
+    session = storage.create_session(["hi"], "qwen")
+    too_long = "x" * (storage.MAX_TITLE_LEN + 1)
+    assert storage.set_title(session, too_long) is False
+    reloaded = storage.load_session(session["id"])
+    assert reloaded["title"] != too_long
+
+
+def test_set_title_updates_and_persists(isolated_dirs):
+    session = storage.create_session(["hi"], "qwen")
+    assert storage.set_title(session, "New title") is True
+    reloaded = storage.load_session(session["id"])
+    assert reloaded["title"] == "New title"
+
+
+# ---------------------------------------------------------------------------
+# resolve_session
+# ---------------------------------------------------------------------------
+
+class DummyArgs:
+    """Minimal stand-in for the argparse.Namespace fields resolve_session reads."""
+
+    def __init__(self, new=False, session=None, prompt=None):
+        self.new = new
+        self.session = session
+        self.prompt = prompt or []
+
+
+def test_resolve_session_new_flag_creates_session(isolated_dirs):
+    args = DummyArgs(new=True, prompt=["fresh", "start"])
+    session = storage.resolve_session(args, "qwen", current_id=None)
+    assert session["title"] == "fresh start"
+
+
+def test_resolve_session_with_explicit_session_id_applies_model_and_sets_current(isolated_dirs):
+    existing = storage.create_session(["first"], "qwen")
+    storage.clear_current()
+
+    args = DummyArgs(session=existing["id"])
+    session = storage.resolve_session(args, "qwen2.5-coder:1.5b", current_id=None)
+
+    assert session["id"] == existing["id"]
+    assert session["model"] == "qwen2.5-coder:1.5b"
+    assert storage.read_current() == existing["id"]
+
+
+def test_resolve_session_falls_back_to_current_id(isolated_dirs):
+    existing = storage.create_session(["first"], "qwen")
+    args = DummyArgs()
+    session = storage.resolve_session(args, "qwen", current_id=existing["id"])
+    assert session["id"] == existing["id"]
+
+
+def test_resolve_session_creates_when_nothing_else_applies(isolated_dirs):
+    args = DummyArgs(prompt=["brand", "new"])
+    session = storage.resolve_session(args, "qwen", current_id=None)
+    assert session["title"] == "brand new"
+
+
+# ---------------------------------------------------------------------------
+# functions that call input() / sys.exit()
+# ---------------------------------------------------------------------------
+
+def test_delete_session_by_id_confirmed_removes_file(isolated_dirs, monkeypatch):
+    session = storage.create_session(["gone soon"], "qwen")
+    monkeypatch.setattr("builtins.input", lambda _: "y")
+    storage.delete_session_by_id(session["id"])
+    assert not (isolated_dirs["sessions_dir"] / f"{session['id']}.json").exists()
+
+
+def test_delete_session_by_id_aborted_keeps_file(isolated_dirs, monkeypatch):
+    session = storage.create_session(["stay"], "qwen")
+    monkeypatch.setattr("builtins.input", lambda _: "n")
+    storage.delete_session_by_id(session["id"])
+    assert (isolated_dirs["sessions_dir"] / f"{session['id']}.json").exists()
+
+
+def test_update_title_on_existing_session_without_prompt_exits_zero(isolated_dirs):
+    session = storage.create_session(["hi"], "qwen")
+    with pytest.raises(SystemExit) as exc_info:
+        storage.update_title(session, "qwen", "New title", prompt=None)
+    assert exc_info.value.code == 0
+    assert storage.load_session(session["id"])["title"] == "New title"
+
+
+def test_update_title_on_existing_session_with_prompt_returns_session(isolated_dirs):
+    session = storage.create_session(["hi"], "qwen")
+    result = storage.update_title(session, "qwen", "New title", prompt=["more"])
+    assert result["title"] == "New title"
+
+
+def test_update_title_with_no_session_confirmed_creates_one(isolated_dirs, monkeypatch):
+    monkeypatch.setattr("builtins.input", lambda _: "y")
+    with pytest.raises(SystemExit):
+        storage.update_title(None, "qwen", "Brand new", prompt=None)
+    sessions = storage.list_sessions()
+    assert any(row[1] == "Brand new" for row in sessions)
+
+    created = storage.load_session(storage.read_current())
+    assert created["created_at"] == created["updated_at"]
+
+
+def test_update_title_with_no_session_aborted_exits_zero(isolated_dirs, monkeypatch):
+    monkeypatch.setattr("builtins.input", lambda _: "n")
+    with pytest.raises(SystemExit) as exc_info:
+        storage.update_title(None, "qwen", "Nope", prompt=None)
+    assert exc_info.value.code == 0

{lemonpie_cli-0.2.0a3 → lemonpie_cli-0.2.0b1}/uv.lock

@@ -33,6 +33,15 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/0b/a7/71ac2cff56fec219ed242bb11b8efb69fcc4bec75db06fb7bfe35de520e6/certifi-2026.7.22-py3-none-any.whl", hash = "sha256:62f22742b58a1a33014a2b6b706588a8d7e2a88ae7bd1a6ebe8c992928483775", size = 136983, upload-time = "2026-07-22T03:35:11.276Z" },
 ]
 
+[[package]]
+name = "colorama"
+version = "0.4.6"
+source = { registry = "https://pypi.org/simple" }
+sdist = { url = "https://files.pythonhosted.org/packages/d8/53/6f443c9a4a8358a93a6792e2acffb9d9d5cb0a5cfd8802644b7b1c9a02e4/colorama-0.4.6.tar.gz", hash = "sha256:08695f5cb7ed6e0531a20572697297273c47b8cae5a63ffc6d6ed5c201be6e44", size = 27697, upload-time = "2022-10-25T02:36:22.414Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/d1/d6/3965ed04c63042e047cb6a3e6ed1a63a35087b6a609aa3a15ed8ac56c221/colorama-0.4.6-py2.py3-none-any.whl", hash = "sha256:4f1d9991f5acc0ca119f9d443620b77f9d6b33703e51011c16baf57afb285fc6", size = 25335, upload-time = "2022-10-25T02:36:20.889Z" },
+]
+
 [[package]]
 name = "h11"
 version = "0.16.0"
@@ -79,6 +88,15 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/58/a2/bb081bab032533a855d44de1d56f8e8426114ff1ba5d1f07a438a0a654f8/idna-3.20-py3-none-any.whl", hash = "sha256:ab7ae7122974553370f0bdb919e1a960b2cd1bc1ef0276416d896db81c14582c", size = 69583, upload-time = "2026-09-17T14:11:03.168Z" },
 ]
 
+[[package]]
+name = "iniconfig"
+version = "2.3.0"
+source = { registry = "https://pypi.org/simple" }
+sdist = { url = "https://files.pythonhosted.org/packages/72/34/14ca021ce8e5dfedc35312d08ba8bf51fdd999c576889fc2c24cb97f4f10/iniconfig-2.3.0.tar.gz", hash = "sha256:c76315c77db068650d49c5b56314774a7804df16fee4402c1f19d6d15d8c4730", size = 20503, upload-time = "2025-10-18T21:55:43.219Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/cb/b1/3846dd7f199d53cb17f49cba7e651e9ce294d8497c8c150530ed11865bb8/iniconfig-2.3.0-py3-none-any.whl", hash = "sha256:f631c04d2c48c52b84d0d0549c99ff3859c98df65b3101406327ecc7d53fbf12", size = 7484, upload-time = "2025-10-18T21:55:41.639Z" },
+]
+
 [[package]]
 name = "lemonpie-cli"
 source = { editable = "." }
@@ -88,6 +106,11 @@ dependencies = [
     { name = "platformdirs" },
 ]
 
+[package.dev-dependencies]
+dev = [
+    { name = "pytest" },
+]
+
 [package.metadata]
 requires-dist = [
     { name = "httpx" },
@@ -95,6 +118,9 @@ requires-dist = [
     { name = "platformdirs" },
 ]
 
+[package.metadata.requires-dev]
+dev = [{ name = "pytest", specifier = ">=8.0" }]
+
 [[package]]
 name = "ollama"
 version = "0.6.2"
@@ -108,6 +134,15 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/c4/ab/d6722beeb2d10f7a3b9ff49375708904fde18f82b5609a0bc4aeb5996a4d/ollama-0.6.2-py3-none-any.whl", hash = "sha256:3ad7daab28e5a973445c36a73882a3ef698c2ebb00e21e308652741577509f7d", size = 15115, upload-time = "2026-04-29T21:21:13.794Z" },
 ]
 
+[[package]]
+name = "packaging"
+version = "26.3"
+source = { registry = "https://pypi.org/simple" }
+sdist = { url = "https://files.pythonhosted.org/packages/7d/fa/3944b40b07da9ce895c0e6303a5ab7d53da063554f534556b134a54d6093/packaging-26.3.tar.gz", hash = "sha256:94edc256424af38762eb31306eed28beb9f0efc50a8837492c9d6fd6004aed79", size = 313412, upload-time = "2026-08-04T18:15:28.737Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/63/34/ba1c580383c9eada3711951fef0795c80b829a078d72188184bcab9dd527/packaging-26.3-py3-none-any.whl", hash = "sha256:d7193f7c8e4e93f444fde0262bf90af30e16fa0ad0ad44cb553c87339b23cd1c", size = 129956, upload-time = "2026-08-04T18:15:27.159Z" },
+]
+
 [[package]]
 name = "platformdirs"
 version = "4.11.11"
@@ -117,6 +152,15 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/39/be/3ae887e84c38c4dd1c549cd5825adc93f48069a94fe1cb8ac52a9da16f15/platformdirs-4.11.11-py3-none-any.whl", hash = "sha256:972ea6b2b387155a536226a0750e46923d731d459a387c4a255c583ed2f64547", size = 24992, upload-time = "2026-09-19T01:18:45.825Z" },
 ]
 
+[[package]]
+name = "pluggy"
+version = "1.6.0"
+source = { registry = "https://pypi.org/simple" }
+sdist = { url = "https://files.pythonhosted.org/packages/f9/e2/3e91f31a7d2b083fe6ef3fa267035b518369d9511ffab804f839851d2779/pluggy-1.6.0.tar.gz", hash = "sha256:7dcc130b76258d33b90f61b658791dede3486c3e6bfb003ee5c9bfb396dd22f3", size = 69412, upload-time = "2025-05-15T12:30:07.975Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/54/20/4d324d65cc6d9205fabedc306948156824eb9f0ee1633355a8f7ec5c66bf/pluggy-1.6.0-py3-none-any.whl", hash = "sha256:e920276dd6813095e9377c0bc5566d94c932c33b27a3e3945d8389c374dd4746", size = 20538, upload-time = "2025-05-15T12:30:06.134Z" },
+]
+
 [[package]]
 name = "pydantic"
 version = "2.13.5"
@@ -234,6 +278,31 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/db/1d/068464f23075f66a8f1b806935e9cd9363ee446636ea70d2c22ee8659dbf/pydantic_core-2.46.5-pp311-pypy311_pp73-win_amd64.whl", hash = "sha256:d0a24b40877af2de4950252be9d21eaf7fb07660f3c2cae1f56c6b599ada5266", size = 2140686, upload-time = "2026-08-28T10:01:28.947Z" },
 ]
 
+[[package]]
+name = "pygments"
+version = "2.21.0"
+source = { registry = "https://pypi.org/simple" }
+sdist = { url = "https://files.pythonhosted.org/packages/49/2e/ced460408999b33da6b31b0021b0f37d329e202d4169aeb164493778f25b/pygments-2.21.0.tar.gz", hash = "sha256:610ca751c9bc2492b38eb9a38a7fbc93edbbb2d7182edaf34e66ae493dee5c8c", size = 5005329, upload-time = "2026-08-17T08:02:48.824Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/71/46/17f022dd3e953bf20a04a028a21ec746d942f8d2af30fa0f124fa0e6a684/pygments-2.21.0-py3-none-any.whl", hash = "sha256:2363c69b61c4a97c838da3b130dcd6468f4848992b21a82f2a63ec34377137d9", size = 1250147, upload-time = "2026-08-17T08:02:44.912Z" },
+]
+
+[[package]]
+name = "pytest"
+version = "9.1.1"
+source = { registry = "https://pypi.org/simple" }
+dependencies = [
+    { name = "colorama", marker = "sys_platform == 'win32'" },
+    { name = "iniconfig" },
+    { name = "packaging" },
+    { name = "pluggy" },
+    { name = "pygments" },
+]
+sdist = { url = "https://files.pythonhosted.org/packages/e4/47/b9efed96c114afcfa3c9d3fe98a76a1d14c74a9e266d397cf6eb64be5e01/pytest-9.1.1.tar.gz", hash = "sha256:1088fbde8f2b49d95a549a195707afa7a76a3ce9bcadc26b6d71f0ffda5fe313", size = 1636369, upload-time = "2026-06-19T10:58:32.857Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/24/25/1de2678b631f5a49215c6c96fff41ba892b0a34df68d6d80292b1b48aa7f/pytest-9.1.1-py3-none-any.whl", hash = "sha256:37a86b45efb9a47a61a36449063e8e18d0cab3161329fc099eb21783169c4f0c", size = 386536, upload-time = "2026-06-19T10:58:31.347Z" },
+]
+
 [[package]]
 name = "typing-extensions"
 version = "4.16.0"

lemonpie_cli-0.2.0a3/.release-please/manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "0.2.0-alpha.3"
-}