copy_tuner_client 1.2.1 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 70c29e81f0f1972e5b8833b0ab7a9f9b079186f055e7d988b207043930144708
-  data.tar.gz: 3277c0fc92c99421c183f19356ea49f8830df54c8bfdf2913c1433a4e30e6268
+  metadata.gz: 4c37179f53ed11fd69930c2b4ff36d6597d0e83ab45e749c5711b7bef254ff2d
+  data.tar.gz: b67cd75f1997c995ba84bee721267b2ceeb1df236907d71c982b5b2058702dc1
 SHA512:
-  metadata.gz: f7ca85e2ebd4d5f8634edc0a2b6c4ac1450d35c49e4bd81740c0bf31166ee8937c1cf29a4af4bda9ba5c1bf30d1985282635c9092a8ee2c6b931a0fcdb78c9f2
-  data.tar.gz: 680d69dcfee71fb565c08a948a41fae83946fd407045e50e4169266fdb83ef366dbb0c4a16326206ef1e7dae646e3d7a375bf98c88e3d5d8217bb1d755acbff4
+  metadata.gz: b17a757326468fa69d31d6e86ea684c9f7df6b76b6792429da5cd5d7db0cb0e15392ebdd9f20f66952ebdb7f8770ff3baa636a77f7742134a4053032a2d6e230
+  data.tar.gz: a2df2195d3096ff198893298f968c453564f479a98393f6bfe93f133fcbc414fec7bd004142e6d9f680df28dbebb79535813f79fc273c1b0448906784edfd4d2

data/.vscode/settings.json

@@ -8,4 +8,7 @@
   "cSpell.words": [
     "mkpath"
   ],
+  "[ruby]": {
+    "editor.defaultFormatter": "rubocop.vscode-rubocop",
+  }
 }

data/lib/copy_tuner_client/dotted_hash.rb

@@ -3,7 +3,9 @@ module CopyTunerClient
     def to_h(dotted_hash)
       hash = {}
       dotted_hash.to_h.transform_keys(&:to_s).sort.each do |key, value|
-        _hash = key.split('.').reverse.inject(value) { |memo, key| { key => memo } }
+        # Rails i18n標準との互換性のため、特定のキーを適切な型に変換
+        converted_value = convert_value_type(key, value)
+        _hash = key.split('.').reverse.inject(converted_value) { |memo, _key| { _key => memo } }
         hash.deep_merge!(_hash)
       end
       hash
@@ -16,8 +18,8 @@ module CopyTunerClient
       all_keys.each_with_index do |key, index|
         prefix = "#{key}."
         conflict_keys = ((index + 1)..Float::INFINITY)
-          .take_while { |i| all_keys[i]&.start_with?(prefix) }
-          .map { |i| all_keys[i] }
+                        .take_while { |i| all_keys[i]&.start_with?(prefix) }
+                        .map { |i| all_keys[i] }
 
         if conflict_keys.present?
           results[key] = conflict_keys
@@ -27,6 +29,22 @@ module CopyTunerClient
       results
     end
 
-    module_function :to_h, :conflict_keys
+    private
+
+    def convert_value_type(key, value)
+      return value unless value.is_a?(String)
+
+      # Rails i18n標準で数値型として扱われるキー
+      if key.end_with?('.precision')
+        value.to_i
+      # Rails i18n標準で真偽値として扱われるキー
+      elsif key.end_with?('.significant', '.strip_insignificant_zeros')
+        value == 'true'
+      else
+        value
+      end
+    end
+
+    module_function :to_h, :conflict_keys, :convert_value_type # rubocop:disable Style/AccessModifierDeclarations
   end
 end

data/lib/copy_tuner_client/i18n_backend.rb

@@ -69,17 +69,18 @@ module CopyTunerClient
       key_with_locale = parts.join('.')
       key_without_locale = parts[1..].join('.')
 
-      if CopyTunerClient::configuration.ignored_keys.include?(key_without_locale)
-        CopyTunerClient::configuration.ignored_key_handler.call(IgnoredKey.new("Ignored key: #{key_without_locale}"))
-      end
-
       # NOTE: local_first_key_regexp にマッチするキーは copy_tuner キャッシュをスキップし、
       # ローカル config/locales（I18n::Backend::Simple）を優先する。段階的にローカルへ移行するための仕組み。
       # ローカルに無い場合は nil（未訳）のまま返し、copy_tuner へのフォールバックも空キー登録も行わない（完全分離）。
+      # ignored_keys より先に評価することで、両方にマッチするキーでも確実にローカルへ委譲する。
       if local_first_key?(key_without_locale)
         return super
       end
 
+      if CopyTunerClient::configuration.ignored_keys.include?(key_without_locale)
+        CopyTunerClient::configuration.ignored_key_handler.call(IgnoredKey.new("Ignored key: #{key_without_locale}"))
+      end
+
       # NOTE: ハッシュ化した場合に削除されるキーに対応するため、最初に完全一致をチェック（旧クライアントの動作を維持）
       # 例: `en.test.key` が `en.test.key.conflict` のように別のキーで上書きされている場合の対応
       exact_match = cache[key_with_locale]
@@ -87,10 +88,9 @@ module CopyTunerClient
         return exact_match
       end
 
-      # NOTE: 色々考慮する必要があることが分かったため暫定対応として、ツリーキャッシュを使用しないようにしている
-      # ensure_tree_cache_current
-      # tree_result = lookup_in_tree_cache(parts)
-      # return tree_result if tree_result
+      ensure_tree_cache_current
+      tree_result = lookup_in_tree_cache(parts)
+      return tree_result if tree_result
 
       content = super
 

data/lib/copy_tuner_client/version.rb

@@ -1,6 +1,6 @@
 module CopyTunerClient
   # Client version
-  VERSION = '1.2.1'.freeze
+  VERSION = '1.2.2'.freeze
 
   # API version being used to communicate with the server
   API_VERSION = '2.0'.freeze

data/spec/copy_tuner_client/dotted_hash_spec.rb

@@ -4,25 +4,25 @@ describe CopyTunerClient::DottedHash do
   describe ".to_h" do
     subject { CopyTunerClient::DottedHash.to_h(dotted_hash) }
 
-    context 'empty keys' do
+    context '空のキーの場合' do
       let(:dotted_hash) { {} }
 
       it { is_expected.to eq({}) }
     end
 
-    context 'with single-level keys' do
+    context '1階層のキーの場合' do
       let(:dotted_hash) { { 'key' => 'test value', other_key: 'other value' } }
 
       it { is_expected.to eq({ 'key' => 'test value', 'other_key' => 'other value' }) }
     end
 
-    context 'array of key value pairs' do
+    context 'キーと値の配列の場合' do
       let(:dotted_hash) { [['key', 'test value'], ['other_key', 'other value']] }
 
       it { is_expected.to eq({ 'key' => 'test value', 'other_key' => 'other value' }) }
     end
 
-    context "with multi-level blurb keys" do
+    context "複数階層のblurbキーの場合" do
       let(:dotted_hash) do
         {
           'en.test.key' => 'en test value',
@@ -31,7 +31,7 @@ describe CopyTunerClient::DottedHash do
         }
       end
 
-      it do
+      it "正しくネストされたハッシュに変換されること" do
         is_expected.to eq({
           'en' => {
             'test' => {
@@ -48,7 +48,7 @@ describe CopyTunerClient::DottedHash do
       end
     end
 
-    context "with conflicting keys" do
+    context "キーの競合がある場合" do
       let(:dotted_hash) do
         {
           'en.test' => 'invalid value',
@@ -58,12 +58,86 @@ describe CopyTunerClient::DottedHash do
 
       it { is_expected.to eq({ 'en' => { 'test' => { 'key' => 'en test value' } } }) }
     end
+
+    context "Rails i18nの数値precisionキーの場合" do
+      let(:dotted_hash) do
+        {
+          'en.number.currency.format.precision' => '2',
+          'en.number.format.precision' => '3',
+        }
+      end
+
+      it "precision値を整数に変換する" do
+        is_expected.to eq({
+          'en' => {
+            'number' => {
+              'currency' => {
+                'format' => {
+                  'precision' => 2,
+                },
+              },
+              'format' => {
+                'precision' => 3,
+              },
+            },
+          },
+        })
+      end
+    end
+
+    context "Rails i18nのbooleanキーの場合" do
+      let(:dotted_hash) do
+        {
+          'en.number.currency.format.significant' => 'false',
+          'en.number.format.strip_insignificant_zeros' => 'true',
+        }
+      end
+
+      it "boolean値を実際の真偽値に変換する" do
+        is_expected.to eq({
+          'en' => {
+            'number' => {
+              'currency' => {
+                'format' => {
+                  'significant' => false,
+                },
+              },
+              'format' => {
+                'strip_insignificant_zeros' => true,
+              },
+            },
+          },
+        })
+      end
+    end
+
+    context "Rails i18n以外で似たパターンを含むキーの場合" do
+      let(:dotted_hash) do
+        {
+          'en.custom.precision' => 'custom_value',
+          'en.other.significant_value' => 'true',
+        }
+      end
+
+      it "Rails i18nパターンで終わるキーのみ変換する" do
+        is_expected.to eq({
+          'en' => {
+            'custom' => {
+              'precision' => 0, # .precision suffix triggers conversion
+            },
+            'other' => {
+              'significant_value' => 'true', # no conversion for non-exact match
+            },
+          },
+        })
+      end
+    end
   end
 
   describe ".conflict_keys" do
     subject { CopyTunerClient::DottedHash.conflict_keys(dotted_hash) }
 
-    context 'valid keys' do
+    context '有効なキーの場合' do
       let(:dotted_hash) do
         {
           'ja.hoge.test' => 'test',
@@ -74,7 +148,7 @@ describe CopyTunerClient::DottedHash do
       it { is_expected.to eq({}) }
     end
 
-    context 'invalid keys' do
+    context '無効なキーの場合' do
       let(:dotted_hash) do
         {
           'ja.hoge.test' => 'test',
@@ -85,7 +159,7 @@ describe CopyTunerClient::DottedHash do
         }
       end
 
-      it do
+      it "競合するキーが正しく検出されること" do
         is_expected.to eq({
           'ja.fuga.test' => %w[ja.fuga.test.hoge],
           'ja.hoge.test' => %w[ja.hoge.test.fuga ja.hoge.test.hoge],

data/spec/copy_tuner_client/i18n_backend_spec.rb

@@ -244,7 +244,7 @@ describe 'CopyTunerClient::I18nBackend' do
   end
 
   # NOTE: 色々考慮する必要があることが分かったため暫定対応として、ツリーキャッシュを使用しないようにしている
-  xdescribe 'ツリー構造のlookup' do # rubocop:disable Metrics/BlockLength
+  describe 'ツリー構造のlookup' do # rubocop:disable Metrics/BlockLength
     subject { build_backend }
 
     context '完全一致が存在する場合' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: copy_tuner_client
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.2
 platform: ruby
 authors:
 - SonicGarden