phronomy 0.2.2 → 0.3.0

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: phronomy
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.3.0
 platform: ruby
 authors:
 - Raizo T.C.S
@@ -65,23 +65,19 @@ files:
 - CHANGELOG.md
 - README.md
 - Rakefile
-- docs/trustworthy_ai_enhancements.md
 - lib/generators/phronomy/install/install_generator.rb
 - lib/generators/phronomy/install/templates/create_phronomy_messages.rb.tt
 - lib/generators/phronomy/install/templates/initializer.rb.tt
 - lib/generators/phronomy/install/templates/message_model.rb.tt
 - lib/phronomy.rb
-- lib/phronomy/active_record/acts_as.rb
-- lib/phronomy/active_record/checkpoint.rb
-- lib/phronomy/active_record/extensions.rb
-- lib/phronomy/active_record/message.rb
-- lib/phronomy/actor.rb
 - lib/phronomy/agent.rb
 - lib/phronomy/agent/base.rb
 - lib/phronomy/agent/before_completion_context.rb
+- lib/phronomy/agent/checkpoint.rb
 - lib/phronomy/agent/handoff.rb
 - lib/phronomy/agent/react_agent.rb
 - lib/phronomy/agent/runner.rb
+- lib/phronomy/agent/suspend_signal.rb
 - lib/phronomy/configuration.rb
 - lib/phronomy/context.rb
 - lib/phronomy/context/assembler.rb
@@ -124,43 +120,18 @@ files:
 - lib/phronomy/loader/csv_loader.rb
 - lib/phronomy/loader/markdown_loader.rb
 - lib/phronomy/loader/plain_text_loader.rb
-- lib/phronomy/memory.rb
-- lib/phronomy/memory/compression.rb
-- lib/phronomy/memory/compression/base.rb
-- lib/phronomy/memory/compression/summary.rb
-- lib/phronomy/memory/compression/tool_output_pruner.rb
-- lib/phronomy/memory/conversation_manager.rb
-- lib/phronomy/memory/retrieval.rb
-- lib/phronomy/memory/retrieval/base.rb
-- lib/phronomy/memory/retrieval/composite.rb
-- lib/phronomy/memory/retrieval/recent.rb
-- lib/phronomy/memory/retrieval/semantic.rb
-- lib/phronomy/memory/storage.rb
-- lib/phronomy/memory/storage/active_record.rb
-- lib/phronomy/memory/storage/base.rb
-- lib/phronomy/memory/storage/in_memory.rb
 - lib/phronomy/output_parser.rb
 - lib/phronomy/output_parser/base.rb
 - lib/phronomy/output_parser/json_parser.rb
 - lib/phronomy/output_parser/structured_parser.rb
 - lib/phronomy/prompt_template.rb
-- lib/phronomy/rails/agent_job.rb
 - lib/phronomy/railtie.rb
+- lib/phronomy/ruby_llm_patches.rb
 - lib/phronomy/runnable.rb
 - lib/phronomy/splitter.rb
 - lib/phronomy/splitter/base.rb
 - lib/phronomy/splitter/fixed_size_splitter.rb
 - lib/phronomy/splitter/recursive_splitter.rb
-- lib/phronomy/state_store.rb
-- lib/phronomy/state_store/active_record.rb
-- lib/phronomy/state_store/base.rb
-- lib/phronomy/state_store/encryptor.rb
-- lib/phronomy/state_store/encryptor/active_support.rb
-- lib/phronomy/state_store/encryptor/base.rb
-- lib/phronomy/state_store/file.rb
-- lib/phronomy/state_store/in_memory.rb
-- lib/phronomy/state_store/redis.rb
-- lib/phronomy/thread_actor_registry.rb
 - lib/phronomy/token_usage.rb
 - lib/phronomy/tool.rb
 - lib/phronomy/tool/agent_tool.rb
@@ -181,6 +152,7 @@ files:
 - lib/phronomy/workflow.rb
 - lib/phronomy/workflow_context.rb
 - lib/phronomy/workflow_runner.rb
+- scripts/check_readme_ruby.rb
 - sig/phronomy.rbs
 homepage: https://github.com/Raizo-TCS/phronomy
 licenses:

data/docs/trustworthy_ai_enhancements.md

@@ -1,332 +0,0 @@
-# Trustworthy AI Enhancements
-
-Specification for features that address the NIST AI Risk Management Framework (AI RMF 1.0)
-trustworthiness characteristics, as applied to the phronomy gem.
-
-Reference: NIST AI 100-1 — https://doi.org/10.6028/NIST.AI.100-1
-Japanese translation: https://aisi.go.jp/assets/pdf/NIST_AI_RMF_jp_20240806.pdf
-
----
-
-## Responsibility Model
-
-Three layers share responsibility for trustworthy AI:
-
-```
-┌─────────────────────────────────────────┐
-│  Application  Domain logic / UX          │
-├─────────────────────────────────────────┤
-│  phronomy     Control flow / observation / boundary enforcement │
-├─────────────────────────────────────────┤
-│  LLM          Probabilistic reasoning / generation              │
-└─────────────────────────────────────────┘
-```
-
-Key principle: **the LLM is untrusted**. phronomy acts as the deterministic control
-layer that validates, constrains, and observes LLM behaviour. Characteristics that
-cannot be delegated to the LLM must be enforced by phronomy or the application layer.
-
----
-
-## Trustworthiness Characteristics — Status and Plan
-
-### 3.1 Valid and Reliable
-
-| Layer | Responsibility | Status |
-|---|---|---|
-| LLM | Base reasoning capability | Model-dependent |
-| **phronomy** | Eval infrastructure, output type validation | ✅ `Eval::Runner`, `Eval::Dataset`, `Eval::Metrics` — see `lib/phronomy/eval/` |
-| **phronomy** | Drift / accuracy monitoring hooks | ❌ Not implemented — **planned** |
-| Application | Test-case design, accuracy thresholds | Application responsibility |
-
-**Planned work:**
-- None in this iteration. `Eval` infrastructure is sufficient for current needs.
-
----
-
-### 3.2 Safe
-
-| Layer | Responsibility | Status |
-|---|---|---|
-| LLM | Basic harmful-content avoidance (RLHF) | Model-dependent, not guaranteed |
-| **phronomy** | Intervention points, iteration limits, approval gates | ✅ `wait_state`/`send_event`, `requires_approval`, `max_iterations` — see `lib/phronomy/workflow.rb`, `lib/phronomy/agent/base.rb` |
-| **phronomy** | Built-in guardrails (PII, prompt injection) | ❌ Not implemented — **planned (Feature A)** |
-| Application | Concrete guardrail logic, approval workflows | Application responsibility |
-
----
-
-### 3.3 Secure and Resilient
-
-| Layer | Responsibility | Status |
-|---|---|---|
-| LLM | Partial prompt-injection resistance | Model-dependent, partial |
-| **phronomy** | State persistence across process restarts | ✅ `StateStore::ActiveRecord` — see `lib/phronomy/state_store/` |
-| **phronomy** | Encrypted state store adapter interface | ❌ Not implemented — **planned (Feature C)** |
-| Application | Authentication / authorisation / infrastructure encryption | Application / infrastructure responsibility |
-
----
-
-### 3.4 Accountable and Transparent
-
-| Layer | Responsibility | Status |
-|---|---|---|
-| LLM | Token usage reporting | ✅ `TokenUsage` — see `lib/phronomy/token_usage.rb` |
-| **phronomy** | Tracing / span recording | ✅ `Tracing::LangfuseTracer`, `OpenTelemetryTracer` — see `lib/phronomy/tracing/` |
-| **phronomy** | Caller identity propagation to tracers | ❌ Not implemented — **planned (Feature B)** |
-| Application | User-facing AI disclosure, business audit requirements | Application responsibility |
-
----
-
-### 3.5 Explainable and Interpretable
-
-| Layer | Responsibility | Status |
-|---|---|---|
-| LLM | Chain-of-thought generation | Prompt-dependent |
-| **phronomy** | Processing step recording via Graph and Tracing | ✅ Partial — `Workflow`/`WorkflowRunner`, `Tracing` |
-| Application | Explanation UI, CoT prompt design | Application responsibility |
-
-**Planned work:** None in this iteration.
-
----
-
-### 3.6 Privacy-Enhanced
-
-| Layer | Responsibility | Status |
-|---|---|---|
-| LLM | Training data handling | Provider responsibility |
-| **phronomy** | Memory compression (data minimisation) | ✅ `Memory::Compression` — see `lib/phronomy/memory/compression/` |
-| **phronomy** | Built-in PII detection guardrail | ❌ Not implemented — **planned (Feature A)** |
-| **phronomy** | TTL and explicit purge API on ConversationManager | ❌ Not implemented — **planned (Feature D)** |
-| Application | Privacy policy, user consent management | Application responsibility |
-
----
-
-### 3.7 Fair — with Harmful Bias Managed
-
-| Layer | Responsibility | Status |
-|---|---|---|
-| LLM | Bias reduction via RLHF | Provider responsibility |
-| **phronomy** | Eval infrastructure for custom metrics | ✅ `Eval::Metrics` — extensible |
-| Application | Fairness test-set design, threshold definition | Application responsibility |
-
-**Planned work:** None in this iteration. The existing `Eval::Metrics` extension
-point is sufficient; fairness metrics are domain-specific and belong to the
-application layer.
-
----
-
-## Planned Features (This Branch)
-
-### Feature A — `Phronomy::Guardrail::Builtin` module
-
-**Addresses:** 3.2 Safe, 3.6 Privacy-Enhanced
-
-**Motivation:**
-Prompt injection and PII leakage are the two most common, high-severity risks for
-any LLM application. They require deterministic, regex/heuristic-based detection
-that the LLM cannot reliably provide. phronomy should ship sensible defaults so
-that applications do not have to re-implement these from scratch.
-
-**Design:**
-- New module: `Phronomy::Guardrail::Builtin`
-- Two concrete classes, both under `lib/phronomy/guardrail/builtin/`:
-  - `PromptInjectionDetector < InputGuardrail`
-  - `PIIPatternDetector < InputGuardrail`
-- Existing base classes (`InputGuardrail`, `OutputGuardrail`) are unchanged — see
-  `lib/phronomy/guardrail/input_guardrail.rb` and `output_guardrail.rb`.
-
-**`PromptInjectionDetector`:**
-- Detects common prompt-injection patterns in input strings:
-  - "ignore previous instructions", "disregard all prior", "system prompt:" prefixes,
-    jailbreak keywords, role-switch attempts.
-- Pattern list is configurable via constructor argument `additional_patterns: []`.
-- Raises `GuardrailError` with message `"Potential prompt injection detected"`.
-
-**`PIIPatternDetector`:**
-- Detects common Japanese and international PII patterns:
-  - Japanese My Number (12-digit number): `/\b\d{4}[- ]?\d{4}[- ]?\d{4}\b/`
-  - Credit card numbers: `/\b(?:\d{4}[- ]?){3}\d{4}\b/`
-  - Email addresses: standard RFC 5322 simplified pattern
-  - Phone numbers (JP): `/\b0\d{1,4}[- ]?\d{1,4}[- ]?\d{4}\b/`
-- Each pattern category is independently togglable via constructor:
-  `PIIPatternDetector.new(detect: [:my_number, :credit_card, :email, :phone])`
-- Default: all four categories active.
-- Raises `GuardrailError` with message `"PII detected in input: <category>"`.
-
-**Usage example:**
-```ruby
-agent = MyAgent.new
-agent.add_input_guardrail(Phronomy::Guardrail::Builtin::PromptInjectionDetector.new)
-agent.add_input_guardrail(Phronomy::Guardrail::Builtin::PIIPatternDetector.new(detect: [:my_number, :credit_card]))
-```
-
-**Files to create:**
-- `lib/phronomy/guardrail/builtin/prompt_injection_detector.rb`
-- `lib/phronomy/guardrail/builtin/pii_pattern_detector.rb`
-- `lib/phronomy/guardrail/builtin.rb` (requires both, defines module)
-- Update `lib/phronomy/guardrail.rb` to require `builtin`
-
-**Tests:**
-- Unit: `spec/phronomy/guardrail/builtin/prompt_injection_detector_spec.rb`
-- Unit: `spec/phronomy/guardrail/builtin/pii_pattern_detector_spec.rb`
-- Integration: extend `spec/integration/tool_guardrail_spec.rb` with builtin guardrail factors
-
----
-
-### Feature B — Caller identity propagation in `config:`
-
-**Addresses:** 3.4 Accountable and Transparent
-
-**Motivation:**
-`Tracing` already records what happened (spans, token usage). What is missing is
-**who** triggered the action. Without a caller identity, audit logs cannot be
-attributed to users or sessions, which is a requirement for accountability under
-NIST AI RMF 3.4.
-
-**Design:**
-- `Agent::Base#invoke` and `WorkflowRunner#invoke` already accept `config: {}` — see
-  `lib/phronomy/agent/base.rb` and `lib/phronomy/graph/workflow_runner.rb`.
-- Add two new optional keys to `config:`:
-  - `user_id:` (String | nil) — caller identity
-  - `session_id:` (String | nil) — session / request identity
-- Both are extracted in `invoke_once` / `call` and forwarded to
-  `Tracing::Base#start_span` as span attributes.
-- `Tracing::Base#start_span` already accepts `**attributes` — no signature change needed.
-- `LangfuseTracer` and `OpenTelemetryTracer` will automatically forward them as
-  metadata/attributes respectively.
-
-**Usage example:**
-```ruby
-agent.invoke("What is the weather?", config: {
-  thread_id: "conv-123",
-  user_id:   "user-42",
-  session_id: "sess-abc"
-})
-```
-
-**Files to modify:**
-- `lib/phronomy/agent/base.rb` — extract `user_id` and `session_id` from config, pass to tracer
-- `lib/phronomy/graph/compiled_graph.rb` — same for graph invocations
-
-**Tests:**
-- Unit: extend `spec/phronomy/agent_spec.rb` with `user_id`/`session_id` forwarding assertions
-- Unit: extend `spec/phronomy/tracing/langfuse_tracer_spec.rb` with attribute forwarding
-
----
-
-### Feature C — `StateStore` encryption adapter interface
-
-**Addresses:** 3.3 Secure and Resilient
-
-**Motivation:**
-`StateStore::ActiveRecord` persists conversation state as plain-text JSON. In
-regulated environments (healthcare, finance, government) this violates data-at-rest
-requirements. phronomy should define a standard interface so that an encryption
-adapter can be layered transparently without modifying `StateStore::ActiveRecord`.
-
-**Design:**
-- New abstract class: `Phronomy::StateStore::Encryptor::Base`
-  - `encrypt(plaintext) → ciphertext` (abstract)
-  - `decrypt(ciphertext) → plaintext` (abstract)
-- New concrete class: `Phronomy::StateStore::Encryptor::ActiveSupport`
-  - Delegates to `ActiveSupport::MessageEncryptor` when available.
-  - Constructor: `ActiveSupport.new(secret_key_base:, cipher: "aes-256-gcm")`
-- `StateStore::ActiveRecord` accepts an optional `encryptor:` constructor argument:
-  - When present, `serialize_state` output is passed through `encryptor.encrypt`
-    before writing to the DB, and `encryptor.decrypt` before `deserialize_state`.
-  - When absent, behaviour is unchanged (backwards compatible).
-
-**Usage example:**
-```ruby
-encryptor = Phronomy::StateStore::Encryptor::ActiveSupport.new(
-  secret_key_base: ENV.fetch("SECRET_KEY_BASE")
-)
-store = Phronomy::StateStore::ActiveRecord.new(
-  model_class: PhronomyStateRecord,
-  encryptor: encryptor
-)
-```
-
-**Files to create:**
-- `lib/phronomy/state_store/encryptor/base.rb`
-- `lib/phronomy/state_store/encryptor/active_support.rb`
-- `lib/phronomy/state_store/encryptor.rb`
-
-**Files to modify:**
-- `lib/phronomy/state_store/active_record.rb` — accept `encryptor:`, apply in save/load
-- `lib/phronomy/state_store.rb` — require `encryptor`
-
-**Tests:**
-- Unit: `spec/phronomy/state_store/encryptor/base_spec.rb`
-- Unit: `spec/phronomy/state_store/encryptor/active_support_spec.rb`
-- Unit: extend `spec/phronomy/state_store_spec.rb` with encrypted save/load round-trip
-
----
-
-### Feature D — TTL and `purge` API on `ConversationManager`
-
-**Addresses:** 3.6 Privacy-Enhanced
-
-**Motivation:**
-Users have a right to be forgotten. `ConversationManager` currently has no way to
-delete stored messages for a given thread, nor does it enforce data retention limits.
-
-**Design:**
-- `ConversationManager#purge(thread_id:)` — deletes all stored messages for the
-  thread from both the storage backend and the retrieval index.
-- Optional `ttl:` constructor argument (Integer seconds | nil):
-  - When set, messages older than `ttl` seconds are filtered out on `load_messages`.
-  - Storage backends that support native TTL (e.g. Redis) should be informed via
-    a `Storage::Base#purge_older_than(thread_id:, older_than:)` hook.
-  - Default: `nil` (no expiry — current behaviour unchanged).
-- `Storage::ActiveRecord` gains a `purge_older_than` implementation using
-  `where("created_at < ?", Time.now - ttl).destroy_all`.
-
-**Usage example:**
-```ruby
-memory = Phronomy::Memory::ConversationManager.new(
-  storage: Phronomy::Memory::Storage::ActiveRecord.new(model_class: PhronomyMessageRecord),
-  ttl: 60 * 60 * 24 * 30  # 30 days
-)
-# Later:
-memory.purge(thread_id: "conv-123")
-```
-
-**Files to modify:**
-- `lib/phronomy/memory/conversation_manager.rb` — add `purge`, accept `ttl:`
-- `lib/phronomy/memory/storage/base.rb` — add `purge(thread_id:)` abstract method, `purge_older_than` hook
-- `lib/phronomy/memory/storage/active_record.rb` — implement both
-- `lib/phronomy/memory/storage/in_memory.rb` — implement both
-
-**Tests:**
-- Unit: extend `spec/phronomy/memory_spec.rb` with `purge` and TTL filtering tests
-- Unit: extend `spec/phronomy/active_record/message_spec.rb` with `purge_older_than`
-
----
-
-## Implementation Order
-
-| Step | Feature | Rationale |
-|---|---|---|
-| 1 | Feature A — BuiltinGuardrails | Self-contained, no dependencies, highest safety impact |
-| 2 | Feature B — Caller identity | Small change, high accountability value |
-| 3 | Feature C — Encryptor I/F | More complex, depends on no other feature |
-| 4 | Feature D — TTL / purge | Touches storage layer, do last to avoid churn |
-
-Each feature follows the same workflow:
-1. Implement source files
-2. Run StandardRB on new files
-3. Run unit tests: `bundle exec rspec <spec_file>`
-4. Run full unit suite: `bundle exec rspec --format progress`
-5. Run integration suite: `bundle exec rspec --tag integration --format progress`
-6. Present diff for commit approval
-
----
-
-## Out of Scope (This Branch)
-
-- Fairness metrics / demographic parity (`Eval::Metrics` extension) — domain-specific,
-  belongs to application layer
-- Kill switch / forced shutdown — infrastructure concern
-- Differential privacy — academic/research topic, not yet practical for gem scope
-- Authentication / authorisation — application / infrastructure concern

data/lib/phronomy/active_record/acts_as.rb

@@ -1,48 +0,0 @@
-# frozen_string_literal: true
-
-module Phronomy
-  module ActiveRecord
-    # DSL mixin that can be included in ApplicationRecord subclasses
-    # to declaratively associate them with Phronomy persistence roles.
-    #
-    # @example
-    #   class PhronomyMessage < ApplicationRecord
-    #     acts_as_phronomy_message
-    #   end
-    module ActsAs
-      def self.included(base)
-        base.extend(ClassMethods)
-      end
-
-      module ClassMethods
-        # Configures this model as a Phronomy checkpoint store.
-        # Applies validations and exposes a convenience checkpointer factory.
-        #
-        # @param encryptor [StateStore::Encryptor::Base, nil] optional encryptor
-        #   for encrypting +state_json+ at rest.
-        def acts_as_phronomy_checkpoint(encryptor: nil)
-          include ::Phronomy::ActiveRecord::Checkpoint
-
-          define_singleton_method(:phronomy_checkpointer) do |enc: encryptor|
-            ::Phronomy::StateStore::ActiveRecord.new(model_class: self, encryptor: enc)
-          end
-        end
-
-        # Configures this model as a Phronomy message store.
-        # Applies validations and exposes a convenience factory method.
-        #
-        # @return [Phronomy::Memory::ConversationManager] a ready-to-use memory object
-        def acts_as_phronomy_message
-          include ::Phronomy::ActiveRecord::Message
-
-          define_singleton_method(:phronomy_memory) do
-            ::Phronomy::Memory::ConversationManager.new(
-              storage: ::Phronomy::Memory::Storage::ActiveRecord.new(model_class: self),
-              retrieval: ::Phronomy::Memory::Retrieval::Recent.new
-            )
-          end
-        end
-      end
-    end
-  end
-end

data/lib/phronomy/active_record/checkpoint.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-module Phronomy
-  module ActiveRecord
-    # Mixin for models backed by the phronomy_checkpoints table.
-    # Validates the required columns and exposes a convenience factory.
-    #
-    # @example
-    #   class PhronomyCheckpoint < ApplicationRecord
-    #     include Phronomy::ActiveRecord::ActsAs
-    #     acts_as_phronomy_checkpoint
-    #   end
-    module Checkpoint
-      def self.included(base)
-        base.validates :thread_id, presence: true
-        base.validates :state_json, presence: true
-      end
-    end
-  end
-end

data/lib/phronomy/active_record/extensions.rb

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-module Phronomy
-  module ActiveRecord
-    # Convenience namespace loaded by Railtie when ActiveRecord is available.
-    # Ensures all Phronomy::ActiveRecord mixins are eager-loaded in Rails context.
-    module Extensions
-    end
-  end
-end
-
-require_relative "checkpoint"
-require_relative "message"
-require_relative "acts_as"

data/lib/phronomy/active_record/message.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-module Phronomy
-  module ActiveRecord
-    # Mixin for models backed by the phronomy_messages table.
-    #
-    # @example
-    #   class PhronomyMessage < ApplicationRecord
-    #     include Phronomy::ActiveRecord::Message
-    #   end
-    module Message
-      def self.included(base)
-        base.validates :thread_id, presence: true
-        base.validates :role, presence: true
-        # content may be blank for assistant messages that carry only tool calls
-        base.validates :content, presence: false, allow_nil: true
-      end
-    end
-  end
-end

data/lib/phronomy/actor.rb

@@ -1,68 +0,0 @@
-# frozen_string_literal: true
-
-module Phronomy
-  # Lightweight synchronous actor backed by a dedicated +Thread+ and a +Queue+.
-  #
-  # A caller submits work via {#call}, which blocks until the actor's thread
-  # finishes executing the block and then returns the result (or re-raises any
-  # exception that occurred inside the actor).
-  #
-  # === Reentrant safety
-  #
-  # If {#call} is invoked from within the actor's own thread (i.e. from inside
-  # a block that is already executing on this actor), the block is executed
-  # directly in the current thread instead of being pushed onto the queue.
-  # This prevents deadlocks in deeply nested call paths without requiring
-  # callers to track whether they are already "inside" the actor.
-  #
-  # === Usage
-  #
-  #   actor = Phronomy::Actor.new
-  #   result = actor.call { expensive_operation() }   # blocks caller; runs on actor thread
-  #   actor.stop                                       # graceful shutdown
-  class Actor
-    def initialize
-      @queue = Queue.new
-      @thread = Thread.new do
-        loop do
-          task = @queue.pop
-          break if task == :stop
-          task.call
-        end
-      end
-    end
-
-    # Run +block+ on the actor's thread and return its result.
-    #
-    # If the current thread is already the actor's thread (reentrant call),
-    # the block is executed inline to prevent deadlocks.
-    #
-    # Any exception raised inside the block is captured and re-raised in the
-    # calling thread.
-    #
-    # @yield block to execute on the actor's thread
-    # @return the return value of the block
-    def call(&block)
-      return block.call if Thread.current == @thread
-
-      done = Queue.new
-      @queue.push(-> {
-        begin
-          done.push([true, block.call])
-        rescue => e
-          done.push([false, e])
-        end
-      })
-      success, value = done.pop
-      raise value unless success
-      value
-    end
-
-    # Send a +:stop+ sentinel to gracefully terminate the actor's thread.
-    # Pending tasks already in the queue will still be processed before
-    # the thread exits.
-    def stop
-      @queue.push(:stop)
-    end
-  end
-end

data/lib/phronomy/memory/compression/base.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-module Phronomy
-  module Memory
-    module Compression
-      # Abstract base class for compression strategies.
-      #
-      # Two kinds of compression exist:
-      #
-      # * Content pruning (e.g. ToolOutputPruner) — modifies individual message
-      #   content in-place (e.g. truncates oversized tool outputs). The original
-      #   data loss is limited and intentional (tool outputs are auxiliary).
-      #   These subclasses return { messages: Array, compaction: nil }.
-      #
-      # * Compaction (e.g. Summary) — replaces multiple messages with an LLM
-      #   summary. Originals are preserved in Storage via a compaction record.
-      #   These subclasses return { messages: Array, compaction: Hash } where
-      #   compaction is { start_seq:, end_seq:, summary_text: }.
-      #
-      # ConversationManager inspects the :compaction key and persists the record
-      # in Storage when present.
-      #
-      # @abstract Subclass and implement #compress.
-      class Base
-        # Compress a message array and return a result hash.
-        #
-        # @param thread_id  [String]  thread identifier (used by stateful compressors)
-        # @param messages   [Array]   message history to compress
-        # @param seq_offset [Integer] seq number of messages[0] in the raw history
-        # @return [Hash] { messages: Array, compaction: Hash|nil }
-        def compress(thread_id:, messages:, seq_offset: 0)
-          raise NotImplementedError, "#{self.class}#compress is not implemented"
-        end
-      end
-    end
-  end
-end