copy_tuner_client 1.2.1 → 1.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 70c29e81f0f1972e5b8833b0ab7a9f9b079186f055e7d988b207043930144708
-  data.tar.gz: 3277c0fc92c99421c183f19356ea49f8830df54c8bfdf2913c1433a4e30e6268
+  metadata.gz: f671e8161b117c8d8d46b5ae56810ccd0e2fc88dbe9d35ab2175d303c89df11c
+  data.tar.gz: 5cf906cd134cda913900d2e15e970a094b23f01a03f9914c746b0e2da3ee6c26
 SHA512:
-  metadata.gz: f7ca85e2ebd4d5f8634edc0a2b6c4ac1450d35c49e4bd81740c0bf31166ee8937c1cf29a4af4bda9ba5c1bf30d1985282635c9092a8ee2c6b931a0fcdb78c9f2
-  data.tar.gz: 680d69dcfee71fb565c08a948a41fae83946fd407045e50e4169266fdb83ef366dbb0c4a16326206ef1e7dae646e3d7a375bf98c88e3d5d8217bb1d755acbff4
+  metadata.gz: 84f11e2d259e496f0638b3b8914cc3f88ceda67c6ae1e497f37192fd55347b0f8948abe115c5b91d33ee8feb3acdfb7994fd0af0b4a3118a64ee763dfc458b89
+  data.tar.gz: 97f296346a8eff7bb3b09ae4b6de6757973ec859e771492962031440626df13548dc25705ebcb355a14e94038444d1223b325c49caff3a1d87f3ec034bde1561

data/.vscode/settings.json

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

data/CLAUDE.md

@@ -18,6 +18,11 @@ CopyTuner の Ruby クライアント gem。Rails アプリの I18n を CopyTune
 - Rack middleware `RequestSync` / `CopyrayMiddleware` — 開発環境でのリクエスト毎同期とオーバーレイ
 Rails 統合は engine.rb のイニシャライザ経由（ヘルパー/SimpleForm フック、アセット precompile）。
 
+## 開発スタイル
+- **RED/TDD で進める**: 実装前に必ず失敗するテストを書き、テストが RED になることを確認してから実装する
+- 新機能・バグ修正ともに「テスト追加 → RED 確認 → 実装 → GREEN」のサイクルを守る
+- テストを書かずに実装を先行させない
+
 ## Gotchas
 - **フロントエンドは `src/*.ts` を編集する。`app/assets/*` は Vite のビルド成果物なので直接編集しない**
   （vite.config.ts が `src/main.ts` → `app/assets/javascripts/copytuner.js` を出力）。

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

@@ -21,12 +21,13 @@ module CopyTunerClient
       @cache = cache
       @tree_cache = nil
       @cache_version = nil
-    end    # Translates the given local and key. See the I18n API documentation for details.
+    end
+
     #
     # @return [Object] the translated key (usually a String)
     def translate(locale, key, options = {})
       # I18nの標準処理に任せる（内部でlookupが呼ばれる）
-      content = super(locale, key, options)
+      content = super
 
       return content if content.nil? || content.is_a?(Hash)
 
@@ -43,8 +44,9 @@ module CopyTunerClient
     # @return [Array<String>] available locales
     def available_locales
       return @available_locales if defined?(@available_locales)
+
       cached_locales = cache.keys.map { |key| key.split('.').first }
-      @available_locales = (cached_locales + super).uniq.map { |locale| locale.to_sym }
+      @available_locales = (cached_locales + super).uniq.map(&:to_sym)
     end
 
     # Stores the given translations.
@@ -62,24 +64,26 @@ module CopyTunerClient
 
     private
 
-    def lookup(locale, key, scope = [], options = {})
+    def lookup(locale, key, scope = [], options = {}) # rubocop:disable Metrics/CyclomaticComplexity,Metrics/MethodLength
       return nil if !key.is_a?(String) && !key.is_a?(Symbol)
 
       parts = I18n.normalize_keys(locale, key, scope, options[:separator])
       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
 
+      config = CopyTunerClient.configuration
+      if config.ignored_keys.include?(key_without_locale)
+        config.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 +91,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
 
@@ -114,7 +117,9 @@ module CopyTunerClient
     def lookup_in_tree_cache(keys)
       return nil if @tree_cache.nil?
 
-      symbol_keys = keys.map(&:to_sym)
+      # NOTE: keys は I18n.normalize_keys 済みの配列で、数字だけのセグメントは Integer になっている
+      # （例: "...body_temperature.36.5" は [..., 36, 5] に分割される）。Integer#to_sym は無いため to_s を経由する。
+      symbol_keys = keys.map { |k| k.to_s.to_sym }
       begin
         result = @tree_cache.dig(*symbol_keys)
         result.is_a?(Hash) ? result : nil
@@ -146,21 +151,22 @@ module CopyTunerClient
     end
 
     def default(locale, object, subject, options = {})
-      content = super(locale, object, subject, options)
+      content = super
       return content if !object.is_a?(String) && !object.is_a?(Symbol)
 
       if content.respond_to?(:to_str)
         parts = I18n.normalize_keys(locale, object, options[:scope], options[:separator])
         # NOTE: ActionView::Helpers::TranslationHelper#translate wraps default String in an Array
         # NOTE: local_first キーのアップロード抑止は Cache#[]= 側に集約している
-        if subject.is_a?(String) || (subject.is_a?(Array) && subject.size == 1 && subject.first.is_a?(String))
-          key = parts.join('.')
-          cache[key] = content.to_str
-        end
+        cache[parts.join('.')] = content.to_str if default_string_subject?(subject)
       end
       content
     end
 
+    def default_string_subject?(subject)
+      subject.is_a?(String) || (subject.is_a?(Array) && subject.size == 1 && subject.first.is_a?(String))
+    end
+
     attr_reader :cache
   end
 end

data/lib/copy_tuner_client/version.rb

@@ -1,6 +1,6 @@
 module CopyTunerClient
   # Client version
-  VERSION = '1.2.1'.freeze
+  VERSION = '1.2.3'.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
@@ -425,6 +425,23 @@ describe 'CopyTunerClient::I18nBackend' do
         result = subject.translate('ja', 'hoge.hello', default: nil)
         expect(result).to be_nil
       end
+
+      it '数値セグメントを含むキー（数値enum）でクラッシュしないこと' do
+        # NOTE: enumerize の Float range など、normalize_keys が末尾を Integer に分割するキー。
+        # exact_match（完全一致）を回避するため別キーを格納し、tree_cache 探索経路を必ず通す。
+        cache['ja.dummy'] = 'dummy'
+
+        expect {
+          subject.translate('ja', 'enumerize.infection_control.body_temperature.36.5', default: nil)
+        }.not_to raise_error
+      end
+
+      it '数値セグメントキーが存在しても通常キーのlookupが壊れないこと' do
+        cache['ja.views.hoge'] = 'normal'
+
+        expect { subject.translate('ja', 'enumerize.body_temperature.36.5', default: nil) }.not_to raise_error
+        expect(subject.translate('ja', 'views.hoge')).to eq('normal')
+      end
     end
   end
 

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.3
 platform: ruby
 authors:
 - SonicGarden