llm.rb 15.0.1 → 15.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3ce996689c82e7a0d62a7f09ab2545d10b36462fc75cc2feeb57ce26f6523585
-  data.tar.gz: 74cf140b0fd11b077aea57bde027f0b00cf67f4e099ad35778bc5042b958469b
+  metadata.gz: c86e9ccde2cc0844b5172935bc222c1adf3cda8b0f49e84730179bcc635b2d2b
+  data.tar.gz: 60c740c3dc2c20d8b1d86de55ae5ef25eb7bbe5b5b567f3766c059b37520efcc
 SHA512:
-  metadata.gz: 4274138e7054d9db54e121dcad674f307b0e1bdbfc568ac16bb3b13181e873a77a57d19f4d2eef2479216555ba0ed9213fa7372cf051d4fa73f6fe73dae594cd
-  data.tar.gz: 12219c398f482c7f949bc9964038a2597e74a592b78023b101e96a337605c2e2fb8963f5526fbd4ca8f1ffd8b14b843e1a10334369d2f62a1e393eb409f4821d
+  metadata.gz: fd1901c5bebd659b242db55ed91ea4f538832c24b4ab119b80f46ae987fc5166823fa70174d9f545c079addd90487eb379b1dfd6eeea2365d045a831b861134e
+  data.tar.gz: 5949efbaa7381fb0cab7328e64c08af4022a8b976a24b8b24901c167c67cd060f912a7054956a7942cd7cf67b3f0aa226ad5aec608bcd273bfa1aa25d3ffb5d0

data/CHANGELOG.md

@@ -15,6 +15,24 @@
 
 ## What's next
 
+## v15.0.2
+
+Changes since `v15.0.1`.
+
+This release fixes a concurrency race in MCP tool calls by
+reference-counting the transport session, so overlapping tool calls
+reuse the running transport instead of racing `start`/`stop`.
+
+### Fix
+
+* **mcp: fix concurrent MCP tool call race** <br>
+  `LLM::MCP` now reference-counts its transport session: the first
+  caller starts the transport and the last caller stops it. Concurrent
+  or overlapping tool calls reuse the running transport instead of
+  racing `start`/`stop`, avoiding "MCP transport is not running" errors
+  that could occur with the `async` strategy. An externally started
+  transport is never stopped by a borrower.
+
 ## v15.0.1
 
 Changes since `v15.0.0`.

data/README.md

@@ -25,6 +25,13 @@ Once you learn the fundamentals, everything else falls into place
 naturally. Some features, such as ActiveRecord support, require
 optional dependencies that are opt-in.
 
+The best way to learn about llm.rb is to ask [the official chatbot](https://r.uby.dev)
+a question. It is connected to the llm.rb GitHub repository, backed by
+ActiveRecord and uses the builtin MCP feature to connect to GitHub. If
+you want to know more then simply [ask the chatbot](https://r.uby.dev) and
+you should get an answer that is grounded in the source code. It is 100%
+free to use and powered by DeepSeek under the hood. 
+
 ## Install
 
 ```bash

data/lib/llm/mcp.rb

@@ -87,7 +87,10 @@ class LLM::MCP
   # @return [LLM::MCP] A new MCP instance
   def initialize(stdio: nil, http: nil, timeout: 30)
     @timeout = timeout
-    if stdio && http
+    @lock = Mutex.new
+    @borrowers = 0
+    @owned = false
+    if stdio and http
       raise ArgumentError, "stdio and http are mutually exclusive"
     elsif stdio
       @command = Command.new(**stdio)
@@ -125,7 +128,10 @@ class LLM::MCP
   #  Propagates errors raised by {#start}, the block itself, or {#stop}
   # @return [void]
   def run
-    start
+    @lock.synchronize do
+      start
+      @owned = false
+    end
     yield
   ensure
     stop
@@ -182,13 +188,32 @@ class LLM::MCP
 
   attr_reader :command, :transport, :timeout
 
+  ##
+  # Borrows the transport for the duration of the block.
+  #
+  # The first borrower starts the transport, concurrent borrowers reuse
+  # it, and the last borrower stops it, so overlapping tool calls never
+  # race `start`/`stop` and trip over "MCP transport is not running".
+  # An externally started transport is never stopped by a borrower.
+  # @yield Runs while the transport is running
+  # @return [void]
   def with_session
-    return yield if transport.running?
-    session_started = true
-    start
+    @lock.synchronize do
+      @borrowers += 1
+      unless transport.running?
+        start
+        @owned = true
+      end
+    end
     yield
   ensure
-    stop if session_started
+    @lock.synchronize do
+      @borrowers -= 1
+      if @borrowers.zero? and @owned
+        @owned = false
+        stop
+      end
+    end
   end
 
   def adapt_content(content)

data/lib/llm/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LLM
-  VERSION = "15.0.1"
+  VERSION = "15.0.2"
 end

data/llm.gemspec

@@ -40,13 +40,13 @@ DESCRIPTION
   spec.post_install_message = "\n" \
                               "Got a question about llm.rb? " \
                               "\n" \
-                              "Ask our chatbot." \
+                              "Ask the https://r.uby.dev chatbot." \
                               "\n" \
                               "It is connected to the official GitHub repository." \
                               "\n" \
                               "100% free to use." \
                               "\n" \
-                              "https://r.uby.dev" \
+                              "Built with llm.rb and DeepSeek." \
                               "\n\n"
 
   spec.add_development_dependency "webmock", "~> 3.24.0"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: llm.rb
 version: !ruby/object:Gem::Version
-  version: 15.0.1
+  version: 15.0.2
 platform: ruby
 authors:
 - Robert Gleeson
@@ -652,8 +652,9 @@ metadata:
   source_code_uri: https://github.com/r-uby-dev/llm
   documentation_uri: https://r.uby.dev
   changelog_uri: https://github.com/r-uby-dev/llm/blob/main/CHANGELOG.md
-post_install_message: "\nGot a question about llm.rb? \nAsk our chatbot.\nIt is connected
-  to the official GitHub repository.\n100% free to use.\nhttps://r.uby.dev\n\n"
+post_install_message: "\nGot a question about llm.rb? \nAsk the https://r.uby.dev
+  chatbot.\nIt is connected to the official GitHub repository.\n100% free to use.\nBuilt
+  with llm.rb and DeepSeek.\n\n"
 rdoc_options: []
 require_paths:
 - lib