agentx-python 0.8.24__tar.gz → 0.8.25__tar.gz

{agentx_python-0.8.24/agentx_python.egg-info → agentx_python-0.8.25}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentx-python
-Version: 0.8.24
+Version: 0.8.25
 Summary: Official Python SDK for AgentX (https://www.agentx.so/)
 Home-page: https://github.com/AgentX-ai/AgentX-python
 Author: Robin Wang and AgentX Team
@@ -108,7 +108,9 @@ Also see [SDK Developer Docs](https://developers.agentx.so), [API Reference Docs
 pip install --upgrade agentx-python
 ```
 
-Requires Python 3.9 or newer.
+Requires Python 3.9 or newer for the core SDK. Some integration extras have higher floors set
+by their upstream packages - `[crewai]`, `[autogen]`, and `[databricks]` need Python 3.10+ (`[all]`
+therefore does too); the core tracer and every REST surface stay 3.9-compatible.
 
 #### Run the self-host governance suite locally
 

{agentx_python-0.8.24 → agentx_python-0.8.25}/README.md

@@ -43,7 +43,9 @@ Also see [SDK Developer Docs](https://developers.agentx.so), [API Reference Docs
 pip install --upgrade agentx-python
 ```
 
-Requires Python 3.9 or newer.
+Requires Python 3.9 or newer for the core SDK. Some integration extras have higher floors set
+by their upstream packages - `[crewai]`, `[autogen]`, and `[databricks]` need Python 3.10+ (`[all]`
+therefore does too); the core tracer and every REST surface stay 3.9-compatible.
 
 #### Run the self-host governance suite locally
 

{agentx_python-0.8.24 → agentx_python-0.8.25}/agentx/agentx.py

@@ -20,8 +20,8 @@ class AgentX:
         # The api_key is NOT written back into os.environ (it used to be): every sub-client
         # below receives it explicitly, and mutating process-global state from a constructor
         # re-pointed unrelated code - the same leak the base_url write below had (deep-dive
-        # round 3, bug #1). Static flows that still read the env (AgentX.list_workforces,
-        # bare get_headers()) now require the caller to set AGENTX_API_KEY themselves.
+        # round 3, bug #1). Flows that still read the env (bare get_headers()) now require
+        # the caller to set AGENTX_API_KEY themselves.
         self.api_key = api_key or os.getenv("AGENTX_API_KEY")
 
         # base_url overrides AGENTX_API_BASE_URL env var (and the SDK default). It is
@@ -147,7 +147,7 @@ class AgentX:
         response = requests.get(url, headers=get_headers(self.api_key))
         # Check if response was successful
         if response.status_code == 200:
-            return Agent(**response.json())
+            return Agent(**response.json())._bind(self.api_key, self.base_url)
         else:
             raise AgentXError(
                 f"Failed to retrieve agent: {response.reason}. This endpoint is "
@@ -165,22 +165,26 @@ class AgentX:
         response = requests.get(url, headers=get_headers(self.api_key))
         # Check if response was successful
         if response.status_code == 200:
-            return [Agent(**agent) for agent in response.json()]
+            return [Agent(**agent)._bind(self.api_key, self.base_url) for agent in response.json()]
         else:
             raise AgentXError(
                 f"Failed to list agents: {response.reason}. This endpoint is "
                 "hosted-platform only - on self-host use client.monitor.agents.list()."
             )
 
-    @staticmethod
-    def list_workforces() -> List["Workforce"]:
-        """List all workforces/teams. Static, so it reads AGENTX_API_KEY from the environment
-        directly - the constructor no longer writes ``api_key`` into os.environ, so set the
-        env var yourself before calling this."""
-        url = f"{api_base()}/access/teams"
-        response = requests.get(url, headers=get_headers())
+    def list_workforces(self) -> List["Workforce"]:
+        """List all workforces/teams, each bound to this client's credentials - including each
+        workforce's ``manager`` and ``agents``, so their calls authenticate the same way.
+
+        This used to be documented as a static call (``AgentX.list_workforces()``); that form
+        was broken (the old staticmethod body referenced ``self`` and raised NameError on any
+        non-empty response). Construct a client instead - ``AgentX().list_workforces()`` picks
+        up AGENTX_API_KEY / AGENTX_API_BASE_URL from the environment, which is what the static
+        form effectively did."""
+        url = f"{self.base_url or api_base()}/access/teams"
+        response = requests.get(url, headers=get_headers(self.api_key))
         if response.status_code == 200:
-            return [Workforce(**workforce) for workforce in response.json()]
+            return [Workforce(**workforce)._bind(self.api_key, self.base_url) for workforce in response.json()]
         else:
             raise Exception(
                 f"Failed to list workforces: {response.status_code} - {response.reason}"

{agentx_python-0.8.24 → agentx_python-0.8.25}/agentx/integrations/crewai.py

@@ -144,14 +144,14 @@ class AgentXCrewObserver:
                     TaskFailedEvent,
                     TaskStartedEvent,
                 )
-            except ImportError:
+            except Exception:  # noqa: BLE001 - crewai import raises TypeError (PEP 604) on py3.9, not just ImportError
                 from crewai.utilities.events import crewai_event_bus
                 from crewai.utilities.events.task_events import (
                     TaskCompletedEvent,
                     TaskFailedEvent,
                     TaskStartedEvent,
                 )
-        except ImportError:
+        except Exception:  # noqa: BLE001 - crewai import raises TypeError (PEP 604) on py3.9, not just ImportError
             if not _warned_no_event_bus:
                 _warned_no_event_bus = True
                 logger.warning(

{agentx_python-0.8.24 → agentx_python-0.8.25}/agentx/monitor/__init__.py

@@ -10,6 +10,9 @@ from agentx.monitor.judge_scorers import (
 from agentx.monitor.models import MonitorPattern, MonitorProfile, MonitorSignal, SignalOccurrence
 from agentx.monitor.patterns import MonitorPatternBuilder, MonitorPatternClient
 from agentx.monitor.profile import MonitorProfileClient
+from agentx.monitor.review_queue import ReviewQueueClient, ReviewQueueItem
+from agentx.monitor.rules import MonitorRule, MonitorRulesClient
+from agentx.monitor.scorers import AgentXScorersError, ScorersClient
 from agentx.monitor.scorer_groups import AgentXScorerGroupsError, ScorerGroup, ScorerGroupsClient
 from agentx.monitor.sessions import MonitorSessionClient
 from agentx.monitor.signals import MonitorSignalClient
@@ -19,6 +22,7 @@ __all__ = [
     "AgentXJudgeScorersError",
     "AgentXMonitorError",
     "AgentXScorerGroupsError",
+    "AgentXScorersError",
     "ImprovementGroupsClient",
     "JudgeScorer",
     "JudgeScorerBuilder",
@@ -30,10 +34,15 @@ __all__ = [
     "MonitorPatternClient",
     "MonitorProfile",
     "MonitorProfileClient",
+    "MonitorRule",
+    "MonitorRulesClient",
     "MonitorSessionClient",
     "MonitorSignal",
     "MonitorSignalClient",
+    "ReviewQueueClient",
+    "ReviewQueueItem",
     "ScorerGroup",
+    "ScorersClient",
     "ScorerGroupsClient",
     "SignalOccurrence",
 ]

agentx_python-0.8.25/agentx/monitor/_transport.py

@@ -0,0 +1,48 @@
+"""Shared HTTP transport for the monitor sub-clients that own their ``_request`` (scorers,
+judge_scorers, scorer_groups, improvement_groups): one retry schedule mirroring
+``MonitorClient._request``, so ``retry=False`` means the same thing everywhere the client.py
+comment promises it ("retry=False for ANY non-idempotent write")."""
+
+from __future__ import annotations
+
+import logging
+import time
+from typing import Any, Optional
+
+import requests
+
+logger = logging.getLogger(__name__)
+
+# Same schedule as MonitorClient._request (agentx/monitor/client.py).
+_RETRYABLE_STATUS = {429, 500, 502, 503, 504}
+_RETRY_BACKOFF = [1.0, 2.0, 4.0]
+
+
+def request_with_retries(
+    method: str, url: str, *, retry: bool = True, **kwargs: Any
+) -> requests.Response:
+    """``requests.request`` with MonitorClient's transport posture: when ``retry`` is true,
+    connection errors and retryable statuses (429/5xx) walk the backoff schedule; the last
+    response (whatever its status) is returned for the caller's own error taxonomy.
+
+    ``retry=False`` is single-shot - for non-idempotent writes (creates, deletes) and
+    judge-billing POSTs, where a client-side timeout must not fire the same work twice.
+    Transport errors keep their ``requests`` exception type (callers guard on
+    ``requests.Timeout`` for judge-billing endpoints)."""
+    schedule = [0.0] + _RETRY_BACKOFF if retry else [0.0]
+    last_exc: Optional[Exception] = None
+    for attempt, wait in enumerate(schedule):
+        if wait:
+            time.sleep(wait)
+        try:
+            resp = requests.request(method, url, **kwargs)
+        except requests.RequestException as e:
+            last_exc = e
+            logger.debug("Request error (attempt %d): %s", attempt + 1, e)
+            continue
+        if retry and resp.status_code in _RETRYABLE_STATUS and attempt < len(schedule) - 1:
+            logger.debug("Retryable status %d (attempt %d)", resp.status_code, attempt + 1)
+            continue
+        return resp
+    assert last_exc is not None  # every non-raising path returned above
+    raise last_exc

{agentx_python-0.8.24 → agentx_python-0.8.25}/agentx/monitor/client.py

@@ -205,9 +205,11 @@ class MonitorClient:
     def _request(
         self, method: str, path: str, timeout: int = 30, base: Optional[str] = None, retry: bool = True, **kwargs
     ) -> Any:
-        # retry=False for non-idempotent judge-spending POSTs (sweep, coherence, portability,
-        # tuning): a client-side timeout must not fire the same LLM-billing work a second time
-        # while the first invocation is still running server-side. Same precedent as
+        # retry=False for ANY non-idempotent write - duplicating creates, deletes (a lost
+        # response + retry turns success into a spurious 404), and judge-spending POSTs
+        # (sweep, coherence, portability, tuning: a client-side timeout must not fire the
+        # same LLM-billing work twice while the first invocation still runs server-side).
+        # The judge list is the example set, not the rule. Same precedent as
         # EvaluationsClient._request / analyze_run.
         url = f"{base or self._base_url}{path}"
         last_exc: Optional[Exception] = None

{agentx_python-0.8.24 → agentx_python-0.8.25}/agentx/monitor/improvement_groups.py

@@ -4,6 +4,7 @@ from typing import Any, Dict, List, Optional
 
 import requests
 
+from agentx.monitor._transport import request_with_retries
 from agentx.util import api_base, get_headers
 from agentx.exceptions import AgentXError, AgentXAuthError, AgentXValidationError
 
@@ -45,7 +46,9 @@ class ImprovementGroupsClient:
         self._workspace_id = workspace_id
         self._base_url = (base_url or api_base()).rstrip("/")
 
-    def _request(self, method: str, path: str, json: Any = None, timeout: int = 120) -> Any:
+    def _request(
+        self, method: str, path: str, json: Any = None, timeout: int = 120, retry: bool = True
+    ) -> Any:
         params = None
         if self._workspace_id:
             # Mirrors MonitorClient._workspace_params/_with_workspace: GETs (and DELETEs)
@@ -57,9 +60,12 @@ class ImprovementGroupsClient:
                     json = {**json, "workspaceId": self._workspace_id}
             else:
                 params = {"workspaceId": self._workspace_id}
-        resp = requests.request(
+        # retry=False for ANY non-idempotent write (member deletes, the report-generating
+        # POST) - MonitorClient._request's posture, via the shared monitor transport.
+        resp = request_with_retries(
             method,
             f"{self._base_url}/agent-monitoring{path}",
+            retry=retry,
             headers={**get_headers(self._api_key), "Content-Type": "application/json"},
             json=json,
             params=params,
@@ -92,7 +98,9 @@ class ImprovementGroupsClient:
 
     def remove_member(self, group_id: str, member_id: str) -> None:
         """Prune a member before spending the group (a confirm that turned out uninteresting)."""
-        self._request("DELETE", f"/improvement-groups/{group_id}/members/{member_id}")
+        # retry=False: a lost response + transport retry would turn a successful delete
+        # into a spurious 404.
+        self._request("DELETE", f"/improvement-groups/{group_id}/members/{member_id}", retry=False)
 
     def generate_report(self, group_id: str, model: Optional[str] = None) -> Dict[str, Any]:
         """Spend the group: one real LLM call clustering the confirmed failures into issues
@@ -101,7 +109,11 @@ class ImprovementGroupsClient:
         payload: Dict[str, Any] = {}
         if model is not None:
             payload["model"] = model
-        return self._request("POST", f"/improvement-groups/{group_id}/report", json=payload, timeout=300)["report"]
+        # retry=False: spends the group (real LLM billing) - a client-side timeout must not
+        # fire the same generation twice while the first still runs server-side.
+        return self._request(
+            "POST", f"/improvement-groups/{group_id}/report", json=payload, timeout=300, retry=False
+        )["report"]
 
     def list_reports(self) -> List[Dict[str, Any]]:
         return self._request("GET", "/improvement-reports").get("improvementReports", [])

{agentx_python-0.8.24 → agentx_python-0.8.25}/agentx/monitor/judge_scorers.py

@@ -5,6 +5,7 @@ from typing import Any, Dict, List, Optional
 
 import requests
 
+from agentx.monitor._transport import request_with_retries
 from agentx.util import api_base, get_headers
 from agentx.exceptions import AgentXError, AgentXAuthError, AgentXValidationError
 
@@ -97,7 +98,9 @@ class JudgeScorersClient:
         # Captured once at construction so two clients with different bases can coexist.
         self._base_url = (base_url or api_base()).rstrip("/")
 
-    def _request(self, method: str, path: str, json: Any = None, timeout: int = 60) -> Any:
+    def _request(
+        self, method: str, path: str, json: Any = None, timeout: int = 60, retry: bool = True
+    ) -> Any:
         params = None
         if self._workspace_id:
             # Mirrors MonitorClient._workspace_params/_with_workspace: GETs (and DELETEs)
@@ -109,9 +112,13 @@ class JudgeScorersClient:
                     json = {**json, "workspaceId": self._workspace_id}
             else:
                 params = {"workspaceId": self._workspace_id}
-        resp = requests.request(
+        # retry=False for ANY non-idempotent write (creates, deletes) and judge-spending POST
+        # (tune/validate/publish) - MonitorClient._request's posture, via the shared monitor
+        # transport.
+        resp = request_with_retries(
             method,
             f"{self._base_url}/agent-monitoring{path}",
+            retry=retry,
             headers={**get_headers(self._api_key), "Content-Type": "application/json"},
             json=json,
             params=params,
@@ -263,7 +270,10 @@ class JudgeScorersClient:
             payload["offline"] = offline
         if online is not None:
             payload["online"] = online
-        return JudgeScorer(self._request("POST", "/judge-scorers", json=payload)["judgeScorer"])
+        # Server-side create: a timeout + transport retry would create the scorer twice.
+        return JudgeScorer(
+            self._request("POST", "/judge-scorers", json=payload, retry=False)["judgeScorer"]
+        )
 
     def get(self, scorer_id: str) -> JudgeScorer:
         return JudgeScorer(self._request("GET", f"/judge-scorers/{scorer_id}")["judgeScorer"])
@@ -300,7 +310,9 @@ class JudgeScorersClient:
     def delete(self, scorer_id: str) -> None:
         """Delete the scorer: rubric, version history, and online profile together.
         Irreversible; refused for the built-in Session Baseline Judge."""
-        self._request("DELETE", f"/judge-scorers/{scorer_id}")
+        # retry=False: a lost response + transport retry would turn a successful delete
+        # into a spurious 404.
+        self._request("DELETE", f"/judge-scorers/{scorer_id}", retry=False)
 
     # ------------------------------------------------------------------
     # Online-profile pass-throughs (calibration / tuning / ratings / events)
@@ -334,8 +346,14 @@ class JudgeScorersClient:
     def tune(self, scorer_id: str, window: str = "7d") -> dict:
         """Propose a rewrite of the rubric from calibration disagreements (LLM call, slow).
         ``window`` accepts the same values as :meth:`calibration`, including "rubric"."""
+        # retry=False (judge-spending POST, MonitorClient.propose_online_evaluator_tuning's
+        # posture): a client-side timeout must not fire the same LLM-billing work twice.
         data = self._request(
-            "POST", f"/online-evaluators/{self._profile_id(scorer_id)}/tune", json={"window": window}, timeout=300
+            "POST",
+            f"/online-evaluators/{self._profile_id(scorer_id)}/tune",
+            json={"window": window},
+            timeout=300,
+            retry=False,
         )
         # The wire wraps the proposal ({"proposal": {...}}); unwrap like the legacy client so
         # proposal["reasoning"] / the criteria fields are directly addressable.
@@ -344,11 +362,13 @@ class JudgeScorersClient:
     def validate_tuning(self, scorer_id: str, criteria: Dict[str, Any], window: str = "7d") -> dict:
         """Re-judge the disagreement + control cases with candidate criteria (LLM calls, slow)."""
         # The wire takes the criteria fields at the TOP level of the body, not nested.
+        # retry=False (judge-spending POST) - same posture as tune() above.
         return self._request(
             "POST",
             f"/online-evaluators/{self._profile_id(scorer_id)}/tune/validate",
             json={**criteria, "window": window},
             timeout=600,
+            retry=False,
         )
 
     def publish_tuning(
@@ -382,7 +402,14 @@ class JudgeScorersClient:
             payload["validation"] = validation_payload
         if force:
             payload["force"] = True
-        return self._request("POST", f"/online-evaluators/{self._profile_id(scorer_id)}/tune/publish", json=payload)
+        # retry=False: a non-idempotent write (each publish appends a rubric version) -
+        # MonitorClient.publish_online_evaluator_tuning's posture.
+        return self._request(
+            "POST",
+            f"/online-evaluators/{self._profile_id(scorer_id)}/tune/publish",
+            json=payload,
+            retry=False,
+        )
 
     def ratings(self, scorer_id: str, window: str = "7d") -> "List[OnlineEvaluatorRatingPoint]":
         """Bucketed average-rating-over-time for this scorer's live checks - same typed points

agentx_python-0.8.25/agentx/monitor/patterns.py

@@ -0,0 +1,250 @@
+from __future__ import annotations
+
+import logging
+from typing import Any, Dict, List, Optional, TYPE_CHECKING
+
+from agentx.monitor.models import MonitorPattern
+
+if TYPE_CHECKING:
+    from agentx.monitor.client import MonitorClient
+
+logger = logging.getLogger(__name__)
+
+
+class MonitorPatternBuilder:
+    """Fluent builder for creating a custom pattern. ``detector_kind`` selects which of
+    ``include_terms``/``regex``/``semantic_prompt`` is used:
+
+    - ``"contains"`` (default): ``include_terms`` - a match if any (or all, with
+      ``match_mode="all"``) phrase appears in the target text.
+    - ``"regex"``: ``regex`` - a single regular expression.
+    - ``"semantic"``: ``semantic_prompt`` - an LLM judges whether the response violates the
+      described rubric.
+
+    ``conditions`` (self-host) is the engine's full N-condition model - a list of condition
+    dicts, each with its own detector kind and match settings - passed through verbatim to
+    the create payload; when set, the engine honors it as the pattern's whole rule set and
+    the flat fields above are only legacy display metadata.
+    """
+
+    def __init__(
+        self,
+        client: "MonitorClient",
+        name: str,
+        description: Optional[str] = None,
+        category: Optional[str] = None,
+        detector_kind: str = "contains",
+        match_target: Optional[List[str]] = None,
+        match_mode: str = "any",
+        include_terms: Optional[List[str]] = None,
+        exclude_terms: Optional[List[str]] = None,
+        regex: Optional[str] = None,
+        semantic_prompt: Optional[str] = None,
+        severity: str = "medium",
+        polarity: str = "failure",
+        enabled: bool = True,
+        sample_rate: float = 1.0,
+        scope_mode: str = "all",
+        agent_ids: Optional[List[str]] = None,
+        conditions: Optional[List[dict]] = None,
+    ):
+        self._client = client
+        self._payload: Dict[str, Any] = {
+            "name": name,
+            "description": description,
+            "category": category,
+            "detectorKind": detector_kind,
+            "matchTarget": match_target or ["response"],
+            "matchMode": match_mode,
+            "includeTerms": include_terms or [],
+            "excludeTerms": exclude_terms or [],
+            "regex": regex,
+            "semanticPrompt": semantic_prompt,
+            "severity": severity,
+            # A "failure" pattern (default) raises a signal to triage; a "proper" pattern logs
+            # a healthy tally instead.
+            "polarity": polarity,
+            "enabled": enabled,
+            "sampleRate": sample_rate,
+            "scopeMode": scope_mode,
+            "agentIds": agent_ids or [],
+        }
+        # Passed through verbatim - the engine honors body.conditions as the full
+        # N-condition model (see the class docstring).
+        if conditions is not None:
+            self._payload["conditions"] = conditions
+
+    def publish(self) -> MonitorPattern:
+        logger.info("Publishing monitor pattern '%s'", self._payload["name"])
+        return self._client.create_pattern(self._payload)
+
+
+class MonitorPatternClient:
+    """Thin wrapper surfaced as ``client.monitor.patterns``."""
+
+    def __init__(self, client: "MonitorClient"):
+        self._client = client
+
+    def builder(
+        self,
+        name: str,
+        description: Optional[str] = None,
+        category: Optional[str] = None,
+        detector_kind: str = "contains",
+        match_target: Optional[List[str]] = None,
+        match_mode: str = "any",
+        include_terms: Optional[List[str]] = None,
+        exclude_terms: Optional[List[str]] = None,
+        regex: Optional[str] = None,
+        semantic_prompt: Optional[str] = None,
+        severity: str = "medium",
+        polarity: str = "failure",
+        enabled: bool = True,
+        sample_rate: float = 1.0,
+        scope_mode: str = "all",
+        agent_ids: Optional[List[str]] = None,
+        conditions: Optional[List[dict]] = None,
+    ) -> MonitorPatternBuilder:
+        return MonitorPatternBuilder(
+            self._client,
+            name=name,
+            description=description,
+            category=category,
+            detector_kind=detector_kind,
+            match_target=match_target,
+            match_mode=match_mode,
+            include_terms=include_terms,
+            exclude_terms=exclude_terms,
+            regex=regex,
+            semantic_prompt=semantic_prompt,
+            severity=severity,
+            polarity=polarity,
+            enabled=enabled,
+            sample_rate=sample_rate,
+            scope_mode=scope_mode,
+            agent_ids=agent_ids,
+            conditions=conditions,
+        )
+
+    def delete(self, pattern_id: str) -> None:
+        """Delete a pattern. Its historical signals remain as history."""
+        # retry=False: a lost response + transport retry would turn a successful
+        # delete into a spurious 404.
+        self._client._request(
+            "DELETE",
+            f"/agent-monitoring/patterns/{pattern_id}",
+            base=self._client._api_root(),
+            retry=False,
+        )
+
+    # snake_case -> wire camelCase, same posture as rules.update: the engine reads only the
+    # camelCase key and silently keeps the stored value for anything it does not recognize -
+    # update(sample_rate=0.05) used to 200 with the rate unchanged.
+    _UPDATE_ALIASES = {
+        "sample_rate": "sampleRate",
+        "scope_mode": "scopeMode",
+        "agent_ids": "agentIds",
+        "detector_kind": "detectorKind",
+        "include_terms": "includeTerms",
+        "exclude_terms": "excludeTerms",
+        "regex": "regex",
+        "semantic_prompt": "semanticPrompt",
+        "match_mode": "matchMode",
+        "match_target": "matchTarget",
+    }
+
+    # The engine's PUT rebuilds the pattern's WHOLE conditions array (legacyPayloadToConditions:
+    # "a full replace, not a sparse patch") exactly when the body carries one of these - its own
+    # sentConditionFields set, minus "conditions". Bodies without any of them keep the stored
+    # conditions untouched.
+    _TRIGGER_FIELDS = ("includeTerms", "regex", "semanticPrompt")
+
+    # Sent alone, these look sparse but cannot land: without a trigger field the engine never
+    # rebuilds conditions (the values are silently ignored). For excludeTerms/matchMode there is
+    # also nothing to merge them over client-side - the wire GET reports display-only
+    # placeholders (includeTerms/excludeTerms always [], matchMode always "any"; conditions is
+    # the only truth). matchTarget IS reported faithfully, but the engine only reads it during a
+    # rebuild, so alone it is the same silent no-op.
+    _UNMERGEABLE_ALONE = ("excludeTerms", "matchMode", "matchTarget")
+
+    def update(self, pattern_id: str, **fields: Any) -> MonitorPattern:
+        """Update a pattern and return the updated :class:`MonitorPattern`. Accepts snake_case
+        kwargs (``sample_rate=0.05``) or the wire's camelCase; an unrecognized snake_case key
+        raises instead of silently changing nothing.
+
+        The real contract on self-host: the stored truth is the pattern's ``conditions`` array,
+        and the flat fields the wire GET returns are display-only placeholders (``includeTerms``/
+        ``excludeTerms`` always ``[]``, ``matchMode`` always ``"any"``, ``regex``/
+        ``semanticPrompt`` omitted). The engine's PUT rebuilds the WHOLE conditions array
+        whenever the body carries ``include_terms``, ``regex``, or ``semantic_prompt`` (or an
+        explicit ``conditions`` list, which wins outright). Consequences:
+
+        - ``update(pid, regex=...)`` (or include_terms/semantic_prompt) is a full detector
+          rewrite; the detector kind follows the trigger field actually sent (``regex`` ->
+          ``"regex"``, ``semantic_prompt`` -> ``"semantic"``, ``include_terms`` ->
+          ``"contains"``), so cross-kind updates work, and this client back-fills only
+          ``matchTarget`` from the stored pattern so the rebuild keeps its target. It never
+          back-fills ``includeTerms``/``excludeTerms``/``matchMode`` - the GET values are
+          placeholders, and copying them in would destroy real conditions.
+        - ``exclude_terms=``, ``match_mode=``, or ``match_target=`` alone raises ValueError:
+          the engine silently ignores them without a rebuild. Pass ``conditions=[...]`` (built
+          from ``get(pattern_id).conditions``) instead, or pass them alongside the
+          include_terms/regex/semantic_prompt they should be rebuilt with.
+        - Everything else stays a sparse metadata edit that leaves the stored conditions
+          untouched."""
+        payload: Dict[str, Any] = {}
+        for key, value in fields.items():
+            wire_key = self._UPDATE_ALIASES.get(key, key)
+            if "_" in wire_key:
+                raise ValueError(
+                    f"Unknown pattern field {key!r} - the engine reads camelCase keys and would "
+                    "silently ignore this (see MonitorPattern for the field names)."
+                )
+            payload[wire_key] = value
+        sends_conditions = "conditions" in payload
+        triggered = any(k in payload for k in self._TRIGGER_FIELDS)
+        if not sends_conditions and not triggered:
+            offending = [k for k in self._UNMERGEABLE_ALONE if k in payload]
+            if offending:
+                raise ValueError(
+                    f"{' and '.join(offending)} cannot be updated on their own: the engine only "
+                    "rebuilds a pattern's conditions when includeTerms/regex/semanticPrompt is "
+                    "sent (alone they are silently ignored), and the wire GET returns display-"
+                    "only placeholders (includeTerms/excludeTerms always [], matchMode always "
+                    "'any'), so there is no stored value to merge them over. Pass "
+                    "conditions=[...] built from get(pattern_id).conditions instead, or send "
+                    "them alongside the trigger field they should be rebuilt with."
+                )
+        if triggered and not sends_conditions:
+            stored = self.get(pattern_id)
+            # The rebuild's detector kind follows the trigger field actually sent - back-filling
+            # the STORED kind 400s a kind change (regex= on a "contains" pattern) and silently
+            # corrupts the inverse (include_terms= on a regex pattern would write phrase
+            # conditions while keeping detectorKind "regex").
+            if "regex" in payload:
+                inferred_kind = "regex"
+            elif "semanticPrompt" in payload:
+                inferred_kind = "semantic"
+            else:
+                inferred_kind = "contains"
+            payload.setdefault("detectorKind", inferred_kind)
+            # matchTarget is the one field the wire reports faithfully - it rides along so the
+            # server-side full-replace rebuild keeps the stored target. NEVER includeTerms/
+            # excludeTerms/matchMode - see _UNMERGEABLE_ALONE's comment.
+            payload.setdefault("matchTarget", stored.match_target)
+        data = self._client._request(
+            "PUT",
+            f"/agent-monitoring/patterns/{pattern_id}",
+            base=self._client._api_root(),
+            json=payload,
+            # An idempotent merge server-side (same payload, same result) - but the read-
+            # merge-write above is not atomic, so keep the single-shot posture.
+            retry=False,
+        )
+        return MonitorPattern(**data["pattern"])
+
+    def get(self, pattern_id: str) -> MonitorPattern:
+        return self._client.get_pattern(pattern_id)
+
+    def list(self) -> List[MonitorPattern]:
+        return self._client.list_patterns()

{agentx_python-0.8.24 → agentx_python-0.8.25}/agentx/monitor/rules.py

@@ -70,9 +70,22 @@ class MonitorRulesClient:
 
     def update(self, rule_id: str, **fields: Any) -> MonitorRule:
         """Sparse update. snake_case keys are mapped to the wire (``sample_rate`` ->
-        ``sampleRate``, ``action_config`` -> ``actionConfig``)."""
+        ``sampleRate``, ``action_config`` -> ``actionConfig``); an unrecognized snake_case
+        key raises instead of 200ing with the rule unchanged (the engine reads only camelCase
+        and silently keeps the stored value for keys it does not know)."""
         aliases = {"sample_rate": "sampleRate", "action_config": "actionConfig"}
-        payload = {aliases.get(k, k): v for k, v in fields.items()}
+        payload: Dict[str, Any] = {}
+        for key, value in fields.items():
+            wire_key = aliases.get(key, key)
+            if "_" in wire_key:
+                raise ValueError(
+                    f"Unknown rule field {key!r} - the engine reads camelCase keys and would "
+                    "silently ignore this (see MonitorRule for the field names)."
+                )
+            payload[wire_key] = value
+        # PUT /rules/:id is an idempotent full-body merge (same payload, same result), so the
+        # transport's default retry is safe - and skipping it just drops legitimate edits on a
+        # transient failure.
         data = self._request("PUT", f"/agent-monitoring/rules/{rule_id}", json=payload)
         return MonitorRule(data.get("rule", data))