Skip to content

chore(tests): bootstrap pytest infra + CI guards#20

Open
isac322 wants to merge 2 commits into
mainfrom
overhaul/pr0-tests-ci-guards
Open

chore(tests): bootstrap pytest infra + CI guards#20
isac322 wants to merge 2 commits into
mainfrom
overhaul/pr0-tests-ci-guards

Conversation

@isac322
Copy link
Copy Markdown
Owner

@isac322 isac322 commented May 5, 2026

Why

Shared infrastructure for the backend overhaul series (pytest + CI grep guards + docs-seo trigger update).

What

  • tests/conftest.py: virtual_session fixture (Session.start/stop)
  • tests/test_smoke.py: D-Bus reachability check
  • pyproject.toml: pytest>=8.0, pytest-asyncio>=0.23 dev deps + [tool.pytest.ini_options] (@pytest.mark.kwin marker)
  • scripts/ci_guards.sh: 4 grep guards
    1. No print() reachable from MCP server paths
    2. Every kwin_wayland invocation must include --virtual
    3. No module-top accessibility_worker import in core.py (Pool worker isolation invariant)
    4. No os.fork / multiprocessing.Process( (spawn context only)
    • Excludes non-launch references (pkill / --version / echo)
  • .github/workflows/ci.yml: guards job
  • CLAUDE.md: docs-seo trigger table extended with dbus_args.py, window.py, accessibility_worker.py, docs/design/*.md

Verify

uv run pytest tests/test_smoke.py        # → 1 passed
bash scripts/ci_guards.sh                # → all 4 pass

Series: Part of a 7-PR series. PR 0/1/1b are based on main (parallel), PR 2–5 are stacked. Merge order: PR 0 → (1, 1b in parallel) → 2 → 3 → 4 → 5.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

📝 Docs & SEO Review

Source files changed in this PR:

pyproject.toml

Consistency check results:

✅  All documentation/plugin SEO checks passed.

Run @docs-seo in Claude Code to perform a full documentation review.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@isac322 isac322 changed the base branch from main to launch/backend-overhaul May 5, 2026 13:34
@isac322 isac322 force-pushed the overhaul/pr0-tests-ci-guards branch 2 times, most recently from 2dec01a to a650583 Compare May 5, 2026 14:00
@isac322 isac322 force-pushed the overhaul/pr0-tests-ci-guards branch from a650583 to 5e9bb96 Compare May 5, 2026 14:12
Base automatically changed from launch/backend-overhaul to main May 29, 2026 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant