Skip to content

Conversation

@nedithgar
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings November 25, 2025 07:20
Copilot finished reviewing on behalf of nedithgar November 25, 2025 07:22
Copy link

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 debug logging capabilities to the ast-grep MCP server, enabling verbose debugging output when diagnosing issues with command execution and tool calls.

Key changes include:

  • Introduction of a debug logging system using TaskLocal for thread-safe context propagation
  • Implementation of thread-safe output buffering using async readabilityHandler for process pipes
  • Addition of a --debug flag to enable verbose logging to stderr

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
Sources/ast-grep-mcp-swift/ast_grep_mcp_swift.swift Adds DebugContext, OutputBuffer class for thread-safe pipe reading, debug logging throughout command execution and tool handling, and --debug flag configuration
README.md Documents the new --debug flag usage example

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nedithgar nedithgar requested a review from Copilot November 25, 2025 07:45
Copilot finished reviewing on behalf of nedithgar November 25, 2025 07:46
Copy link

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nedithgar nedithgar merged commit bb777e1 into main Nov 25, 2025
@nedithgar nedithgar deleted the feat/debug-info branch November 25, 2025 07:58
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.

2 participants