steerdev 1.1.26__tar.gz → 1.1.28__tar.gz

{steerdev-1.1.26 → steerdev-1.1.28}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: steerdev
-Version: 1.1.26
+Version: 1.1.28
 Summary: Backend task runner for steerdev.com - orchestrates CLI coding agents with activity reporting
 Project-URL: Homepage, https://github.com/pentoai/steerdev-agent
 Project-URL: Repository, https://github.com/pentoai/steerdev-agent

{steerdev-1.1.26 → steerdev-1.1.28}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "steerdev"
-version = "1.1.26"
+version = "1.1.28"
 description = "Backend task runner for steerdev.com - orchestrates CLI coding agents with activity reporting"
 readme = "README.md"
 authors = [

{steerdev-1.1.26 → steerdev-1.1.28}/src/steerdev_agent/agent_loop.py

@@ -582,10 +582,15 @@ class AgentLoop(CommandExecutor):
             # Step 2: Recover stale commands
             await self._recover_stale_commands()
 
-            # Step 3: Start heartbeat
+            # Step 3: Reconcile .claude config (hooks) for this checkout, so a
+            # repo that predates newer steerdev hooks (e.g. the PostToolUse
+            # checkpoint-push hook) self-heals before any task launches Claude.
+            self._reconcile_settings()
+
+            # Step 4: Start heartbeat
             self._heartbeat_task = asyncio.create_task(self._heartbeat_loop())
 
-            # Step 4: Main poll loop
+            # Step 5: Main poll loop
             await self._poll_loop()
 
         except AgentLoopError as e:
@@ -647,6 +652,23 @@ class AgentLoop(CommandExecutor):
                 )
                 await self._commands_client.mark_pending(cmd.id)
 
+    def _reconcile_settings(self) -> None:
+        """Re-apply steerdev-owned .claude/settings.json hooks for this checkout.
+
+        Single-project mode runs against an already-existing repo that never
+        went through project bootstrap, so its .claude config can be missing or
+        predate newer steerdev hooks. Reconciling here (idempotent, best-effort)
+        ensures the PostToolUse checkpoint-push hook is present before Claude
+        launches. A setup failure must never block the agent.
+        """
+        from steerdev_agent.setup.claude_setup import ClaudeSetup
+
+        try:
+            setup = ClaudeSetup(self.working_directory, install_target="project")
+            setup.setup_settings(force=False, include_hooks=True)
+        except Exception:
+            logger.exception(f"Failed to reconcile .claude settings for {self.working_directory}")
+
     async def _poll_loop(self) -> None:
         """Main loop: poll for commands and execute them."""
         logger.info("Entering poll loop")

{steerdev-1.1.26 → steerdev-1.1.28}/src/steerdev_agent/runner.py

@@ -1109,6 +1109,23 @@ class Runner:
                     except Exception:
                         logger.debug(f"Failed to cleanup worktree {worktree_branch}", exc_info=True)
 
+    def _reconcile_settings(self) -> None:
+        """Re-apply steerdev-owned .claude/settings.json hooks for this checkout.
+
+        `steerdev run` operates on an existing or freshly-cloned repo that never
+        went through project bootstrap (notably sandbox dispatch, which clones
+        and immediately runs). Reconciling here (idempotent, best-effort)
+        installs/heals the PostToolUse checkpoint-push hook before Claude
+        launches, so commits push. A setup failure must never block the run.
+        """
+        from steerdev_agent.setup.claude_setup import ClaudeSetup
+
+        try:
+            setup = ClaudeSetup(self.working_directory, install_target="project")
+            setup.setup_settings(force=False, include_hooks=True)
+        except Exception:
+            logger.exception(f"Failed to reconcile .claude settings for {self.working_directory}")
+
     async def run(self) -> dict[str, Any]:
         """Run the agent.
 
@@ -1137,6 +1154,13 @@ class Runner:
         try:
             await self._notify_state_change(RunState.RUNNING)
 
+            # Reconcile .claude config (hooks) for this checkout before any task
+            # launches Claude. `steerdev run` (used by sandbox dispatch) operates
+            # on a fresh clone with no .claude config, so without this the
+            # PostToolUse checkpoint-push hook is absent and sandbox commits
+            # never push. Idempotent and best-effort.
+            self._reconcile_settings()
+
             from steerdev_agent.retry import execute_with_retry
 
             # Multi-task loop

{steerdev-1.1.26 → steerdev-1.1.28}/src/steerdev_agent/workspace/preparation.py

@@ -322,6 +322,13 @@ class WorkspacePreparation:
         if wave_context and current_branch == target_branch:
             logger.info(f"Continuing wave on branch {target_branch}, fetching only")
             await _run_git("fetch", "origin", cwd=cwd, check=False)
+            # Persist provenance here too: a wave-continuation task arrives
+            # already on the branch, so it skips the create/checkout path
+            # below where provenance is normally written. Without this, the
+            # second+ task in a wave never records its branch on the task.
+            await persist_branch_provenance(
+                task_id=task_id, branch=target_branch, base_branch=default_branch
+            )
             return PreparationResult(
                 success=True,
                 branch_name=target_branch,

{steerdev-1.1.26 → steerdev-1.1.28}/tests/test_agent_loop.py

@@ -72,6 +72,37 @@ class TestAgentLoopError:
         assert str(err) == "test error"
 
 
+class TestAgentLoopReconcileSettings:
+    """Tests for single-project AgentLoop hook reconciliation."""
+
+    def _make_agent_loop(self, working_directory):
+        from steerdev_agent.agent_loop import AgentLoop
+
+        return AgentLoop(
+            project_id="proj-1",
+            agent_name="test-agent",
+            working_directory=working_directory,
+            api_key="test-key",
+        )
+
+    def test_reconciles_settings_at_working_directory(self, tmp_path):
+        agent = self._make_agent_loop(tmp_path)
+
+        with patch("steerdev_agent.setup.claude_setup.ClaudeSetup") as cls:
+            agent._reconcile_settings()
+
+        cls.assert_called_once_with(tmp_path, install_target="project")
+        cls.return_value.setup_settings.assert_called_once_with(force=False, include_hooks=True)
+
+    def test_reconcile_swallows_errors(self, tmp_path):
+        agent = self._make_agent_loop(tmp_path)
+
+        with patch("steerdev_agent.setup.claude_setup.ClaudeSetup") as cls:
+            cls.return_value.setup_settings.side_effect = RuntimeError("disk full")
+            # Must not raise — durability must never block the agent.
+            agent._reconcile_settings()
+
+
 class TestCommandExecutor:
     """Tests for CommandExecutor._execute_command dispatch logic."""
 

{steerdev-1.1.26 → steerdev-1.1.28}/tests/test_preparation.py

@@ -232,7 +232,10 @@ class TestWorkspacePreparationInPlace:
                 return (0, "", "")
             return (0, "", "")
 
-        with patch("steerdev_agent.workspace.preparation._run_git", side_effect=mock_run_git):
+        with (
+            patch("steerdev_agent.workspace.preparation._run_git", side_effect=mock_run_git),
+            patch("steerdev_agent.workspace.preparation.persist_branch_provenance") as mock_persist,
+        ):
             result = await prep.prepare(sample_task, wave_ctx)
 
         assert result.success is True
@@ -240,6 +243,11 @@ class TestWorkspacePreparationInPlace:
         # Should NOT have checkout commands (stayed on branch)
         checkout_cmds = [c for c in calls if c[0] == "checkout"]
         assert len(checkout_cmds) == 0
+        # Wave-continuation must still record branch provenance on the task,
+        # even though it skips the create/checkout path.
+        mock_persist.assert_awaited_once_with(
+            task_id=sample_task["id"], branch="wave-a1b2c3", base_branch="main"
+        )
 
     @pytest.mark.asyncio
     async def test_dirty_repo_stashes(self, worktree_disabled, default_config, sample_task):

{steerdev-1.1.26 → steerdev-1.1.28}/tests/test_runner_worktrees.py

@@ -1,5 +1,8 @@
 """Tests for worktree configuration and Runner worktree integration."""
 
+from pathlib import Path
+from unittest.mock import patch
+
 from steerdev_agent.config.models import EvidenceConfig, WorktreeConfig
 from steerdev_agent.prompt.builder import WaveContext
 from steerdev_agent.runner import Runner
@@ -50,6 +53,27 @@ class TestComputeBranchName:
         assert compute_branch_name(task) == "task/JIRA-456-fix"
 
 
+class TestRunnerReconcileSettings:
+    """Hook reconciliation for `steerdev run` (sandbox + local)."""
+
+    def test_reconciles_settings_at_working_directory(self):
+        r = Runner(project_id="p", api_key="k", working_directory=Path("/code/repo"))
+
+        with patch("steerdev_agent.setup.claude_setup.ClaudeSetup") as cls:
+            r._reconcile_settings()
+
+        cls.assert_called_once_with(Path("/code/repo"), install_target="project")
+        cls.return_value.setup_settings.assert_called_once_with(force=False, include_hooks=True)
+
+    def test_reconcile_swallows_errors(self):
+        r = Runner(project_id="p", api_key="k", working_directory=Path("/code/repo"))
+
+        with patch("steerdev_agent.setup.claude_setup.ClaudeSetup") as cls:
+            cls.return_value.setup_settings.side_effect = RuntimeError("disk full")
+            # Must not raise — durability must never block the run.
+            r._reconcile_settings()
+
+
 class TestRunnerWorktreeConfig:
     def test_disabled_by_default(self):
         r = Runner(project_id="p", api_key="k")