mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-07-11 13:41:53 +00:00
The extracted normalize_skill_lookup_name() resolved trusted roots via agent.skill_utils.get_skills_dir(), but skill_view() enforces tools.skills_tool.SKILLS_DIR — a separate module attribute that callers and 60+ existing tests patch directly. With the helper reading a different symbol than the enforcer, any SKILLS_DIR patch (or future divergence between the two resolvers) makes normalization disagree with enforcement and absolute-path loads regress silently. Read SKILLS_DIR at call time (deferred import, cycle-safe) with get_skills_dir() as the fallback, and align the new tests to patch the enforced symbol. Follow-up to the salvage of #59829 by @HexLab98. |
||
|---|---|---|
| .. | ||
| __init__.py | ||
| conftest.py | ||
| test_blueprint_catalog.py | ||
| test_claim_job_for_fire.py | ||
| test_codex_execution_paths.py | ||
| test_compute_next_run_last_run_at.py | ||
| test_cron_context_from.py | ||
| test_cron_inactivity_timeout.py | ||
| test_cron_no_agent.py | ||
| test_cron_profile_isolation.py | ||
| test_cron_prompt_injection_skill.py | ||
| test_cron_provider_pin.py | ||
| test_cron_script.py | ||
| test_cron_workdir.py | ||
| test_cronjob_schema.py | ||
| test_file_permissions.py | ||
| test_jobs.py | ||
| test_jobs_changed_notify.py | ||
| test_jobs_crossprocess_lock.py | ||
| test_parallel_pool.py | ||
| test_rewrite_skill_refs.py | ||
| test_run_one_job.py | ||
| test_scheduler.py | ||
| test_scheduler_mcp_init.py | ||
| test_scheduler_provider.py | ||
| test_scheduler_shutdown_guard.py | ||
| test_suggestions.py | ||
| test_terminal_cwd_lock.py | ||