claude-dev-env 4.1.0 → 4.3.0

package/.agents/skills/pull-request/scripts/test_pull_request.py

@@ -0,0 +1,398 @@
+"""Contract tests for process-local pull request publication."""
+
+from __future__ import annotations
+
+import os
+import subprocess
+import sys
+import threading
+from collections.abc import Callable
+from concurrent.futures import ThreadPoolExecutor
+from pathlib import Path
+from unittest.mock import Mock
+
+import pytest
+
+_SCRIPTS_DIRECTORY = Path(__file__).resolve().parent
+if str(_SCRIPTS_DIRECTORY) not in sys.path:
+    sys.path.insert(0, str(_SCRIPTS_DIRECTORY))
+
+import pull_request
+
+REPOSITORY = "jl-cmd/claude-dev-env"
+PR_NUMBER = "2518"
+PR_TITLE = "feat(skills): add guarded pull request publication"
+ACCOUNT_A = "JonEcho"
+ACCOUNT_B = "ReviewBot"
+BODY_FILENAME = "pull-request-body.md"
+LINTER_PATH = (
+    _SCRIPTS_DIRECTORY.parents[1] / "_shared/pr-loop/scripts/durable_post_lint.py"
+)
+COMMENT_CLI = ["comment", "--repo", REPOSITORY, "--number", PR_NUMBER, "--body-file"]
+ACTION_CASES = [
+    (
+        [
+            "create",
+            "--repo",
+            REPOSITORY,
+            "--base",
+            "main",
+            "--head",
+            "feature/post",
+            "--title",
+            PR_TITLE,
+            "--body-file",
+            BODY_FILENAME,
+        ],
+        "pr-create",
+        [
+            "gh",
+            "pr",
+            "create",
+            "--draft",
+            "--repo",
+            REPOSITORY,
+            "--base",
+            "main",
+            "--head",
+            "feature/post",
+            "--title",
+            PR_TITLE,
+            "--body-file",
+            BODY_FILENAME,
+        ],
+    ),
+    (
+        [
+            "edit",
+            "--repo",
+            REPOSITORY,
+            "--number",
+            PR_NUMBER,
+            "--title",
+            PR_TITLE,
+            "--body-file",
+            BODY_FILENAME,
+        ],
+        "pr-edit",
+        [
+            "gh",
+            "pr",
+            "edit",
+            PR_NUMBER,
+            "--repo",
+            REPOSITORY,
+            "--title",
+            PR_TITLE,
+            "--body-file",
+            BODY_FILENAME,
+        ],
+    ),
+    (
+        [*COMMENT_CLI, BODY_FILENAME],
+        "pr-comment",
+        [
+            "gh",
+            "pr",
+            "comment",
+            PR_NUMBER,
+            "--repo",
+            REPOSITORY,
+            "--body-file",
+            BODY_FILENAME,
+        ],
+    ),
+    (
+        [
+            "review",
+            "--repo",
+            REPOSITORY,
+            "--number",
+            PR_NUMBER,
+            "--body-file",
+            BODY_FILENAME,
+        ],
+        "pr-review",
+        [
+            "gh",
+            "pr",
+            "review",
+            PR_NUMBER,
+            "--repo",
+            REPOSITORY,
+            "--comment",
+            "--body-file",
+            BODY_FILENAME,
+        ],
+    ),
+]
+
+
+def _completion(
+    all_arguments: list[str], code: int = 0, stdout: str = "", stderr: str = ""
+) -> subprocess.CompletedProcess[str]:
+    return subprocess.CompletedProcess(all_arguments, code, stdout, stderr)
+
+
+@pytest.fixture
+def body_file(tmp_path: Path) -> Path:
+    post_file = tmp_path / BODY_FILENAME
+    post_file.write_text("Durable body text.", encoding="utf-8")
+    return post_file
+
+
+def _comment_arguments(body_file: Path) -> list[str]:
+    return [*COMMENT_CLI, str(body_file)]
+
+
+def _account_text(account_name: str) -> str:
+    return account_name[::-1]
+
+
+def _calls(command_runner: Mock) -> list[list[str]]:
+    return [each_call.args[0] for each_call in command_runner.call_args_list]
+
+
+def _run_comment(
+    body_file: Path, command_runner: Mock, environment: dict[str, str]
+) -> int:
+    return pull_request.main(
+        _comment_arguments(body_file),
+        all_environment=environment,
+        command_runner=command_runner,
+    )
+
+
+def _account_runner(
+    *, auth_code: int = 0, auth_stdout: str, action_code: int = 0, auth_stderr: str = ""
+) -> Mock:
+    return Mock(
+        side_effect=[
+            _completion([]),
+            _completion([], auth_code, auth_stdout, auth_stderr),
+            _completion([], action_code),
+        ]
+    )
+
+
+@pytest.mark.parametrize(
+    ("all_cli_arguments", "linter_action", "expected_gh"), ACTION_CASES
+)
+def test_actions_run_exact_linter_then_gh_arguments(
+    all_cli_arguments: list[str], linter_action: str, expected_gh: list[str]
+) -> None:
+    command_runner = Mock(side_effect=[_completion([]), _completion([])])
+    exit_code = pull_request.main(
+        all_cli_arguments, all_environment={}, command_runner=command_runner
+    )
+    title_arguments = ["--title", PR_TITLE] if "--title" in all_cli_arguments else []
+    expected_linter = [
+        sys.executable,
+        str(LINTER_PATH),
+        "--action",
+        linter_action,
+        *title_arguments,
+        "--body-file",
+        BODY_FILENAME,
+    ]
+    assert exit_code == 0
+    assert _calls(command_runner) == [expected_linter, expected_gh]
+
+
+def test_body_transport_uses_files_only(body_file: Path) -> None:
+    command_runner = Mock(side_effect=[_completion([]), _completion([])])
+    assert _run_comment(body_file, command_runner, {}) == 0
+    for each_command in _calls(command_runner):
+        assert "--body-file" in each_command
+        assert "--body" not in each_command
+        assert "Durable body text." not in each_command
+
+
+def test_linter_failure_stops_credential_lookup_and_action(body_file: Path) -> None:
+    command_runner = Mock(return_value=_completion([], 1))
+    exit_code = _run_comment(
+        body_file, command_runner, {"GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A}
+    )
+    assert exit_code == 1
+    assert command_runner.call_count == 1
+    assert Path(_calls(command_runner)[0][1]).name == "durable_post_lint.py"
+
+
+def test_account_lookup_scopes_action_environment(body_file: Path) -> None:
+    account_text = _account_text(ACCOUNT_A)
+    command_runner = _account_runner(auth_stdout=f"{account_text}\n")
+    parent_environment = {
+        "GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A,
+        "GITHUB_TOKEN": "parent-setting",
+        "UNCHANGED": "same",
+    }
+    assert _run_comment(body_file, command_runner, parent_environment) == 0
+    assert _calls(command_runner)[1] == ["gh", "auth", "token", "--user", ACCOUNT_A]
+    action_environment = command_runner.call_args_list[2].kwargs["env"]
+    assert action_environment["GH_TOKEN"] == account_text
+    assert "GITHUB_TOKEN" not in action_environment
+    assert action_environment["UNCHANGED"] == "same"
+
+
+def test_absent_account_uses_copied_parent_environment(body_file: Path) -> None:
+    command_runner = Mock(side_effect=[_completion([]), _completion([])])
+    parent_environment = {"GITHUB_TOKEN": "parent-setting", "UNCHANGED": "same"}
+    assert _run_comment(body_file, command_runner, parent_environment) == 0
+    action_environment = command_runner.call_args_list[1].kwargs["env"]
+    assert action_environment == parent_environment
+    assert action_environment is not parent_environment
+    assert command_runner.call_count == 2
+
+
+def test_token_lookup_failure_prints_generic_safe_stderr(
+    body_file: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    account_text = _account_text(ACCOUNT_A)
+    command_runner = _account_runner(
+        auth_code=1, auth_stdout=account_text, auth_stderr=f"unsafe {account_text}"
+    )
+    exit_code = _run_comment(
+        body_file, command_runner, {"GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A}
+    )
+    captured = capsys.readouterr()
+    assert exit_code == 1
+    assert captured.err == "error: GitHub account lookup failed\n"
+    assert account_text not in captured.out + captured.err
+    assert command_runner.call_count == 2
+    command_runner = Mock(side_effect=[_completion([]), OSError("unavailable")])
+    assert (
+        _run_comment(body_file, command_runner, {"GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A})
+        == 1
+    )
+    assert capsys.readouterr().err == "error: GitHub account lookup failed\n"
+
+
+def test_empty_account_token_stops_before_action(
+    body_file: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    command_runner = _account_runner(auth_stdout=" \n")
+    exit_code = _run_comment(
+        body_file, command_runner, {"GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A}
+    )
+    captured = capsys.readouterr()
+    assert exit_code == 1
+    assert captured.err == "error: GitHub account lookup returned no value\n"
+    assert command_runner.call_count == 2
+
+
+def test_action_child_results_pass_through_or_report_safely(
+    body_file: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    command_runner = Mock(side_effect=[_completion([]), _completion([], 7)])
+    assert _run_comment(body_file, command_runner, {}) == 7
+    command_runner = Mock(side_effect=[_completion([]), OSError("unavailable")])
+    assert _run_comment(body_file, command_runner, {}) == 1
+    assert capsys.readouterr().err == "error: GitHub action failed\n"
+
+
+def test_supplied_parent_environment_remains_unchanged(body_file: Path) -> None:
+    command_runner = _account_runner(auth_stdout=_account_text(ACCOUNT_A))
+    parent_environment = {
+        "GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A,
+        "GITHUB_TOKEN": "parent-setting",
+        "GH_TOKEN": "parent-setting",
+    }
+    original_environment = dict(parent_environment)
+    assert _run_comment(body_file, command_runner, parent_environment) == 0
+    assert parent_environment == original_environment
+
+
+def test_process_environment_remains_unchanged(
+    body_file: Path, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    monkeypatch.setenv("GITHUB_DEFAULT_ACCOUNT", ACCOUNT_A)
+    monkeypatch.setenv("GITHUB_TOKEN", "parent-setting")
+    monkeypatch.setenv("GH_TOKEN", "parent-setting")
+    original_environment = dict(os.environ)
+    command_runner = _account_runner(auth_stdout=_account_text(ACCOUNT_A))
+    exit_code = pull_request.main(
+        _comment_arguments(body_file),
+        all_environment=os.environ,
+        command_runner=command_runner,
+    )
+    assert exit_code == 0
+    assert dict(os.environ) == original_environment
+
+
+def _concurrent_runner(
+    all_environments: list[dict[str, str]],
+    all_commands: list[list[str]],
+    record_lock: threading.Lock,
+) -> Callable[..., subprocess.CompletedProcess[str]]:
+    def run_command(
+        all_arguments: list[str], **keyword_arguments: object
+    ) -> subprocess.CompletedProcess[str]:
+        with record_lock:
+            all_commands.append(all_arguments)
+        if all_arguments[0] == sys.executable:
+            return _completion(all_arguments)
+        if all_arguments[:3] == ["gh", "auth", "token"]:
+            if all_arguments[-1] == ACCOUNT_A:
+                return _completion(all_arguments, stdout=_account_text(ACCOUNT_A))
+            return _completion(all_arguments, 1)
+        child_environment = keyword_arguments["env"]
+        assert isinstance(child_environment, dict)
+        with record_lock:
+            all_environments.append(dict(child_environment))
+        return _completion(all_arguments)
+
+    return run_command
+
+
+def test_concurrent_accounts_keep_tokens_isolated(body_file: Path) -> None:
+    all_environments: list[dict[str, str]] = []
+    all_commands: list[list[str]] = []
+    command_runner = _concurrent_runner(
+        all_environments, all_commands, threading.Lock()
+    )
+    with ThreadPoolExecutor(max_workers=2) as executor:
+        first_run = executor.submit(
+            pull_request.main,
+            _comment_arguments(body_file),
+            all_environment={"GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A},
+            command_runner=command_runner,
+        )
+        second_run = executor.submit(
+            pull_request.main,
+            _comment_arguments(body_file),
+            all_environment={"GITHUB_DEFAULT_ACCOUNT": ACCOUNT_B},
+            command_runner=command_runner,
+        )
+    assert {first_run.result(), second_run.result()} == {0, 1}
+    assert len(all_environments) == 1
+    assert all_environments[0]["GH_TOKEN"] == _account_text(ACCOUNT_A)
+    assert all(
+        each_command[:3] != ["gh", "auth", "switch"] for each_command in all_commands
+    )
+
+
+def test_token_never_reaches_captured_output(
+    body_file: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    account_text = _account_text(ACCOUNT_A)
+    command_runner = _account_runner(auth_stdout=f"{account_text}\n")
+    assert (
+        _run_comment(body_file, command_runner, {"GITHUB_DEFAULT_ACCOUNT": ACCOUNT_A})
+        == 0
+    )
+    captured = capsys.readouterr()
+    assert account_text not in captured.out + captured.err
+
+
+def test_cli_rejects_inline_body(capsys: pytest.CaptureFixture[str]) -> None:
+    command_runner = Mock()
+    with pytest.raises(SystemExit) as raised_exit:
+        pull_request.main(
+            [*COMMENT_CLI[:-1], "--body", "Inline body text."],
+            all_environment={},
+            command_runner=command_runner,
+        )
+    captured = capsys.readouterr()
+    assert raised_exit.value.code == 2
+    assert "--body-file" in captured.err
+    command_runner.assert_not_called()

package/.agents/skills/pull-request/scripts/test_recover_legacy_author.py

@@ -0,0 +1,221 @@
+"""Contract tests for explicit legacy GitHub author recovery."""
+
+from __future__ import annotations
+
+import json
+import os
+import stat
+import subprocess
+import sys
+from collections.abc import Callable
+from pathlib import Path
+from unittest.mock import Mock
+
+import pytest
+
+_SCRIPTS_DIRECTORY = Path(__file__).resolve().parent
+if str(_SCRIPTS_DIRECTORY) not in sys.path:
+    sys.path.insert(0, str(_SCRIPTS_DIRECTORY))
+
+import recover_legacy_author
+
+CURRENT_TIME_SECONDS = 2_000_000.0
+CURRENT_USER_ID = 41
+LEGACY_ACCOUNT = "previous-account"
+OTHER_ACCOUNT = "other-account"
+STATE_FILENAME = "gh_pr_author_swap_session-a.json"
+
+
+def _completion(
+    code: int = 0, stdout: str = "", stderr: str = ""
+) -> subprocess.CompletedProcess[str]:
+    return subprocess.CompletedProcess([], code, stdout, stderr)
+
+
+def _write_state(
+    directory: Path,
+    *,
+    filename: str = STATE_FILENAME,
+    payload: object | None = None,
+    age_seconds: float = 1801.0,
+) -> Path:
+    state_file = directory / filename
+    state_payload = {"original_account": LEGACY_ACCOUNT} if payload is None else payload
+    state_file.write_text(json.dumps(state_payload), encoding="utf-8")
+    os.utime(
+        state_file,
+        (CURRENT_TIME_SECONDS - age_seconds, CURRENT_TIME_SECONDS - age_seconds),
+    )
+    if hasattr(os, "getuid"):
+        state_file.chmod(0o600)
+    return state_file
+
+
+def _run(
+    state_file: Path,
+    command_runner: Callable[..., subprocess.CompletedProcess[str]],
+    *,
+    confirm_inactive: bool = True,
+) -> int:
+    all_arguments = [str(state_file)]
+    if confirm_inactive:
+        all_arguments.append("--confirm-inactive")
+    return recover_legacy_author.main(
+        all_arguments,
+        now_seconds=CURRENT_TIME_SECONDS,
+        current_user_id=os.getuid() if hasattr(os, "getuid") else None,
+        command_runner=command_runner,
+    )
+
+
+def test_success_restores_selected_record_and_preserves_neighbor(
+    tmp_path: Path,
+) -> None:
+    selected_file = _write_state(tmp_path)
+    neighbor_file = _write_state(
+        tmp_path,
+        filename="gh_pr_author_swap_session-b.json",
+        payload={"original_account": OTHER_ACCOUNT},
+    )
+    command_runner = Mock(return_value=_completion())
+    assert _run(selected_file, command_runner) == 0
+    command_runner.assert_called_once_with(
+        ["gh", "auth", "switch", "--user", LEGACY_ACCOUNT],
+        capture_output=True,
+        check=False,
+        shell=False,
+        text=True,
+    )
+    assert not selected_file.exists()
+    assert neighbor_file.exists()
+
+
+def test_unconfirmed_record_is_unresolved_without_side_effects(
+    tmp_path: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    state_file = _write_state(tmp_path)
+    command_runner = Mock()
+    assert _run(state_file, command_runner, confirm_inactive=False) == 3
+    assert "confirm inactive" in capsys.readouterr().err
+    command_runner.assert_not_called()
+    assert state_file.exists()
+
+
+def test_active_record_is_rejected_without_side_effects(
+    tmp_path: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    state_file = _write_state(tmp_path, age_seconds=10.0)
+    command_runner = Mock()
+    assert _run(state_file, command_runner) == 2
+    assert "record is active" in capsys.readouterr().err
+    command_runner.assert_not_called()
+    assert state_file.exists()
+
+
+@pytest.mark.parametrize(
+    "payload",
+    [
+        [],
+        {},
+        {"original_account": ""},
+        {"original_account": 7},
+        {"original_account": LEGACY_ACCOUNT, "extra": True},
+    ],
+)
+def test_invalid_schema_is_rejected(
+    tmp_path: Path, payload: object, capsys: pytest.CaptureFixture[str]
+) -> None:
+    state_file = _write_state(tmp_path, payload=payload)
+    command_runner = Mock()
+    assert _run(state_file, command_runner) == 2
+    assert "record is invalid" in capsys.readouterr().err
+    command_runner.assert_not_called()
+    assert state_file.exists()
+
+
+def test_invalid_utf8_is_rejected(tmp_path: Path) -> None:
+    state_file = _write_state(tmp_path)
+    state_file.write_bytes(b"\xff")
+    os.utime(state_file, (CURRENT_TIME_SECONDS - 1801, CURRENT_TIME_SECONDS - 1801))
+    command_runner = Mock()
+    assert _run(state_file, command_runner) == 2
+    command_runner.assert_not_called()
+    assert state_file.exists()
+
+
+def test_wrong_filename_is_rejected(tmp_path: Path) -> None:
+    state_file = _write_state(tmp_path, filename="other.json")
+    command_runner = Mock()
+    assert _run(state_file, command_runner) == 2
+    command_runner.assert_not_called()
+
+
+def test_symlink_is_rejected(tmp_path: Path) -> None:
+    target_file = _write_state(tmp_path)
+    link_file = tmp_path / "gh_pr_author_swap_link.json"
+    try:
+        link_file.symlink_to(target_file)
+    except OSError as error:
+        pytest.skip(f"symlink unavailable: {error}")
+    command_runner = Mock()
+    assert _run(link_file, command_runner) == 2
+    command_runner.assert_not_called()
+    assert target_file.exists()
+
+
+def test_failed_restore_preserves_record_and_hides_child_output(
+    tmp_path: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    state_file = _write_state(tmp_path)
+    command_runner = Mock(
+        return_value=_completion(1, LEGACY_ACCOUNT, f"unsafe {LEGACY_ACCOUNT}")
+    )
+    assert _run(state_file, command_runner) == 1
+    captured = capsys.readouterr()
+    assert captured.err == "error: GitHub account restore failed\n"
+    assert LEGACY_ACCOUNT not in captured.out + captured.err
+    assert state_file.exists()
+
+
+def test_restore_launch_failure_preserves_record(
+    tmp_path: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    state_file = _write_state(tmp_path)
+    command_runner = Mock(side_effect=OSError("unavailable"))
+    assert _run(state_file, command_runner) == 1
+    assert capsys.readouterr().err == "error: GitHub account restore failed\n"
+    assert state_file.exists()
+
+
+def test_changed_record_after_restore_is_preserved(
+    tmp_path: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    state_file = _write_state(tmp_path)
+
+    def change_record(
+        all_arguments: list[str], **keyword_arguments: object
+    ) -> subprocess.CompletedProcess[str]:
+        state_file.write_text(
+            json.dumps({"original_account": OTHER_ACCOUNT}), encoding="utf-8"
+        )
+        return _completion()
+
+    assert _run(state_file, change_record) == 1
+    assert "record changed" in capsys.readouterr().err
+    assert state_file.exists()
+
+
+def test_secure_posix_metadata_requires_regular_owner_mode() -> None:
+    regular_secure_mode = stat.S_IFREG | 0o600
+    assert recover_legacy_author._posix_metadata_is_secure(
+        regular_secure_mode, CURRENT_USER_ID, CURRENT_USER_ID
+    )
+    assert not recover_legacy_author._posix_metadata_is_secure(
+        regular_secure_mode, CURRENT_USER_ID + 1, CURRENT_USER_ID
+    )
+    assert not recover_legacy_author._posix_metadata_is_secure(
+        stat.S_IFREG | 0o644, CURRENT_USER_ID, CURRENT_USER_ID
+    )
+    assert not recover_legacy_author._posix_metadata_is_secure(
+        stat.S_IFLNK | 0o600, CURRENT_USER_ID, CURRENT_USER_ID
+    )

package/.agents/skills/test-runner/SKILL.md

@@ -0,0 +1,69 @@
+---
+name: test-runner
+description: >-
+  Run pytest or Playwright through the repository's test command.
+  Checks Django and Playwright readiness before launching the
+  selected child process. Use when running tests in a repository with local
+  Django or frontend services.
+---
+
+# test-runner
+
+Run Django and Playwright readiness checks, then launch the selected test
+command. Pass one supported runner after `--`. The command forwards those
+arguments without a shell.
+
+## Command
+
+Run from the repository that owns the tests:
+
+```text
+python "${CLAUDE_SKILL_DIR}/scripts/run_tests.py" -- python -m pytest <pytest-arguments>
+python "${CLAUDE_SKILL_DIR}/scripts/run_tests.py" -- npx playwright test <playwright-arguments>
+```
+
+Use `--project <path>` when the test repository is not the current directory:
+
+```text
+python "${CLAUDE_SKILL_DIR}/scripts/run_tests.py" --project "<repository>" -- python -m pytest <pytest-arguments>
+```
+
+The supported runner forms are `pytest`, `python -m pytest`, `playwright test`,
+and `npx playwright test`. The child receives the exact arguments after `--`,
+inherits the current environment, writes to the current output streams, runs
+with the selected project as its working directory, and returns its exit
+status.
+
+## Preflight
+
+The command returns nonzero and launches no child when a required check fails.
+
+- A pytest run in a project without `manage.py` runs without preflight.
+- A pytest run in a Django project requires `db.sqlite3` and a healthy Django
+  server. The URL defaults to `http://localhost:8000`. A URL in the child
+  arguments overrides that default.
+- A Playwright run checks its target server, validates a detected Django
+  `runserver` process uses `--test-db`, rejects conflicting Django servers from
+  different worktrees, and uses the URL default `http://localhost:3000`.
+- When the project contains `frontend/`, the command runs `npm run build` in
+  that directory, then `python manage.py collectstatic --noinput` in the
+  project directory before Playwright starts.
+
+The command does not parse shell syntax. Keep pipes, redirects, `cd`, and
+background operators outside this command. Run this command directly so its
+exit status reaches the caller.
+
+## Unsupported commands
+
+Use the normal runner for anything else. This command accepts only the four
+runner forms listed above. It does not install dependencies, start a server,
+run migrations, or repair a failed preflight.
+
+## Files
+
+- `SKILL.md` covers invocation, checks, and the command boundary.
+- `scripts/run_tests.py` parses the command, runs preflight, and launches the child.
+- `scripts/preflight_checks.py` runs Django and Playwright readiness checks.
+- `scripts/test_run_tests.py` covers the command and preflight.
+- `scripts/test_preflight_checks.py` covers the readiness checks.
+- `scripts/test_runner_constants/` holds command names, messages, and limits.