phronomy 0.5.1 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ca63f10e2d505005a6011a8755135e0dea1c3d8e8013ae3b2ea1d3b5ecf13d3
-  data.tar.gz: 420ec691725b6450a0430cdce90372c13da4e73d7013a630b2aa33a7b72e496d
+  metadata.gz: 49b20f3defaed56477f9f1ee375a450d26a770d004c052754cc5c045746587cc
+  data.tar.gz: 1d2fe811e467c7d04208b82cc9d9ca5fca9b17d0bf6061aa952a2fb862c23a53
 SHA512:
-  metadata.gz: 300b155e482fe0b0015bba5d3985c8d37f2599828d9d87763cc5d9925c3bf399cbee46a658d48a79e152d9b410505609e35a89d3b02db632d51b85280938dc8c
-  data.tar.gz: 88184d595d04eb1ed1be8c6ec145476a5e1f7e6e02e52658f9d80ad3893a29fbcfc377c736b7c0de10deb14e6105bd9594d6b950bd2fb3afda800e770a2219c4
+  metadata.gz: 763cf25297e0c8799ad76bcd362ecb5f1899a9ccd0d90791e119d2d0946c59f7c076f7a00d92e01e64735e90b45d7e1aa5462e41efceee9147daf45ac214551f
+  data.tar.gz: 3a30e9198008dd9e4e512c374324b4c1cfda40c2a41762ff160f90ffa8ac98c0669f700d810e899fa661bd68af3a14db0145a00afc4d0e97d560d7de27989db0

data/CHANGELOG.md

@@ -7,7 +7,61 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ---
 
-## [0.5.1] - 2026-05-21
+## [0.5.3] - 2026-05-20
+
+### Bug Fixes
+
+- **Ensure `from_server` closes transport on error** (#95): The short-lived
+  transport created inside `McpTool.from_server` is now wrapped in
+  `begin/ensure`, so the underlying child process (stdio) is always
+  terminated even when `fetch_tool` raises.
+- **Correct `McpTool#close` documentation** (#94): The comment previously
+  stated that calling `execute` after `close` raises an error; in practice
+  the transport reopens automatically. The comment now reflects actual
+  behaviour.
+
+### Documentation
+
+- **Fix CHANGELOG date for v0.5.1** (#96): The v0.5.1 entry had an
+  incorrect date of 2026-05-21; corrected to 2026-05-20.
+
+---
+
+## [0.5.2] - 2026-05-20
+
+### Bug Fixes
+
+- **CHANGELOG correction for v0.5.1 MCP fix** (#90): The v0.5.1 entry
+  incorrectly stated that a `Mutex` was added to `StdioTransport#rpc_call`.
+  The actual fix was per-instance transport ownership (each `McpTool` instance
+  creates its own transport in `initialize`). Corrected the description.
+
+### Enhancements
+
+- **Add `McpTool#close`** (#92): Tool instances now expose a `close` method
+  that shuts down the underlying stdio child process (`StdioTransport`) or
+  releases the HTTP connection (`HttpTransport`). This gives callers a
+  deterministic way to clean up resources instead of relying on GC.
+
+### Maintenance
+
+- **Archive stale Rails integration design doc** (#91): Added an archived
+  notice to `spec/design/17_rails_integration.md` clarifying that Rails
+  integration was removed in v0.3.0–v0.5.1 and the document is for
+  historical reference only.
+
+- **Remove zombie `register_workflow_context` API** (#93): The
+  `Phronomy.register_workflow_context`, `workflow_context_registry`, and
+  `reset_workflow_context_registry!` methods (along with the backing
+  `@workflow_context_registry` and `@registry_mutex` module-level variables)
+  were removed from `lib/phronomy.rb`. These existed to support the
+  `StateStore` deserialization guard, which was removed in a prior release.
+  The API had no remaining callers in the codebase and was not listed in
+  the README stability table.
+
+---
+
+## [0.5.1] - 2026-05-20
 
 ### Bug Fixes
 
@@ -17,10 +71,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   `lib/generators/`, `lib/phronomy/railtie.rb`, and all references in
   `lib/phronomy.rb`.
 
-- **Fix thread-safety of `StdioTransport#rpc_call`** (#86): Concurrent calls
-  to the same `McpTool` instance could interleave JSON-RPC writes and reads,
-  corrupting request/response pairing. A `Mutex` is now held around each
-  write+read cycle. Also adds the missing `require "securerandom"`.
+- **Fix MCP transport ownership** (#86): `McpTool` no longer stores a shared
+  transport at class level. `from_server` now uses a short-lived transport only
+  to fetch tool metadata and calls `close` immediately after. Each tool instance
+  creates its own `StdioTransport` or `HttpTransport` in `initialize`, so
+  concurrent callers (e.g. via `Orchestrator#dispatch_parallel`) never share
+  stdio streams. No `Mutex` is needed. Also adds missing
+  `require "securerandom"` and a no-op `HttpTransport#close` for interface
+  consistency.
 
 ### Documentation
 

data/README.md

@@ -391,6 +391,13 @@ search_tool = Phronomy::Tool::McpTool.from_server(
 )
 ```
 
+Call `close` when the tool is no longer needed to shut down the underlying
+child process (stdio transport) or release the HTTP connection:
+
+```ruby
+search_tool.close
+```
+
 ### Conversation History — passing prior messages
 
 Phronomy does not manage conversation history internally. The application owns the

data/lib/phronomy/tool/mcp_tool.rb

@@ -41,8 +41,11 @@ module Phronomy
           # then close it.  Each McpTool instance creates its own transport
           # so that concurrent callers never share IO streams.
           transport = build_transport(server_uri)
-          tool_def = transport.fetch_tool(tool_name)
-          transport.close
+          begin
+            tool_def = transport.fetch_tool(tool_name)
+          ensure
+            transport.close
+          end
           build_tool_class(tool_name, server_uri, tool_def).new
         end
 
@@ -82,6 +85,15 @@ module Phronomy
             @mcp_transport.call_tool(tool_name, args)
           end
 
+          # Allow callers to deterministically shut down the underlying child
+          # process (stdio) or release the HTTP connection.  For HttpTransport
+          # this is a no-op.  After close, calling execute will reopen the
+          # transport automatically (stdio restarts the child process; HTTP
+          # opens a fresh connection per call).
+          klass.define_method(:close) do
+            @mcp_transport.close
+          end
+
           klass
         end
       end

data/lib/phronomy/vector_store/in_memory.rb

@@ -28,7 +28,13 @@ module Phronomy
       # @param k               [Integer]
       # @return [Array<Hash>] sorted by descending score
       def search(query_embedding:, k: 5)
-        results = @documents.map do |id, doc|
+        # Take an atomic snapshot before iterating.  Hash#dup is a C-level
+        # call that completes without releasing the GVL, so it is atomic with
+        # respect to any other Ruby thread.  Iterating the copy instead of
+        # @documents directly prevents "can't add a new key into hash during
+        # iteration" when a concurrent thread calls #add.
+        snapshot = @documents.dup
+        results = snapshot.map do |id, doc|
           score = cosine_similarity(query_embedding, doc[:embedding])
           {id: id, score: score, metadata: doc[:metadata]}
         end

data/lib/phronomy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Phronomy
-  VERSION = "0.5.1"
+  VERSION = "0.5.3"
 end

data/lib/phronomy.rb

@@ -50,42 +50,7 @@ module Phronomy
     end
   end
 
-  # Registry for WorkflowContext classes that may be serialized to external stores
-  # (Redis, DB). Call +register_workflow_context+ at application startup so that
-  # only known classes can be deserialized.
-  @workflow_context_registry = nil
-  @registry_mutex = Mutex.new
-
   class << self
-    # Register one or more WorkflowContext classes that are allowed to be
-    # deserialized by StateStore backends. When at least one class is registered,
-    # only registered classes will be accepted by
-    # +StateStore::Base#safe_state_class+.
-    #
-    # Call this once at application startup (e.g. in a Rails initializer).
-    #
-    # @param classes [Array<Class>] classes including Phronomy::WorkflowContext
-    # @example
-    #   Phronomy.register_workflow_context(ScanContext, OtherContext)
-    def register_workflow_context(*classes)
-      @registry_mutex.synchronize do
-        @workflow_context_registry ||= {}
-        classes.each do |klass|
-          raise ArgumentError, "#{klass.inspect} is not a Class" unless klass.is_a?(Class)
-          @workflow_context_registry[klass.name] = klass
-        end
-      end
-    end
-
-    # Returns the current registry Hash, or nil when no class has been registered.
-    # @return [Hash{String => Class}, nil]
-    attr_reader :workflow_context_registry
-
-    # Clears the registry. Primarily used in tests.
-    def reset_workflow_context_registry!
-      @registry_mutex.synchronize { @workflow_context_registry = nil }
-    end
-
     def configuration
       @configuration ||= Configuration.new
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: phronomy
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.3
 platform: ruby
 authors:
 - Raizo T.C.S