reactor-runtime 2.8.0__tar.gz → 2.9.0__tar.gz

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: reactor_runtime
-Version: 2.8.0
+Version: 2.9.0
 Summary: Reactor runtime with public model API
 Author-email: Reactor <team@reactor.inc>
 Requires-Python: >=3.9

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "reactor_runtime"
-version = "2.8.0"
+version = "2.9.0"
 description = "Reactor runtime with public model API"
 authors = [
 	{ name = "Reactor", email = "team@reactor.inc" }
@@ -90,4 +90,4 @@ gst = [
 # Legacy releases used `X.Y.Z-g<short-sha>` (e.g. `1.4.1-g5660eed`); the wheel
 # stripped the `-g<sha>` suffix. fetch-proto.py still understands that shape
 # for back-compat with old pins, but new bumps should track the calendar format.
-version = "1.20260603.15086"
+version = "1.20260625.16626"

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime/runtime_api.py

@@ -379,8 +379,16 @@ class Runtime(ModelStateMachine, ABC):
         self._session_moderator: Optional["SessionModerator"] = None
         moderation_enabled = self.config.reactor_config.moderation.enabled
         if _MODERATION_AVAILABLE:
+            # REA-2601: pass the subclass-owned SupervisorClient (if any)
+            # so the default ``supervisor`` backend can proxy moderation
+            # over the existing gRPC channel. Subclasses without a
+            # Supervisor connection (e.g. HttpRuntime) simply leave the
+            # attribute unset; ``from_reactor_config`` raises a
+            # ``ModerationConfigError`` at startup so the misconfiguration
+            # never reaches a live session.
             self._moderator = ContentModerator.from_reactor_config(
-                self.config.reactor_config
+                self.config.reactor_config,
+                supervisor_client=getattr(self, "_supervisor_client", None),
             )
             if self._moderator is not None:
                 logger.info("Content moderation enabled")

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime/serve/commands/run.py

@@ -23,12 +23,37 @@ from reactor_runtime.utils.paths import _set_configured_weights_path
 logger = get_logger(__name__)
 
 _MODEL_NAME_ENV_VAR = "MODEL_NAME"
-_MODEL_NAME_MAX_LENGTH = 128
-_MODEL_NAME_PATTERN = re.compile(r"^[a-zA-Z0-9][a-zA-Z0-9-]*$")
+_MODEL_NAME_MAX_LENGTH = 129
+_MODEL_NAME_PATTERN = re.compile(
+    r"^[a-zA-Z0-9][a-zA-Z0-9._-]*(/[a-zA-Z0-9][a-zA-Z0-9._-]*)?$"
+)
 
 _REACTOR_YAML = "reactor.yaml"
 
 
+def _advertised_name_matches(advertised: str, canonical: str) -> bool:
+    """Check a reactor.yaml-advertised name against the assigned MODEL_NAME.
+
+    The Coordinator assigns a fully-qualified canonical name
+    (``namespace/slug``, e.g. ``reactor/echo``). A model author is allowed
+    to advertise either that canonical name or the bare slug — the
+    namespace is derived from the authenticated account, so omitting it is
+    legitimate. Any other value (a different slug, or a namespace the
+    author does not own) is a mismatch.
+
+    Args:
+        advertised: ``model.name`` from the partner-authored reactor.yaml.
+        canonical: The platform-assigned ``MODEL_NAME``.
+
+    Returns:
+        True if ``advertised`` resolves to ``canonical``.
+    """
+    if advertised == canonical:
+        return True
+    _, _, slug = canonical.partition("/")
+    return bool(slug) and advertised == slug
+
+
 class RunCommand:
     @staticmethod
     def register_subcommand(subparsers):
@@ -126,10 +151,27 @@ class RunCommand:
             return
 
         # ---------------------------------------------------------
-        # Model name (from YAML or env override)
+        # Model identity
+        #
+        # In production the Coordinator assigns the canonical model name
+        # (e.g. "reactor/echo") and the Supervisor injects it as the
+        # MODEL_NAME env var on the generated deployment. That value is the
+        # trusted identity: the runtime keys its streams, logs and metering
+        # off it, never off the partner-authored reactor.yaml.
+        #
+        # A model author may still advertise a name in reactor.yaml
+        # (model.name). When MODEL_NAME is set we treat that as a claim that
+        # must be honoured — it has to resolve to the assigned name (the
+        # canonical name, or the bare slug since the namespace is derived
+        # from the authenticated account). A mismatch means untrusted model
+        # code is trying to run under a name it was not granted, so we
+        # refuse to start.
+        #
+        # Without MODEL_NAME (local `reactor run`) there is no assigning
+        # authority, so the advertised name — then the entrypoint spec — is
+        # used as a best-effort label.
         # ---------------------------------------------------------
-        if not reactor_config.name:
-            reactor_config.name = reactor_config.model
+        advertised_name = reactor_config.name
         env_name = os.environ.get(_MODEL_NAME_ENV_VAR)
         if env_name:
             if len(env_name) > _MODEL_NAME_MAX_LENGTH:
@@ -138,7 +180,19 @@ class RunCommand:
             if not _MODEL_NAME_PATTERN.match(env_name):
                 print(f"Error: {_MODEL_NAME_ENV_VAR} has invalid chars")
                 return
+            if advertised_name and not _advertised_name_matches(
+                advertised_name, env_name
+            ):
+                print(
+                    f"Error: reactor.yaml advertises model name "
+                    f"'{advertised_name}', which does not match the "
+                    f"assigned {_MODEL_NAME_ENV_VAR} '{env_name}'. A model "
+                    f"may only run under the name the platform assigned it."
+                )
+                return
             reactor_config.name = env_name
+        elif not reactor_config.name:
+            reactor_config.name = reactor_config.model
 
         # ---------------------------------------------------------
         # Resolve model config path + CLI overrides

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime/transports/config.py

@@ -258,7 +258,8 @@ def resolve_default_transport() -> TransportType:
     bindings import and that every required element is present, raising
     a clear ``ImportError`` (with an install hint) when they are not.
     Failing loudly here surfaces a misconfigured environment at startup
-    rather than at the first ``WebRTCTransport.create`` call.
+    rather than at the first ``WebRTCTransport.create`` call, and
+    pre-warms Gst.init() so the first connection does not pay that cost.
     """
     try:
         validate_transport_available(TransportType.GSTREAMER)

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime/transports/events.py

@@ -178,6 +178,7 @@ class TrackStats:
     packets_received: Optional[int] = None
     packets_sent: Optional[int] = None
     jitter: Optional[float] = None
+    qos: Optional[float] = None
 
 
 @dataclass

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime/transports/gstreamer/client.py

@@ -70,6 +70,7 @@ from reactor_runtime.transports.gstreamer.gst_helpers import (
 from reactor_runtime.transports.gstreamer.signals import SignalManager
 from reactor_runtime.transports.gstreamer.sender import AudioSender, VideoSender
 from reactor_runtime.transports.gstreamer.receiver import VideoReceiver, AudioReceiver
+from reactor_runtime.transports.gstreamer.quality import aggregate_qos_score
 from reactor_runtime.transports.gstreamer.settings import (
     SUPPORTED_RTP_HEADER_EXTENSION_URIS,
     VIDEO_BWE_MIN_BITRATE_KBPS,
@@ -104,6 +105,9 @@ ICE_GATHERING_TIMEOUT_MS = 3000
 # Frame queue drain timeout interval in milliseconds
 FRAME_Q_DRAIN_TIMEOUT_MS = 10
 
+# Quality score measurement interval
+QUALITY_METER_INTERVAL_MS = 1000
+
 # =============================================================================
 # Helpers for converting transport-agnostic types to GStreamer expected ones
 # =============================================================================
@@ -489,6 +493,38 @@ class GStreamerTransport(WebRTCTransport):
         src.set_callback(_drain)
         src.attach(self._main_ctx)
 
+    def _gst_arm_quality_meter(self) -> None:
+        """GLib thread: log a per-second video quality score for every active video sender."""
+        if self._main_ctx is None:
+            return
+
+        src = GLib.timeout_source_new(QUALITY_METER_INTERVAL_MS)
+        self._track_source(src)
+
+        def _tick(*_args) -> bool:
+            if self._stopping:
+                return GLib.SOURCE_REMOVE
+
+            scores = [
+                s
+                for sender in self._senders.values()
+                if isinstance(sender, VideoSender)
+                for s in [sender.qos()]
+                if s is not None
+            ]
+            score = aggregate_qos_score(scores)
+            if score is not None:
+                logger.info(
+                    "video quality score",
+                    score=score,
+                    senders=len(scores),
+                )
+
+            return GLib.SOURCE_CONTINUE
+
+        src.set_callback(_tick)
+        src.attach(self._main_ctx)
+
     def _require_webrtc(self) -> Gst.Element:
         if self._webrtc is None:
             raise RuntimeError("webrtcbin element is not initialized")
@@ -874,6 +910,7 @@ class GStreamerTransport(WebRTCTransport):
             if not self._connected.is_set():
                 self._connected.set()
                 self._gst_emit_event(ConnectedEvent())
+                self._gst_arm_quality_meter()
         elif state in (
             GstWebRTC.WebRTCPeerConnectionState.FAILED,
             GstWebRTC.WebRTCPeerConnectionState.CLOSED,
@@ -906,10 +943,10 @@ class GStreamerTransport(WebRTCTransport):
 
         if label == "control":
             self._control_channel = data_channel
-            self._run_on_gst_thread(self._setup_control_channel_handlers)
+            self._setup_control_channel_handlers()
         else:
             self._data_channel = data_channel
-            self._run_on_gst_thread(self._setup_data_channel_handlers)
+            self._setup_data_channel_handlers()
 
     def _gst_on_pad_added(self, element, pad) -> None:
         if self._stopping or self._stop_event.is_set():
@@ -1642,6 +1679,7 @@ class GStreamerTransport(WebRTCTransport):
                     width: Optional[int] = None
                     height: Optional[int] = None
                     codec_name: Optional[str] = None
+                    qos: Optional[float] = None
                     sender = self._senders.get(mid)
                     if isinstance(sender, VideoSender):
                         bitrate_bps = sender.current_bitrate_kbps * 1000
@@ -1649,6 +1687,7 @@ class GStreamerTransport(WebRTCTransport):
                         width = sender.width
                         height = sender.height
                         codec_name = sender.codec_name
+                        qos = sender.qos()
                     elif isinstance(sender, AudioSender):
                         bitrate_bps = sender.current_bitrate_kbps * 1000
                         codec_name = sender.codec_name
@@ -1678,6 +1717,7 @@ class GStreamerTransport(WebRTCTransport):
                             packet_loss=packet_loss,
                             packets_sent=packets_sent,
                             jitter=jitter,
+                            qos=qos,
                         )
                     )
 

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime/transports/gstreamer/gst_helpers.py

@@ -75,7 +75,14 @@ def add_many(
       the element will remain in NULL state until explicitly set.
     """
     for e in elements:
-        if not bin.add(e):
+        # Gst.Bin.add() returns the C gboolean on most PyGObject builds, but some
+        # return None even on success — so verify the element actually landed in
+        # this bin rather than trusting the binding's return value. Checking
+        # `is not bin` (not just `is None`) also catches a failed add of an element
+        # that already belongs to another bin, where get_parent() returns that
+        # other bin.
+        bin.add(e)
+        if e.get_parent() is not bin:
             raise RuntimeError(f"Failed to add {e.get_name()} to {bin.get_name()}")
         if sync_with_parent:
             e.sync_state_with_parent()

reactor_runtime-2.9.0/src/reactor_runtime/transports/gstreamer/quality.py

@@ -0,0 +1,71 @@
+# Copyright (c) 2026 Reactor Technologies, Inc. All rights reserved.
+
+"""
+Video quality scoring utilities.
+
+Provides a formula-based estimate of the *expected* bitrate for a given
+resolution, frame-rate, and codec, and a 0–10 quality score comparing
+a measured bitrate against that expectation.
+"""
+
+from typing import Optional
+
+# =========================================================================
+# Reference tables
+# =========================================================================
+
+# Bits-per-pixel for H.264 at "medium" quality (general content, no motion
+# extremes).  Other codecs scale from this baseline via _CODEC_EFFICIENCY.
+_BPP_H264_MEDIUM = 0.08
+
+# Compression efficiency relative to H.264 (lower → fewer bits needed for
+# equal perceived quality).  Keys are RTP encoding-name strings (uppercase),
+# as returned by GStreamer and SDP negotiation.
+_CODEC_EFFICIENCY: dict[str, float] = {
+    "H264": 1.00,
+    "H265": 0.55,
+    "VP8": 1.00,
+    "VP9": 0.65,
+    "AV1": 0.45,
+}
+
+# =========================================================================
+# Per-track helpers
+# =========================================================================
+
+
+def expected_video_bitrate_bps(
+    width: int, height: int, fps: float, codec_name: str
+) -> int:
+    """Return the target bitrate in bps for a good-quality encode."""
+    efficiency = _CODEC_EFFICIENCY.get(codec_name.upper(), 1.0)
+    return int(width * height * fps * _BPP_H264_MEDIUM * efficiency)
+
+
+def video_qos_score(
+    current_bps: int, width: int, height: int, fps: float, codec_name: str
+) -> float:
+    """Return a 0–10 quality score for a single video track.
+
+    10.0 means at or above the expected bitrate for good quality.
+    Score scales linearly down to 0 as bitrate approaches zero.
+    """
+    expected = expected_video_bitrate_bps(width, height, fps, codec_name)
+    if expected <= 0:
+        return 10.0
+    return round(min(10.0, current_bps / expected * 10.0), 1)
+
+
+# =========================================================================
+# Aggregate score across multiple senders
+# =========================================================================
+
+
+def aggregate_qos_score(scores: list[float]) -> Optional[float]:
+    """Return the average 0–10 quality score across all video senders.
+
+    Returns None when the list is empty (no ready senders).
+    """
+    if not scores:
+        return None
+    return round(sum(scores) / len(scores), 1)

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime/transports/gstreamer/sender/video.py

@@ -9,6 +9,7 @@ from typing import List, NamedTuple, Optional
 import numpy as np
 
 from reactor_runtime.transports.gstreamer.encoders import EncoderFactory
+from reactor_runtime.transports.gstreamer.quality import video_qos_score
 from reactor_runtime.transports.gstreamer.settings import VIDEO_BWE_TARGET_BITRATE_KBPS
 from reactor_runtime.transports.gstreamer.sdp.codec import CodecEntry
 from reactor_runtime.transports.gstreamer.sdp.extmap import SdpExtmap
@@ -208,6 +209,15 @@ class VideoSender(_SenderStreamBase):
         """Forward target bitrate to the encoder bin."""
         self._encoder.set_target_bitrate_kbps(int(new_kbps))
 
+    def qos(self) -> Optional[float]:
+        """Return the 0–10 quality score for this sender, or None if not ready."""
+        w, h, fps = self.width, self.height, self.fps
+        if not w or not h or not fps:
+            return None
+        return video_qos_score(
+            self.current_bitrate_kbps * 1000, w, h, fps, self._codec_name
+        )
+
     def push_buffer(self, frame: np.ndarray) -> bool:
         """
         Extract dimensions and bytes from the frame (e.g. numpy array with .shape and .tobytes()),

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: reactor_runtime
-Version: 2.8.0
+Version: 2.9.0
 Summary: Reactor runtime with public model API
 Author-email: Reactor <team@reactor.inc>
 Requires-Python: >=3.9

{reactor_runtime-2.8.0 → reactor_runtime-2.9.0}/src/reactor_runtime.egg-info/SOURCES.txt

@@ -85,6 +85,7 @@ src/reactor_runtime/transports/gstreamer/__init__.py
 src/reactor_runtime/transports/gstreamer/client.py
 src/reactor_runtime/transports/gstreamer/gst.py
 src/reactor_runtime/transports/gstreamer/gst_helpers.py
+src/reactor_runtime/transports/gstreamer/quality.py
 src/reactor_runtime/transports/gstreamer/settings.py
 src/reactor_runtime/transports/gstreamer/signals.py
 src/reactor_runtime/transports/gstreamer/decoders/__init__.py