Small cleanup for .env, README and strings in the code#137
Merged
Conversation
* Added CODEOWNERS * Added .gitignore --------- Co-authored-by: Johan Broberg <johanb@microsoft.com>
Co-authored-by: Johan Broberg <johanb@microsoft.com>
Co-authored-by: Johan Broberg <johanb@microsoft.com>
Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
Co-authored-by: Johan Broberg <johanb@microsoft.com>
Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
* Adding Python Agent Framework Sample * update to latest --------- Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
* Revise README for Agent 365 Sample Agent Updated the README to improve clarity and correct typos. Added sections on prerequisites, running the sample, and troubleshooting. * Update README to remove external sample reference Removed reference to the semantic-kernel-multiturn sample.
* update agents sdk version for langchain * Accept more recent versions * remove authority field, this is set by default --------- Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
* Adding Notifications on AF * Update readme * PR comments * small fix * removed observability * update manifest * changes * setup fix
Added comprehensive instructions for GitHub Copilot to follow during code reviews, including rules for checking keywords, copyright headers, and Agent 365 sample validation.
* update with api change * update claude to tool service api name change * name change --------- Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
* Add Vercel AI SDK sample * fix comments --------- Co-authored-by: Dominik Bezic <dominikbezic@microsoft.com>
* introducing perplexity sample agent * applying changes from code review * updated readme file with features list --------- Co-authored-by: aubreyquinn <aubreyquinn+odspmdb@microsoft.com>
* Add OpenAI Nodejs Sample * fix: model to string * Add copyright headers * Add build script * fix typo * fix typo in doc * take comments * update doc for env name change * update license path in readme * update walkthrough to follow same structure as AF * include agentsplayground in readme --------- Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
…ion (#31) * Initial plan * Rename "Microsoft Agents A365" to "Microsoft Agent 365" Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>
Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
* add updated nodejs claude implementation * fix typos * change mcp endpoint to prod * review comments * add auth from agents-hosting * remove lock
* quickstart initial code * rename folder * update docs on quickstart * add links and update docs * include rename change * copilot suggestions --------- Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
* Introducing playground notification handling in Perplexity agent * Introducing playground Teams messaging notification handling in Perplexity agent * applying changes from code review * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * applying changes from code review * Add missing copyright header to playgroundActivityTypes.ts (#36) * Initial plan * Add Microsoft copyright header to playgroundActivityTypes.ts Co-authored-by: aubreyquinn <80953505+aubreyquinn@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: aubreyquinn <80953505+aubreyquinn@users.noreply.github.com> * Add copyright header to playgroundActivityTypes.ts (#35) * Initial plan * Add Microsoft copyright header to playgroundActivityTypes.ts Co-authored-by: aubreyquinn <80953505+aubreyquinn@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: aubreyquinn <80953505+aubreyquinn@users.noreply.github.com> --------- Co-authored-by: aubreyquinn <aubreyquinn+odspmdb@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
* Cursor IDE prompt usage --------- Co-authored-by: shirahman <shirahman@microsoft.com>
* Update samples to remove environment id from calls and settings * remove env id from newer samples * Update python/agent-framework/sample-agent/.env.template Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fix formatting of 'Microsoft Agent365' to 'Microsoft Agent 365' * Update README to reflect Microsoft Agent 365 SDK * Fix typo in Agent365 authentication comment * Fix comment formatting in client.ts Updated comments to improve clarity and consistency. * Fix typo in AGENT-TESTING.md * Fix description formatting in pyproject.toml * Update package description for clarity * Fix formatting of Agent 365 configuration comment * Update README to reference Microsoft Agent 365 SDK * Fix typo in Agent 365 Observability SDK comment * Update package description for clarity * Fix description formatting in pyproject.toml * Fix formatting in .env.template for Agent 365 * Update README.md * Update README.md * Fix naming in README for Agent Framework and SDK Updated README to correct naming conventions for Agent Framework and Microsoft Agent 365 SDK. * Update license link in README.md * Update perplexityClient.ts * Update README to reference Microsoft Agent 365 SDK * Fix typo in LangChain client documentation * Update README to reflect Microsoft Agent 365 SDK
…le README (#108) * Initial plan * Fix broken SECURITY.md and LICENSE.md links in LangChain Node.js README Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>
* Add contributing guidelines to CONTRIBUTING.md Added contributing guidelines to the project, including sections on code of conduct, how to contribute, development process, and licensing. * Update CONTRIBUTING.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update CONTRIBUTING.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Johan Broberg <johan@pontemonti.net> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n for auto instrument for OpenAi Agent sample. (#96) * fix manual instrument * Configure OpenAI Agents instrumentation * fix PR build error * Update nodejs/openai/sample-agent/src/client.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix invoke scope --------- Co-authored-by: jsl517 <pefan@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Johan Broberg <johanb@microsoft.com>
* Introduce per application validation for python Samples * Introduce per application validation for python Samples * Introduce per application validation for python Samples * Introduce per application validation for python Samples * Introduce per application validation for python Samples * Introduce per application validation for python Samples * Introduce per application validation for python Samples * Introduce per application validation for python Samples
* Initial plan * Fix nodemon watch pattern for macOS compatibility in Node.js samples Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>
* Add survey section to README for user feedback Added a survey link for feedback on the SDK and CLI. * Update README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Remove Top Contributor badge from README Removed the Top Contributor badge from the README. --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Add thumbnail images for ATK sample onboarding * Add files via upload * Update OpenAI sample * Updating sample thumbnails * Add missing zod peer dependency to OpenAI and Claude samples * Revert "Add missing zod peer dependency to OpenAI and Claude samples" This reverts commit d863f27. * Revert changes to uv.lock
* Create EmailResponse Activity when replying to email * Add default response handling for notifications --------- Co-authored-by: Jesus Terrazas <jterrazas@microsoft.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request performs cleanup tasks for the Vercel AI SDK sample agent, removing legacy "compliance" references from the code and improving configuration documentation. The changes align the sample's naming with its actual purpose (Vercel AI SDK integration) and enhance the developer experience by adding clarifying comments and fixing navigation links.
Key changes:
- Updated agent and tenant identifiers from "compliance" terminology to "vercel-ai-sdk" naming
- Fixed LICENSE file path in README to point to the correct location in the repository root
- Cleaned up .env.example by removing Azure/OpenAI-specific telemetry settings that don't apply to this Anthropic-based sample and added helpful inline comments for service connection settings
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| nodejs/vercel-sdk/sample-agent/src/client.ts | Renamed agent and tenant identifiers from "compliance" references to "vercel-ai-sdk" terminology for consistency with the sample's purpose |
| nodejs/vercel-sdk/sample-agent/README.md | Corrected LICENSE file path from relative to absolute repository path (../../../LICENSE.md) |
| nodejs/vercel-sdk/sample-agent/.env.example | Removed unnecessary Azure/OpenAI telemetry configuration, added helpful comments for service connection settings, and cleaned up formatting |
- trailing whitespace Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
pontemonti
previously approved these changes
Dec 19, 2025
f93d40d to
060a096
Compare
…osoft/Agent365-Samples into dominikbezic/cleanup-vercel # Conflicts: # nodejs/vercel-sdk/sample-agent/.env.example
The merge-base changed after approval.
pontemonti
approved these changes
Dec 30, 2025
walterluna
approved these changes
Dec 30, 2025
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.