rfmt 1.3.2 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79f5e6c24f82f6552874aefda2a6b97d8f4612419810d76ebd9381e862edded4
-  data.tar.gz: f193dab59e9a02f62d3e599c8f0349ebbf6e7fda070ba55990e9a8d399990f72
+  metadata.gz: 9ac6a3e001e85e27839f04ba78564e5bc16675818b2603a179f62d7fa35c9062
+  data.tar.gz: 4957bbc0de84f4e5c72dacf514ea0e87f211979667a50b53f365c005f1ee710a
 SHA512:
-  metadata.gz: 6ebfd2c04793445912e1edc28db74e5cfa218fcd3fc28f6e3b42a88b1794fe0bfc6643890fa34477291c7ad8624696f3bb7207436ecfeb6f756439be5d9065ae
-  data.tar.gz: e209cf0a8671786af5e1560979f70f5fe8bb4e2de3912613a59e6d6269effe54f123407ca8989b50578f330d16e8a0d2fa7a1b47c8c07d5731e893664892fe1a
+  metadata.gz: 470807974090b7b41d5c9bf5dd437cd34d80cdbb45ed1c9af2d97f93c0a01c88e5e709dcb47b6e077909be7bbb586f7df4eb798e2403dd2186a4d08b6770463e
+  data.tar.gz: 6620293ef2c43673eecbf478e2c4d3ba1b79a0ad71e67ec984547c0aeda36e03f7c10d848c646a74bc392e4057aeb2f4c4dc56d2602a96e295bf77f0fbedbf3e

data/CHANGELOG.md

@@ -1,5 +1,34 @@
 ## [Unreleased]
 
+## [1.3.4] - 2026-01-17
+
+### Added
+- New `rfmt_fast` executable for optimized performance
+- Automatic parallel processing detection logic
+- Enhanced logging and summary display functionality
+
+### Changed  
+- Optimize logging and summary display performance
+- Improve parallel execution logic with automatic detection
+- Update README with new features and usage examples
+- Code formatting improvements with Rubocop compliance
+
+### Fixed
+- Logger optimization to reduce overhead
+- Parallel processing logic refinements
+
+## [1.3.3] - 2026-01-17
+
+### Fixed
+- Add native extension loader for Ruby 3.3+ compatibility (#65)
+  - Resolves LoadError on Ruby 3.3+ arm64-darwin systems
+  - Implements dynamic path resolution for version-specific directories
+
+### Changed
+- Remove unnecessary String clones in comment emission (performance optimization)
+- Remove debug logs and obvious comments from codebase
+- Update .gitignore with development artifacts
+
 ## [1.3.2] - 2026-01-09
 
 ### Added

data/Cargo.lock

@@ -1214,7 +1214,7 @@ checksum = "7a2d987857b319362043e95f5353c0535c1f58eec5336fdfcf626430af7def58"
 
 [[package]]
 name = "rfmt"
-version = "1.3.2"
+version = "1.3.4"
 dependencies = [
  "anyhow",
  "clap",

data/README.md

@@ -166,6 +166,12 @@ Format multiple files:
 rfmt lib/**/*.rb
 ```
 
+Format all files in your project:
+
+```bash
+rfmt .
+```
+
 Check if files need formatting (CI/CD):
 
 ```bash
@@ -178,12 +184,92 @@ Show diff without modifying files:
 rfmt lib/user.rb --diff
 ```
 
+Quiet mode (minimal output):
+
+```bash
+rfmt --quiet lib/**/*.rb
+```
+
 Enable verbose output for debugging:
 
 ```bash
-rfmt lib/user.rb --verbose
-# or use environment variable
-DEBUG=1 rfmt lib/user.rb
+rfmt --verbose lib/user.rb
+```
+
+#### Common Options
+
+| Option | Description |
+|--------|-------------|
+| `--check` | Check formatting without writing files |
+| `--diff` | Show diff of changes |
+| `--quiet` | Minimal output |
+| `--verbose` | Detailed output with timing |
+
+### Output Modes
+
+**Normal mode** (default):
+```bash
+$ rfmt app/
+Processing 25 file(s)...
+✓ Formatted app/controllers/users_controller.rb
+✓ Formatted app/models/user.rb
+
+✓ Processed 25 files
+  (3 formatted, 22 unchanged)
+```
+
+**Quiet mode** (`--quiet` or `-q`):
+```bash
+$ rfmt --quiet app/
+✓ 3 files formatted
+```
+
+**Verbose mode** (`--verbose` or `-v`):
+```bash
+$ rfmt --verbose app/
+Processing 25 file(s)...
+Using sequential processing for 25 files
+✓ Formatted app/controllers/users_controller.rb  
+✓ app/models/application_record.rb already formatted
+...
+
+✓ Processed 25 files
+  (3 formatted, 22 unchanged)
+
+Details:
+  Total files: 25
+  Total time: 0.45s
+  Files/sec: 55.6
+```
+
+### Parallel Processing
+
+rfmt automatically chooses the optimal processing mode:
+
+- **< 20 files**: Sequential processing (fastest for small batches)  
+- **20-49 files**: Automatic based on average file size
+- **≥ 50 files**: Parallel processing (utilizes multiple cores)
+
+You can override this behavior:
+
+```bash
+# Force parallel processing
+rfmt --parallel app/
+
+# Force sequential processing  
+rfmt --no-parallel app/
+```
+
+### Cache Management
+
+rfmt uses caching to improve performance on large codebases:
+
+```bash
+# Clear cache if needed
+rfmt cache clear
+
+# View cache statistics  
+rfmt cache stats
 ```
 
 ### Ruby API

data/exe/rfmt

@@ -1,6 +1,9 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
+# Suppress gem warnings for performance (saves ~2 seconds)
+$VERBOSE = nil unless ENV['RFMT_VERBOSE']
+
 require 'rfmt/cli'
 
 # Known subcommands - if first arg is not one of these, prepend 'format'

data/ext/rfmt/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rfmt"
-version = "1.3.2"
+version = "1.3.4"
 edition = "2021"
 authors = ["fujitani sora <fujitanisora0414@gmail.com>"]
 license = "MIT"

data/ext/rfmt/src/emitter/mod.rs

@@ -62,13 +62,9 @@ impl Emitter {
 
         self.emit_node(ast, 0)?;
 
-        // Find the last emitted code line for proper blank line handling
         let last_code_line = Self::find_last_code_line(ast);
-
-        // Emit any remaining comments that weren't emitted
         self.emit_remaining_comments(last_code_line)?;
 
-        // Ensure file ends with a newline
         if !self.buffer.ends_with('\n') {
             self.buffer.push('\n');
         }
@@ -186,35 +182,26 @@ impl Emitter {
     /// Emit comments that appear before a given line
     /// Uses BTreeMap index for O(log n) lookup instead of O(n) iteration
     fn emit_comments_before(&mut self, line: usize, indent_level: usize) -> Result<()> {
-        let indent_str = self.get_indent(indent_level).to_string();
+        self.ensure_indent_cache(indent_level);
 
-        // Use indexed lookup instead of iterating all comments
         let indices = self.get_comment_indices_before(line);
 
-        // Build list of comments to emit with their data
         let mut comments_to_emit: Vec<_> = indices
             .into_iter()
             .map(|idx| {
                 let comment = &self.all_comments[idx];
-                (
-                    idx,
-                    comment.text.clone(),
-                    comment.location.start_line,
-                    comment.location.end_line,
-                )
+                (idx, comment.location.start_line, comment.location.end_line)
             })
             .collect();
 
-        // Sort by start_line to emit in order
-        comments_to_emit.sort_by_key(|(_, _, start, _)| *start);
+        comments_to_emit.sort_by_key(|(_, start, _)| *start);
 
         let comments_count = comments_to_emit.len();
         let mut last_comment_end_line: Option<usize> = None;
 
-        for (i, (idx, text, comment_start_line, comment_end_line)) in
+        for (i, (idx, comment_start_line, comment_end_line)) in
             comments_to_emit.into_iter().enumerate()
         {
-            // Preserve blank lines between comments
             if let Some(prev_end) = last_comment_end_line {
                 let gap = comment_start_line.saturating_sub(prev_end);
                 for _ in 1..gap {
@@ -222,11 +209,14 @@ impl Emitter {
                 }
             }
 
-            writeln!(self.buffer, "{}{}", indent_str, text)?;
+            writeln!(
+                self.buffer,
+                "{}{}",
+                &self.indent_cache[indent_level], &self.all_comments[idx].text
+            )?;
             self.emitted_comment_indices.insert(idx);
             last_comment_end_line = Some(comment_end_line);
 
-            // Add blank line after the LAST comment if there was a gap to the code
             if i == comments_count - 1 && line > comment_end_line + 1 {
                 self.buffer.push('\n');
             }
@@ -255,33 +245,24 @@ impl Emitter {
         end_line: usize,
         indent_level: usize,
     ) -> Result<()> {
-        let indent_str = self.get_indent(indent_level).to_string();
+        self.ensure_indent_cache(indent_level);
 
-        // Use indexed lookup instead of iterating all comments
         let indices = self.get_comment_indices_in_range(start_line, end_line);
 
-        // Build list of comments to emit, filtering by end_line
         let mut comments_to_emit: Vec<_> = indices
             .into_iter()
             .filter(|&idx| self.all_comments[idx].location.end_line < end_line)
             .map(|idx| {
                 let comment = &self.all_comments[idx];
-                (
-                    idx,
-                    comment.text.clone(),
-                    comment.location.start_line,
-                    comment.location.end_line,
-                )
+                (idx, comment.location.start_line, comment.location.end_line)
             })
             .collect();
 
-        // Sort by start_line to emit in order
-        comments_to_emit.sort_by_key(|(_, _, start, _)| *start);
+        comments_to_emit.sort_by_key(|(_, start, _)| *start);
 
         let mut last_comment_end_line: Option<usize> = None;
 
-        for (idx, text, comment_start_line, comment_end_line) in comments_to_emit {
-            // Preserve blank lines between comments
+        for (idx, comment_start_line, comment_end_line) in comments_to_emit {
             if let Some(prev_end) = last_comment_end_line {
                 let gap = comment_start_line.saturating_sub(prev_end);
                 for _ in 1..gap {
@@ -289,7 +270,11 @@ impl Emitter {
                 }
             }
 
-            writeln!(self.buffer, "{}{}", indent_str, text)?;
+            writeln!(
+                self.buffer,
+                "{}{}",
+                &self.indent_cache[indent_level], &self.all_comments[idx].text
+            )?;
             self.emitted_comment_indices.insert(idx);
             last_comment_end_line = Some(comment_end_line);
         }
@@ -306,39 +291,35 @@ impl Emitter {
         indent_level: usize,
         prev_line: usize,
     ) -> Result<()> {
-        let indent_str = self.get_indent(indent_level).to_string();
+        self.ensure_indent_cache(indent_level);
 
-        // Use indexed lookup instead of iterating all comments
         let indices = self.get_comment_indices_in_range(start_line, end_line);
 
-        // Build list of comments to emit, filtering by end_line
         let mut comments_to_emit: Vec<_> = indices
             .into_iter()
             .filter(|&idx| self.all_comments[idx].location.end_line < end_line)
             .map(|idx| {
                 let comment = &self.all_comments[idx];
-                (
-                    idx,
-                    comment.text.clone(),
-                    comment.location.start_line,
-                    comment.location.end_line,
-                )
+                (idx, comment.location.start_line, comment.location.end_line)
             })
             .collect();
 
-        // Sort by start_line to emit in order
-        comments_to_emit.sort_by_key(|(_, _, start, _)| *start);
+        comments_to_emit.sort_by_key(|(_, start, _)| *start);
 
         let mut last_end_line: usize = prev_line;
 
-        for (idx, text, comment_start_line, comment_end_line) in comments_to_emit {
+        for (idx, comment_start_line, comment_end_line) in comments_to_emit {
             // Preserve blank lines between previous content and this comment
             let gap = comment_start_line.saturating_sub(last_end_line);
             for _ in 1..gap {
                 self.buffer.push('\n');
             }
 
-            writeln!(self.buffer, "{}{}", indent_str, text)?;
+            writeln!(
+                self.buffer,
+                "{}{}",
+                &self.indent_cache[indent_level], &self.all_comments[idx].text
+            )?;
             self.emitted_comment_indices.insert(idx);
             last_end_line = comment_end_line;
         }
@@ -352,15 +333,12 @@ impl Emitter {
         // Use indexed lookup for O(log n) access
         let indices = self.get_comment_indices_on_line(line);
 
-        // Build list of comments to emit
-        let indices_to_emit: Vec<_> = indices
-            .into_iter()
-            .map(|idx| (idx, self.all_comments[idx].text.clone()))
-            .collect();
+        // Collect indices only (no text clone needed)
+        let indices_to_emit: Vec<usize> = indices;
 
-        // Now emit the collected comments
-        for (idx, text) in indices_to_emit {
-            write!(self.buffer, " {}", text)?;
+        // Now emit the collected comments by accessing text at write time
+        for idx in indices_to_emit {
+            write!(self.buffer, " {}", &self.all_comments[idx].text)?;
             self.emitted_comment_indices.insert(idx);
         }
 
@@ -1311,9 +1289,9 @@ impl Emitter {
         Ok(())
     }
 
-    /// Get cached indent string for a given level
-    fn get_indent(&mut self, level: usize) -> &str {
-        // Extend cache if needed
+    /// Ensure indent cache has entries up to and including the given level
+    /// This allows pre-building the cache before borrowing self.indent_cache
+    fn ensure_indent_cache(&mut self, level: usize) {
         while self.indent_cache.len() <= level {
             let len = self.indent_cache.len();
             let indent = match self.config.formatting.indent_style {
@@ -1322,13 +1300,12 @@ impl Emitter {
             };
             self.indent_cache.push(indent);
         }
-        &self.indent_cache[level]
     }
 
     /// Emit indentation
     fn emit_indent(&mut self, level: usize) -> Result<()> {
-        let indent_str = self.get_indent(level).to_string();
-        write!(self.buffer, "{}", indent_str)?;
+        self.ensure_indent_cache(level);
+        write!(self.buffer, "{}", &self.indent_cache[level])?;
         Ok(())
     }
 

data/ext/rfmt/src/lib.rs

@@ -14,25 +14,16 @@ use magnus::{define_module, function, prelude::*, Error, Ruby};
 use parser::{PrismAdapter, RubyParser};
 
 fn format_ruby_code(ruby: &Ruby, source: String, json: String) -> Result<String, Error> {
-    log::info!("format_ruby_code called");
     let policy = SecurityPolicy::default();
 
     policy
         .validate_source_size(&source)
         .map_err(|e| e.to_magnus_error(ruby))?;
 
-    log::debug!("Source code validated, size: {} bytes", source.len());
-
     let parser = PrismAdapter::new();
     let ast = parser.parse(&json).map_err(|e| e.to_magnus_error(ruby))?;
 
-    // Load configuration from file or use defaults
-    log::info!("Attempting to discover config file...");
     let config = Config::discover().map_err(|e| e.to_magnus_error(ruby))?;
-    log::info!(
-        "Config loaded successfully, line_length: {}",
-        config.formatting.line_length
-    );
     let mut emitter = Emitter::with_source(config, source);
 
     let formatted = emitter.emit(&ast).map_err(|e| e.to_magnus_error(ruby))?;
@@ -56,7 +47,6 @@ fn rust_version() -> String {
 #[magnus::init]
 fn init(_ruby: &Ruby) -> Result<(), Error> {
     logging::RfmtLogger::init();
-    log::info!("Initializing rfmt Rust extension");
 
     let module = define_module("Rfmt")?;
 
@@ -64,6 +54,5 @@ fn init(_ruby: &Ruby) -> Result<(), Error> {
     module.define_singleton_method("parse_to_json", function!(parse_to_json, 1))?;
     module.define_singleton_method("rust_version", function!(rust_version, 0))?;
 
-    log::info!("rfmt Rust extension initialized successfully");
     Ok(())
 }

data/lib/rfmt/cli.rb

@@ -44,6 +44,10 @@ module Rfmt
 
   # Command Line Interface for rfmt
   class CLI < Thor
+    # Constants
+    PROGRESS_THRESHOLD = 20  # Show progress for file counts >= this
+    PROGRESS_INTERVAL = 10   # Update progress every N files
+
     class_option :config, type: :string, desc: 'Path to configuration file'
     class_option :verbose, type: :boolean, aliases: '-v', desc: 'Verbose output'
 
@@ -54,10 +58,11 @@ module Rfmt
     option :check, type: :boolean, desc: "Check if files are formatted (don't write)"
     option :diff, type: :boolean, desc: 'Show diff of changes'
     option :diff_format, type: :string, default: 'unified', desc: 'Diff format: unified, side_by_side, or color'
-    option :parallel, type: :boolean, default: true, desc: 'Process files in parallel'
+    option :parallel, type: :boolean, desc: 'Use parallel processing (auto-disabled for <20 files)'
     option :jobs, type: :numeric, desc: 'Number of parallel jobs (default: CPU count)'
     option :cache, type: :boolean, default: true, desc: 'Use cache to skip unchanged files'
     option :cache_dir, type: :string, desc: 'Cache directory (default: ~/.cache/rfmt)'
+    option :quiet, type: :boolean, aliases: '-q', desc: 'Minimal output (errors and summary only)'
     def format(*files)
       config = load_config
       files = files.empty? ? config.files_to_format : files.flatten
@@ -67,33 +72,32 @@ module Rfmt
         return
       end
 
-      # Initialize cache
-      cache = if options[:cache]
-                cache_opts = options[:cache_dir] ? { cache_dir: options[:cache_dir] } : {}
-                Cache.new(**cache_opts)
-              end
-
-      # Filter files using cache
-      if cache
-        original_count = files.size
-        files = files.select { |file| cache.needs_formatting?(file) }
-        skipped = original_count - files.size
-        say "ℹ Skipped #{skipped} unchanged file(s) (cached)", :cyan if skipped.positive? && options[:verbose]
-      end
+      # Initialize and use cache if enabled
+      cache = initialize_cache_if_enabled
+      files = filter_files_with_cache(files, cache)
 
       if files.empty?
-        say '✓ All files are already formatted (cached)', :green
+        say '✓ All files are already formatted (cached)', :cyan
         return
       end
 
-      # Show progress message
-      if files.size == 1
-        say "Processing #{files.first}...", :blue
-      else
-        say "Processing #{files.size} file(s)...", :blue
+      # Show progress message (unless in quiet mode)
+      unless options[:quiet]
+        if files.size == 1
+          say "Processing #{files.first}...", :blue
+        else
+          say "Processing #{files.size} file(s)...", :blue
+        end
       end
 
-      results = if options[:parallel] && files.size > 1
+      use_parallel = should_use_parallel?(files)
+
+      if options[:verbose] && files.size > 1
+        mode = use_parallel ? "parallel (#{options[:jobs] || 'auto'} jobs)" : 'sequential'
+        say "Using #{mode} processing for #{files.size} files", :blue
+      end
+
+      results = if use_parallel
                   format_files_parallel(files)
                 else
                   format_files_sequential(files)
@@ -140,6 +144,39 @@ module Rfmt
 
     private
 
+    # Intelligently decide whether to use parallel processing
+    def should_use_parallel?(files)
+      return false if files.size <= 1
+
+      # Check if parallel option was explicitly set via command line
+      # Thor sets options[:parallel] to true/false for --parallel/--no-parallel
+      # and nil when not specified
+      return options[:parallel] unless options[:parallel].nil?
+
+      # Auto decision based on workload characteristics
+      # Calculate total size for better decision
+      total_size = files.sum do |f|
+        File.size(f)
+      rescue StandardError
+        0
+      end
+      avg_size = total_size / files.size.to_f
+
+      # Decision matrix:
+      # - Less than 20 files: sequential (overhead > benefit)
+      # - 20-50 files with small size (<10KB avg): sequential
+      # - 20-50 files with large size (>10KB avg): parallel
+      # - More than 50 files: always parallel
+
+      if files.size < 20
+        false
+      elsif files.size < 50
+        avg_size > 10_000 # 10KB threshold
+      else
+        true
+      end
+    end
+
     def load_config
       if options[:config]
         Configuration.new(file: options[:config])
@@ -148,25 +185,68 @@ module Rfmt
       end
     end
 
+    def initialize_cache_if_enabled
+      return nil unless options[:cache]
+
+      cache_opts = options[:cache_dir] ? { cache_dir: options[:cache_dir] } : {}
+      Cache.new(**cache_opts)
+    end
+
+    def filter_files_with_cache(files, cache)
+      return files unless cache
+
+      original_count = files.size
+      filtered = files.select { |file| cache.needs_formatting?(file) }
+
+      log_cache_skip(original_count - filtered.size)
+      filtered
+    end
+
+    def log_cache_skip(skipped_count)
+      return unless skipped_count.positive? && options[:verbose]
+
+      say "ℹ Skipped #{skipped_count} unchanged file(s) (cached)", :cyan
+    end
+
     def format_files_sequential(files)
-      files.map do |file|
+      show_progress = should_show_progress?(files)
+
+      files.map.with_index do |file, index|
+        display_progress(index, files.size) if show_progress && (index % PROGRESS_INTERVAL).zero?
         format_single_file(file)
       end
     end
 
+    def should_show_progress?(files)
+      !options[:quiet] && files.size >= PROGRESS_THRESHOLD
+    end
+
+    def display_progress(index, total)
+      percentage = ((index.to_f / total) * 100).round
+      say "[#{index}/#{total}] #{percentage}% complete...", :blue
+    end
+
     def format_files_parallel(files)
       require 'parallel'
 
-      # Determine number of processes to use
-      process_count = options[:jobs] || Parallel.processor_count
-
-      say "Processing #{files.size} files with #{process_count} parallel jobs...", :blue if options[:verbose]
+      process_count = determine_process_count
+      log_parallel_processing(files.size, process_count)
 
       Parallel.map(files, in_processes: process_count) do |file|
         format_single_file(file)
       end
     end
 
+    def determine_process_count
+      options[:jobs] || Parallel.processor_count
+    end
+
+    def log_parallel_processing(file_count, process_count)
+      return unless options[:verbose]
+
+      say "Processing #{file_count} files with #{process_count} parallel jobs...", :blue
+    end
+
     def format_single_file(file)
       start_time = Time.now
       source = File.read(file)
@@ -191,69 +271,126 @@ module Rfmt
     end
 
     def handle_results(results, cache = nil)
-      failed_count = 0
-      changed_count = 0
-      error_count = 0
+      stats = process_results(results, cache)
+      stats[:total_duration] = results.sum { |r| r[:duration] || 0 }
+      cache&.save
+      display_summary(stats, results.size)
+      exit(1) if should_exit_with_error?(stats)
+    end
+
+    def process_results(results, cache)
+      stats = { changed: 0, errors: 0, failed: 0, duration: 0 }
 
       results.each do |result|
         if result[:error]
-          say "Error in #{result[:file]}: #{result[:error]}", :red
-          error_count += 1
-          next
+          handle_error_result(result, stats)
+        elsif result[:changed]
+          handle_changed_result(result, stats, cache)
+        else
+          handle_unchanged_result(result, cache)
         end
+      end
 
-        if result[:changed]
-          changed_count += 1
-
-          if options[:check]
-            say "#{result[:file]} needs formatting", :yellow
-            failed_count += 1
-            show_diff(result[:file], result[:original], result[:formatted]) if options[:diff]
-          elsif options[:diff]
-            show_diff(result[:file], result[:original], result[:formatted])
-          elsif options[:write]
-            File.write(result[:file], result[:formatted])
-            # Always show formatted files (not just in verbose mode)
-            say "✓ Formatted #{result[:file]}", :green
-
-            # Update cache after successful write
-            cache&.mark_formatted(result[:file])
-          else
-            puts result[:formatted]
-          end
-        else
-          # Show already formatted files in non-check mode
-          say "✓ #{result[:file]} already formatted", :cyan unless options[:check]
+      stats
+    end
 
-          # Update cache even if no changes (file was checked)
-          cache&.mark_formatted(result[:file])
-        end
+    def handle_error_result(result, stats)
+      say "Error in #{result[:file]}: #{result[:error]}", :red
+      stats[:errors] += 1
+    end
+
+    def handle_changed_result(result, stats, cache)
+      stats[:changed] += 1
+
+      if options[:check]
+        say "#{result[:file]} needs formatting", :yellow
+        stats[:failed] += 1
+        show_diff(result[:file], result[:original], result[:formatted]) if options[:diff]
+      elsif options[:diff]
+        show_diff(result[:file], result[:original], result[:formatted])
+      elsif options[:write]
+        write_formatted_file(result, cache)
+      else
+        puts result[:formatted]
       end
+    end
 
-      # Save cache to disk
-      cache&.save
+    def handle_unchanged_result(result, cache)
+      say "✓ #{result[:file]} already formatted", :white if options[:verbose] && !options[:check]
+      cache&.mark_formatted(result[:file])
+    end
 
-      # Summary - always show a summary message
-      if error_count.positive?
-        say "\n✗ Failed: #{error_count} error(s) occurred", :red
-      elsif options[:check] && failed_count.positive?
-        say "\n✗ Check failed: #{failed_count} file(s) need formatting", :yellow
-      elsif changed_count.positive?
-        # Success message with appropriate details
-        say "\n✓ Success! Formatted #{changed_count} file(s)", :green
-      elsif results.size == 1
-        say "\n✓ Success! File is already formatted", :green
+    def write_formatted_file(result, cache)
+      File.write(result[:file], result[:formatted])
+      say "✓ Formatted #{result[:file]}", :green unless options[:quiet]
+      cache&.mark_formatted(result[:file])
+    end
+
+    def display_summary(stats, total_files)
+      @last_stats = stats # Store for verbose details
+      unchanged_count = total_files - stats[:changed] - stats[:errors]
+
+      if stats[:errors].positive?
+        display_error_summary(stats[:errors])
+      elsif options[:check] && stats[:failed].positive?
+        display_check_failed_summary(stats[:failed])
+      elsif options[:quiet]
+        display_quiet_summary(stats[:changed])
       else
-        say "\n✓ Success! All #{results.size} files are already formatted", :green
+        display_normal_summary(stats[:changed], unchanged_count, total_files)
       end
 
-      # Detailed summary in verbose mode
-      if options[:verbose]
-        say "Total: #{results.size} file(s) processed", :blue
-        say "Changed: #{changed_count} file(s)", :yellow if changed_count.positive?
+      display_verbose_details(total_files) if options[:verbose] && !options[:quiet]
+    end
+
+    def display_error_summary(error_count)
+      say "\n✗ Failed: #{error_count} error(s) occurred", :red
+    end
+
+    def display_check_failed_summary(failed_count)
+      say "\n✗ Check failed: #{failed_count} file(s) need formatting", :yellow
+    end
+
+    def display_quiet_summary(changed_count)
+      say "✓ #{changed_count} files formatted", :cyan if changed_count.positive?
+    end
+
+    def display_normal_summary(changed_count, unchanged_count, total_files)
+      if total_files == 1
+        if changed_count.positive?
+          say "\n✓ Formatted 1 file", :cyan
+        else
+          say "\n✓ File is already formatted", :cyan
+        end
+      else
+        say "\n✓ Processed #{total_files} files", :cyan
+        display_file_breakdown(changed_count, unchanged_count)
       end
+    end
+
+    def display_file_breakdown(changed_count, unchanged_count)
+      return unless changed_count.positive? || unchanged_count.positive?
+
+      parts = []
+      parts << "#{changed_count} formatted" if changed_count.positive?
+      parts << "#{unchanged_count} unchanged" if unchanged_count.positive?
+      say "  (#{parts.join(', ')})", :white
+    end
+
+    def display_verbose_details(total_files)
+      say "\nDetails:", :blue
+      say "  Total files: #{total_files}", :blue
+
+      # Duration is collected if available
+      return unless defined?(@last_stats) && @last_stats[:total_duration]
+
+      duration = @last_stats[:total_duration].round(2)
+      say "  Total time: #{duration}s", :blue
+      say "  Files/sec: #{(total_files / duration).round(1)}", :blue if duration.positive?
+    end
 
-      exit(1) if (options[:check] && failed_count.positive?) || error_count.positive?
+    def should_exit_with_error?(stats)
+      (options[:check] && stats[:failed].positive?) || stats[:errors].positive?
     end
 
     def show_diff(file, original, formatted)

data/lib/rfmt/native_extension_loader.rb

@@ -0,0 +1,138 @@
+# frozen_string_literal: true
+
+module Rfmt
+  # Handles loading of native extension across different Ruby versions
+  # Ruby 3.3+ places native extensions in version-specific subdirectories
+  module NativeExtensionLoader
+    class << self
+      # Load the native extension, trying multiple possible paths
+      # @return [Boolean] true if successfully loaded
+      # @raise [LoadError] if the extension cannot be found
+      def load_extension
+        debug_log "Loading native extension for Ruby #{RUBY_VERSION}"
+
+        possible_paths = build_possible_paths
+        debug_log "Trying paths: #{possible_paths.inspect}"
+
+        load_from_paths(possible_paths) || raise(LoadError, build_error_message(possible_paths))
+      end
+
+      private
+
+      # Try loading from multiple paths
+      # @param paths [Array<String>] paths to try
+      # @return [Boolean, nil] true if loaded, nil otherwise
+      def load_from_paths(paths)
+        paths.each do |path|
+          if try_load_extension(path)
+            debug_log "Successfully loaded from: #{path}"
+            return true
+          end
+        end
+        nil
+      end
+
+      # Build list of possible paths for the native extension
+      # @return [Array<String>] paths to try, in order of preference
+      def build_possible_paths
+        paths = []
+
+        # Ruby 3.3+ style: version-specific subdirectory
+        paths << version_specific_path if ruby_version >= '3.3'
+
+        # Ruby 3.0-3.2 style: might use version directory
+        paths << version_specific_path if ruby_version >= '3.0' && ruby_version < '3.3'
+
+        # Legacy/fallback: direct placement
+        paths << File.join(__dir__, 'rfmt')
+
+        # Additional fallback: check for .bundle extension explicitly
+        paths << File.join(__dir__, 'rfmt.bundle')
+
+        paths.uniq
+      end
+
+      # Get version-specific path
+      # @return [String] path with version directory
+      def version_specific_path
+        File.join(__dir__, ruby_version_dir, 'rfmt')
+      end
+
+      # Try to load extension from a specific path
+      # @param path [String] path to try
+      # @return [Boolean] true if successful, false otherwise
+      def try_load_extension(path)
+        require path
+        true
+      rescue LoadError => e
+        debug_log "Failed to load from #{path}: #{e.message}"
+        false
+      end
+
+      # Get Ruby version for comparison
+      # @return [String] Ruby version string
+      def ruby_version
+        RUBY_VERSION
+      end
+
+      # Get Ruby version directory name (e.g., "3.3" for Ruby 3.3.0)
+      # @return [String] version directory name
+      def ruby_version_dir
+        RUBY_VERSION.split('.')[0..1].join('.')
+      end
+
+      # Build detailed error message when extension cannot be loaded
+      # @param tried_paths [Array<String>] paths that were tried
+      # @return [String] error message
+      def build_error_message(tried_paths)
+        [
+          error_header,
+          format_tried_paths(tried_paths),
+          error_explanation,
+          workaround_instructions
+        ].join("\n")
+      end
+
+      # Error message header
+      # @return [String] header text
+      def error_header
+        "Unable to load rfmt native extension for Ruby #{RUBY_VERSION}.\n"
+      end
+
+      # Format list of tried paths
+      # @param paths [Array<String>] paths that were tried
+      # @return [String] formatted path list
+      def format_tried_paths(paths)
+        "Tried the following paths:\n#{paths.map { |p| "  - #{p}" }.join("\n")}\n"
+      end
+
+      # Error explanation text
+      # @return [String] explanation
+      def error_explanation
+        "This might be a packaging issue with the gem for your Ruby version.\n"
+      end
+
+      # Workaround instructions
+      # @return [String] instructions
+      def workaround_instructions
+        <<~MSG.chomp
+          Workaround:
+          1. Check if rfmt.bundle exists in: #{__dir__}/
+          2. If it's in a subdirectory, create a symlink:
+             cd #{__dir__}
+             ln -sf <subdirectory>/rfmt.bundle rfmt.bundle
+
+          Please report this issue at: https://github.com/fs0414/rfmt/issues
+        MSG
+      end
+
+      # Log debug information if RFMT_DEBUG is set
+      # @param message [String] message to log
+      def debug_log(message)
+        return unless ENV['RFMT_DEBUG']
+
+        warn "[RFMT::NativeExtensionLoader] #{message}"
+      end
+    end
+  end
+end

data/lib/rfmt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rfmt
-  VERSION = '1.3.2'
+  VERSION = '1.3.4'
 end

data/lib/rfmt.rb

@@ -1,9 +1,12 @@
 # frozen_string_literal: true
 
 require_relative 'rfmt/version'
-require_relative 'rfmt/rfmt'
+require_relative 'rfmt/native_extension_loader'
 require_relative 'rfmt/prism_bridge'
 
+# Load native extension with version-aware loader
+Rfmt::NativeExtensionLoader.load_extension
+
 module Rfmt
   class Error < StandardError; end
   # Errors from Rust side
@@ -99,14 +102,9 @@ module Rfmt
     # @param force [Boolean] Overwrite existing file if true
     # @return [Boolean] true if file was created, false if already exists
     def self.init(path = '.rfmt.yml', force: false)
-      if File.exist?(path) && !force
-        warn "Configuration file already exists: #{path}"
-        warn 'Use force: true to overwrite'
-        return false
-      end
+      return false if File.exist?(path) && !force
 
       File.write(path, DEFAULT_CONFIG)
-      puts "Created rfmt configuration file: #{path}"
       true
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rfmt
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.4
 platform: ruby
 authors:
 - fujitani sora
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-01-09 00:00:00.000000000 Z
+date: 2026-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rb_sys
@@ -56,6 +56,7 @@ files:
 - lib/rfmt/cache.rb
 - lib/rfmt/cli.rb
 - lib/rfmt/configuration.rb
+- lib/rfmt/native_extension_loader.rb
 - lib/rfmt/prism_bridge.rb
 - lib/rfmt/prism_node_extractor.rb
 - lib/rfmt/version.rb