Merge branch 'main' into eliminate-unbounded-queue

Author: CoolCat467

.github/workflows/autodeps.yml

@@ -28,8 +28,8 @@ jobs:
         run: |
           python -m pip install -U pip pre-commit
           python -m pip install -r test-requirements.txt
-          uv pip compile --no-strip-markers --python-version=3.8 --upgrade test-requirements.in -o test-requirements.txt
-          uv pip compile --no-strip-markers --python-version=3.8 --upgrade docs-requirements.in -o docs-requirements.txt
+          uv pip compile --universal --python-version=3.8 --upgrade test-requirements.in -o test-requirements.txt
+          uv pip compile --universal --python-version=3.8 --upgrade docs-requirements.in -o docs-requirements.txt
           pre-commit autoupdate --jobs 0
       - name: Black
         run: |

.pre-commit-config.yaml

@@ -23,7 +23,7 @@ repos:
     hooks:
       - id: black
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.5.1
+    rev: v0.5.5
     hooks:
       - id: ruff
         types: [file]

check.sh

@@ -78,10 +78,10 @@ fi
 
 # Check pip compile is consistent
 echo "::group::Pip Compile - Tests"
-uv pip compile --no-strip-markers --python-version=3.8 test-requirements.in -o test-requirements.txt
+uv pip compile --universal --python-version=3.8 test-requirements.in -o test-requirements.txt
 echo "::endgroup::"
 echo "::group::Pip Compile - Docs"
-uv pip compile --no-strip-markers --python-version=3.8 docs-requirements.in -o docs-requirements.txt
+uv pip compile --universal --python-version=3.8 docs-requirements.in -o docs-requirements.txt
 echo "::endgroup::"
 
 if git status --porcelain | grep -q "requirements.txt"; then

docs-requirements.txt

@@ -1,5 +1,5 @@
 # This file was autogenerated by uv via the following command:
-#    uv pip compile --no-strip-markers --python-version=3.8 docs-requirements.in -o docs-requirements.txt
+#    uv pip compile --universal --python-version=3.8 docs-requirements.in -o docs-requirements.txt
 alabaster==0.7.13
     # via sphinx
 attrs==23.2.0
@@ -12,12 +12,18 @@ beautifulsoup4==4.12.3
     # via sphinx-codeautolink
 certifi==2024.7.4
     # via requests
-cffi==1.16.0 ; platform_python_implementation != 'PyPy'
-    # via cryptography
+cffi==1.16.0 ; os_name == 'nt' or platform_python_implementation != 'PyPy'
+    # via
+    #   -r docs-requirements.in
+    #   cryptography
 charset-normalizer==3.3.2
     # via requests
 click==8.1.7
     # via towncrier
+colorama==0.4.6 ; platform_system == 'Windows' or sys_platform == 'win32'
+    # via
+    #   click
+    #   sphinx
 cryptography==42.0.8
     # via pyopenssl
 docutils==0.20.1
@@ -51,7 +57,7 @@ outcome==1.3.0.post0
     # via -r docs-requirements.in
 packaging==24.1
     # via sphinx
-pycparser==2.22 ; platform_python_implementation != 'PyPy'
+pycparser==2.22 ; os_name == 'nt' or platform_python_implementation != 'PyPy'
     # via cffi
 pygments==2.18.0
     # via sphinx

src/trio/_core/_ki.py

@@ -143,6 +143,7 @@ def decorator(fn: Callable[ArgsT, RetT]) -> Callable[ArgsT, RetT]:
             def wrapper(*args: ArgsT.args, **kwargs: ArgsT.kwargs) -> RetT:  # type: ignore[misc]
                 # See the comment for regular generators below
                 coro = fn(*args, **kwargs)
+                assert coro.cr_frame is not None, "Coroutine frame should exist"
                 coro.cr_frame.f_locals[LOCALS_KEY_KI_PROTECTION_ENABLED] = enabled
                 return coro  # type: ignore[return-value]
 

src/trio/_core/_run.py

@@ -1782,6 +1782,7 @@ async def python_wrapper(orig_coro: Awaitable[RetT]) -> RetT:
                 return await orig_coro
 
             coro = python_wrapper(coro)
+        assert coro.cr_frame is not None, "Coroutine frame should exist"
         coro.cr_frame.f_locals.setdefault(LOCALS_KEY_KI_PROTECTION_ENABLED, system_task)
 
         ######

src/trio/_core/_tests/test_run.py

@@ -2704,7 +2704,7 @@ async def start_raiser() -> None:
     else:
         # Check attributes, not identity, because should_be_raiser_exc may be a
         # copy of raiser_exc rather than raiser_exc by identity.
-        assert type(should_be_raiser_exc) == type(raiser_exc)
+        assert type(should_be_raiser_exc) is type(raiser_exc)
         assert should_be_raiser_exc.message == raiser_exc.message
         assert should_be_raiser_exc.exceptions == raiser_exc.exceptions
 

src/trio/_socket.py

@@ -528,7 +528,7 @@ def __init__(self) -> None:
         # make sure this __init__ works with multiple inheritance
         super().__init__()
         # and only raises error if it's directly constructed
-        if type(self) == SocketType:
+        if type(self) is SocketType:
             raise TypeError(
                 "SocketType is an abstract class; use trio.socket.socket if you "
                 "want to construct a socket object"

src/trio/_tests/test_util.py

@@ -269,7 +269,7 @@ def test_fixup_module_metadata() -> None:
     assert mod.SomeClass.method.__module__ == "trio.somemodule"  # type: ignore[attr-defined]
     assert mod.SomeClass.method.__qualname__ == "SomeClass.method"  # type: ignore[attr-defined]
     # Make coverage happy.
-    non_trio_module.some_func()  # type: ignore[no-untyped-call]
-    mod.some_func()  # type: ignore[no-untyped-call]
-    mod._private()  # type: ignore[no-untyped-call]
+    non_trio_module.some_func()
+    mod.some_func()
+    mod._private()
     mod.SomeClass().method()

test-requirements.in

@@ -15,7 +15,7 @@ mypy; implementation_name == "cpython"
 types-pyOpenSSL; implementation_name == "cpython" # and annotations
 ruff >= 0.4.3
 astor          # code generation
-uv >= 0.2.22
+uv >= 0.2.24
 codespell
 
 # https://github.com/python-trio/trio/pull/654#issuecomment-420518745