hermes - 💡(How to fix) Fix [Feature]: configurable goal-judge timeout and terminal foreground max timeout [1 pull requests]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

Fix Action

Fixed

RAW_BUFFERClick to expand / collapse

Problem or Use Case

The judge and the foreground cap uses a fixed default timeout in code rather than a first-class config.yaml setting.

Users who run slow local/auxiliary models or who want a higher or lower foreground cap to run longer tasks without making them into background tasks should be able to tune these values from ~/.hermes/config.yaml without digging into code or relying on .env for the foreground cap.

Proposed Solution

Add two documented keys under the existing schema:

goals.judge_timeout_seconds — upper bound (seconds) for the OpenAI-compatible client timeout on each goal-judge call (default unchanged from current behavior, e.g. 30s unless maintainers choose otherwise).

terminal.max_foreground_timeout_seconds — maximum allowed explicit foreground timeout= passed to the terminal tool before requiring background=true (default unchanged, e.g. 600s).

Both should appear in defaults / cli-config.yaml.example, user docs (configuration reference), and hermes config show so values are discoverable. Foreground max should be driven by config.yaml only (no parallel .env override for this knob), so configuration stays in one place.

Alternatives Considered

No response

Feature Type

Configuration option

Scope

Small (single file, < 50 lines)

Contribution

  • I'd like to implement this myself and submit a PR

Debug Report (optional)

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

hermes - 💡(How to fix) Fix [Feature]: configurable goal-judge timeout and terminal foreground max timeout [1 pull requests]