auto-l18n 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9eaeb61d41c636035f955c83afb385e40a020e4806bcc2553b965032b3966c64
-  data.tar.gz: c062d54dd2f5b7e639eeb9304e31993aa0a7485703a4236868d86a731ccc1518
+  metadata.gz: d827a80d2d978e8855592c2fe411c97f169a5c44b59d2a7e3a87da698bad39ad
+  data.tar.gz: 82096e37eb8ad1dad0473c5762c1cd04b5e2f078b952747212bc84656bbacab8
 SHA512:
-  metadata.gz: ae940be66b33102dce802addaa5f2d3abacde2f3e934383b8ab0a663329a5672c3ef84065d5020332de26cca3f43831a2e214c61ffecfcae661b34d2e2249d0c
-  data.tar.gz: 31b3b158aa2d1978828874f1d30e7815e5e8b868e599beb3e93fb1c7f68335476386c03d198e3506ce967208f66dec2e985c4e03570fa3ff860fcc434928da01
+  metadata.gz: b5cbf6914f5e5c1ae1c42d3f7a18895596b621280eee99071110234c878279096d49a226886b6f8a3a9c583fec8ef55c06d5652c18779fe8d2e63d978d58aa58
+  data.tar.gz: 945cb23e8fd293d78a81e89a717f3aea3ebfac24358b84df5bc0ac10bd36c765ecb681b8f49090b1888e2e1fa2a5d38ceacb1a3e792661f643d8470b2699bc1a

data/README.md

@@ -50,7 +50,7 @@ texts.each { |t| puts "- #{t}" }
 # Preview changes (dry run)
 result = Auto::L18n.auto_internationalize(
   "app/views/posts/show.html.erb",
-  namespace: "views.posts.show",
+  # namespace: "views.posts.show", # Optional – will be derived from the file path if omitted
   dry_run: true
 )
 
@@ -58,8 +58,7 @@ puts "Would replace #{result[:total_replaced]} strings"
 
 # Apply changes
 result = Auto::L18n.auto_internationalize(
-  "app/views/posts/show.html.erb",
-  namespace: "views.posts.show"
+  "app/views/posts/show.html.erb"
 )
 ```
 
@@ -70,12 +69,13 @@ result = Auto::L18n.auto_internationalize(
 ruby exe/auto-l18n app/views/posts/show.html.erb
 
 # Replace with I18n calls (dry run)
+# Note: If --namespace is omitted, it will be derived from the file path (e.g., app/views/admin/users/show.html.erb -> views.admin.users.show)
 ruby exe/auto-l18n app/views/posts/show.html.erb \
-  --replace --namespace views.posts.show --dry-run
+  --replace --dry-run
 
 # Actually apply changes
 ruby exe/auto-l18n app/views/posts/show.html.erb \
-  --replace --namespace views.posts.show
+  --replace
 ```
 
 ## Example Transformation
@@ -178,7 +178,7 @@ Options:
   --ext=EXTS              File extensions (default: .html.erb)
   --replace               Replace hardcoded text with I18n calls
   --locale-path=PATH      Locale file path (default: config/locales/en.yml)
-  --namespace=NS          Translation key namespace (e.g., views.posts)
+  --namespace=NS          Translation key namespace (e.g., views.posts). If omitted, it is derived from the file path (folder hierarchy).
   --dry-run               Preview changes without modifying files
   --no-backup             Don't create backup files
   -h, --help              Show help

data/exe/auto-l18n

@@ -42,7 +42,7 @@ parser = OptionParser.new do |opts|
     options[:locale_path] = path
   end
 
-  opts.on("--namespace=NS", "Namespace for translation keys (e.g., views.posts)") do |ns|
+  opts.on("--namespace=NS", "Namespace for translation keys (e.g., views.posts). If omitted, it is derived from the file path (folder hierarchy).") do |ns|
     options[:namespace] = ns
   end
 
@@ -101,16 +101,47 @@ if paths.empty?
 end
 
 if options[:mode] == "find"
-  # Original behavior: find and list hardcoded text
+  # Find and list hardcoded text; in --dry-run also show the keys that would be used
   total = 0
   paths.each do |p|
     found = Auto::L18n.find_text(p)
     next if found.empty?
 
-    total += found.size
     puts "\nFile: #{p}"
-    found.each do |s|
-      puts "  - #{s}"
+    # List found strings
+    total += found.size
+    found.each { |s| puts "  - #{s}" }
+
+    # In dry-run, show the keys that would be used for replacement
+    if options[:dry_run]
+      structured = Auto::L18n.find_text(p, structured: true)
+      # mimic the sort used by exchange_text_for_l18n_placeholder (by line desc)
+      sorted = structured.sort_by { |f| -(f.line || 0) }
+
+      # Determine effective namespace as in the library code
+      effective_ns = options[:namespace]
+      if (effective_ns.nil? || effective_ns.empty?)
+        # Call private helper via send to derive from path
+        begin
+          effective_ns = Auto::L18n.send(:derive_namespace_from_path, p, {})
+          effective_ns = nil if effective_ns.nil? || effective_ns.empty?
+        rescue NoMethodError
+          # Fallback: no derived namespace available
+          effective_ns = options[:namespace]
+        end
+      end
+
+      keys = sorted.each_with_index.map do |f, idx|
+        begin
+          Auto::L18n.send(:generate_translation_key, f.text, f.type, effective_ns, idx)
+        rescue NoMethodError
+          # Extremely unlikely; just show placeholder when method not accessible
+          "generated_key_#{idx}"
+        end
+      end
+
+      puts "  Keys that would be used:"
+      keys.each { |k| puts "    - #{k}" }
     end
   end
 

data/lib/auto/l18n/version.rb

@@ -2,6 +2,6 @@
 
 module Auto
   module L18n
-    VERSION = "0.1.0"
+    VERSION = "0.1.2"
   end
 end

data/lib/auto/l18n.rb

@@ -64,7 +64,7 @@ module Auto
       record = lambda do |str, type:, source:, original_position: nil|
         return if str.nil?
         
-        # Normalize whitespace
+  # Normalize whitespace
         s = str.gsub(/\s+/, " ").strip
         return if s.empty?
         return if s.length < opts[:min_length]
@@ -85,14 +85,63 @@ module Auto
         # Only skip if multiple curly braces (likely interpolation)
         return if s.scan(/\{/).size > 1 && s.scan(/\}/).size > 1
         
-        # Skip file paths
+        
+        #  - Common field names that often appear as technical tokens rather than UI text
+        return if s.downcase == 'email'
+  # Skip file paths
         return if s =~ %r{\A\.?/?[\w\-]+(/[\w\-\.]+)+\z}
+
+        # Heuristics to avoid common non-translatable strings
+        #  - CSS class/id-like tokens (snake/kebab case, possibly space-separated list)
+        if (s.include?('_') || s.include?('-')) && s =~ /\A[a-z0-9 _\-]+\z/
+          # e.g., "btn btn-secondary", "group_show", "event-link"
+          return
+        end
+
+        #  - CamelCase or lowerCamelCase identifiers (likely variable/ID names)
+        return if s =~ /\A(?:[a-z]+(?:[A-Z][a-z0-9]+)+|[A-Z][a-z0-9]+(?:[A-Z][a-z0-9]+)+)\z/
+
+        #  - Hex colors like #fff or #4CAF50
+        return if s =~ /\A#(?:[0-9a-fA-F]{3}|[0-9a-fA-F]{6})\b/
+
+        #  - File names with extensions (e.g., JA_Logo.png, Chart.bundle)
+        return if s =~ /\A[\w\-]+(\.[A-Za-z0-9]{2,6})+\z/
+
+        #  - Date/time format strings (strftime-style like %Y-%m-%d, %I:%M %p)
+        return if s =~ /%[a-zA-Z]/
+
+  #  - Mostly numeric values (amounts, percentages)
+        return if s =~ /\A\d+(?:[.,]\d+)?(?:%|[a-zA-Z]*)?\z/
+
+  #  - Rails-style parameter names with bracket notation used in form helpers
+  #    e.g., "webhook_event_call[event][]", "permission_ids[]"
+  return if s =~ /\A[a-zA-Z0-9_]+(?:\[[^\]]*\])+(?:\[\])?\z/
+
+  #  - Common developer token seen as IDs/variables but not visible copy
+  return if s == 'breadcrumb'
+
+  #  - MIME types (single or comma-separated), e.g., "image/*", "image/png,image/jpeg"
+  return if s =~ /\A(?:[a-zA-Z0-9.+-]+\/[a-zA-Z0-9+*.-]+)(?:\s*,\s*[a-zA-Z0-9.+-]+\/[a-zA-Z0-9+*.-]+)*\z/
+
+  #  - CSS inline style declarations (property: value; ...)
+  #    Detect at least one property:value; pair
+  return if s =~ /\b[a-zA-Z\-]+\s*:\s*[^;]+;/
         
         # Skip code-like syntax
-        return if s =~ /[;=>]{2,}/ || s =~ /function\s*\(/ || s =~ /\b(?:var|const|let)\s+\w+/
+  return if s =~ /[;=>]{2,}/ || s =~ /function\s*\(/ || s =~ /\b(?:var|const|let)\s+\w+/
+  # Method-call chains typical of JS (e.g., this.form.submit(); or foo.bar())
+  return if s =~ /[A-Za-z_$][\w$]*(?:\.[A-Za-z_$][\w$]*)+\s*\([^)]*\)\s*;?/
         
         # Normalize quotes for comparison
         normalized = s.tr('""''', %q{"''"})
+
+        # Skip common non-visible tokens that appear as ERB strings (helper options, class toggles)
+        if type == :erb_string
+          # Single lowercase identifier, typical for CSS classes or symbols used in helpers
+          return if s =~ /\A[a-z][a-z0-9_\-]*\z/
+          # Known nav/resource tokens commonly used as classes or symbols
+          return if %w[active members events partners permissions groups].include?(s)
+        end
         
         # Try to estimate line number
         line = estimate_line(original_position, line_map) if original_position
@@ -118,9 +167,40 @@ module Auto
           
           # Skip if it's an I18n call
           next if code =~ /\b(?:I18n\.)?(?:t|translate)\s*\(/
+
+          # Skip asset/helper calls where strings are not visible user text
+          # e.g., stylesheet_link_tag "webhooks", media: "all", "data-turbo-track": "reload"
+          next if code =~ /\b(?:stylesheet_link_tag|javascript_include_tag|javascript_pack_tag|stylesheet_pack_tag|asset_path|image_tag|image_pack_tag|font_path|font_url|asset_url)\b/
+
+          # Skip render calls where string literals represent partial/template names, not visible text
+          # e.g., render 'form', render partial: 'form'
+          next if code =~ /\brender\b/
+
+          # Skip check_box_tag and radio_button_tag: string args are names/values, not visible copy
+          next if code =~ /\b(?:check_box_tag|radio_button_tag)\b/
+
+          # Helper-aware extraction: only consider visible-content helpers
+          # Allow: link_to, button_to, submit_tag, label_tag, content_tag, form builder label/submit
+          helper_in_code = (
+            code =~ /\b(?:link_to|button_to|submit_tag|label_tag|content_tag)\b/ ||
+            code =~ /\b\w+\.(?:label|submit)\b/
+          )
+
+          # If no known content-bearing helper is present, skip extracting strings from this ERB block
+          unless helper_in_code
+            next
+          end
           
           # Extract double-quoted strings
           code.scan(/"((?:[^"\\]|\\.)*)"/m).each do |string_match|
+            # Skip inline conditional class injections like: "calculated" if condition
+            # or other inline if/unless patterns composed solely of a literal
+            next if code.strip =~ /\A["'][a-z0-9 _\-]+["']\s+(?:if|unless)\b/i
+            # Skip option-hash values like: class: "...", id: "...", data: "...", style: "..."
+            literal = string_match[0]
+            escaped_lit = Regexp.escape(literal)
+            next if code =~ /\b[a-zA-Z_]\w*\s*:\s*"#{escaped_lit}"/
+            next if code =~ /"#{escaped_lit}"\s*=>/
             unescaped = string_match[0].gsub(/\\(.)/, '\1')
             record.call(
               unescaped,
@@ -132,6 +212,13 @@ module Auto
           
           # Extract single-quoted strings
           code.scan(/'((?:[^'\\]|\\.)*)'/).each do |string_match|
+            # Skip inline conditional class injections like: 'calculated' if condition
+            next if code.strip =~ /\A["'][a-z0-9 _\-]+["']\s+(?:if|unless)\b/i
+            # Skip option-hash values like: class: '...'
+            literal = string_match[0]
+            escaped_lit = Regexp.escape(literal)
+            next if code =~ /\b[a-zA-Z_]\w*\s*:\s*'#{escaped_lit}'/
+            next if code =~ /'#{escaped_lit}'\s*=>/
             unescaped = string_match[0].gsub(/\\(.)/, '\1')
             record.call(
               unescaped,
@@ -229,10 +316,30 @@ module Auto
       fragment.css(selector).each do |el|
         all_attrs.each do |attr|
           next unless el[attr]
-          
-          # Skip empty values or single characters for 'value' attribute
-          next if attr == 'value' && el[attr].length < 2
-          
+
+          # Skip title attributes (tooltips) per project convention to reduce false positives
+          next if attr == 'title'
+
+          # Special handling for the 'value' attribute:
+          # - Only treat as visible text for input types that display their value as a label
+          #   (submit, button, reset). For all other cases (hidden, text, radio, checkbox,
+          #   option values, etc.) the value is not a visible label and should be ignored.
+          if attr == 'value'
+            tag = el.name.to_s.downcase
+            if tag == 'input'
+              input_type = (el['type'] || '').downcase
+              visible_button_types = %w[submit button reset]
+              # If not a visible button-like input, skip capturing the value
+              next unless visible_button_types.include?(input_type)
+            else
+              # For non-input tags (e.g., option, button), don't treat value as visible label
+              next
+            end
+          end
+
+          # Skip empty values or single characters (after applying 'value' rules above)
+          next if el[attr].strip.length < 2
+
           position = find_position_in_original(el[attr], raw)
           record.call(
             el[attr],
@@ -317,7 +424,7 @@ module Auto
       results
     end
 
-      # Exchange hardcoded text for I18n placeholders
+    # Exchange hardcoded text for I18n placeholders
     # 
     # This method replaces hardcoded strings in a file with I18n translation calls
     # and adds the translations to a locale file (default: en.yml).
@@ -333,7 +440,7 @@ module Auto
     # @option options [Boolean] :backup (true) Create backup files before modifying
     #
     # @return [Hash] Summary of changes made
-    def self.exchange_text_for_l18n_placeholder(path, options = {})
+  def self.exchange_text_for_l18n_placeholder(path, options = {})
       raise ArgumentError, "path must be a String" unless path.is_a?(String)
       raise ArgumentError, "File not found: #{path}" unless File.file?(path)
 
@@ -342,6 +449,9 @@ module Auto
         locale_path: "config/locales/en.yml",
         locale: "en",
         namespace: nil,
+        namespace_from_path: true, # derive namespace from folder/file path when none provided
+        path_root: nil,            # optional root to strip (e.g., "app/views")
+        namespace_prefix: nil,     # optional prefix to prepend (e.g., "views")
         dry_run: false,
         min_length: 2,
         ignore_patterns: [],
@@ -367,9 +477,17 @@ module Auto
       # Process findings in reverse order by position to maintain string positions
       sorted_findings = findings.sort_by { |f| -(f.line || 0) }
       
+      # Determine effective namespace: user-provided or derived from file path
+      effective_namespace = opts[:namespace]
+      if (effective_namespace.nil? || effective_namespace.empty?) && opts[:namespace_from_path]
+        effective_namespace = derive_namespace_from_path(path, opts)
+        # Normalize to nil if empty
+        effective_namespace = nil if effective_namespace.nil? || effective_namespace.empty?
+      end
+
       sorted_findings.each_with_index do |finding, idx|
         # Generate translation key
-        key = generate_translation_key(finding.text, finding.type, opts[:namespace], idx)
+        key = generate_translation_key(finding.text, finding.type, effective_namespace, idx)
         
         # Add to locale file
         set_nested_key(locale_data, key, finding.text, opts[:locale])
@@ -496,6 +614,64 @@ module Auto
       parts.join('.')
     end
 
+    # Derive a namespace from the file path, producing a dotted hierarchy
+    # Examples:
+    #  - app/views/admin/users/show.html.erb -> "views.admin.users.show"
+    #  - app/views/posts/index.html.erb -> "views.posts.index"
+    #  - src/templates/home.html.erb (with namespace_prefix: nil) -> "src.templates.home"
+    #  - test/test.html.erb (no root) -> "test.test"
+    def self.derive_namespace_from_path(path, options = {})
+      require 'pathname'
+      abs_path = File.expand_path(path)
+
+      # Resolve root to strip
+      root = options[:path_root]
+      prefix = options[:namespace_prefix]
+
+      # Auto-detect common Rails views root and default prefix
+      if root.nil? && abs_path.include?(File.join('app', 'views'))
+        root = abs_path.split(File.join('app', 'views')).first + File.join('app', 'views')
+        prefix ||= 'views'
+      end
+
+      rel_path = nil
+      if root && abs_path.start_with?(File.expand_path(root) + File::SEPARATOR)
+        rel_path = Pathname.new(abs_path).relative_path_from(Pathname.new(File.expand_path(root))).to_s
+      else
+        # Try relative to current working directory
+        cwd = Dir.pwd
+        if abs_path.start_with?(cwd + File::SEPARATOR)
+          rel_path = abs_path.sub(cwd + File::SEPARATOR, '')
+        else
+          rel_path = File.basename(abs_path)
+        end
+      end
+
+      # Remove all extensions (handle multi-extensions like .html.erb)
+      base = rel_path.dup
+      loop do
+        ext = File.extname(base)
+        break if ext.nil? || ext.empty?
+        base = base.chomp(ext)
+      end
+
+      # Split into segments, normalize, and join with dots
+      segments = base.split(File::SEPARATOR).map do |seg|
+        seg.downcase
+           .gsub(/[^\w\-\.]+/, '_')
+           .gsub(/[\-\.]/, '_')
+           .gsub(/_+/, '_')
+           .gsub(/^_|_$/, '')
+      end.reject(&:empty?)
+
+      derived = segments.join('.')
+      if prefix && !prefix.to_s.empty?
+        [prefix, derived].reject(&:empty?).join('.')
+      else
+        derived
+      end
+    end
+
     # Load locale file (YAML)
     def self.load_locale_file(path, locale)
       if File.exist?(path)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: auto-l18n
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - Nicolas Reiner
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-10-27 00:00:00.000000000 Z
+date: 2025-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -44,6 +44,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- auto-l18n-0.1.0.gem
 - exe/auto-l18n
 - lib/auto/l18n.rb
 - lib/auto/l18n/version.rb
@@ -56,7 +57,7 @@ metadata:
   source_code_uri: https://github.com/NicolasReiner/auto-l18n
   changelog_uri: https://github.com/NicolasReiner/auto-l18n/blob/master/CHANGELOG.md
   bug_tracker_uri: https://github.com/NicolasReiner/auto-l18n/issues
-  rubygems_mfa_required: 'true'
+  rubygems_mfa_required: 'false'
 post_install_message:
 rdoc_options: []
 require_paths: