workstream-cli 0.0.1__py3-none-any.whl

workstream/test/conftest.py

@@ -0,0 +1,221 @@
+"""Shared pytest fixtures for workstream tests."""
+
+from __future__ import annotations
+
+from pathlib import Path
+from textwrap import dedent
+
+import pytest
+
+from workstream.config import Config
+
+
+# ---------------------------------------------------------------------------
+# Raw markdown text fixture
+# ---------------------------------------------------------------------------
+
+SAMPLE_WORKSTREAM_TEXT = dedent("""\
+    ---
+    id: 2026-03-15-a1b2c3d4e5
+    title: API Redesign
+    status: active
+    size: week
+    tags:
+      - code
+      - backend
+    repos:
+      - myapp
+    created: '2026-03-15'
+    updated: '2026-03-25'
+    ---
+
+    # API Redesign
+    #code #backend
+
+    ## Thread
+    ### 2026-03-25
+    Refactored authentication middleware to use the new token format.
+    Endpoint tests all passing now.
+
+    ### 2026-03-20
+    Sketched out the new resource hierarchy. Going with /v2/ prefix
+    to avoid breaking existing clients.
+
+    ## Decisions
+    - Use /v2/ prefix for all new endpoints
+    - Adopt JSON:API response envelope
+
+    ## Plans
+    <!-- Updated by ws sweep -->
+    - myapp `.plans/api-v2-migration.md` (active)
+
+    ## Branches
+    <!-- Updated by ws sweep -->
+    - myapp: `feature/api-v2` (+12 ahead)
+
+    ## Next
+    - Write migration guide for consumers
+    - Add rate-limiting to /v2/ endpoints
+""")
+
+
+@pytest.fixture()
+def sample_workstream_text() -> str:
+    """Complete workstream markdown with frontmatter and all sections."""
+    return SAMPLE_WORKSTREAM_TEXT
+
+
+# ---------------------------------------------------------------------------
+# Directory of workstream files
+# ---------------------------------------------------------------------------
+
+_WORKSTREAM_SPECS: list[dict] = [
+    {
+        "filename": "2026-03-15-a1b2c3d4e5-api-redesign.md",
+        "id": "2026-03-15-a1b2c3d4e5",
+        "title": "API Redesign",
+        "status": "active",
+        "size": "week",
+        "tags": ["code"],
+        "created": "2026-03-15",
+        "updated": "2026-03-25",
+        "last_activity": "2026-03-25",
+        "thread_date": "2026-03-25",
+        "thread_body": "Refactored auth middleware. Tests passing.",
+        "next": ["Write migration guide", "Add rate-limiting"],
+    },
+    {
+        "filename": "2026-03-10-b2c3d4e5f6-docs-overhaul.md",
+        "id": "2026-03-10-b2c3d4e5f6",
+        "title": "Docs Overhaul",
+        "status": "active",
+        "size": "day",
+        "tags": ["code", "docs"],
+        "created": "2026-03-10",
+        "updated": "2026-03-22",
+        "last_activity": "2026-03-22",
+        "thread_date": "2026-03-22",
+        "thread_body": "Finished API reference section. Need to add examples.",
+        "next": ["Add code examples to each endpoint"],
+    },
+    {
+        "filename": "2026-03-01-c3d4e5f6a7-conference-talk.md",
+        "id": "2026-03-01-c3d4e5f6a7",
+        "title": "Conference Talk",
+        "status": "snoozed",
+        "size": "day",
+        "tags": ["community"],
+        "snooze_reason": "Waiting on CFP results",
+        "created": "2026-03-01",
+        "updated": "2026-03-12",
+        "last_activity": "2026-03-12",
+        "thread_date": "2026-03-12",
+        "thread_body": "Submitted CFP abstract. Fingers crossed.",
+        "next": ["Prepare slide outline once accepted"],
+    },
+    {
+        "filename": "2026-03-20-d4e5f6a7b8-accessibility-audit.md",
+        "id": "2026-03-20-d4e5f6a7b8",
+        "title": "Accessibility Audit",
+        "status": "active",
+        "size": "hour",
+        "tags": ["code"],
+        "created": "2026-03-20",
+        "updated": "2026-03-20",
+        "last_activity": "2026-03-20",
+        "thread_date": "2026-03-20",
+        "thread_body": "Initial idea: run axe-core across all pages.",
+        "next": ["Pick an audit tool"],
+    },
+    {
+        "filename": "2026-03-05-e5f6a7b8c9-sponsorship-outreach.md",
+        "id": "2026-03-05-e5f6a7b8c9",
+        "title": "Sponsorship Outreach",
+        "status": "active",
+        "size": "week",
+        "tags": ["business"],
+        "created": "2026-03-05",
+        "updated": "2026-03-18",
+        "last_activity": "2026-03-18",
+        "thread_date": "2026-03-18",
+        "thread_body": "Sent first batch of emails. Three positive replies so far.",
+        "next": ["Follow up with remaining prospects", "Draft sponsorship tiers"],
+    },
+]
+
+
+def _render_workstream_file(spec: dict) -> str:
+    """Build a minimal but valid workstream markdown file from a spec dict."""
+    lines = [
+        "---",
+        f"id: {spec['id']}",
+        f"title: {spec['title']}",
+        f"status: {spec['status']}",
+        f"size: {spec['size']}",
+    ]
+
+    tags = spec.get("tags", [])
+    if tags:
+        lines.append("tags:")
+        for t in tags:
+            lines.append(f"  - {t}")
+
+    if spec.get("snooze_reason"):
+        lines.append(f"snooze_reason: '{spec['snooze_reason']}'")
+
+    lines.append(f"created: '{spec['created']}'")
+    lines.append(f"updated: '{spec['updated']}'")
+    if spec.get('last_activity'):
+        lines.append(f"last_activity: '{spec['last_activity']}'")
+    lines.append("---")
+    lines.append("")
+    lines.append(f"# {spec['title']}")
+    lines.append("")
+    lines.append("## Thread")
+    lines.append(f"### {spec['thread_date']}")
+    lines.append(spec["thread_body"])
+    lines.append("")
+    lines.append("## Next")
+    for action in spec.get("next", []):
+        lines.append(f"- {action}")
+    lines.append("")
+
+    return "\n".join(lines)
+
+
+@pytest.fixture()
+def workstreams_dir(tmp_path: Path) -> Path:
+    """Temp directory populated with 5 sample workstream files."""
+    ws_dir = tmp_path / "workstreams"
+    ws_dir.mkdir()
+
+    for spec in _WORKSTREAM_SPECS:
+        (ws_dir / spec["filename"]).write_text(
+            _render_workstream_file(spec), encoding="utf-8"
+        )
+
+    return ws_dir
+
+
+# ---------------------------------------------------------------------------
+# Config fixture
+# ---------------------------------------------------------------------------
+
+@pytest.fixture()
+def test_config(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> Config:
+    """Config with workstreams_dir in a temp directory.
+
+    Patches global config path and WS_DIR env to prevent leaking to the real
+    filesystem.  Most command tests also monkeypatch ``load_config`` directly.
+    """
+    ws_dir = tmp_path / "workstreams"
+    ws_dir.mkdir()
+
+    # Prevent global config reads/writes from hitting the real home dir
+    config_file = tmp_path / "global_config.yaml"
+    monkeypatch.setattr("workstream.config.CONFIG_PATH", config_file)
+    monkeypatch.setattr("workstream.config.CONFIG_DIR", tmp_path)
+    # Prevent env var and cwd walk-up from finding a real project
+    monkeypatch.delenv("WS_DIR", raising=False)
+
+    return Config(workstreams_dir=str(ws_dir))

workstream/test/fixtures/sample_sprint_note.md

@@ -0,0 +1,10 @@
+## 2026-03-26
+- TODO
+  - Fix onboarding bug
+  - Review PR #42
+- Thoughts
+  - Should we add confetti to the welcome screen?
+  - Time to rethink the API versioning strategy
+  - That agent knowledge sharing idea keeps coming up
+- Done
+  - Deployed landing page

workstream/test/fixtures/sample_workstream.md

@@ -0,0 +1,41 @@
+---
+id: 2026-03-15-a1b2c3d4e5
+title: API Redesign
+status: active
+size: week
+tags:
+  - code
+  - backend
+repos:
+  - myapp
+created: '2026-03-15'
+updated: '2026-03-25'
+---
+
+# API Redesign
+#code #backend
+
+## Thread
+### 2026-03-25
+Refactored authentication middleware to use the new token format.
+Endpoint tests all passing now.
+
+### 2026-03-20
+Sketched out the new resource hierarchy. Going with /v2/ prefix
+to avoid breaking existing clients.
+
+## Decisions
+- Use /v2/ prefix for all new endpoints
+- Adopt JSON:API response envelope
+
+## Plans
+<!-- Updated by ws sweep -->
+- myapp `.plans/api-v2-migration.md` (active)
+
+## Branches
+<!-- Updated by ws sweep -->
+- myapp: `feature/api-v2` (+12 ahead)
+
+## Next
+- Write migration guide for consumers
+- Add rate-limiting to /v2/ endpoints

workstream/test/test_backfill.py

@@ -0,0 +1,180 @@
+"""Tests for workstream.commands.backfill — plan ID assignment and workstream inference."""
+
+from __future__ import annotations
+
+from pathlib import Path
+from textwrap import dedent
+
+import pytest
+
+from workstream.commands.backfill import _slug_overlap, _infer_workstream
+from workstream.config import Config, RepoConfig
+from workstream.markdown import parse_frontmatter
+from workstream.models import Workstream
+
+
+# ── Slug overlap ─────────────────────────────────────────────────────
+
+def test_slug_overlap_match():
+    """Overlapping tokens return True."""
+    assert _slug_overlap('api-redesign-migration', 'api-redesign')
+
+
+def test_slug_overlap_no_match():
+    """No overlapping tokens return False."""
+    assert not _slug_overlap('perf-improvements', 'api-redesign')
+
+
+def test_slug_overlap_short_tokens_ignored():
+    """Single-char tokens are ignored."""
+    assert not _slug_overlap('a-b-c', 'a-b-d')
+
+
+# ── Workstream inference ─────────────────────────────────────────────
+
+def test_infer_single_match():
+    """Exactly one matching workstream returns its ID."""
+    ws = Workstream(id='ws-123', title='API Redesign', status='active', repos=['myapp'])
+    assert _infer_workstream('myapp', 'api-redesign-migration', [ws]) == 'ws-123'
+
+
+def test_infer_no_repo_match():
+    """No matching repo returns None."""
+    ws = Workstream(id='ws-123', title='API Redesign', status='active', repos=['otherapp'])
+    assert _infer_workstream('myapp', 'api-redesign', [ws]) is None
+
+
+def test_infer_no_slug_overlap():
+    """Matching repo but no slug overlap returns None."""
+    ws = Workstream(id='ws-123', title='Fundraising', status='active', repos=['myapp'])
+    assert _infer_workstream('myapp', 'api-redesign', [ws]) is None
+
+
+def test_infer_multiple_matches_returns_none():
+    """Multiple matching workstreams returns None (ambiguous)."""
+    ws1 = Workstream(id='ws-1', title='API Redesign', status='active', repos=['myapp'])
+    ws2 = Workstream(id='ws-2', title='API Migration', status='active', repos=['myapp'])
+    assert _infer_workstream('myapp', 'api-migration-redesign', [ws1, ws2]) is None
+
+
+# ── Full backfill handler ────────────────────────────────────────────
+
+def test_backfill_adds_id(tmp_path: Path):
+    """Backfill adds an ID to a plan file that lacks one."""
+    # Set up repo with a plan missing an ID
+    repo = tmp_path / 'myapp'
+    plans_dir = repo / '.plans'
+    plans_dir.mkdir(parents=True)
+
+    plan_text = dedent("""\
+        ---
+        title: My Plan
+        status: active
+        ---
+
+        Plan content here.
+    """)
+    (plans_dir / 'my-plan.md').write_text(plan_text, encoding='utf-8')
+
+    # Set up empty workstreams dir
+    ws_dir = tmp_path / 'workstreams'
+    ws_dir.mkdir()
+
+    config = Config(
+        workstreams_dir=str(ws_dir),
+        repos=[RepoConfig(path=str(repo), name='myapp')],
+    )
+
+
+    from workstream.commands.backfill import _backfill_handler
+    _backfill_handler(config=config, dry_run=False)
+
+    # The original file should be renamed
+    remaining = list(plans_dir.glob('*.md'))
+    assert len(remaining) == 1
+
+    new_file = remaining[0]
+    # Should have the pattern: <id>-my-plan.md
+    assert 'my-plan' in new_file.name
+    assert len(new_file.name) > len('my-plan.md')  # has ID prefix
+
+    # Verify frontmatter has an ID
+    meta, _ = parse_frontmatter(new_file.read_text(encoding='utf-8'))
+    assert 'id' in meta
+    assert meta['id'] in new_file.name
+
+
+def test_backfill_dry_run(tmp_path: Path, capsys):
+    """Dry run shows what would change without modifying files."""
+    repo = tmp_path / 'myapp'
+    plans_dir = repo / '.plans'
+    plans_dir.mkdir(parents=True)
+
+    plan_text = dedent("""\
+        ---
+        title: Another Plan
+        status: active
+        ---
+
+        Plan content.
+    """)
+    original_name = 'another-plan.md'
+    (plans_dir / original_name).write_text(plan_text, encoding='utf-8')
+
+    ws_dir = tmp_path / 'workstreams'
+    ws_dir.mkdir()
+
+    config = Config(
+        workstreams_dir=str(ws_dir),
+        repos=[RepoConfig(path=str(repo), name='myapp')],
+    )
+
+
+    from workstream.commands.backfill import _backfill_handler
+    _backfill_handler(config=config, dry_run=True)
+
+    # Original file should be unchanged
+    assert (plans_dir / original_name).exists()
+    meta, _ = parse_frontmatter((plans_dir / original_name).read_text(encoding='utf-8'))
+    assert 'id' not in meta  # not modified
+
+    out = capsys.readouterr().out
+    assert 'would add id' in out
+    assert 'dry run' in out.lower()
+
+
+def test_backfill_preserves_existing_id(tmp_path: Path):
+    """Plans with existing IDs are not re-IDed."""
+    repo = tmp_path / 'myapp'
+    plans_dir = repo / '.plans'
+    plans_dir.mkdir(parents=True)
+
+    plan_text = dedent("""\
+        ---
+        id: 2026-03-20-existing123
+        title: Existing Plan
+        status: active
+        ---
+
+        Already has an ID.
+    """)
+    (plans_dir / '2026-03-20-existing123-existing-plan.md').write_text(plan_text, encoding='utf-8')
+
+    ws_dir = tmp_path / 'workstreams'
+    ws_dir.mkdir()
+
+    config = Config(
+        workstreams_dir=str(ws_dir),
+        repos=[RepoConfig(path=str(repo), name='myapp')],
+    )
+
+
+    from workstream.commands.backfill import _backfill_handler
+    _backfill_handler(config=config, dry_run=False)
+
+    # File should still exist with same name
+    assert (plans_dir / '2026-03-20-existing123-existing-plan.md').exists()
+    meta, _ = parse_frontmatter(
+        (plans_dir / '2026-03-20-existing123-existing-plan.md').read_text(encoding='utf-8')
+    )
+    assert meta['id'] == '2026-03-20-existing123'

workstream/test/test_batch_writeback.py

@@ -0,0 +1,81 @@
+"""Tests for batch review plan status writeback mechanics."""
+
+from datetime import date
+
+from workstream.markdown import parse_frontmatter, write_frontmatter
+
+
+def test_plan_status_writeback(tmp_path):
+    """Simulate batch review updating a plan file's status."""
+    plan_dir = tmp_path / '.plans'
+    plan_dir.mkdir()
+    plan_file = plan_dir / 'myplan.md'
+    plan_file.write_text(
+        '---\ntitle: My Plan\nstatus: active\nworkstream: test\n---\n\nPlan content.',
+        encoding='utf-8',
+    )
+
+    # Simulate classification
+    cl = {'plan': 'myplan.md', 'status': 'implemented', 'reason': 'Commits match', 'ideas': []}
+    today = date.today().isoformat()
+
+    # Apply the same logic as _batch_review_plans_handler
+    plan_text = plan_file.read_text(encoding='utf-8')
+    plan_meta, plan_body = parse_frontmatter(plan_text)
+    assert plan_meta.get('status') == 'active'
+
+    if plan_meta and plan_meta.get('status') != cl['status']:
+        plan_meta['status'] = cl['status']
+        plan_meta['reviewed'] = today
+        plan_file.write_text(
+            write_frontmatter(plan_meta, plan_body),
+            encoding='utf-8',
+        )
+
+    # Verify
+    updated_text = plan_file.read_text(encoding='utf-8')
+    updated_meta, _ = parse_frontmatter(updated_text)
+    assert updated_meta['status'] == 'implemented'
+    assert updated_meta['reviewed'] == today
+
+
+def test_plan_status_writeback_no_change(tmp_path):
+    """Don't write if status already matches."""
+    plan_dir = tmp_path / '.plans'
+    plan_dir.mkdir()
+    plan_file = plan_dir / 'myplan.md'
+    original = '---\ntitle: My Plan\nstatus: active\n---\n\nContent.'
+    plan_file.write_text(original, encoding='utf-8')
+
+    cl = {'plan': 'myplan.md', 'status': 'active', 'reason': 'Still active', 'ideas': []}
+
+    plan_text = plan_file.read_text(encoding='utf-8')
+    plan_meta, plan_body = parse_frontmatter(plan_text)
+
+    # Same status — should not rewrite
+    if plan_meta and plan_meta.get('status') != cl['status']:
+        plan_meta['status'] = cl['status']
+        plan_file.write_text(write_frontmatter(plan_meta, plan_body), encoding='utf-8')
+
+    # File should be unchanged
+    assert plan_file.read_text(encoding='utf-8') == original
+
+
+def test_plan_status_writeback_no_frontmatter(tmp_path):
+    """Plans without frontmatter should be skipped."""
+    plan_dir = tmp_path / '.plans'
+    plan_dir.mkdir()
+    plan_file = plan_dir / 'legacy.md'
+    original = '<!-- STATUS: FINALIZED -->\n\nLegacy plan.'
+    plan_file.write_text(original, encoding='utf-8')
+
+    plan_text = plan_file.read_text(encoding='utf-8')
+    plan_meta, plan_body = parse_frontmatter(plan_text)
+
+    # No frontmatter — should skip
+    if plan_meta and plan_meta.get('status') != 'implemented':
+        plan_meta['status'] = 'implemented'
+        plan_file.write_text(write_frontmatter(plan_meta, plan_body), encoding='utf-8')
+
+    # File unchanged
+    assert plan_file.read_text(encoding='utf-8') == original