phronomy 0.33.0 → 0.34.0

data/lib/phronomy/context/context_policy.rb

@@ -35,7 +35,7 @@ module Phronomy
           role: role,
           content: content,
           content_format: content_format || inferred_content_format(content),
-          estimated_tokens: estimate_content(content),
+          estimated_tokens: estimate_tokens(content),
           required: required,
           provenance: generated_provenance,
           metadata: metadata
@@ -56,7 +56,7 @@ module Phronomy
           role: role,
           content: content,
           content_format: content_format || inferred_content_format(content),
-          estimated_tokens: estimate_content(content),
+          estimated_tokens: estimate_tokens(content),
           required: required,
           provenance: generated_provenance,
           metadata: metadata
@@ -82,7 +82,7 @@ module Phronomy
           content: content,
           content_format: content_format || inferred_content_format(content),
           sequence: sequence,
-          estimated_tokens: estimate_content(content),
+          estimated_tokens: estimate_tokens(content),
           required: required,
           provenance: generated_provenance,
           tool_call_id: tool_call_id,
@@ -121,10 +121,6 @@ module Phronomy
       def inferred_content_format(content)
         content.is_a?(String) ? :text : :json
       end
-
-      def estimate_content(content)
-        estimate_tokens(content)
-      end
     end
   end
 end

data/lib/phronomy/context/llm_input_manifest.rb

@@ -149,9 +149,9 @@ module Phronomy
         freeze
       end
 
-      # Current-format-only durable decoder. Historical format conversion is an
-      # explicit migration operation and is never attempted here. Unlike the
-      # Ruby construction API, this decoder never coerces durable field types.
+      # Current-format-only durable decoder. Historical versions are rejected.
+      # Unlike the Ruby construction API, this decoder never coerces durable
+      # field types.
       def self.from_h(hash)
         source = strict_source!(
           hash,

data/lib/phronomy/engine/runtime/shutdown_result.rb

@@ -52,11 +52,6 @@ module Phronomy
           %i[not_started terminated].include?(@event_loop_status) &&
           @task_registry_status == :empty
       end
-
-      # Runtime reset is safe when cleanup completed, even if execution failed.
-      def success?
-        cleanup_complete?
-      end
     end
   end
 end

data/lib/phronomy/execution/execution.rb

@@ -127,22 +127,11 @@ module Phronomy
       observed
     end
 
-    # Scope/admission hooks shared with framework-owned operations.
-    # @api private
-    def __while_open(&block)
-      @scope.__while_open(&block)
-    end
-
     # @api private
     def __open?
       @scope.__open?
     end
 
-    # @api private
-    def __cancellation_token
-      @token
-    end
-
     # @api private
     def __cancellation_error
       @scope.__cancellation_error

data/lib/phronomy/multi_agent/shared_state.rb

@@ -323,14 +323,11 @@ module Phronomy
         # application's definition-version namespace independent from framework
         # instrumentation.
         instrumented_def = instrumented_definition_for(researcher_class)
-        Class.new(researcher_class) do
-          model researcher_class.model
-          agent_definition(
-            id: instrumented_def.fetch(:id),
-            version: instrumented_def.fetch(:version)
-          )
-          tools(definitions)
-        end
+        researcher_class.with_tools(
+          definitions,
+          id: instrumented_def.fetch(:id),
+          version: instrumented_def.fetch(:version)
+        )
       end
 
       def build_read_tool(store)

data/lib/phronomy/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: phronomy
 version: !ruby/object:Gem::Version
-  version: 0.33.0
+  version: 0.34.0
 platform: ruby
 authors:
 - Raizo T.C.S
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-11-14 00:00:00.000000000 Z
+date: 2026-11-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby_llm
@@ -345,8 +345,10 @@ files:
 - docs/migrations/0.19.md
 - docs/migrations/0.22.md
 - docs/migrations/0.31.md
+- docs/migrations/cleanup-audit.md
 - docs/migrations/durable-semantic-coordination-v2.md
 - docs/migrations/handoff-runner-multi-agent.md
+- docs/migrations/legacy-format-removal.md
 - docs/migrations/neutral-storage-spi.md
 - docs/migrations/parallel-tool-chat-removal.md
 - docs/migrations/persistence-failure-contracts.md
@@ -561,7 +563,6 @@ files:
 - lib/phronomy/execution/runnable.rb
 - lib/phronomy/execution/runtime_shutdown_error.rb
 - lib/phronomy/execution/runtime_shutdown_reentrancy_error.rb
-- lib/phronomy/execution/scheduler_reentrancy_error.rb
 - lib/phronomy/execution/task_result.rb
 - lib/phronomy/execution/timeout_error.rb
 - lib/phronomy/execution/wait_policy.rb
@@ -636,7 +637,6 @@ files:
 - lib/phronomy/output_parser/structured_parser.rb
 - lib/phronomy/persistence/content_repository.rb
 - lib/phronomy/persistence/errors.rb
-- lib/phronomy/persistence/migration/initial_format_migration.rb
 - lib/phronomy/persistence/persistence.rb
 - lib/phronomy/persistence/save_outcome.rb
 - lib/phronomy/persistence/snapshot_comparison.rb
@@ -749,16 +749,10 @@ files:
 - lib/phronomy/workflow/runtime_binding/workflow_phase_machine_builder.rb
 - lib/phronomy/workflow/runtime_binding/workflow_task_session.rb
 - lib/phronomy/workflow/runtime_binding/workflow_terminal_policy.rb
-- scripts/add_to_h_to_token_doubles.rb
-- scripts/add_to_h_unnamed_doubles.rb
 - scripts/api_snapshot.rb
 - scripts/check_api_annotations.rb
 - scripts/check_readme_ruby.rb
 - scripts/check_readme_runnable.rb
-- scripts/migrate_generator_verifier_result.rb
-- scripts/migrate_spec_agent_definition.rb
-- scripts/migrate_spec_agent_definition_pass2.rb
-- scripts/migrate_spec_inline_pass3.rb
 - scripts/run_mutation.sh
 - scripts/storage_spi_snapshot.rb
 - sig/_private/phronomy/agent/settings.rbs
@@ -820,7 +814,6 @@ files:
 - sig/phronomy/multi_agent/team_not_found_error.rbs
 - sig/phronomy/persistence.rbs
 - sig/phronomy/persistence/errors.rbs
-- sig/phronomy/persistence/migration.rbs
 - sig/phronomy/persistence/transaction.rbs
 - sig/phronomy/persistence_composition.rbs
 - sig/phronomy/runtime_composition/configuration.rbs

data/lib/phronomy/execution/scheduler_reentrancy_error.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "event_loop_reentrancy_error"
-
-module Phronomy
-  # Backward-compatible error class name for callers that still rescue the old
-  # scheduler-oriented exception. New code should use EventLoopReentrancyError.
-  class SchedulerReentrancyError < EventLoopReentrancyError; end
-end

data/lib/phronomy/persistence/migration/initial_format_migration.rb

@@ -1,226 +0,0 @@
-# frozen_string_literal: true
-
-module Phronomy
-  class Persistence
-    module Migration
-      # Explicit logical conversion from the pre-S3 unversioned representations
-      # present at the ACS-06 fixed base to the initial 0.1 durable formats.
-      #
-      # This module does not scan a database or write records. A backend-specific
-      # migration runner is responsible for physical enumeration, transactions,
-      # and replacement. Normal Runtime load never calls these methods.
-      #
-      # @api public
-      module InitialFormatMigration
-        PRE_S3_AGENT_ROOT_KEYS = Phronomy::Agent::AgentRoot::ATTRIBUTES.map(&:to_s).freeze
-        PRE_S3_AGENT_EXECUTION_KEYS = Phronomy::Agent::AgentExecution::ATTRIBUTES.map(&:to_s).freeze
-        PRE_S3_JOURNAL_KEYS = Phronomy::Agent::JournalRecord::ATTRIBUTES.map(&:to_s).freeze
-        PRE_S3_LLM_CALL_KEYS = Phronomy::Agent::LLMCallRecord::ATTRIBUTES.map(&:to_s).freeze
-        PRE_S3_APPROVAL_REQUIRED_KEYS = %w[id items created_at].freeze
-        PRE_S3_APPROVAL_PARENT_KEYS = %w[execution_id agent_invocation_id].freeze
-        PRE_S3_APPROVAL_OPTIONAL_KEYS = %w[approved].freeze
-        PRE_S3_APPROVAL_ITEM_KEYS = %w[
-          tool_invocation_id tool_call_id tool_name arguments facts reason origin metadata
-        ].freeze
-
-        module_function
-
-        def agent_root(hash)
-          source = stringify_keys(hash)
-          validate_exact_keys!(
-            source,
-            PRE_S3_AGENT_ROOT_KEYS,
-            label: "pre-S3 AgentRoot"
-          )
-          Phronomy::Agent::Persistence::Codec.encode_agent_root(
-            Phronomy::Agent::AgentRoot.from_h(source)
-          )
-        rescue Phronomy::Storage::SerializationError
-          raise
-        rescue => error
-          migration_error("AgentRoot", error)
-        end
-
-        def agent_execution(hash)
-          source = stringify_keys(hash)
-          validate_exact_keys!(
-            source,
-            PRE_S3_AGENT_EXECUTION_KEYS,
-            label: "pre-S3 AgentExecution"
-          )
-          source["working_records"] = Array(source.fetch("working_records")).map.with_index do |record, index|
-            normalize_legacy_journal_record(
-              record,
-              label: "pre-S3 AgentExecution working_records[#{index}]"
-            )
-          end
-          source["llm_calls"] = Array(source.fetch("llm_calls")).map.with_index do |call, index|
-            normalized = stringify_keys(call)
-            validate_exact_keys!(
-              normalized,
-              PRE_S3_LLM_CALL_KEYS,
-              label: "pre-S3 AgentExecution llm_calls[#{index}]"
-            )
-            normalized
-          end
-          source["approval_request"] = normalize_legacy_approval_request(
-            source["approval_request"],
-            execution_id: source.fetch("execution_id")
-          )
-          execution = Phronomy::Agent::AgentExecution.from_h(source)
-          Phronomy::Agent::Persistence::Codec.encode_agent_execution(execution)
-        rescue Phronomy::Storage::SerializationError
-          raise
-        rescue => error
-          migration_error("AgentExecution", error)
-        end
-
-        def journal_record(hash)
-          source = normalize_legacy_journal_record(hash, label: "pre-S3 JournalRecord")
-          Phronomy::Agent::Persistence::Codec.encode_journal_record(
-            Phronomy::Agent::JournalRecord.from_h(source)
-          )
-        rescue Phronomy::Storage::SerializationError
-          raise
-        rescue => error
-          migration_error("JournalRecord", error)
-        end
-
-        def workflow_state(workflow_instance_id:, revision:, snapshot:)
-          Phronomy::Workflow::Persistence::Codec.encode_workflow_state(
-            workflow_instance_id: workflow_instance_id,
-            workflow_revision: Integer(revision),
-            snapshot: snapshot
-          )
-        rescue Phronomy::Storage::SerializationError
-          raise
-        rescue => error
-          migration_error("Workflow state", error)
-        end
-
-        # LLMInputManifest is a ContentStore codec boundary, not a
-        # Storage::DurableRecord. Its pre-CG-07 integer version 1 is migrated
-        # explicitly to the pre-1.0 string version "0.1".
-        def llm_input_manifest(hash)
-          source = stringify_keys(hash)
-          old_version = source.fetch("version")
-          unless old_version == 1 || old_version == "1"
-            raise Phronomy::Storage::SerializationError,
-              "unsupported pre-S3 LLMInputManifest version: #{old_version.inspect}"
-          end
-          source["version"] = Phronomy::Context::LLMInputManifest::VERSION
-          Phronomy::Context::LLMInputManifest.from_h(source).to_h
-        rescue Phronomy::Storage::SerializationError
-          raise
-        rescue => error
-          migration_error("LLMInputManifest", error)
-        end
-
-        def normalize_legacy_journal_record(hash, label:)
-          source = stringify_keys(hash)
-          validate_allowed_keys!(
-            source,
-            required_keys: PRE_S3_JOURNAL_KEYS,
-            optional_keys: ["correlation_id"],
-            label: label
-          )
-          source.delete("correlation_id")
-          source
-        end
-        private_class_method :normalize_legacy_journal_record
-
-        def normalize_legacy_approval_request(request, execution_id:)
-          return nil unless request
-
-          source = stringify_keys(request)
-          validate_allowed_keys!(
-            source,
-            required_keys: PRE_S3_APPROVAL_REQUIRED_KEYS,
-            optional_keys: PRE_S3_APPROVAL_PARENT_KEYS + PRE_S3_APPROVAL_OPTIONAL_KEYS,
-            label: "pre-S3 approval_request"
-          )
-          parents = PRE_S3_APPROVAL_PARENT_KEYS.select { |key| source.key?(key) }
-          unless parents.length == 1
-            raise Phronomy::Storage::SerializationError,
-              "pre-S3 approval_request must contain exactly one parent identity, got #{parents.inspect}"
-          end
-
-          items = source.fetch("items")
-          unless items.is_a?(Array) && !items.empty?
-            raise Phronomy::Storage::SerializationError,
-              "pre-S3 approval_request items must be a non-empty Array"
-          end
-          source["items"] = items.map.with_index do |item, index|
-            normalized = stringify_keys(item)
-            validate_exact_keys!(
-              normalized,
-              PRE_S3_APPROVAL_ITEM_KEYS,
-              label: "pre-S3 approval_request items[#{index}]"
-            )
-            normalized
-          end
-
-          if source.key?("agent_invocation_id")
-            source.delete("agent_invocation_id")
-            source["execution_id"] = execution_id.to_s
-          elsif source.fetch("execution_id").to_s != execution_id.to_s
-            raise Phronomy::Storage::SerializationError,
-              "pre-S3 approval_request execution_id does not match AgentExecution"
-          end
-          source
-        end
-        private_class_method :normalize_legacy_approval_request
-
-        def validate_allowed_keys!(hash, required_keys:, optional_keys:, label:)
-          actual = hash.keys.sort
-          missing = required_keys.sort - actual
-          unknown = actual - (required_keys + optional_keys).sort
-          return hash if missing.empty? && unknown.empty?
-
-          details = []
-          details << "missing=#{missing.inspect}" unless missing.empty?
-          details << "unknown=#{unknown.inspect}" unless unknown.empty?
-          raise Phronomy::Storage::SerializationError,
-            "#{label} schema mismatch (#{details.join(", ")})"
-        end
-        private_class_method :validate_allowed_keys!
-
-        def validate_exact_keys!(hash, expected_keys, label:)
-          validate_allowed_keys!(
-            hash,
-            required_keys: expected_keys,
-            optional_keys: [],
-            label: label
-          )
-        end
-        private_class_method :validate_exact_keys!
-
-        def stringify_keys(hash)
-          unless hash.is_a?(Hash)
-            raise Phronomy::Storage::SerializationError,
-              "migration input must be a Hash"
-          end
-          hash.each_with_object({}) do |(key, value), result|
-            unless key.is_a?(String) || key.is_a?(Symbol)
-              raise Phronomy::Storage::SerializationError,
-                "migration key must be String or Symbol, got #{key.class}"
-            end
-            string_key = key.to_s
-            if result.key?(string_key)
-              raise Phronomy::Storage::SerializationError,
-                "duplicate migration key after normalization: #{string_key.inspect}"
-            end
-            result[string_key] = value
-          end
-        end
-        private_class_method :stringify_keys
-
-        def migration_error(label, error)
-          raise Phronomy::Storage::SerializationError,
-            "cannot migrate pre-S3 #{label}: #{error.class}: #{error.message}"
-        end
-        private_class_method :migration_error
-      end
-    end
-  end
-end

data/scripts/add_to_h_to_token_doubles.rb

@@ -1,33 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# Adds to_h to all Tokens-like doubles so ExecutionCoordinator can serialize usage.
-
-files = Dir.glob("spec/**/*.rb").sort
-
-total = 0
-files.each do |f|
-  content = File.read(f)
-  modified = content.gsub(
-    /double\(("[^"]+"),\s*(input: \d+, output: \d+, cached: \d+, cache_creation: \d+)\)/
-  ) do
-    name = $1
-    attrs = $2
-    next $& if $&.include?("to_h")
-
-    h_entries = attrs.split(",").map(&:strip).map do |pair|
-      k, v = pair.split(":").map(&:strip)
-      "\"#{k}\" => #{v}"
-    end.join(", ")
-
-    total += 1
-    "double(#{name}, #{attrs}, to_h: {#{h_entries}})"
-  end
-
-  if modified != content
-    File.write(f, modified)
-    puts "  patched: #{f}"
-  end
-end
-
-puts "\nAdded to_h to #{total} Tokens doubles."

data/scripts/add_to_h_unnamed_doubles.rb

@@ -1,27 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# Adds to_h to unnamed (no-string-name) Tokens doubles like double(input: 5, ...)
-
-files = Dir.glob("spec/**/*.rb").sort
-
-total = 0
-files.each do |f|
-  content = File.read(f)
-  modified = content.gsub(
-    /double\((input: (\d+), output: (\d+), cached: (\d+), cache_creation: (\d+))\)/
-  ) do
-    attrs, a, b, c, d = $1, $2, $3, $4, $5
-    next $& if $&.include?("to_h")
-
-    total += 1
-    "double(#{attrs}, to_h: {\"input\" => #{a}, \"output\" => #{b}, \"cached\" => #{c}, \"cache_creation\" => #{d}})"
-  end
-
-  if modified != content
-    File.write(f, modified)
-    puts "  patched: #{f}"
-  end
-end
-
-puts "\nAdded to_h to #{total} unnamed Tokens doubles."

data/scripts/migrate_generator_verifier_result.rb

@@ -1,18 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# Explicit one-time conversion of a pre-structured-output Result JSON object.
-# Never infer old formats in the runtime reader; never overwrite the source.
-require_relative "../lib/phronomy"
-
-abort "Usage: ruby scripts/migrate_generator_verifier_result.rb OLD.json NEW.json" unless ARGV.length == 2
-source, destination = ARGV
-value = JSON.parse(File.read(source), allow_duplicate_key: false, allow_nan: false,
-  allow_comments: false, allow_invalid_escape: false, allow_control_characters: false)
-abort "Expected an old Result object without failure" unless value.is_a?(Hash) && !value.key?("failure")
-value["failure"] = nil
-result = Phronomy::GeneratorVerifier::Result.from_h(value)
-File.open(destination, File::WRONLY | File::CREAT | File::EXCL) do |file|
-  file.write(JSON.pretty_generate(result.to_h))
-  file.write("\n")
-end

data/scripts/migrate_spec_agent_definition.rb

@@ -1,108 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# Migrates spec files to add agent_definition to Agent::Base subclasses.
-# - Named classes: inserts after the class declaration line
-# - Anonymous blocks (do...end): inserts after the opening `do`
-# - Inline anonymous ({ ... }): expands to multi-line with agent_definition
-# Safe: skips classes that already have agent_definition
-
-require "fileutils"
-
-SPEC_DIR = File.expand_path("../spec", __dir__)
-AGENT_BASE_PATTERN = /Phronomy::Agent::Base/
-
-def snake_case(class_name)
-  # Remove trailing Agent/Tool suffixes, snake_case the name
-  name = class_name.split("::").last
-  name.gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
-    .gsub(/([a-z\d])([A-Z])/, '\1_\2')
-    .downcase
-end
-
-counter = 0
-files_modified = 0
-
-Dir.glob("#{SPEC_DIR}/**/*.rb").sort.each do |file|
-  lines = File.readlines(file, chomp: false)
-  new_lines = []
-  modified = false
-  i = 0
-
-  while i < lines.length
-    line = lines[i]
-
-    # ── Named class: class FooAgent < Phronomy::Agent::Base ──────────────────
-    if line =~ /^(\s*)class\s+(\w+)\s*<\s*(?:::)?Phronomy::Agent::Base\s*$/
-      indent = $1
-      class_name = $2
-      id = snake_case(class_name).sub(/_agent$/, "-agent").tr("_", "-")
-      id = "#{id}-agent" unless id.end_with?("-agent")
-      new_lines << line
-      i += 1
-      # Check if agent_definition already present in the class body
-      look_ahead = lines[i..]&.take(10) || []
-      unless look_ahead.any? { |l| l =~ /agent_definition/ }
-        new_lines << "#{indent}  agent_definition id: #{id.inspect}, version: 1\n"
-        counter += 1
-        modified = true
-      end
-      next
-    end
-
-    # ── Anonymous block (do...end): Class.new(Phronomy::Agent::Base) do ─────
-    if line =~ /^(\s*)([\w.]*\s*=\s*)?Class\.new\((?:::)?Phronomy::Agent::Base\)\s+do\s*(\|\w+\|)?\s*$/
-      indent = $1
-      # Check if agent_definition already present
-      look_ahead = lines[i + 1..]&.take(10) || []
-      new_lines << line
-      i += 1
-      unless look_ahead.any? { |l| l =~ /agent_definition/ }
-        # Derive indentation from the `do` block body (first non-blank line)
-        inner = lines[i..]&.take(5)&.find { |l| l.strip.length > 0 } || ""
-        inner_indent = inner.match(/^(\s*)/)[1]
-        inner_indent = "#{indent}  " if inner_indent.strip.length > 0 && inner_indent.length < indent.length + 2
-        new_lines << "#{inner_indent}agent_definition id: \"test-agent-#{counter += 1}\", version: 1\n"
-        modified = true
-      end
-      next
-    end
-
-    # ── Inline anonymous: Class.new(Phronomy::Agent::Base) { ... } ───────────
-    if line =~ /^(\s*)((?:[\w.]*\s*=\s*)?)Class\.new\((?:::)?Phronomy::Agent::Base\)\s*\{(.+)\}\s*$/
-      indent = $1
-      assignment = $2
-      body = $3
-      # Skip if agent_definition already present
-      if /agent_definition/.match?(body)
-        new_lines << line
-      else
-        id = counter += 1
-        definition_line = "agent_definition id: \"test-agent-#{id}\", version: 1;"
-        new_lines << "#{indent}#{assignment}Class.new(Phronomy::Agent::Base) do\n"
-        new_lines << "#{indent}  #{definition_line}\n"
-        # Rewrite the original body lines, stripping leading spaces
-        body.strip.split(";").each do |stmt|
-          stmt = stmt.strip
-          next if stmt.empty?
-          new_lines << "#{indent}  #{stmt}\n"
-        end
-        new_lines << "#{indent}end\n"
-        modified = true
-      end
-      i += 1
-      next
-    end
-
-    new_lines << line
-    i += 1
-  end
-
-  if modified
-    File.write(file, new_lines.join)
-    files_modified += 1
-    puts "  patched: #{file.sub("#{SPEC_DIR}/", "spec/")}"
-  end
-end
-
-puts "\nDone: #{counter} agent_definition insertions in #{files_modified} files."

data/scripts/migrate_spec_agent_definition_pass2.rb

@@ -1,53 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-# Second pass: handles patterns missed by the first script
-#   - Class.new(Phronomy::Agent::Base) { ... }.new  (inline + .new suffix)
-#   - Class.new(Phronomy::Agent::Base).new           (no block + .new)
-#   - { Class.new(Phronomy::Agent::Base) }           (no block, inside outer { })
-
-require "fileutils"
-
-SPEC_DIR = File.expand_path("../spec", __dir__)
-counter = 200
-
-files_modified = 0
-
-Dir.glob("#{SPEC_DIR}/**/*.rb").sort.each do |file|
-  content = File.read(file)
-  new_content = content.dup
-
-  # Pattern A: Class.new(Phronomy::Agent::Base) { body }.new
-  new_content = new_content.gsub(
-    /Class\.new\(Phronomy::Agent::Base\) \{([^}]*)\}\.new/
-  ) do
-    body = $1
-    next $& if body.include?("agent_definition")
-    counter += 1
-    "Class.new(Phronomy::Agent::Base) { agent_definition id: \"test-agent-#{counter}\", version: 1;#{body}}.new"
-  end
-
-  # Pattern B: Class.new(Phronomy::Agent::Base).new  (no block, chained .new)
-  new_content = new_content.gsub(
-    /Class\.new\(Phronomy::Agent::Base\)\.new\b/
-  ) do
-    counter += 1
-    "Class.new(Phronomy::Agent::Base) { agent_definition id: \"test-agent-#{counter}\", version: 1 }.new"
-  end
-
-  # Pattern C: { Class.new(Phronomy::Agent::Base) }  (no block, inside outer block)
-  new_content = new_content.gsub(
-    "{ Class.new(Phronomy::Agent::Base) }"
-  ) do
-    counter += 1
-    "{ Class.new(Phronomy::Agent::Base) { agent_definition id: \"test-agent-#{counter}\", version: 1 } }"
-  end
-
-  if new_content != content
-    File.write(file, new_content)
-    files_modified += 1
-    puts "  patched: #{file.sub("#{SPEC_DIR}/", "spec/")}"
-  end
-end
-
-puts "\nSecond pass done: #{files_modified} files modified."