worklog-opsdevnz 0.2.0__tar.gz → 0.2.1__tar.gz

{worklog_opsdevnz-0.2.0/src/worklog_opsdevnz.egg-info → worklog_opsdevnz-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: worklog-opsdevnz
-Version: 0.2.0
+Version: 0.2.1
 Summary: Configurable worklog management CLI for dated development logs
 Author-email: "OpsDev.nz Collective" <john@opsdev.nz>
 License: Apache-2.0
@@ -50,6 +50,9 @@ worklog-opsdevnz --version
 
 # Override the editor for this run
 worklog-opsdevnz --editor nvim
+
+# Open the most recent previous entry
+worklog-opsdevnz -p
 ```
 
 ## Configuration
@@ -85,6 +88,7 @@ title = "Next"
 - Configurable section headers per project
 - Custom body templates with `{{DATE}}` and `{{TITLE}}` placeholders
 - Editor integration: `-e` flag → config → `$VISUAL` → `$EDITOR`
+- `-p` / `--previous` flag to open the most recent past entry
 - `--version` flag for installed version
 
 ## Requirements

{worklog_opsdevnz-0.2.0 → worklog_opsdevnz-0.2.1}/README.md

@@ -16,6 +16,9 @@ worklog-opsdevnz --version
 
 # Override the editor for this run
 worklog-opsdevnz --editor nvim
+
+# Open the most recent previous entry
+worklog-opsdevnz -p
 ```
 
 ## Configuration
@@ -51,6 +54,7 @@ title = "Next"
 - Configurable section headers per project
 - Custom body templates with `{{DATE}}` and `{{TITLE}}` placeholders
 - Editor integration: `-e` flag → config → `$VISUAL` → `$EDITOR`
+- `-p` / `--previous` flag to open the most recent past entry
 - `--version` flag for installed version
 
 ## Requirements

{worklog_opsdevnz-0.2.0 → worklog_opsdevnz-0.2.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "worklog-opsdevnz"
-version = "0.2.0"
+version = "0.2.1"
 description = "Configurable worklog management CLI for dated development logs"
 readme = "README.md"
 license = {text = "Apache-2.0"}
@@ -65,6 +65,11 @@ addopts = "-v --cov=src/worklog_opsdevnz --cov-report=term-missing"
 line-length = 88
 target-version = "py312"
 
+# Explicit rule set (ruff's historical defaults) so CI is stable across
+# ruff upgrades — DTZ/I/etc. are intentionally not enabled: local-date
+# handling is this tool's core domain, not an oversight.
+lint.select = ["E4", "E7", "E9", "F"]
+
 [tool.mypy]
 python_version = "3.12"
 warn_return_any = true

{worklog_opsdevnz-0.2.0 → worklog_opsdevnz-0.2.1}/src/worklog_opsdevnz/cli.py

@@ -9,7 +9,7 @@ import click
 
 from worklog_opsdevnz import __version__
 from worklog_opsdevnz.config import get_config
-from worklog_opsdevnz.paths import resolve_path
+from worklog_opsdevnz.paths import find_previous, resolve_path
 from worklog_opsdevnz.template import generate_content
 
 
@@ -21,11 +21,27 @@ from worklog_opsdevnz.template import generate_content
     default=None,
     help="Override the editor command (overrides config, $VISUAL, $EDITOR).",
 )
-def main(editor: str | None) -> None:
-    """Create or open today's worklog entry."""
+@click.option(
+    "-p",
+    "--previous",
+    is_flag=True,
+    help="Open the most recent worklog entry before today (never creates).",
+)
+def main(editor: str | None, previous: bool) -> None:
+    """Create or open today's worklog entry, or the previous one with -p."""
     entry_date = date.today()
 
     config = get_config()
+
+    if previous:
+        target = find_previous(config, entry_date)
+        if target is None:
+            click.echo("No previous worklog entry found.", err=True)
+            return
+        print(f"Opening previous worklog: {target}")
+        _open_editor(target, editor, config.get("editor"))
+        return
+
     target = resolve_path(config, entry_date)
     target.parent.mkdir(parents=True, exist_ok=True)
 

worklog_opsdevnz-0.2.1/src/worklog_opsdevnz/paths.py

@@ -0,0 +1,73 @@
+"""Path resolution for worklog files."""
+
+from datetime import date, datetime
+from pathlib import Path
+from typing import Any
+
+
+def resolve_path(
+    config: dict[str, Any],
+    entry_date: date,
+) -> Path:
+    """Compute the target file path based on config structure."""
+    base = Path(config.get("worklog_dir", "docs/worklog"))
+    structure = config.get("structure", "year")
+    suffix = config.get("suffix", "worklog")
+
+    if structure == "flat":
+        filename = f"{entry_date.isoformat()}-{suffix}.md"
+        return base / filename
+    elif structure == "year-month":
+        year = entry_date.strftime("%Y")
+        month = entry_date.strftime("%m")
+        filename = f"{entry_date.strftime('%d-%m-%Y')}-{suffix}.md"
+        return base / year / month / filename
+    else:  # "year" (default)
+        year = entry_date.strftime("%Y")
+        filename = f"{entry_date.strftime('%d-%m-%Y')}-{suffix}.md"
+        return base / year / filename
+
+
+_DATE_FORMATS = ("%Y-%m-%d", "%d-%m-%Y")
+
+
+def _parse_entry_date(filename: str, suffix: str) -> date | None:
+    """Extract the entry date from a worklog filename, or None if not parseable."""
+    stem = filename.removesuffix(".md")
+    prefix = stem.removesuffix(f"-{suffix}")
+    for fmt in _DATE_FORMATS:
+        try:
+            return datetime.strptime(prefix, fmt).date()
+        except ValueError:
+            continue
+    return None
+
+
+def find_previous(config: dict[str, Any], today: date) -> Path | None:
+    """Locate the most recent worklog entry strictly before *today*.
+
+    Scans the directory tree implied by the configured structure mode
+    (flat, year, or year-month) for files matching ``*-{suffix}.md``,
+    parses their dates, and returns the newest one older than today.
+    Returns None when no previous entry exists. Never creates anything.
+    """
+    base = Path(config.get("worklog_dir", "docs/worklog"))
+    suffix = config.get("suffix", "worklog")
+    structure = config.get("structure", "year")
+
+    pattern = f"*-{suffix}.md"
+    if structure == "flat":
+        candidates = base.glob(pattern)
+    elif structure == "year-month":
+        candidates = base.glob(f"*/*/{pattern}")
+    else:  # "year" (default)
+        candidates = base.glob(f"*/{pattern}")
+
+    best: tuple[date, Path] | None = None
+    for path in candidates:
+        entry_date = _parse_entry_date(path.name, suffix)
+        if entry_date is None or entry_date >= today:
+            continue
+        if best is None or entry_date > best[0]:
+            best = (entry_date, path)
+    return best[1] if best else None

{worklog_opsdevnz-0.2.0 → worklog_opsdevnz-0.2.1/src/worklog_opsdevnz.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: worklog-opsdevnz
-Version: 0.2.0
+Version: 0.2.1
 Summary: Configurable worklog management CLI for dated development logs
 Author-email: "OpsDev.nz Collective" <john@opsdev.nz>
 License: Apache-2.0
@@ -50,6 +50,9 @@ worklog-opsdevnz --version
 
 # Override the editor for this run
 worklog-opsdevnz --editor nvim
+
+# Open the most recent previous entry
+worklog-opsdevnz -p
 ```
 
 ## Configuration
@@ -85,6 +88,7 @@ title = "Next"
 - Configurable section headers per project
 - Custom body templates with `{{DATE}}` and `{{TITLE}}` placeholders
 - Editor integration: `-e` flag → config → `$VISUAL` → `$EDITOR`
+- `-p` / `--previous` flag to open the most recent past entry
 - `--version` flag for installed version
 
 ## Requirements

worklog_opsdevnz-0.2.1/tests/test_cli.py

@@ -0,0 +1,137 @@
+"""Tests for CLI entry point."""
+
+from datetime import date, timedelta
+
+from click.testing import CliRunner
+
+from worklog_opsdevnz.cli import main
+
+
+def test_main_creates_file(tmp_path, monkeypatch):
+    """Running with no args creates today's worklog in tmp_path."""
+    monkeypatch.chdir(tmp_path)
+    runner = CliRunner()
+    result = runner.invoke(main, [])
+    assert result.exit_code == 0
+    assert "Created" in result.output
+    expected_leaf = (
+        f"docs/worklog/{date.today().year}/{date.today():%d-%m-%Y}-worklog.md"
+    )
+    assert expected_leaf in result.output
+
+
+def test_main_opens_existing(tmp_path, monkeypatch):
+    """Running again opens the existing file."""
+    monkeypatch.chdir(tmp_path)
+    runner = CliRunner()
+    runner.invoke(main, [])
+    result = runner.invoke(main, [])
+    assert result.exit_code == 0
+    assert "Opening existing" in result.output
+
+
+def test_main_help():
+    """--help works."""
+    runner = CliRunner()
+    result = runner.invoke(main, ["--help"])
+    assert result.exit_code == 0
+    assert "--editor" in result.output
+
+
+def test_main_version():
+    """--version prints the installed version."""
+    runner = CliRunner()
+    result = runner.invoke(main, ["--version"])
+    assert result.exit_code == 0
+    assert "worklog-opsdevnz, version" in result.output
+    # Extract and verify it's a non-empty version string
+    version_str = result.output.strip().split(", version ")[-1]
+    assert version_str
+
+
+def test_main_editor_from_config(tmp_path, monkeypatch):
+    """Editor in worklog.toml is resolved before $VISUAL/$EDITOR."""
+    monkeypatch.chdir(tmp_path)
+    (tmp_path / "worklog.toml").write_text('editor = "fake-editor-test"\n')
+    runner = CliRunner()
+    result = runner.invoke(main, [])
+    assert result.exit_code == 0
+    # config-based editor picked up but not found in PATH
+    assert "Editor 'fake-editor-test' not found" in result.output
+
+
+def test_main_editor_cli_overrides_config(tmp_path, monkeypatch):
+    """-e flag overrides the editor set in worklog.toml."""
+    monkeypatch.chdir(tmp_path)
+    (tmp_path / "worklog.toml").write_text('editor = "fake-editor-test"\n')
+    runner = CliRunner()
+    result = runner.invoke(main, ["-e", "override-editor-test"])
+    assert result.exit_code == 0
+    # CLI override takes priority over config
+    assert "Editor 'override-editor-test' not found" in result.output
+
+
+def test_previous_no_entries(tmp_path, monkeypatch):
+    """-p with no prior entries: message to stderr, exit 0, creates nothing."""
+    monkeypatch.chdir(tmp_path)
+    runner = CliRunner()
+    result = runner.invoke(main, ["-p"])
+    assert result.exit_code == 0
+    assert "No previous worklog entry found." in result.stderr
+    assert not (tmp_path / "docs").exists()
+
+
+def test_previous_opens_most_recent_before_today(tmp_path, monkeypatch):
+    """-p opens the newest prior entry, not today's, and creates no file."""
+    monkeypatch.chdir(tmp_path)
+    today = date.today()
+    year_dir = tmp_path / "docs" / "worklog" / str(today.year)
+    year_dir.mkdir(parents=True)
+    older = (today - timedelta(days=3)).strftime("%d-%m-%Y")
+    recent = (today - timedelta(days=1)).strftime("%d-%m-%Y")
+    (year_dir / f"{older}-worklog.md").write_text("older")
+    (year_dir / f"{recent}-worklog.md").write_text("recent")
+    today_file = year_dir / f"{today:%d-%m-%Y}-worklog.md"
+    today_file.write_text("today")
+
+    runner = CliRunner()
+    result = runner.invoke(main, ["--previous"])
+    assert result.exit_code == 0
+    assert recent in result.output
+    assert today.strftime("%d-%m-%Y") not in result.output
+
+
+def test_previous_does_not_create_today(tmp_path, monkeypatch):
+    """-p must not create today's entry even though it does not exist."""
+    monkeypatch.chdir(tmp_path)
+    today = date.today()
+    year_dir = tmp_path / "docs" / "worklog" / str(today.year)
+    year_dir.mkdir(parents=True)
+    (year_dir / f"{(today - timedelta(days=2)):%d-%m-%Y}-worklog.md").write_text("x")
+
+    runner = CliRunner()
+    runner.invoke(main, ["-p"])
+    assert not (year_dir / f"{today:%d-%m-%Y}-worklog.md").exists()
+
+
+def test_previous_editor_override(tmp_path, monkeypatch):
+    """-p combined with -e applies the editor override to the previous entry."""
+    monkeypatch.chdir(tmp_path)
+    today = date.today()
+    year_dir = tmp_path / "docs" / "worklog" / str(today.year)
+    year_dir.mkdir(parents=True)
+    recent = (today - timedelta(days=1)).strftime("%d-%m-%Y")
+    (year_dir / f"{recent}-worklog.md").write_text("recent")
+
+    runner = CliRunner()
+    result = runner.invoke(main, ["-p", "-e", "override-editor-test"])
+    assert result.exit_code == 0
+    assert "Editor 'override-editor-test' not found" in result.output
+
+
+def test_previous_help():
+    """--help mentions the --previous flag."""
+    runner = CliRunner()
+    result = runner.invoke(main, ["--help"])
+    assert result.exit_code == 0
+    assert "--previous" in result.output

worklog_opsdevnz-0.2.1/tests/test_paths.py

@@ -0,0 +1,79 @@
+"""Tests for path resolution."""
+
+from datetime import date
+
+from worklog_opsdevnz.paths import find_previous, resolve_path
+
+
+def test_resolve_path_flat():
+    config = {"worklog_dir": "docs/worklog", "structure": "flat", "suffix": "worklog"}
+    path = resolve_path(config, date(2026, 5, 23))
+    assert str(path) == "docs/worklog/2026-05-23-worklog.md"
+
+
+def test_resolve_path_year():
+    config = {"worklog_dir": "logs", "structure": "year", "suffix": "log"}
+    path = resolve_path(config, date(2026, 5, 23))
+    assert str(path) == "logs/2026/23-05-2026-log.md"
+
+
+def test_resolve_path_year_month():
+    config = {"worklog_dir": "logs", "structure": "year-month", "suffix": "ops"}
+    path = resolve_path(config, date(2026, 5, 23))
+    assert str(path) == "logs/2026/05/23-05-2026-ops.md"
+
+
+def test_resolve_path_custom_suffix():
+    config = {"structure": "year", "suffix": "retro"}
+    path = resolve_path(config, date(2026, 1, 1))
+    assert str(path).endswith("-retro.md")
+
+
+def test_find_previous_year(tmp_path):
+    base = tmp_path / "logs" / "2026"
+    base.mkdir(parents=True)
+    (base / "28-08-2026-worklog.md").write_text("old")
+    (base / "01-09-2026-worklog.md").write_text("prev")
+    (base / "03-09-2026-worklog.md").write_text("today")
+    config = {"worklog_dir": str(tmp_path / "logs"), "structure": "year", "suffix": "worklog"}
+    result = find_previous(config, date(2026, 9, 3))
+    assert result is not None
+    assert result.name == "01-09-2026-worklog.md"
+
+
+def test_find_previous_year_month(tmp_path):
+    base = tmp_path / "docs" / "worklogs"
+    (base / "2026" / "08").mkdir(parents=True)
+    (base / "2026" / "09").mkdir(parents=True)
+    (base / "2026" / "08" / "27-08-2026-worklog.md").write_text("aug")
+    (base / "2026" / "09" / "01-09-2026-worklog.md").write_text("sep")
+    config = {"worklog_dir": str(base), "structure": "year-month", "suffix": "worklog"}
+    result = find_previous(config, date(2026, 9, 2))
+    assert result is not None
+    assert result.name == "01-09-2026-worklog.md"
+
+
+def test_find_previous_flat_iso_names(tmp_path):
+    base = tmp_path / "wl"
+    base.mkdir()
+    (base / "2026-08-30-worklog.md").write_text("a")
+    (base / "2026-09-01-worklog.md").write_text("b")
+    config = {"worklog_dir": str(base), "structure": "flat", "suffix": "worklog"}
+    result = find_previous(config, date(2026, 9, 2))
+    assert result is not None
+    assert result.name == "2026-09-01-worklog.md"
+
+
+def test_find_previous_ignores_today_unrelated_and_wrong_suffix(tmp_path):
+    base = tmp_path / "logs" / "2026"
+    base.mkdir(parents=True)
+    (base / "03-09-2026-worklog.md").write_text("today")
+    (base / "README.md").write_text("not an entry")
+    (base / "01-09-2026-journal.md").write_text("wrong suffix")
+    config = {"worklog_dir": str(tmp_path / "logs"), "structure": "year", "suffix": "worklog"}
+    assert find_previous(config, date(2026, 9, 3)) is None
+
+
+def test_find_previous_missing_dir(tmp_path):
+    config = {"worklog_dir": str(tmp_path / "nothing"), "structure": "year", "suffix": "worklog"}
+    assert find_previous(config, date(2026, 9, 3)) is None

worklog_opsdevnz-0.2.0/src/worklog_opsdevnz/paths.py

@@ -1,28 +0,0 @@
-"""Path resolution for worklog files."""
-
-from datetime import date
-from pathlib import Path
-from typing import Any
-
-
-def resolve_path(
-    config: dict[str, Any],
-    entry_date: date,
-) -> Path:
-    """Compute the target file path based on config structure."""
-    base = Path(config.get("worklog_dir", "docs/worklog"))
-    structure = config.get("structure", "year")
-    suffix = config.get("suffix", "worklog")
-
-    if structure == "flat":
-        filename = f"{entry_date.isoformat()}-{suffix}.md"
-        return base / filename
-    elif structure == "year-month":
-        year = entry_date.strftime("%Y")
-        month = entry_date.strftime("%m")
-        filename = f"{entry_date.strftime('%d-%m-%Y')}-{suffix}.md"
-        return base / year / month / filename
-    else:  # "year" (default)
-        year = entry_date.strftime("%Y")
-        filename = f"{entry_date.strftime('%d-%m-%Y')}-{suffix}.md"
-        return base / year / filename

worklog_opsdevnz-0.2.0/tests/test_cli.py

@@ -1,71 +0,0 @@
-"""Tests for CLI entry point."""
-
-from datetime import date
-
-from click.testing import CliRunner
-
-from worklog_opsdevnz.cli import main
-
-
-def test_main_creates_file(tmp_path, monkeypatch):
-    """Running with no args creates today's worklog in tmp_path."""
-    monkeypatch.chdir(tmp_path)
-    runner = CliRunner()
-    result = runner.invoke(main, [])
-    assert result.exit_code == 0
-    assert "Created" in result.output
-    expected_leaf = (
-        f"docs/worklog/{date.today().year}/{date.today():%d-%m-%Y}-worklog.md"
-    )
-    assert expected_leaf in result.output
-
-
-def test_main_opens_existing(tmp_path, monkeypatch):
-    """Running again opens the existing file."""
-    monkeypatch.chdir(tmp_path)
-    runner = CliRunner()
-    runner.invoke(main, [])
-    result = runner.invoke(main, [])
-    assert result.exit_code == 0
-    assert "Opening existing" in result.output
-
-
-def test_main_help():
-    """--help works."""
-    runner = CliRunner()
-    result = runner.invoke(main, ["--help"])
-    assert result.exit_code == 0
-    assert "--editor" in result.output
-
-
-def test_main_version():
-    """--version prints the installed version."""
-    runner = CliRunner()
-    result = runner.invoke(main, ["--version"])
-    assert result.exit_code == 0
-    assert "worklog-opsdevnz, version" in result.output
-    # Extract and verify it's a non-empty version string
-    version_str = result.output.strip().split(", version ")[-1]
-    assert version_str
-
-
-def test_main_editor_from_config(tmp_path, monkeypatch):
-    """Editor in worklog.toml is resolved before $VISUAL/$EDITOR."""
-    monkeypatch.chdir(tmp_path)
-    (tmp_path / "worklog.toml").write_text('editor = "fake-editor-test"\n')
-    runner = CliRunner()
-    result = runner.invoke(main, [])
-    assert result.exit_code == 0
-    # config-based editor picked up but not found in PATH
-    assert "Editor 'fake-editor-test' not found" in result.output
-
-
-def test_main_editor_cli_overrides_config(tmp_path, monkeypatch):
-    """-e flag overrides the editor set in worklog.toml."""
-    monkeypatch.chdir(tmp_path)
-    (tmp_path / "worklog.toml").write_text('editor = "fake-editor-test"\n')
-    runner = CliRunner()
-    result = runner.invoke(main, ["-e", "override-editor-test"])
-    assert result.exit_code == 0
-    # CLI override takes priority over config
-    assert "Editor 'override-editor-test' not found" in result.output

worklog_opsdevnz-0.2.0/tests/test_paths.py

@@ -1,29 +0,0 @@
-"""Tests for path resolution."""
-
-from datetime import date
-
-from worklog_opsdevnz.paths import resolve_path
-
-
-def test_resolve_path_flat():
-    config = {"worklog_dir": "docs/worklog", "structure": "flat", "suffix": "worklog"}
-    path = resolve_path(config, date(2026, 5, 23))
-    assert str(path) == "docs/worklog/2026-05-23-worklog.md"
-
-
-def test_resolve_path_year():
-    config = {"worklog_dir": "logs", "structure": "year", "suffix": "log"}
-    path = resolve_path(config, date(2026, 5, 23))
-    assert str(path) == "logs/2026/23-05-2026-log.md"
-
-
-def test_resolve_path_year_month():
-    config = {"worklog_dir": "logs", "structure": "year-month", "suffix": "ops"}
-    path = resolve_path(config, date(2026, 5, 23))
-    assert str(path) == "logs/2026/05/23-05-2026-ops.md"
-
-
-def test_resolve_path_custom_suffix():
-    config = {"structure": "year", "suffix": "retro"}
-    path = resolve_path(config, date(2026, 1, 1))
-    assert str(path).endswith("-retro.md")