copy_tuner_client 1.2.2 → 1.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4c37179f53ed11fd69930c2b4ff36d6597d0e83ab45e749c5711b7bef254ff2d
-  data.tar.gz: b67cd75f1997c995ba84bee721267b2ceeb1df236907d71c982b5b2058702dc1
+  metadata.gz: f671e8161b117c8d8d46b5ae56810ccd0e2fc88dbe9d35ab2175d303c89df11c
+  data.tar.gz: 5cf906cd134cda913900d2e15e970a094b23f01a03f9914c746b0e2da3ee6c26
 SHA512:
-  metadata.gz: b17a757326468fa69d31d6e86ea684c9f7df6b76b6792429da5cd5d7db0cb0e15392ebdd9f20f66952ebdb7f8770ff3baa636a77f7742134a4053032a2d6e230
-  data.tar.gz: a2df2195d3096ff198893298f968c453564f479a98393f6bfe93f133fcbc414fec7bd004142e6d9f680df28dbebb79535813f79fc273c1b0448906784edfd4d2
+  metadata.gz: 84f11e2d259e496f0638b3b8914cc3f88ceda67c6ae1e497f37192fd55347b0f8948abe115c5b91d33ee8feb3acdfb7994fd0af0b4a3118a64ee763dfc458b89
+  data.tar.gz: 97f296346a8eff7bb3b09ae4b6de6757973ec859e771492962031440626df13548dc25705ebcb355a14e94038444d1223b325c49caff3a1d87f3ec034bde1561

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/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,7 +64,7 @@ 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])
@@ -77,8 +79,9 @@ module CopyTunerClient
         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}"))
+      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: ハッシュ化した場合に削除されるキーに対応するため、最初に完全一致をチェック（旧クライアントの動作を維持）
@@ -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.2'.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/i18n_backend_spec.rb

@@ -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.2
+  version: 1.2.3
 platform: ruby
 authors:
 - SonicGarden