Strip ANSI + accept `_create(...)` in devx tests

Two version-compat fixes for the `devx` debugger
test-suite, both about matching upstream output that
got more verbose w/ recent lib releases.

ANSI stripping (`tests/devx/conftest.py`),
- Add `ansi_strip(text)` helper + `_ansi_re` pattern
  (regex per https://stackoverflow.com/a/14693789).
- Apply inside `in_prompt_msg()` + `assert_before()` so
  substring matches against REPL/traceback output stay
  robust to color leakage.
- Motivated by py3.13's colored tracebacks +
  `pdbp`/pygments highlighting leaking ANSI even when
  `PYTHON_COLORS=0` is set in the `spawn` fixture (not
  every renderer in the spawned subproc honors it).
- Replaces the longstanding inline TODO that linked
  the SO answer w/o impl'ing.

trio 0.30+ `Cancelled._create(` match (`test_debugger`),
- In `test_shield_pause` swap the two
  `"raise Cancelled._create()"` assertion patterns →
  `"raise Cancelled._create("` (open-paren form, no
  closing).
- trio >=0.30 raises a multi-line
  `raise Cancelled._create(source=.., reason=..,
  source_task=..)` w/ cancel-reason metadata, so the
  legacy bare-`()` form no longer matches. Inline
  comment documents the trio-version pivot.

(this commit msg was generated in some part by [`claude-code`][claude-code-gh])
[claude-code-gh]: https://github.com/anthropics/claude-code

(cherry picked from commit 3854cf5ecb)
(cherry picked from commit c07cf2546b)

pyproject.toml

@@ -104,7 +104,7 @@ testing = [
 repl = [
   "pyperclip>=1.9.0",
   "prompt-toolkit>=3.0.50",
-  "xonsh>=0.23.8",
+  "xonsh>=0.23.0",
   "psutil>=7.0.0",
 ]
 lint = [

tests/devx/test_proctitle.py

@@ -8,9 +8,9 @@ after `Actor` construction, so any spawned sub-actor process
 should:
 
   - have `argv[0]` (== `/proc/<pid>/cmdline`) start with
-    `<_def_prefix>[<aid.reprol()>]` (currently `_subactor[…]`)
-  - have `/proc/<pid>/comm` start with `<_def_prefix>[`
-    (kernel truncates to ~15 bytes)
+    `tractor[<aid.reprol()>]`
+  - have `/proc/<pid>/comm` start with `tractor[` (kernel
+    truncates to ~15 bytes)
   - be detected as a tractor sub-actor by
     `_is_tractor_subactor(pid)` via the cmdline marker.
 
@@ -27,10 +27,7 @@ import trio
 import tractor
 
 from tractor.runtime._runtime import Actor
-from tractor.devx._proctitle import (
-    set_actor_proctitle,
-    _def_prefix,
-)
+from tractor.devx._proctitle import set_actor_proctitle
 from tractor._testing._reap import (
     _is_tractor_subactor,
     _read_cmdline,
@@ -44,9 +41,8 @@ _non_linux: bool = platform.system() != 'Linux'
 def test_set_actor_proctitle_format():
     '''
     `set_actor_proctitle()` returns the canonical
-    `<_def_prefix>[<aid.reprol()>]` form (currently
-    `_subactor[…]`) and actually mutates the running
-    proc's title.
+    `tractor[<aid.reprol()>]` form and actually mutates
+    the running proc's title.
 
     '''
     pytest.importorskip(
@@ -64,14 +60,12 @@ def test_set_actor_proctitle_format():
         )
         title: str = set_actor_proctitle(actor)
 
-        # canonical wrapping: `<_def_prefix>[<aid.reprol()>]`.
-        # We source BOTH the prefix (`_def_prefix`) and the
-        # runtime-computed `reprol()` rather than hard-coding,
-        # so the test stays decoupled from the prefix shape
-        # (flipped to `_subactor` in `3a45dbd5`) AND from
-        # `Aid.reprol()`'s internal format (currently
-        # `<name>@<pid>`, but could evolve).
-        expected: str = f'{_def_prefix}[{actor.aid.reprol()}]'
+        # canonical wrapping: `tractor[<aid.reprol()>]`. We
+        # compare against the runtime-computed `reprol()`
+        # rather than a hard-coded value so the test stays
+        # decoupled from `Aid.reprol()`'s internal format
+        # (currently `<name>@<pid>`, but could evolve).
+        expected: str = f'tractor[{actor.aid.reprol()}]'
         assert title == expected
         # sanity: the actor's name must be in the title
         # somewhere (so a future `reprol()` change that
@@ -146,17 +140,15 @@ def test_subactor_proctitle_visible_via_proc():
     )
 
     pid, info = matched[0]
-    # canonical proctitle prefix in cmdline (full form);
-    # prefix sourced from `_def_prefix` so it tracks the
-    # `3a45dbd5` flip (`tractor[` -> `_subactor[`).
-    assert info['cmdline'].startswith(f'{_def_prefix}[proctitle_boi@'), (
-        f'cmdline missing `{_def_prefix}[proctitle_boi@…]` prefix: '
+    # canonical proctitle prefix in cmdline (full form)
+    assert info['cmdline'].startswith('tractor[proctitle_boi@'), (
+        f'cmdline missing `tractor[proctitle_boi@…]` prefix: '
         f'{info["cmdline"]!r}'
     )
     # comm is kernel-truncated to ~15 bytes — just check the
-    # `<_def_prefix>[` prefix made it.
-    assert info['comm'].startswith(f'{_def_prefix}['), (
-        f'comm missing `{_def_prefix}[` prefix: {info["comm"]!r}'
+    # `tractor[` prefix made it.
+    assert info['comm'].startswith('tractor['), (
+        f'comm missing `tractor[` prefix: {info["comm"]!r}'
     )
     # intrinsic-signal detector should match.
     assert info['is_tractor'] is True

tests/test_log_sys.py

@@ -162,66 +162,6 @@ def test_implicit_mod_name_applied_for_child(
     assert submod.log.logger in sub_logs
 
 
-def test_io_custom_level_registered():
-    '''
-    The `IO`(21) level (registered via `add_log_level()` at
-    import, for `tractor.trionics._subproc`'s std-stream relay)
-    is fully wired and SHOWN BY DEFAULT at `info`-level consoles
-    since `21 >= INFO(20)`.
-
-    '''
-    import logging
-    assert log.CUSTOM_LEVELS.get('IO') == 21
-    assert logging.getLevelName(21) == 'IO'
-    assert log.STD_PALETTE.get('IO')
-    assert log.BOLD_PALETTE['bold'].get('IO')
-
-    iolog = log.get_logger('io_lvl_test')
-    assert callable(getattr(iolog, 'io', None))
-    # emit must not raise
-    iolog.io('hello from the IO level')
-
-    # 21 >= INFO(20) -> shown when console set to `info`
-    assert 21 >= logging.INFO
-
-
-def test_add_log_level_pluggable():
-    '''
-    `add_log_level()` is the single pluggable entry-point: one
-    call wires `CUSTOM_LEVELS` + `addLevelName` + both palettes +
-    a same-named `StackLevelAdapter` emit method (so
-    `get_logger()`'s per-level audit passes).
-
-    '''
-    import logging
-    name: str = 'XLVL'
-    val: int = 19
-    try:
-        log.add_log_level(name, val, 'cyan')
-
-        assert log.CUSTOM_LEVELS[name] == val
-        assert logging.getLevelName(val) == name
-        assert log.STD_PALETTE[name] == 'cyan'
-        assert log.BOLD_PALETTE['bold'][name] == 'bold_cyan'
-
-        # the audit in `get_logger()` (asserts a method per
-        # `CUSTOM_LEVELS` entry) must still pass.
-        xlog = log.get_logger('xlvl_test')
-        emit = getattr(xlog, name.lower(), None)
-        assert callable(emit)
-        emit('hello from a plugged-in level')
-
-    finally:
-        # best-effort cleanup of our module-global mutations so
-        # later `get_logger()` audits don't see a half-removed
-        # level.
-        log.CUSTOM_LEVELS.pop(name, None)
-        log.STD_PALETTE.pop(name, None)
-        log.BOLD_PALETTE['bold'].pop(name, None)
-        if hasattr(log.StackLevelAdapter, name.lower()):
-            delattr(log.StackLevelAdapter, name.lower())
-
-
 # TODO, moar tests against existing feats:
 # ------ - ------
 # - [ ] color settings?

tractor/_root.py

@@ -155,6 +155,7 @@ async def maybe_block_bp(
             os.environ.pop('PYTHONBREAKPOINT', None)
 
 
+
 @acm
 async def open_root_actor(
     *,
@@ -185,7 +186,6 @@ async def open_root_actor(
     # enables the multi-process debugger support
     debug_mode: bool = False,
     maybe_enable_greenback: bool = False,  # `.pause_from_sync()/breakpoint()` support
-
     # ^XXX NOTE^ the perf implications of use,
     # https://greenback.readthedocs.io/en/latest/principle.html#performance
     enable_stack_on_sig: bool = False,

tractor/_testing/_reap.py

@@ -90,6 +90,7 @@ keys are caller-defined).
 
 '''
 from __future__ import annotations
+
 import os
 import pathlib
 import re
@@ -98,9 +99,6 @@ import stat
 import sys
 import time
 
-
-from tractor.devx import _proctitle
-
 # `/dev/shm` is the POSIX-shm filesystem on Linux + FreeBSD.
 # macOS uses `shm_open` syscalls without a fs-visible path,
 # so the shm helpers refuse to run there.
@@ -232,9 +230,9 @@ def _read_comm(pid: int) -> str:
 #   while `cmdline` for zombies often reads as empty.
 _TRACTOR_PROC_CMDLINE_MARKERS: tuple[str, ...] = (
     'tractor._child',
-    _proctitle._def_prefix,
+    'tractor[',
 )
-_TRACTOR_PROC_COMM_MARKER: str = _proctitle._def_prefix
+_TRACTOR_PROC_COMM_MARKER: str = 'tractor['
 
 
 def _is_tractor_subactor(pid: int) -> bool:

tractor/devx/_proctitle.py

@@ -24,10 +24,7 @@ which" at a glance without needing to read full
 `/proc/<pid>/cmdline`.
 
 Format:
-  ``<_def_prefix>[<aid.reprol()>]``  e.g. ``_subactor[doggy@1027301b]``
-(prefix from the `_def_prefix` const, flipped `tractor` ->
-`_subactor` so sub-actor procs are visually distinct from the
-root in `ps`/`htop` and the reap-recognition markers.)
+  ``tractor[<aid.reprol()>]``    e.g. ``tractor[doggy@1027301b]``
 
 Uses the canonical `Aid.reprol()` form
 (``<name>@<uuid_short>``) so the proc-title matches the
@@ -55,13 +52,7 @@ except ImportError:
     _stp = None
 
 
-_def_prefix: str = '_subactor'
-
-
-def set_actor_proctitle(
-    actor: 'Actor',
-    prefix: str = _def_prefix,
-) -> str | None:
+def set_actor_proctitle(actor: 'Actor') -> str | None:
     '''
     Set the calling process's proc-title to identify it as a
     tractor sub-actor.
@@ -78,6 +69,6 @@ def set_actor_proctitle(
     if _stp is None:
         return None
 
-    title: str = f'{prefix}[{actor.aid.reprol()}]'
+    title: str = f'tractor[{actor.aid.reprol()}]'
     _stp.setproctitle(title)
     return title

tractor/log.py

@@ -262,63 +262,6 @@ class StackLevelAdapter(LoggerAdapter):
         )
 
 
-def add_log_level(
-    name: str,
-    value: int,
-    color: str = 'white',
-) -> None:
-    '''
-    Register a new custom log level with `tractor`'s logging
-    machinery in ONE call — the single pluggable entry-point that
-    keeps the (otherwise hand-synced) pieces consistent:
-
-    - `CUSTOM_LEVELS[name]` (drives the `stacklevel` bump in
-      `StackLevelAdapter.log()` + the `get_logger()` audit).
-    - `logging.addLevelName()` registration.
-    - `STD_PALETTE`/`BOLD_PALETTE` color entries (consumed when
-      `get_console_log()` builds its `ColoredFormatter`).
-    - a same-named (lowercase) emit method bound on
-      `StackLevelAdapter` so `log.<name>('msg')` works (and so
-      `get_logger()`'s per-level method audit passes).
-
-    Idempotent: re-registering an existing name is a no-op-ish
-    refresh (won't clobber an already-bound method).
-
-    '''
-    name_up: str = name.upper()
-    name_lo: str = name.lower()
-
-    CUSTOM_LEVELS[name_up] = value
-    logging.addLevelName(value, name_up)
-    STD_PALETTE[name_up] = color
-    BOLD_PALETTE['bold'][name_up] = f'bold_{color}'
-
-    if not hasattr(StackLevelAdapter, name_lo):
-        # bind via default-arg so `value` is captured (not
-        # late-bound); delegates to `.log()` exactly like the
-        # hand-written level methods above.
-        def _emit(
-            self,
-            msg: str,
-            *,
-            _level: int = value,
-        ) -> None:
-            return self.log(_level, msg)
-
-        _emit.__name__ = name_lo
-        _emit.__qualname__ = f'StackLevelAdapter.{name_lo}'
-        setattr(StackLevelAdapter, name_lo, _emit)
-
-
-# `IO`: child-subproc std-stream relay (see
-# `tractor.trionics._subproc`). Value 21 sits just ABOVE
-# `INFO`(20) so it's SHOWN BY DEFAULT at the usual `info`/`devx`
-# console levels (a `runtime`(15) relay would be silently
-# filtered) yet still distinctly labelled/colored + separately
-# filterable.
-add_log_level('IO', 21, 'purple')
-
-
 # TODO IDEAs:
 # -[ ] move to `.devx.pformat`?
 # -[ ] do per task-name and actor-name color coding

uv.lock

@@ -797,7 +797,7 @@ dev = [
     { name = "pytest-timeout", specifier = ">=2.3" },
     { name = "stackscope", specifier = ">=0.2.2,<0.3" },
     { name = "typing-extensions", specifier = ">=4.14.1" },
-    { name = "xonsh", specifier = ">=0.23.8" },
+    { name = "xonsh", specifier = ">=0.23.0" },
 ]
 devx = [
     { name = "stackscope", specifier = ">=0.2.2,<0.3" },
@@ -809,7 +809,7 @@ repl = [
     { name = "prompt-toolkit", specifier = ">=3.0.50" },
     { name = "psutil", specifier = ">=7.0.0" },
     { name = "pyperclip", specifier = ">=1.9.0" },
-    { name = "xonsh", specifier = ">=0.23.8" },
+    { name = "xonsh", specifier = ">=0.23.0" },
 ]
 subints = [{ name = "msgspec", marker = "python_full_version >= '3.14'", specifier = ">=0.21.0" }]
 sync-pause = [{ name = "greenback", marker = "python_full_version == '3.13.*'", specifier = ">=1.2.1,<2" }]

xontrib/tractor_diag.xsh

@@ -488,7 +488,6 @@ def _tractor_reap(args):
         reap,
         reap_shm,
         reap_uds,
-        _TRACTOR_PROC_CMDLINE_MARKERS,
     )
 
     rc: int = 0
@@ -501,8 +500,9 @@ def _tractor_reap(args):
         else:
             pids = find_orphans()
             mode = (
-                f'orphans (PPid==1, intrinsic '
-                f'cmdline/comm match — {_TRACTOR_PROC_CMDLINE_MARKERS}'
+                'orphans (PPid==1, intrinsic '
+                'cmdline/comm match — `tractor[…]` or '
+                '`tractor._child`)'
             )
 
         if not pids: