fix: address PR #41 review comments

- Fix versioning scheme description inconsistency in CHANGELOG.md
- Extract _extract_action_values() helper in Teams adapter (was
  duplicated between _handle_message_action and _handle_adaptive_card_action)
- GitHub raw dict duplication noted but not extracted (only 2 call sites
  with slight differences in the type/review_comment branching)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: patrick-chinchill

CHANGELOG.md

@@ -12,7 +12,7 @@ Synced to [Vercel Chat 4.25.0](https://github.com/vercel/chat). New versioning:
 - **Slack OAuth redirect fix**: `handle_oauth_callback` correctly forwards `redirect_uri` option.
 
 ### Versioning
-- Version scheme changed from `0.0.1aX` to `0.{upstream_minor}.{patch}`
+- Version scheme changed from `0.0.1aX` to `0.{upstream_major}.{upstream_minor}[.patch]`
 - `UPSTREAM_PARITY` constant in `chat_sdk.__init__` for programmatic access
 - Sync procedure documented in [UPSTREAM_SYNC.md](docs/UPSTREAM_SYNC.md)
 

src/chat_sdk/adapters/teams/adapter.py

@@ -350,6 +350,20 @@ async def _handle_message_activity(
 
         self._chat.process_message(self, thread_id, message, options)
 
+    @staticmethod
+    def _extract_action_values(action_data: dict[str, Any]) -> tuple[str, Any]:
+        """Extract action ID and submitted values from a Teams action payload.
+
+        For plain buttons: ``{"actionId": "btn", "value": "x"}`` → ``("btn", "x")``
+        For ChoiceSet: ``{"actionId": "__auto_submit", "sel": "opt"}`` → ``("__auto_submit", {"sel": "opt"})``
+        """
+        action_id = action_data.get("actionId", "")
+        submitted_values: Any = {k: v for k, v in action_data.items() if k != "actionId"}
+        # Unwrap single "value" key for plain button backward compat
+        if list(submitted_values.keys()) == ["value"]:
+            submitted_values = submitted_values["value"]
+        return action_id, submitted_values
+
     def _handle_message_action(
         self,
         activity: dict[str, Any],
@@ -377,13 +391,7 @@ def _handle_message_action(
             )
         )
 
-        # Extract action_id; pass remaining keys as value so ChoiceSet
-        # input values are not dropped.
-        action_id = action_value.get("actionId", "")
-        submitted_values = {k: v for k, v in action_value.items() if k != "actionId"}
-        # If there's a single "value" key, unwrap it for simple button clicks
-        if list(submitted_values.keys()) == ["value"]:
-            submitted_values = submitted_values["value"]
+        action_id, submitted_values = self._extract_action_values(action_value)
 
         from_user = activity.get("from", {})
         self._chat.process_action(
@@ -426,10 +434,7 @@ async def _handle_adaptive_card_action(
             )
         )
 
-        action_id = action_data.get("actionId", "")
-        submitted_values = {k: v for k, v in action_data.items() if k != "actionId"}
-        if list(submitted_values.keys()) == ["value"]:
-            submitted_values = submitted_values["value"]
+        action_id, submitted_values = self._extract_action_values(action_data)
 
         from_user = activity.get("from", {})
         self._chat.process_action(