Move agent tests under tests/unit/ai and tests/integration/ai

Nests the migrated agent tests as an `ai` topic-subdir under Conductor's
pre-existing tests/unit and tests/integration parents (matching the
convention already used for automator/, orkes/, etc.), instead of the old
tests/ai/{unit,integration} structure that stood apart as a parallel
top-level grouping. tests/ai/fixtures is unaffected.

Fixes test_skill.py's FIXTURES path, which assumed fixtures was a sibling
one level up (true under the old layout) and pointed at a
nonexistent directory after the move. Updates the PR CI step and
scripts/run_agent_tests.sh's hardcoded paths to match.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Author: Kowser

.github/workflows/pull_request.yml

@@ -33,6 +33,11 @@ jobs:
         run: |
           python -c "import conductor.ai.agents"
 
+      - name: Install agents extra
+        run: |
+          pip install -e '.[agents]'
+          pip install pytest-asyncio
+
       - name: Prepare coverage directory
         run: |
           mkdir -p ${{ env.COVERAGE_DIR }}
@@ -70,19 +75,6 @@ jobs:
         run: |
           coverage run -m pytest tests/serdesertest -v
 
-      - name: Install agents extra
-        run: |
-          pip install -e '.[agents]'
-          pip install pytest-asyncio
-
-      - name: Run agent tests
-        id: agent_tests
-        continue-on-error: true
-        env:
-          COVERAGE_FILE: ${{ env.COVERAGE_DIR }}/.coverage.agents
-        run: |
-          coverage run -m pytest tests/ai -m "not integration and not e2e" -v
-
       - name: Generate coverage report
         id: coverage_report
         run: |
@@ -110,5 +102,5 @@ jobs:
           file: coverage.xml
 
       - name: Check test results
-        if: steps.unit_tests.outcome == 'failure' || steps.bc_tests.outcome == 'failure' || steps.serdeser_tests.outcome == 'failure' || steps.agent_base_import.outcome == 'failure' || steps.agent_tests.outcome == 'failure'
+        if: steps.unit_tests.outcome == 'failure' || steps.bc_tests.outcome == 'failure' || steps.serdeser_tests.outcome == 'failure' || steps.agent_base_import.outcome == 'failure'
         run: exit 1