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().

[giulio-leone]

Rebased this branch onto the latest main and force-pushed refreshed head 19173e7.

Local validation I ran from the rebased branch:

• uv run hatch run test-format
• uv run hatch run test-lint
• uv run hatch run test -- tests/strands/multiagent/test_graph.py -q -> 50 passed
• repeated the same full suite a second time with no code changes between runs -> 50 passed
• after the rebase surfaced one formatter-only drift in the new regression test, I committed the repo-native formatting fix and reran the same suite from clean head 19173e7; it passed again from a clean worktree

Real branch-vs-main proof on exact source trees for the nested-graph reset corruption path:

• refreshed branch (19173e7): before reset GraphState, after reset GraphState, preserves_graph_state == True, node status reset to pending, result cleared
• current origin/main (fd8168a): before reset GraphState, after reset JSONSerializableDict, preserves_graph_state == False, node status reset to pending, result cleared

So the refreshed branch still fixes the real issue from #1775: GraphNode.reset_executor_state() no longer corrupts nested MultiAgentBase / Graph executor state during reset, while current main still does.