phronomy 0.5.1 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ca63f10e2d505005a6011a8755135e0dea1c3d8e8013ae3b2ea1d3b5ecf13d3
-  data.tar.gz: 420ec691725b6450a0430cdce90372c13da4e73d7013a630b2aa33a7b72e496d
+  metadata.gz: f2129e4cbe20c3831530f0a8ef810abe272174195cbce5198207fec6bf255eff
+  data.tar.gz: f9a0152759518cb2d85126ee5fb7a57b96e00a0ed7b8acffd54c57de91ed0c18
 SHA512:
-  metadata.gz: 300b155e482fe0b0015bba5d3985c8d37f2599828d9d87763cc5d9925c3bf399cbee46a658d48a79e152d9b410505609e35a89d3b02db632d51b85280938dc8c
-  data.tar.gz: 88184d595d04eb1ed1be8c6ec145476a5e1f7e6e02e52658f9d80ad3893a29fbcfc377c736b7c0de10deb14e6105bd9594d6b950bd2fb3afda800e770a2219c4
+  metadata.gz: 0af940bc5279c64221d000e4c545c5e2696f2f14c405326785786b94cf9afc567619e94e79aefbfc68b0656d761ebe88b7604d1cbacf75e82ae3aca6d39b9e3f
+  data.tar.gz: 71d4ef2f73a4e914d4dd85fffef0643a1725857dc6bf51dce5c39e78f34a0bfdc69d2de59be574396175f3a2ea5841e502bde75d082ddb30f15ee30a1a598037

data/CHANGELOG.md

@@ -7,6 +7,40 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ---
 
+## [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-21
 
 ### Bug Fixes
@@ -17,10 +51,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

@@ -82,6 +82,13 @@ 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.  Calling execute after close raises an error.
+          klass.define_method(:close) do
+            @mcp_transport.close
+          end
+
           klass
         end
       end

data/lib/phronomy/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Phronomy
-  VERSION = "0.5.1"
+  VERSION = "0.5.2"
 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.2
 platform: ruby
 authors:
 - Raizo T.C.S