Open
Conversation
heyitsaamir
previously approved these changes
Apr 8, 2026
lilyydu
changed the title
added 2 commits
April 13, 2026 11:16
Contributor
There was a problem hiding this comment.
Pull request overview
Removes Python DevTools and Python AI documentation content from the Teams SDK docs site (teams.md) by restricting AI templates to TS/C# and deleting Python AI include fragments, plus updating the Python getting-started examples accordingly.
Changes:
- Restrict AI template generation to
typescript/csharp(and some pages totypescriptonly) and updatestatic/missing-pages.jsonto hide Python versions. - Delete Python AI include fragments (A2A/MCP/chat/function-calling/etc.).
- Update Python quickstart/code-basics includes to remove DevTools references and adjust the sample output (port 3978).
Reviewed changes
Copilot reviewed 27 out of 27 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| teams.md/static/missing-pages.json | Marks AI pages as unavailable for Python (and some pages for C#) so language switching doesn’t offer missing pages. |
| teams.md/src/pages/templates/in-depth-guides/ai/setup-and-prereqs.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/in-depth-guides/ai/mcp/mcp-server.mdx | Removes Python from languages list (TS-only). |
| teams.md/src/pages/templates/in-depth-guides/ai/mcp/mcp-client.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/in-depth-guides/ai/mcp/README.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/in-depth-guides/ai/keeping-state.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/in-depth-guides/ai/function-calling.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/in-depth-guides/ai/chat.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/in-depth-guides/ai/best-practices.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/in-depth-guides/ai/a2a/a2a-server.mdx | Removes Python from languages list (TS-only). |
| teams.md/src/pages/templates/in-depth-guides/ai/a2a/a2a-client.mdx | Removes Python from languages list (TS-only). |
| teams.md/src/pages/templates/in-depth-guides/ai/a2a/README.mdx | Removes Python from languages list (TS-only). |
| teams.md/src/pages/templates/in-depth-guides/ai/README.mdx | Adds languages frontmatter to exclude Python generation. |
| teams.md/src/pages/templates/getting-started/code-basics.mdx | Removes “Implement AI” from next-steps list. |
| teams.md/src/components/include/in-depth-guides/ai/setup-and-prereqs/python.incl.md | Removes Python-only AI setup/prereqs include. |
| teams.md/src/components/include/in-depth-guides/ai/python.incl.md | Removes Python-only AI package-name include. |
| teams.md/src/components/include/in-depth-guides/ai/mcp/mcp-server/python.incl.md | Removes Python MCP server include content. |
| teams.md/src/components/include/in-depth-guides/ai/mcp/mcp-client/python.incl.md | Removes Python MCP client include content. |
| teams.md/src/components/include/in-depth-guides/ai/keeping-state/python.incl.md | Removes Python keeping-state include content. |
| teams.md/src/components/include/in-depth-guides/ai/function-calling/python.incl.md | Removes Python function-calling include content. |
| teams.md/src/components/include/in-depth-guides/ai/chat/python.incl.md | Removes Python chat include content. |
| teams.md/src/components/include/in-depth-guides/ai/best-practices/python.incl.md | Removes Python best-practices include content. |
| teams.md/src/components/include/in-depth-guides/ai/a2a/python.incl.md | Removes Python A2A overview include content. |
| teams.md/src/components/include/in-depth-guides/ai/a2a/a2a-server/python.incl.md | Removes Python A2A server include content. |
| teams.md/src/components/include/in-depth-guides/ai/a2a/a2a-client/python.incl.md | Removes Python A2A client include content. |
| teams.md/src/components/include/getting-started/quickstart/python.incl.md | Removes DevTools log lines and updates the shown port to 3978. |
| teams.md/src/components/include/getting-started/code-basics/python.incl.md | Removes DevToolsPlugin usage from Python snippet. |
| @@ -21,9 +21,8 @@ quote-agent/ | |||
| ```python title="src/main.py" | |||
| from microsoft_teams.api import MessageActivity, TypingActivityInput | |||
| from microsoft_teams.apps import ActivityContext, App, AppOptions | |||
There was a problem hiding this comment.
[HIGH] This snippet still imports AppOptions but no longer uses it after removing DevToolsPlugin. Drop the unused import (or show an AppOptions usage). Also ensure the surrounding getting-started/code-basics.mdx template text (which references DevTools as already running) is updated/conditioned for Python now that DevToolsPlugin is gone.
Suggested change
| from microsoft_teams.apps import ActivityContext, App, AppOptions | |
| from microsoft_teams.apps import ActivityContext, App |
Comment on lines
70
to
74
| After that, you can: | ||
|
|
||
| - Add more activity handlers for different types of interactions. See [Listening to Activities](../essentials/on-activity) for more details. | ||
| - Integrate with external services using the [API Client](../essentials/api). | ||
| - Add interactive [cards](../in-depth-guides/adaptive-cards) and [dialogs](../in-depth-guides/dialogs). |
There was a problem hiding this comment.
[HIGH] Removing the AI link here affects all languages (TS/C#/Python). But this PR only removes Python AI docs and the AI templates are still available for TS/C#. If AI should remain discoverable for TS/C#, consider making this bullet language-conditional (hide only for Python) instead of removing it globally.
| INFO: Uvicorn running on http://0.0.0.0:3979 (Press CTRL+C to quit) | ||
| INFO: Uvicorn running on http://0.0.0.0:3978 (Press CTRL+C to quit) | ||
| ``` | ||
|
|
There was a problem hiding this comment.
[HIGH] Python quickstart output no longer shows DevTools (3979), but getting-started/quickstart.mdx still states a devtools server starts on 3979 and instructs users to open /devtools. Make the devtools section language-specific/conditional (or restore devtools for Python) so Python instructions match the actual runtime behavior/ports.
Suggested change
| The Python quickstart starts your app's HTTP server on port `3978` only. It does not start a DevTools server on port `3979`, and there is no `/devtools` page for this runtime. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.