Merge branch 'main' into deepwork/deepwork_jobs-add-platform-20260112

Author: nhorton

.github/workflows/validate.yml

@@ -4,18 +4,12 @@ on:
   pull_request:
     branches: ["*"]
 
-permissions:
-  contents: write
-
 jobs:
-  format:
+  tests:
     runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@v4
-        with:
-          ref: ${{ github.head_ref }}
-          token: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Install uv
         uses: astral-sh/setup-uv@v4
@@ -30,56 +24,10 @@ jobs:
       - name: Install dependencies
         run: uv sync --extra dev
 
-      - name: Run ruff formatting
-        run: uv run ruff format src/ tests/
-
-      - name: Run ruff linting with auto-fix
-        run: uv run ruff check --fix src/ tests/
-
-      - name: Re-run formatting after auto-fix
-        run: uv run ruff format src/ tests/
-
-      - name: Verify all issues are fixed
+      - name: Check formatting (ruff)
         run: |
-          uv run ruff check src/ tests/
           uv run ruff format --check src/ tests/
-
-      - name: Check for changes
-        id: check_changes
-        run: |
-          if [[ -n "$(git status --porcelain)" ]]; then
-            echo "has_changes=true" >> $GITHUB_OUTPUT
-          else
-            echo "has_changes=false" >> $GITHUB_OUTPUT
-          fi
-
-      - name: Commit and push formatting changes
-        if: steps.check_changes.outputs.has_changes == 'true'
-        run: |
-          git config --local user.email "github-actions[bot]@users.noreply.github.com"
-          git config --local user.name "github-actions[bot]"
-          git add -A
-          git commit -m "style: auto-format code with ruff"
-          git push
-
-  tests:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: Install uv
-        uses: astral-sh/setup-uv@v4
-        with:
-          version: "latest"
-
-      - name: Set up Python
-        uses: actions/setup-python@v5
-        with:
-          python-version: "3.11"
-
-      - name: Install dependencies
-        run: uv sync --extra dev
+          uv run ruff check src/ tests/
 
       - name: Run tests
         run: uv run pytest tests/ -v

src/deepwork/core/adapters.py

@@ -103,8 +103,7 @@ def get(cls, name: str) -> type[AgentAdapter]:
         """
         if name not in cls._registry:
             raise AdapterError(
-                f"Unknown adapter '{name}'. "
-                f"Supported adapters: {', '.join(cls._registry.keys())}"
+                f"Unknown adapter '{name}'. Supported adapters: {', '.join(cls._registry.keys())}"
             )
         return cls._registry[name]
 

src/deepwork/core/generator.py

@@ -83,9 +83,7 @@ def _is_standalone_step(self, job: JobDefinition, step: Step) -> bool:
 
         return True
 
-    def _build_hook_context(
-        self, job: JobDefinition, hook_action: Any
-    ) -> dict[str, Any]:
+    def _build_hook_context(self, job: JobDefinition, hook_action: Any) -> dict[str, Any]:
         """
         Build context for a single hook action.
 
@@ -177,8 +175,7 @@ def _build_step_context(
 
         # Backward compatibility: stop_hooks is after_agent hooks
         stop_hooks = hooks.get(
-            adapter.get_platform_hook_name(CommandLifecycleHook.AFTER_AGENT) or "Stop",
-            []
+            adapter.get_platform_hook_name(CommandLifecycleHook.AFTER_AGENT) or "Stop", []
         )
 
         return {

src/deepwork/core/parser.py

@@ -132,9 +132,7 @@ def from_dict(cls, data: dict[str, Any]) -> "Step":
         if "stop_hooks" in data and data["stop_hooks"]:
             # Merge with any existing after_agent hooks
             after_agent_hooks = hooks.get("after_agent", [])
-            after_agent_hooks.extend(
-                [HookAction.from_dict(h) for h in data["stop_hooks"]]
-            )
+            after_agent_hooks.extend([HookAction.from_dict(h) for h in data["stop_hooks"]])
             hooks["after_agent"] = after_agent_hooks
 
         return cls(

tests/unit/test_adapters.py

@@ -156,9 +156,7 @@ def test_sync_hooks_merges_with_existing(self, temp_dir: Path) -> None:
 
         adapter = ClaudeAdapter(temp_dir)
         hooks = {
-            "PreToolUse": [
-                {"matcher": "", "hooks": [{"type": "command", "command": "test.sh"}]}
-            ]
+            "PreToolUse": [{"matcher": "", "hooks": [{"type": "command", "command": "test.sh"}]}]
         }
 
         adapter.sync_hooks(temp_dir, hooks)

tests/unit/test_stop_hooks.py

@@ -485,9 +485,7 @@ def test_build_context_with_prompt_hook(
     ) -> None:
         """Test context building includes prompt stop hook."""
         adapter = ClaudeAdapter()
-        context = generator._build_step_context(
-            job_with_hooks, job_with_hooks.steps[0], 0, adapter
-        )
+        context = generator._build_step_context(job_with_hooks, job_with_hooks.steps[0], 0, adapter)
         assert "stop_hooks" in context
         assert len(context["stop_hooks"]) == 1
         assert context["stop_hooks"][0]["type"] == "prompt"