fix(graph): prevent reset_executor_state from corrupting MultiAgentBase state

[giulio-leone]

Summary

Fixes #1775 — GraphNode.reset_executor_state() corrupts MultiAgentBase executor state by overwriting GraphState with AgentState.

Root Cause

reset_executor_state() checks hasattr(self.executor, 'state') without verifying the state type:

# Before (broken)
if hasattr(self.executor, 'state'):
    self.executor.state = AgentState(self._initial_state.get())  # Overwrites GraphState!

__post_init__ already had the correct guard (hasattr(self.executor.state, 'get')), but reset_executor_state() was missing it.

Impact

Two call sites affected:

Call site	Trigger
_execute_node	reset_on_revisit=True with nested graph in a cycle
deserialize_state	Always — iterates all nodes when restoring a completed run

Fix

Mirror the __post_init__ guard:

# After (fixed)
if hasattr(self.executor, 'state') and hasattr(self.executor.state, 'get'):
    self.executor.state = AgentState(self._initial_state.get())

Test

Added test_reset_executor_state_preserves_graph_state_for_nested_graph — creates a GraphNode with a nested Graph executor and verifies reset_executor_state() preserves the GraphState type.

All 49 graph tests pass.

[giulio-leone]

Friendly ping — prevents reset_executor_state from corrupting MultiAgentBase state by preserving agent-level state that should persist across executor resets.

[giulio-leone]

Rebased this branch onto the latest main and force-pushed it. The only manual rebase resolution was in tests/strands/multiagent/test_graph.py, where I kept both the new upstream test and this PR's regression test.

Local validation I ran:

• .venv/bin/python -m pytest tests/strands/multiagent/test_graph.py::test_reset_executor_state_preserves_graph_state_for_nested_graph -q -> 1 passed

Real branch-vs-main proof (using the repo venv and explicit PYTHONPATH=<checkout>/src so the imported module path is unambiguous):

• main imported sdk-python-main/src/strands/multiagent/graph.py
  • before reset: GraphState
  • after reset: JSONSerializableDict
  • node status reset to PENDING, result cleared
• this branch imported sdk-python/src/strands/multiagent/graph.py
  • before reset: GraphState
  • after reset: GraphState
  • node status reset to PENDING, result cleared

So the rebased branch still fixes the real corruption path from #1775: nested graph executor state no longer gets overwritten to the wrong state type during reset_executor_state().