tapps-agents 3.5.40__py3-none-any.whl → 3.5.41__py3-none-any.whl

tapps_agents/__init__.py

@@ -24,8 +24,8 @@ Example:
     ```
 """
 
-__version__: str = "3.5.40"
+__version__: str = "3.5.41"
 
 # Also expose as _version_ for compatibility with some import mechanisms
 # This helps with editable installs where __version__ might not be importable
-_version_: str = "3.5.40"
+_version_: str = "3.5.41"

tapps_agents/cli/commands/task.py

@@ -207,13 +207,21 @@ def _handle_run(args: object) -> None:
             spec = spec.model_copy(update={"status": "done"})
             save_task_spec(spec, root)
             print(f"Task {task_id} completed.")
+        elif final_state.status == "blocked":
+            # BUG-003B fix: Set status to "blocked" when workflow fails
+            spec = spec.model_copy(update={"status": "blocked"})
+            save_task_spec(spec, root)
+            print(f"Workflow blocked: {final_state.error or 'Required step failed'}", file=sys.stderr)
+            sys.exit(1)
         else:
+            # Other statuses (paused, etc.) - set to todo for retry
             spec = spec.model_copy(update={"status": "todo"})
             save_task_spec(spec, root)
             print(f"Workflow ended with status: {final_state.status}", file=sys.stderr)
             sys.exit(1)
     except Exception as e:
-        spec = spec.model_copy(update={"status": "todo"})
+        # BUG-003B fix: Set status to "blocked" on exception
+        spec = spec.model_copy(update={"status": "blocked"})
         save_task_spec(spec, root)
         print(f"Error running workflow: {e}", file=sys.stderr)
         sys.exit(1)

tapps_agents/workflow/cursor_executor.py

@@ -27,6 +27,7 @@ from __future__ import annotations
 import asyncio
 import hashlib
 import os
+import traceback
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
 from dataclasses import asdict
@@ -52,7 +53,7 @@ from .event_bus import FileBasedEventBus
 from .events import EventType, WorkflowEvent
 from .logging_helper import WorkflowLogger
 from .marker_writer import MarkerWriter
-from .models import Artifact, StepExecution, Workflow, WorkflowState, WorkflowStep
+from .models import Artifact, StepExecution, StepResult, Workflow, WorkflowState, WorkflowStep
 from .parallel_executor import ParallelStepExecutor
 from .progress_manager import ProgressUpdateManager
 from .skill_invoker import SkillInvoker
@@ -602,11 +603,74 @@ class CursorWorkflowExecutor:
 
                 # Execute ready steps in parallel
                 running_step_ids.update(step.id for step in ready_steps)
-                
+
+                # Store completed steps with their results for dependency validation (BUG-003B)
+                completed_step_results: dict[str, StepResult] = {}
+
                 async def execute_step_wrapper(step: WorkflowStep) -> dict[str, Any]:
-                    """Wrapper to adapt _execute_step_for_parallel to parallel executor interface."""
-                    artifacts = await self._execute_step_for_parallel(step=step, target_path=target_path)
-                    return artifacts or {}
+                    """Wrapper to adapt _execute_step_for_parallel to parallel executor interface (BUG-003B fix)."""
+                    # Validate dependencies before execution (BUG-003B)
+                    can_execute, skip_reason = self._can_execute_step(step, completed_step_results)
+
+                    if not can_execute:
+                        # Create skipped StepResult
+                        now = datetime.now()
+                        skipped_result = StepResult(
+                            step_id=step.id,
+                            status="skipped",
+                            success=False,
+                            duration=0.0,
+                            started_at=now,
+                            completed_at=now,
+                            skip_reason=skip_reason,
+                            artifacts=[],
+                        )
+                        completed_step_results[step.id] = skipped_result
+
+                        # Print skip message
+                        from ..core.unicode_safe import safe_print
+                        safe_print(f"\n⏭️  Skipping step '{step.id}': {skip_reason}\n")
+
+                        # Return empty artifacts (step was skipped)
+                        return {}
+
+                    # Execute step
+                    step_result = await self._execute_step_for_parallel(step=step, target_path=target_path)
+                    completed_step_results[step.id] = step_result
+
+                    # Check if step failed (BUG-003B)
+                    if not step_result.success:
+                        # Check if step is required
+                        is_required = step.condition == "required"
+
+                        if is_required:
+                            # Halt workflow for required step failure
+                            from ..core.unicode_safe import safe_print
+                            safe_print(
+                                f"\n❌ Workflow halted: Required step '{step.id}' failed\n"
+                                f"Error: {step_result.error}\n"
+                            )
+
+                            # Update workflow status
+                            if self.state:
+                                self.state.status = "blocked"
+                                self.state.error = step_result.error
+
+                        # Raise error to stop execution
+                        raise RuntimeError(step_result.error or "Step failed")
+
+                    # Convert StepResult artifacts (list of names) back to dict format for compatibility
+                    artifacts_dict: dict[str, dict[str, Any]] = {}
+                    for artifact_name in step_result.artifacts:
+                        artifacts_dict[artifact_name] = {
+                            "name": artifact_name,
+                            "path": artifact_name,
+                            "status": "complete",
+                            "created_by": step.id,
+                            "created_at": step_result.completed_at.isoformat(),
+                        }
+
+                    return artifacts_dict
 
                 try:
                     results = await self.parallel_executor.execute_parallel(
@@ -1325,11 +1389,14 @@ class CursorWorkflowExecutor:
 
     async def _execute_step_for_parallel(
         self, step: WorkflowStep, target_path: Path | None
-    ) -> dict[str, dict[str, Any]] | None:
+    ) -> StepResult:
         """
-        Execute a single workflow step using Cursor Skills and return artifacts (for parallel execution).
-        
-        This is similar to _execute_step but returns artifacts instead of updating state.
+        Execute a single workflow step using Cursor Skills and return result (BUG-003B fix).
+
+        This method now returns StepResult with proper error handling:
+        - success=True + artifacts on success
+        - success=False + error details on failure (no exception raised)
+
         State updates (step_execution tracking) are handled by ParallelStepExecutor.
         """
         if not self.state or not self.workflow:
@@ -1356,8 +1423,17 @@ class CursorWorkflowExecutor:
 
         # Handle completion/finalization steps that don't require agent execution
         if agent_name == "orchestrator" and action in ["finalize", "complete"]:
-            # Return empty artifacts for completion steps
-            return {}
+            # Return successful result for completion steps (no artifacts)
+            now = datetime.now()
+            return StepResult(
+                step_id=step.id,
+                status="completed",
+                success=True,
+                duration=0.0,
+                started_at=now,
+                completed_at=now,
+                artifacts=[],
+            )
 
         # Track step start time for duration calculation
         step_started_at = datetime.now()
@@ -1407,16 +1483,12 @@ class CursorWorkflowExecutor:
                     # Worktree is only used for skill invocation fallback
                     created_artifacts_list = await handler.execute(step, action, target_path)
 
-                    # Convert handler artifacts to dict format
-                    artifacts_dict: dict[str, dict[str, Any]] = {}
-                    for art in (created_artifacts_list or []):
-                        artifacts_dict[art["name"]] = art
-
                     # Write success marker
                     step_completed_at = datetime.now()
                     duration = (step_completed_at - step_started_at).total_seconds()
 
                     found_artifact_paths = [art["path"] for art in (created_artifacts_list or [])]
+                    artifact_names = [art["name"] for art in (created_artifacts_list or [])]
 
                     marker_path = self.marker_writer.write_done_marker(
                         workflow_id=self.state.workflow_id,
@@ -1438,7 +1510,16 @@ class CursorWorkflowExecutor:
                             marker_path=str(marker_path),
                         )
 
-                    return artifacts_dict if artifacts_dict else None
+                    # Return successful StepResult (BUG-003B fix)
+                    return StepResult(
+                        step_id=step.id,
+                        status="completed",
+                        success=True,
+                        duration=duration,
+                        started_at=step_started_at,
+                        completed_at=step_completed_at,
+                        artifacts=artifact_names,
+                    )
                 else:
                     # Fall back to SkillInvoker for steps without handlers
                     safe_print(f"\n[EXEC] Executing {agent_name}/{action} via skill...", flush=True)
@@ -1459,19 +1540,12 @@ class CursorWorkflowExecutor:
                         step=step,
                     )
 
-                    # Convert artifacts to dict format
-                    artifacts_dict: dict[str, dict[str, Any]] = {}
+                    # Extract artifact paths and names
                     found_artifact_paths = []
+                    artifact_names = []
                     for artifact in artifacts:
-                        artifacts_dict[artifact.name] = {
-                            "name": artifact.name,
-                            "path": artifact.path,
-                            "status": artifact.status,
-                            "created_by": artifact.created_by,
-                            "created_at": artifact.created_at.isoformat() if artifact.created_at else None,
-                            "metadata": artifact.metadata or {},
-                        }
                         found_artifact_paths.append(artifact.path)
+                        artifact_names.append(artifact.name)
 
                     # Write DONE marker for successful completion
                     step_completed_at = datetime.now()
@@ -1497,8 +1571,17 @@ class CursorWorkflowExecutor:
                             marker_path=str(marker_path),
                         )
 
+                    # Return successful StepResult (BUG-003B fix)
                     # Worktree cleanup is handled by context manager
-                    return artifacts_dict if artifacts_dict else None
+                    return StepResult(
+                        step_id=step.id,
+                        status="completed",
+                        success=True,
+                        duration=duration,
+                        started_at=step_started_at,
+                        completed_at=step_completed_at,
+                        artifacts=artifact_names,
+                    )
                 
             except (TimeoutError, RuntimeError) as e:
                 # Write FAILED marker for timeout or execution errors
@@ -1506,7 +1589,8 @@ class CursorWorkflowExecutor:
                 duration = (step_failed_at - step_started_at).total_seconds()
                 error_type = type(e).__name__
                 error_msg = str(e)
-                
+                error_tb = traceback.format_exc()
+
                 # Try to get completion status if available (for missing artifacts)
                 found_artifact_paths = []
                 try:
@@ -1518,7 +1602,7 @@ class CursorWorkflowExecutor:
                     found_artifact_paths = completion_status.get("found_artifacts", [])
                 except Exception:
                     pass
-                
+
                 marker_path = self.marker_writer.write_failed_marker(
                     workflow_id=self.state.workflow_id,
                     step_id=step.id,
@@ -1537,30 +1621,41 @@ class CursorWorkflowExecutor:
                         "marker_location": f".tapps-agents/workflows/markers/{self.state.workflow_id}/step-{step.id}/FAILED.json",
                     },
                 )
-                
+
                 if self.logger:
                     self.logger.warning(
                         f"FAILED marker written for step {step.id}",
                         marker_path=str(marker_path),
                         error=error_msg,
                     )
-                
+
                 # Include marker location in error message for better troubleshooting
                 from ..core.unicode_safe import safe_print
                 safe_print(
                     f"\n[INFO] Failure marker written to: {marker_path}",
                     flush=True,
                 )
-                
-                # Re-raise the exception
-                raise
+
+                # Return failed StepResult (BUG-003B fix - don't raise)
+                return StepResult(
+                    step_id=step.id,
+                    status="failed",
+                    success=False,
+                    duration=duration,
+                    started_at=step_started_at,
+                    completed_at=step_failed_at,
+                    error=error_msg,
+                    error_traceback=error_tb,
+                    artifacts=[],
+                )
             except Exception as e:
                 # Write FAILED marker for unexpected errors
                 step_failed_at = datetime.now()
                 duration = (step_failed_at - step_started_at).total_seconds()
                 error_type = type(e).__name__
                 error_msg = str(e)
-                
+                error_tb = traceback.format_exc()
+
                 marker_path = self.marker_writer.write_failed_marker(
                     workflow_id=self.state.workflow_id,
                     step_id=step.id,
@@ -1579,7 +1674,7 @@ class CursorWorkflowExecutor:
                         "marker_location": f".tapps-agents/workflows/markers/{self.state.workflow_id}/step-{step.id}/FAILED.json",
                     },
                 )
-                
+
                 if self.logger:
                     self.logger.error(
                         f"FAILED marker written for step {step.id} (unexpected error)",
@@ -1587,9 +1682,19 @@ class CursorWorkflowExecutor:
                         error=error_msg,
                         exc_info=True,
                     )
-                
-                # Re-raise the exception
-                raise
+
+                # Return failed StepResult (BUG-003B fix - don't raise)
+                return StepResult(
+                    step_id=step.id,
+                    status="failed",
+                    success=False,
+                    duration=duration,
+                    started_at=step_started_at,
+                    completed_at=step_failed_at,
+                    error=error_msg,
+                    error_traceback=error_tb,
+                    artifacts=[],
+                )
 
     @asynccontextmanager
     async def _worktree_context(
@@ -1843,6 +1948,42 @@ class CursorWorkflowExecutor:
         finally:
             self.save_state()
 
+    def _can_execute_step(
+        self,
+        step: WorkflowStep,
+        completed_steps: dict[str, StepResult]
+    ) -> tuple[bool, str]:
+        """
+        Check if step can execute based on dependencies (BUG-003B fix).
+
+        Validates that all required dependencies have been executed and succeeded.
+        If any dependency is missing or failed, the step cannot execute.
+
+        Args:
+            step: Step to check
+            completed_steps: Results of previously executed steps
+
+        Returns:
+            (can_execute, skip_reason) tuple:
+            - (True, "") if all dependencies met
+            - (False, reason) if dependencies not met
+
+        Example:
+            can_run, reason = self._can_execute_step(step, completed_steps)
+            if not can_run:
+                # Skip step with reason
+                skip_result = StepResult(status="skipped", skip_reason=reason, ...)
+        """
+        for dep in step.requires or []:
+            if dep not in completed_steps:
+                return False, f"Dependency '{dep}' not executed"
+
+            dep_result = completed_steps[dep]
+            if not dep_result.success:
+                return False, f"Dependency '{dep}' failed: {dep_result.error}"
+
+        return True, ""
+
     def _normalize_action(self, action: str) -> str:
         """
         Normalize action name to use underscores (Python convention).

tapps_agents/workflow/models.py

@@ -5,7 +5,7 @@ Workflow models - Data structures for workflow definitions.
 from dataclasses import dataclass, field
 from datetime import datetime
 from enum import Enum
-from typing import Any
+from typing import Any, Literal
 
 
 class WorkflowType(Enum):
@@ -95,6 +95,43 @@ class StepExecution:
     error: str | None = None
 
 
+@dataclass
+class StepResult:
+    """Result of step execution with proper error handling.
+
+    This class represents the outcome of executing a workflow step,
+    including success/failure status, error details, and artifacts created.
+    Used by CursorExecutor to properly track step outcomes and halt
+    workflows when required steps fail.
+    """
+
+    step_id: str
+    status: Literal["completed", "failed", "skipped"]
+    success: bool
+    duration: float
+    started_at: datetime
+    completed_at: datetime
+    error: str | None = None
+    error_traceback: str | None = None
+    artifacts: list[str] = field(default_factory=list)
+    skip_reason: str | None = None
+
+    def to_dict(self) -> dict[str, Any]:
+        """Convert to dictionary for serialization."""
+        return {
+            "step_id": self.step_id,
+            "status": self.status,
+            "success": self.success,
+            "duration": self.duration,
+            "started_at": self.started_at.isoformat(),
+            "completed_at": self.completed_at.isoformat(),
+            "error": self.error,
+            "error_traceback": self.error_traceback,
+            "artifacts": self.artifacts,
+            "skip_reason": self.skip_reason,
+        }
+
+
 @dataclass
 class WorkflowState:
     """Workflow execution state."""

{tapps_agents-3.5.40.dist-info → tapps_agents-3.5.41.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tapps-agents
-Version: 3.5.40
+Version: 3.5.41
 Summary: A specification framework for defining, configuring, and orchestrating coding agents
 Author: Tapps
 Maintainer: Tapps

{tapps_agents-3.5.40.dist-info → tapps_agents-3.5.41.dist-info}/RECORD

@@ -1,4 +1,4 @@
-tapps_agents/__init__.py,sha256=PRiG3PBenRzWH5o_UapUUWxh2l2A7N_mHD-ute_PzjY,1180
+tapps_agents/__init__.py,sha256=ZnBHeqZtNulBHToeSdRA1bK58J9YrYT2DvheuT3MrEQ,1180
 tapps_agents/agents/__init__.py,sha256=w0Aj0bEdmIoPY1OcVTK0KFRSkYeAk-66CM2zmGTLjW0,698
 tapps_agents/agents/analyst/__init__.py,sha256=zLFkB6j1PkcpeaOk8Vnx76oYzVYyUqdULWj-pOyfLZY,135
 tapps_agents/agents/analyst/agent.py,sha256=YgPvWOqdZfIA24jIrbmHSdj3EVGIrGpwVIv5ep9uc8U,32033
@@ -133,7 +133,7 @@ tapps_agents/cli/commands/planner.py,sha256=dB43ux1hF5Z_bJImP06CpKocCRWcSGd2LG90
 tapps_agents/cli/commands/reviewer.py,sha256=TjuTgaQaCfaFfZfW-7Hm4X6JjUKz9vVzDodO1uA6Om4,78081
 tapps_agents/cli/commands/simple_mode.py,sha256=JjuaaPKo0DZXOqMNyvFNNqz98sNE78EZFeMIcu77lOU,31820
 tapps_agents/cli/commands/status.py,sha256=XrkCnxdiaieZrXExAW0_YrCDlkVGa9iRIiJ74DwP_5o,10661
-tapps_agents/cli/commands/task.py,sha256=YQTEopFPw7QUeSHG5xe98aJUG8YNy-tEOE794Kl0Vq4,7956
+tapps_agents/cli/commands/task.py,sha256=11mWEyAL-Mo3PzH6P1cg84o7Mn4xKuoCzzBCOy1IB5o,8445
 tapps_agents/cli/commands/tester.py,sha256=ZGwSw8O6wMOx1UO5kCgmawZUccC0MBL3kvRH-7dEic4,8481
 tapps_agents/cli/commands/top_level.py,sha256=dTh_rdoSiXrLgnDN04YIvlNITPnEnxoek7E8NusARho,149034
 tapps_agents/cli/help/__init__.py,sha256=2hO-whwGhNM4efFc7fc7ihTmE5PmxFl7jaQ2H7ywUbA,147
@@ -659,7 +659,7 @@ tapps_agents/workflow/confirmation_handler.py,sha256=iH6WCI08pDin5FBztzMYgRMWaO3
 tapps_agents/workflow/context_analyzer.py,sha256=-Nxq8okaqQD1rTAsOtNy0BNN_AcMQweM6KCxV8S6j34,8455
 tapps_agents/workflow/context_artifact.py,sha256=ps8sZgMXIemKNK1nADxpHfi5NQdFIVtmDMp48jDb5rI,8125
 tapps_agents/workflow/cursor_chat.py,sha256=e_2eXbr3yFYRdnu9iYcWIsuRgubd5-R68DsJrFN9buU,3180
-tapps_agents/workflow/cursor_executor.py,sha256=9wbMDAX4k18OVdGQYvPmd8IYS6Sxt2La18Pqo8tBWjQ,93712
+tapps_agents/workflow/cursor_executor.py,sha256=pdWZk3C4xMSD5JazEJFXwWK5SSSbB3363U__HYcKKEw,99476
 tapps_agents/workflow/cursor_skill_helper.py,sha256=QZCNCIBpwdZiP5piH9qB2l3Fseh4bMulCg8QcHyAfVA,19481
 tapps_agents/workflow/dependency_resolver.py,sha256=f_2a0uUBADHPEgNvdOdA-tCs-qYXwKyMkbvep2mwlrI,8923
 tapps_agents/workflow/design_artifact.py,sha256=HgEyNndiItMBPTTalOtiUrB600LF3EwwIL557034w7k,5510
@@ -693,7 +693,7 @@ tapps_agents/workflow/metadata_models.py,sha256=xsigrFQgIgFzLJn7qwCa242w49iEv-Qw
 tapps_agents/workflow/metrics_enhancements.py,sha256=6u5EnXCJTyLbi5BcouKamW2Pzutmzjv-Fy-ZGCHVgxk,25800
 tapps_agents/workflow/metrics_integration.py,sha256=NXhbw77zrS4X0xE61hCCNLt4lucfadKH1dSjANYowYI,7511
 tapps_agents/workflow/migration_utils.py,sha256=46kMrHV3yxujDpcDxtPytg9aL7N_BPRch1bEUIP_Ahc,3877
-tapps_agents/workflow/models.py,sha256=1lDPzWaKDEqfVNjRe20fSfe892nPvpLrItb85nFLfWk,2959
+tapps_agents/workflow/models.py,sha256=-IuecSogkRpYjOWjm4Qx-dpZ4uCnSbeVGcGPq7kvYHQ,4235
 tapps_agents/workflow/nlp_config.py,sha256=KesPjPzDfBa3WnnWT5IRkaNzzLbS4MLfr5UqdmTkYSI,6790
 tapps_agents/workflow/nlp_error_handler.py,sha256=Kgdu1XrPFiHTTJ3WctbaezsBP4v4DwfSO7Nc6BSEdKo,7352
 tapps_agents/workflow/nlp_executor.py,sha256=IiWv4ybBHWtccn0rN8Tp0ckpVVMdRA4V0DmiV_VbkFE,6806
@@ -752,9 +752,9 @@ tapps_agents/workflow/agent_handlers/planner_handler.py,sha256=9oVSsA6Lso5NncYPS
 tapps_agents/workflow/agent_handlers/registry.py,sha256=aYMU59da4SBgDgqMy4Basu1yL8CvDvdi929n6XM50ro,3613
 tapps_agents/workflow/agent_handlers/reviewer_handler.py,sha256=D4spVUk4l9mJsSsDIKC88EL-2k43YObNz5v439EtDRc,4285
 tapps_agents/workflow/agent_handlers/tester_handler.py,sha256=e44Eqn1CIpcqYqCrjaTK748yb4k7PlzGdK9zSne-SfY,2249
-tapps_agents-3.5.40.dist-info/licenses/LICENSE,sha256=xwT0RCg0KluJ63lZcgkSOiYkpEIEXYlIGqRtkGGsPe0,1085
-tapps_agents-3.5.40.dist-info/METADATA,sha256=jkhzTrjCsNOgFVIFPZwvARmAQ7EstqhNdZYHtMcUrWU,41413
-tapps_agents-3.5.40.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-tapps_agents-3.5.40.dist-info/entry_points.txt,sha256=b8WhZxYg2xA-ocElPeY0LDSqRCOgzw6PuEksZD6BB2E,55
-tapps_agents-3.5.40.dist-info/top_level.txt,sha256=AR0UfcM1nJhQ413h8j1ObAGsUrbfys55D-qKUBGzYtk,13
-tapps_agents-3.5.40.dist-info/RECORD,,
+tapps_agents-3.5.41.dist-info/licenses/LICENSE,sha256=xwT0RCg0KluJ63lZcgkSOiYkpEIEXYlIGqRtkGGsPe0,1085
+tapps_agents-3.5.41.dist-info/METADATA,sha256=WeP5io4GWolPJjOHH-c1GO6jHwqQ6YY-SJKF3hJvVO8,41413
+tapps_agents-3.5.41.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+tapps_agents-3.5.41.dist-info/entry_points.txt,sha256=b8WhZxYg2xA-ocElPeY0LDSqRCOgzw6PuEksZD6BB2E,55
+tapps_agents-3.5.41.dist-info/top_level.txt,sha256=AR0UfcM1nJhQ413h8j1ObAGsUrbfys55D-qKUBGzYtk,13
+tapps_agents-3.5.41.dist-info/RECORD,,