copy_tuner_client 1.2.4 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ab3f862812c6f70f5157abe63b5d7cc860037be7b429173b33f5e845fca8d5f
-  data.tar.gz: f14a689fb3bc5e39de2cbe7489b81c5371f502feec84cb3367de3d57ddf2b673
+  metadata.gz: 10935db209e8b915ac8101dfa324bf5a40da15978124e8300746c0be13407862
+  data.tar.gz: faa6e39b7149540c342ec9fcefce9f0c47425614c4f361fa8b8b597ff932d69c
 SHA512:
-  metadata.gz: ac8dd0b44dc790805421993444bf09e8b28c63a7532796367acfc90e366c7b806e412f6e04d8c451f5c204ebd283e23d15bd9b32ec8912bbbc820e994d184fd4
-  data.tar.gz: bcce2f96d4b748280edba1bf962a7e19f93a42e9af74d37a6b10c914e7e98408c1dcc856e771763c8dededa4aa379828d408da133167dad4c512e52c8a6503b8
+  metadata.gz: 83ea46e846b20d110366e5d8d7ecb5c415a082a700e45682931e2303a7516e50c6d1b0fcdce5ea7deada42dd3ef7e6f9df07ff4084c3b211db0cb42d1de1d422
+  data.tar.gz: 194dc1188ab23b9776b030ec08c453444513a8f55d6b674d4b06b6953c652409951690bbfad569b1e2524a02c7951f5faeb55a4fe0d1eafbfd77c61d72cb2a45

data/.github/dependabot.yml

@@ -0,0 +1,18 @@
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    cooldown:
+      default-days: 5
+    ignore:
+      - dependency-name: "ruby/setup-ruby"
+        update-types:
+          - "version-update:semver-patch"
+          - "version-update:semver-minor"
+      - dependency-name: "actions/checkout"
+        update-types:
+          - "version-update:semver-patch"
+          - "version-update:semver-minor"
+    schedule:
+      # Check for updates to GitHub Actions every weekday
+      interval: "weekly"

data/.github/workflows/rspec.yml

@@ -22,14 +22,12 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: ["3.1", "3.2", "3.3", "3.4"]
-        gemfile: ["7.1", "7.2", "8.0", "main"]
+        ruby: ["3.3", "3.4", "4.0"]
+        gemfile: ["8.0", "8.1", "main"]
         exclude:
-          - ruby: "3.1"
-            gemfile: "8.0"
-          - ruby: "3.1"
+          - ruby: "3.3"
             gemfile: "main"
-          - ruby: "3.2"
+          - ruby: "3.4"
             gemfile: "main"
     steps:
       - name: Install packages
@@ -37,7 +35,7 @@ jobs:
           sudo apt update -y
           sudo apt install -y libsqlite3-dev
 
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v6
 
       - name: Set up Ruby
         uses: ruby/setup-ruby@v1

data/.rubocop.yml

@@ -1,2 +1,7 @@
 inherit_gem:
   sgcop: ruby/rubocop.yml
+
+# .ruby-version が RuboCop 未対応の新しい Ruby を指すと解析が落ちるため、
+# サポート最低バージョンを明示する
+AllCops:
+  TargetRubyVersion: 3.3

data/.ruby-version

@@ -1 +1 @@
-3.4.5
+4.0.5

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/copy_tuner_client.gemspec

@@ -2,11 +2,11 @@ $LOAD_PATH.push File.expand_path('lib', __dir__)
 require 'copy_tuner_client/version'
 
 Gem::Specification.new do |s|
-  s.required_ruby_version = '>= 3.1.0'
+  s.required_ruby_version = '>= 3.3.0'
   s.add_dependency 'i18n', '>= 0.5.0'
   s.add_dependency 'json'
   s.add_dependency 'nokogiri'
-  s.add_development_dependency 'rails', '~> 7.1'
+  s.add_development_dependency 'rails', '~> 8.0'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec'
   s.add_development_dependency 'sham_rack'

data/gemfiles/{7.1.gemfile → 8.1.gemfile}

@@ -1,5 +1,5 @@
 source "http://rubygems.org"
 
-gem "rails", "~> 7.1"
+gem "rails", "~> 8.1"
 
 gemspec :path => "../"

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/dotted_hash.rb

@@ -3,9 +3,7 @@ module CopyTunerClient
     def to_h(dotted_hash)
       hash = {}
       dotted_hash.to_h.transform_keys(&:to_s).sort.each do |key, value|
-        # Rails i18n標準との互換性のため、特定のキーを適切な型に変換
-        converted_value = convert_value_type(key, value)
-        _hash = key.split('.').reverse.inject(converted_value) { |memo, _key| { _key => memo } }
+        _hash = key.split('.').reverse.inject(value) { |memo, _key| { _key => memo } }
         hash.deep_merge!(_hash)
       end
       hash
@@ -29,22 +27,6 @@ module CopyTunerClient
       results
     end
 
-    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
+    module_function :to_h, :conflict_keys # rubocop:disable Style/AccessModifierDeclarations
   end
 end

data/lib/copy_tuner_client/version.rb

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

data/skills/copy-tuner/SKILL.md

@@ -18,7 +18,18 @@ license: MIT
 | 判別 | 管理場所 | 操作 |
 |---|---|---|
 | マッチする | config/locales（copy_tuner と完全分離） | `config/locales/*.yml` を Read / Edit |
-| マッチしない・未設定 | copy_tuner | MCP ツール（下記） |
+| マッチしない・未設定 | copy_tuner | **MCP ツール経由でのみ探す**（下記） |
+
+**copy_tuner 管理のキーは必ず MCP ツール経由で探すこと。** ローカルの `.yml` を grep して見つけたつもりになってはならない。
+
+**例外（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` 以外の export 済み yml は参照禁止:**
+環境によっては copy_tuner から export したキャッシュ用の `.yml`（`config/locales` 配下以外、例: アプリ直下や `tmp/`、`db/` などに置かれた export ファイル）が存在することがある。
+これらは古いスナップショットであり権威ある情報源ではない。**読んで判断材料にしてはならない。** copy_tuner 管理キーは常に MCP ツールで最新を確認する。
 
 **config/locales 管理キーの落とし穴:**
 - copy_tuner には存在しないので、MCP ツールで探しても見つからない。`config/locales` を見ること。
@@ -33,7 +44,8 @@ license: MIT
 | キー名（一部）から翻訳を調べる | `search_key`（引数は英語キーワード） |
 | 画面テキストからキーを逆引き | `search_translations` |
 | 新しいキーを登録する | `create_i18n_key` |
-| 既存キーの翻訳を更新する | `get_edit_url`（ブラウザ編集） |
+| 既存キーのドラフト訳を更新する | `update_i18n_key` |
+| 既存キーの翻訳を更新する（ブラウザ） | `get_edit_url` |
 | 使用中ロケールを確認 | `get_locales` |
 
 ## 新規キー登録
@@ -41,6 +53,19 @@ license: MIT
 1. `search_key` で関連キーを検索し、既存の命名パターンに合わせる（例: `search_key("move_out")`）。
 2. ja は必須。他は `get_locales` で確認し、存在するロケール分を登録する。
 
+## 既存キーの訳を更新する
+
+`update_i18n_key` を使う。`key` と `translations`（ロケール・値のペア配列）は両方必須。
+
+**制約**: ドラフト状態または未発行のキーのみ更新可能。発行済み（published）かどうかを事前に知る手段はなく、API にリクエストしてエラーが返るかどうかで初めてわかる。
+
+**published キーへの対応方針**: 値を直接書き換えるのはリスクが高いため、原則として以下の手順を推奨する:
+1. 新しいキーにバージョン番号を付けて登録する（例: `views.foo_v2`）→ **新規キー登録の手順に従う**
+2. コード側の参照を新キーに差し替える
+3. 古いキーを `config/initializers/copy_tuner.rb` の `ignored_keys` に追加する
+
+どうしても既存キーを書き換えたい場合は `get_edit_url` でブラウザから編集する。
+
 ## 不要なキーの処理
 
 削除せず `config/initializers/copy_tuner.rb` の `ignored_keys` に追加する。

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/dotted_hash_spec.rb

@@ -59,75 +59,22 @@ describe CopyTunerClient::DottedHash do
       it { is_expected.to eq({ 'en' => { 'test' => { 'key' => 'en test value' } } }) }
     end
 
-    context "Rails i18nの数値precisionキーの場合" do
+    # NOTE: number.*.format 配下のキー（precision 等）は I18nBackend の local_first_key? ガードで
+    # tree_cache をバイパスしローカル YAML 優先になるため、ここで型変換せず文字列のまま保持する。
+    # （number_to_currency が壊れないことは i18n_backend_spec の number ローカル優先テストで担保）
+    context "number.*.format 配下の値を含む場合" 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
+      it "型変換せず値を文字列のまま保持する" do
         is_expected.to eq({
           'en' => {
-            'custom' => {
-              'precision' => 0, # .precision suffix triggers conversion
-            },
-            'other' => {
-              'significant_value' => 'true', # no conversion for non-exact match
-            },
+            'number' => { 'currency' => { 'format' => { 'precision' => '2' } } },
+            'custom' => { 'precision' => 'custom_value' },
           },
         })
       end

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.4.0
 platform: ruby
 authors:
 - SonicGarden
@@ -57,14 +57,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '8.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '8.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -185,6 +185,7 @@ files:
 - ".babelrc"
 - ".eslintrc.js"
 - ".gitattributes"
+- ".github/dependabot.yml"
 - ".github/workflows/rspec.yml"
 - ".gitignore"
 - ".rspec"
@@ -200,9 +201,8 @@ files:
 - app/assets/javascripts/copytuner.js
 - app/assets/stylesheets/copytuner.css
 - copy_tuner_client.gemspec
-- gemfiles/7.1.gemfile
-- gemfiles/7.2.gemfile
 - gemfiles/8.0.gemfile
+- gemfiles/8.1.gemfile
 - gemfiles/main.gemfile
 - index.html
 - lib/copy_tuner_client.rb
@@ -288,14 +288,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.1.0
+      version: 3.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.10
 specification_version: 4
 summary: Client for the CopyTuner copy management service
 test_files: []

data/gemfiles/7.2.gemfile

@@ -1,5 +0,0 @@
-source "http://rubygems.org"
-
-gem "rails", "~> 7.2"
-
-gemspec :path => "../"