locallingo 0.5.0 → 0.6.0

data/lib/locallingo/validators/manual_edits.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative "../state_store"
+require_relative "../drift"
 
 module Locallingo
   module Validators
@@ -12,19 +12,15 @@ module Locallingo
     # it was edited by a human — surface it so the operator can protect it with
     # `accept-edits` before the next translate run overwrites it.
     class ManualEdits
-      def initialize(cli_name: "lingo")
+      def initialize(cli_name: "lingo", drift: Drift.new)
         @cli_name = cli_name
+        @drift = drift
       end
 
       # +target+ is the flat target hash; +locale_state+ its loaded state.
       def call(target:, locale_state:, locale:)
         target.filter_map do |key, value|
-          entry = locale_state[key]
-          next unless entry.is_a?(Hash)
-          next if entry["manual"]
-
-          stored = entry["target_hash"]
-          next unless stored && stored != StateStore.hash(value)
+          next unless @drift.hand_edited?(target: value, entry: locale_state[key])
 
           {
             type: :manual_edit,

data/lib/locallingo/validators/outdated.rb

@@ -1,14 +1,15 @@
 # frozen_string_literal: true
 
-require_relative "../state_store"
+require_relative "../drift"
 
 module Locallingo
   module Validators
     # Reports target keys whose recorded source hash no longer matches the
     # current source value — i.e. the English text changed after translation.
     class Outdated
-      def initialize(cli_name: "lingo")
+      def initialize(cli_name: "lingo", drift: Drift.new)
         @cli_name = cli_name
+        @drift = drift
       end
 
       # +source+ is the flat source hash; +locale_state+ is the target locale's
@@ -27,8 +28,7 @@ module Locallingo
       # Keys whose stored source_hash differs from the current source hash.
       def outdated_keys(source, locale_state)
         source.filter_map do |key, value|
-          stored = locale_state.dig(key, "source_hash")
-          key if stored && stored != StateStore.hash(value)
+          key if @drift.outdated?(source: value, entry: locale_state[key])
         end
       end
 
@@ -37,7 +37,7 @@ module Locallingo
       # Manual keys must not be pushed toward machine retranslation — the value
       # is hand-curated, so the human updates it and re-accepts.
       def suggestion_for(key, locale, locale_state)
-        if locale_state.dig(key, "manual")
+        if @drift.manual?(locale_state[key])
           "Source changed for a manually-curated key. Update the #{locale} value by hand, " \
             "then run: #{@cli_name} accept-edits --locale #{locale} --key #{key}"
         else

data/lib/locallingo/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Locallingo
-  VERSION = "0.5.0"
+  VERSION = "0.6.0"
 end

data/lib/locallingo.rb

@@ -1,13 +1,18 @@
 # frozen_string_literal: true
 
+require "monitor"
 require_relative "locallingo/version"
 require_relative "locallingo/settings"
 require_relative "locallingo/configuration"
 require_relative "locallingo/json_extraction"
 require_relative "locallingo/key_flattener"
 require_relative "locallingo/state_store"
+require_relative "locallingo/drift"
 require_relative "locallingo/providers/ruby_llm"
 require_relative "locallingo/batch_translator"
+require_relative "locallingo/prompt"
+require_relative "locallingo/translator"
+require_relative "locallingo/record_translator"
 require_relative "locallingo/validators/missing"
 require_relative "locallingo/validators/outdated"
 require_relative "locallingo/validators/duplicate_values"
@@ -38,20 +43,59 @@ module Locallingo
     Configuration.load(root_path:, package:)
   end
 
-  # Code-level settings (provider credentials) — see Locallingo::Settings.
-  def self.settings
-    @settings ||= Settings.new
+  # Boot-time state, finalized: always a frozen Settings, so any thread or
+  # Ractor reads finished settings and only `configure` replaces them. With
+  # String keys (or none) it is Ractor-shareable; a callable key is not, so it
+  # keeps the settings readable on the main Ractor only.
+  #
+  # rubocop:disable ThreadSafety/ClassInstanceVariable -- @settings is written only by configure and reset_settings!, at boot on the main Ractor, and always holds a frozen Settings; @configure_lock (engines without Ractor only) is set once, at load
+  @settings = Settings.new.freeze
+
+  # Serializes configure: one Monitor per Ractor, in Ractor-local storage (only
+  # the main Ractor can publish @settings, so in practice the main Ractor's).
+  # The main Ractor's is made here, at load, never lazily. A Monitor, so a
+  # configure nested in a configure block does not deadlock. Engines without
+  # Ractor keep it in @configure_lock.
+  if defined?(::Ractor)
+    ::Ractor.current[:locallingo_configure_lock] = Monitor.new
+  else
+    @configure_lock = Monitor.new
   end
 
+  # Code-level settings (provider credentials) — see Locallingo::Settings.
+  def self.settings = @settings
+
   # Configure the gem from Ruby — the credentials path for apps whose keys
   # don't live in ENV (call it from a Rails initializer or a `.locallingo.rb`
   # setup file next to `.locallingo.yml`; the CLI loads the latter on start).
+  # Yields a copy of the current settings, then publishes it frozen in one
+  # assignment: a concurrent reader sees the old settings or the new, never
+  # half of them. Concurrent calls run one at a time, so neither drops the
+  # other's keys. Calling it again keeps the keys it does not set.
   def self.configure
-    yield settings
-    settings
+    configure_lock.synchronize do
+      draft = settings.dup
+      yield draft
+      @settings = draft.freeze
+    end
   end
 
+  # Back to frozen, blank defaults (for tests).
   def self.reset_settings!
-    @settings = nil
+    @settings = Settings.new.freeze
+  end
+
+  # Another Ractor's lock is made on first use: atomically on Ruby 3.4+; below,
+  # two of its threads could each make one, which is harmless because a
+  # non-main Ractor cannot publish settings anyway.
+  def self.configure_lock
+    return @configure_lock unless defined?(::Ractor)
+
+    key = :locallingo_configure_lock
+    return ::Ractor.store_if_absent(key) { Monitor.new } if ::Ractor.respond_to?(:store_if_absent)
+
+    ::Ractor.current[key] ||= Monitor.new
   end
+  private_class_method :configure_lock
+  # rubocop:enable ThreadSafety/ClassInstanceVariable
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: locallingo
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Mikael Henriksson
@@ -69,21 +69,27 @@ files:
 - config/locallingo.default.yml
 - exe/lingo
 - lib/locallingo.rb
+- lib/locallingo/active_record.rb
 - lib/locallingo/batch_translator.rb
 - lib/locallingo/cli.rb
 - lib/locallingo/configuration.rb
+- lib/locallingo/content_leaves.rb
+- lib/locallingo/drift.rb
 - lib/locallingo/json_extraction.rb
 - lib/locallingo/key_flattener.rb
 - lib/locallingo/manager.rb
+- lib/locallingo/prompt.rb
 - lib/locallingo/providers/ruby_llm.rb
 - lib/locallingo/quality/british_spellings.rb
 - lib/locallingo/quality/static_rules.rb
 - lib/locallingo/quality/terminology.rb
 - lib/locallingo/quality_checker.rb
+- lib/locallingo/record_translator.rb
 - lib/locallingo/reporter.rb
 - lib/locallingo/rubocop.rb
 - lib/locallingo/settings.rb
 - lib/locallingo/state_store.rb
+- lib/locallingo/translator.rb
 - lib/locallingo/validators/duplicate_values.rb
 - lib/locallingo/validators/manual_edits.rb
 - lib/locallingo/validators/missing.rb