omlish 0.0.0.dev21__py3-none-any.whl → 0.0.0.dev23__py3-none-any.whl

omlish/__about__.py

@@ -1,5 +1,5 @@
-__version__ = '0.0.0.dev21'
-__revision__ = '4cb4af9e253f312d5f301449f08dfa7168234e07'
+__version__ = '0.0.0.dev23'
+__revision__ = '1d019b6879a9552ea31622e40f50d260d6b08824'
 
 
 #
@@ -33,10 +33,10 @@ class Project(ProjectBase):
             'anyio ~= 4.4',
             'sniffio ~= 1.3',
 
-            'greenlet ~= 3.0; python_version < "3.13"',
+            'greenlet ~= 3.1',
 
             'trio ~= 0.26',
-            'trio-asyncio ~= 0.15; python_version < "3.13"',
+            'trio-asyncio ~= 0.15',
         ],
 
         'compression': [
@@ -46,12 +46,15 @@ class Project(ProjectBase):
         ],
 
         'diag': [
+            'asttokens ~= 2.4',
+            'executing ~= 2.1',
+
             'psutil ~= 6.0',
         ],
 
         'formats': [
             'orjson ~= 3.10',
-            # 'ujson ~= 5.10',
+            'ujson ~= 5.10',
 
             'json5 ~= 0.9',
 
@@ -74,11 +77,14 @@ class Project(ProjectBase):
         ],
 
         'sql': [
-            'sqlalchemy ~= 2.0; python_version ~= "3.13"',
-            'sqlalchemy[asyncio] ~= 2.0; python_version < "3.13"',
+            'sqlalchemy[asyncio] ~= 2.0',
 
             'pg8000 ~= 1.31',
+            # 'psycopg2 ~= 2.9',
+
             'pymysql ~= 1.1',
+            # 'mysql-connector-python ~= 9.0',
+            # 'mysqlclient ~= 2.2',
 
             'aiomysql ~= 0.2',
             'aiosqlite ~= 0.20',
@@ -108,7 +114,10 @@ class SetuptoolsBase:
     include_package_data = False
 
     find_packages = {
-        'exclude': ['*.tests', '*.tests.*'],
+        'exclude': [
+            '*.tests',
+            '*.tests.*',
+        ],
     }
 
 

omlish/asyncs/bridge.py

@@ -8,6 +8,9 @@ KEEP THE SPACE SHUTTLE FLYING.
 
 TODO:
  - reuse greenlet if nested somehow?
+
+See:
+ - https://greenback.readthedocs.io/en/latest/
 """
 import itertools
 import sys
@@ -15,22 +18,21 @@ import types
 import typing as ta
 import weakref
 
+from .. import check
 from .. import lang
+from .. import sync
+from ..concurrent import threadlets
 from .asyncs import sync_await
 
 
 if ta.TYPE_CHECKING:
     import asyncio
 
-    import greenlet
-
     from . import anyio as aiu
 
 else:
     asyncio = lang.proxy_import('asyncio')
 
-    greenlet = lang.proxy_import('greenlet')
-
     aiu = lang.proxy_import('.anyio', __package__)
 
 
@@ -56,6 +58,20 @@ def trivial_a_to_s(fn):
 # https://gist.github.com/snaury/202bf4f22c41ca34e56297bae5f33fef
 
 
+_THREADLETS_IMPL = threadlets.GreenletThreadlets
+# from ..concurrent.tests.real import RealThreadlets
+# _THREADLETS_IMPL = RealThreadlets
+
+_THREADLETS = sync.LazyFn(lambda: _THREADLETS_IMPL())
+
+
+def _threadlets() -> threadlets.Threadlets:
+    return _THREADLETS.get()
+
+
+#
+
+
 class BridgeAwaitRequiredError(Exception):
     pass
 
@@ -101,7 +117,7 @@ def _make_transition(seq: int, a_to_s: bool, obj: ta.Any) -> _BridgeTransition:
 
 _BRIDGED_TASKS: ta.MutableMapping[ta.Any, list[_BridgeTransition]] = weakref.WeakKeyDictionary()
 
-_BRIDGE_GREENLET_ATTR = f'__{__package__.replace(".", "__")}__bridge_greenlet__'
+_BRIDGE_THREADLET_ATTR = f'__{__package__.replace(".", "__")}__bridge_threadlet__'
 
 
 def _push_transition(a_to_s: bool, l: list[_BridgeTransition], t: _BridgeTransition) -> _BridgeTransition:
@@ -129,9 +145,9 @@ def _get_transitions() -> list[_BridgeTransition]:
         else:
             l.extend(tl)
 
-    g = greenlet.getcurrent()
+    g = _threadlets().get_current()
     try:
-        gl = getattr(g, _BRIDGE_GREENLET_ATTR)
+        gl = getattr(g.underlying, _BRIDGE_THREADLET_ATTR)
     except AttributeError:
         pass
     else:
@@ -158,9 +174,9 @@ def is_in_bridge() -> bool:
     else:
         last_t = None
 
-    g = greenlet.getcurrent()
+    g = _threadlets().get_current()
     try:
-        gl = getattr(g, _BRIDGE_GREENLET_ATTR)
+        gl = getattr(g.underlying, _BRIDGE_THREADLET_ATTR)
     except AttributeError:
         last_g = None
     else:
@@ -194,13 +210,13 @@ def _safe_cancel_awaitable(awaitable: ta.Awaitable[ta.Any]) -> None:
 
 
 def s_to_a_await(awaitable: ta.Awaitable[T]) -> T:
-    g = greenlet.getcurrent()
+    g = _threadlets().get_current()
 
-    if not getattr(g, _BRIDGE_GREENLET_ATTR, False):
+    if not getattr(g.underlying, _BRIDGE_THREADLET_ATTR, False):
         _safe_cancel_awaitable(awaitable)
         raise MissingBridgeGreenletError
 
-    return g.parent.switch(awaitable)
+    return check.not_none(g.parent).switch(awaitable)
 
 
 def s_to_a(fn, *, require_await=False):
@@ -210,7 +226,7 @@ def s_to_a(fn, *, require_await=False):
             try:
                 return fn(*args, **kwargs)
             finally:
-                if (gl2 := getattr(g, _BRIDGE_GREENLET_ATTR)) is not gl:  # noqa
+                if (gl2 := getattr(g.underlying, _BRIDGE_THREADLET_ATTR)) is not gl:  # noqa
                     raise UnexpectedBridgeNestingError
                 if (cur_g := _pop_transition(False, gl)) is not added_g:  # noqa
                     raise UnexpectedBridgeNestingError
@@ -219,8 +235,8 @@ def s_to_a(fn, *, require_await=False):
 
         seq = next(_BRIDGE_TRANSITIONS_SEQ)
 
-        g = greenlet.greenlet(inner)
-        setattr(g, _BRIDGE_GREENLET_ATTR, gl := [])  # type: ignore
+        g = _threadlets().spawn(inner)
+        setattr(g.underlying, _BRIDGE_THREADLET_ATTR, gl := [])  # type: ignore
         added_g = _push_transition(False, gl, _make_transition(seq, False, g))
 
         if (t := aiu.get_current_backend_task()) is not None:
@@ -270,11 +286,11 @@ def a_to_s(fn):
         else:
             added_t = None
 
-        g = greenlet.getcurrent()
+        g = _threadlets().get_current()
         try:
-            gl = getattr(g, _BRIDGE_GREENLET_ATTR)
+            gl = getattr(g.underlying, _BRIDGE_THREADLET_ATTR)
         except AttributeError:
-            setattr(g, _BRIDGE_GREENLET_ATTR, gl := [])
+            setattr(g.underlying, _BRIDGE_THREADLET_ATTR, gl := [])
         added_g = _push_transition(True, gl, _make_transition(seq, True, g))
 
         try:
@@ -306,7 +322,7 @@ def a_to_s(fn):
                 if (cur_t := _pop_transition(True, tl)) is not added_t:  # noqa
                     raise UnexpectedBridgeNestingError
 
-            if (gl2 := getattr(g, _BRIDGE_GREENLET_ATTR)) is not gl:  # noqa
+            if (gl2 := getattr(g.underlying, _BRIDGE_THREADLET_ATTR)) is not gl:  # noqa
                 raise UnexpectedBridgeNestingError
             if (cur_g := _pop_transition(True, gl)) is not added_g:  # noqa
                 raise UnexpectedBridgeNestingError

omlish/bootstrap/__init__.py

@@ -1,3 +1,42 @@
+from .base import (  # noqa
+    Bootstrap,
+    ContextBootstrap,
+    SimpleBootstrap,
+)
+
+from .diag import (  # noqa
+    CheckBootstrap,
+    CprofileBootstrap,
+    PycharmBootstrap,
+    ThreadDumpBootstrap,
+    TimebombBootstrap,
+)
+
 from .harness import (  # noqa
     bootstrap,
 )
+
+from .sys import (  # noqa
+    CwdBootstrap,
+    EnvBootstrap,
+    FaulthandlerBootstrap,
+    FdsBootstrap,
+    GcBootstrap,
+    GcDebugFlag,
+    ImportBootstrap,
+    LogBootstrap,
+    NiceBootstrap,
+    PidfileBootstrap,
+    PrctlBootstrap,
+    PrintPidBootstrap,
+    RlimitBootstrap,
+    SetuidBootstrap,
+)
+
+
+##
+
+
+from ..lang.imports import _register_conditional_import  # noqa
+
+_register_conditional_import('..marshal', '.marshal', __package__)

omlish/bootstrap/base.py

@@ -2,6 +2,8 @@ import abc
 import dataclasses as dc
 import typing as ta
 
+from omlish import lang
+
 
 BootstrapConfigT = ta.TypeVar('BootstrapConfigT', bound='Bootstrap.Config')
 
@@ -9,7 +11,7 @@ BootstrapConfigT = ta.TypeVar('BootstrapConfigT', bound='Bootstrap.Config')
 ##
 
 
-class Bootstrap(abc.ABC, ta.Generic[BootstrapConfigT]):
+class Bootstrap(abc.ABC, lang.PackageSealed, ta.Generic[BootstrapConfigT]):
     @dc.dataclass(frozen=True)
     class Config(abc.ABC):  # noqa
         pass

omlish/bootstrap/diag.py

@@ -5,28 +5,56 @@ import signal
 import sys
 import typing as ta
 
+from .. import check
 from .. import lang
 from .base import Bootstrap
 from .base import ContextBootstrap
+from .base import SimpleBootstrap
 
 
 if ta.TYPE_CHECKING:
     import cProfile  # noqa
     import pstats
 
+    from ..diag import pycharm as diagpc
     from ..diag import threads as diagt
 
 else:
     cProfile = lang.proxy_import('cProfile')  # noqa
     pstats = lang.proxy_import('pstats')
 
+    diagpc = lang.proxy_import('..diag.pycharm', __package__)
     diagt = lang.proxy_import('..diag.threads', __package__)
 
 
 ##
 
 
-class ProfilingBootstrap(ContextBootstrap['ProfilingBootstrap.Config']):
+class CheckBootstrap(ContextBootstrap['CheckBootstrap.Config']):
+    @dc.dataclass(frozen=True)
+    class Config(Bootstrap.Config):
+        breakpoint: bool = False
+
+    @staticmethod
+    def _breakpoint(exc: Exception) -> None:  # noqa
+        breakpoint()  # noqa
+
+    @contextlib.contextmanager
+    def enter(self) -> ta.Iterator[None]:
+        if not self._config.breakpoint:
+            return
+
+        check.register_on_raise(CheckBootstrap._breakpoint)
+        try:
+            yield
+        finally:
+            check.unregister_on_raise(CheckBootstrap._breakpoint)
+
+
+##
+
+
+class CprofileBootstrap(ContextBootstrap['CprofileBootstrap.Config']):
     @dc.dataclass(frozen=True)
     class Config(Bootstrap.Config):
         enable: bool = False
@@ -70,6 +98,7 @@ class ThreadDumpBootstrap(ContextBootstrap['ThreadDumpBootstrap.Config']):
     @dc.dataclass(frozen=True)
     class Config(Bootstrap.Config):
         interval_s: ta.Optional[float] = None
+        nodaemon: bool = False
 
         on_sigquit: bool = False
 
@@ -79,6 +108,7 @@ class ThreadDumpBootstrap(ContextBootstrap['ThreadDumpBootstrap.Config']):
             tdt = diagt.create_thread_dump_thread(
                 interval_s=self._config.interval_s,
                 start=True,
+                nodaemon=self._config.nodaemon,
             )
         else:
             tdt = None
@@ -97,7 +127,7 @@ class ThreadDumpBootstrap(ContextBootstrap['ThreadDumpBootstrap.Config']):
             yield
 
         finally:
-            if tdt is not None:
+            if tdt is not None and not self._config.nodaemon:
                 tdt.stop_nowait()
 
             if prev_sq.present:
@@ -126,3 +156,22 @@ class TimebombBootstrap(ContextBootstrap['TimebombBootstrap.Config']):
             yield
         finally:
             tbt.stop_nowait()
+
+
+##
+
+
+class PycharmBootstrap(SimpleBootstrap['PycharmBootstrap.Config']):
+    @dc.dataclass(frozen=True)
+    class Config(Bootstrap.Config):
+        debug_host: ta.Optional[str] = None
+        debug_port: ta.Optional[int] = None
+        debug_version: ta.Optional[str] = None
+
+    def run(self) -> None:
+        if self._config.debug_port is not None:
+            diagpc.pycharm_remote_debugger_attach(
+                self._config.debug_host,
+                self._config.debug_port,
+                version=self._config.debug_version,
+            )

omlish/bootstrap/main.py

@@ -156,8 +156,9 @@ def _main() -> int:
     with bootstrap(*cfgs):
         tgt = args.target
 
+        sys.argv = [tgt, *(args.args or ())]
+
         if args.module:
-            sys.argv = [tgt, *(args.args or ())]
             runpy._run_module_as_main(tgt)  # type: ignore  # noqa
 
         else:

omlish/bootstrap/marshal.py

@@ -0,0 +1,18 @@
+from .. import lang
+from .. import marshal as msh
+from .base import Bootstrap
+from .harness import BOOTSTRAP_TYPES_BY_NAME
+
+
+@lang.cached_function
+def _install_standard_marshalling() -> None:
+    cfgs_poly = msh.Polymorphism(
+        Bootstrap.Config,
+        [msh.Impl(b.Config, n) for n, b in BOOTSTRAP_TYPES_BY_NAME.items()],
+    )
+
+    msh.STANDARD_MARSHALER_FACTORIES[0:0] = [msh.PolymorphismMarshalerFactory(cfgs_poly)]
+    msh.STANDARD_UNMARSHALER_FACTORIES[0:0] = [msh.PolymorphismUnmarshalerFactory(cfgs_poly)]
+
+
+_install_standard_marshalling()

omlish/check.py

@@ -3,6 +3,7 @@ TODO:
  - def maybe(v: lang.Maybe[T])
 """
 import collections
+import threading
 import typing as ta
 
 
@@ -21,6 +22,62 @@ _callable = callable
 ##
 
 
+_CONFIG_LOCK = threading.RLock()
+
+
+OnRaiseFn: ta.TypeAlias = ta.Callable[[Exception], None]
+_ON_RAISE: ta.Sequence[OnRaiseFn] = []
+
+
+def register_on_raise(fn: OnRaiseFn) -> None:
+    global _ON_RAISE
+    with _CONFIG_LOCK:
+        _ON_RAISE = [*_ON_RAISE, fn]
+
+
+def unregister_on_raise(fn: OnRaiseFn) -> None:
+    global _ON_RAISE
+    with _CONFIG_LOCK:
+        _ON_RAISE = [e for e in _ON_RAISE if e != fn]
+
+
+#
+
+
+_render_args: ta.Callable[..., str | None] | None = None
+
+
+def enable_args_rendering() -> bool:
+    global _render_args
+    if _render_args is not None:
+        return True
+
+    with _CONFIG_LOCK:
+        if _render_args is not None:
+            return True  # type: ignore
+
+        try:
+            from .diag.asts import ArgsRenderer
+
+            ArgsRenderer.smoketest()
+
+        except Exception:  # noqa
+            return False
+
+        def _real_render_args(fmt: str, *args: ta.Any) -> str | None:
+            ra = ArgsRenderer(back=3).render_args(*args)
+            if ra is None:
+                return None
+
+            return fmt % tuple(str(a) for a in ra)
+
+        _render_args = _real_render_args
+        return True
+
+
+#
+
+
 def _default_exception_factory(exc_cls: type[Exception], *args, **kwargs) -> Exception:
     return exc_cls(*args, **kwargs)  # noqa
 
@@ -28,20 +85,45 @@ def _default_exception_factory(exc_cls: type[Exception], *args, **kwargs) -> Exc
 _EXCEPTION_FACTORY = _default_exception_factory
 
 
+class _Args:
+    def __init__(self, *args, **kwargs):
+        self.args = args
+        self.kwargs = kwargs
+
+
 def _raise(
         exception_type: type[Exception],
         default_message: str,
         message: Message,
-        *args: ta.Any,
-        **kwargs: ta.Any,
+        ak: _Args = _Args(),
+        *,
+        render_fmt: str | None = None,
 ) -> ta.NoReturn:
+    exc_args = ()
     if _callable(message):
-        message = ta.cast(ta.Callable, message)(*args, **kwargs)
+        message = ta.cast(ta.Callable, message)(*ak.args, **ak.kwargs)
         if _isinstance(message, tuple):
-            message, *args = message  # type: ignore
+            message, *exc_args = message  # type: ignore
+
     if message is None:
         message = default_message
-    exc = _EXCEPTION_FACTORY(exception_type, message, *args, **kwargs)
+
+    if render_fmt is not None and _render_args is not None:
+        rendered_args = _render_args(render_fmt, *ak.args)
+        if rendered_args is not None:
+            message = f'{message} : {rendered_args}'
+
+    exc = _EXCEPTION_FACTORY(
+        exception_type,
+        message,
+        *exc_args,
+        *ak.args,
+        **ak.kwargs,
+    )
+
+    for fn in _ON_RAISE:
+        fn(exc)
+
     raise exc
 
 
@@ -60,7 +142,7 @@ def _unpack_isinstance_spec(spec: ta.Any) -> tuple:
 
 def isinstance(v: ta.Any, spec: type[T] | tuple, msg: Message = None) -> T:  # noqa
     if not _isinstance(v, _unpack_isinstance_spec(spec)):
-        _raise(TypeError, 'Must be instance', msg, v, spec)
+        _raise(TypeError, 'Must be instance', msg, _Args(v, spec))
     return v
 
 
@@ -73,7 +155,7 @@ def of_isinstance(spec: type[T] | tuple, msg: Message = None) -> ta.Callable[[ta
 
 def cast(v: ta.Any, cls: type[T], msg: Message = None) -> T:  # noqa
     if not _isinstance(v, cls):
-        _raise(TypeError, 'Must be instance', msg, v, cls)
+        _raise(TypeError, 'Must be instance', msg, _Args(v, cls))
     return v
 
 
@@ -86,7 +168,7 @@ def of_cast(cls: type[T], msg: Message = None) -> ta.Callable[[T], T]:
 
 def not_isinstance(v: T, spec: ta.Any, msg: Message = None) -> T:  # noqa
     if _isinstance(v, _unpack_isinstance_spec(spec)):
-        _raise(TypeError, 'Must not be instance', msg, v, spec)
+        _raise(TypeError, 'Must not be instance', msg, _Args(v, spec))
     return v
 
 
@@ -102,13 +184,13 @@ def of_not_isinstance(spec: ta.Any, msg: Message = None) -> ta.Callable[[T], T]:
 
 def issubclass(v: type[T], spec: ta.Any, msg: Message = None) -> type[T]:  # noqa
     if not _issubclass(v, spec):
-        _raise(TypeError, 'Must be subclass', msg, v, spec)
+        _raise(TypeError, 'Must be subclass', msg, _Args(v, spec))
     return v
 
 
 def not_issubclass(v: type[T], spec: ta.Any, msg: Message = None) -> type[T]:  # noqa
     if _issubclass(v, spec):
-        _raise(TypeError, 'Must not be subclass', msg, v, spec)
+        _raise(TypeError, 'Must not be subclass', msg, _Args(v, spec))
     return v
 
 
@@ -117,32 +199,43 @@ def not_issubclass(v: type[T], spec: ta.Any, msg: Message = None) -> type[T]:  #
 
 def in_(v: T, c: ta.Container[T], msg: Message = None) -> T:
     if v not in c:
-        _raise(ValueError, 'Must be in', msg, v, c)
+        _raise(ValueError, 'Must be in', msg, _Args(v, c))
     return v
 
 
 def not_in(v: T, c: ta.Container[T], msg: Message = None) -> T:
     if v in c:
-        _raise(ValueError, 'Must not be in', msg, v, c)
+        _raise(ValueError, 'Must not be in', msg, _Args(v, c))
     return v
 
 
 def empty(v: SizedT, msg: Message = None) -> SizedT:
     if len(v) != 0:
-        _raise(ValueError, 'Must be empty', msg, v)
+        _raise(ValueError, 'Must be empty', msg, _Args(v))
+    return v
+
+
+def iterempty(v: ta.Iterable[T], msg: Message = None) -> ta.Iterable[T]:
+    it = iter(v)
+    try:
+        next(it)
+    except StopIteration:
+        pass
+    else:
+        _raise(ValueError, 'Must be empty', msg, _Args(v))
     return v
 
 
 def not_empty(v: SizedT, msg: Message = None) -> SizedT:
     if len(v) == 0:
-        _raise(ValueError, 'Must not be empty', msg, v)
+        _raise(ValueError, 'Must not be empty', msg, _Args(v))
     return v
 
 
 def unique(it: ta.Iterable[T], msg: Message = None) -> ta.Iterable[T]:
     dupes = [e for e, c in collections.Counter(it).items() if c > 1]
     if dupes:
-        _raise(ValueError, 'Must be unique', msg, it, dupes)
+        _raise(ValueError, 'Must be unique', msg, _Args(it, dupes))
     return it
 
 
@@ -150,7 +243,7 @@ def single(obj: ta.Iterable[T], message: Message = None) -> T:
     try:
         [value] = obj
     except ValueError:
-        _raise(ValueError, 'Must be single', message, obj)
+        _raise(ValueError, 'Must be single', message, _Args(obj))
     else:
         return value
 
@@ -165,7 +258,7 @@ def optional_single(obj: ta.Iterable[T], message: Message = None) -> T | None:
         next(it)
     except StopIteration:
         return value  # noqa
-    _raise(ValueError, 'Must be empty or single', message, obj)
+    _raise(ValueError, 'Must be empty or single', message, _Args(obj))
 
 
 ##
@@ -173,48 +266,45 @@ def optional_single(obj: ta.Iterable[T], message: Message = None) -> T | None:
 
 def none(v: ta.Any, msg: Message = None) -> None:
     if v is not None:
-        _raise(ValueError, 'Must be None', msg, v)
+        _raise(ValueError, 'Must be None', msg, _Args(v))
 
 
 def not_none(v: T | None, msg: Message = None) -> T:
     if v is None:
-        _raise(ValueError, 'Must not be None', msg, v)
+        _raise(ValueError, 'Must not be None', msg, _Args(v))
     return v
 
 
 ##
 
 
-def equal(v: T, *os: ta.Any, msg: Message = None) -> T:
-    for o in os:
-        if o != v:
-            _raise(ValueError, 'Must be equal', msg, v, os)
+def equal(v: T, o: ta.Any, msg: Message = None) -> T:
+    if o != v:
+        _raise(ValueError, 'Must be equal', msg, _Args(v, o), render_fmt='%s != %s')
     return v
 
 
-def is_(v: T, *os: ta.Any, msg: Message = None) -> T:
-    for o in os:
-        if o is not v:
-            _raise(ValueError, 'Must be the same', msg, v, os)
+def is_(v: T, o: ta.Any, msg: Message = None) -> T:
+    if o is not v:
+        _raise(ValueError, 'Must be the same', msg, _Args(v, o), render_fmt='%s is not %s')
     return v
 
 
-def is_not(v: T, *os: ta.Any, msg: Message = None) -> T:
-    for o in os:
-        if o is v:
-            _raise(ValueError, 'Must not be the same', msg, v, os)
+def is_not(v: T, o: ta.Any, msg: Message = None) -> T:
+    if o is v:
+        _raise(ValueError, 'Must not be the same', msg, _Args(v, o), render_fmt='%s is %s')
     return v
 
 
 def callable(v: T, msg: Message = None) -> T:  # noqa
     if not _callable(v):
-        _raise(TypeError, 'Must be callable', msg, v)
+        _raise(TypeError, 'Must be callable', msg, _Args(v))
     return v  # type: ignore
 
 
 def non_empty_str(v: str | None, msg: Message = None) -> str:
     if not _isinstance(v, str) or not v:
-        _raise(ValueError, 'Must be non-empty str', msg, v)
+        _raise(ValueError, 'Must be non-empty str', msg, _Args(v))
     return v