ci: fix wheel building

.github/workflows/pypi.yml

@@ -4,17 +4,18 @@ on:
   push:
     branches: [main]
     tags: [v*]
+  pull_request:
 
 jobs:
-  test:
-    uses: pyapp-kit/workflows/.github/workflows/test-pyrepo.yml@v2
-    with:
-      hatch-build-hooks-enable: true
-      coverage-upload: none
+  # test:
+  #   uses: pyapp-kit/workflows/.github/workflows/test-pyrepo.yml@v2
+  #   with:
+  #     hatch-build-hooks-enable: true
+  #     coverage-upload: none
 
   deploy-sdist:
-    needs: test
-    if: success() && startsWith(github.ref, 'refs/tags/v')
+    # needs: test
+    # if: success() && startsWith(github.ref, 'refs/tags/v')
     runs-on: ubuntu-latest
     permissions:
       id-token: write
@@ -34,36 +35,26 @@ jobs:
           python -m pip install build
           python -m build
 
-      - name: 🚢 Publish to PyPI
-        uses: pypa/gh-action-pypi-publish@release/v1
+      # - name: 🚢 Publish to PyPI
+      #   uses: pypa/gh-action-pypi-publish@release/v1
 
-      - uses: softprops/action-gh-release@v2
-        with:
-          generate_release_notes: true
-          files: "./dist/*"
+      # - uses: softprops/action-gh-release@v2
+      #   with:
+      #     generate_release_notes: true
+      #     files: "./dist/*"
 
   deploy-wheel:
-    if: startsWith(github.event.ref, 'refs/tags/v')
-    needs: deploy-sdist
-    name: mypyc wheels (${{ matrix.name }})
+    # if: startsWith(github.event.ref, 'refs/tags/v')
+    # needs: deploy-sdist
+    name: mypyc wheels (${{ matrix.os }})
     runs-on: ${{ matrix.os }}
     permissions:
       id-token: write
       contents: write
     strategy:
       fail-fast: false
       matrix:
-        include:
-          - os: ubuntu-latest
-            name: linux-x86_64
-          - os: windows-latest
-            name: windows-amd64
-          - os: macos-13
-            name: macos-x86_64
-            macos_arch: "x86_64"
-          - os: macos-14
-            name: macos-arm64
-            macos_arch: "arm64"
+        os: [ubuntu-latest, windows-latest, macos-13, macos-latest]
 
     steps:
       - uses: actions/checkout@v4
@@ -74,14 +65,12 @@ jobs:
         uses: pypa/[email protected]
         with:
           output-dir: dist
-        env:
-          CIBW_ARCHS_MACOS: "${{ matrix.macos_arch }}"
 
       - name: Upload wheels as workflow artifacts
         uses: actions/upload-artifact@v4
         with:
-          name: ${{ matrix.name }}-mypyc-wheels
+          name: ${{ runner.os }}-${{ runner.arch }}-mypyc-wheels
           path: ./dist/*.whl
 
-      - name: 🚢 Publish to PyPI
-        uses: pypa/gh-action-pypi-publish@release/v1
+      # - name: 🚢 Publish to PyPI
+      #   uses: pypa/gh-action-pypi-publish@release/v1

pyproject.toml

@@ -32,17 +32,6 @@ dependencies = [
 # extras
 # https://peps.python.org/pep-0621/#dependencies-optional-dependencies
 [project.optional-dependencies]
-dev = [
-    "ipython",
-    "mypy",
-    "mypy_extensions",
-    "pre-commit",
-    "PyQt5",
-    "pytest-mypy-plugins",
-    "rich",
-    "ruff",
-    "typing-extensions",
-]
 docs = [
     "griffe==0.25.5",
     "mkdocs-material==8.5.10",
@@ -55,19 +44,32 @@ docs = [
 proxy = ["wrapt"]
 pydantic = ["pydantic"]
 test = [
-    "dask",
+    "dask[array]>=2024.0.0",
     "attrs",
-    "numpy",
+    "numpy >1.21.6",
     "pydantic",
     "pyinstaller>=4.0",
     "pytest>=6.0",
     "pytest-cov",
     "wrapt",
-    "msgspec; python_version < '3.13'",
+    "msgspec",
     "toolz",
 ]
 testqt = ["pytest-qt", "qtpy"]
 
+dev = [
+    "psygnal[test, testqt]",
+    "PyQt6",
+    "ipython",
+    "mypy",
+    "mypy_extensions",
+    "pre-commit",
+    "pytest-mypy-plugins",
+    "rich",
+    "ruff",
+    "typing-extensions",
+]
+
 [project.urls]
 homepage = "https://github.com/pyapp-kit/psygnal"
 repository = "https://github.com/pyapp-kit/psygnal"
@@ -118,6 +120,10 @@ test-extras = ["test"]
 test-command = "pytest {project}/tests -v"
 test-skip = ["*-musllinux*", "cp312-win*", "*-macosx_arm64"]
 
+[tool.cibuildwheel.macos]
+# https://cibuildwheel.readthedocs.io/en/stable/faq/#apple-silicon
+archs = ["x86_64", "arm64"]
+
 [[tool.cibuildwheel.overrides]]
 select = "*-manylinux_i686*"
 before-all = "yum install -y python3-devel"

src/psygnal/_signal.py

@@ -128,7 +128,6 @@ def ensure_at_least_20(val: int):
 from __future__ import annotations
 
 import inspect
-import sys
 import threading
 import warnings
 import weakref
@@ -182,7 +181,14 @@ def ensure_at_least_20(val: int):
 
 _NULL = object()
 F = TypeVar("F", bound=Callable)
-RECURSION_LIMIT = sys.getrecursionlimit()
+
+# using 300 instead of sys.getrecursionlimit()
+# in a mypyc-compiled program, hitting an actual RecursionError can cause
+# a segfault (rather than raise a python exception), so we really MUST
+# avoid it.  Windows has a lower stack limit than other platforms, so we
+# use 300 as a cross-platform "safe" limit, determined via testing.  It's
+# probably plenty large for most reasonable use-cases.
+RECURSION_LIMIT = 300
 
 ReemissionVal = Literal["immediate", "queued", "latest-only"]
 VALID_REEMISSION = set(ReemissionVal.__args__)  # type: ignore
@@ -1223,6 +1229,13 @@ def emit_fast(self, *args: Any) -> None:
             self._args_queue.append(args)
             return
 
+        if self._recursion_depth >= RECURSION_LIMIT:
+            raise RecursionError(
+                f"Psygnal recursion limit ({RECURSION_LIMIT}) reached when emitting "
+                f"signal {self.name!r} with args {args}"
+            )
+
+        self._recursion_depth += 1
         try:
             for caller in self._slots:
                 caller.cb(args)
@@ -1238,6 +1251,9 @@ def emit_fast(self, *args: Any) -> None:
             )
             # this comment will show up in the traceback
             raise loop_err from cb_err  # emit() call ABOVE || callback error BELOW
+        finally:
+            if self._recursion_depth > 0:
+                self._recursion_depth -= 1
 
     def __call__(
         self, *args: Any, check_nargs: bool = False, check_types: bool = False
@@ -1246,6 +1262,9 @@ def __call__(
         return self.emit(*args, check_nargs=check_nargs, check_types=check_types)
 
     def _run_emit_loop(self, args: tuple[Any, ...]) -> None:
+        if self._recursion_depth >= RECURSION_LIMIT:
+            raise RecursionError("Recursion limit reached!")
+
         with self._lock:
             self._emit_queue.append(args)
             if len(self._emit_queue) > 1: