fix(cp2): extract_action handles punctuation-wrapped field tokens (e.g. 'up?')

proteus/agents/parsing.py

@@ -33,9 +33,10 @@ def extract_action(text: str, valid: list[str]) -> str | None:
     # Strategy 1: last ACTION: <token> field.
     field_matches = _ACTION_FIELD.findall(text)
     for raw in reversed(field_matches):
-        token = raw.strip().strip(".!,;:'\"`").lower()
-        # take the first word of the field value
-        first = token.split()[0] if token.split() else ""
+        # First alphabetic run of the field value, ignoring any wrapping
+        # punctuation (e.g. "up?", "up." -> "up").
+        match = _WORD.search(raw)
+        first = match.group(0).lower() if match else ""
         if first in valid_set:
             return first
 

tests/agents/test_parsing.py

@@ -18,3 +18,9 @@ def test_bare_word_fallback_when_no_action_field():
 
 def test_returns_none_when_no_valid_action():
     assert extract_action("no idea", VALID) is None
+
+
+def test_action_field_with_trailing_question_mark():
+    # LLMs often append "?"; the field token must still be extracted, not
+    # fall through to a later prose word.
+    assert extract_action("ACTION: up?\nThen move right", VALID) == "up"