From fb14bc4e11d67a281d6acfa3a7307ef447d9e4c9 Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Thu, 3 Sep 2026 09:31:54 -0700 Subject: [PATCH] review-fix(whitespace): strip trailing whitespace and EOF blank lines introduced by this PR Trailing-whitespace-only edits so 'git diff --check BASE HEAD' is clean (16 diagnostics across 11 files). No code changes. --- agent/web_search_registry.py | 1 - cli.py | 4 ++-- cron/jobs.py | 4 ++-- gateway/run_turn.py | 1 - hermes_cli/cli_session_mixin.py | 2 +- hermes_cli/config.py | 4 ++-- hermes_cli/models.py | 2 -- plugins/platforms/telegram/adapter.py | 1 - run_agent.py | 6 +++--- tools/skills_hub.py | 1 - tools/tts_tool_lifecycle.py | 1 - 11 files changed, 10 insertions(+), 17 deletions(-) diff --git a/agent/web_search_registry.py b/agent/web_search_registry.py index 5913f2fa90..813f06bd95 100644 --- a/agent/web_search_registry.py +++ b/agent/web_search_registry.py @@ -193,4 +193,3 @@ def get_active_search_provider() -> Optional[WebSearchProvider]: def get_active_extract_provider() -> Optional[WebSearchProvider]: """Resolve the currently-active web extract provider.""" return _resolve(_configured_backend("extract"), capability="extract") - diff --git a/cli.py b/cli.py index 533d69564c..0363da0528 100644 --- a/cli.py +++ b/cli.py @@ -3031,7 +3031,7 @@ class HermesCLI(CLIAgentSetupMixin, CLICommandsMixin, CLIBillingMixin, CLITuiMix terminal_env = os.getenv("TERMINAL_ENV", "local") terminal_cwd = os.getenv("TERMINAL_CWD", os.getcwd()) terminal_timeout = os.getenv("TERMINAL_TIMEOUT", "60") - + config_path = _hermes_home / 'config.yaml' if not config_path.exists(): config_path = Path(__file__).parent / 'cli-config.yaml' @@ -3087,7 +3087,7 @@ class HermesCLI(CLIAgentSetupMixin, CLICommandsMixin, CLIBillingMixin, CLITuiMix for label, value in rows: print(f" {label} {value}") print() - + # canonical command -> (method name, pass cmd_original?). Absent commands resolve to # ``_handle__command(cmd)``. Looked up via getattr at dispatch time so # monkeypatching works. A handler returning False exits the REPL. diff --git a/cron/jobs.py b/cron/jobs.py index 70b015a18b..b78ffaf5e7 100644 --- a/cron/jobs.py +++ b/cron/jobs.py @@ -2614,7 +2614,7 @@ def _repair_timezone_shifted_cron(d: _DueJob) -> bool: def _rearm_stale_error_recurring(d: _DueJob) -> datetime: """Re-arm a recurring job wedged in persisted last_status=error; returns the effective next_run_dt. - + Such a job errored, mark_job_run parked next_run_at in the future, and nothing re-dispatched it (the in-memory stale-claim sweep cannot see it). Interval jobs re-arm to now (always a legal fire); cron jobs re-arm to the next LEGAL occurrence, since re-arming to now would fire at times @@ -2680,7 +2680,7 @@ def _reanchor_stale_cron(d: _DueJob) -> bool: def _fast_forward_missed_recurring(d: _DueJob, grace: int) -> None: """Recurring job past its grace window: skip the accumulated misses, fire once now. - + The fast-forward is persisted immediately — NOT redundant with advance_next_run/mark_job_run: it protects the crash window before mark_job_run and covers the external fire_due path, which never diff --git a/gateway/run_turn.py b/gateway/run_turn.py index 42e8870b86..2363d1a48c 100644 --- a/gateway/run_turn.py +++ b/gateway/run_turn.py @@ -3693,4 +3693,3 @@ class GatewayTurnMixin: await self._run_agent_mark_streamed_delivery(response, turn_ctx) self._run_agent_schedule_bubble_cleanup(response, _cleanup_adapter, turn_ctx) return response - diff --git a/hermes_cli/cli_session_mixin.py b/hermes_cli/cli_session_mixin.py index 9613686c52..dc3df7e1dd 100644 --- a/hermes_cli/cli_session_mixin.py +++ b/hermes_cli/cli_session_mixin.py @@ -181,7 +181,7 @@ class CLISessionMixin: if is_session_yolo_enabled(session_key): return enable_session_yolo(session_key) - _dim_notice(self, + _dim_notice(self, "⚡ YOLO mode restored from session — all commands auto-approved. /yolo to turn off.", quiet) diff --git a/hermes_cli/config.py b/hermes_cli/config.py index 2ce83382de..21615522c5 100644 --- a/hermes_cli/config.py +++ b/hermes_cli/config.py @@ -2228,9 +2228,9 @@ def save_config( if is_managed(): managed_error("save configuration") return - + config = _strip_managed_keys_for_save(config) - + ensure_hermes_home() config_path = get_config_path() require_readable_config_before_write(config_path) diff --git a/hermes_cli/models.py b/hermes_cli/models.py index 6817505625..f8e738f965 100644 --- a/hermes_cli/models.py +++ b/hermes_cli/models.py @@ -2369,5 +2369,3 @@ def cached_fetch_api_models( if _cache_entry_valid(entry, fp): return list(entry["models"]) return live - - diff --git a/plugins/platforms/telegram/adapter.py b/plugins/platforms/telegram/adapter.py index 9afec38fcf..3a56697339 100644 --- a/plugins/platforms/telegram/adapter.py +++ b/plugins/platforms/telegram/adapter.py @@ -6196,4 +6196,3 @@ def register(ctx) -> None: install_hint="Run `hermes setup` to install Telegram support.", setup_fn=interactive_setup, apply_yaml_config_fn=_apply_yaml_config, allowed_users_env="TELEGRAM_ALLOWED_USERS", allow_all_env="TELEGRAM_ALLOW_ALL_USERS", cron_deliver_env_var="TELEGRAM_HOME_CHANNEL", standalone_sender_fn=_standalone_send, max_message_length=4096, emoji="✈️", allow_update_command=True) - diff --git a/run_agent.py b/run_agent.py index 191801bd10..999dcfe1ff 100644 --- a/run_agent.py +++ b/run_agent.py @@ -1341,13 +1341,13 @@ _LIST_TOOLS_USAGE = """ python run_agent.py --enabled_toolsets=research --query='search for Python news' python run_agent.py --enabled_toolsets=development --query='debug this code' python run_agent.py --enabled_toolsets=safe --query='analyze without terminal' - + # Combine multiple toolsets python run_agent.py --enabled_toolsets=web,vision --query='analyze website' - + # Disable toolsets python run_agent.py --disabled_toolsets=terminal --query='no command execution' - + # Run with trajectory saving enabled python run_agent.py --save_trajectories --query='your question here'""" diff --git a/tools/skills_hub.py b/tools/skills_hub.py index 9d0d9aeeff..74ecb088df 100644 --- a/tools/skills_hub.py +++ b/tools/skills_hub.py @@ -342,4 +342,3 @@ def ensure_hub_dirs() -> None: ): if not path.exists(): path.write_text(initial, encoding="utf-8") - diff --git a/tools/tts_tool_lifecycle.py b/tools/tts_tool_lifecycle.py index 7c30bb059c..6f1ff7d712 100644 --- a/tools/tts_tool_lifecycle.py +++ b/tools/tts_tool_lifecycle.py @@ -166,4 +166,3 @@ def tts_lease_holders() -> List[str]: def _reset_tts_leases_for_tests() -> None: with _tts_lease_lock: _tts_leases.clear() -