Feat/cron enabled toolsets by say8hi · Pull Request #6130 · NousResearch/hermes-agent

say8hi · 2026-04-08T11:29:06Z

What does this PR do?

Cron jobs currently load the full default toolset on every run - including
browser automation (11 tools), vision, TTS, MoA, skills management, and
other tools that most jobs never use. All of these inflate input token count
purely from tool definitions on every turn.

This PR adds an optional enabled_toolsets field to cron job config. When
set, only the specified toolsets are loaded for that job. When omitted,
behavior is unchanged.

Example: a job that only needs web search, terminal, file reads, and
delegation goes from loading ~25 tools to 6, reducing tool definition tokens
by ~60–70%.

{
  "name": "my-monitor",
  "enabled_toolsets": ["web", "terminal", "file", "delegation"]
}

The agent that creates cron jobs (cronjob tool) now also accepts and
persists enabled_toolsets, and infers the right value from the job prompt
based on the tool description guidance.

Related Issue

Fixes #

Type of Change

🐛 Bug fix (non-breaking change that fixes an issue)
✨ New feature (non-breaking change that adds functionality)
🔒 Security fix
📝 Documentation update
✅ Tests (adding or improving test coverage)
♻️ Refactor (no behavior change)
🎯 New skill (bundled or hub)

Changes Made

cron/scheduler.py: pass job.get("enabled_toolsets") to AIAgent when spawning the cron agent
cron/jobs.py: add enabled_toolsets parameter to create_job(), normalize and persist it in the job dict
tools/cronjob_tools.py: expose enabled_toolsets in cronjob() function, tool schema, _format_job(), and registry handler — supports both create and update actions

How to Test

Create a cron job with "enabled_toolsets": ["web", "terminal"] in ~/.hermes/cron/jobs.json
Trigger the job and observe input token count in logs - only web and terminal tool definitions should appear
Remove enabled_toolsets and retrigger — token count returns to default (all tools)
Ask the agent to create a cron job: it should automatically populate enabled_toolsets based on the tools referenced in the prompt

Checklist

Code

I've read the Contributing Guide
My commit messages follow Conventional Commits (fix(scope):, feat(scope):, etc.)
I searched for existing PRs to make sure this isn't a duplicate
My PR contains only changes related to this fix/feature (no unrelated commits)
I've run pytest tests/ -q and all tests pass
I've added tests for my changes (required for bug fixes, strongly encouraged for features)
I've tested on my platform: Linux 6.18.8-arch2-1

Documentation & Housekeeping

I've updated relevant documentation (README, docs/, docstrings) — or N/A
I've updated cli-config.yaml.example if I added/changed config keys — or N/A
I've updated CONTRIBUTING.md or AGENTS.md if I changed architecture or workflows — or N/A
I've considered cross-platform impact (Windows, macOS) per the compatibility guide — or N/A
I've updated tool descriptions/schemas if I changed tool behavior — or N/A

Screenshots / Logs

Before:

After:

Swift42 · 2026-04-10T13:58:59Z

That would not only save tokens, it would greatly increase security!
Currently I can just do
"prompt": "Copy the file /opt/data/foo.txt to /opt/data/bar.txt using the command: cp /opt/data/foo.txt /opt/data/bar.txt",
And the Agent then happily executes the command.
So a prompt injection attack could have severe consequences (e.g. when the Agent reads a mail or some other user-generated message).
Therefore I see this as a must-have security fix.

teknium1 · 2026-04-23T22:16:28Z

Merged via #14767 — your three commits were cherry-picked with authorship preserved (rebase merge). Thanks for the feature!

alt-glitch · 2026-04-23T22:28:51Z

Superseded by #14767 which salvaged this work and has been merged.

Cron now resolves its toolset from the same per-platform config the gateway uses — `_get_platform_tools(cfg, 'cron')` — instead of blindly loading every default toolset. Existing cron jobs without a per-job override automatically lose `moa`, `homeassistant`, and `rl` (the `_DEFAULT_OFF_TOOLSETS` set), which stops the "surprise $4.63 mixture_of_agents run" class of bug (Norbert, Discord). Precedence inside `run_job`: 1. per-job `enabled_toolsets` (PR #14767 / #6130) — wins if set 2. `_get_platform_tools(cfg, 'cron')` — new, the blanket gate 3. `None` fallback (legacy) — only on resolver exception Changes: - hermes_cli/platforms.py: register 'cron' with default_toolset 'hermes-cron' - toolsets.py: add 'hermes-cron' toolset (mirrors 'hermes-cli'; `_get_platform_tools` then filters via `_DEFAULT_OFF_TOOLSETS`) - cron/scheduler.py: add `_resolve_cron_enabled_toolsets(job, cfg)`, call it at the `AIAgent(...)` kwargs site - tests/cron/test_scheduler.py: replace the 'None when not set' test (outdated contract) with an invariant ('moa not in default cron toolset') + new per-job-wins precedence test - tests/hermes_cli/test_tools_config.py: mark 'cron' as non-messaging in the gateway-toolset-coverage test

…search#14798) Cron now resolves its toolset from the same per-platform config the gateway uses — `_get_platform_tools(cfg, 'cron')` — instead of blindly loading every default toolset. Existing cron jobs without a per-job override automatically lose `moa`, `homeassistant`, and `rl` (the `_DEFAULT_OFF_TOOLSETS` set), which stops the "surprise $4.63 mixture_of_agents run" class of bug (Norbert, Discord). Precedence inside `run_job`: 1. per-job `enabled_toolsets` (PR NousResearch#14767 / NousResearch#6130) — wins if set 2. `_get_platform_tools(cfg, 'cron')` — new, the blanket gate 3. `None` fallback (legacy) — only on resolver exception Changes: - hermes_cli/platforms.py: register 'cron' with default_toolset 'hermes-cron' - toolsets.py: add 'hermes-cron' toolset (mirrors 'hermes-cli'; `_get_platform_tools` then filters via `_DEFAULT_OFF_TOOLSETS`) - cron/scheduler.py: add `_resolve_cron_enabled_toolsets(job, cfg)`, call it at the `AIAgent(...)` kwargs site - tests/cron/test_scheduler.py: replace the 'None when not set' test (outdated contract) with an invariant ('moa not in default cron toolset') + new per-job-wins precedence test - tests/hermes_cli/test_tools_config.py: mark 'cron' as non-messaging in the gateway-toolset-coverage test

say8hi added 3 commits April 8, 2026 15:39

feat(cron): support enabled_toolsets per job to reduce token overhead

d316f36

feat(cron): expose enabled_toolsets in cronjob tool and create_job()

878c434

test(cron): add tests for enabled_toolsets in create_job and run_job

014bd6c

teknium1 mentioned this pull request Apr 23, 2026

feat(cron): per-job enabled_toolsets to cap token overhead + cost #14767

Merged

teknium1 added a commit that referenced this pull request Apr 23, 2026

chore(release): map say8hi author for #6130 salvage

470389e

teknium1 closed this in #14767 Apr 23, 2026

alt-glitch added type/feature New feature or request P2 Medium — degraded but workaround exists comp/cron Cron scheduler and job management comp/tools Tool registry, model_tools, toolsets labels Apr 23, 2026

alt-glitch added the duplicate This issue or pull request already exists label Apr 23, 2026

teknium1 mentioned this pull request Apr 23, 2026

feat(cron): honor hermes tools config for the cron platform #14798

Merged

nekorytaylor666 pushed a commit to nekorytaylor666/hermes-agent that referenced this pull request Apr 24, 2026

chore(release): map say8hi author for NousResearch#6130 salvage

6c747ab

ulasbilgen pushed a commit to ulasbilgen/hermes-adhd-agent that referenced this pull request May 1, 2026

chore(release): map say8hi author for NousResearch#6130 salvage

2959d41

aj-nt pushed a commit to aj-nt/hermes-agent that referenced this pull request May 1, 2026

chore(release): map say8hi author for NousResearch#6130 salvage

9a38953

donald131 pushed a commit to donald131/hermes-agent that referenced this pull request May 2, 2026

chore(release): map say8hi author for NousResearch#6130 salvage

62e5da3

02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026

chore(release): map say8hi author for NousResearch#6130 salvage

3a8ea10

gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026

chore(release): map say8hi author for NousResearch#6130 salvage

fb80421

Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026

chore(release): map say8hi author for NousResearch#6130 salvage

ea35439

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/cron enabled toolsets#6130

Feat/cron enabled toolsets#6130
say8hi wants to merge 3 commits into
NousResearch:mainfrom
say8hi:feat/cron-enabled-toolsets

say8hi commented Apr 8, 2026 •

edited

Loading

Uh oh!

Swift42 commented Apr 10, 2026

Uh oh!

teknium1 commented Apr 23, 2026

Uh oh!

alt-glitch commented Apr 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

say8hi commented Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What does this PR do?

Related Issue

Type of Change

Changes Made

How to Test

Checklist

Code

Documentation & Housekeeping

Screenshots / Logs

Uh oh!

Swift42 commented Apr 10, 2026

Uh oh!

teknium1 commented Apr 23, 2026

Uh oh!

alt-glitch commented Apr 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

say8hi commented Apr 8, 2026 •

edited

Loading