auto-l18n 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3e1461d71a3c1c714d84295c11605c26cb984a8939c07f265c61489036a1cc58
-  data.tar.gz: c5805b85181f8e87a65a25371c651d4cf936084e9c67d1e2e1c49e09e8934a45
+  metadata.gz: d827a80d2d978e8855592c2fe411c97f169a5c44b59d2a7e3a87da698bad39ad
+  data.tar.gz: 82096e37eb8ad1dad0473c5762c1cd04b5e2f078b952747212bc84656bbacab8
 SHA512:
-  metadata.gz: d33f864be73e4bfa056f17d1897e581e2f8456d93a307871e7173ee19c995f7d09c426af5c8d82b5e77e22d6ede15286aa952525ad46ed2ff444896944e5f825
-  data.tar.gz: 3868fec113b95cd2d64f0e371630c58e27854792dc5b98462d2d1fbe13c55c40cdfd56880464b65b150b39f97c2c74a753cbd0ba0de75c2737bda64274200d2a
+  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.1"
+    VERSION = "0.1.2"
   end
 end

data/lib/auto/l18n.rb

@@ -424,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).
@@ -440,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)
 
@@ -449,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: [],
@@ -474,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])
@@ -603,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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: auto-l18n
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Nicolas Reiner