Update Teams CLI install command#88103
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 29, 2026, 5:18 PM ET / 21:18 UTC. Summary PR surface: Docs -4. Total -4 across 1 file. Reproducibility: not applicable. as a runtime bug; source inspection shows current main and the latest release still contain the stale preview command, while the PR removes it. Review metrics: none identified. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Next step before merge
Security Review detailsBest possible solution: Land the focused docs update after normal maintainer review so the public Teams setup page follows the stable Teams CLI install path. Do we have a high-confidence way to reproduce the issue? Not applicable as a runtime bug; source inspection shows current main and the latest release still contain the stale preview command, while the PR removes it. Is this the best way to solve the issue? Yes. The stale wording is isolated to the Microsoft Teams docs page, and replacing the preview-tag install command with the bare package matches the current npm registry state without touching runtime, config, or plugin behavior. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 15de9d881a84. Label changesLabel justifications:
Evidence reviewedPR surface: Docs -4. Total -4 across 1 file. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
Review history (1 earlier review cycle)
|
|
The docs change itself looks right. The current npm package state supports using the stable install command: The blocker is the failing Real behavior proof gate. The PR has the required field-style proof under Please reshape the PR body so the required fields live under the real proof section. For this docs-only change, something like this should be enough if it includes actual terminal output or a terminal screenshot from the patched checkout: ## Real behavior proof
Behavior addressed: Microsoft Teams docs no longer tell users to install `@microsoft/teams.cli@preview` now that the stable package is available.
Real environment tested: local OpenClaw docs source checkout.
Exact steps or command run after this patch:
```bash
npm view @microsoft/teams.cli dist-tags version --json
rg -n "@microsoft/teams\\.cli@preview|Teams CLI is currently in preview|teams\\.cli@preview" docs/channels/msteams.md || true
git diff --check
```
Evidence after fix: terminal output or screenshot showing the npm `latest` tag, no remaining deprecated preview command in `docs/channels/msteams.md`, and a clean whitespace check.
Observed result after fix: docs now show `npm install -g @microsoft/teams.cli` and no longer show the preview warning.
What was not tested: docs site rendering was not built.The npm screenshot is useful supporting evidence, but the gate is asking for after-fix evidence from the patched OpenClaw setup as well. |
|
Updated and included the screenshot from my terminal. |
|
Re-reviewed after the update. The terminal screenshot helps, and the docs diff still looks correct. The remaining blocker is still structural in the PR body, not the patch. The latest Those fields are still under No code change is needed. Move or copy the required fields into the proof section, and put the screenshot/link in ## Real behavior proof
Behavior addressed: Microsoft Teams docs no longer mention `@microsoft/teams.cli@preview`.
Real environment tested: local docs source checkout.
Exact steps or command run after this patch: `rg -n "@microsoft/teams\\.cli@preview|Teams CLI is currently in preview|teams\\.cli@preview" || true`; `git diff --check`.
Evidence after fix: terminal screenshot showing no deprecated preview install command found and whitespace check passed; npm screenshot showing `latest` is stable `3.0.0`.
Observed result after fix: docs now show `npm install -g @microsoft/teams.cli`.
What was not tested: docs site rendering was not built; this is a one-line docs command update.Once that is in |
|
@BradGroux good now 👍🏽 |
|
This pull request has been automatically marked as stale due to inactivity. |
Summary
Updates the Microsoft Teams setup docs to install
@microsoft/teams.cliwithout the old@previewtag.Why
Teams CLI is out of preview now, so the docs should point folks at the current stable package. Less vintage npm incantation, more happy setup.
Interesting bits
Also removed the note warning that Teams CLI is currently in preview, since that is no longer true.
Real behavior proof
Behavior addressed: Microsoft Teams setup docs now install the GA
@microsoft/teams.clipackage instead of the old@previewtag.Real environment tested: Local OpenClaw docs source checkout, plus the public npm registry state for
@microsoft/teams.cli.Exact steps or command run after this patch:
Evidence after fix:
The stale-preview
rgcommand produced no matches, andgit diff --checkproduced no output.Supporting screenshots:
Observed result after fix: No stale
@microsoft/teams.cli@previewinstall command remains, and the docs now shownpm install -g @microsoft/teams.cli.What was not tested: Docs site rendering was not built; this is a docs-only command update.