From a262b2e372130fd59b013f7b10a7ed2ecb7f9357 Mon Sep 17 00:00:00 2001 From: kshitijk4poor <82637225+kshitijk4poor@users.noreply.github.com> Date: Sun, 6 Sep 2026 20:41:16 +0530 Subject: [PATCH] refactor(update): drop the tombstone comment at the old matcher site and a duplicated exactness leg --- hermes_cli/update_cmd_fleet.py | 4 ---- tests/hermes_cli/test_restart_plan_reconciliation.py | 7 ------- 2 files changed, 11 deletions(-) diff --git a/hermes_cli/update_cmd_fleet.py b/hermes_cli/update_cmd_fleet.py index aeaacecba2..a4246b6a69 100644 --- a/hermes_cli/update_cmd_fleet.py +++ b/hermes_cli/update_cmd_fleet.py @@ -543,10 +543,6 @@ def _surviving_gateway_pids_after_failed_restart(): return None -# `_gateway_service_matches_profile` is imported from update_inventory so plan -# reconciliation and abort-recovery share one systemd/launchd/s6 matcher. - - _MANUAL_GATEWAY_SKIP_REASON = ( "manual gateway has no supervisor relaunch authority; left running for explicit operator restart" ) diff --git a/tests/hermes_cli/test_restart_plan_reconciliation.py b/tests/hermes_cli/test_restart_plan_reconciliation.py index ea25d915d8..4066d57c57 100644 --- a/tests/hermes_cli/test_restart_plan_reconciliation.py +++ b/tests/hermes_cli/test_restart_plan_reconciliation.py @@ -162,13 +162,6 @@ def test_launchd_named_profile_and_failed_label(): failed_units=["ai.hermes.gateway"], ) assert failed[0]["outcome"] == "failed" - # Sibling label must not credit the default profile (exact match, not prefix). - sibling = match_runtime_outcomes( - _plan(_rt("default", 403, supervisor="launchd")), - restarted_services=["ai.hermes.gateway-foo"], relaunched_profiles=[], - externally_supervised_profiles=[], killed_pids=set(), failed_units=[], - ) - assert sibling[0]["outcome"] == "unaccounted" def test_untouched_runtime_is_unaccounted_and_escalates(capsys):