Skip to content

Commit 306240a

Browse files
committed
Debug
1 parent aea19b6 commit 306240a

File tree

3 files changed

+9
-3
lines changed

3 files changed

+9
-3
lines changed

conformance/expected_failures.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,3 @@ server:
55
- tools-call-elicitation
66
- elicitation-sep1034-defaults
77
- elicitation-sep1330-enums
8-
# Fails on CI (Ubuntu, Ruby 4.0) but passes locally. Root cause under investigation.
9-
- tools-call-with-logging

conformance/server.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,11 +159,15 @@ class << self
159159
attr_accessor :server_ref
160160

161161
def call(**_args)
162+
$stderr.puts "[DIAG] notify_log_message #1, transport=#{server_ref&.transport.class}"
162163
server_ref&.notify_log_message(data: "Tool execution started", level: "info", logger: "test_logger")
164+
$stderr.puts "[DIAG] notify_log_message #1 done"
163165
sleep(0.05)
164166
server_ref&.notify_log_message(data: "Tool processing data", level: "info", logger: "test_logger")
167+
$stderr.puts "[DIAG] notify_log_message #2 done"
165168
sleep(0.05)
166169
server_ref&.notify_log_message(data: "Tool execution completed", level: "info", logger: "test_logger")
170+
$stderr.puts "[DIAG] notify_log_message #3 done, logging_notification=#{server_ref&.logging_message_notification.inspect}"
167171
MCP::Tool::Response.new([{ type: "text", text: "Logging complete (3 messages sent)" }])
168172
end
169173
end

lib/mcp/server.rb

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,14 +169,18 @@ def notify_resources_list_changed
169169
end
170170

171171
def notify_log_message(data:, level:, logger: nil)
172+
$stderr.puts "[DIAG] notify_log_message: transport=#{@transport.class}, notification=#{logging_message_notification.inspect}, should_notify=#{logging_message_notification&.should_notify?(level)}"
172173
return unless @transport
173174
return unless logging_message_notification&.should_notify?(level)
174175

175176
params = { "data" => data, "level" => level }
176177
params["logger"] = logger if logger
177178

178-
@transport.send_notification(Methods::NOTIFICATIONS_MESSAGE, params)
179+
result = @transport.send_notification(Methods::NOTIFICATIONS_MESSAGE, params)
180+
$stderr.puts "[DIAG] send_notification result=#{result.inspect}"
181+
result
179182
rescue => e
183+
$stderr.puts "[DIAG] notify_log_message ERROR: #{e.class}: #{e.message}"
180184
report_exception(e, { notification: "log_message" })
181185
end
182186

0 commit comments

Comments
 (0)