Skip to content

Comments

Add docstrings to ai packages#146

Merged
heyitsaamir merged 2 commits intomainfrom
aamirj/addDocStrings
Sep 10, 2025
Merged

Add docstrings to ai packages#146
heyitsaamir merged 2 commits intomainfrom
aamirj/addDocStrings

Conversation

@heyitsaamir
Copy link
Collaborator

@heyitsaamir heyitsaamir commented Sep 10, 2025

PR Dependency Tree

This tree was auto-generated by Charcoal

Copilot AI review requested due to automatic review settings September 10, 2025 23:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive docstrings to AI packages, improving code documentation across the Teams AI framework. The changes focus on adding detailed docstrings to classes, methods, and protocols to enhance developer understanding and API usability.

  • Adds detailed class and method docstrings with parameter descriptions, return types, and usage notes
  • Documents AI model implementations, plugin systems, and core messaging protocols
  • Includes inline comments for field explanations and behavioral notes

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/openai/src/microsoft/teams/openai/responses_chat_model.py Added docstrings for OpenAI Responses API model class and generate_text method
packages/openai/src/microsoft/teams/openai/function_utils.py Added comprehensive docstrings for function schema utilities
packages/openai/src/microsoft/teams/openai/completions_model.py Added docstrings for Chat Completions model class and internal data structures
packages/openai/src/microsoft/teams/openai/common.py Added detailed docstrings for OpenAI base configuration class
packages/mcpplugin/src/microsoft/teams/mcpplugin/server_plugin.py Added docstrings for MCP server plugin and wrapper methods
packages/mcpplugin/src/microsoft/teams/mcpplugin/models/tool.py Enhanced tool model with Field descriptions
packages/mcpplugin/src/microsoft/teams/mcpplugin/models/params.py Converted to dataclass with inline field documentation
packages/mcpplugin/src/microsoft/teams/mcpplugin/models/cache.py Converted to dataclass with field documentation
packages/mcpplugin/src/microsoft/teams/mcpplugin/ai_plugin.py Added comprehensive docstrings for MCP client plugin
packages/ai/src/microsoft/teams/ai/plugin.py Added detailed protocol and base class documentation
packages/ai/src/microsoft/teams/ai/message.py Added docstrings for all message types with usage explanations
packages/ai/src/microsoft/teams/ai/memory.py Added memory protocol and implementation documentation
packages/ai/src/microsoft/teams/ai/function.py Added comprehensive function and type documentation
packages/ai/src/microsoft/teams/ai/chat_prompt.py Added detailed ChatPrompt class and method documentation
packages/ai/src/microsoft/teams/ai/ai_model.py Added AIModel protocol documentation with usage guidelines
packages/ai/src/microsoft/teams/ai/agent.py Enhanced Agent class documentation with persistent memory explanation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@heyitsaamir heyitsaamir merged commit edb2b38 into main Sep 10, 2025
6 checks passed
@heyitsaamir heyitsaamir deleted the aamirj/addDocStrings branch September 10, 2025 23:55
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.

1 participant