workstream-cli 0.0.1__py3-none-any.whl

workstream/test/test_models.py

@@ -0,0 +1,506 @@
+"""Tests for workstream.models — pure data types and helpers."""
+
+import re
+from datetime import datetime
+
+from workstream.models import (
+    ACTIVE_STATUSES,
+    SIZES,
+    STATUSES,
+    TERMINAL_STATUSES,
+    IdeaEntry,
+    LogEntry,
+    PlanRef,
+    ThoughtEntry,
+    ThreadEntry,
+    Workstream,
+    generate_id,
+    slugify,
+)
+
+_ID_RE = re.compile(r"^\d{4}-\d{2}-\d{2}-[0-9a-f]{10}$")
+
+
+def test_generate_id_format():
+    assert _ID_RE.match(generate_id())
+
+
+def test_generate_id_uniqueness():
+    assert generate_id() != generate_id()
+
+
+def test_slugify_basic():
+    assert slugify("API Redesign") == "api-redesign"
+
+
+def test_slugify_special_chars():
+    assert slugify("Hello, World! (v2)") == "hello-world-v2"
+
+
+def test_slugify_empty():
+    assert slugify("") == "untitled"
+
+
+def test_slugify_unicode():
+    # Non-ASCII chars are stripped by encode('ascii','ignore')
+    assert slugify("Über Cool") == "ber-cool"
+
+
+def test_workstream_defaults():
+    ws = Workstream(id="x", title="y")
+    assert ws.status == "active"
+    assert ws.size == "day"
+    assert ws.tags == []
+    assert ws.repos == []
+    assert ws.created == ""
+    assert ws.updated == ""
+    assert ws.snooze_reason == ""
+    assert ws.ideas == []
+    assert ws.log == []
+    assert ws.thread == []
+    assert ws.decisions == []
+    assert ws.plans == []
+    assert ws.branches == []
+    assert ws.next_actions == []
+    assert ws.source_path is None
+
+
+def test_workstream_filename():
+    ws = Workstream(id="2026-03-28-abc1234567", title="API Redesign")
+    assert ws.filename == "2026-03-28-abc1234567-api-redesign.md"
+
+
+def test_frontmatter_dict_minimal():
+    ws = Workstream(id="x", title="y")
+    d = ws.frontmatter_dict()
+    assert d == {"id": "x", "title": "y", "status": "active", "size": "day"}
+
+
+def test_frontmatter_dict_full():
+    ws = Workstream(
+        id="x",
+        title="y",
+        status="active",
+        size="week",
+        tags=["infra"],
+        repos=["repo-a"],
+        created="2026-01-01",
+        updated="2026-03-28",
+        snooze_reason="waiting on review",
+    )
+    d = ws.frontmatter_dict()
+    assert d == {
+        "id": "x",
+        "title": "y",
+        "status": "active",
+        "size": "week",
+        "tags": ["infra"],
+        "repos": ["repo-a"],
+        "created": "2026-01-01",
+        "updated": "2026-03-28",
+        "snooze_reason": "waiting on review",
+    }
+
+
+def test_days_idle_recent():
+    today = datetime.now().strftime("%Y-%m-%d")
+    ws = Workstream(id="x", title="y", last_activity=today)
+    assert ws.days_idle() == 0
+
+
+def test_days_idle_empty():
+    ws = Workstream(id="x", title="y")
+    assert ws.days_idle() == 0
+
+
+def test_last_thread_entry_empty():
+    ws = Workstream(id="x", title="y")
+    assert ws.last_thread_entry is None
+
+
+def test_last_thread_entry_present():
+    e1 = ThreadEntry(date="2026-03-28", body="first")
+    e2 = ThreadEntry(date="2026-03-27", body="second")
+    ws = Workstream(id="x", title="y", thread=[e1, e2])
+    assert ws.last_thread_entry is e1
+
+
+def test_statuses_are_tuples():
+    assert isinstance(STATUSES, tuple)
+    assert isinstance(ACTIVE_STATUSES, tuple)
+    assert isinstance(TERMINAL_STATUSES, tuple)
+    assert "active" in STATUSES
+    assert "active" in ACTIVE_STATUSES
+    assert "completed" in TERMINAL_STATUSES
+    assert len(SIZES) > 0
+
+
+
+# ── ThoughtEntry ──────────────────────────────────────────────────────
+
+def test_thought_entry_defaults():
+    t = ThoughtEntry(date='2026-03-28', text='Some idea')
+    assert t.snooze_until == ''
+    assert t.snooze_count == 0
+
+
+def test_thought_entry_with_snooze():
+    t = ThoughtEntry(date='2026-03-28', text='Idea', snooze_until='2026-04-04', snooze_count=2)
+    assert t.snooze_until == '2026-04-04'
+    assert t.snooze_count == 2
+
+
+# ── Hierarchy and snooze on Workstream ────────────────────────────────
+
+def test_workstream_hierarchy_defaults():
+    ws = Workstream(id='x', title='y')
+    assert ws.parent == ''
+    assert ws.snooze_until == ''
+    assert ws.snooze_count == 0
+    assert ws.thoughts == []
+
+
+def test_frontmatter_dict_with_parent_and_snooze():
+    ws = Workstream(
+        id='x', title='y',
+        parent='2026-01-01-abc1234567',
+        snooze_until='2026-04-15',
+        snooze_count=3,
+    )
+    d = ws.frontmatter_dict()
+    assert d['parent'] == '2026-01-01-abc1234567'
+    assert d['snooze_until'] == '2026-04-15'
+    assert d['snooze_count'] == 3
+
+
+def test_frontmatter_dict_omits_empty_parent_and_snooze():
+    ws = Workstream(id='x', title='y')
+    d = ws.frontmatter_dict()
+    assert 'parent' not in d
+    assert 'snooze_until' not in d
+    assert 'snooze_count' not in d
+
+
+# ── Activity dates ──────────────────────────────────────────────────────────
+
+def test_activity_dates_defaults():
+    ws = Workstream(id='x', title='y')
+    assert ws.first_activity == ''
+    assert ws.last_activity == ''
+
+
+def test_frontmatter_dict_with_activity_dates():
+    ws = Workstream(
+        id='x', title='y',
+        first_activity='2026-01-15',
+        last_activity='2026-03-28',
+    )
+    d = ws.frontmatter_dict()
+    assert d['first_activity'] == '2026-01-15'
+    assert d['last_activity'] == '2026-03-28'
+
+
+def test_frontmatter_dict_omits_empty_activity_dates():
+    ws = Workstream(id='x', title='y')
+    d = ws.frontmatter_dict()
+    assert 'first_activity' not in d
+    assert 'last_activity' not in d
+
+
+# ── IdeaEntry / LogEntry ──────────────────────────────────────────────
+
+def test_idea_entry_defaults():
+    idea = IdeaEntry(date='2026-03-28', text='Build a widget')
+    assert idea.snooze_until == ''
+    assert idea.snooze_count == 0
+
+
+def test_log_entry_defaults():
+    entry = LogEntry(date='2026-03-28', event='created')
+    assert entry.detail == ''
+
+
+def test_log_entry_with_detail():
+    entry = LogEntry(date='2026-03-28', event='snoozed', detail='until 2026-04-07: Blocked')
+    assert entry.event == 'snoozed'
+    assert entry.detail == 'until 2026-04-07: Blocked'
+
+
+# ── Summary field ──────────────────────────────────────────────────
+
+def test_summary_default():
+    ws = Workstream(id='x', title='y')
+    assert ws.summary == ''
+
+
+def test_frontmatter_dict_with_summary():
+    ws = Workstream(id='x', title='y', summary='Redesigning the API layer for v2 migration.')
+    d = ws.frontmatter_dict()
+    assert d['summary'] == 'Redesigning the API layer for v2 migration.'
+
+
+def test_frontmatter_dict_omits_empty_summary():
+    ws = Workstream(id='x', title='y')
+    d = ws.frontmatter_dict()
+    assert 'summary' not in d
+
+
+# ── days_idle with last_activity ──────────────────────────────────────
+
+def test_days_idle_uses_last_activity_when_newer():
+    """days_idle uses last_activity exclusively; updated is ignored."""
+    today = datetime.now().strftime('%Y-%m-%d')
+    # updated is old, last_activity is today => 0 days idle
+    ws = Workstream(id='x', title='y', updated='2026-01-01', last_activity=today)
+    assert ws.days_idle() == 0
+
+
+def test_days_idle_ignores_updated():
+    """days_idle only consults last_activity, not updated."""
+    today = datetime.now().strftime('%Y-%m-%d')
+    # updated is today but last_activity is old => days_idle reflects last_activity, not updated
+    ws = Workstream(id='x', title='y', updated=today, last_activity='2026-01-01')
+    assert ws.days_idle() > 0
+
+
+def test_days_idle_only_last_activity():
+    today = datetime.now().strftime('%Y-%m-%d')
+    ws = Workstream(id='x', title='y', last_activity=today)
+    assert ws.days_idle() == 0
+
+
+def test_days_idle_updated_only():
+    """When only updated is set, computed_last_activity falls back to it."""
+    today = datetime.now().strftime('%Y-%m-%d')
+    ws = Workstream(id='x', title='y', updated=today)
+    assert ws.days_idle() == 0
+
+
+def test_last_activity_dt_valid():
+    ws = Workstream(id='x', title='y', last_activity='2026-03-28')
+    assert ws.last_activity_dt is not None
+    assert ws.last_activity_dt.year == 2026
+    assert ws.last_activity_dt.month == 3
+    assert ws.last_activity_dt.day == 28
+
+
+def test_last_activity_dt_empty():
+    ws = Workstream(id='x', title='y')
+    assert ws.last_activity_dt is None
+
+
+def test_last_activity_dt_invalid():
+    ws = Workstream(id='x', title='y', last_activity='not-a-date')
+    assert ws.last_activity_dt is None
+
+
+# ── computed_last_activity ────────────────────────────────────────────
+
+def test_computed_last_activity_prefers_plan_date():
+    """Plan dates that are newer than stored last_activity are used."""
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-03-25',
+        plans=[PlanRef(repo='r', path='p.md', status='active',
+                       title='Plan', date='2026-03-30')],
+    )
+    assert ws.computed_last_activity == '2026-03-30'
+
+
+def test_computed_last_activity_ignores_log_entries():
+    """Log entries are bookkeeping (snooze/wake/import), not work signals."""
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-03-25',
+        log=[LogEntry(date='2026-03-29', event='snoozed')],
+    )
+    assert ws.computed_last_activity == '2026-03-25'
+
+
+def test_computed_last_activity_considers_all_work_sources():
+    """The max date across work-artifact types wins; log and ideas are excluded."""
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-03-20',
+        thread=[ThreadEntry(date='2026-03-22', body='note')],
+        thoughts=[ThoughtEntry(date='2026-03-24', text='hmm')],
+        ideas=[IdeaEntry(date='2026-03-26', text='what if')],  # excluded
+        plans=[PlanRef(repo='r', path='p.md', status='active',
+                       title='Plan', date='2026-03-28')],
+        log=[LogEntry(date='2026-03-30', event='snoozed')],  # excluded
+    )
+    assert ws.computed_last_activity == '2026-03-28'  # plan, not log or idea
+
+
+def test_computed_last_activity_falls_back_to_updated():
+    """When no dated entries exist and last_activity is empty, falls back to updated."""
+    ws = Workstream(id='x', title='y', updated='2026-03-15')
+    assert ws.computed_last_activity == '2026-03-15'
+
+
+def test_computed_last_activity_falls_back_to_created():
+    """When updated is also empty, falls back to created."""
+    ws = Workstream(id='x', title='y', created='2026-03-10')
+    assert ws.computed_last_activity == '2026-03-10'
+
+
+def test_computed_last_activity_empty():
+    """Returns empty string when nothing is available."""
+    ws = Workstream(id='x', title='y')
+    assert ws.computed_last_activity == ''
+
+
+def test_days_idle_reflects_plan_activity():
+    """days_idle uses plan dates — the user's core bug."""
+    today = datetime.now().strftime('%Y-%m-%d')
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-01-01',  # stale cache
+        plans=[PlanRef(repo='r', path='p.md', status='active',
+                       title='Plan', date=today)],  # fresh plan
+    )
+    assert ws.days_idle() == 0  # plan date overrides stale cache
+# ── needs_direction ──────────────────────────────────────────────────
+
+def test_needs_direction_true_when_directionless():
+    """Active ws with no next_actions, no active plans, no recent thread."""
+    ws = Workstream(id='x', title='y', status='active')
+    assert ws.needs_direction() is True
+
+
+def test_needs_direction_false_when_snoozed():
+    ws = Workstream(id='x', title='y', status='snoozed')
+    assert ws.needs_direction() is False
+
+
+def test_needs_direction_false_with_next_actions():
+    ws = Workstream(id='x', title='y', status='active',
+                    next_actions=['Do the thing'])
+    assert ws.needs_direction() is False
+
+
+def test_needs_direction_false_with_active_plan():
+    ws = Workstream(id='x', title='y', status='active',
+                    plans=[PlanRef(repo='r', path='p.md', status='active',
+                                   title='Plan', date='2026-03-01')])
+    assert ws.needs_direction() is False
+
+
+def test_needs_direction_false_with_recent_thread():
+    today = datetime.now().strftime('%Y-%m-%d')
+    ws = Workstream(id='x', title='y', status='active',
+                    thread=[ThreadEntry(date=today, body='Working on it')])
+    assert ws.needs_direction() is False
+
+
+def test_needs_direction_true_with_old_thread():
+    ws = Workstream(id='x', title='y', status='active',
+                    thread=[ThreadEntry(date='2026-01-01', body='Stale entry')])
+    assert ws.needs_direction() is True
+
+
+# ── computed_last_activity excludes ideas ─────────────────────────────
+
+def test_computed_last_activity_excludes_ideas():
+    """Ideas are design-level, not work signals — they should not count."""
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-03-20',
+        ideas=[IdeaEntry(date='2026-03-30', text='some idea')],
+    )
+    # Idea date (03-30) is newer but excluded; last_activity (03-20) wins
+    assert ws.computed_last_activity == '2026-03-20'
+
+
+def test_computed_last_activity_ideas_only_falls_back():
+    """Workstream with only ideas has no work signals — falls back to updated."""
+    ws = Workstream(
+        id='x', title='y',
+        updated='2026-03-10',
+        ideas=[IdeaEntry(date='2026-03-30', text='some idea')],
+    )
+    assert ws.computed_last_activity == '2026-03-10'
+
+
+# ── code_last_activity ────────────────────────────────────────────────
+
+def test_code_last_activity_default_empty():
+    ws = Workstream(id='x', title='y')
+    assert ws.code_last_activity == ''
+
+
+def test_code_last_activity_in_frontmatter_dict():
+    ws = Workstream(id='x', title='y', code_last_activity='2026-03-28')
+    d = ws.frontmatter_dict()
+    assert d['code_last_activity'] == '2026-03-28'
+
+
+def test_code_last_activity_omitted_when_empty():
+    ws = Workstream(id='x', title='y')
+    d = ws.frontmatter_dict()
+    assert 'code_last_activity' not in d
+
+
+# ── blocked status ──────────────────────────────────────────────────────────────────
+
+def test_blocked_in_statuses():
+    assert 'blocked' in STATUSES
+    assert 'blocked' in ACTIVE_STATUSES
+    assert 'blocked' not in TERMINAL_STATUSES
+
+
+def test_blocked_notes_default():
+    ws = Workstream(id='x', title='y')
+    assert ws.blocked_notes == ''
+
+
+def test_frontmatter_dict_with_blocked_notes():
+    ws = Workstream(id='x', title='y', status='blocked', blocked_notes='Waiting for API key')
+    d = ws.frontmatter_dict()
+    assert d['blocked_notes'] == 'Waiting for API key'
+    assert d['status'] == 'blocked'
+
+
+def test_frontmatter_dict_omits_empty_blocked_notes():
+    ws = Workstream(id='x', title='y')
+    d = ws.frontmatter_dict()
+    assert 'blocked_notes' not in d
+
+
+def test_needs_direction_false_when_blocked():
+    """Blocked workstreams have direction (they're stuck, not directionless)."""
+    ws = Workstream(id='x', title='y', status='blocked')
+    assert ws.needs_direction() is False
+
+
+# ── code_last_activity in computed_last_activity ───────────────────────────────────
+
+def test_computed_last_activity_includes_code_last_activity():
+    """code_last_activity is a work signal — repos with recent commits are not stale."""
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-03-20',
+        code_last_activity='2026-03-30',
+    )
+    assert ws.computed_last_activity == '2026-03-30'
+
+
+def test_computed_last_activity_code_older_than_other_sources():
+    """code_last_activity is included but doesn't override newer sources."""
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-03-30',
+        code_last_activity='2026-03-20',
+    )
+    assert ws.computed_last_activity == '2026-03-30'
+
+
+def test_days_idle_reflects_code_activity():
+    """A workstream with recent commits is not idle."""
+    today = datetime.now().strftime('%Y-%m-%d')
+    ws = Workstream(
+        id='x', title='y',
+        last_activity='2026-01-01',  # stale cache
+        code_last_activity=today,     # fresh commits
+    )
+    assert ws.days_idle() == 0