chore(msteams): bump @microsoft/teams.apps&api from 2.0.5 to 2.0.6#54424
chore(msteams): bump @microsoft/teams.apps&api from 2.0.5 to 2.0.6#54424HDYA wants to merge 4 commits intoopenclaw:mainfrom
Conversation
v2.0.6 replaces the '/api*' wildcard Express route (incompatible with
Express 5 / path-to-regexp v8) with explicit route registration via
registerRoute('POST', messagingEndpoint). The messaging endpoint path
is now configurable (defaults to '/api/messages').
Release notes: https://github.com/microsoft/teams.ts/releases/tag/v2.0.6
Relevant PR: microsoft/teams.ts#483
All 338 msteams tests pass.
Greptile SummaryThis PR bumps
Confidence Score: 5/5
Prompt To Fix All With AIThis is a comment left during a code review.
Path: extensions/msteams/package.json
Line: 7
Comment:
**`@microsoft/teams.api` version not bumped to match**
`@microsoft/teams.api` is still pinned at `2.0.5` here while `@microsoft/[email protected]` now bundles `@microsoft/[email protected]` as a direct (non-peer) dependency. This causes pnpm to install both versions simultaneously — `2.0.5` for direct usage in this package, and `2.0.6` inside `teams.apps`.
For a patch-level bump the types are very likely compatible, and the passing tests confirm no runtime breakage. But bumping to `2.0.6` here too would eliminate the duplicate installation and keep all `@microsoft/teams.*` packages aligned:
```suggestion
"@microsoft/teams.api": "2.0.6",
```
How can I resolve this? If you propose a fix, please make it concise.Reviews (1): Last reviewed commit: "chore(msteams): bump @microsoft/teams.ap..." | Re-trigger Greptile |
Aligns @microsoft/teams.api with [email protected] to avoid dual version installation ([email protected] bundles [email protected] as a direct dependency).
|
Good catch — bumped |
|
Closing — these changes were superseded by main. The |
Summary
Bumps
@microsoft/teams.appsfrom 2.0.5 to 2.0.6.Key change
v2.0.6 removes the problematic
'/api*'wildcard Express route that was incompatible with Express 5 / path-to-regexp v8. Routes are now registered via explicitregisterRoute('POST', messagingEndpoint)instead ofexpress.use('/api*', ...).The messaging endpoint path is also now configurable (defaults to
/api/messages).Upstream references
Testing