fix: Remove duplicate method definitions in QueryCursor to eliminate warnings

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: yancya

ext/tree_sitter/query_cursor.c

@@ -204,21 +204,16 @@ void init_query_cursor(void) {
 
   rb_define_alloc_func(cQueryCursor, query_cursor_allocate);
 
-  /* Module methods */
-  rb_define_module_function(cQueryCursor, "exec", query_cursor_exec_static, 2);
-
   /* Class methods */
+  rb_define_singleton_method(cQueryCursor, "exec", query_cursor_exec_static, 2);
+
   // Accessors
   DECLARE_ACCESSOR(cQueryCursor, query_cursor, match_limit)
 
   // Other
   rb_define_method(cQueryCursor, "exec", query_cursor_exec, 2);
   rb_define_method(cQueryCursor, "exceed_match_limit?",
                    query_cursor_did_exceed_match_limit, 0);
-  rb_define_method(cQueryCursor, "match_limit", query_cursor_get_match_limit,
-                   0);
-  rb_define_method(cQueryCursor, "match_limit=", query_cursor_set_match_limit,
-                   1);
   rb_define_method(cQueryCursor,
                    "max_start_depth=", query_cursor_set_max_start_depth, 1);
   rb_define_method(cQueryCursor, "next_capture", query_cursor_next_capture, 0);

test/tree_sitter/query_cursor_test.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require_relative '../test_helper'
+
+describe 'QueryCursor' do
+  it 'must not emit redefined warnings when loading the extension' do
+    lib_path = File.expand_path('../../lib', __dir__)
+    output = IO.popen(
+      [RbConfig.ruby, '-W', '-I', lib_path, '-e', "require 'tree_sitter'"],
+      err: %i[child out],
+      &:read
+    )
+    refute_match(/redefined/, output,
+                 "Loading tree_sitter should not produce 'redefined' warnings:\n#{output}")
+  end
+end