From 6670c7c326889893043a062c0ec79a081715f069 Mon Sep 17 00:00:00 2001 From: ProtocolWarden <32967198+ProtocolWarden@users.noreply.github.com> Date: Thu, 4 Jun 2026 16:26:31 -0400 Subject: [PATCH] =?UTF-8?q?fix(ci):=20green=20main=20=E2=80=94=20register?= =?UTF-8?q?=20reconcile=5Fenforce=20key=20+=20drop=20unused=20import?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Two pre-existing CI failures on main: - Custodian doctor (--strict): the audit key `reconcile_enforce` (added in #238) was never registered in `plugin_audit_keys`, so doctor flagged it as an "unknown audit key (typo?)" and --strict turned the warning into a non-zero exit. It is a legitimate OC plugin config flag — add it to plugin_audit_keys (matching the existing oc1_exempt/oc7_exempt/known_values registrations). - Lint (ruff): `unittest.mock.call` was imported but unused in tests/test_check_signal_collector.py (F401). Removed via `ruff check --fix`; `call` is not referenced elsewhere in the file. Verified locally with the repo's .venv: ruff check . -> All checks passed! custodian-doctor --strict --repo . -> OK pytest tests/test_check_signal_collector.py -> 16 passed Co-Authored-By: Claude Opus 4.8 --- .custodian/config.yaml | 1 + tests/test_check_signal_collector.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/.custodian/config.yaml b/.custodian/config.yaml index b7e4aa21..77d26932 100644 --- a/.custodian/config.yaml +++ b/.custodian/config.yaml @@ -682,6 +682,7 @@ audit: - known_values - oc1_exempt - oc7_exempt + - reconcile_enforce # Additional well-known string values for OC9 to recognise without # requiring a string-literal anchor in src/. diff --git a/tests/test_check_signal_collector.py b/tests/test_check_signal_collector.py index 77dd9954..493369a5 100644 --- a/tests/test_check_signal_collector.py +++ b/tests/test_check_signal_collector.py @@ -6,7 +6,7 @@ import sys from datetime import UTC, datetime from pathlib import Path -from unittest.mock import call, patch +from unittest.mock import patch from operations_center.observer.collectors.check_signal import CheckSignalCollector from operations_center.observer.service import ObserverContext