d56a48d626
Every pre-apply exit of `hermes update` records one closed token on the receipt at the exit itself (update_receipt.record_stop_reason -> stop_class), and shared_metrics_update.update_failure_class maps it to the run's failure_class. aborted_before_apply stays only as the fallback for an exit with no recorded reason. Exits that fire before the receipt opens (update-lock refusal, Git operation in progress, managed install) write no receipt, as before, but now record a metrics-only row with the same closed class. The parked receipt copy keeps stop_class, exit_code, the stop reason's leading label and the restart/user-action flags, so a parked run classifies the same as the in-process one. A run that committed and is owed only an interrupt or the user's parked changes reads outcome partial (failure_class interrupted / local_changes_parked), never failed.
39 lines
1.5 KiB
Python
39 lines
1.5 KiB
Python
"""Shared leaf helpers for the ``hermes update`` modules (no Hermes imports; no cycle)."""
|
|
|
|
import logging
|
|
import sys
|
|
from contextlib import contextmanager
|
|
|
|
# Log-record parity with the origin module.
|
|
logger = logging.getLogger("hermes_cli.update_cmd")
|
|
|
|
|
|
def _record_stop(reason: str, *, without_receipt: "str | None" = None) -> None:
|
|
"""Name the exit about to stop this update as one closed token (see update_receipt.record_stop_reason).
|
|
|
|
``without_receipt`` (``refused``/``failed``) is for an exit that fires before the receipt
|
|
opens: a metrics row only. The receipt module is read from ``sys.modules``, never imported:
|
|
an older updater can load this tree after the swap while its own, older ``update_receipt``
|
|
(without these calls) stays loaded.
|
|
"""
|
|
receipt = sys.modules.get("hermes_cli.update_receipt")
|
|
if without_receipt is None:
|
|
record = getattr(receipt, "record_stop_reason", None)
|
|
if record is not None:
|
|
record(reason)
|
|
elif (record := getattr(receipt, "record_stop_without_receipt", None)) is not None:
|
|
record(reason, without_receipt)
|
|
|
|
|
|
@contextmanager
|
|
def _best_effort(message: str):
|
|
"""Run a non-critical update step; swallow ``Exception`` and log it at debug.
|
|
|
|
The updater must never die on bookkeeping (receipt, notices, cache seeds):
|
|
``message`` is the ``%s``-style debug line the inline ``try/except`` used.
|
|
"""
|
|
try:
|
|
yield
|
|
except Exception as exc:
|
|
logger.debug(message, exc)
|