familia 2.0.0.pre19 → 2.0.0.pre21

data/try/unit/thread_safety_monitor_try.rb

@@ -0,0 +1,149 @@
+# try/unit/thread_safety_monitor_try.rb
+#
+# frozen_string_literal: true
+
+require_relative '../support/helpers/test_helpers'
+
+# Thread safety monitoring tests
+#
+# Tests basic monitoring functionality without complex concurrency
+
+# setup
+Familia.stop_monitoring! if Familia.thread_safety_monitor.enabled
+Familia.thread_safety_monitor.reset_metrics
+
+## Monitor starts and stops correctly
+Familia.start_monitoring!
+started = Familia.thread_safety_monitor.enabled
+Familia.stop_monitoring!
+stopped = !Familia.thread_safety_monitor.enabled
+started && stopped
+#=> true
+
+## Monitor tracks contentions
+Familia.start_monitoring!
+Familia.thread_safety_monitor.record_contention('test_location')
+Familia.thread_safety_monitor.record_contention('test_location')
+report = Familia.thread_safety_report
+report[:summary][:mutex_contentions]
+#=> 2
+
+## Monitor tracks different locations
+Familia.thread_safety_monitor.reset_metrics
+Familia.thread_safety_monitor.record_contention('location_a')
+Familia.thread_safety_monitor.record_contention('location_b')
+Familia.thread_safety_monitor.record_contention('location_a')
+report = Familia.thread_safety_report
+report[:hot_spots].size
+#=> 2
+
+## Monitor tracks race conditions
+Familia.thread_safety_monitor.reset_metrics
+Familia.thread_safety_monitor.record_race_condition('test', 'details')
+report = Familia.thread_safety_report
+report[:summary][:race_detections]
+#=> 1
+
+## Monitor exports metrics
+Familia.thread_safety_monitor.reset_metrics
+Familia.thread_safety_monitor.record_contention('test')
+metrics = Familia.thread_safety_metrics
+metrics['familia.thread_safety.mutex_contentions']
+#=> 1
+
+## Health score starts at maximum
+Familia.thread_safety_monitor.reset_metrics
+report = Familia.thread_safety_report
+report[:health]
+#=> 100
+
+## InstrumentedMutex is used for connection chain
+conn_mutex = Familia.instance_variable_get(:@connection_chain_mutex)
+conn_mutex.is_a?(Familia::ThreadSafety::InstrumentedMutex)
+#=> true
+
+## InstrumentedMutex tracks basic operations
+mutex = Familia::ThreadSafety::InstrumentedMutex.new('test')
+mutex.synchronize { 'work' }
+stats = mutex.stats
+stats[:lock_count]
+#=> 1
+
+## Monitor time_critical_section works
+Familia.thread_safety_monitor.reset_metrics
+result = Familia.thread_safety_monitor.time_critical_section('test') { 37 }
+result
+#=> 37
+
+## Critical sections are tracked
+report = Familia.thread_safety_report
+report[:summary][:critical_sections]
+#=> 1
+
+## Monitor uses microsecond timing for precision
+Familia.thread_safety_monitor.reset_metrics
+start_μs = Familia.now_in_μs
+Familia.thread_safety_monitor.time_critical_section('timing_test') do
+  sleep 0.005  # 5ms sleep
+end
+end_μs = Familia.now_in_μs
+duration_μs = end_μs - start_μs
+# Should be at least 5000 microseconds (5ms)
+duration_μs >= 5000
+#=> true
+
+## Wait time tracking uses microsecond precision
+mutex = Familia::ThreadSafety::InstrumentedMutex.new('timing_mutex')
+Familia.thread_safety_monitor.reset_metrics
+
+# Create intentional contention with timing
+t1_ready = false
+t2_ready = false
+
+t1 = Thread.new do
+  mutex.synchronize do
+    t1_ready = true
+    sleep 0.01 while !t2_ready  # Wait for t2 to be waiting
+    sleep 0.005  # Hold lock for 5ms
+  end
+end
+
+t2 = Thread.new do
+  sleep 0.001 while !t1_ready  # Wait for t1 to acquire lock
+  t2_ready = true
+  mutex.synchronize { 'got lock' }
+end
+
+t1.join
+t2.join
+
+stats = mutex.stats
+# Should show contention occurred due to intentional delay
+stats[:contention_count] > 0
+#=> true
+
+## Monitor preserves microsecond precision
+Familia.thread_safety_monitor.reset_metrics
+Familia.thread_safety_monitor.record_contention('precision_test', 1500)  # 1500μs = 1.5ms
+
+report = Familia.thread_safety_report
+hot_spot = report[:hot_spots].first
+# Should preserve microsecond precision, not convert to seconds
+hot_spot[:avg_wait_μs]
+#=> 1500
+
+## Critical section timing uses microseconds
+Familia.thread_safety_monitor.reset_metrics
+result = Familia.thread_safety_monitor.time_critical_section('precision_timing') do
+  sleep 0.003  # 3ms
+  37
+end
+
+report = Familia.thread_safety_report
+section_perf = report[:section_performance].first
+# Total time should be in microseconds (≥3000μs for 3ms sleep)
+section_perf[:total_time_μs] >= 3000
+#=> true
+
+# teardown
+Familia.stop_monitoring!

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: familia
 version: !ruby/object:Gem::Version
-  version: 2.0.0.pre19
+  version: 2.0.0.pre21
 platform: ruby
 authors:
 - Delano Mandelbaum
@@ -23,6 +23,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.4'
+- !ruby/object:Gem::Dependency
+  name: concurrent-ruby
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
 - !ruby/object:Gem::Dependency
   name: connection_pool
   requirement: !ruby/object:Gem::Requirement
@@ -137,7 +151,6 @@ files:
 - ".github/workflows/ci.yml"
 - ".github/workflows/claude-code-review.yml"
 - ".github/workflows/claude.yml"
-- ".github/workflows/code-quality.yml"
 - ".github/workflows/code-smells.yml"
 - ".github/workflows/docs.yml"
 - ".gitignore"
@@ -153,35 +166,39 @@ files:
 - LICENSE.txt
 - README.md
 - bin/irb
-- changelog.d/20251011_012003_delano_159_datatype_transaction_pipeline_support.rst
-- changelog.d/20251011_203905_delano_next.rst
-- changelog.d/20251011_212633_delano_next.rst
-- changelog.d/20251011_221253_delano_next.rst
+- bin/try
+- bin/tryouts
+- changelog.d/20251105_flexible_external_identifier_format.rst
+- changelog.d/20251107_112554_delano_179_participation_asymmetry.rst
+- changelog.d/20251107_213121_delano_fix_thread_safety_races_011CUumCP492Twxm4NLt2FvL.rst
+- changelog.d/20251107_fix_participates_in_symbol_resolution.rst
+- changelog.d/20251107_optimized_redis_exists_checks.rst
+- changelog.d/20251108_frozen_string_literal_pragma.rst
 - changelog.d/README.md
 - changelog.d/scriv.ini
+- docs/1106-participates_in-bidirectional-solution.md
 - docs/archive/.gitignore
-- docs/archive/FAMILIA_RELATIONSHIPS.md
-- docs/archive/FAMILIA_TECHNICAL.md
-- docs/archive/FAMILIA_UPDATE.md
-- docs/archive/README.md
-- docs/archive/api-reference.md
 - docs/conf.py
 - docs/guides/.gitignore
-- docs/guides/core-field-system.md
+- docs/guides/encryption.md
 - docs/guides/feature-encrypted-fields.md
 - docs/guides/feature-expiration.md
 - docs/guides/feature-external-identifiers.md
 - docs/guides/feature-object-identifiers.md
 - docs/guides/feature-quantization.md
+- docs/guides/feature-relationships-indexing.md
 - docs/guides/feature-relationships-methods.md
+- docs/guides/feature-relationships-participation.md
 - docs/guides/feature-relationships.md
 - docs/guides/feature-system-devs.md
 - docs/guides/feature-system.md
 - docs/guides/feature-transient-fields.md
-- docs/guides/implementation.md
+- docs/guides/field-system.md
 - docs/guides/index.md
-- docs/guides/security-model.md
-- docs/guides/time-utilities.md
+- docs/guides/logging.md
+- docs/guides/optimized-loading.md
+- docs/guides/thread-safety-monitoring.md
+- docs/guides/time-literals.md
 - docs/migrating/.gitignore
 - docs/migrating/v2.0.0-pre.md
 - docs/migrating/v2.0.0-pre11.md
@@ -190,6 +207,7 @@ files:
 - docs/migrating/v2.0.0-pre14.md
 - docs/migrating/v2.0.0-pre18.md
 - docs/migrating/v2.0.0-pre19.md
+- docs/migrating/v2.0.0-pre22.md
 - docs/migrating/v2.0.0-pre5.md
 - docs/migrating/v2.0.0-pre6.md
 - docs/migrating/v2.0.0-pre7.md
@@ -204,6 +222,7 @@ files:
 - examples/json_usage_patterns.rb
 - examples/relationships.rb
 - examples/safe_dump.rb
+- examples/sampling_demo.rb
 - examples/single_connection_transaction_confusions.rb
 - familia.gemspec
 - lib/familia.rb
@@ -255,11 +274,14 @@ files:
 - lib/familia/features/relationships/collection_operations.rb
 - lib/familia/features/relationships/indexing.rb
 - lib/familia/features/relationships/indexing/multi_index_generators.rb
+- lib/familia/features/relationships/indexing/rebuild_strategies.rb
 - lib/familia/features/relationships/indexing/unique_index_generators.rb
 - lib/familia/features/relationships/indexing_relationship.rb
 - lib/familia/features/relationships/participation.rb
 - lib/familia/features/relationships/participation/participant_methods.rb
+- lib/familia/features/relationships/participation/rebuild_strategies.md
 - lib/familia/features/relationships/participation/target_methods.rb
+- lib/familia/features/relationships/participation_membership.rb
 - lib/familia/features/relationships/participation_relationship.rb
 - lib/familia/features/relationships/score_encoding.rb
 - lib/familia/features/safe_dump.rb
@@ -279,6 +301,7 @@ files:
 - lib/familia/horreum/settings.rb
 - lib/familia/horreum/utils.rb
 - lib/familia/identifier_extractor.rb
+- lib/familia/instrumentation.rb
 - lib/familia/json_serializer.rb
 - lib/familia/logging.rb
 - lib/familia/refinements.rb
@@ -287,9 +310,12 @@ files:
 - lib/familia/refinements/time_literals.rb
 - lib/familia/secure_identifier.rb
 - lib/familia/settings.rb
+- lib/familia/thread_safety/instrumented_mutex.rb
+- lib/familia/thread_safety/monitor.rb
 - lib/familia/utils.rb
 - lib/familia/verifiable_identifier.rb
 - lib/familia/version.rb
+- lib/middleware/database_command_counter.rb
 - lib/middleware/database_logger.rb
 - lib/multi_result.rb
 - pr_agent.toml
@@ -337,11 +363,15 @@ files:
 - try/features/quantization/quantization_try.rb
 - try/features/real_feature_integration_try.rb
 - try/features/relationships/indexing_commands_verification_try.rb
+- try/features/relationships/indexing_rebuild_try.rb
 - try/features/relationships/indexing_try.rb
+- try/features/relationships/participation_bidirectional_try.rb
 - try/features/relationships/participation_commands_verification_spec.rb
 - try/features/relationships/participation_commands_verification_try.rb
 - try/features/relationships/participation_performance_improvements_try.rb
 - try/features/relationships/participation_reverse_index_try.rb
+- try/features/relationships/participation_target_class_resolution_try.rb
+- try/features/relationships/participation_unresolved_target_try.rb
 - try/features/relationships/relationships_api_changes_try.rb
 - try/features/relationships/relationships_edge_cases_try.rb
 - try/features/relationships/relationships_performance_minimal_try.rb
@@ -384,10 +414,15 @@ files:
 - try/integration/models/familia_object_try.rb
 - try/integration/persistence_operations_try.rb
 - try/integration/relationships_persistence_round_trip_try.rb
+- try/integration/save_methods_consistency_try.rb
 - try/integration/scenarios_try.rb
 - try/integration/secure_identifier_try.rb
+- try/integration/transaction_safety_core_try.rb
+- try/integration/transaction_safety_workflow_try.rb
 - try/integration/verifiable_identifier_try.rb
+- try/investigation/pipeline_routing/README.md
 - try/performance/benchmarks_try.rb
+- try/performance/transaction_safety_benchmark_try.rb
 - try/support/benchmarks/deserialization_benchmark.rb
 - try/support/benchmarks/deserialization_correctness_test.rb
 - try/support/debugging/README.md
@@ -437,6 +472,18 @@ files:
 - try/support/prototypes/pooling/lib/visualize_stress_results.rb
 - try/support/prototypes/pooling/pool_siege.rb
 - try/support/prototypes/pooling/run_stress_tests.rb
+- try/thread_safety/README.md
+- try/thread_safety/class_connection_chain_race_try.rb
+- try/thread_safety/connection_chain_race_try.rb
+- try/thread_safety/encryption_manager_cache_race_try.rb
+- try/thread_safety/feature_registry_race_try.rb
+- try/thread_safety/fiber_pipeline_isolation_try.rb
+- try/thread_safety/fiber_transaction_isolation_try.rb
+- try/thread_safety/field_registration_race_try.rb
+- try/thread_safety/logger_initialization_race_try.rb
+- try/thread_safety/middleware_registration_race_try.rb
+- try/thread_safety/module_config_race_try.rb
+- try/thread_safety/secure_identifier_cache_race_try.rb
 - try/unit/core/autoloader_try.rb
 - try/unit/core/base_enhancements_try.rb
 - try/unit/core/connection_try.rb
@@ -444,6 +491,9 @@ files:
 - try/unit/core/extensions_try.rb
 - try/unit/core/familia_logger_try.rb
 - try/unit/core/familia_try.rb
+- try/unit/core/middleware_sampling_try.rb
+- try/unit/core/middleware_test_helpers_bug_try.rb
+- try/unit/core/middleware_thread_safety_try.rb
 - try/unit/core/middleware_try.rb
 - try/unit/core/settings_try.rb
 - try/unit/core/time_utils_try.rb
@@ -459,6 +509,7 @@ files:
 - try/unit/data_types/sorted_set_zadd_options_try.rb
 - try/unit/data_types/string_try.rb
 - try/unit/data_types/unsortedset_try.rb
+- try/unit/familia_resolve_class_try.rb
 - try/unit/horreum/auto_indexing_on_save_try.rb
 - try/unit/horreum/automatic_index_validation_try.rb
 - try/unit/horreum/base_try.rb
@@ -471,16 +522,20 @@ files:
 - try/unit/horreum/field_definition_try.rb
 - try/unit/horreum/initialization_try.rb
 - try/unit/horreum/json_type_preservation_try.rb
+- try/unit/horreum/optimized_loading_try.rb
 - try/unit/horreum/relations_try.rb
 - try/unit/horreum/serialization_persistent_fields_try.rb
 - try/unit/horreum/serialization_try.rb
 - try/unit/horreum/settings_try.rb
 - try/unit/horreum/unique_index_edge_cases_try.rb
 - try/unit/horreum/unique_index_guard_validation_try.rb
+- try/unit/middleware/database_command_counter_methods_try.rb
+- try/unit/middleware/database_logger_methods_try.rb
 - try/unit/refinements/dear_json_array_methods_try.rb
 - try/unit/refinements/dear_json_hash_methods_try.rb
 - try/unit/refinements/time_literals_numeric_methods_try.rb
 - try/unit/refinements/time_literals_string_methods_try.rb
+- try/unit/thread_safety_monitor_try.rb
 - try/valkey.conf
 homepage: https://github.com/delano/familia
 licenses:
@@ -494,14 +549,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.4'
+      version: 3.3.6
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 3.7.2
 specification_version: 4
 summary: An ORM for Valkey-compatible databases in Ruby.
 test_files: []

data/.github/workflows/code-quality.yml

@@ -1,138 +0,0 @@
-name: Code Quality
-
-on:
-  pull_request:
-    branches: [ main ]
-  push:
-    branches: [ main ]
-  workflow_dispatch:
-
-permissions:
-  contents: read
-  pull-requests: write  # Needed to post comments on PRs
-
-jobs:
-  reek:
-    name: Reek Code Analysis
-    runs-on: ubuntu-24.04
-    timeout-minutes: 5
-
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-
-      - name: Set up Ruby
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: 3.4
-          bundler-cache: true
-
-      - name: Configure Bundler for secure gem installation
-        run: |
-          bundle config set --local path 'vendor/bundle'
-          bundle config set --local deployment 'false'
-
-      - name: Install dependencies
-        run: bundle install
-
-      - name: Run Reek analysis
-        run: |
-          echo "=== Running Reek code analysis ==="
-          echo "This analysis identifies code smells and potential improvements."
-          echo "Results are informational and won't fail the build."
-          echo ""
-
-          # Run reek and capture output (don't fail on warnings)
-          # Use success-exit-code to prevent failures from stopping the analysis
-          bundle exec reek --format=text --success-exit-code 0 --failure-exit-code 0 || true
-
-          echo ""
-          echo "=== Reek analysis complete ==="
-        continue-on-error: true  # Don't fail the build on code smells
-
-      - name: Generate Reek report (if analysis available)
-        run: |
-          echo "=== Generating detailed Reek report ==="
-
-          # Generate JSON report for potential future processing
-          bundle exec reek --format=json --success-exit-code 0 --failure-exit-code 0 > reek-report.json || true
-
-          # Display summary
-          if [ -s reek-report.json ]; then
-            echo "Reek JSON report generated: $(wc -l < reek-report.json) lines"
-            echo "Top code smell types found:"
-            jq -r '.[].smells[].smell_type' reek-report.json 2>/dev/null | sort | uniq -c | sort -rn | head -10 || echo "Unable to parse JSON report"
-          else
-            echo "No code smells detected or analysis failed"
-          fi
-        continue-on-error: true
-
-      - name: Upload Reek report as artifact
-        uses: actions/upload-artifact@v4
-        if: always()
-        with:
-          name: reek-report
-          path: reek-report.json
-          retention-days: 30
-
-  # Add other code quality checks here
-  additional-quality-checks:
-    name: Additional Quality Checks
-    runs-on: ubuntu-24.04
-    timeout-minutes: 5
-
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-
-      - name: Set up Ruby
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: 3.4
-          bundler-cache: true
-
-      - name: Configure Bundler for secure gem installation
-        run: |
-          bundle config set --local path 'vendor/bundle'
-          bundle config set --local deployment 'false'
-
-      - name: Install dependencies
-        run: bundle install
-
-      - name: Check for TODO/FIXME comments
-        run: |
-          echo "=== Scanning for TODO/FIXME comments ==="
-          echo "This helps track technical debt and action items."
-          echo ""
-
-          # Find TODO/FIXME comments (excluding this workflow file)
-          find . -name "*.rb" -not -path "./vendor/*" -not -path "./tmp/*" | \
-            xargs grep -Hn -i -E "(TODO|FIXME|HACK|XXX|NOTE):" 2>/dev/null | \
-            head -20 || echo "No TODO/FIXME comments found"
-        continue-on-error: true
-
-      - name: Check Ruby file syntax
-        run: |
-          echo "=== Checking Ruby syntax ==="
-          echo "Validates that all Ruby files have correct syntax."
-          echo ""
-
-          find . -name "*.rb" -not -path "./vendor/*" -not -path "./tmp/*" | \
-            while read -r file; do
-              if ! ruby -c "$file" > /dev/null 2>&1; then
-                echo "Syntax error in: $file"
-                ruby -c "$file"
-              fi
-            done
-        continue-on-error: true
-
-      - name: Check for long lines
-        run: |
-          echo "=== Checking for long lines (>120 characters) ==="
-          echo "Identifies potentially hard-to-read code lines."
-          echo ""
-
-          find . -name "*.rb" -not -path "./vendor/*" -not -path "./tmp/*" | \
-            xargs grep -Hn "^.\{121,\}$" | \
-            head -10 || echo "No overly long lines found"
-        continue-on-error: true

data/changelog.d/20251011_012003_delano_159_datatype_transaction_pipeline_support.rst

@@ -1,91 +0,0 @@
-.. Added
-.. -----
-.. New features and capabilities that have been added.
-
-.. Changed
-.. -------
-.. Changes to existing functionality.
-
-.. Deprecated
-.. ----------
-.. Soon-to-be removed features.
-
-.. Removed
-.. -------
-.. Now removed features.
-
-.. Fixed
-.. -----
-.. Bug fixes.
-
-.. Security
-.. --------
-.. Security-related improvements.
-
-Added
------
-
--  **DataType Transaction and Pipeline Support** - DataType objects can now initiate transactions and pipelines independently, enabling atomic operations and batch command execution for both parent-owned and standalone DataType objects. `PR #159 <https://github.com/familia/familia/pull/159>`_
-
-   Key capabilities added:
-
-   * ``transaction`` method for atomic MULTI/EXEC operations on all DataType classes
-   * ``pipelined`` method for batched command execution on all DataType classes
-   * Connection chain pattern with Chain of Responsibility for DataType objects
-   * Two new connection handlers: ``ParentDelegationHandler`` for owned DataTypes and ``StandaloneConnectionHandler`` for independent DataTypes
-   * Enhanced ``direct_access`` method with automatic transaction/pipeline context detection
-   * Shared ``Familia::Connection::Behavior`` module extracting common connection functionality
-
-   This enhancement addresses a critical gap where standalone DataType objects could not guarantee atomicity across multiple operations. A prime example is session storage implementations (similar to Rack::Session stores) where setting session data and expiration must be atomic to prevent memory leaks or security issues. Both parent-owned DataTypes (delegating to parent Horreum objects) and standalone DataTypes now support the full transaction and pipeline API.
-
-   Example usage:
-
-   .. code-block:: ruby
-
-      # Recommended: Use DataType methods for clean, key-free syntax
-      # Parent-owned DataType transaction
-      user.scores.transaction do
-        user.scores.add('level1', 100)
-        user.scores.add('level2', 200)
-      end
-
-      # Standalone DataType transaction (e.g., session storage)
-      session_store = Familia::StringKey.new('session:abc123')
-      session_store.transaction do
-        session_store.set(session_data)
-        session_store.update_expiration(expiration: 3600)
-      end
-
-      # Pipeline for performance optimization
-      leaderboard.pipelined do
-        leaderboard.add('player1', 500)
-        leaderboard.add('player2', 600)
-        leaderboard.size
-      end
-
-      # Advanced: Connection available for low-level Redis commands when needed
-      user.scores.transaction do |conn|
-        conn.zadd(user.scores.dbkey, 100, 'level1')
-        conn.hset(user.profile.dbkey, 'status', 'active')
-      end
-
-Changed
--------
-
--  **DataType URI Construction** - DataType objects with ``logical_database`` settings now return clean URIs without custom port information (e.g., ``redis://127.0.0.1/3`` instead of ``redis://127.0.0.1:2525/3``), ensuring consistent URI representation across the library.
-
--  **Horreum::Connection Refactored** - The ``Horreum::Connection`` module now includes ``Familia::Connection::Behavior``, eliminating code duplication by sharing URI normalization and connection creation methods between Horreum and DataType. This refactoring improves maintainability while preserving all existing functionality.
-
-AI Assistance
--------------
-
-This feature was implemented with significant AI assistance from Claude (Anthropic). The AI helped with:
-
-* Architectural design of the connection chain pattern for DataType objects
-* Implementation of the shared Behavior module to extract common functionality
-* Creation of DataType-specific connection handlers (ParentDelegationHandler, StandaloneConnectionHandler)
-* Comprehensive test coverage including transaction and pipeline integration tests
-* Documentation and changelog preparation
-* Debugging and fixing URI formatting edge cases
-
-The implementation preserves backward compatibility (all 2,216 existing tests pass) while adding 27 new tests specifically for DataType transaction and pipeline support.

data/changelog.d/20251011_203905_delano_next.rst

@@ -1,30 +0,0 @@
-.. A new scriv changelog fragment.
-..
-.. Uncomment the section that is right (remove the leading dots).
-.. For top level release notes, leave all the headers commented out.
-..
-Added
------
-
-- Automatic validation in ``add_to_*`` methods for instance-scoped unique indexes. Previously required manual ``guard_unique_*!`` call before adding to index; now validation happens automatically with clear error messages on duplicate detection.
-
-- Transaction detection in ``save()`` method. Raises ``Familia::OperationModeError`` when ``save()`` is called within an existing transaction, since unique index guards need to read current values which is not possible inside MULTI/EXEC blocks.
-
-Changed
--------
-
-- Instance-scoped unique index ``add_to_*`` methods now automatically validate uniqueness before adding to parent's index. This matches modern ORM expectations where constraint validation happens implicitly during mutation operations.
-
-Documentation
--------------
-
-- Enhanced ``save()`` method documentation to explain transaction restrictions and unique index validation flow.
-
-- Updated ``UniqueIndexGenerators`` documentation to clarify that ``add_to_*`` methods perform automatic validation.
-
-- Added comprehensive test suite (21 test cases) demonstrating automatic validation behavior, transaction detection, and error handling patterns.
-
-AI Assistance
--------------
-
-- Claude Sonnet 4.5 assisted with implementation design, test coverage, and documentation for automatic unique index validation and transaction detection features.

data/changelog.d/20251011_212633_delano_next.rst

@@ -1,13 +0,0 @@
-
-Changed
--------
-
-- **IndexingRelationship**: Added explicit ``:within`` field to preserve the original DSL parameter, replacing brittle ``target_class`` equality checks with clearer ``within.nil?`` checks. This makes the distinction between class-level and instance-scoped indexes more explicit and prevents potential issues with inheritance scenarios.
-
-AI Assistance
--------------
-
-- Design review and architectural analysis by Claude Code (Sonnet 4.5) via second-opinion agent, identifying brittleness in class comparison logic and recommending explicit storage of the ``within`` parameter.
-- Implementation of the ``within`` field addition across IndexingRelationship, generators, and usage sites by Claude Code.
-- All tests verified passing with no behavioral changes.
-..