Skip to content

feat: Warn users if Agent is called with only system messages#9514

Merged
sjrl merged 5 commits intomainfrom
agents-warning-no-messages
Jun 13, 2025
Merged

feat: Warn users if Agent is called with only system messages#9514
sjrl merged 5 commits intomainfrom
agents-warning-no-messages

Conversation

@sjrl
Copy link
Copy Markdown
Contributor

@sjrl sjrl commented Jun 13, 2025

Related Issues

Proposed Changes:

Logs a warning message to users if only system messages are passed to the Agent at runtime causing potentially unexpected behavior.

How did you test it?

Added unit tests

Notes for the reviewer

Checklist

  • I have read the contributors guidelines and the code of conduct
  • I have updated the related issue with new insights and changes
  • I added unit tests and updated the docstrings
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I documented my code
  • I ran pre-commit hooks and fixed any issue

@sjrl sjrl requested review from a team as code owners June 13, 2025 12:24
@sjrl sjrl requested review from dfokina and vblagoje and removed request for a team June 13, 2025 12:24
@github-actions github-actions Bot added topic:tests type:documentation Improvements on the docs labels Jun 13, 2025
@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 15634478084

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 13 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.03%) to 90.284%

Files with Coverage Reduction New Missed Lines %
components/agents/agent.py 13 90.7%
Totals Coverage Status
Change from base Build 15634468219: 0.03%
Covered Lines: 11560
Relevant Lines: 12804

💛 - Coveralls

@sjrl sjrl merged commit 379df4a into main Jun 13, 2025
20 checks passed
@sjrl sjrl deleted the agents-warning-no-messages branch June 13, 2025 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic:tests type:documentation Improvements on the docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Agent to warn the user if an empty message list is passed

3 participants