Merge pull request #285 from tbrandenburg/codex/exclude-crontab-yml-in-worktrees

Author: tbrandenburg

packages/pybackend/tests/unit/test_workflow_service.py

@@ -104,3 +104,41 @@ def test_list_workspace_workflows_collects_repository_workflows(
             }
         ]
     }
+
+
+@patch("workflow_service.read_workflows")
+@patch("workflow_service.get_workspace_home")
+def test_list_workspace_workflows_skips_git_worktrees(
+    mock_workspace_home, mock_read_workflows
+):
+    mock_workspace_home.return_value = Path("/workspace/home")
+
+    repo = Path("/workspace/home/repo-a")
+    worktree = Path("/workspace/home/repo-a-feature")
+
+    with patch.object(Path, "iterdir", return_value=[repo, worktree]), patch.object(
+        Path, "is_dir", return_value=True
+    ), patch.object(Path, "is_file", side_effect=[False, True]):
+        mock_read_workflows.return_value = {
+            "workflows": [
+                {"id": "wf_a", "name": "A", "enabled": True, "schedule": "* * * * *"}
+            ]
+        }
+
+        result = list_workspace_workflows()
+
+    assert result == {
+        "workflows": [
+            {
+                "repository": "repo-a",
+                "id": "wf_a",
+                "name": "A",
+                "enabled": True,
+                "schedule": "* * * * *",
+                "shellScriptPath": None,
+                "lastRun": None,
+                "diagnostics": None,
+            }
+        ]
+    }
+    mock_read_workflows.assert_called_once_with("repo-a")

packages/pybackend/workflow_service.py

@@ -124,6 +124,8 @@ def list_workspace_workflows(
     for repo_path in workspace_home.iterdir():
         if not repo_path.is_dir():
             continue
+        if (repo_path / ".git").is_file():
+            continue
 
         repo_name = repo_path.name
         repository_workflows = read_workflows(repo_name).get("workflows", [])