Update Readme's - #44
Merged
Mrunal Suyoga Hirve (mrunalhirve128) merged 20 commits intoNov 13, 2025
Merged
Mrunal Suyoga Hirve (mrunalhirve128) merged 20 commits into
Mrunal Suyoga Hirve (mrunalhirve128) merged 20 commits into
Conversation
Mrunal Suyoga Hirve (mrunalhirve128)
requested a review
from a team
as a code owner
November 9, 2025 07:35
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request significantly restructures and streamlines the README documentation across multiple packages in the Microsoft Agents A365 SDK. The changes transform verbose, detailed package documentation into concise, user-friendly quickstart guides with consistent formatting and structure.
- Standardized documentation format with npm badges, installation instructions, and basic usage examples across all packages
- Consolidated detailed documentation into cleaner, more accessible formats focused on getting started quickly
- Updated the main repository README to provide clearer project structure, build instructions, and comprehensive navigation links
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/agents-a365-tooling/README.md | Simplified from detailed configuration guide to concise quickstart with tool server discovery examples |
| packages/agents-a365-tooling-extensions-openai/README.md | Reduced from comprehensive guide to streamlined basic usage examples for OpenAI integration |
| packages/agents-a365-tooling-extensions-langchain/README.md | Condensed to essential setup and usage examples for LangChain integration with syntax error in code example |
| packages/agents-a365-tooling-extensions-claude/README.md | Streamlined to core usage patterns for Claude SDK integration |
| packages/agents-a365-runtime/README.md | Enhanced with practical usage examples for authentication and API discovery services, contains markdown formatting errors |
| packages/agents-a365-observability/README.md | Simplified from extensive feature documentation to essential observability setup and span creation examples |
| packages/agents-a365-observability-extensions-openai/README.md | Reduced to concise instrumentation setup guide with duplicate import statement |
| packages/agents-a365-notifications/README.md | Condensed to essential notification handling patterns and email-specific examples |
| README.md | Comprehensive rewrite with enhanced project structure, build instructions, and navigation links to related resources |
Mrunal Suyoga Hirve (mrunalhirve128)
requested a review
from Jim Daly (JimDaly)
November 9, 2025 07:40
Jim Daly (JimDaly)
requested changes
Nov 10, 2025
Jim Daly (JimDaly)
left a comment
Jim Daly (JimDaly)
left a comment
Member
There was a problem hiding this comment.
Let's verify that all the usage examples are in the docs and remove them from these README files.
Copilot finished reviewing on behalf of
Mrunal Suyoga Hirve (mrunalhirve128)
November 13, 2025 00:33
…thub.com/microsoft/Agent365-nodejs into users/mrunalhirve/AddReadMeForProjectsJs
Copilot started reviewing on behalf of
Mrunal Suyoga Hirve (mrunalhirve128)
November 13, 2025 01:27
View session
Copilot finished reviewing on behalf of
Mrunal Suyoga Hirve (mrunalhirve128)
November 13, 2025 01:29
Jesus Daniel Terrazas (JesuTerraz)
previously approved these changes
Nov 13, 2025
Johan Broberg (pontemonti)
previously approved these changes
Nov 13, 2025
Mrunal Suyoga Hirve (mrunalhirve128)
dismissed stale reviews from Johan Broberg (pontemonti) and Jesus Daniel Terrazas (JesuTerraz)
via
November 13, 2025 04:11
7e8e996
Copilot started reviewing on behalf of
Mrunal Suyoga Hirve (mrunalhirve128)
November 13, 2025 04:13
View session
Copilot finished reviewing on behalf of
Mrunal Suyoga Hirve (mrunalhirve128)
November 13, 2025 04:15
Copilot started reviewing on behalf of
Mrunal Suyoga Hirve (mrunalhirve128)
November 13, 2025 04:20
View session
Copilot finished reviewing on behalf of
Mrunal Suyoga Hirve (mrunalhirve128)
November 13, 2025 04:21
Johan Broberg (pontemonti)
approved these changes
Nov 13, 2025
Johan Broberg (pontemonti)
deleted the
users/mrunalhirve/AddReadMeForProjectsJs
branch
December 5, 2025 17:09
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.
Update Readme's