actionmcp 0.80.0 → 0.80.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ea96963047c8bb5e9fa9fd88163a14798c8780a531fa1600e3681bd92ca1b4fe
-  data.tar.gz: aae9fc897bd554fba0442735b184d9eac03a381c3b07e052869ce76ce714555d
+  metadata.gz: 6251e9ce3e87878bf86bb3ca1bbfd3363fb4fedccd9567a0b9b6c42fbc87241a
+  data.tar.gz: ee8f4bdd5ad42c611f7f652f836e40ce0477510e9c0e856a0a32eaa447b77718
 SHA512:
-  metadata.gz: 47947a7bba2e39a33b1793cbf8af4a38dcf26e0eebcc19989b143d4279f8f71fc05c646169d4739dd8e52f23c2f34df5a232d4cd414e6872cc636ab4c5a22ae5
-  data.tar.gz: 8e2b06ddaf6b1174035dc3e4a6315aaf8d624ac5a096d647f252c40c533f5d20ebf02b3b8b5355d81617de98dfccf63d4342ec540558f8dbc5072043aee11c4a
+  metadata.gz: 80712d1d14d91fb087bf8bd35c4cedb61ea22015a958dc5c66df64dfa7872d0755ea08a4ecbe02c1b577bb14eff8bcd30aaaa78ffba61c4b6e16ea37c336c313
+  data.tar.gz: '0920ffdcb46a86910789282365a7ff6391ec4a1bf424be3dee7b8174497953df4aa73098e89a58559bb59d8157791d7620045339e763b3b27c885037b5e420fd'

data/app/models/concerns/{mcp_console_helpers.rb → action_mcp/mcp_console_helpers.rb}

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-# app/models/concerns/mcp_console_helpers.rb
-module MCPConsoleHelpers
-  extend ActiveSupport::Concern
+module ActionMCP
+  module MCPConsoleHelpers
+    extend ActiveSupport::Concern
 
   class_methods do
     def pretty_messages(session_or_messages, limit: 10)
@@ -65,4 +65,5 @@ module MCPConsoleHelpers
   def message_flow(limit: 50)
     self.class.message_flow(self, limit: limit)
   end
+  end
 end

data/app/models/concerns/{mcp_message_inspect.rb → action_mcp/mcp_message_inspect.rb}

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-# app/models/concerns/mcp_message_inspect.rb
-module MCPMessageInspect
-  extend ActiveSupport::Concern
+module ActionMCP
+  module MCPMessageInspect
+    extend ActiveSupport::Concern
 
   def inspect(show_data: false)
     if show_data
@@ -70,4 +70,5 @@ module MCPMessageInspect
 
     "#{colors[color]}#{text}\e[0m"
   end
+  end
 end

data/lib/action_mcp/resource_template.rb

@@ -274,6 +274,7 @@ module ActionMCP
           else
             @response.add_content(result)
           end
+          @response
         end
       rescue StandardError => e
         @response.mark_as_resolution_failed!("template://#{self.class.name}", e.message)

data/lib/action_mcp/server/resources.rb

@@ -45,10 +45,11 @@ module ActionMCP
       # @example Output:
       #   # Sends: {"jsonrpc":"2.0","id":"req-789","result":{"contents":[{"uri":"file:///example.txt","text":"Example content"}]}}
       def send_resource_read(id, params)
-        template = ResourceTemplatesRegistry.find_template_for_uri(params[:uri])
+        uri = params["uri"]
+        template = ResourceTemplatesRegistry.find_template_for_uri(uri)
 
         unless template
-          send_jsonrpc_error(id, :resource_not_found, "No resource template found for URI: #{params[:uri]}")
+          send_jsonrpc_error(id, :method_not_found, "No resource template found for URI: '#{uri}'")
           return
         end
 
@@ -65,21 +66,21 @@ module ActionMCP
 
         begin
           # Create template instance and set execution context
-          record = template.process(params[:uri])
+          record = template.process(uri)
           record.with_context({ session: session })
 
           response = record.call
 
           if response.error?
-            # Convert ResourceResponse errors to JSON-RPC errors
-            error_info = response.to_h
-            send_jsonrpc_error(id, error_info[:code], error_info[:message], error_info[:data])
+            # response.to_h works properly when error? is true:
+            send_jsonrpc_response(id, error: response.to_h)
           else
             # Handle successful response - ResourceResponse.contents is already an array
             send_jsonrpc_response(id, result: { contents: response.contents.map(&:to_h) })
           end
         rescue StandardError => e
-          log_error(e, { resource_uri: params[:uri], template: template.name })
+          # Should rather `ErrorHandling` module be included here? Then we could use `log_error(e)` directly.
+          Rails.logger.error "[MCP Error] #{e.class}: #{e.message}"
           send_jsonrpc_error(id, :internal_error, "Failed to read resource: #{e.message}")
         end
       end

data/lib/action_mcp/version.rb

@@ -2,7 +2,7 @@
 
 require_relative "gem_version"
 module ActionMCP
-  VERSION = "0.80.0"
+  VERSION = "0.80.1"
 
   class << self
     alias version gem_version

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: actionmcp
 version: !ruby/object:Gem::Version
-  version: 0.80.0
+  version: 0.80.1
 platform: ruby
 authors:
 - Abdelkader Boudih
@@ -142,8 +142,8 @@ files:
 - app/models/action_mcp/session/resource.rb
 - app/models/action_mcp/session/sse_event.rb
 - app/models/action_mcp/session/subscription.rb
-- app/models/concerns/mcp_console_helpers.rb
-- app/models/concerns/mcp_message_inspect.rb
+- app/models/concerns/action_mcp/mcp_console_helpers.rb
+- app/models/concerns/action_mcp/mcp_message_inspect.rb
 - config/routes.rb
 - db/migrate/20250512154359_consolidated_migration.rb
 - db/migrate/20250715070713_add_consents_to_action_mcp_sess.rb