release: v0.4.25 — synced to Vercel Chat 4.25.0

* fix: review findings + version scheme + integration tests

Must-fix:
- Teams Action.Submit now passes all submitted input values (was dropping
  ChoiceSet selections, making Select/RadioSelect unusable). For plain
  buttons with a single "value" key, the value is unwrapped for backward
  compat.
- Added end-to-end Teams card input tests: render Select -> submit
  Action.Submit -> verify process_action receives selected values.

Review fixes from previous round:
- P1: GitHub fetch_thread issue threads missing github: prefix
- P1: _content_to_plain_text could return non-string
- P2: asyncio.ensure_future -> create_task in Plan
- P2: Redundant isinstance(value, object) -> value is not None
- P2: GitHub issue metadata camelCase -> snake_case
- P2: Silent exception suppression -> logged in _enqueue_edit
- P3: Added test for Plan edit error path

Version scheme: 0.4.25 (embeds upstream version 4.25)

Process updates in UPSTREAM_SYNC.md:
- Upstream behavior is one input, not source of truth
- Every new adapter feature needs integration-style tests
- Every fetch_thread() needs round-trip test with channel APIs
- Intentional divergences must be recorded before publishing

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

* 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>

* docs: add PyPI, Python version, CI, and license badges to README

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

* fix: GitHub raw dict helper + PostableObject history cache

- Extract _build_raw_message() in GitHub adapter (was duplicated between
  post_message and edit_message)
- PostableObject posts now cached in message history for both Thread and
  Channel, so Plan objects appear in messages()/all_messages() like
  regular posts. Uses fallback text as the cached message content.

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

* fix: strip msteams transport keys from action values (P1) + revert PostableObject history (P2)

P1: _extract_action_values now strips both actionId and msteams keys.
The msteams key is injected by our own card renderer (task/fetch for
modal buttons) and is transport metadata, not user input. Without this,
modal button clicks would deliver {"value": "x", "msteams": {...}}
instead of just "x".

P2: Reverted PostableObject history caching — upstream TS does not cache
PostableObjects in message history either. The previous fix used a
synthetic "postable-obj" ID which would have caused ambiguous history
entries. Matches upstream behavior.

Added regression test for msteams key stripping.

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

* fix: cache PostableObjects in history with real message ID

PostableObject posts (e.g., Plan) now cached in thread/channel message
history using the real adapter-returned message ID and fallback text.
This is an intentional divergence from upstream TS, which skips history
for PostableObjects (a gap, not a design choice — regular posts and
streaming posts both cache, only PostableObjects were missed).

Changes:
- post_postable_object() now returns the RawMessage
- Thread/Channel _handle_postable_object returns RawMessage
- History append uses raw.id (not synthetic "postable-obj" constant)
- Documented as intentional divergence in UPSTREAM_SYNC.md non-parity

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

---------

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

Author: patrick-chinchill

CHANGELOG.md

@@ -1,8 +1,8 @@
 # Changelog
 
-## 0.25.0 (2026-04-10)
+## 0.4.25 (2026-04-10)
 
-Synced to [Vercel Chat 4.25.0](https://github.com/vercel/chat). New versioning: our minor version matches the upstream minor version.
+Synced to [Vercel Chat 4.25.0](https://github.com/vercel/chat). New versioning: `0.{upstream_major}.{upstream_minor}` embeds the upstream version directly.
 
 ### New features (from upstream 4.25.0)
 - **Plan blocks**: `Plan` PostableObject for structured task lists with live updates. Post a plan to a thread, then `add_task()`, `update_task()`, and `complete()` with automatic card rendering.
@@ -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)
 

CLAUDE.md

@@ -19,10 +19,11 @@ uv run pytest tests/ --tb=short -q
 ```
 
 ## Version Mapping
-Our version tracks the upstream Vercel Chat minor version:
-- `0.25.0` = synced to upstream `4.25.0`
-- `0.25.1` = Python-only fixes on top of `4.25.0`
-- `0.26.0` = synced to upstream `4.26.0`
+Our version embeds the upstream Vercel Chat version: `0.{upstream_major}.{upstream_minor}[.patch]`
+- `0.4.25` = synced to upstream `4.25.0`
+- `0.4.25.1` = Python-only fix on top of `4.25.0`
+- `0.4.26` = synced to upstream `4.26.0`
+- `0.4.26a1` = alpha while porting upstream `4.26.0`
 - `UPSTREAM_PARITY` constant in `__init__.py` = programmatic access
 
 ## Architecture

README.md

@@ -1,8 +1,13 @@
 # chat-sdk-python
 
+[![PyPI](https://img.shields.io/pypi/v/chat-sdk)](https://pypi.org/project/chat-sdk/)
+[![Python](https://img.shields.io/pypi/pyversions/chat-sdk)](https://pypi.org/project/chat-sdk/)
+[![Tests](https://github.com/Chinchill-AI/chat-sdk-python/actions/workflows/test.yml/badge.svg)](https://github.com/Chinchill-AI/chat-sdk-python/actions/workflows/test.yml)
+[![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
+
 Multi-platform async chat SDK for Python. Port of [Vercel Chat](https://github.com/vercel/chat).
 
-> **Status: Alpha (0.25.0 — synced to [Vercel Chat 4.25.0](https://github.com/vercel/chat))** — API may change.
+> **Status: Alpha (0.4.25 — synced to [Vercel Chat 4.25.0](https://github.com/vercel/chat))** — API may change.
 
 ## Why chat-sdk?
 

docs/UPSTREAM_SYNC.md

@@ -4,13 +4,14 @@ How to keep `chat-sdk-python` in sync with the [Vercel Chat TS SDK](https://gith
 
 ## Version Mapping
 
-Our version number tracks the upstream Vercel Chat minor version:
+Our version embeds the upstream Vercel Chat version: `0.{upstream_major}.{upstream_minor}[.patch]`
 
 | Python version | Upstream version | Meaning |
 |---|---|---|
-| `0.25.0` | `4.25.0` | Synced to upstream 4.25.0 |
-| `0.25.1` | `4.25.0` | Python-only fix on top of 4.25.0 |
-| `0.26.0` | `4.26.0` | Synced to upstream 4.26.0 |
+| `0.4.25` | `4.25.0` | Synced to upstream 4.25.0 |
+| `0.4.25.1` | `4.25.0` | Python-only fix on top of 4.25.0 |
+| `0.4.25a1` | `4.25.0` | Alpha while porting 4.25.0 |
+| `0.4.26` | `4.26.0` | Synced to upstream 4.26.0 |
 
 The `UPSTREAM_PARITY` constant in `chat_sdk/__init__.py` provides programmatic access
 to the upstream version this release is synced to.
@@ -43,7 +44,7 @@ uv run python scripts/verify_test_fidelity.py
 uv run pytest tests/ --tb=short -q
 
 # 6. Update version
-#    - pyproject.toml: version = "0.26.0"
+#    - pyproject.toml: version = "0.4.26"
 #    - README.md: status line
 #    - __init__.py: UPSTREAM_PARITY = "4.26.0"
 #    - CLAUDE.md: version reference
@@ -57,10 +58,19 @@ gh pr create --title "sync: upstream v4.26.0"
 
 - [ ] New types or fields → add to `types.py`
 - [ ] New methods on Thread/Channel → add to `thread.py`/`channel.py`
-- [ ] New adapter features → update the adapter + tests
+- [ ] New adapter features → update the adapter + integration-style tests
 - [ ] New TS tests → run fidelity script, port missing tests
 - [ ] Changed behavior → verify Python matches with regression tests
 - [ ] Review the porting hazards below for each change
+- [ ] Every new `fetch_thread()` behavior → round-trip test with channel APIs
+- [ ] Every new adapter feature → at least one end-to-end test (not just unit/conversion)
+
+### Upstream behavior is one input, not source of truth
+
+If upstream tests are missing coverage for a feature, add Python-only regression
+tests. If upstream tests lock in inconsistent behavior, choose one of:
+- **Preserve parity** and document the inconsistency in the non-parity section below
+- **Intentionally diverge** and document the divergence in the non-parity section
 
 ## How to Diff Upstream Changes
 
@@ -382,6 +392,8 @@ stay explicit instead of being rediscovered in code review.
 | `ast_to_plain_text` | Joins blocks with `\n` | Concatenates directly | More readable output |
 | `renderPostable` on unknown input | Returns `str(message)` | Throws `Error` | More resilient |
 | Chat resolver | 3-level: explicit → ContextVar → global | Process-global singleton | See [DECISIONS.md](DECISIONS.md#why-3-level-chat-resolver) |
+| PostableObject history | Cached in message history with real message ID | Not cached (skips history) | Upstream gap — posted messages should appear in thread/channel history |
+| Teams `msteams` transport key | Stripped from action values | Not stripped | Upstream gap — SDK-injected metadata should not leak to handlers |
 
 ### Platform-specific gaps
 

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "chat-sdk"
-version = "0.25.0"
+version = "0.4.25"
 description = "Multi-platform async chat SDK for Python — port of Vercel Chat"
 readme = "README.md"
 license = {text = "MIT"}

src/chat_sdk/adapters/github/adapter.py

@@ -476,24 +476,10 @@ async def post_message(self, thread_id: str, message: AdapterPostableMessage) ->
                 {"body": body},
             )
 
-        raw: dict[str, Any] = {
-            "type": "review_comment" if decoded.review_comment_id else "issue_comment",
-            "comment": comment,
-            "repository": {
-                "id": 0,
-                "name": decoded.repo,
-                "full_name": f"{decoded.owner}/{decoded.repo}",
-                "owner": {"id": 0, "login": decoded.owner, "type": "User"},
-            },
-            "pr_number": decoded.pr_number,
-        }
-        if not decoded.review_comment_id:
-            raw["thread_type"] = decoded.type or "pr"
-
         return RawMessage(
             id=str(comment["id"]),
             thread_id=thread_id,
-            raw=raw,
+            raw=self._build_raw_message(decoded, comment),
         )
 
     async def edit_message(self, thread_id: str, message_id: str, message: AdapterPostableMessage) -> RawMessage:
@@ -518,24 +504,10 @@ async def edit_message(self, thread_id: str, message_id: str, message: AdapterPo
                 {"body": body},
             )
 
-        raw: dict[str, Any] = {
-            "type": "review_comment" if decoded.review_comment_id else "issue_comment",
-            "comment": comment,
-            "repository": {
-                "id": 0,
-                "name": decoded.repo,
-                "full_name": f"{decoded.owner}/{decoded.repo}",
-                "owner": {"id": 0, "login": decoded.owner, "type": "User"},
-            },
-            "pr_number": decoded.pr_number,
-        }
-        if not decoded.review_comment_id:
-            raw["thread_type"] = decoded.type or "pr"
-
         return RawMessage(
             id=str(comment["id"]),
             thread_id=thread_id,
-            raw=raw,
+            raw=self._build_raw_message(decoded, comment),
         )
 
     async def stream(
@@ -553,6 +525,24 @@ async def stream(
                 text += chunk.text
         return await self.post_message(thread_id, {"markdown": text})
 
+    @staticmethod
+    def _build_raw_message(decoded: Any, comment: dict[str, Any]) -> dict[str, Any]:
+        """Build the raw message dict for post/edit responses."""
+        raw: dict[str, Any] = {
+            "type": "review_comment" if decoded.review_comment_id else "issue_comment",
+            "comment": comment,
+            "repository": {
+                "id": 0,
+                "name": decoded.repo,
+                "full_name": f"{decoded.owner}/{decoded.repo}",
+                "owner": {"id": 0, "login": decoded.owner, "type": "User"},
+            },
+            "pr_number": decoded.pr_number,
+        }
+        if not decoded.review_comment_id:
+            raw["thread_type"] = decoded.type or "pr"
+        return raw
+
     async def delete_message(self, thread_id: str, message_id: str) -> None:
         """Delete a message."""
         decoded = self.decode_thread_id(thread_id)
@@ -703,15 +693,15 @@ async def fetch_thread(self, thread_id: str) -> ThreadInfo:
 
             return ThreadInfo(
                 id=thread_id,
-                channel_id=f"{decoded.owner}/{decoded.repo}",
+                channel_id=f"github:{decoded.owner}/{decoded.repo}",
                 channel_name=f"{decoded.repo} #{decoded.pr_number}",
                 is_dm=False,
                 metadata={
                     "owner": decoded.owner,
                     "repo": decoded.repo,
-                    "issueNumber": decoded.pr_number,
-                    "issueTitle": issue.get("title"),
-                    "issueState": issue.get("state"),
+                    "issue_number": decoded.pr_number,
+                    "issue_title": issue.get("title"),
+                    "issue_state": issue.get("state"),
                     "type": "issue",
                 },
             )

src/chat_sdk/adapters/teams/adapter.py

@@ -350,13 +350,40 @@ async def _handle_message_activity(
 
         self._chat.process_message(self, thread_id, message, options)
 
+    # Keys injected by the SDK's card renderer or Teams transport — not user input.
+    _ACTION_TRANSPORT_KEYS = frozenset({"actionId", "msteams"})
+
+    @staticmethod
+    def _extract_action_values(action_data: dict[str, Any]) -> tuple[str, Any]:
+        """Extract action ID and submitted values from a Teams action payload.
+
+        Strips transport keys (``actionId``, ``msteams``) that are injected by
+        the SDK's card renderer or Teams infrastructure and are not user input.
+
+        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 not in TeamsAdapter._ACTION_TRANSPORT_KEYS}
+        # 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],
         action_value: dict[str, Any],
         options: WebhookOptions | None = None,
     ) -> None:
-        """Handle Action.Submit button clicks sent as message activities."""
+        """Handle Action.Submit button clicks sent as message activities.
+
+        For plain buttons, ``action_value`` looks like ``{"actionId": "btn_id", "value": "clicked"}``.
+        For ChoiceSet (Select/RadioSelect) submissions, it looks like
+        ``{"actionId": "__auto_submit", "my_select": "option_1"}``.
+        In both cases, we pass the full dict (minus ``actionId``) as ``value``
+        so handlers receive all submitted input values.
+        """
         if not self._chat:
             return
 
@@ -370,11 +397,13 @@ def _handle_message_action(
             )
         )
 
+        action_id, submitted_values = self._extract_action_values(action_value)
+
         from_user = activity.get("from", {})
         self._chat.process_action(
             ActionEvent(
-                action_id=action_value.get("actionId", ""),
-                value=action_value.get("value"),
+                action_id=action_id,
+                value=submitted_values,
                 user=Author(
                     user_id=from_user.get("id", "unknown"),
                     user_name=from_user.get("name", "unknown"),
@@ -411,11 +440,13 @@ async def _handle_adaptive_card_action(
             )
         )
 
+        action_id, submitted_values = self._extract_action_values(action_data)
+
         from_user = activity.get("from", {})
         self._chat.process_action(
             ActionEvent(
-                action_id=action_data.get("actionId", ""),
-                value=action_data.get("value"),
+                action_id=action_id,
+                value=submitted_values,
                 user=Author(
                     user_id=from_user.get("id", "unknown"),
                     user_name=from_user.get("name", "unknown"),

src/chat_sdk/channel.py

@@ -282,8 +282,13 @@ async def post(
         If the message is a PostableObject (e.g. Plan), it is posted via
         native adapter support or fallback text.
         """
+        # Handle PostableObject (e.g. Plan)
         if is_postable_object(message):
-            await self._handle_postable_object(message)
+            raw = await self._handle_postable_object(message)
+            if self._message_history is not None and raw is not None:
+                fallback = message.get_fallback_text() if hasattr(message, "get_fallback_text") else ""
+                sent = self._create_sent_message(raw.id, PostableMarkdown(markdown=fallback), raw.thread_id)
+                await self._message_history.append(self._id, _to_message(sent))
             return message
 
         if _is_async_iterable(message):
@@ -312,7 +317,7 @@ async def _post_single_message(
 
         return sent
 
-    async def _handle_postable_object(self, obj: Any) -> None:
+    async def _handle_postable_object(self, obj: Any) -> Any:
         """Post a PostableObject using native adapter support or fallback."""
         adapter = self.adapter
 
@@ -321,7 +326,7 @@ async def _post_fn(thread_id: str, message: str) -> Any:
                 return await adapter.post_channel_message(thread_id, message)
             return await adapter.post_message(thread_id, message)
 
-        await post_postable_object(obj, adapter, self._id, _post_fn)
+        return await post_postable_object(obj, adapter, self._id, _post_fn)
 
     async def post_ephemeral(
         self,

src/chat_sdk/plan.py

@@ -9,7 +9,6 @@
 from __future__ import annotations
 
 import asyncio
-import contextlib
 import uuid
 from dataclasses import dataclass, field
 from typing import Any, Literal
@@ -115,7 +114,8 @@ def _content_to_plain_text(content: PlanContent | None) -> str:
     if isinstance(content, str):
         return content
     if isinstance(content, dict) and "markdown" in content:
-        return content["markdown"]
+        md = content["markdown"]
+        return str(md) if md is not None else ""
     if isinstance(content, dict):
         # For ast dicts, return empty -- full rendering not needed for titles
         pass
@@ -125,7 +125,7 @@ def _content_to_plain_text(content: PlanContent | None) -> str:
 def is_postable_object(value: Any) -> bool:
     """Check if a value is a PostableObject (has the required protocol methods)."""
     return (
-        isinstance(value, object)
+        value is not None
         and hasattr(value, "kind")
         and hasattr(value, "get_fallback_text")
         and hasattr(value, "get_post_data")
@@ -145,9 +145,12 @@ async def post_postable_object(
     thread_id: str,
     post_fn: Any,
     logger: Logger | None = None,
-) -> None:
+) -> Any:
     """Post a PostableObject using the adapter's native support or fallback text.
 
+    Returns the ``RawMessage`` from the adapter so callers can use the
+    real message ID for history caching.
+
     Parameters
     ----------
     obj:
@@ -176,6 +179,7 @@ def _make_context(raw: Any) -> PostableObjectContext:
     else:
         raw = await post_fn(thread_id, obj.get_fallback_text())
         obj.on_posted(_make_context(raw))
+    return raw
 
 
 # =============================================================================
@@ -419,11 +423,14 @@ async def _do_edit() -> None:
 
         # Chain edits: wait for previous edit to finish before starting new one
         if bound.update_chain is not None:
-            with contextlib.suppress(Exception):
+            try:
                 await bound.update_chain
+            except Exception as prev_exc:
+                if bound.logger:
+                    bound.logger.warn("Previous plan edit failed", prev_exc)
 
         try:
-            bound.update_chain = asyncio.ensure_future(_do_edit())
+            bound.update_chain = asyncio.get_running_loop().create_task(_do_edit())
             await bound.update_chain
         except Exception as exc:
             if bound.logger: