fix: Agent event dispatching

[monoxgas]

• Added AgentEnd/Reacted to dispatch.
• Added agent tests and fixed some small bugs.
• Added RegexRefAgent mixin.
• Removed some non-working type introspection behavior on hooks.

---

Generated Summary:

• Implemented RegexRefAgent mixin for dynamic text referencing using regex patterns in tool calls, improving tool call efficiency by reducing repetition.
• Added automatic instructions for the regex {find:<pattern>} syntax to guide users on its usage.
• Enhanced Agent class with the model_post_init call during the with_ method to ensure consistency in initialization.
• Refactored the hooks mechanism in Agent to streamline behavior and remove redundant code, potentially improving performance and clarity.
• Enhanced TaskAgent initialization to ensure required tools and hooks are appended only if they are not already present.
• Introduced comprehensive testing for the new implementations, targeting regex functionality and ensuring the correctness of the agent's behavior.
• Revised the agent's event handling to ensure proper reporting and to improve clarity on the stopping conditions during execution.
• Updated the max_steps handling logic to correct the finishing step calculation and improve overall agent flow control.

These changes collectively enhance the functionality and reliability of the agents within the SDK.

This summary was generated with ❤️ by rigging

[Copilot]

Pull Request Overview

This PR adds comprehensive test coverage for the Agent system and refactors the agent execution logic. Key changes include adding a new RegexRefAgent mixin, fixing event dispatching for hooks, improving step counting logic, and moving AgentStopReason to a more appropriate module.

• Added 629 lines of comprehensive tests for Agent initialization, execution, hooks, and reactions
• Refactored agent execution flow to properly dispatch Reacted events and handle stalling conditions
• Introduced RegexRefAgent mixin for dynamic regex-based references in tool arguments

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tests/test_agent.py	New comprehensive test suite covering Agent initialization, execution, hooks, reactions, and edge cases
pyproject.toml	Added PERF exclusion for test files to allow performance-related patterns in tests
dreadnode/agent/result.py	Added stop_reason field to AgentResult and moved AgentStopReason type definition here
dreadnode/agent/events.py	Moved AgentStopReason import and added quotes around forward reference
dreadnode/agent/agent.py	Major refactor: removed hook mapping, improved event dispatching, added model_post_init re-triggering, introduced RegexRefAgent, fixed step counting logic
docs/sdk/agent.mdx	Updated documentation for TaskAgent and added RegexRefAgent documentation

Comments suppressed due to low confidence (1)

dreadnode/agent/agent.py:706

• Inconsistent step counting: The log message uses step - 1 but the AgentResult at line 732 uses step directly. This creates a mismatch where the logged steps value differs from the result.steps value. Both should use the same value for consistency. Based on the loop logic starting from 0 and incrementing at the beginning, step already contains the correct final step count, so the - 1 here appears incorrect.

            f"steps={step - 1}, "

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.