4b6817381b
Benchmark image — build + push to ECR (any adapter) / build + push (push) Waiting to run
CI / quality (ubuntu-latest) (push) Waiting to run
CI / test (tools-runtime) (push) Waiting to run
CI / test (e2e-general) (push) Waiting to run
CI / test (cli-runtime) (push) Waiting to run
CI / test (e2e-provider-and-openclaw) (push) Waiting to run
CI / test (integrations-and-misc) (push) Waiting to run
CI / coverage-report (push) Blocked by required conditions
CI / test-kubernetes (push) Waiting to run
CI / should-run-thorough (push) Waiting to run
CI / test-thorough (cloudwatch-demo) (push) Blocked by required conditions
CI / test-thorough (flink-ecs) (push) Blocked by required conditions
CI / test-thorough (upstream-lambda) (push) Blocked by required conditions
CI / test-thorough (prefect-ecs-fargate) (push) Blocked by required conditions
CodeQL / Analyze (python) (push) Waiting to run
Release / build-binaries (zip, opensre.exe, onefile, windows-latest, windows-x64) (push) Blocked by required conditions
Release / publish-release (push) Blocked by required conditions
Release / publish-main-release (push) Blocked by required conditions
Release / prepare (push) Waiting to run
Release / verify (push) Blocked by required conditions
Release / build-python-dist (push) Blocked by required conditions
Release / build-binaries (tar.gz, opensre, onedir, macos-15-intel, darwin-x64) (push) Blocked by required conditions
Release / build-binaries (tar.gz, opensre, onedir, macos-latest, darwin-arm64) (push) Blocked by required conditions
Release / build-binaries (tar.gz, opensre, onedir, ubuntu-22.04, linux-x64) (push) Blocked by required conditions
Release / build-binaries (tar.gz, opensre, onedir, ubuntu-22.04-arm, linux-arm64) (push) Blocked by required conditions
Synthetic Deterministic Tests / Synthetic offline (deterministic) (push) Waiting to run
Interactive Shell Live (PR + post-merge) / turn-checks (no-LLM) (push) Waiting to run
Interactive Shell Live (PR + post-merge) / turn-live shard ${{ matrix.shard_index }} (push) Waiting to run
CI (OpenClaw E2E) / openclaw test (push) Has been cancelled
95 lines
3.2 KiB
Python
95 lines
3.2 KiB
Python
"""Shared error-reporting helpers for boundary exception handling.
|
|
|
|
Use these helpers at every ``except`` site that intentionally swallows or
|
|
re-raises an exception, so the failure is always visible in Sentry and logs.
|
|
|
|
Tagging conventions (pass via ``tags``):
|
|
surface — cli | interactive_shell | service_client | tool | node |
|
|
pipeline | integration | remote_server | analytics | auth |
|
|
webapp | mcp | sandbox | deployment | masking
|
|
component — module-level identifier, e.g. ``integrations.grafana.tempo``
|
|
integration — vendor when applicable: grafana | splunk | vercel | …
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import logging
|
|
from collections.abc import Iterator
|
|
from contextlib import contextmanager
|
|
from typing import Any
|
|
|
|
from platform.observability.errors.sentry import capture_exception
|
|
|
|
|
|
def report_exception(
|
|
exc: BaseException,
|
|
*,
|
|
logger: logging.Logger,
|
|
message: str,
|
|
severity: str = "error",
|
|
tags: dict[str, str] | None = None,
|
|
extras: dict[str, Any] | None = None,
|
|
include_traceback: bool = True,
|
|
) -> None:
|
|
"""Log + Sentry-capture an exception with structured context.
|
|
|
|
Use at boundaries where an exception is intentionally swallowed (the
|
|
function returns a degraded value to its caller).
|
|
"""
|
|
log_fn = getattr(logger, severity, logger.error)
|
|
# Some expected fallback paths (e.g. remote network probe timeouts) should
|
|
# remain visible in logs without dumping a full traceback into the terminal UI.
|
|
log_fn("%s", message, exc_info=exc if include_traceback else False)
|
|
# Info-severity reports are expected/graceful fallbacks; skip Sentry to avoid noise.
|
|
if severity == "info":
|
|
return
|
|
combined: dict[str, Any] = {}
|
|
if tags:
|
|
combined.update({f"tag.{k}": v for k, v in tags.items()})
|
|
if extras:
|
|
combined.update(extras)
|
|
if tags:
|
|
capture_exception(exc, extra=combined or None, tags=tags)
|
|
else:
|
|
capture_exception(exc, extra=combined or None)
|
|
|
|
|
|
@contextmanager
|
|
def report_and_swallow(
|
|
*,
|
|
logger: logging.Logger,
|
|
message: str,
|
|
tags: dict[str, str] | None = None,
|
|
extras: dict[str, Any] | None = None,
|
|
swallow: type[BaseException] | tuple[type[BaseException], ...] = Exception,
|
|
) -> Iterator[None]:
|
|
"""Context manager that logs + reports a matching exception, then swallows it.
|
|
|
|
Replaces bare ``try/except Exception: pass`` patterns where the caller does
|
|
not need the value but does want the failure visible in Sentry.
|
|
"""
|
|
try:
|
|
yield
|
|
except swallow as exc:
|
|
report_exception(exc, logger=logger, message=message, tags=tags, extras=extras)
|
|
|
|
|
|
@contextmanager
|
|
def report_and_reraise(
|
|
*,
|
|
logger: logging.Logger,
|
|
message: str,
|
|
tags: dict[str, str] | None = None,
|
|
extras: dict[str, Any] | None = None,
|
|
) -> Iterator[None]:
|
|
"""Context manager that captures to Sentry + re-raises (for top-level boundaries)."""
|
|
try:
|
|
yield
|
|
except Exception as exc:
|
|
report_exception(exc, logger=logger, message=message, tags=tags, extras=extras)
|
|
raise
|
|
|
|
|
|
class OpenSRESilentFallback(Warning):
|
|
"""Warning class for debug-only fallback paths that should still reach Sentry."""
|