copy_tuner_client 1.2.2 → 1.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4c37179f53ed11fd69930c2b4ff36d6597d0e83ab45e749c5711b7bef254ff2d
-  data.tar.gz: b67cd75f1997c995ba84bee721267b2ceeb1df236907d71c982b5b2058702dc1
+  metadata.gz: 4ab3f862812c6f70f5157abe63b5d7cc860037be7b429173b33f5e845fca8d5f
+  data.tar.gz: f14a689fb3bc5e39de2cbe7489b81c5371f502feec84cb3367de3d57ddf2b673
 SHA512:
-  metadata.gz: b17a757326468fa69d31d6e86ea684c9f7df6b76b6792429da5cd5d7db0cb0e15392ebdd9f20f66952ebdb7f8770ff3baa636a77f7742134a4053032a2d6e230
-  data.tar.gz: a2df2195d3096ff198893298f968c453564f479a98393f6bfe93f133fcbc414fec7bd004142e6d9f680df28dbebb79535813f79fc273c1b0448906784edfd4d2
+  metadata.gz: ac8dd0b44dc790805421993444bf09e8b28c63a7532796367acfc90e366c7b806e412f6e04d8c451f5c204ebd283e23d15bd9b32ec8912bbbc820e994d184fd4
+  data.tar.gz: bcce2f96d4b748280edba1bf962a7e19f93a42e9af74d37a6b10c914e7e98408c1dcc856e771763c8dededa4aa379828d408da133167dad4c512e52c8a6503b8

data/CLAUDE.md

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

data/lib/copy_tuner_client/cache.rb

@@ -7,7 +7,7 @@ module CopyTunerClient
   # this class directly.
   #
   # Responsible for locking down access to data used by both threads.
-  class Cache
+  class Cache # rubocop:disable Metrics/ClassLength
     STATUS_NOT_READY = :not_ready
     STATUS_PENDING = :pending
     STATUS_READY = :ready
@@ -33,6 +33,10 @@ module CopyTunerClient
       @status = STATUS_NOT_READY
     end
 
+    # blank_keys を公開しているのは、MCP ツール等の外部利用者が「キーは登録済みだが翻訳なし」と
+    # 「キー未登録」を区別するため（このリポジトリ内では参照箇所がない）。
+    attr_reader :last_downloaded_at, :last_uploaded_at, :queued, :blurbs, :blank_keys
+
     # Returns content for the given blurb.
     # @param key [String] the key of the desired blurb
     # @return [String] the contents of the blurb
@@ -121,10 +125,12 @@ module CopyTunerClient
       @status = STATUS_PENDING unless ready?
 
       res = client.download(cache_fallback: pending?) do |downloaded_blurbs|
-        blank_blurbs, blurbs = downloaded_blurbs.partition { |_key, value| value == '' }
+        blank_keys = Set.new
+        blurbs = {}
+        downloaded_blurbs.each { |key, value| value == '' ? blank_keys << key : blurbs[key] = value }
         lock do
-          @blank_keys = Set.new(blank_blurbs.map(&:first))
-          @blurbs = blurbs.to_h
+          @blank_keys = blank_keys
+          @blurbs = blurbs
         end
       end
 
@@ -143,8 +149,6 @@ module CopyTunerClient
       flush
     end
 
-    attr_reader :last_downloaded_at, :last_uploaded_at, :queued, :blurbs
-
     def inspect
       "#<CopyTunerClient::Cache:#{object_id}>"
     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,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.4'.freeze
 
   # API version being used to communicate with the server
   API_VERSION = '2.0'.freeze

data/spec/copy_tuner_client/cache_spec.rb

@@ -136,6 +136,7 @@ describe 'CopyTunerClient::Cache' do
 
     expect(cache['en.test.key']).to eq('test value')
     expect(cache['en.test.empty']).to eq(nil)
+    expect(cache.blank_keys).to contain_exactly('en.test.empty')
 
     cache['en.test.empty'] = ''
     expect(cache.queued).to be_empty

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