Skip to content

fix: preserve configured prompt in oneshot mode - #74646

Closed
KaDarius wants to merge 2 commits into
NousResearch:mainfrom
KaDarius:codex/oneshot-prompt-contract
Closed

fix: preserve configured prompt in oneshot mode#74646
KaDarius wants to merge 2 commits into
NousResearch:mainfrom
KaDarius:codex/oneshot-prompt-contract

Conversation

@KaDarius

Copy link
Copy Markdown

Summary

  • Forward the configured agent system prompt through the one-shot (hermes -z) entrypoint.
  • Add a focused regression test for configured prompt propagation.

Why

One-shot mode constructed AIAgent without the configured agent.system_prompt, so persistent boot contracts did not apply to one-shot invocations.

Verification

  • pytest -q tests/hermes_cli/test_tui_resume_flow.py -k "oneshot_run_agent_forwards_configured_system_prompt or defaults_ephemeral_prompt"
  • Result: 2 passed, 70 deselected.

@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard P2 Medium — degraded but workaround exists duplicate This issue or pull request already exists labels Jul 30, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Duplicate of #34863: both forward the configured system prompt into the hermes -z AIAgent construction. #69957 is a related later repair at the same seam.

@x7peeps

x7peeps commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

✅ Quality optimization: Added additional regression tests for oneshot prompt preservation. Tests pass locally. PR upgraded from A-grade (7/10) to S-grade (8/10).

@KaDarius

Copy link
Copy Markdown
Author

Closing in favor of downstream maintenance in KaDarius/hermes-agent. The focused one-shot prompt-parity repair remains maintained on the KaDarius fork.

@KaDarius KaDarius closed this Jul 30, 2026
@teknium1 teknium1 added 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 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard duplicate This issue or pull request already exists P2 Medium — degraded but workaround exists 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 type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants