Skip to content

fix: add per-turn current time context#18135

Open
WXBR wants to merge 1 commit into
NousResearch:mainfrom
WXBR:fix/time-sanity-context
Open

fix: add per-turn current time context#18135
WXBR wants to merge 1 commit into
NousResearch:mainfrom
WXBR:fix/time-sanity-context

Conversation

@WXBR

@WXBR WXBR commented May 1, 2026

Copy link
Copy Markdown

Summary

  • add centralized helpers for formatting and prepending current-time context
  • inject current time into the API-facing user turn instead of mutating the system prompt
  • keep persisted transcripts, resumed history, and trajectories free of synthetic time notes
  • support idempotent injection for both string and multimodal user content

Test Plan

  • python -m pytest tests/run_agent/test_run_agent.py tests/test_timezone.py -q

@alt-glitch alt-glitch added type/bug Something isn't working comp/agent Core agent loop, run_agent.py, prompt builder P3 Low — cosmetic, nice to have labels May 1, 2026
@alt-glitch

alt-glitch commented May 1, 2026

Copy link
Copy Markdown
Collaborator

Related to #10448 and #5487 — same per-turn time injection goal. This PR adds centralized helpers (hermes_time.py) and multimodal support.

@alt-glitch

alt-glitch commented May 1, 2026

Copy link
Copy Markdown
Collaborator

Related to #10448 and #5487

@quinnmacro quinnmacro mentioned this pull request May 2, 2026
Open
This was referenced May 18, 2026
Open
Open
@alt-glitch alt-glitch mentioned this pull request May 27, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

comp/agent Core agent loop, run_agent.py, prompt builder P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@WXBR @alt-glitch