monoco-toolkit 0.2.5__py3-none-any.whl → 0.2.8__py3-none-any.whl

monoco/features/issue/engine/config.py

@@ -0,0 +1,172 @@
+from monoco.core.config import IssueSchemaConfig, IssueTypeConfig, TransitionConfig, StateMachineConfig
+
+DEFAULT_ISSUE_CONFIG = IssueSchemaConfig(
+    types=[
+        IssueTypeConfig(name="epic", label="Epic", prefix="EPIC", folder="Epics"),
+        IssueTypeConfig(name="feature", label="Feature", prefix="FEAT", folder="Features"),
+        IssueTypeConfig(name="chore", label="Chore", prefix="CHORE", folder="Chores"),
+        IssueTypeConfig(name="fix", label="Fix", prefix="FIX", folder="Fixes"),
+    ],
+    statuses=["open", "closed", "backlog"],
+    stages=["draft", "doing", "review", "done", "freezed"],
+    solutions=["implemented", "cancelled", "wontfix", "duplicate"],
+    workflows=[
+        # --- UNIVERSAL AGENT ACTIONS ---
+
+
+        # --- OPEN -> OPEN Transitions (Stage changes) ---
+        TransitionConfig(
+            name="start",
+            label="Start",
+            icon="$(play)",
+            from_status="open",
+            from_stage="draft",
+            to_status="open",
+            to_stage="doing",
+            command_template="monoco issue start {id}",
+            description="Start working on the issue"
+        ),
+        
+
+
+        TransitionConfig(
+            name="stop",
+            label="Stop",
+            icon="$(stop)",
+            from_status="open",
+            from_stage="doing",
+            to_status="open",
+            to_stage="draft",
+            command_template="monoco issue stop {id}",
+            description="Stop working and return to draft"
+        ),
+        TransitionConfig(
+            name="submit",
+            label="Submit",
+            icon="$(check)",
+            from_status="open",
+            from_stage="doing",
+            to_status="open",
+            to_stage="review",
+            command_template="monoco issue submit {id}",
+            description="Submit for review"
+        ),
+        TransitionConfig(
+            name="reject",
+            label="Reject",
+            icon="$(error)",
+            from_status="open",
+            from_stage="review",
+            to_status="open",
+            to_stage="doing",
+            command_template="monoco issue update {id} --stage doing",
+            description="Reject review and return to doing"
+        ),
+
+        # --- OPEN -> CLOSED Transitions ---
+        TransitionConfig(
+            name="accept",
+            label="Accept",
+            icon="$(pass-filled)",
+            from_status="open",
+            from_stage="review",
+            to_status="closed",
+            to_stage="done",
+            required_solution="implemented",
+            command_template="monoco issue close {id} --solution implemented",
+            description="Accept and close issue"
+        ),
+        TransitionConfig(
+            name="close_done",
+            label="Close",
+            icon="$(close)",
+            from_status="open",
+            from_stage="done",
+            to_status="closed",
+            to_stage="done",
+            required_solution="implemented",
+            command_template="monoco issue close {id} --solution implemented",
+            description="Close completed issue"
+        ),
+        TransitionConfig(
+            name="cancel",
+            label="Cancel",
+            icon="$(trash)",
+            from_status="open",
+            # Allowed from any stage except DONE (though core.py had a check for it)
+            to_status="closed",
+            to_stage="done",
+            required_solution="cancelled",
+            command_template="monoco issue cancel {id}",
+            description="Cancel the issue"
+        ),
+        TransitionConfig(
+            name="wontfix",
+            label="Won't Fix",
+            icon="$(circle-slash)",
+            from_status="open",
+            to_status="closed",
+            to_stage="done",
+            required_solution="wontfix",
+            command_template="monoco issue close {id} --solution wontfix",
+            description="Mark as won't fix"
+        ),
+
+        # --- BACKLOG Transitions ---
+        TransitionConfig(
+            name="push",
+            label="Push to Backlog",
+            icon="$(archive)",
+            from_status="open",
+            to_status="backlog",
+            to_stage="freezed",
+            command_template="monoco issue backlog push {id}",
+            description="Move issue to backlog"
+        ),
+
+        TransitionConfig(
+            name="pull",
+            label="Pull",
+            icon="$(arrow-up)",
+            from_status="backlog",
+            to_status="open",
+            to_stage="draft",
+            command_template="monoco issue backlog pull {id}",
+            description="Restore issue from backlog"
+        ),
+        TransitionConfig(
+            name="cancel_backlog",
+            label="Cancel",
+            icon="$(trash)",
+            from_status="backlog",
+            to_status="closed",
+            to_stage="done",
+            required_solution="cancelled",
+            command_template="monoco issue cancel {id}",
+            description="Cancel backlog issue"
+        ),
+
+        # --- CLOSED Transitions ---
+        TransitionConfig(
+            name="reopen",
+            label="Reopen",
+            icon="$(refresh)",
+            from_status="closed",
+            to_status="open",
+            to_stage="draft",
+            command_template="monoco issue open {id}",
+            description="Reopen a closed issue"
+        ),
+        TransitionConfig(
+            name="reopen_from_done",
+            label="Reopen",
+            icon="$(refresh)",
+            from_status="open",
+            from_stage="done",
+            to_status="open",
+            to_stage="draft",
+            command_template="monoco issue open {id}",
+            description="Reopen a done issue"
+        ),
+    ]
+)

monoco/features/issue/engine/machine.py

@@ -0,0 +1,185 @@
+from typing import List, Optional, Dict
+from monoco.core.config import IssueSchemaConfig, TransitionConfig
+from ..models import IssueStatus, IssueStage, IssueMetadata, IssueSolution, IssueType
+
+class StateMachine:
+    def __init__(self, config: IssueSchemaConfig):
+        self.issue_config = config
+        self.transitions = config.workflows or []
+
+    def get_type_config(self, type_name: str):
+        if not self.issue_config.types:
+            return None
+        for t in self.issue_config.types:
+            if t.name == type_name:
+                return t
+        return None
+
+    def get_prefix_map(self) -> Dict[str, str]:
+        if not self.issue_config.types:
+            return {}
+        return {t.name: t.prefix for t in self.issue_config.types}
+
+    def get_folder_map(self) -> Dict[str, str]:
+        if not self.issue_config.types:
+            return {}
+        return {t.name: t.folder for t in self.issue_config.types}
+
+    def get_all_types(self) -> List[str]:
+        if not self.issue_config.types:
+            return []
+        return [t.name for t in self.issue_config.types]
+
+    def can_transition(self, current_status: str, current_stage: Optional[str], 
+                       target_status: str, target_stage: Optional[str]) -> bool:
+        """Check if a transition path exists."""
+        for t in self.transitions:
+            if t.from_status and t.from_status != current_status:
+                continue
+            if t.from_stage and t.from_stage != current_stage:
+                continue
+            
+            if t.to_status == target_status:
+                if target_stage is None or t.to_stage == target_stage:
+                    return True
+        return False
+
+    def get_available_transitions(self, meta: IssueMetadata) -> List[TransitionConfig]:
+        """Get all transitions allowed from the current state of the issue."""
+        allowed = []
+        for t in self.transitions:
+            # Universal actions (no from_status/stage) are always allowed
+            if t.from_status is None and t.from_stage is None:
+                allowed.append(t)
+                continue
+
+            # Match status
+            if t.from_status and t.from_status != meta.status:
+                continue
+            
+            # Match stage
+            if t.from_stage and t.from_stage != meta.stage:
+                continue
+            
+            # Special case for 'Cancel': don't show if already DONE or CLOSED
+            if t.name == "cancel" and meta.stage == "done":
+                continue
+
+            allowed.append(t)
+        return allowed
+
+    def find_transition(self, from_status: str, from_stage: Optional[str],
+                        to_status: str, to_stage: Optional[str],
+                        solution: Optional[str] = None) -> Optional[TransitionConfig]:
+        """Find a specific transition rule."""
+        candidates = []
+        for t in self.transitions:
+            # Skip non-transitions (agent actions with same status/stage)
+            if t.from_status is None and t.from_stage is None:
+                continue
+
+            if t.from_status and t.from_status != from_status:
+                continue
+            if t.from_stage and t.from_stage != from_stage:
+                continue
+            
+            # Check if this transition matches the target
+            if t.to_status == to_status:
+                if to_stage is None or t.to_stage == to_stage:
+                    candidates.append(t)
+        
+        if not candidates:
+            return None
+        
+        # If we have a solution, find the transition that requires it
+        if solution:
+            for t in candidates:
+                if t.required_solution == solution:
+                    return t
+            # If solution provided but none of the transitions match it,
+            # we should return None (unless there is a transition with NO required_solution)
+            for t in candidates:
+                if t.required_solution is None:
+                    return t
+            return None
+        
+        # Otherwise return the first one that has NO required_solution
+        for t in candidates:
+            if t.required_solution is None:
+                return t
+        
+        return candidates[0]
+
+    def validate_transition(self, from_status: str, from_stage: Optional[str],
+                            to_status: str, to_stage: Optional[str],
+                            solution: Optional[str] = None) -> None:
+        """
+        Validate if a transition is allowed. Raises ValueError if not.
+        """
+        if from_status == to_status and from_stage == to_stage:
+            return # No change is always allowed (unless we want to enforce specific updates)
+
+        transition = self.find_transition(from_status, from_stage, to_status, to_stage, solution)
+        
+        if not transition:
+            raise ValueError(f"Lifecycle Policy: Transition from {from_status}({from_stage if from_stage else 'None'}) "
+                             f"to {to_status}({to_stage if to_stage else 'None'}) is not defined.")
+
+        if transition.required_solution and solution != transition.required_solution:
+             raise ValueError(f"Lifecycle Policy: Transition '{transition.label}' requires solution '{transition.required_solution}'.")
+
+    def enforce_policy(self, meta: IssueMetadata) -> None:
+        """
+        Apply consistency rules to IssueMetadata.
+        """
+        from ..models import current_time
+        
+        if meta.status == "backlog":
+            meta.stage = "freezed"
+        
+        elif meta.status == "closed":
+            if meta.stage != "done":
+                meta.stage = "done"
+            if not meta.closed_at:
+                meta.closed_at = current_time()
+        
+        elif meta.status == "open":
+            if meta.stage is None:
+                meta.stage = "draft"
+
+    def validate_transition(self, from_status: str, from_stage: Optional[str],
+                            to_status: str, to_stage: Optional[str],
+                            solution: Optional[str] = None) -> None:
+        """
+        Validate if a transition is allowed. Raises ValueError if not.
+        """
+        if from_status == to_status and from_stage == to_stage:
+            return # No change is always allowed (unless we want to enforce specific updates)
+
+        transition = self.find_transition(from_status, from_stage, to_status, to_stage, solution)
+        
+        if not transition:
+            raise ValueError(f"Lifecycle Policy: Transition from {from_status}({from_stage if from_stage else 'None'}) "
+                             f"to {to_status}({to_stage if to_stage else 'None'}) is not defined.")
+
+        if transition.required_solution and solution != transition.required_solution:
+             raise ValueError(f"Lifecycle Policy: Transition '{transition.label}' requires solution '{transition.required_solution}'.")
+
+    def enforce_policy(self, meta: IssueMetadata) -> None:
+        """
+        Apply consistency rules to IssueMetadata.
+        """
+        from ..models import current_time
+        
+        if meta.status == "backlog":
+            meta.stage = "freezed"
+        
+        elif meta.status == "closed":
+            if meta.stage != "done":
+                meta.stage = "done"
+            if not meta.closed_at:
+                meta.closed_at = current_time()
+        
+        elif meta.status == "open":
+            if meta.stage is None:
+                meta.stage = "draft"

monoco/features/issue/engine/models.py

@@ -0,0 +1,18 @@
+from typing import List, Optional, Any
+from pydantic import BaseModel
+
+class Transition(BaseModel):
+    name: str
+    label: str
+    icon: Optional[str] = None
+    from_status: Optional[str] = None  # None means any
+    from_stage: Optional[str] = None    # None means any
+    to_status: str
+    to_stage: Optional[str] = None
+    required_solution: Optional[str] = None
+    description: str = ""
+    command_template: Optional[str] = None
+
+class StateMachineConfig(BaseModel):
+    transitions: List[Transition]
+    # We can add more config like default stages for statuses etc.

monoco/features/issue/linter.py

@@ -4,13 +4,40 @@ from rich.console import Console
 from rich.table import Table
 import typer
 import re
-
+from monoco.core import git
 from . import core
 from .validator import IssueValidator
 from monoco.core.lsp import Diagnostic, DiagnosticSeverity
 
 console = Console()
 
+def check_environment_policy(project_root: Path):
+    """
+    Guardrail: Prevent direct modifications on protected branches (main/master).
+    """
+    # Only enforce if it is a git repo
+    try:
+        if not git.is_git_repo(project_root):
+            return
+
+        current_branch = git.get_current_branch(project_root)
+        # Standard protected branches
+        if current_branch in ["main", "master", "production"]:
+            # Check if dirty (uncommitted changes)
+            changed_files = git.get_git_status(project_root)
+            if changed_files:
+                console.print(f"\n[bold red]🛑 Environment Policy Violation[/bold red]")
+                console.print(f"You are modifying code directly on protected branch: [bold cyan]{current_branch}[/bold cyan]")
+                console.print(f"Found {len(changed_files)} uncommitted changes.")
+                console.print(f"[yellow]Action Required:[/yellow] Please stash your changes and switch to a feature branch.")
+                console.print(f"  > git stash")
+                console.print(f"  > monoco issue start <ID> --branch")
+                console.print(f"  > git stash pop")
+                raise typer.Exit(code=1)
+    except Exception:
+        # Fail safe: Do not block linting if git check fails unexpectedly
+        pass
+
 def check_integrity(issues_root: Path, recursive: bool = False) -> List[Diagnostic]:
     """
     Verify the integrity of the Issues directory using LSP Validator.
@@ -38,27 +65,48 @@ def check_integrity(issues_root: Path, recursive: bool = False) -> List[Diagnost
                     meta = core.parse_issue(f)
                     if meta:
                         local_id = meta.id
-                        full_id = f"{project_name}::{local_id}" if project_name != "local" else local_id
+                        full_id = f"{project_name}::{local_id}"
                         
                         all_issue_ids.add(local_id)
-                        if project_name != "local":
-                            all_issue_ids.add(full_id)
+                        all_issue_ids.add(full_id)
                         
                         project_issues.append((f, meta))
         return project_issues
 
-    all_issues.extend(collect_project_issues(issues_root, "local"))
+    from monoco.core.config import get_config
+    conf = get_config(str(issues_root.parent))
+    
+    # Identify local project name
+    local_project_name = "local"
+    if conf and conf.project and conf.project.name:
+        local_project_name = conf.project.name.lower()
+
+    # Find Topmost Workspace Root
+    workspace_root = issues_root.parent
+    for parent in [workspace_root] + list(workspace_root.parents):
+        if (parent / ".monoco" / "workspace.yaml").exists() or (parent / ".monoco" / "project.yaml").exists():
+            workspace_root = parent
+            
+    # Collect from local issues_root
+    all_issues.extend(collect_project_issues(issues_root, local_project_name))
     
     if recursive:
         try:
-            from monoco.core.config import get_config
-            project_root = issues_root.parent
-            conf = get_config(str(project_root))
-            for member_name, rel_path in conf.project.members.items():
-                member_root = (project_root / rel_path).resolve()
+            # Re-read config from workspace root to get all members
+            ws_conf = get_config(str(workspace_root))
+            
+            # Index Root project if different from current
+            if workspace_root != issues_root.parent:
+                root_issues_dir = workspace_root / "Issues"
+                if root_issues_dir.exists():
+                    all_issues.extend(collect_project_issues(root_issues_dir, ws_conf.project.name.lower()))
+
+            # Index all members
+            for member_name, rel_path in ws_conf.project.members.items():
+                member_root = (workspace_root / rel_path).resolve()
                 member_issues_dir = member_root / "Issues"
-                if member_issues_dir.exists():
-                    collect_project_issues(member_issues_dir, member_name)
+                if member_issues_dir.exists() and member_issues_dir != issues_root:
+                    all_issues.extend(collect_project_issues(member_issues_dir, member_name.lower()))
         except Exception:
             pass
 
@@ -89,6 +137,10 @@ def run_lint(issues_root: Path, recursive: bool = False, fix: bool = False, form
         format: Output format (table, json)
         file_path: Optional path to a single file to validate (LSP mode)
     """
+    # 0. Environment Policy Check (Guardrail)
+    # We assume issues_root.parent is the project root or close enough for git context
+    check_environment_policy(issues_root.parent)
+
     # Single-file mode (for LSP integration)
     if file_path:
         file = Path(file_path).resolve()
@@ -210,6 +262,56 @@ def run_lint(issues_root: Path, recursive: bool = False, fix: bool = False, form
                              new_content = new_content.rstrip() + "\n\n## Review Comments\n\n- [ ] Self-Review\n"
                              has_changes = True
 
+                    if "Malformed ID" in d.message:
+                        lines = new_content.splitlines()
+                        if d.range and d.range.start.line < len(lines):
+                            line_idx = d.range.start.line
+                            line = lines[line_idx]
+                            # Remove # from quoted strings or raw values
+                            new_line = line.replace("'#", "'").replace('"#', '"')
+                            if new_line != line:
+                                lines[line_idx] = new_line
+                                new_content = "\n".join(lines) + "\n"
+                                has_changes = True
+                    
+                    if "Tag Check: Missing required context tags" in d.message:
+                         # Extract missing tags from message
+                         # Message format: "Tag Check: Missing required context tags: #TAG1, #TAG2"
+                         try:
+                             parts = d.message.split(": ")
+                             if len(parts) >= 3:
+                                 tags_str = parts[-1]
+                                 missing_tags = [t.strip() for t in tags_str.split(",")]
+                                 
+                                 # We need to update content via core.update_issue logic effectively
+                                 # But we are in a loop potentially with other string edits.
+                                 # IMPORTANT: Mixed strategy (Regex vs Object Update) is risky.
+                                 # However, tags are in YAML frontmatter.
+                                 # Since we might have modified new_content already (string), using core.update_issue on file is dangerous (race condition with memory).
+                                 # Better to append to tags list in YAML via regex or yaml parser on new_content.
+                                 
+                                 # Parsing Frontmatter from new_content
+                                 fm_match = re.search(r"^---(.*?)---", new_content, re.DOTALL | re.MULTILINE)
+                                 if fm_match:
+                                      import yaml
+                                      fm_text = fm_match.group(1)
+                                      data = yaml.safe_load(fm_text) or {}
+                                      current_tags = data.get('tags', [])
+                                      if not isinstance(current_tags, list): current_tags = []
+                                      
+                                      # Add missing
+                                      updated_tags = sorted(list(set(current_tags) | set(missing_tags)))
+                                      data['tags'] = updated_tags
+                                      
+                                      # Dump back
+                                      new_fm_text = yaml.dump(data, sort_keys=False, allow_unicode=True)
+                                      
+                                      # Replace FM block
+                                      new_content = new_content.replace(fm_match.group(1), "\n" + new_fm_text)
+                                      has_changes = True
+                         except Exception as ex:
+                             console.print(f"[red]Failed to fix tags: {ex}[/red]")
+
                 if has_changes:
                     path.write_text(new_content)
                     fixed_count += 1
@@ -267,5 +369,9 @@ def run_lint(issues_root: Path, recursive: bool = False, fix: bool = False, form
         console.print(table)
         
         if any(d.severity == DiagnosticSeverity.Error for d in issues):
+            console.print("\n[yellow]Tip: Run 'monoco issue lint --fix' to attempt automatic repairs.[/yellow]")
             raise typer.Exit(code=1)
+        
+        if issues:
+             console.print("\n[yellow]Tip: Run 'monoco issue lint --fix' to attempt automatic repairs.[/yellow]")
 

monoco/features/issue/lsp/__init__.py

@@ -0,0 +1,3 @@
+from .definition import DefinitionProvider
+
+__all__ = ["DefinitionProvider"]

monoco/features/issue/lsp/definition.py

@@ -0,0 +1,72 @@
+from pathlib import Path
+from typing import Optional, List
+from monoco.core.lsp import Location, Position, Range
+from ..domain.parser import MarkdownParser
+from ..domain.workspace import WorkspaceSymbolIndex, IssueLocation
+
+class DefinitionProvider:
+    def __init__(self, workspace_root: Path):
+        self.workspace_root = workspace_root
+        self.index = WorkspaceSymbolIndex(workspace_root)
+        # Lazy indexing handled by the index class itself
+
+    def provide_definition(self, file_path: Path, position: Position) -> List[Location]:
+        """
+        Resolve definition at the given position in the file.
+        """
+        if not file_path.exists():
+            return []
+
+        content = file_path.read_text()
+        
+        # 1. Parse the document to find spans
+        # We only need to find the span at the specific line
+        issue = MarkdownParser.parse(content, path=str(file_path))
+        
+        target_span = None
+        for block in issue.body.blocks:
+            # Check if position is within block
+             # Note: block.line_start is inclusive, line_end is exclusive for content
+            if block.line_start <= position.line < block.line_end:
+                 for span in block.spans:
+                     if span.range.start.line == position.line:
+                         # Check character range
+                         if span.range.start.character <= position.character <= span.range.end.character:
+                             target_span = span
+                             break
+            if target_span:
+                break
+        
+        if not target_span:
+            return []
+
+        # 2. Resolve based on span type
+        if target_span.type in ["wikilink", "issue_id"]:
+            issue_id = target_span.metadata.get("issue_id")
+            if issue_id:
+                # Resolve using Workspace Index
+                location = self.index.resolve(issue_id, context_project=self._get_context_project(file_path))
+                if location:
+                    return [
+                        Location(
+                            uri=f"file://{location.file_path}",
+                            range=Range(
+                                start=Position(line=0, character=0),
+                                end=Position(line=0, character=0)
+                            )
+                        )
+                    ]
+        
+        return []
+
+    def _get_context_project(self, file_path: Path) -> Optional[str]:
+        # Simple heuristic: look for parent directory name if it's a known project structure?
+        # Or rely on configuration. 
+        # For now, let's assume the index handles context if passed, or we pass None.
+        # Actually resolving context project from file path is tricky without config loaded for that specific root.
+        # Let's try to deduce from path relative to workspace root.
+        try:
+             rel = file_path.relative_to(self.workspace_root)
+             return rel.parts[0] # First dir is likely project name in a workspace
+        except ValueError:
+             return "local"