agentic-devtools 0.2.25__tar.gz → 0.2.26__tar.gz

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-devtools
-Version: 0.2.25
+Version: 0.2.26
 Summary: AI assistant helper commands for the Dragonfly platform
 Author: Dragonfly Team
 License-Expression: MIT

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/agentic_devtools/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.2.25'
-__version_tuple__ = version_tuple = (0, 2, 25)
+__version__ = version = '0.2.26'
+__version_tuple__ = version_tuple = (0, 2, 26)
 
 __commit_id__ = commit_id = None

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/agentic_devtools/cli/workflows/commands.py

@@ -52,8 +52,8 @@ def initiate_pull_request_review_workflow(
     Args:
         pull_request_id: ID of the pull request to review.
         issue_key: Jira issue key to find the PR by branch name.
-        interactive: Whether to start the Copilot session interactively (default: True).
-            Set to False for pipeline/non-interactive mode.
+        interactive: Whether to start the Copilot session interactively (default: False).
+            Set to True for interactive mode.
         _argv: Command line arguments (for testing). Pass [] in tests to avoid parsing sys.argv
             (this function always calls parse_known_args, so unknown flags are silently ignored,
             but passing [] prevents any accidental parsing of the test runner's sys.argv).
@@ -98,7 +98,7 @@ Examples:
         "--interactive",
         dest="interactive",
         default=None,
-        help="Start Copilot session interactively (default: true). Pass 'false' for pipeline mode.",
+        help="Start Copilot session interactively (default: false). Pass 'true' for interactive mode.",
     )
     args, _unknown = parser.parse_known_args(_argv)
 
@@ -110,7 +110,7 @@ Examples:
     if interactive is None and args.interactive is not None:
         interactive = args.interactive.lower() not in ("false", "0", "no")
     if interactive is None:
-        interactive = True
+        interactive = False
 
     # Set provided values in state (use set_value directly since we handle cross-lookup below)
     if pull_request_id:
@@ -206,6 +206,7 @@ Examples:
         ]
         if resolved_issue_key:
             auto_execute_command.extend(["--issue-key", resolved_issue_key])
+        auto_execute_command.extend(["--interactive", "true" if interactive else "false"])
 
         # Automatically set up the environment with the PR's source branch
         if perform_auto_setup(

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/agentic_devtools/cli/workflows/preflight.py

@@ -246,7 +246,7 @@ def perform_auto_setup(
     additional_params: Optional[dict] = None,
     auto_execute_command: Optional[list[str]] = None,
     auto_execute_timeout: int = 300,
-    interactive: bool = True,
+    interactive: bool = False,
 ) -> bool:
     """
     Automatically set up a worktree environment for the issue as a background task.
@@ -277,7 +277,7 @@ def perform_auto_setup(
         auto_execute_timeout: Timeout in seconds for the auto-execute command
             (default: 300).
         interactive: Whether to start the Copilot session interactively after
-            the worktree is ready (default: True). Set to False for pipeline mode.
+            the worktree is ready (default: False). Set to True for interactive mode.
 
     Returns:
         True if the background task was started, False otherwise

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/agentic_devtools/cli/workflows/worktree_setup.py

@@ -976,7 +976,7 @@ def _wait_for_prompt_file(
 
 def _start_copilot_session_for_pr_review(
     worktree_path: str,
-    interactive: bool = True,
+    interactive: bool = False,
 ) -> None:
     """
     Wait for the workflow setup to complete, then start a ``gh copilot``
@@ -1054,7 +1054,7 @@ def setup_worktree_in_background_sync(
     additional_params: Optional[dict] = None,
     auto_execute_command: Optional[list[str]] = None,
     auto_execute_timeout: int = 300,
-    interactive: bool = True,
+    interactive: bool = False,
 ) -> None:
     """
     Perform worktree setup synchronously (called from background task).
@@ -1063,10 +1063,8 @@ def setup_worktree_in_background_sync(
     It performs the full worktree setup and prints the continuation prompt.
 
     For ``pull-request-review`` workflows with an ``auto_execute_command``,
-    the function additionally:
-    1. Waits for the review initiate prompt to be generated by the auto-execute.
-    2. Loads repo-specific focus areas from ``{worktree}/.github/agdt-config.json``.
-    3. Starts a ``gh copilot`` session with the integrated prompt.
+    the auto-execute command re-runs the full workflow inside the worktree,
+    which handles starting the Copilot session itself.
 
     Args:
         issue_key: The Jira issue key
@@ -1084,7 +1082,7 @@ def setup_worktree_in_background_sync(
         auto_execute_timeout: Timeout in seconds for the auto-execute command
             (default: 300).
         interactive: Whether to start the Copilot session interactively after
-            setup (default: True). Set to False for pipeline/non-interactive mode.
+            setup (default: False). Set to True for interactive mode.
     """
     from ...state import set_value
 
@@ -1106,9 +1104,6 @@ def setup_worktree_in_background_sync(
             exit_code = _run_auto_execute_command(auto_execute_command, existing_path, auto_execute_timeout)
             set_value("worktree_setup.auto_execute_exit_code", str(exit_code))
 
-            if workflow_name == "pull-request-review" and exit_code == 0:
-                _start_copilot_session_for_pr_review(existing_path, interactive=interactive)
-
         print("\n✅ Environment ready!")
         print(get_worktree_continuation_prompt(issue_key, workflow_name, user_request, additional_params))
         print("\n" + "=" * 80)
@@ -1141,9 +1136,6 @@ can copy and paste it into the new VS Code window that just opened:
             exit_code = _run_auto_execute_command(auto_execute_command, result.worktree_path, auto_execute_timeout)
             set_value("worktree_setup.auto_execute_exit_code", str(exit_code))
 
-            if workflow_name == "pull-request-review" and exit_code == 0:
-                _start_copilot_session_for_pr_review(result.worktree_path, interactive=interactive)
-
         print("\n✅ Environment setup complete!")
         print(f"   Worktree: {result.worktree_path}")
         print(f"   Branch: {result.branch_name}")
@@ -1211,8 +1203,8 @@ def _setup_worktree_from_state() -> None:
         except ValueError:
             pass
 
-    # Default interactive to True; stored as "false" string to disable
-    interactive = interactive_str != "false"
+    # Default interactive to False; stored as "true" string to enable
+    interactive = interactive_str == "true"
 
     if not issue_key:
         raise ValueError("worktree_setup.issue_key not set in state")
@@ -1242,7 +1234,7 @@ def start_worktree_setup_background(
     additional_params: Optional[dict] = None,
     auto_execute_command: Optional[list[str]] = None,
     auto_execute_timeout: int = 300,
-    interactive: bool = True,
+    interactive: bool = False,
 ) -> str:
     """
     Start worktree setup as a background task.
@@ -1267,7 +1259,7 @@ def start_worktree_setup_background(
         auto_execute_timeout: Timeout in seconds for the auto-execute command
             (default: 300).
         interactive: Whether to start the Copilot session interactively after
-            setup (default: True). Set to False for pipeline/non-interactive mode.
+            setup (default: False). Set to True for interactive mode.
 
     Returns:
         The background task ID for tracking progress

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/tests/unit/cli/workflows/commands/test_initiate_pull_request_review_workflow.py

@@ -345,10 +345,10 @@ class TestInitiatePRReviewWorkflowInteractive:
         call_kwargs = mock_setup.call_args[1]
         assert call_kwargs["interactive"] is False
 
-    def test_interactive_defaults_to_true(
+    def test_interactive_defaults_to_false(
         self, temp_state_dir, clear_state_before, mock_workflow_state_clearing, capsys
     ):
-        """Test that interactive defaults to True when not specified."""
+        """Test that interactive defaults to False when not specified."""
         state.set_value("pull_request_id", "456")
         state.set_value("jira.issue_key", "DFLY-5678")
 
@@ -371,7 +371,7 @@ class TestInitiatePRReviewWorkflowInteractive:
                     commands.initiate_pull_request_review_workflow(_argv=[])
 
         call_kwargs = mock_setup.call_args[1]
-        assert call_kwargs["interactive"] is True
+        assert call_kwargs["interactive"] is False
 
     def test_auto_execute_command_passed_with_pr_id_and_issue_key(
         self, temp_state_dir, clear_state_before, mock_workflow_state_clearing, capsys
@@ -405,6 +405,8 @@ class TestInitiatePRReviewWorkflowInteractive:
             "789",
             "--issue-key",
             "DFLY-9999",
+            "--interactive",
+            "false",
         ]
         assert call_kwargs["auto_execute_command"] == expected_cmd
 
@@ -440,6 +442,39 @@ class TestInitiatePRReviewWorkflowInteractive:
         assert "--pull-request-id" in auto_cmd
         assert "111" in auto_cmd
         assert "--issue-key" not in auto_cmd
+        assert "--interactive" in auto_cmd
+        assert "false" in auto_cmd
+
+    def test_auto_execute_command_includes_interactive_true(
+        self, temp_state_dir, clear_state_before, mock_workflow_state_clearing, capsys
+    ):
+        """Test that auto_execute_command includes --interactive true when interactive is explicitly set."""
+        state.set_value("pull_request_id", "789")
+        state.set_value("jira.issue_key", "DFLY-9999")
+
+        with patch("agentic_devtools.cli.azure_devops.helpers.get_pull_request_source_branch") as mock_src:
+            mock_src.return_value = "feature/DFLY-9999/impl"
+
+            with patch("agentic_devtools.cli.workflows.commands.check_worktree_and_branch") as mock_preflight:
+                from agentic_devtools.cli.workflows.preflight import PreflightResult
+
+                mock_preflight.return_value = PreflightResult(
+                    folder_valid=False,
+                    branch_valid=False,
+                    folder_name="wrong",
+                    branch_name="main",
+                    issue_key="DFLY-9999",
+                )
+
+                with patch("agentic_devtools.cli.workflows.preflight.perform_auto_setup") as mock_setup:
+                    mock_setup.return_value = True
+                    commands.initiate_pull_request_review_workflow(_argv=["--interactive", "true"])
+
+        call_kwargs = mock_setup.call_args[1]
+        auto_cmd = call_kwargs["auto_execute_command"]
+        assert "--interactive" in auto_cmd
+        interactive_idx = auto_cmd.index("--interactive")
+        assert auto_cmd[interactive_idx + 1] == "true"
 
 
 class TestInitiatePRReviewWorkflowCopilotSession:
@@ -486,12 +521,12 @@ class TestInitiatePRReviewWorkflowCopilotSession:
         call_kwargs = mock_session.call_args
         assert call_kwargs[0][0] == "/fake/repo-root"
 
-    def test_copilot_session_interactive_default_is_true(
+    def test_copilot_session_interactive_default_is_false(
         self, temp_state_dir, clear_state_before, mock_workflow_state_clearing
     ):
-        """_start_copilot_session_for_pr_review is called with interactive=True by default."""
+        """_start_copilot_session_for_pr_review is called with interactive=False by default."""
         mock_session = self._run_with_preflight_passing("999", "feature/some-branch")
-        mock_session.assert_called_once_with("/fake/repo-root", interactive=True)
+        mock_session.assert_called_once_with("/fake/repo-root", interactive=False)
 
     def test_copilot_session_respects_interactive_false(
         self, temp_state_dir, clear_state_before, mock_workflow_state_clearing
@@ -499,3 +534,10 @@ class TestInitiatePRReviewWorkflowCopilotSession:
         """_start_copilot_session_for_pr_review is called with interactive=False when --interactive false."""
         mock_session = self._run_with_preflight_passing("999", "feature/some-branch", argv=["--interactive", "false"])
         mock_session.assert_called_once_with("/fake/repo-root", interactive=False)
+
+    def test_copilot_session_interactive_true_when_explicitly_set(
+        self, temp_state_dir, clear_state_before, mock_workflow_state_clearing
+    ):
+        """_start_copilot_session_for_pr_review is called with interactive=True when --interactive true."""
+        mock_session = self._run_with_preflight_passing("999", "feature/some-branch", argv=["--interactive", "true"])
+        mock_session.assert_called_once_with("/fake/repo-root", interactive=True)

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/tests/unit/cli/workflows/preflight/test_perform_auto_setup.py

@@ -32,7 +32,7 @@ class TestPerformAutoSetup:
             additional_params=None,
             auto_execute_command=None,
             auto_execute_timeout=300,
-            interactive=True,
+            interactive=False,
         )
         captured = capsys.readouterr()
         assert "task-12345" in captured.out
@@ -139,8 +139,8 @@ class TestPerformAutoSetup:
         assert call_kwargs["interactive"] is False
 
     @patch("agentic_devtools.cli.workflows.worktree_setup.start_worktree_setup_background")
-    def test_passes_interactive_true_by_default(self, mock_start_background, capsys):
-        """Test that interactive defaults to True when not specified."""
+    def test_passes_interactive_false_by_default(self, mock_start_background, capsys):
+        """Test that interactive defaults to False when not specified."""
         mock_start_background.return_value = "task-default"
 
         perform_auto_setup(
@@ -150,4 +150,4 @@ class TestPerformAutoSetup:
 
         mock_start_background.assert_called_once()
         call_kwargs = mock_start_background.call_args[1]
-        assert call_kwargs["interactive"] is True
+        assert call_kwargs["interactive"] is False

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/tests/unit/cli/workflows/worktree_setup/test__setup_worktree_from_state.py

@@ -39,7 +39,7 @@ class TestSetupWorktreeFromState:
             additional_params={"pr_id": "123"},
             auto_execute_command=None,
             auto_execute_timeout=300,
-            interactive=True,
+            interactive=False,
         )
 
     @patch("agentic_devtools.state.get_value")
@@ -108,7 +108,7 @@ class TestSetupWorktreeFromState:
             additional_params=None,
             auto_execute_command=None,
             auto_execute_timeout=300,
-            interactive=True,
+            interactive=False,
         )
 
     @patch("agentic_devtools.cli.workflows.worktree_setup.setup_worktree_in_background_sync")
@@ -210,8 +210,8 @@ class TestSetupWorktreeFromState:
 
     @patch("agentic_devtools.cli.workflows.worktree_setup.setup_worktree_in_background_sync")
     @patch("agentic_devtools.state.get_value")
-    def test_defaults_interactive_to_true_when_not_stored(self, mock_get_value, mock_setup_sync):
-        """Test that interactive defaults to True when not set in state."""
+    def test_defaults_interactive_to_false_when_not_stored(self, mock_get_value, mock_setup_sync):
+        """Test that interactive defaults to False when not set in state."""
         mock_get_value.side_effect = lambda key: {
             "worktree_setup.issue_key": "DFLY-1234",
             "worktree_setup.branch_prefix": "feature",
@@ -230,4 +230,4 @@ class TestSetupWorktreeFromState:
         _setup_worktree_from_state()
 
         call_kwargs = mock_setup_sync.call_args[1]
-        assert call_kwargs["interactive"] is True
+        assert call_kwargs["interactive"] is False

{agentic_devtools-0.2.25 → agentic_devtools-0.2.26}/tests/unit/cli/workflows/worktree_setup/test_setup_worktree_in_background_sync.py

@@ -285,7 +285,7 @@ class TestSetupWorktreeInBackgroundSync:
     @patch("agentic_devtools.cli.workflows.worktree_setup.get_worktree_continuation_prompt")
     @patch("agentic_devtools.cli.workflows.worktree_setup.setup_worktree_environment")
     @patch("agentic_devtools.cli.workflows.worktree_setup.check_worktree_exists")
-    def test_copilot_session_started_for_pr_review_workflow(
+    def test_copilot_session_not_started_for_pr_review_workflow_with_auto_execute(
         self,
         mock_check_exists,
         mock_setup,
@@ -295,7 +295,7 @@ class TestSetupWorktreeInBackgroundSync:
         mock_set_value,
         mock_copilot,
     ):
-        """Test that Copilot session is started for pull-request-review workflow with auto_execute."""
+        """Test that Copilot session is NOT started when auto_execute_command re-runs the workflow (new worktree)."""
         mock_check_exists.return_value = None
         mock_setup.return_value = WorktreeSetupResult(
             success=True,
@@ -313,7 +313,7 @@ class TestSetupWorktreeInBackgroundSync:
             interactive=True,
         )
 
-        mock_copilot.assert_called_once_with("/repos/DFLY-1234", interactive=True)
+        mock_copilot.assert_not_called()
 
     @patch("agentic_devtools.cli.workflows.worktree_setup._start_copilot_session_for_pr_review")
     @patch("agentic_devtools.cli.workflows.worktree_setup.get_ai_agent_continuation_prompt")
@@ -384,7 +384,7 @@ class TestSetupWorktreeInBackgroundSync:
     @patch("agentic_devtools.cli.workflows.worktree_setup.get_worktree_continuation_prompt")
     @patch("agentic_devtools.cli.workflows.worktree_setup.open_vscode_workspace")
     @patch("agentic_devtools.cli.workflows.worktree_setup.check_worktree_exists")
-    def test_copilot_session_started_for_existing_pr_review_worktree(
+    def test_copilot_session_not_started_for_existing_pr_review_worktree_with_auto_execute(
         self,
         mock_check_exists,
         mock_open_vscode,
@@ -394,7 +394,7 @@ class TestSetupWorktreeInBackgroundSync:
         mock_set_value,
         mock_copilot,
     ):
-        """Test that Copilot session is started when worktree already exists for PR review."""
+        """Test that Copilot session is NOT started when auto_execute re-runs the workflow (existing worktree)."""
         mock_check_exists.return_value = "/repos/DFLY-1234"
         mock_open_vscode.return_value = True
         mock_continuation_prompt.return_value = "Continue..."
@@ -408,7 +408,7 @@ class TestSetupWorktreeInBackgroundSync:
             interactive=False,
         )
 
-        mock_copilot.assert_called_once_with("/repos/DFLY-1234", interactive=False)
+        mock_copilot.assert_not_called()
 
     @patch("agentic_devtools.cli.workflows.worktree_setup._start_copilot_session_for_pr_review")
     @patch("agentic_devtools.state.set_value")