refactor: Format code with RuboCop

Author: wudidapaopao

.rubocop.yml

@@ -5,9 +5,11 @@ AllCops:
 
   NewCops: enable
 
-  TargetRubyVersion: 3.4
+  TargetRubyVersion: 3.1
 
   AutoCorrect: true
+  
+  SuggestExtensions: false
 
 Style:
   StringLiterals:
@@ -27,11 +29,10 @@ Layout:
     Max: 120
 
   FirstParameterIndentation:
-   恩abled: true
+    Enabled: true
     EnforcedStyle: indented
 
-Indentation:
-  Width:
+  Indentation:
     Width: 2
 
 Metrics:

Gemfile

@@ -1,15 +1,17 @@
-source "https://rubygems.org"
+# frozen_string_literal: true
+
+source 'https://rubygems.org'
 
 gemspec
 
 group :test do
-  gem "rspec", "3.12.0"
-  gem "ruby_memcheck", "3.0.1" if Gem::Platform.local.os == "linux"
-  gem "rake-compiler", "1.2.9"
-  gem "rake-compiler-dock", "1.9.1"
+  gem 'rake-compiler', '1.2.9'
+  gem 'rake-compiler-dock', '1.9.1'
+  gem 'rspec', '3.12.0'
+  gem 'ruby_memcheck', '3.0.1' if Gem::Platform.local.os == 'linux'
 end
 
 group :development do
-  gem "rdoc", "6.12.0"
-  gem "rubocop", "1.59.0", require: false
+  gem 'rdoc', '6.12.0'
+  gem 'rubocop', '1.74.0', require: false
 end

Rakefile

@@ -1,17 +1,17 @@
 # frozen_string_literal: true
 
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
-require "rake/extensiontask"
-require "rubocop/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'rake/extensiontask'
+require 'rubocop/rake_task'
 
 # require "bundler"
-CHDB_SPEC = Bundler.load_gemspec("chdb.gemspec")
+CHDB_SPEC = Bundler.load_gemspec('chdb.gemspec')
 
 RSpec::Core::RakeTask.new(:spec)
 
-Rake::ExtensionTask.new("chdb") do |ext|
-  ext.lib_dir = "lib/chdb"
+Rake::ExtensionTask.new('chdb') do |ext|
+  ext.lib_dir = 'lib/chdb'
 end
 
 RuboCop::RakeTask.new

chdb.gemspec

@@ -1,56 +1,58 @@
+# frozen_string_literal: true
+
 begin
-  require_relative "lib/chdb/version"
+  require_relative 'lib/chdb/version'
 rescue LoadError
-  puts "WARNING: could not load ChDB::VERSION"
+  puts 'WARNING: could not load ChDB::VERSION'
 end
 
 Gem::Specification.new do |s|
-  s.name = "chdb"
-  s.version = defined?(ChDB::VERSION) ? ChDB::VERSION : "0.0.0"
+  s.name = 'chdb'
+  s.version = defined?(ChDB::VERSION) ? ChDB::VERSION : '0.0.0'
 
-  s.summary = "Ruby library to interface with the chDB database engine (https://clickhouse.com/docs/chdb)."
+  s.summary = 'Ruby library to interface with the chDB database engine (https://clickhouse.com/docs/chdb).'
   s.description = <<~TEXT
     Ruby library to interface with the chDB database engine (https://clickhouse.com/docs/chdb). Precompiled
     binaries are available for common platforms for recent versions of Ruby.
   TEXT
 
-  s.authors = ["Xiaozhe Yu", "Auxten Wang"]
+  s.authors = ['Xiaozhe Yu', 'Auxten Wang']
 
-  s.licenses = ["Apache-2.0"]
+  s.licenses = ['Apache-2.0']
 
-  s.required_ruby_version = Gem::Requirement.new(">= 3.1")
+  s.required_ruby_version = Gem::Requirement.new('>= 3.1')
 
-  s.homepage = "https://github.com/chdb-io/chdb-ruby"
+  s.homepage = 'https://github.com/chdb-io/chdb-ruby'
   s.metadata = {
-    "homepage_uri" => "https://github.com/chdb-io/chdb-ruby",
-    "bug_tracker_uri" => "https://github.com/chdb-io/chdb-ruby/issues",
-    "changelog_uri" => "https://github.com/chdb-io/chdb-ruby/blob/main/CHANGELOG.md",
-    "source_code_uri" => "https://github.com/chdb-io/chdb-ruby",
+    'homepage_uri' => 'https://github.com/chdb-io/chdb-ruby',
+    'bug_tracker_uri' => 'https://github.com/chdb-io/chdb-ruby/issues',
+    'changelog_uri' => 'https://github.com/chdb-io/chdb-ruby/blob/main/CHANGELOG.md',
+    'source_code_uri' => 'https://github.com/chdb-io/chdb-ruby',
 
     # https://guides.rubygems.org/mfa-requirement-opt-in/
-    "rubygems_mfa_required" => "true"
+    'rubygems_mfa_required' => 'true'
   }
 
   s.files = [
-    "CHANGELOG.md",
-    "LICENSE",
-    "README.md",
-    "lib/chdb.rb",
-    "lib/chdb/constants.rb",
-    "lib/chdb/data_path.rb",
-    "lib/chdb/database.rb",
-    "lib/chdb/errors.rb",
-    "lib/chdb/local_result.rb",
-    "lib/chdb/parameter_binding.rb",
-    "lib/chdb/result_handler.rb",
-    "lib/chdb/result_set.rb",
-    "lib/chdb/sql_processor.rb",
-    "lib/chdb/statement.rb",
-    "lib/chdb/version_info.rb",
-    "lib/chdb/version.rb"
+    'CHANGELOG.md',
+    'LICENSE',
+    'README.md',
+    'lib/chdb.rb',
+    'lib/chdb/constants.rb',
+    'lib/chdb/data_path.rb',
+    'lib/chdb/database.rb',
+    'lib/chdb/errors.rb',
+    'lib/chdb/local_result.rb',
+    'lib/chdb/parameter_binding.rb',
+    'lib/chdb/result_handler.rb',
+    'lib/chdb/result_set.rb',
+    'lib/chdb/sql_processor.rb',
+    'lib/chdb/statement.rb',
+    'lib/chdb/version_info.rb',
+    'lib/chdb/version.rb'
   ]
 
-  s.rdoc_options = ["--main", "README.md"]
+  s.rdoc_options = ['--main', 'README.md']
 
-  s.extensions << "ext/chdb/extconf.rb"
+  s.extensions << 'ext/chdb/extconf.rb'
 end

ext/chdb/chdb.c

@@ -8,7 +8,8 @@
 VALUE cChDBError;
 VALUE cLocalResult;
 
-void init_chdb_constants() {
+void init_chdb_constants()
+{
     VALUE mChDB = rb_define_module("ChDB");
     VALUE mChDBConstants = rb_define_module_under(mChDB, "Constants");
     VALUE mmChDBOpen = rb_define_module_under(mChDBConstants, "Open");
@@ -18,15 +19,21 @@ void init_chdb_constants() {
     rb_define_const(mmChDBOpen, "CREATE", INT2FIX(CHDB_OPEN_CREATE));
 }
 
-void init_exception() {
+void init_exception()
+{
     VALUE mChDB = rb_define_module("ChDB");
     if (rb_const_defined(mChDB, rb_intern("Exception")))
+    {
         cChDBError = rb_const_get(mChDB, rb_intern("Exception"));
+    }
     else
+    {
         cChDBError = rb_define_class_under(mChDB, "Exception", rb_eStandardError);
+    }
 }
 
-void init_local_result() {
+void init_local_result()
+{
     VALUE mChDB = rb_define_module("ChDB");
     cLocalResult = rb_define_class_under(mChDB, "LocalResult", rb_cObject);
     rb_define_alloc_func(cLocalResult, local_result_alloc);
@@ -36,7 +43,8 @@ void init_local_result() {
     rb_define_method(cLocalResult, "bytes_read", local_result_bytes_read, 0);
 }
 
-void init_connection() {
+void init_connection()
+{
     VALUE mChDB = rb_define_module("ChDB");
     VALUE cConnection = rb_define_class_under(mChDB, "Connection", rb_cObject);
     rb_define_alloc_func(cConnection, connection_alloc);
@@ -49,11 +57,11 @@ void init_connection() {
 void Init_chdb(void)
 {
     DEBUG_PRINT("Initializing chdb extension");
-    
+
     init_chdb_constants();
-    init_exception(); 
+    init_exception();
     init_local_result();
-    init_connection(); 
+    init_connection();
 
     DEBUG_PRINT("chdb extension initialized successfully");
 }

ext/chdb/chdb.h

@@ -17,9 +17,9 @@ extern "C" {
 #define CHDB_EXPORT __attribute__((visibility("default")))
 struct local_result
 {
-    char * buf;
+    char *buf;
     size_t len;
-    void * _vec; // std::vector<char> *, for freeing
+    void *_vec;  // std::vector<char> *, for freeing
     double elapsed;
     uint64_t rows_read;
     uint64_t bytes_read;
@@ -28,31 +28,31 @@ struct local_result
 #ifdef __cplusplus
 struct local_result_v2
 {
-    char * buf = nullptr;
+    char *buf = nullptr;
     size_t len = 0;
-    void * _vec = nullptr; // std::vector<char> *, for freeing
+    void *_vec = nullptr;  // std::vector<char> *, for freeing
     double elapsed = 0.0;
     uint64_t rows_read = 0;
     uint64_t bytes_read = 0;
-    char * error_message = nullptr;
+    char *error_message = nullptr;
 };
 #else
 struct local_result_v2
 {
-    char * buf;
+    char *buf;
     size_t len;
-    void * _vec; // std::vector<char> *, for freeing
+    void *_vec;  // std::vector<char> *, for freeing
     double elapsed;
     uint64_t rows_read;
     uint64_t bytes_read;
-    char * error_message;
+    char *error_message;
 };
 #endif
 
-CHDB_EXPORT struct local_result * query_stable(int argc, char ** argv);
+CHDB_EXPORT struct local_result *query_stable(int argc, char ** argv);
 CHDB_EXPORT void free_result(struct local_result * result);
 
-CHDB_EXPORT struct local_result_v2 * query_stable_v2(int argc, char ** argv);
+CHDB_EXPORT struct local_result_v2 *query_stable_v2(int argc, char ** argv);
 CHDB_EXPORT void free_result_v2(struct local_result_v2 * result);
 
 #ifdef __cplusplus
@@ -68,7 +68,7 @@ struct query_queue
     std::condition_variable query_cv; // For query submission
     std::condition_variable result_cv; // For query result retrieval
     query_request current_query;
-    local_result_v2 * current_result = nullptr;
+    local_result_v2 *current_result = nullptr;
     bool has_query = false;
     bool shutdown = false;
     bool cleanup_done = false;
@@ -81,42 +81,42 @@ struct query_queue
  */
 struct chdb_conn
 {
-    void * server; /* ClickHouse LocalServer instance */
+    void *server;  /* ClickHouse LocalServer instance */
     bool connected; /* Connection state flag */
-    void * queue; /* Query processing queue */
+    void *queue;  /* Query processing queue */
 };
 
 /**
  * Creates a new chDB connection.
  * Only one active connection is allowed per process.
  * Creating a new connection with different path requires closing existing connection.
- * 
+ *
  * @param argc Number of command-line arguments
  * @param argv Command-line arguments array (--path=<db_path> to specify database location)
  * @return Pointer to connection pointer, or NULL on failure
  * @note Default path is ":memory:" if not specified
  */
-CHDB_EXPORT struct chdb_conn ** connect_chdb(int argc, char ** argv);
+CHDB_EXPORT struct chdb_conn **connect_chdb(int argc, char ** argv);
 
 /**
  * Closes an existing chDB connection and cleans up resources.
  * Thread-safe function that handles connection shutdown and cleanup.
- * 
+ *
  * @param conn Pointer to connection pointer to close
  */
 CHDB_EXPORT void close_conn(struct chdb_conn ** conn);
 
 /**
  * Executes a query on the given connection.
  * Thread-safe function that handles query execution in a separate thread.
- * 
+ *
  * @param conn Connection to execute query on
  * @param query SQL query string to execute
  * @param format Output format string (e.g., "CSV", default format)
  * @return Query result structure containing output or error message
  * @note Returns error result if connection is invalid or closed
  */
-CHDB_EXPORT struct local_result_v2 * query_conn(struct chdb_conn * conn, const char * query, const char * format);
+CHDB_EXPORT struct local_result_v2 *query_conn(struct chdb_conn * conn, const char * query, const char * format);
 
 #ifdef __cplusplus
 }