fix: keep system reminders out of conversation history

[kidding-404]

Summary

Mark injected system reminders as temporary content. The reminders remain available to the model for the current request but are excluded from persisted conversation history, preventing them from polluting later context and repeatedly consuming tokens.

The existing main-agent test is extended to cover both the current-request and persistence behavior.

Testing

• uv run ruff format .
• uv run ruff check .
• uv run pytest tests/unit/test_astr_main_agent.py

Fixes #9779

Summary by Sourcery

Prevent system reminders from polluting future conversation context and consuming tokens repeatedly.

Bug Fixes:

• Exclude injected system reminders from persisted conversation history while keeping them available during the current model request.

Tests:

• Extend main-agent coverage to verify system reminders are included in the current request but omitted from saved messages.

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Needs a human reviewer. If the temporary marker is wrong, system reminders will be omitted from saved conversation history, so reconstructed conversations could lack that context. Reverting restores persistence for future messages but cannot recover reminders that were already skipped, although the reminders are derived and the impact is bounded.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}