refactor: address review feedback - trim tests, support Sequence

Review feedback from @Unshure:

1. hooks param now accepts Sequence (list, tuple) instead of just list
   - Changed type hint to Sequence[HookProvider | HookCallback]
   - Added Sequence import from collections.abc

2. Moved tests to existing test_agent_hooks.py and trimmed down
   - Removed separate test_agent_hooks_callable.py file
   - Kept 5 essential tests: callable, mixed list, tuple, invalid, lifecycle
   - Removed duplicate HookProvider test (already covered by test_agent__init__hooks)
   - Removed redundant tests (None, empty list, async, lambda, multiple)

3. Removed duplicated test_hooks_param_accepts_hook_provider
   - This was already covered by existing test_agent__init__hooks

Author: strands-agent

src/strands/agent/agent.py

@@ -12,7 +12,7 @@
 import logging
 import threading
 import warnings
-from collections.abc import AsyncGenerator, AsyncIterator, Callable, Mapping
+from collections.abc import AsyncGenerator, AsyncIterator, Callable, Mapping, Sequence
 from typing import (
     TYPE_CHECKING,
     Any,
@@ -129,7 +129,7 @@ def __init__(
         description: str | None = None,
         state: AgentState | dict | None = None,
         plugins: list[Plugin] | None = None,
-        hooks: list[HookProvider | HookCallback] | None = None,
+        hooks: Sequence[HookProvider | HookCallback] | None = None,
         session_manager: SessionManager | None = None,
         structured_output_prompt: str | None = None,
         tool_executor: ToolExecutor | None = None,
@@ -183,7 +183,8 @@ def __init__(
                 Plugins are initialized with the agent instance after construction and can register hooks,
                 modify agent attributes, or perform other setup tasks.
                 Defaults to None.
-            hooks: Hooks to be added to the agent hook registry. Accepts HookProvider instances
+            hooks: Hooks to be added to the agent hook registry. Accepts any sequence
+                (list, tuple) of HookProvider instances
                 or plain callable hook callbacks (functions with typed event parameters).
                 Defaults to None.
             session_manager: Manager for handling agent sessions including conversation history and state.

tests/strands/agent/test_agent_hooks.py

@@ -1021,3 +1021,68 @@ def interrupt_tool(event: BeforeToolCallEvent):
     assert result.stop_reason == "end_turn"
     assert result.message["content"][0]["text"] == "Final response"
     assert agent._interrupt_state.activated is False
+
+
+def test_hooks_param_accepts_callable():
+    """Verify that a plain callable can be passed via hooks parameter."""
+    events_received = []
+
+    def my_callback(event: AgentInitializedEvent) -> None:
+        events_received.append(event)
+
+    agent = Agent(hooks=[my_callback], callback_handler=None)
+
+    assert len(events_received) == 1
+    assert isinstance(events_received[0], AgentInitializedEvent)
+    assert events_received[0].agent is agent
+
+
+def test_hooks_param_accepts_mixed_list():
+    """Verify that a mix of HookProviders and callables can be passed."""
+    callback_events = []
+
+    def my_callback(event: AgentInitializedEvent) -> None:
+        callback_events.append(event)
+
+    provider = MockHookProvider(event_types=[AgentInitializedEvent])
+
+    agent = Agent(hooks=[provider, my_callback], callback_handler=None)
+
+    assert len(callback_events) == 1
+    assert callback_events[0].agent is agent
+    length, _ = provider.get_events()
+    assert length == 1
+
+
+def test_hooks_param_accepts_tuple():
+    """Verify that a tuple of hooks can be passed (Sequence support)."""
+    events_received = []
+
+    def my_callback(event: AgentInitializedEvent) -> None:
+        events_received.append(event)
+
+    agent = Agent(hooks=(my_callback,), callback_handler=None)
+
+    assert len(events_received) == 1
+    assert events_received[0].agent is agent
+
+
+def test_hooks_param_invalid_hook_raises_error():
+    """Verify that passing an invalid hook raises ValueError."""
+    with pytest.raises(ValueError, match="Invalid hook"):
+        Agent(hooks=["not_a_hook"], callback_handler=None)  # type: ignore
+
+
+def test_hooks_param_callable_invoked_during_lifecycle():
+    """Verify callable hooks fire during agent lifecycle."""
+    before_events = []
+
+    def on_before(event: BeforeInvocationEvent) -> None:
+        before_events.append(event)
+
+    mock_model = MockedModelProvider([{"role": "assistant", "content": [{"text": "Hello!"}]}])
+    agent = Agent(model=mock_model, hooks=[on_before], callback_handler=None)
+    agent("test")
+
+    assert len(before_events) == 1
+    assert isinstance(before_events[0], BeforeInvocationEvent)