jekyll-l10n 2.0.0 → 3.0.0

data/lib/jekyll-l10n/utils/logger_formatter.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../constants'
+
 module Jekyll
   module L10n
     # Formats and conditionally logs messages with component prefixes.
@@ -25,7 +27,7 @@ module Jekyll
       # @param message [String] Message to log
       # @return [void]
       def self.info(component, message)
-        Jekyll.logger.info 'Localization', "[#{component}] #{message}"
+        Jekyll.logger.info Constants::LOG_PREFIX, "[#{component}] #{message}"
       end
 
       # Log a debug message with component prefix.
@@ -34,7 +36,7 @@ module Jekyll
       # @param message [String] Message to log
       # @return [void]
       def self.debug(component, message)
-        Jekyll.logger.debug 'Localization', "[#{component}] #{message}"
+        Jekyll.logger.debug Constants::LOG_PREFIX, "[#{component}] #{message}"
       end
 
       # Log a warning message with component prefix.
@@ -43,7 +45,7 @@ module Jekyll
       # @param message [String] Message to log
       # @return [void]
       def self.warn(component, message)
-        Jekyll.logger.warn 'Localization', "[#{component}] #{message}"
+        Jekyll.logger.warn Constants::LOG_PREFIX, "[#{component}] #{message}"
       end
 
       # Log an error message with component prefix.
@@ -52,7 +54,7 @@ module Jekyll
       # @param message [String] Message to log
       # @return [void]
       def self.error(component, message)
-        Jekyll.logger.error 'Localization', "[#{component}] #{message}"
+        Jekyll.logger.error Constants::LOG_PREFIX, "[#{component}] #{message}"
       end
 
       # Check if debug logging is enabled.

data/lib/jekyll-l10n/utils/msgstr_validator.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+require_relative '../constants'
+require_relative 'html_parser'
+
+module Jekyll
+  module L10n
+    # Checks that a msgstr carries the same markup as its msgid.
+    #
+    # A block's msgid holds placeholders (<g id="N">, <x id="N"></x>) and
+    # literal elements such as <code>. Translation writes the msgstr into the
+    # page as HTML, so a msgstr that drops or repeats a placeholder, or adds
+    # markup of its own (a <script>, an attribute), would change the page
+    # structure. Such entries are dropped and the page shows the fallback.
+    #
+    # Both strings are parsed, and their elements are compared with their
+    # attributes, in any order and nesting: a translation may reorder
+    # placeholders, but not change them.
+    #
+    # @example
+    #   MsgstrValidator.valid?('See <g id="1">docs</g>.', 'Voir <g id="1">docs</g>.')  # => true
+    #   MsgstrValidator.valid?('See <g id="1">docs</g>.', 'Voir docs.')               # => false
+    module MsgstrValidator
+      module_function
+
+      # @param msgid [String]
+      # @param msgstr [String]
+      # @return [Boolean] true when both hold the same elements and attributes
+      def valid?(msgid, msgstr)
+        return true unless msgid.include?('<') || msgstr.include?('<')
+
+        # The answer depends only on the two strings, and a build checks the
+        # same pairs each time a compendium is loaded again.
+        results.fetch([msgid, msgstr]) { results[[msgid, msgstr]] = markup(msgid) == markup(msgstr) }
+      end
+
+      # Drop the entries whose msgstr markup does not match their msgid.
+      #
+      # A compendium also keeps entries for text the site no longer has, so
+      # each one is only logged at debug level here; CoverageReport warns about
+      # the ones the site uses, once per build.
+      #
+      # @param translations [Hash{String => String}] msgid => msgstr
+      # @param po_path [String] The PO file the entries come from, for the log
+      # @return [Hash{String => String}] The valid entries
+      def reject_invalid(translations, po_path)
+        translations.select do |msgid, msgstr|
+          next true if msgstr.empty? || valid?(msgid, msgstr)
+
+          log_invalid(msgid, po_path)
+          false
+        end
+      end
+
+      def markup(text)
+        HtmlParser.parse_fragment(text).css('*').map do |element|
+          [element.name, element.attribute_nodes.map { |attr| [attr.name, attr.value] }.sort]
+        end.sort
+      end
+
+      def log_invalid(msgid, po_path)
+        Jekyll.logger.debug Constants::LOG_PREFIX,
+                            "#{po_path}: msgstr markup does not match its msgid, dropping the " \
+                            "entry: #{msgid[0..Constants::LOG_TRUNCATE_LONG]}"
+      end
+
+      def results
+        @results ||= {}
+      end
+
+      private_class_method :markup, :log_invalid, :results
+    end
+  end
+end

data/lib/jekyll-l10n/utils/page_locales_config.rb

@@ -1,20 +1,24 @@
 # frozen_string_literal: true
 
+require_relative '../configuration'
 require_relative '../constants'
 
 module Jekyll
   module L10n
-    # Wraps a `with_locales_data` hash and answers questions about it, filling in
-    # the defaults from Constants for anything not set.
+    # Wraps the plugin's settings and answers questions about them, filling in
+    # the defaults from Schema::SETTINGS for anything not set.
     #
-    # It raises InvalidConfigurationError right away for bad locale codes or bad
-    # LibreTranslate settings, so mistakes stop the build early.
+    # When LibreTranslate is enabled but its URL, source locale or format is
+    # invalid, it warns once and turns machine translation off, so the build
+    # still finishes. Bad locale codes never reach it: Configuration::Schema
+    # drops them, with a warning.
     #
     # Build it from the right data:
     # - For one page, use SiteConfigAccessor.effective_page_data(page), never raw
     #   `page.data`; otherwise the page loses the site defaults.
-    # - For site-wide settings, use
-    #   `{ 'with_locales_data' => SiteConfigAccessor.extract_locales_data(site) }`.
+    # - For site-wide settings (the `jekyll_l10n` key), use
+    #   SiteConfigAccessor.site_config(site). A page's config has none of
+    #   them, so its site-wide readers return the defaults.
     #
     # @example
     #   config = PageLocalesConfig.new(SiteConfigAccessor.effective_page_data(page))
@@ -22,40 +26,24 @@ module Jekyll
     #   config.fallback_mode # => "english"
     #
     class PageLocalesConfig
-      # Shortcuts to the defaults in Constants.
-      LOCALE_PATTERN = Constants::LOCALE_PATTERN
-      DEFAULT_LOCALES_DIR = Constants::DEFAULT_LOCALES_DIR
-      DEFAULT_FALLBACK_MODE = Constants::DEFAULT_FALLBACK_MODE
-      DEFAULT_TRANSLATABLE_ATTRIBUTES = Constants::DEFAULT_TRANSLATABLE_ATTRIBUTES
-      DEFAULT_LIBRETRANSLATE_TIMEOUT = Constants::DEFAULT_LIBRETRANSLATE_TIMEOUT
-      DEFAULT_LIBRETRANSLATE_BATCH_SIZE = Constants::DEFAULT_LIBRETRANSLATE_BATCH_SIZE
-      DEFAULT_LIBRETRANSLATE_RETRY_ATTEMPTS = Constants::DEFAULT_LIBRETRANSLATE_RETRY_ATTEMPTS
-      DEFAULT_LIBRETRANSLATE_RETRY_DELAY = Constants::DEFAULT_LIBRETRANSLATE_RETRY_DELAY
-      DEFAULT_LIBRETRANSLATE_STOP_ON_ERROR = Constants::DEFAULT_LIBRETRANSLATE_STOP_ON_ERROR
-      DEFAULT_LIBRETRANSLATE_PROGRESS_INTERVAL = Constants::DEFAULT_LIBRETRANSLATE_PROGRESS_INTERVAL
-      DEFAULT_LIBRETRANSLATE_SOURCE_LOCALE = Constants::DEFAULT_LIBRETRANSLATE_SOURCE_LOCALE
-      DEFAULT_LIBRETRANSLATE_FORMAT = Constants::DEFAULT_LIBRETRANSLATE_FORMAT
-
       # @!attribute [r] data
       #   The hash this config was built from
       #   @return [Hash]
       attr_reader :data
 
-      # @param page_data [Hash] A hash with a `with_locales_data` key, and
-      #   optionally `path` (used in error messages)
-      # @raise [Jekyll::Errors::InvalidConfigurationError] If a locale code or a
-      #   LibreTranslate setting is invalid
+      # @param page_data [Hash] A hash with a `with_locales_data` key
       def initialize(page_data)
-        @config = page_data['with_locales_data'] || {}
+        # Cleaned without warnings: Configuration warns where values are written.
+        # Both scopes are kept, so SiteConfigAccessor.site_config can pass the
+        # site-wide settings merged in.
+        @config = Configuration::Schema.normalize(page_data[Configuration::PAGE_DATA_KEY], source: nil, scope: :all)
         @data = page_data
-        @page_path = page_data['path'] || 'unknown'
-
-        validate_locales!
-        validate_libretranslate!
+        @libretranslate_problem = libretranslate_problem if libretranslate_requested?
+        warn_libretranslate_off if @libretranslate_problem
       end
 
-      # Locales to translate into. The constructor has already checked each code
-      # against LOCALE_PATTERN.
+      # Locales to translate into. Configuration::Schema has already dropped
+      # any code that does not match Constants::LOCALE_PATTERN.
       #
       # @return [Array<String>] Locale codes such as ["es", "pt_BR"], or []
       def locales
@@ -64,35 +52,35 @@ module Jekyll
 
       # @return [String] Directory for PO files, relative to the site source
       def locales_dir
-        @config['locales_dir'] || DEFAULT_LOCALES_DIR
+        @config['locales_dir'] || schema_default('locales_dir')
       end
 
       # @return [Boolean] false only when `extract_on_build` is false
       def extract_on_build?
-        @config['extract_on_build'] != false
+        @config.fetch('extract_on_build', schema_default('extract_on_build'))
       end
 
       # Whether to merge page PO files into the locale compendium. A site-wide
-      # setting: read it from the site-wide config, not from a page.
+      # setting: read it from SiteConfigAccessor.site_config, not from a page.
       #
       # @return [Boolean] false only when `update_compendium` is false
       def update_compendium?
-        @config['update_compendium'] != false
+        @config.fetch('update_compendium', schema_default('update_compendium'))
       end
 
       # Whether to print the extraction summary at the end of the build. A
-      # site-wide setting: Extractor reads it from the site-wide config.
+      # site-wide setting: read it from SiteConfigAccessor.site_config.
       #
       # @return [Boolean] false only when `logging.show_statistics` is false
       def show_statistics?
-        @config.dig('logging', 'show_statistics') != false
+        logging_setting('show_statistics')
       end
 
       # Turns on HtmlTranslator's log of each translation lookup for this page.
       #
       # @return [Boolean] true when `logging.debug` is true
       def debug_logging?
-        @config.dig('logging', 'debug') == true
+        logging_setting('debug')
       end
 
       # Turns on per-entry LibreTranslate logging. Those lines go to Jekyll's
@@ -100,7 +88,7 @@ module Jekyll
       #
       # @return [Boolean] true when `logging.trace` or `logging.debug` is true
       def trace_logging?
-        @config.dig('logging', 'trace') == true || debug_logging?
+        logging_setting('trace') || debug_logging?
       end
 
       # What to show when a string has no translation: "english" (the original
@@ -108,15 +96,32 @@ module Jekyll
       #
       # @return [String] The fallback mode
       def fallback_mode
-        @config.dig('translation', 'fallback') || DEFAULT_FALLBACK_MODE
+        @config.dig('translation', 'fallback') || schema_default('translation.fallback')
+      end
+
+      # The share of each locale's compendium that must be translated, or the
+      # build fails. A site-wide setting: read it from
+      # SiteConfigAccessor.site_config.
+      #
+      # @return [Integer, nil] A percentage, or nil when no minimum is set
+      def min_coverage
+        @config.dig('translation', 'min_coverage')
       end
 
       # HTML attributes whose values are extracted and translated. Defaults to
-      # DEFAULT_TRANSLATABLE_ATTRIBUTES.
+      # Constants::DEFAULT_TRANSLATABLE_ATTRIBUTES.
       #
       # @return [Array<String>] Attribute names
       def translatable_attributes
-        @config.dig('extraction', 'translatable_attributes') || DEFAULT_TRANSLATABLE_ATTRIBUTES
+        @config.dig('extraction', 'translatable_attributes') || schema_default('extraction.translatable_attributes')
+      end
+
+      # CSS selectors of elements left out of extraction. Defaults to
+      # Constants::DEFAULT_EXCLUDE_SELECTORS.
+      #
+      # @return [Array<String>] Selectors
+      def exclude_selectors
+        @config.dig('extraction', 'exclude_selectors') || schema_default('extraction.exclude_selectors')
       end
 
       # @return [Boolean] true when at least one locale is configured
@@ -128,37 +133,34 @@ module Jekyll
       # `libretranslate_enabled` always wins. Without it, only a
       # `libretranslate_api_url` you set yourself turns it on; the default URL
       # does not, so an unconfigured site never tries to reach a server.
+      # Either way, it stays off while its URL, source locale or format is
+      # invalid.
       #
       # @return [Boolean] true if LibreTranslate is enabled
       def libretranslate_enabled?
-        if libretranslate_config.key?('libretranslate_enabled')
-          return libretranslate_config['libretranslate_enabled'] == true
-        end
-
-        url = libretranslate_config['libretranslate_api_url']
-        url.is_a?(String) && !url.strip.empty?
+        libretranslate_requested? && @libretranslate_problem.nil?
       end
 
       # @return [String] Locale code of the original content. Defaults to
-      #   DEFAULT_LIBRETRANSLATE_SOURCE_LOCALE.
+      #   Constants::DEFAULT_LIBRETRANSLATE_SOURCE_LOCALE.
       def libretranslate_source_locale
-        libretranslate_setting('libretranslate_source_locale', DEFAULT_LIBRETRANSLATE_SOURCE_LOCALE)
+        libretranslate_setting('libretranslate_source_locale')
       end
 
       # How strings are sent to LibreTranslate. "html" keeps inline markup intact.
       #
-      # @return [String] "text" or "html". Defaults to DEFAULT_LIBRETRANSLATE_FORMAT.
+      # @return [String] "text" or "html". Default from Configuration::Schema::SETTINGS.
       def libretranslate_format
-        libretranslate_setting('libretranslate_format', DEFAULT_LIBRETRANSLATE_FORMAT)
+        libretranslate_setting('libretranslate_format')
       end
 
       # Base URL of the LibreTranslate server, without `/translate`, which
       # LibreTranslator appends. Returning the default here does not turn
       # LibreTranslate on; see {#libretranslate_enabled?}.
       #
-      # @return [String] The URL. Defaults to Constants::DEFAULT_LIBRETRANSLATE_API_URL.
+      # @return [String] The URL. Default from Configuration::Schema::SETTINGS.
       def libretranslate_api_url
-        libretranslate_setting('libretranslate_api_url', Constants::DEFAULT_LIBRETRANSLATE_API_URL)
+        libretranslate_setting('libretranslate_api_url')
       end
 
       # @return [String, nil] API key for servers that require one, or nil
@@ -167,45 +169,60 @@ module Jekyll
       end
 
       # @return [Integer] Request timeout in seconds. Defaults to
-      #   DEFAULT_LIBRETRANSLATE_TIMEOUT.
+      #   Constants::DEFAULT_LIBRETRANSLATE_TIMEOUT.
       def libretranslate_timeout
-        libretranslate_setting('libretranslate_timeout', DEFAULT_LIBRETRANSLATE_TIMEOUT)
+        libretranslate_setting('libretranslate_timeout')
       end
 
-      # Strings sent per request. Larger batches mean fewer requests but may
-      # exceed the server's size limit.
+      # Entries a worker takes from the queue at a time. Each entry is still
+      # its own request.
       #
-      # @return [Integer] Defaults to DEFAULT_LIBRETRANSLATE_BATCH_SIZE.
+      # @return [Integer] Default from Configuration::Schema::SETTINGS.
       def libretranslate_batch_size
-        libretranslate_setting('libretranslate_batch_size', DEFAULT_LIBRETRANSLATE_BATCH_SIZE)
+        libretranslate_setting('libretranslate_batch_size')
+      end
+
+      # Requests sent to LibreTranslate at once. 1 sends them one at a time.
+      #
+      # @return [Integer] Default from Configuration::Schema::SETTINGS.
+      def libretranslate_concurrency
+        libretranslate_setting('libretranslate_concurrency')
       end
 
       # @return [Integer] Retries for a failed request. Defaults to
-      #   DEFAULT_LIBRETRANSLATE_RETRY_ATTEMPTS.
+      #   Constants::DEFAULT_LIBRETRANSLATE_RETRY_ATTEMPTS.
       def libretranslate_retry_attempts
-        libretranslate_setting('libretranslate_retry_attempts', DEFAULT_LIBRETRANSLATE_RETRY_ATTEMPTS)
+        libretranslate_setting('libretranslate_retry_attempts')
       end
 
       # @return [Integer] Seconds to wait between retries. Defaults to
-      #   DEFAULT_LIBRETRANSLATE_RETRY_DELAY.
+      #   Constants::DEFAULT_LIBRETRANSLATE_RETRY_DELAY.
       def libretranslate_retry_delay
-        libretranslate_setting('libretranslate_retry_delay', DEFAULT_LIBRETRANSLATE_RETRY_DELAY)
+        libretranslate_setting('libretranslate_retry_delay')
       end
 
       # When true, the first API error stops machine translation. When false, the
       # error is logged and the remaining entries are still translated.
       #
-      # @return [Boolean] Defaults to Constants::DEFAULT_LIBRETRANSLATE_STOP_ON_ERROR.
+      # @return [Boolean] Default from Configuration::Schema::SETTINGS.
       def libretranslate_stop_on_error?
         val = libretranslate_config['libretranslate_stop_on_error']
-        val.nil? ? Constants::DEFAULT_LIBRETRANSLATE_STOP_ON_ERROR : val != false
+        val.nil? ? schema_default('translation.libretranslate_stop_on_error') : val != false
+      end
+
+      # When true, machine translations are written as fuzzy entries: pages
+      # keep the fallback until a person reviews each one and removes the flag.
+      #
+      # @return [Boolean] Default from Configuration::Schema::SETTINGS.
+      def libretranslate_mark_fuzzy?
+        libretranslate_config.fetch('libretranslate_mark_fuzzy', schema_default('translation.libretranslate_mark_fuzzy'))
       end
 
       # Log progress after every this many translated entries; 0 turns it off.
       #
-      # @return [Integer] Defaults to DEFAULT_LIBRETRANSLATE_PROGRESS_INTERVAL.
+      # @return [Integer] Default from Configuration::Schema::SETTINGS.
       def libretranslate_progress_interval
-        libretranslate_setting('libretranslate_progress_interval', DEFAULT_LIBRETRANSLATE_PROGRESS_INTERVAL)
+        libretranslate_setting('libretranslate_progress_interval')
       end
 
       private
@@ -214,55 +231,57 @@ module Jekyll
         @config['translation'] || {}
       end
 
-      def libretranslate_setting(key, default)
-        libretranslate_config[key] || default
+      # Reads a logging setting by its leaf key, falling back to the default
+      # recorded in Schema::SETTINGS for "logging.<key>".
+      #
+      # @param key [String] leaf key under the `logging` section
+      # @return [Boolean]
+      def logging_setting(key)
+        (@config['logging'] || {}).fetch(key, schema_default("logging.#{key}"))
       end
 
-      def validate_locales!
-        invalid = locales.grep_v(LOCALE_PATTERN)
-
-        return if invalid.empty?
-
-        error_msg = build_validation_error_message(invalid)
-
-        Jekyll.logger.error('Localization', error_msg)
-
-        raise Jekyll::Errors::InvalidConfigurationError, error_msg
+      # Returns the configured value for a libretranslate setting, or the default
+      # recorded in Schema::SETTINGS for "translation.<key>".
+      #
+      # @param key [String] leaf key under the `translation` section
+      # @return [Object]
+      def libretranslate_setting(key)
+        libretranslate_config[key] || schema_default("translation.#{key}")
       end
 
-      def build_validation_error_message(invalid_codes)
-        <<~ERROR
-          Invalid locale codes in #{@page_path}: #{invalid_codes.join(', ')}
-
-          Expected ISO 639-1 language code, optionally with ISO 3166-1 alpha-2 country (e.g., 'es', 'fr', 'pt_BR')
-          Valid pattern: ^[a-z]{2}(_[A-Z]{2})?$
+      # Whether the settings ask for LibreTranslate, before they are checked.
+      def libretranslate_requested?
+        if libretranslate_config.key?('libretranslate_enabled')
+          return libretranslate_config['libretranslate_enabled'] == true
+        end
 
-          Common mistakes:
-          - Use lowercase language code: 'es' not 'ES' or 'ESP'
-          - Use underscore for country: 'pt_BR' not 'pt-BR'
-          - Use ISO codes: 'es' not 'español'
-          - Minimum 2 characters: 'es' not 'e'
-          - Country code uppercase: 'pt_BR' not 'pt_br'
-        ERROR
+        url = libretranslate_config['libretranslate_api_url']
+        url.is_a?(String) && !url.strip.empty?
       end
 
-      def validate_libretranslate!
-        return unless libretranslate_enabled?
-
-        if libretranslate_api_url.nil? || libretranslate_api_url.empty?
-          raise Jekyll::Errors::InvalidConfigurationError,
-                'LibreTranslate is enabled but libretranslate_api_url is not configured'
-        end
-
-        unless LOCALE_PATTERN.match?(libretranslate_source_locale)
-          raise Jekyll::Errors::InvalidConfigurationError,
-                "Invalid libretranslate_source_locale: #{libretranslate_source_locale}"
+      # @return [String, nil] Why the LibreTranslate settings cannot be used,
+      #   or nil when they can
+      def libretranslate_problem
+        if libretranslate_api_url.strip.empty?
+          'libretranslate_api_url is empty'
+        elsif !Constants::LOCALE_PATTERN.match?(libretranslate_source_locale)
+          "libretranslate_source_locale #{libretranslate_source_locale.inspect} is not a locale code such as en or pt_BR"
+        elsif !%w[text html].include?(libretranslate_format)
+          "libretranslate_format #{libretranslate_format.inspect} must be \"text\" or \"html\""
         end
+      end
 
-        return if %w[text html].include?(libretranslate_format)
+      def warn_libretranslate_off
+        Configuration.warn_once("LibreTranslate is turned off: #{@libretranslate_problem}; " \
+                                'fix it under `jekyll_l10n.translation` in _config.yml.')
+      end
 
-        raise Jekyll::Errors::InvalidConfigurationError,
-              "Invalid libretranslate_format: #{libretranslate_format} (must be 'text' or 'html')"
+      # Returns the default value for a setting path from Schema::SETTINGS.
+      #
+      # @param path [String] dotted setting path, e.g. "locales_dir"
+      # @return [Object] the default value
+      def schema_default(path)
+        Configuration::Schema::SETTINGS.fetch(path)[:default]
       end
     end
   end

data/lib/jekyll-l10n/utils/po_entry_converter.rb

@@ -14,40 +14,12 @@ module Jekyll
     # * Convert translation hashes to POEntry objects
     # * Convert POEntry objects back to hash format
     # * Preserve metadata (references, fuzzy flags) during conversion
-    # * Handle both hash and array return formats
     #
     # @example
     #   hash = { "Hello" => { msgstr: "Hola", reference: "file.html:10" } }
     #   entries = PoEntryConverter.hash_to_po_entry_array(hash)
     #   # entries is array of GetText::POEntry objects
     class PoEntryConverter
-      # Convert translation hash to POEntry hash.
-      #
-      # Converts simple translation hash { msgid => msgstr } or metadata format
-      # { msgid => { msgstr: "...", reference: "...", fuzzy: false } } to a
-      # hash of GetText::POEntry objects keyed by msgid.
-      #
-      # @param hash [Hash] Translation hash
-      # @return [Hash] Hash of { msgid => POEntry }
-      def self.hash_to_po_entries(hash)
-        return {} if hash.nil? || hash.empty?
-
-        hash.each_with_object({}) do |(key, value), result|
-          entry = ::GetText::POEntry.new(:normal)
-          entry.msgid = key
-
-          if value.is_a?(Hash)
-            entry.msgstr = value[:msgstr]
-            entry.add_comment(value[:reference]) if value[:reference]
-            entry.flag = 'fuzzy' if value[:fuzzy]
-          else
-            entry.msgstr = value
-          end
-
-          result[key] = entry
-        end
-      end
-
       # Convert translation hash to array of POEntry objects.
       #
       # Converts hash format to array of GetText::POEntry objects, preserving
@@ -75,21 +47,6 @@ module Jekyll
         entry
       end
 
-      # Convert POEntry objects to translation hash.
-      #
-      # @param entries [Array<GetText::POEntry>] Array of POEntry objects
-      # @return [Hash] Hash of { msgid => { msgstr: "...", reference: "..." } }
-      def self.po_entries_to_hash(entries)
-        return {} if entries.nil? || entries.empty?
-
-        entries.to_h do |entry|
-          [entry.msgid, {
-            msgstr: entry.msgstr.to_s,
-            reference: entry.extracted_comment
-          }]
-        end
-      end
-
       # Convert POEntry objects to array of hashes.
       #
       # Each entry becomes a hash with :msgid and :msgstr (and optional :reference).