Remove JSON::State#[] and JSON::State#[]=

Author: byroot

CHANGES.md

@@ -8,6 +8,7 @@
 - Removed `JSON.load_default_options`.
 - Removed `JSON.unsafe_load_default_options`.
 - Removed `JSON.dump_default_options`.
+- Removed `JSON::State#[]` and `JSON::State#[]=`.
 
 ### 2026-07-13 (2.21.1)
 

java/src/json/ext/GeneratorState.java

@@ -270,30 +270,6 @@ public IRubyObject generate(ThreadContext context, IRubyObject obj) {
         return generate(context, obj, context.nil);
     }
 
-    @JRubyMethod(name="[]")
-    public IRubyObject op_aref(ThreadContext context, IRubyObject vName) {
-        String name = vName.asJavaString();
-        if (getMetaClass().isMethodBound(name, true)) {
-            return send(context, vName, Block.NULL_BLOCK);
-        } else {
-            IRubyObject value = getInstanceVariables().getInstanceVariable("@" + name);
-            return value == null ? context.nil : value;
-        }
-    }
-
-    @JRubyMethod(name="[]=")
-    public IRubyObject op_aset(ThreadContext context, IRubyObject vName, IRubyObject value) {
-        checkFrozen();
-        String name = vName.asJavaString();
-        String nameWriter = name + "=";
-        if (getMetaClass().isMethodBound(nameWriter, true)) {
-            return send(context, context.runtime.newString(nameWriter), value, Block.NULL_BLOCK);
-        } else {
-            getInstanceVariables().setInstanceVariable("@" + name, value);
-        }
-        return context.nil;
-    }
-
     public ByteList getIndent() {
         return indent;
     }

lib/json/ext/generator/state.rb

@@ -71,33 +71,6 @@ def to_h
         end
 
         alias_method :to_hash, :to_h
-
-        # call-seq: [](name)
-        #
-        # Returns the value returned by method +name+.
-        def [](name)
-          ::JSON.deprecation_warning("JSON::State#[] is deprecated and will be removed in json 3.0.0")
-
-          if respond_to?(name)
-            __send__(name)
-          else
-            instance_variable_get("@#{name}") if
-              instance_variables.include?("@#{name}".to_sym) # avoid warning
-          end
-        end
-
-        # call-seq: []=(name, value)
-        #
-        # Sets the attribute name to value.
-        def []=(name, value)
-          ::JSON.deprecation_warning("JSON::State#[]= is deprecated and will be removed in json 3.0.0")
-
-          if respond_to?(name_writer = "#{name}=")
-            __send__ name_writer, value
-          else
-            instance_variable_set "@#{name}", value
-          end
-        end
       end
     end
   end

test/json/json_generator_test.rb

@@ -287,10 +287,6 @@ def test_states
     json = generate({1=>2}, nil)
     assert_equal('{"1":2}', json)
     s = JSON.state.new
-    assert s.check_circular?
-    assert_deprecated_warning(/JSON::State/) do
-      assert s[:check_circular?]
-    end
     h = { 1=>2 }
     h[3] = h
     assert_raise(JSON::NestingError) {  generate(h) }
@@ -299,10 +295,6 @@ def test_states
     a = [ 1, 2 ]
     a << a
     assert_raise(JSON::NestingError) {  generate(a, s) }
-    assert s.check_circular?
-    assert_deprecated_warning(/JSON::State/) do
-      assert s[:check_circular?]
-    end
   end
 
   def test_falsy_state
@@ -578,35 +570,6 @@ def to_s
     bignum.class.define_method(:to_s, original_to_s) if original_to_s
   end
 
-  def test_hash_likeness_set_symbol
-    assert_deprecated_warning(/JSON::State/) do
-      state = JSON.state.new
-      assert_equal nil, state[:foo]
-      assert_equal nil.class, state[:foo].class
-      assert_equal nil, state['foo']
-      state[:foo] = :bar
-      assert_equal :bar, state[:foo]
-      assert_equal :bar, state['foo']
-      state_hash = state.to_hash
-      assert_kind_of Hash, state_hash
-      assert_equal :bar, state_hash[:foo]
-    end
-  end
-
-  def test_hash_likeness_set_string
-    assert_deprecated_warning(/JSON::State/) do
-      state = JSON.state.new
-      assert_equal nil, state[:foo]
-      assert_equal nil, state['foo']
-      state['foo'] = :bar
-      assert_equal :bar, state[:foo]
-      assert_equal :bar, state['foo']
-      state_hash = state.to_hash
-      assert_kind_of Hash, state_hash
-      assert_equal :bar, state_hash[:foo]
-    end
-  end
-
   def test_json_state_to_h_roundtrip
     state = JSON.state.new
     assert_equal state.to_h, JSON.state.new(state.to_h).to_h