Skip to content

Release 2.0.7 (Libraries): exclude quoted replies#510

Merged
corinagum merged 1 commit into
releases/v2from
cg/prep-release-v2.0.7
May 15, 2026
Merged

Release 2.0.7 (Libraries): exclude quoted replies#510
corinagum merged 1 commit into
releases/v2from
cg/prep-release-v2.0.7

Conversation

@corinagum

@corinagum corinagum commented May 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

Brings main into releases/v2 for the 2.0.7 Libraries release, with quoted-replies (PR #389) excluded. Scope is Legacy / Libraries only — core/ is not in scope for this release.

Single squashed commit.

What's in this release

What's NOT in this release

Quoted-replies feature (PR #389) is excluded:

  • MessageActivity.AddQuote(), PrependQuote() builder methods — absent
  • Context.Reply() quote-aware behavior — reverted to legacy replyToId + blockquote
  • Samples/Samples.Quoting — removed
  • QR-related tests — removed

Kept intentionally:

  • QuotedReplyEntity and QuotedReplyData types stay in Microsoft.Teams.Api.Entities (still annotated [Experimental("ExperimentalTeamsQuotedReplies")]) so inbound activities carrying quotedReply entities still parse
  • MessageActivity.AddTargetedMessageInfo runtime strip-QR logic — preserves prompt-preview cleanup
  • Entity.cs serializer cases for quotedReply (with existing #pragma warning disable ExperimentalTeamsQuotedReplies)

Version

version.json: 2.0.6 → 2.0.7

Scope

Libraries only. core/ (the releases/core track) is not affected by this PR — when this prep merges to releases/v2, the core/ tree may appear in the branch as part of the merge, but the Legacy publish pipeline only builds and ships from Libraries/, Samples/, and Tests/.

Test plan

  • dotnet build Microsoft.Teams.sln --configuration Release — 0 errors, 5 pre-existing warnings
  • dotnet test Microsoft.Teams.sln --configuration Release — all suites pass across net8.0 and net10.0 (Api 407+407, Apps 136+136, Common 63+63, AI 17+17, McpClient 9+9, Graph 3+3, AspNetCore 25+25 with 1 pre-existing skip)
  • dotnet pack — nupkgs produced; consumer install verified (fresh console project, dotnet add package Microsoft.Teams.Apps --prerelease, dotnet run confirms QuotedReplyEntity ships, AddQuote/PrependQuote absent, AddTargetedMessageInfo present, ReactionClient.AddAsync/DeleteAsync present)
  • Teams.NET-ESRP pipeline with packageSet=Legacy, publishType=Public (post-merge)

🤖 Generated with Claude Code

@corinagum @claude
Prep release: include main, exclude quoted replies (Libraries only)
a8eef59 
Squashed merge of origin/main into releases/v2, with quoted-replies (PR #389)
reverted. Keeps release branch's version.json (2.0.6); version.json will be
bumped to 2.0.7 on this branch before publishing. core/ is out of scope for
this release.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
Copilot AI review requested due to automatic review settings May 14, 2026 23:14
Copilot AI reviewed May 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@corinagum

corinagum commented May 15, 2026

Copy link
Copy Markdown
Collaborator Author

Merging via admin override — why we're not waiting for Build & Test & Lint

The required Build & Test & Lint GitHub Actions check doesn't run automatically on this PR because the workflow's trigger is filtered to pull_request: branches: ['main']. PRs against releases/v2 are silently skipped — by design, to avoid redundant builds.

When manually dispatched (run 25892409468), the workflow's Pack step failed for an infrastructure reason unrelated to this PR's content:

Error: Unable to process file command 'env' successfully.
Error: Invalid format '0.7'

dotnet pack itself succeeded — every nupkg under Libraries/ was produced (e.g., Microsoft.Teams.Apps.2.0.7-ga8eef5947c.nupkg). The failure is in the GitHub runner reading $GITHUB_ENV after the step completes: an nbgv-written cloud-build env entry got line-broken, and the runner can't parse it. The trigger is the prep branch's unusual stamping combo (stable 2.0.7 root version + per-project preview override on the external MCP plugins producing 2.0.7-preview-0001-...). Main doesn't hit this because its root version still has -preview.{height}.

What we have instead

Check Source Status
Teams.NET-PR (Build/Test/Pack — Legacy) ADO ✅ pass
Teams.NET-PR (Build/Test/Pack — Core) ADO ✅ pass
Local dotnet build + dotnet test on Microsoft.Teams.sln macOS ✅ pass (0 errors, 146+146 + 407+407 + 63+63 + 17+17 + 9+9 + 3+3 + 25+25 tests across net8.0/net10.0)
Local dotnet pack + consumer-install smoke test macOS ✅ pass (QuotedReplyEntity ships; no AddQuote/PrependQuote symbols; ReactionClient.AddAsync/DeleteAsync present)

Publish path

The actual release pipeline is Teams.NET-ESRP in Azure DevOps, not GitHub Actions. The env-file fragility doesn't apply there.

@corinagum
corinagum merged commit beaf890 into releases/v2 May 15, 2026
5 of 6 checks passed
@corinagum
corinagum deleted the cg/prep-release-v2.0.7 branch May 15, 2026 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments
+1 more reviewer
@rido-min rido-min rido-min approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@corinagum @rido-min