Skip to content

feat: infer goal turn budgets up to 250 - #32536

Open
unreliable-machine wants to merge 1 commit into
NousResearch:mainfrom
unreliable-machine:goal-turn-budget-250
Open

feat: infer goal turn budgets up to 250#32536
unreliable-machine wants to merge 1 commit into
NousResearch:mainfrom
unreliable-machine:goal-turn-budget-250

Conversation

@unreliable-machine

Copy link
Copy Markdown

Summary\n- Treat goals.max_turns as a ceiling capped at 250 instead of a fixed budget for every /goal.\n- Infer per-goal budgets from prompt complexity: simple goals keep 20 turns; complex plan-style or explicit persistence prompts scale up automatically.\n- Update CLI/gateway config handling, docs, and focused goal tests.\n\n## Tests\n- python -m pytest -o 'addopts=' tests/hermes_cli/test_goals.py tests/gateway/test_goal_max_turns_config.py tests/tui_gateway/test_goal_command.py -q\n- python -m py_compile hermes_cli/goals.py cli.py gateway/run.py hermes_cli/config.py\n- Manual heuristic assertions for simple, complex, and explicit-turn prompts\n

@alt-glitch alt-glitch added type/feature New feature or request comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have labels May 26, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the contribution. The premise is still live: current main assigns each goal the configured fixed budget in hermes_cli/goals.py:1143-1152.

Problems

  • The PR updates goal-specific documentation but leaves the canonical configuration reference at website/docs/user-guide/configuration.md:842-851 describing goals.max_turns as a fixed 20-turn safety cap.
  • Current defaults explicitly define that cap as protection against judge false negatives and unbounded spend (hermes_cli/config.py:2271-2284). Automatic escalation to 250 changes that contract, so the intended default/opt-in policy needs maintainer confirmation.

Suggested changes

  • If approved, update the configuration reference and add CLI, gateway, and TUI coverage for simple versus escalated goals under a configured 250-turn cap, including a lower configured cap.

Automated hermes-sweeper review.

@teknium1 teknium1 added sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants