[PR #25] [MERGED] break: remove winston in favor of the debug package #51

New Issue

2026-02-16T08:17:24-05:00

yindo commented

2026-02-16 08:17:24 -05:00

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langchainjs-mcp-adapters/pull/25
Author: @benjamincburns
Created: 3/19/2025
Status: ✅ Merged
Merged: 3/19/2025
Merged by: @benjamincburns

Base: main ← Head: ben/debug-logger

📝 Commits (2)

db22ac9 break: remove winston in favor of the debug package
00c795f fix: add missing test coverage dev dependency

📊 Changes

11 files changed (+739 additions, -581 deletions)

View changed files

📝 README.md (+16 -20)
➖ __tests__/logger.test.ts (+0 -109)
📝 examples/firecrawl_multiple_servers_example.ts (+1 -4)
📝 package-lock.json (+645 -267)
📝 package.json (+2 -1)
📝 src/client.ts (+56 -45)
📝 src/index.ts (+0 -1)
➖ src/logger.ts (+0 -124)
📝 src/tools.ts (+15 -8)
📝 tsconfig.examples.json (+2 -1)
📝 tsconfig.json (+2 -1)

📄 Description

Breaking change - drops the support for winston logging in favor of the lighter-weight, and more fit-for-purpose debug module.

Also includes a missing dev dependency for calculating test coverage.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langchainjs-mcp-adapters/pull/25 **Author:** [@benjamincburns](https://github.com/benjamincburns) **Created:** 3/19/2025 **Status:** ✅ Merged **Merged:** 3/19/2025 **Merged by:** [@benjamincburns](https://github.com/benjamincburns) **Base:** `main` ← **Head:** `ben/debug-logger` --- ### 📝 Commits (2) - [`db22ac9`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/db22ac9bbdfe4ae79ab3812bb98ec6d39e6dbd63) break: remove winston in favor of the debug package - [`00c795f`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/00c795fa6ef1832863021ce59e0f1b3eb57bbc84) fix: add missing test coverage dev dependency ### 📊 Changes **11 files changed** (+739 additions, -581 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+16 -20) ➖ `__tests__/logger.test.ts` (+0 -109) 📝 `examples/firecrawl_multiple_servers_example.ts` (+1 -4) 📝 `package-lock.json` (+645 -267) 📝 `package.json` (+2 -1) 📝 `src/client.ts` (+56 -45) 📝 `src/index.ts` (+0 -1) ➖ `src/logger.ts` (+0 -124) 📝 `src/tools.ts` (+15 -8) 📝 `tsconfig.examples.json` (+2 -1) 📝 `tsconfig.json` (+2 -1) </details> ### 📄 Description Breaking change - drops the support for `winston` logging in favor of the lighter-weight, and more fit-for-purpose `debug` module. Also includes a missing dev dependency for calculating test coverage. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

yindo added the pull-request label 2026-02-16 08:17:24 -05:00

yindo closed this issue

2026-02-16 08:17:24 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: langchain-ai/langchainjs-mcp-adapters#51