copy_tuner_client 1.2.4 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ab3f862812c6f70f5157abe63b5d7cc860037be7b429173b33f5e845fca8d5f
-  data.tar.gz: f14a689fb3bc5e39de2cbe7489b81c5371f502feec84cb3367de3d57ddf2b673
+  metadata.gz: d298d7f59a4cacd30c925e2a0af2b6026fbd3610d4564fae70e5b6efbad4ad0c
+  data.tar.gz: 0f7e09019505a1b49353550b696a2f4a0a4a40eaf74573e4a4e916ec547bd1e9
 SHA512:
-  metadata.gz: ac8dd0b44dc790805421993444bf09e8b28c63a7532796367acfc90e366c7b806e412f6e04d8c451f5c204ebd283e23d15bd9b32ec8912bbbc820e994d184fd4
-  data.tar.gz: bcce2f96d4b748280edba1bf962a7e19f93a42e9af74d37a6b10c914e7e98408c1dcc856e771763c8dededa4aa379828d408da133167dad4c512e52c8a6503b8
+  metadata.gz: 934b99a1624d8dd517c9f2df68c37ffb667128beca73222db458b5f973534398eda88f01c6550c48219e54205fb14b6e47d9731f9556ec0c55d06ce5b47b3729
+  data.tar.gz: 11e6f16e6792603ee6d5c74f4888f293895cf483d10f88d57298bc80bd54a05cb50da5dcdba2c9ade055bf6d57afe66a398b355de806a87a0528dea9e19efaae

data/README.md

@@ -53,6 +53,16 @@ CopyTuner で一元管理している翻訳を、`views.*` のような単位で
 - ローカル YAML にも存在しない場合は未訳（`nil` / MissingTranslation）となります。CopyTuner へのフォールバックや新規キーのアップロードは行いません。これにより移行漏れを未訳として検知できます。
 - マッチしたキーには、ビューヘルパー（`t` / `translate`）および SimpleForm のラベルで CopyRay オーバーレイマーカー（`<!--COPYRAY key-->`）を注入しません。これらのキーは CopyTuner 上で編集できないため、編集可能だと誤認させないためです。
 
+### Rails 標準の数値フォーマットキーは常にローカル優先（組み込み）
+
+`local_first_key_regexp` の設定有無にかかわらず、以下の Rails 標準キーは**常にローカル YAML 優先**になります（CopyTuner をバイパス）。
+
+- `number.format` / `number.currency.format` / `number.percentage.format` / `number.human.format`（およびその配下）
+
+これらは `precision`（整数）や `significant` / `strip_insignificant_zeros`（真偽値）といった非文字列値を含みます。CopyTuner は文字列値しか保持できないため、経由するとこれらが欠落し、`number_to_currency` などが意図しない表示（小数桁数や記号の崩れ）になります。これを防ぐため gem 側で固定的にローカル優先にしています。
+
+アプリ独自の `number.*` キー（例 `number.gift_amount`）は対象外で、従来どおり CopyTuner で管理できます。
+
 `exclude_key_regexp` との違い:
 
 | オプション | 対象 | 作用するタイミング |

data/lib/copy_tuner_client/cache.rb

@@ -56,8 +56,8 @@ module CopyTunerClient
 
       # NOTE: config/locales以下のファイルに除外キーが残っていた場合の対応
       key_without_locale = key.split('.')[1..].join('.')
-      # NOTE: local_first_key_regexp にマッチするキーは copy_tuner と完全分離するためアップロードしない
-      return if @local_first_key_regexp && key_without_locale.match?(@local_first_key_regexp)
+      # NOTE: local_first キー（組み込みの Rails number.*.format + ユーザー設定）は copy_tuner と完全分離するためアップロードしない
+      return if local_first_key?(key_without_locale)
 
       if @ignored_keys.include?(key_without_locale)
         @ignored_key_handler.call(IgnoredKey.new("Ignored key: #{key_without_locale}"))
@@ -165,6 +165,14 @@ module CopyTunerClient
 
     attr_reader :client, :logger
 
+    # NOTE: 組み込みの Rails number.*.format キーは lookup 経路（Configuration#local_first_key?）と
+    # アップロード抑止経路（ここ）で同じ判定を共有する。判定本体は Configuration に集約し付け忘れの穴を防ぐ。
+    def local_first_key?(key_without_locale)
+      return true if Configuration.builtin_local_first_key?(key_without_locale)
+
+      @local_first_key_regexp && key_without_locale.match?(@local_first_key_regexp)
+    end
+
     def with_queued_changes
       changes_to_push = nil
 

data/lib/copy_tuner_client/configuration.rb

@@ -21,6 +21,21 @@ module CopyTunerClient
                  ca_file exclude_key_regexp local_first_key_regexp s3_host locales ignored_keys ignored_key_handler
                  download_cache_dir].freeze
 
+    # NOTE: Rails 標準ロケールで非文字列値（precision: Integer, significant: Boolean,
+    # strip_insignificant_zeros: Boolean）を含むのは number.*.format 配下のみ。store_item が文字列しか
+    # 保持できず lookup_in_tree_cache が tree cache をローカル YAML より優先するため、CopyTuner 経由だと
+    # precision 等が欠落し number_to_currency 等が壊れる。これらの Rails 標準 format キーだけを常にローカル
+    # 優先にする（number 全体ではなく、アプリ独自の number.* キーは対象外）。末尾 (\.|\z) で
+    # number.currency.format（ハッシュ lookup）と number.currency.format.precision（葉キー）の両方に対応する。
+    BUILTIN_LOCAL_FIRST_KEY_REGEXP =
+      /\Anumber\.(format|currency\.format|percentage\.format|human\.format)(\.|\z)/
+
+    # lookup 経路（Configuration#local_first_key?）と upload 抑止経路（Cache#[]=）で同じ組み込み判定を
+    # 共有する。判定を 1 箇所に集約することで number 以外を足す際の同期漏れを防ぐ。
+    def self.builtin_local_first_key?(key_without_locale)
+      key_without_locale.to_s.match?(BUILTIN_LOCAL_FIRST_KEY_REGEXP)
+    end
+
     # @return [String] The API key for your project, found on the project edit form.
     attr_reader :api_key
 
@@ -363,9 +378,13 @@ module CopyTunerClient
     # @param key_without_locale [String, Symbol, nil] locale prefix を除いたキー（例: "views.foo.bar"）
     # @return [Boolean]
     def local_first_key?(key_without_locale)
-      return false if local_first_key_regexp.nil? || key_without_locale.nil?
+      return false if key_without_locale.nil?
+
+      normalized = key_without_locale.to_s
+      return true if self.class.builtin_local_first_key?(normalized)
+      return false if local_first_key_regexp.nil?
 
-      key_without_locale.to_s.match?(local_first_key_regexp)
+      normalized.match?(local_first_key_regexp)
     end
 
     private

data/lib/copy_tuner_client/version.rb

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

data/skills/copy-tuner/SKILL.md

@@ -20,6 +20,11 @@ license: MIT
 | マッチする | config/locales（copy_tuner と完全分離） | `config/locales/*.yml` を Read / Edit |
 | マッチしない・未設定 | copy_tuner | MCP ツール（下記） |
 
+**例外（gem 組み込み・設定不要で常に config/locales 管理）:** Rails 標準の `number.format` /
+`number.currency.format` / `number.percentage.format` / `number.human.format`（およびその配下）。
+`precision` 等の非文字列値を含み copy_tuner では正しく扱えないため、gem が常にバイパスする。
+これらは `config/locales/*.yml` を見る・編集すること（アプリ独自の `number.gift_amount` 等は対象外で通常どおり copy_tuner）。
+
 **config/locales 管理キーの落とし穴:**
 - copy_tuner には存在しないので、MCP ツールで探しても見つからない。`config/locales` を見ること。
 - copy_tuner に登録しても lookup でバイパスされ無効。必ず `.yml` 側に書く。

data/spec/copy_tuner_client/cache_spec.rb

@@ -55,6 +55,18 @@ describe 'CopyTunerClient::Cache' do
     expect(cache.queued.keys).to match_array(%w[ja.views.foo ja.messages.bar])
   end
 
+  it 'Rails標準の number.*.format キーは設定なしでもアップロードしないこと' do
+    cache = build_cache(local_first_key_regexp: nil)
+    cache['ja.number.currency.format.unit'] = '$'
+    cache['ja.number.gift_amount']          = 'app key'
+    cache['ja.messages.bar']                = 'copy tuner value'
+
+    cache.download
+
+    # number.*.format のみ抑止。アプリ独自 number キーと通常キーは従来どおりアップロード
+    expect(cache.queued.keys).to match_array(%w[ja.number.gift_amount ja.messages.bar])
+  end
+
   it '変更がない場合はアップロードしないこと' do
     cache = build_cache
     cache.flush

data/spec/copy_tuner_client/configuration_spec.rb

@@ -217,6 +217,40 @@ describe CopyTunerClient::Configuration do
         expect(config.local_first_key?(:'views.foo')).to eq true
       end
     end
+
+    # NOTE: Rails 標準の number.*.format 配下は precision 等の非文字列値を含み CopyTuner 経由だと壊れるため、
+    # ユーザー設定の有無によらず常にローカル優先（組み込み判定）になる
+    context 'with built-in Rails number format keys' do
+      it 'returns true for built-in number format keys even when local_first_key_regexp is nil' do
+        expect(config.local_first_key?('number.format')).to eq true
+        expect(config.local_first_key?('number.currency.format')).to eq true
+        expect(config.local_first_key?('number.currency.format.precision')).to eq true
+        expect(config.local_first_key?('number.percentage.format')).to eq true
+        expect(config.local_first_key?('number.human.format.significant')).to eq true
+      end
+
+      it 'returns false for app-defined number keys (not Rails format subtrees)' do
+        expect(config.local_first_key?('number.gift_amount')).to eq false
+        expect(config.local_first_key?('number.my_currency.unit')).to eq false
+      end
+
+      it 'returns false for string-only number subtrees and non-number keys' do
+        expect(config.local_first_key?('number.human.storage_units.units.byte.one')).to eq false
+        expect(config.local_first_key?('date.formats.default')).to eq false
+        expect(config.local_first_key?('time.formats.short')).to eq false
+        expect(config.local_first_key?('datetime.distance_in_words.x')).to eq false
+        expect(config.local_first_key?('views.foo')).to eq false
+        expect(config.local_first_key?('numbers.foo')).to eq false
+      end
+
+      it 'keeps protecting built-in keys without breaking a user-set regexp' do
+        config.local_first_key_regexp = /\Aviews\./
+
+        expect(config.local_first_key?('number.currency.format')).to eq true
+        expect(config.local_first_key?('views.foo')).to eq true
+        expect(config.local_first_key?('models.foo')).to eq false
+      end
+    end
   end
 
   describe '#exclude_key_regexp= (deprecated)' do

data/spec/copy_tuner_client/i18n_backend_spec.rb

@@ -445,7 +445,7 @@ describe 'CopyTunerClient::I18nBackend' do
     end
   end
 
-  describe 'local_first_key_regexp（ローカル優先キー）' do
+  describe 'local_first_key_regexp（ローカル優先キー）' do # rubocop:disable Metrics/BlockLength
     after { CopyTunerClient.configuration.local_first_key_regexp = nil }
 
     # ローカル config/locales 相当のデータを I18n::Backend::Simple 側だけに格納する。
@@ -468,6 +468,24 @@ describe 'CopyTunerClient::I18nBackend' do
 
         expect(subject.translate('ja', 'views.foo')).to eq('copy tuner value')
       end
+
+      # NOTE: number.*.format は precision 等の非文字列値が store_item で落ちるため、
+      # tree cache を優先するとローカル YAML の正しい format が壊れる。組み込み判定で常にローカル優先にする。
+      it 'number.*.format は cache に値があってもローカル YAML を優先すること' do
+        CopyTunerClient.configuration.local_first_key_regexp = nil
+        cache['ja.number.currency.format.unit'] = '$'
+        store_local(subject, :ja, number: { currency: { format: { unit: '円' } } })
+
+        expect(subject.translate('ja', 'number.currency.format')).to eq(unit: '円')
+      end
+
+      it 'アプリ独自の number キーは従来どおり copy_tuner を優先すること' do
+        CopyTunerClient.configuration.local_first_key_regexp = nil
+        cache['ja.number.gift_amount'] = 'copy tuner value'
+        store_local(subject, :ja, number: { gift_amount: 'local value' })
+
+        expect(subject.translate('ja', 'number.gift_amount')).to eq('copy tuner value')
+      end
     end
 
     context 'local_first_key_regexp = /\Aviews\./ のとき' do

metadata

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