Skip to content

Conversation

@abdulanu0
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 18, 2025 19:02
@abdulanu0 abdulanu0 requested a review from a team as a code owner December 18, 2025 19:02
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 removes the deprecated TOOLS_MODE environment variable configuration from all Node.js sample agent projects. The TOOLS_MODE configuration option is no longer used in the codebase, as the samples now directly use the MCP Platform without requiring mode selection.

Key Changes:

  • Removed TOOLS_MODE environment variable from configuration templates across all Node.js samples
  • Cleaned up obsolete configuration comments referencing mode options (MockMCPServer | MCPPlatform)

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
nodejs/openai/sample-agent/.env.template Removed TOOLS_MODE configuration and related comment
nodejs/n8n/sample-agent/Agent-Code-Walkthrough.MD Removed TOOLS_MODE from example configuration snippet
nodejs/n8n/sample-agent/.env.template Removed TOOLS_MODE configuration line
nodejs/langchain/sample-agent/.env.example Removed TOOLS_MODE configuration and related comment
nodejs/claude/sample-agent/.env.template Removed TOOLS_MODE configuration and related comment

The changes are clean and complete. All references to TOOLS_MODE have been successfully removed from configuration files, and no code references to this environment variable exist in the source files. This simplifies the configuration for developers using these samples.

@abdulanu0 abdulanu0 merged commit 72efbb7 into main Dec 19, 2025
16 of 18 checks passed
@abdulanu0 abdulanu0 deleted the users/anabdul/removingtoolsmode branch December 19, 2025 20:39
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.

3 participants