Merge pull request #87 from 0xbyt4/fix/cli-max-turns-sentinel
fix(cli): respect explicit --max-turns value even when it equals default
This commit is contained in:
commit
ae8d25faca
1 changed files with 4 additions and 4 deletions
8
cli.py
8
cli.py
|
|
@ -765,14 +765,14 @@ class HermesCLI:
|
||||||
provider: str = None,
|
provider: str = None,
|
||||||
api_key: str = None,
|
api_key: str = None,
|
||||||
base_url: str = None,
|
base_url: str = None,
|
||||||
max_turns: int = 60,
|
max_turns: int = None,
|
||||||
verbose: bool = False,
|
verbose: bool = False,
|
||||||
compact: bool = False,
|
compact: bool = False,
|
||||||
resume: str = None,
|
resume: str = None,
|
||||||
):
|
):
|
||||||
"""
|
"""
|
||||||
Initialize the Hermes CLI.
|
Initialize the Hermes CLI.
|
||||||
|
|
||||||
Args:
|
Args:
|
||||||
model: Model to use (default: from env or claude-sonnet)
|
model: Model to use (default: from env or claude-sonnet)
|
||||||
toolsets: List of toolsets to enable (default: all)
|
toolsets: List of toolsets to enable (default: all)
|
||||||
|
|
@ -815,7 +815,7 @@ class HermesCLI:
|
||||||
self._nous_key_expires_at: Optional[str] = None
|
self._nous_key_expires_at: Optional[str] = None
|
||||||
self._nous_key_source: Optional[str] = None
|
self._nous_key_source: Optional[str] = None
|
||||||
# Max turns priority: CLI arg > env var > config file (agent.max_turns or root max_turns) > default
|
# Max turns priority: CLI arg > env var > config file (agent.max_turns or root max_turns) > default
|
||||||
if max_turns != 60: # CLI arg was explicitly set
|
if max_turns is not None:
|
||||||
self.max_turns = max_turns
|
self.max_turns = max_turns
|
||||||
elif os.getenv("HERMES_MAX_ITERATIONS"):
|
elif os.getenv("HERMES_MAX_ITERATIONS"):
|
||||||
self.max_turns = int(os.getenv("HERMES_MAX_ITERATIONS"))
|
self.max_turns = int(os.getenv("HERMES_MAX_ITERATIONS"))
|
||||||
|
|
@ -2669,7 +2669,7 @@ def main(
|
||||||
provider: str = None,
|
provider: str = None,
|
||||||
api_key: str = None,
|
api_key: str = None,
|
||||||
base_url: str = None,
|
base_url: str = None,
|
||||||
max_turns: int = 60,
|
max_turns: int = None,
|
||||||
verbose: bool = False,
|
verbose: bool = False,
|
||||||
compact: bool = False,
|
compact: bool = False,
|
||||||
list_tools: bool = False,
|
list_tools: bool = False,
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue