Files
hermes-agent/tests/agent/test_proactive_prune_restart_safety.py
ethernet 890bbbda1f Merge remote-tracking branch 'origin/main' into ethie/pm-clean
# Conflicts:
#	apps/desktop/e2e/archived-hidden-session-recoverable.spec.ts
#	apps/desktop/e2e/bot-chat-message-agent-friendly-name.spec.ts
#	apps/desktop/e2e/bot-mailbox-unreadable-ticket.spec.ts
#	apps/desktop/e2e/bot-mode-roster-localized.spec.ts
#	apps/desktop/e2e/bot-mode-row-click-mirrors-registry.spec.ts
#	apps/desktop/e2e/bot-mode-tab-shows-bot-name.spec.ts
#	apps/desktop/e2e/bot-roster-group-row-organisation.spec.ts
#	apps/desktop/e2e/bot-roster-ignores-infra-dirs.spec.ts
#	apps/desktop/e2e/bot-roster-timestamp-meta.spec.ts
#	apps/desktop/e2e/bot-roster-user-sections.spec.ts
#	apps/desktop/e2e/bot-routines-pane-narrow.spec.ts
#	apps/desktop/e2e/bot-row-open-recent-session.spec.ts
#	apps/desktop/e2e/bot-tile-ignores-ambient-composer-model.spec.ts
#	apps/desktop/e2e/group-composer-auto-grow.spec.ts
#	apps/desktop/e2e/group-create-gate-remote-roster.spec.ts
#	apps/desktop/e2e/group-prompt-renamed-primary-handle.spec.ts
#	apps/desktop/e2e/hosted-room-backend-continuity.spec.ts
#	apps/desktop/e2e/hosted-room-legacy-store-migration.spec.ts
#	apps/desktop/e2e/settings-scope-chips-bot-title.spec.ts
#	apps/desktop/e2e/worktree-branch-status.spec.ts
#	apps/desktop/electron/backend-probes.test.ts
#	apps/desktop/electron/connection-apply.test.ts
#	apps/desktop/electron/desktop-electron-pin.test.ts
#	apps/desktop/electron/desktop-uninstall.test.ts
#	apps/desktop/electron/gateway-file-download-transport.test.ts
#	apps/desktop/electron/gateway-stop-before-update.test.ts
#	apps/desktop/electron/github-api-auth.test.ts
#	apps/desktop/electron/registry-primary-profile-scope.test.ts
#	apps/desktop/electron/update-api-check.test.ts
#	apps/desktop/electron/update-handoff-marker.test.ts
#	apps/desktop/electron/venv-blocker-scan.test.ts
#	apps/desktop/scripts/after-extract.test.mjs
#	apps/desktop/scripts/local-pack-publish.test.mjs
#	apps/desktop/scripts/tasks-scroll.test.mjs
#	apps/desktop/src/app/settings/model-settings.test.tsx
#	apps/desktop/src/app/updates-overlay.blockers.test.tsx
#	apps/desktop/src/components/desktop-install-overlay.test.tsx
#	apps/desktop/src/lib/update-copy.test.ts
#	scripts/ci/check_os_marker_fakes.py
#	tests-js/desktop-mac-usage-descriptions.test.ts
#	tests-js/node-engine-alignment.test.ts
#	tests/agent/lsp/test_install_and_lint_fixes.py
#	tests/agent/test_command_token_source.py
#	tests/agent/test_compression_boundary_hook.py
#	tests/agent/test_create_openai_client_ssl_verify.py
#	tests/agent/test_custom_provider_ca_probes.py
#	tests/agent/test_endpoint_blackhole.py
#	tests/agent/test_estimator_parity.py
#	tests/agent/test_in_place_compaction.py
#	tests/agent/test_moa_loop_mode.py
#	tests/agent/test_model_metadata.py
#	tests/agent/test_skill_session_platform_gate.py
#	tests/agent/test_skill_utils.py
#	tests/agent/test_ssl_ca_guard.py
#	tests/computer_use/test_doctor.py
#	tests/cron/test_codex_execution_paths.py
#	tests/cron/test_cron_bot_chat_delivery.py
#	tests/cron/test_cron_script.py
#	tests/cron/test_media_delivery_parity.py
#	tests/cron/test_misfire_catchup.py
#	tests/cron/test_parallel_pool.py
#	tests/cron/test_recurring_eagain_redispatch.py
#	tests/gateway/test_choice_picker.py
#	tests/gateway/test_control_socket_windows_live.py
#	tests/gateway/test_dingtalk.py
#	tests/gateway/test_feishu.py
#	tests/gateway/test_feishu_onboard.py
#	tests/gateway/test_gateway_shutdown.py
#	tests/gateway/test_matrix.py
#	tests/gateway/test_model_command_custom_providers.py
#	tests/gateway/test_reasoning_command.py
#	tests/gateway/test_runtime_footer.py
#	tests/gateway/test_session.py
#	tests/gateway/test_session_hygiene.py
#	tests/gateway/test_status.py
#	tests/gateway/test_teams.py
#	tests/gateway/test_turn_lease.py
#	tests/gateway/test_whatsapp_connect.py
#	tests/hermes_cli/test_approvals_command.py
#	tests/hermes_cli/test_auth_store_lock_concurrent.py
#	tests/hermes_cli/test_backup.py
#	tests/hermes_cli/test_banner_git_state.py
#	tests/hermes_cli/test_certifi_repair.py
#	tests/hermes_cli/test_cmd_update.py
#	tests/hermes_cli/test_compat_manifest_targets.py
#	tests/hermes_cli/test_computer_use_cli.py
#	tests/hermes_cli/test_cpr_local_leak.py
#	tests/hermes_cli/test_dashboard_auth_gate.py
#	tests/hermes_cli/test_dashboard_procs_kill_grace.py
#	tests/hermes_cli/test_desktop_lifecycle_windows_live.py
#	tests/hermes_cli/test_doctor.py
#	tests/hermes_cli/test_doctor_command_install.py
#	tests/hermes_cli/test_fleet_config_migration_windows_live.py
#	tests/hermes_cli/test_gateway.py
#	tests/hermes_cli/test_gateway_platform_gating.py
#	tests/hermes_cli/test_gateway_restart_loop.py
#	tests/hermes_cli/test_gateway_task_probe.py
#	tests/hermes_cli/test_gateway_wsl.py
#	tests/hermes_cli/test_gui_command.py
#	tests/hermes_cli/test_install_cua_driver.py
#	tests/hermes_cli/test_kanban_db.py
#	tests/hermes_cli/test_lazy_command_exports.py
#	tests/hermes_cli/test_lazy_refresh_venv_repair.py
#	tests/hermes_cli/test_linux_desktop_entry.py
#	tests/hermes_cli/test_local_runtime.py
#	tests/hermes_cli/test_local_runtime_updates.py
#	tests/hermes_cli/test_managed_uv.py
#	tests/hermes_cli/test_mcp_reload_confirm_gate.py
#	tests/hermes_cli/test_nous_subscription.py
#	tests/hermes_cli/test_npm_engine.py
#	tests/hermes_cli/test_personality_none.py
#	tests/hermes_cli/test_pet_toggle.py
#	tests/hermes_cli/test_plan_reconciliation_windows_live.py
#	tests/hermes_cli/test_plugin_event_bus.py
#	tests/hermes_cli/test_plugin_manifest_v2.py
#	tests/hermes_cli/test_plugin_packs.py
#	tests/hermes_cli/test_plugins_cmd.py
#	tests/hermes_cli/test_plugins_cmd_enable_disable_nested.py
#	tests/hermes_cli/test_process_identity.py
#	tests/hermes_cli/test_profiles.py
#	tests/hermes_cli/test_profiles_sidebar_cache.py
#	tests/hermes_cli/test_pty_bridge.py
#	tests/hermes_cli/test_resolve_turn_limit.py
#	tests/hermes_cli/test_serve_runtime_inventory.py
#	tests/hermes_cli/test_session_vacuum_config.py
#	tests/hermes_cli/test_set_config_value.py
#	tests/hermes_cli/test_signal_handler_kanban_worker.py
#	tests/hermes_cli/test_slash_confirm_windows.py
#	tests/hermes_cli/test_stale_pid_guard.py
#	tests/hermes_cli/test_startup_fast_guards.py
#	tests/hermes_cli/test_status.py
#	tests/hermes_cli/test_telegram_managed_bot.py
#	tests/hermes_cli/test_tools_config.py
#	tests/hermes_cli/test_update_apply_shallow_count.py
#	tests/hermes_cli/test_update_autostash.py
#	tests/hermes_cli/test_update_concurrent_quarantine.py
#	tests/hermes_cli/test_update_fetch_failure_classifier.py
#	tests/hermes_cli/test_update_fleet_probe_resume_token.py
#	tests/hermes_cli/test_update_handoff_backend_reap.py
#	tests/hermes_cli/test_update_handoff_desktop_rebuild.py
#	tests/hermes_cli/test_update_head_moved_gate.py
#	tests/hermes_cli/test_update_host_obligation.py
#	tests/hermes_cli/test_update_import_guard.py
#	tests/hermes_cli/test_update_interrupted_recovery.py
#	tests/hermes_cli/test_update_inventory.py
#	tests/hermes_cli/test_update_launchd_unloaded_gateway.py
#	tests/hermes_cli/test_update_missing_configured_deps.py
#	tests/hermes_cli/test_update_modified_notice.py
#	tests/hermes_cli/test_update_multiplex_migration_hook.py
#	tests/hermes_cli/test_update_no_gateway_restart.py
#	tests/hermes_cli/test_update_orphan_backend_reap.py
#	tests/hermes_cli/test_update_parked_branch_guard.py
#	tests/hermes_cli/test_update_post_pull_syntax_guard.py
#	tests/hermes_cli/test_update_receipt.py
#	tests/hermes_cli/test_update_self_lock.py
#	tests/hermes_cli/test_update_shim_fail_closed.py
#	tests/hermes_cli/test_update_shim_self_lock.py
#	tests/hermes_cli/test_update_sqlite_remediation.py
#	tests/hermes_cli/test_update_stale_dashboard.py
#	tests/hermes_cli/test_update_stale_virtualenv.py
#	tests/hermes_cli/test_update_venv_health.py
#	tests/hermes_cli/test_update_venv_ownership_preflight.py
#	tests/hermes_cli/test_update_wedged_gateway.py
#	tests/hermes_cli/test_update_yes_flag.py
#	tests/hermes_cli/test_update_zip_two_phase.py
#	tests/hermes_cli/test_urllib_security.py
#	tests/hermes_cli/test_ux_messages_auth_config.py
#	tests/hermes_cli/test_ux_messages_startup.py
#	tests/hermes_cli/test_venv_holder_classifier.py
#	tests/hermes_cli/test_verify_console_scripts.py
#	tests/hermes_cli/test_verify_core_dependencies.py
#	tests/hermes_cli/test_web_server.py
#	tests/hermes_cli/test_web_server_console_ws.py
#	tests/hermes_cli/test_web_server_ws_ping.py
#	tests/hermes_cli/test_web_ui_build.py
#	tests/hermes_state/test_fts_rebuild_admission.py
#	tests/hermes_state/test_hermes_state.py
#	tests/plugins/memory/test_memory_lazy_install.py
#	tests/plugins/test_google_meet_plugin.py
#	tests/plugins/test_langfuse_plugin.py
#	tests/plugins/test_security_guidance_plugin.py
#	tests/plugins/test_transform_llm_output_hook.py
#	tests/scripts/desktop_update/test_desktop_update_windows_gateway_flag.py
#	tests/scripts/desktop_update/test_desktop_update_windows_python_handoff.py
#	tests/scripts/desktop_update/test_desktop_update_windows_timestamp.py
#	tests/scripts/install/test_install_clone_throttle_fallback.py
#	tests/scripts/install/test_install_lockfile_churn.py
#	tests/scripts/install/test_install_no_initial_commit.py
#	tests/scripts/install/test_install_sh_browser_install.py
#	tests/scripts/install/test_install_sh_node_prerelease.py
#	tests/scripts/install/test_install_sh_symlink_stomp.py
#	tests/scripts/install/test_install_sh_uv_lock_config.py
#	tests/scripts/install/test_install_unmerged_index.py
#	tests/scripts/test_contributor_map.py
#	tests/scripts/test_run_tests_parallel.py
#	tests/skills/test_competitor_news_monitor_skill.py
#	tests/skills/test_document_to_action_items_skill.py
#	tests/skills/test_google_workspace_setup.py
#	tests/skills/test_google_workspace_setup_deps.py
#	tests/skills/test_grounded_citations_skill.py
#	tests/skills/test_ip_as_logo_skill.py
#	tests/skills/test_live_dashboard_skill.py
#	tests/skills/test_mcp_oauth_remote_gateway_skill.py
#	tests/skills/test_office_document_skills.py
#	tests/skills/test_openclaw_migration.py
#	tests/skills/test_product_price_monitor_skill.py
#	tests/skills/test_scrollcraft_skill.py
#	tests/skills/test_setup_wizard_generator_skill.py
#	tests/skills/test_weekly_review_planning_skill.py
#	tests/test_engines_satisfiable.py
#	tests/test_fast_safe_load.py
#	tests/test_hermes_bootstrap.py
#	tests/test_hermes_constants.py
#	tests/test_hermes_logging.py
#	tests/test_managed_runtime_resolution.py
#	tests/test_model_tools_async_bridge.py
#	tests/test_packaging_build_guard.py
#	tests/test_packaging_metadata.py
#	tests/test_yaml_indent_consistency.py
#	tests/tools/test_approval_timeout_overflow.py
#	tests/tools/test_base_environment.py
#	tests/tools/test_bot_mode_dm.py
#	tests/tools/test_browser_chromium_check.py
#	tests/tools/test_browser_hardening.py
#	tests/tools/test_browser_homebrew_paths.py
#	tests/tools/test_browser_npx_warmup.py
#	tests/tools/test_browser_orphan_reaper.py
#	tests/tools/test_browser_real_profile.py
#	tests/tools/test_browser_use_cli.py
#	tests/tools/test_clipboard.py
#	tests/tools/test_code_execution.py
#	tests/tools/test_code_execution_modes.py
#	tests/tools/test_code_execution_windows_env.py
#	tests/tools/test_computer_use.py
#	tests/tools/test_delegate_liveness_timeout.py
#	tests/tools/test_execute_code_approval_cluster.py
#	tests/tools/test_execution_flag_detection.py
#	tests/tools/test_fal_common.py
#	tests/tools/test_file_operations.py
#	tests/tools/test_file_tools.py
#	tests/tools/test_file_tools_cwd_resolution.py
#	tests/tools/test_file_tools_live.py
#	tests/tools/test_lazy_deps.py
#	tests/tools/test_lazy_deps_durable_target.py
#	tests/tools/test_lazy_deps_managed.py
#	tests/tools/test_local_env_blocklist.py
#	tests/tools/test_local_tempdir.py
#	tests/tools/test_macos_protected_search.py
#	tests/tools/test_mcp_npx_cached_bin.py
#	tests/tools/test_oneshot_completion_linger.py
#	tests/tools/test_process_registry.py
#	tests/tools/test_read_file_schema_gating.py
#	tests/tools/test_skill_improvements.py
#	tests/tools/test_skills_sync.py
#	tests/tools/test_termux_api_detection.py
#	tests/tools/test_tirith_security.py
#	tests/tools/test_transcription_tools.py
#	tests/tools/test_tts_streaming.py
#	tests/tools/test_wake_word.py
#	tests/tui_gateway/test_compute_host_borrowed_lease.py
#	tests/tui_gateway/test_compute_host_turn_protocol.py
#	tests/tui_gateway/test_isolated_orphan_activity.py
#	tests/tui_gateway/test_protocol.py
#	tests/tui_gateway/test_slash_worker_profile_home.py
#	tests/tui_gateway/test_subprocess_encoding.py
#	tests/tui_gateway/test_tui_gateway_server.py
#	ui-tui/src/__tests__/terminalParity.test.ts
#	ui-tui/src/__tests__/termuxComposerLayout.test.ts
#	ui-tui/src/__tests__/textInputFastEcho.test.ts
2026-09-23 07:02:44 -04:00

234 lines
9.0 KiB
Python

"""Restart-safety regressions for proactive tool-result pruning."""
from __future__ import annotations
import json
import os
from pathlib import Path
from unittest.mock import patch
import pytest
from agent.context_compressor import _estimate_msg_budget_tokens
from hermes_state import SessionDB
_REARM_KEY = "_proactive_prune_rearm_tokens"
def _assistant_call(call_id: str) -> dict:
return {
"role": "assistant",
"content": "",
"tool_calls": [{
"id": call_id,
"type": "function",
"function": {"name": "terminal", "arguments": '{"cmd":"ls"}'},
}],
}
def _tool_result(call_id: str, content: str) -> dict:
return {"role": "tool", "tool_call_id": call_id, "content": content}
def _history(*, large_chars: int = 24_000) -> list[dict]:
messages: list[dict] = [{"role": "user", "content": "start"}]
for index in range(8):
call_id = f"call_{index}"
messages.append(_assistant_call(call_id))
content = chr(65 + index) * large_chars if index < 3 else "ok"
messages.append(_tool_result(call_id, content))
return messages
def _build_agent(db: SessionDB, session_id: str, *, platform: str = "telegram"):
with patch.dict(os.environ, {"OPENROUTER_API_KEY": "test-key"}):
from run_agent import AIAgent
return AIAgent(
api_key="test-key",
base_url="https://openrouter.ai/api/v1",
model="test/model",
quiet_mode=True,
session_db=db,
session_id=session_id,
platform=platform,
skip_context_files=True,
skip_memory=True,
)
def _configure_pruning(agent) -> None:
compressor = agent.context_compressor
compressor.proactive_prune_tokens = 48_000
compressor.proactive_prune_min_result_chars = 8_000
compressor.proactive_prune_min_reclaim_tokens = 4_096
compressor.protect_first_n = 2
compressor.protect_last_n = 4
def _model_config(db: SessionDB, session_id: str) -> dict:
raw = db.get_session(session_id)["model_config"]
return json.loads(raw) if raw else {}
def test_gateway_eviction_reload_keeps_prune_and_durable_runway(tmp_path: Path) -> None:
"""A fresh gateway agent must reload both the pruned body and its runway."""
db = SessionDB(db_path=tmp_path / "state.db")
session_id = "GATEWAY_PRUNE_RESTART"
db.create_session(
session_id, source="telegram", model_config={"keep": "value"},
)
db.append_messages_batch(session_id, _history())
first_agent = _build_agent(db, session_id)
_configure_pruning(first_agent)
before = db.get_messages_as_conversation(session_id)
pruned, count = first_agent.context_compressor.prune_tool_results_only(
before, current_tokens=120_000,
)
assert count >= 1
durable = db.get_messages_as_conversation(session_id)
assert [message["content"] for message in durable] == [
message["content"] for message in pruned
]
assert len(durable[2]["content"]) < 24_000
stored_runway = _model_config(db, session_id)[_REARM_KEY]
assert _model_config(db, session_id)["keep"] == "value"
assert stored_runway > sum(map(_estimate_msg_budget_tokens, durable))
# Simulate gateway cache eviction / process restart: construct a wholly
# new AIAgent and load the active transcript from SQLite.
resumed_agent = _build_agent(db, session_id)
_configure_pruning(resumed_agent)
assert resumed_agent.context_compressor._proactive_prune_rearm_tokens == stored_runway
reloaded = db.get_messages_as_conversation(session_id)
archived_before = len(db.get_messages(session_id, include_inactive=True))
result, second_count = resumed_agent.context_compressor.prune_tool_results_only(
reloaded, current_tokens=1_000_000,
)
assert result is reloaded
assert second_count == 0
assert len(db.get_messages(session_id, include_inactive=True)) == archived_before
def test_fresh_agent_rearms_after_durable_history_regrowth_once(tmp_path: Path) -> None:
db = SessionDB(db_path=tmp_path / "state.db")
session_id = "PRUNE_DURABLE_REGROWTH"
db.create_session(session_id, source="telegram")
db.append_messages_batch(session_id, _history())
first_agent = _build_agent(db, session_id)
_configure_pruning(first_agent)
first, first_count = first_agent.context_compressor.prune_tool_results_only(
db.get_messages_as_conversation(session_id), current_tokens=120_000,
)
assert first_count >= 1
first_runway = _model_config(db, session_id)[_REARM_KEY]
growth = [
_assistant_call("regrown_large"),
_tool_result("regrown_large", "z" * 240_000),
_assistant_call("tail_1"),
_tool_result("tail_1", "ok"),
_assistant_call("tail_2"),
_tool_result("tail_2", "ok"),
]
db.append_messages_batch(session_id, growth)
resumed = _build_agent(db, session_id)
_configure_pruning(resumed)
grown = db.get_messages_as_conversation(session_id)
assert sum(map(_estimate_msg_budget_tokens, grown)) >= first_runway
second, second_count = resumed.context_compressor.prune_tool_results_only(
grown, current_tokens=1_000_000,
)
assert second_count >= 1
second_runway = _model_config(db, session_id)[_REARM_KEY]
assert second_runway > first_runway
restarted = _build_agent(db, session_id)
_configure_pruning(restarted)
durable = db.get_messages_as_conversation(session_id)
result, third_count = restarted.context_compressor.prune_tool_results_only(
durable, current_tokens=1_000_000,
)
assert result is durable
assert third_count == 0
assert restarted.context_compressor._proactive_prune_rearm_tokens == second_runway
def test_prune_persistence_failure_is_a_noop(tmp_path: Path) -> None:
db = SessionDB(db_path=tmp_path / "state.db")
session_id = "PRUNE_PERSISTENCE_FAILURE"
db.create_session(session_id, source="telegram")
db.append_messages_batch(session_id, _history())
agent = _build_agent(db, session_id)
_configure_pruning(agent)
messages = db.get_messages_as_conversation(session_id)
original_contents = [message["content"] for message in messages]
with patch.object(
db, "archive_and_compact", side_effect=RuntimeError("disk full"),
):
result, count = agent.context_compressor.prune_tool_results_only(
messages, current_tokens=120_000,
)
assert result is messages
assert count == 0
assert agent.context_compressor._proactive_prune_rearm_tokens == 0
assert [message["content"] for message in messages] == original_contents
assert [message["content"] for message in db.get_messages_as_conversation(session_id)] == original_contents
assert _REARM_KEY not in _model_config(db, session_id)
def test_archive_model_config_patch_rolls_back_with_transcript(tmp_path: Path) -> None:
db = SessionDB(db_path=tmp_path / "state.db")
session_id = "PRUNE_ATOMIC_ARCHIVE_FAILURE"
db.create_session(
session_id,
source="telegram",
model_config={"keep": "value", _REARM_KEY: 120_000},
)
original = [{"role": "user", "content": "original"}]
db.append_messages_batch(session_id, original)
with patch.object(
db, "_insert_message_rows", side_effect=RuntimeError("insert failed"),
):
with pytest.raises(RuntimeError, match="insert failed"):
db.archive_and_compact(
session_id,
[{"role": "user", "content": "replacement"}],
model_config_patch={_REARM_KEY: None},
)
assert db.get_messages_as_conversation(session_id)[0]["content"] == "original"
assert _model_config(db, session_id) == {"keep": "value", _REARM_KEY: 120_000}
def test_model_switch_clears_durable_runway(tmp_path: Path) -> None:
"""update_model must clear BOTH the in-memory and the durable runway."""
db = SessionDB(db_path=tmp_path / "state.db")
session_id = "MODEL_SWITCH_CLEARS_RUNWAY"
db.create_session(
session_id,
source="telegram",
model_config={"keep": "value", _REARM_KEY: 120_000},
)
agent = _build_agent(db, session_id)
compressor = agent.context_compressor
assert compressor._proactive_prune_rearm_tokens == 120_000
compressor.update_model("other/model", 200_000)
assert compressor._proactive_prune_rearm_tokens == 0
assert _REARM_KEY not in _model_config(db, session_id)
assert _model_config(db, session_id)["keep"] == "value"
def test_patch_session_model_config_merge_and_delete(tmp_path: Path) -> None:
db = SessionDB(db_path=tmp_path / "state.db")
session_id = "PATCH_MODEL_CONFIG"
db.create_session(
session_id, source="cli", model_config={"keep": "value", "drop": 1},
)
db.patch_session_model_config(session_id, {"drop": None, "added": 7})
assert _model_config(db, session_id) == {"keep": "value", "added": 7}
# Missing rows and empty patches are no-ops, never errors.
db.patch_session_model_config("NO_SUCH_SESSION", {"x": 1})
db.patch_session_model_config(session_id, {})