@pennyfarthing/core 10.0.2 → 10.0.3

package/pennyfarthing_scripts/sprint/status.py

@@ -0,0 +1,122 @@
+"""
+Sprint status operations.
+
+Provides functions for getting and displaying sprint status.
+"""
+
+from typing import Any
+
+from pennyfarthing_scripts.sprint.loader import get_all_stories, get_sprint_info, load_sprint
+
+
+def get_sprint_status(filter_status: str | None = None) -> dict[str, Any]:
+    """Get sprint status with story counts.
+
+    Args:
+        filter_status: Optional status to filter by
+
+    Returns:
+        Dict with sprint status information
+    """
+    sprint_info = get_sprint_info()
+    stories = get_all_stories()
+
+    if not stories:
+        return {}
+
+    # Count by status
+    status_counts: dict[str, int] = {}
+    total_points = 0
+    completed_points = 0
+
+    for story in stories:
+        status = story.get("status", "backlog")
+        points = story.get("points", 0) or 0
+
+        status_counts[status] = status_counts.get(status, 0) + 1
+        total_points += points
+
+        if status in ("done", "completed"):
+            completed_points += points
+
+    return {
+        "sprint": sprint_info,
+        "total_stories": len(stories),
+        "by_status": status_counts,
+        "backlog": status_counts.get("backlog", 0),
+        "in_progress": status_counts.get("in_progress", 0),
+        "review": status_counts.get("review", 0),
+        "completed": status_counts.get("done", 0) + status_counts.get("completed", 0),
+        "total_points": total_points,
+        "completed_points": completed_points,
+    }
+
+
+def format_status(status: dict[str, Any]) -> str:
+    """Format sprint status as human-readable string.
+
+    Args:
+        status: Status dict from get_sprint_status
+
+    Returns:
+        Formatted status string
+    """
+    if not status:
+        return "No sprint data available"
+
+    lines = []
+
+    # Sprint header
+    sprint = status.get("sprint", {})
+    if sprint:
+        lines.append(f"Sprint: {sprint.get('name', 'Unknown')}")
+        lines.append(f"Status: {sprint.get('status', 'unknown')}")
+        if sprint.get("goal"):
+            lines.append(f"Goal: {sprint['goal']}")
+        lines.append("")
+
+    # Story counts
+    lines.append(f"Total Stories: {status.get('total_stories', 0)}")
+    lines.append(f"  Backlog: {status.get('backlog', 0)}")
+    lines.append(f"  In Progress: {status.get('in_progress', 0)}")
+    lines.append(f"  In Review: {status.get('review', 0)}")
+    lines.append(f"  Completed: {status.get('completed', 0)}")
+    lines.append("")
+
+    # Points
+    lines.append(f"Points: {status.get('completed_points', 0)}/{status.get('total_points', 0)}")
+
+    return "\n".join(lines)
+
+
+def main(args: list[str] | None = None) -> int:
+    """CLI entry point for sprint status.
+
+    Args:
+        args: Command line arguments
+
+    Returns:
+        Exit code
+    """
+    import argparse
+    import sys
+
+    parser = argparse.ArgumentParser(description="Show sprint status")
+    parser.add_argument(
+        "filter",
+        nargs="?",
+        choices=["backlog", "todo", "in-progress", "review", "done"],
+        help="Filter by status",
+    )
+
+    parsed = parser.parse_args(args)
+
+    status = get_sprint_status(parsed.filter)
+    print(format_status(status))
+
+    return 0
+
+
+if __name__ == "__main__":
+    import sys
+    sys.exit(main())

package/pennyfarthing_scripts/sprint/story_add.py

@@ -0,0 +1,187 @@
+"""Sprint story add command.
+
+Story: MSSCI-14256 - Sprint story add command
+
+This module provides:
+- generate_story_id(sprint_data, epic) -> str
+- add_story(sprint_path, epic_id, title, points, ...) -> dict
+- story_add_command (Click command for CLI registration)
+"""
+
+from pathlib import Path
+from typing import Any
+
+import click
+from ruamel.yaml.comments import CommentedMap
+
+from pennyfarthing_scripts.sprint.loader import find_epic
+from pennyfarthing_scripts.sprint.validator import validate_full_sprint
+from pennyfarthing_scripts.sprint.yaml_io import (
+    STORY_KEY_ORDER,
+    read_sprint,
+    write_sprint,
+)
+
+
+def _extract_epic_num(epic: Any) -> str:
+    """Extract the numeric part from an epic ID like 'epic-76' -> '76'."""
+    epic_id = str(epic.get("id", ""))
+    return epic_id.replace("epic-", "")
+
+
+def generate_story_id(sprint_data: Any, epic: Any) -> str:
+    """Generate the next story ID for an epic.
+
+    Args:
+        sprint_data: Full sprint YAML data
+        epic: The target epic dict/CommentedMap
+
+    Returns:
+        Next story ID string (e.g., "76-3")
+    """
+    epic_num = _extract_epic_num(epic)
+    stories = epic.get("stories", [])
+
+    max_seq = 0
+    for story in stories:
+        story_id = str(story.get("id", ""))
+        parts = story_id.split("-")
+        if len(parts) >= 2:
+            try:
+                seq = int(parts[-1])
+                if seq > max_seq:
+                    max_seq = seq
+            except ValueError:
+                pass
+
+    return f"{epic_num}-{max_seq + 1}"
+
+
+def add_story(
+    sprint_path: Path,
+    epic_id: str,
+    title: str,
+    points: int,
+    *,
+    story_type: str | None = None,
+    priority: str = "P1",
+    workflow: str = "tdd",
+    jira: str | None = None,
+) -> dict[str, Any]:
+    """Add a new story to an epic in the sprint YAML.
+
+    Args:
+        sprint_path: Path to sprint YAML file
+        epic_id: Epic ID to add the story to
+        title: Story title
+        points: Story points
+        story_type: Optional story type (feature, bug, chore, refactor)
+        priority: Priority (default: P1)
+        workflow: Workflow (default: tdd)
+        jira: Optional Jira key
+
+    Returns:
+        Dict with success status and story_id or error
+    """
+    data = read_sprint(sprint_path)
+
+    epic = find_epic(data, epic_id)
+    if epic is None:
+        available = []
+        for e in data.get("epics", []):
+            available.append(str(e.get("id", "")))
+        return {
+            "success": False,
+            "error": f"Epic '{epic_id}' not found. Available epics: {', '.join(available)}",
+        }
+
+    story_id = generate_story_id(data, epic)
+
+    # Build story as CommentedMap with canonical key ordering
+    story = CommentedMap()
+    fields: dict[str, Any] = {
+        "id": story_id,
+        "title": title,
+        "points": points,
+        "priority": priority,
+        "status": "backlog",
+        "workflow": workflow,
+    }
+    if jira is not None:
+        fields["jira"] = jira
+    if story_type is not None:
+        fields["type"] = story_type
+
+    # Insert keys in STORY_KEY_ORDER, then any extras
+    for key in STORY_KEY_ORDER:
+        if key in fields:
+            story[key] = fields[key]
+    for key in fields:
+        if key not in STORY_KEY_ORDER:
+            story[key] = fields[key]
+
+    # Append to epic's stories list
+    if "stories" not in epic:
+        from ruamel.yaml.comments import CommentedSeq
+        epic["stories"] = CommentedSeq()
+    epic["stories"].append(story)
+
+    # Validate before writing
+    result = validate_full_sprint(data)
+    if not result.valid:
+        # Remove the story we just added to avoid corrupting data
+        epic["stories"].pop()
+        return {
+            "success": False,
+            "error": f"Validation failed after insertion: {result.errors}",
+        }
+
+    write_sprint(sprint_path, data)
+
+    return {
+        "success": True,
+        "story_id": story_id,
+    }
+
+
+@click.command("add")
+@click.argument("epic_id", type=str)
+@click.argument("title", type=str)
+@click.argument("points", type=int)
+@click.option("--type", "story_type", type=click.Choice(["feature", "bug", "chore", "refactor"]), default="feature")
+@click.option("--priority", type=click.Choice(["P0", "P1", "P2", "P3"]), default="P1")
+@click.option("--workflow", type=click.Choice(["tdd", "trivial", "bdd"]), default="tdd")
+@click.option("--jira", "jira_id", type=str, default=None)
+@click.option("--sprint-file", type=click.Path(), default=None, help="Path to sprint YAML file")
+def story_add_command(
+    epic_id: str,
+    title: str,
+    points: int,
+    story_type: str,
+    priority: str,
+    workflow: str,
+    jira_id: str | None,
+    sprint_file: str | None,
+) -> None:
+    """Add a new story to an epic."""
+    if sprint_file is None:
+        from pennyfarthing_scripts.common.config import get_project_root
+        path = get_project_root() / "sprint" / "current-sprint.yaml"
+    else:
+        path = Path(sprint_file)
+
+    result = add_story(
+        sprint_path=path,
+        epic_id=epic_id,
+        title=title,
+        points=points,
+        story_type=story_type if story_type != "feature" else None,
+        priority=priority,
+        workflow=workflow,
+        jira=jira_id,
+    )
+
+    if result["success"]:
+        click.echo(f"Added story {result['story_id']}: {title} [{points}pts]")
+    else:
+        raise click.ClickException(result["error"])

package/pennyfarthing_scripts/sprint/story_update.py

@@ -0,0 +1,181 @@
+"""Sprint story update command.
+
+Story: MSSCI-14257 - Sprint story update command
+
+This module provides:
+- update_story(sprint_path, story_id, ...) -> dict
+- story_update_command (Click command for CLI registration)
+"""
+
+from datetime import date
+from pathlib import Path
+from typing import Any
+
+import click
+
+from pennyfarthing_scripts.sprint.loader import find_epic, find_story
+from pennyfarthing_scripts.sprint.validator import VALID_STORY_STATUSES, validate_full_sprint
+from pennyfarthing_scripts.sprint.yaml_io import read_sprint, write_sprint
+
+
+def update_story(
+    sprint_path: Path,
+    story_id: str,
+    *,
+    status: str | None = None,
+    points: int | None = None,
+    priority: str | None = None,
+    assigned_to: str | None = None,
+    completed_date: str | None = None,
+    started_date: str | None = None,
+    dry_run: bool = False,
+) -> dict[str, Any]:
+    """Update fields on a story in the sprint YAML.
+
+    Args:
+        sprint_path: Path to sprint YAML file
+        story_id: Story ID (e.g., "76-4")
+        status: New status value
+        points: New points value
+        priority: New priority value
+        assigned_to: New assignee
+        completed_date: Completed date (ISO format)
+        started_date: Started date (ISO format)
+        dry_run: If True, report changes without writing
+
+    Returns:
+        Dict with success status and optional error
+    """
+    # Validate status before reading file
+    if status is not None and status not in VALID_STORY_STATUSES:
+        return {
+            "success": False,
+            "error": f"Invalid status '{status}'. Must be one of: {', '.join(sorted(VALID_STORY_STATUSES))}",
+        }
+
+    # Parse story ID to find epic
+    parts = story_id.split("-")
+    if len(parts) < 2:
+        return {
+            "success": False,
+            "error": f"Invalid story ID format '{story_id}'. Expected format: <epic>-<seq> (e.g., '76-4')",
+        }
+
+    epic_num = parts[0]
+
+    data = read_sprint(sprint_path)
+
+    epic = find_epic(data, epic_num)
+    if epic is None:
+        return {
+            "success": False,
+            "error": f"Epic '{epic_num}' not found for story '{story_id}'",
+        }
+
+    story = find_story(epic, story_id)
+    if story is None:
+        return {
+            "success": False,
+            "error": f"Story '{story_id}' not found in epic '{epic_num}'",
+        }
+
+    # Apply field updates
+    if status is not None:
+        story["status"] = status
+    if points is not None:
+        story["points"] = points
+    if priority is not None:
+        story["priority"] = priority
+    if assigned_to is not None:
+        story["assigned_to"] = assigned_to
+    if completed_date is not None:
+        story["completed"] = completed_date
+    if started_date is not None:
+        story["started"] = started_date
+
+    # Auto-cleanup rules
+    if status == "done":
+        # Remove assigned_to
+        if "assigned_to" in story:
+            del story["assigned_to"]
+        # Auto-set completed if not explicitly provided
+        if completed_date is None and "completed" not in story:
+            story["completed"] = date.today().isoformat()
+        elif completed_date is not None:
+            story["completed"] = completed_date
+
+    if status == "in_progress":
+        # Auto-set started if not already present
+        if "started" not in story:
+            story["started"] = date.today().isoformat()
+
+    # Validate after mutation
+    result = validate_full_sprint(data)
+    if not result.valid:
+        return {
+            "success": False,
+            "error": f"Validation failed after update: {result.errors}",
+        }
+
+    if dry_run:
+        return {
+            "success": True,
+            "dry_run": True,
+            "story_id": story_id,
+        }
+
+    write_sprint(sprint_path, data)
+
+    return {
+        "success": True,
+        "story_id": story_id,
+    }
+
+
+@click.command("update")
+@click.argument("story_id")
+@click.option("--status", type=click.Choice(["backlog", "ready", "in_progress", "done", "canceled"]))
+@click.option("--completed", "completed_date", default=None)
+@click.option("--assigned-to", default=None)
+@click.option("--points", type=int, default=None)
+@click.option("--priority", default=None)
+@click.option("--started", "started_date", default=None)
+@click.option("--dry-run", is_flag=True)
+@click.option("--sprint-file", type=click.Path(), default=None, help="Path to sprint YAML file")
+def story_update_command(
+    story_id: str,
+    status: str | None,
+    completed_date: str | None,
+    assigned_to: str | None,
+    points: int | None,
+    priority: str | None,
+    started_date: str | None,
+    dry_run: bool,
+    sprint_file: str | None,
+) -> None:
+    """Update a story's fields by ID."""
+    if sprint_file is None:
+        from pennyfarthing_scripts.common.config import get_project_root
+        path = get_project_root() / "sprint" / "current-sprint.yaml"
+    else:
+        path = Path(sprint_file)
+
+    result = update_story(
+        sprint_path=path,
+        story_id=story_id,
+        status=status,
+        points=points,
+        priority=priority,
+        assigned_to=assigned_to,
+        completed_date=completed_date,
+        started_date=started_date,
+        dry_run=dry_run,
+    )
+
+    if result["success"]:
+        if result.get("dry_run"):
+            click.echo(f"[DRY-RUN] Would update story {result['story_id']}")
+        else:
+            click.echo(f"Updated story {result['story_id']}")
+    else:
+        raise click.ClickException(result["error"])