monoco-toolkit 0.3.5__py3-none-any.whl → 0.3.9__py3-none-any.whl

monoco/features/issue/core.py

@@ -15,12 +15,18 @@ from .models import (
     current_time,
     generate_uid,
 )
+from .criticality import (
+    CriticalityLevel,
+    CriticalityTypeMapping,
+    CriticalityInheritanceService,
+)
 from monoco.core import git
 from monoco.core.config import get_config, MonocoConfig
 from monoco.core.lsp import DiagnosticSeverity
 from .validator import IssueValidator
 
 from .engine import get_engine
+from .git_service import IssueGitService
 
 
 def get_prefix_map(issues_root: Path) -> Dict[str, str]:
@@ -137,6 +143,10 @@ def _serialize_metadata(metadata: IssueMetadata) -> str:
         elif k == "parent":
             ordered_data[k] = None
 
+    # Add criticality if present
+    if "criticality" in data:
+        ordered_data["criticality"] = data["criticality"]
+
     # Add remaining
     for k, v in data.items():
         if k not in ordered_data:
@@ -211,6 +221,7 @@ def create_issue_file(
     subdir: Optional[str] = None,
     sprint: Optional[str] = None,
     tags: List[str] = [],
+    criticality: Optional[CriticalityLevel] = None,
 ) -> Tuple[IssueMetadata, Path]:
     # Validation
     for dep_id in dependencies:
@@ -221,6 +232,42 @@ def create_issue_file(
         if not find_issue_path(issues_root, rel_id):
             raise ValueError(f"Related issue {rel_id} not found.")
 
+    # Auto-assign default parent for non-epic types if not provided
+    if issue_type != IssueType.EPIC and not parent:
+        parent = "EPIC-0000"
+
+    # Determine criticality
+    # 1. Use provided criticality if specified
+    # 2. Check parent for inheritance
+    # 3. Apply type-based default
+    effective_criticality = criticality
+
+    # Get issue type string for mapping lookup
+    issue_type_str = (
+        issue_type.value if isinstance(issue_type, IssueType) else str(issue_type)
+    )
+
+    if effective_criticality is None:
+        # Check parent inheritance
+        if parent:
+            parent_path = find_issue_path(issues_root, parent)
+            if parent_path:
+                parent_meta = parse_issue(parent_path)
+                if parent_meta and parent_meta.criticality:
+                    # Child must inherit at least parent's criticality
+                    default_type_criticality = CriticalityTypeMapping.get_default(
+                        issue_type_str
+                    )
+                    effective_criticality = (
+                        CriticalityInheritanceService.resolve_child_criticality(
+                            parent_meta.criticality, default_type_criticality
+                        )
+                    )
+
+        # Fall back to type-based default
+        if effective_criticality is None:
+            effective_criticality = CriticalityTypeMapping.get_default(issue_type_str)
+
     issue_id = find_next_id(issue_type, issues_root)
     base_type_dir = get_issue_dir(issue_type, issues_root)
     target_dir = base_type_dir / status
@@ -264,6 +311,7 @@ def create_issue_file(
         sprint=sprint,
         tags=final_tags,
         opened_at=current_time() if status == IssueStatus.OPEN else None,
+        criticality=effective_criticality,
     )
 
     # Enforce lifecycle policies
@@ -399,6 +447,9 @@ def update_issue(
     related: Optional[List[str]] = None,
     tags: Optional[List[str]] = None,
     files: Optional[List[str]] = None,
+    criticality: Optional[CriticalityLevel] = None,
+    no_commit: bool = False,
+    project_root: Optional[Path] = None,
 ) -> IssueMetadata:
     path = find_issue_path(issues_root, issue_id)
     if not path:
@@ -460,6 +511,9 @@ def update_issue(
         except ValueError:
             pass
 
+    # Reconstruct temporary metadata for policy validation
+    temp_meta = IssueMetadata(**data)
+
     # Use engine to validate the transition
     engine.validate_transition(
         from_status=current_status,
@@ -467,6 +521,7 @@ def update_issue(
         to_status=target_status,
         to_stage=target_stage,
         solution=effective_solution,
+        meta=temp_meta,
     )
 
     if target_status == "closed":
@@ -532,6 +587,23 @@ def update_issue(
     if files is not None:
         data["files"] = files
 
+    # Criticality update (only through escalation workflow)
+    if criticality is not None:
+        current_criticality = data.get("criticality")
+        if current_criticality:
+            current_level = CriticalityLevel(current_criticality)
+            # Only allow escalation (increase), never lowering
+            if criticality > current_level:
+                data["criticality"] = criticality.value
+            elif criticality < current_level:
+                raise ValueError(
+                    f"Cannot lower criticality from {current_level.value} to {criticality.value}. "
+                    "Criticality is immutable and can only be increased through escalation workflow."
+                )
+        else:
+            # Set if not previously set
+            data["criticality"] = criticality.value
+
     # Lifecycle Hooks
     # 1. Opened At: If transitioning to OPEN
     if target_status == IssueStatus.OPEN and current_status != IssueStatus.OPEN:
@@ -614,6 +686,38 @@ def update_issue(
     if updated_meta.parent:
         recalculate_parent(issues_root, updated_meta.parent)
 
+    # Auto-commit issue file changes (FEAT-0115)
+    if not no_commit:
+        # Determine the action type for commit message
+        action = "update"
+        if status and status != current_status:
+            action = status  # "open", "closed", "backlog"
+        elif stage and stage != current_stage:
+            action = stage  # "draft", "doing", "review", "done"
+
+        # Resolve project root if not provided
+        if project_root is None:
+            project_root = issues_root.parent
+
+        # Only auto-commit if we're in a git repo
+        git_service = IssueGitService(project_root)
+        if git_service.is_git_repository():
+            # Determine old path if status changed (file was moved)
+            old_path_for_git = None
+            if status and status != current_status:
+                # The original path before move
+                old_path_for_git = find_issue_path(issues_root, issue_id)
+
+            commit_result = git_service.commit_issue_change(
+                issue_id=issue_id,
+                action=action,
+                issue_file_path=path,
+                old_file_path=old_path_for_git if old_path_for_git != path else None,
+                no_commit=no_commit,
+            )
+            # Attach commit result to metadata for optional inspection
+            updated_meta.commit_result = commit_result
+
     # Update returned metadata with final absolute path
     updated_meta.path = str(path.absolute())
     updated_meta.actions = get_available_actions(updated_meta)