Skip to content

Mark reactions API as GA and fix sample remove flow#427

Merged
corinagum merged 2 commits into
mainfrom
cg/reactions-ga
May 14, 2026
Merged

Mark reactions API as GA and fix sample remove flow#427
corinagum merged 2 commits into
mainfrom
cg/reactions-ga

Conversation

@corinagum
Copy link
Copy Markdown
Contributor

Summary

Removes the @experimental("ExperimentalTeamsReactions") decorator and the .. warning:: Preview docstring from ReactionClient. The reactions feature is in sync across all three SDKs and the Teams service — no longer preview.

Also fixes the sample's unreact command, which previously called ctx.api.reactions.delete() on an incoming message that had no reaction. It now adds the reaction first, waits 2 seconds, then deletes — so the demo visibly shows the full add/delete cycle.

Cross-SDK coordination

Part of the cross-SDK Reactions-GA pass. Sibling PRs:

  • microsoft/teams.ts — Mark reactions API as GA
  • microsoft/teams.net — Mark reactions API as GA (Libraries + core)
  • microsoft/teams-sdk — drop .NET opt-in tip from the reactions guide

Test plan

  • uv run ruff check — all checks passed
  • uv run ruff format --check — 403 files already formatted
  • uv run pyright — 0 errors, 0 warnings
  • uv run pytest — 596 passed
  • Smoke run of examples/reactions showing the cycle in Teams

Versioning

No version.json change in this PR. Bump happens as part of the release flow per RELEASE.md.

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings May 14, 2026 19:52
corinagum added a commit to microsoft/teams-sdk that referenced this pull request May 14, 2026
## Summary

Removes the `:::tip[.NET]` callout block from the reactions in-depth
guide's C# include. The block instructed readers to suppress
`ExperimentalTeamsReactions` via `#pragma` or `<NoWarn>` — that's no
longer applicable now that the `[Experimental]` attribute is being
removed from teams.net (sibling PR).

## Cross-SDK coordination

Part of the cross-SDK Reactions-GA pass. Sibling PRs:
- microsoft/teams.ts#575 — Mark reactions API as GA
- microsoft/teams.py#427 — Mark reactions API as GA
- microsoft/teams.net#509 — Mark reactions API as GA (Libraries + core)

## Test plan

- [x] `npm start` regenerates docs cleanly with no content gaps
- [x] C# in-depth guide renders without the opt-in callout (verified in
local dev server)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@corinagum corinagum merged commit 580935c into main May 14, 2026
11 checks passed
@corinagum corinagum deleted the cg/reactions-ga branch May 14, 2026 20:51
corinagum added a commit that referenced this pull request May 14, 2026
## Summary

Brings `main` into `release` for the 2.0.11 release, with quoted-replies
(PR #321) excluded. Also bumps `version.json` from `2.0.0` to `2.0.11`
to align with the teams.ts release cadence.

Single squashed commit.

> **Branch name note**: the branch is `cg/prep-release-2.0.1` (left over
from an earlier draft when alignment hadn't been decided yet). The
actual published version per `version.json` is **2.0.11**.

## What's in this release

- All commits merged to main since 2.0.0 (the previous release),
notably:
  - **Reactions GA** (PR #427) — `@experimental` removed; sample fixed
  - Prompt Preview Support (PR #397)
  - Various dependency bumps and bug fixes

## What's NOT in this release

**Quoted-replies feature (PR #321) is excluded:**
- `prepend_quote()`, `add_quote()` builder methods — absent
- `ctx.quote()` and quote-aware `ctx.reply()` behavior — reverted to
legacy `reply_to_id` + blockquote
- `examples/quoting` — removed
- QR-related tests — removed

**Kept intentionally:**
- `QuotedReplyEntity` and `QuotedReplyData` types stay in the `Entity`
union so inbound activities carrying `quotedReply` entities still parse
- `add_targeted_message_info` runtime strip-QR logic — preserves
prompt-preview cleanup

## Version

`version.json`: `2.0.0` → `2.0.11`

This one-time jump aligns teams.py with teams.ts. Going forward, both
progress on a shared 2.0.x patch cadence.

## Test plan

- [x] `uv run ruff check` — all checks passed
- [x] `uv run pyright` — 0 errors, 0 warnings
- [x] `uv run pytest` — 571/571 tests
- [x] `uv build --all-packages` — wheels produced; consumer install
verified (fresh venv, `pip install --no-index --find-links` succeeds,
`QuotedReplyEntity` importable, `prepend_quote`/`add_quote` absent,
`add_targeted_message_info` present)
- [ ] ADO Public publish pipeline (post-merge)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants