rfmt 1.2.5 → 1.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 67e0ceb39e5630237f92ac2d78fcf2438313dcd883c72ec44066ede9e9868735
-  data.tar.gz: 69b818c97c55f1ffc4f09518393c2249b71e0e084207249ba8fa631679d66167
+  metadata.gz: 5ab6ecad407c85286dad54ee0f744f6adc601c9778d0a2171bbae197d7d5c58d
+  data.tar.gz: c3f7f178538d64891d60936e9303b93e63a28403aa6e4f50adab4b7eaa1f641b
 SHA512:
-  metadata.gz: 2e7cc3e550d2506c0f6691f6b4823f76a2c88e94519d46ca080c2d3b57a7330d5e3df4bca1ab4e18ae1551ea6821c92210638d657997cb6a72631512e47d2bc5
-  data.tar.gz: '0491cb8029834d053692a9cc251da952c5bbbf6e38045bcfe3a90dfb2a9ebf31cb3dfb09bfa640b0c22b427d1ab47f0e801ecd44a88c79626f05ab4898d19c64'
+  metadata.gz: a1f272f73406b14d1d5a735bdb0e66138f753b64ba26d369262a824cefda6d2fc0df14665a2c4f85ae62f3139845c8c01a8f9ad39c2d5fa3db4ad821b7aa7c29
+  data.tar.gz: 0a40da5d6084cfd3e3c4ccfd66d3b4be39d7f0a534a5ba7161ca151d98cce039820c587de0109a3065585cceee0a7eeaaaae2c8a4889ec7eb3bd60f843801bc6

data/CHANGELOG.md

@@ -1,11 +1,22 @@
 ## [Unreleased]
 
+## [1.2.7] - 2026-01-04
+
+### Changed
+- Remove OpenSSL dependency: Use mtime instead of SHA256 hash for cache invalidation
+
+## [1.2.6] - 2026-01-04
+
+### Changed
+- Version bump
+
 ## [1.2.5] - 2026-01-04
 
 ### Fixed
 - Fix trailing comments on `end` keyword (e.g., `end # rubocop:disable`)
 - Fix block internal comments being moved outside the block
 - Fix blank line preservation between code and comments inside blocks
+- Fix leading blank line being added to comment-only files
 
 ## [1.2.4] - 2026-01-04
 

data/Cargo.lock

@@ -1219,7 +1219,7 @@ checksum = "caf4aa5b0f434c91fe5c7f1ecb6a5ece2130b02ad2a590589dda5146df959001"
 
 [[package]]
 name = "rfmt"
-version = "1.2.5"
+version = "1.2.7"
 dependencies = [
  "anyhow",
  "clap",

data/ext/rfmt/Cargo.toml

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

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

@@ -79,24 +79,37 @@ impl Emitter {
     /// Emit all comments that haven't been emitted yet
     fn emit_remaining_comments(&mut self, last_code_line: usize) -> Result<()> {
         let mut last_end_line: Option<usize> = Some(last_code_line);
+        let mut is_first_comment = true;
+
         for (idx, comment) in self.all_comments.iter().enumerate() {
             if self.emitted_comment_indices.contains(&idx) {
                 continue;
             }
-            // Ensure we start on a new line for remaining comments
-            if !self.buffer.ends_with('\n') {
+
+            // For the first remaining comment:
+            // - If buffer is empty, don't add any leading newline
+            // - If buffer has content, ensure we start on a new line
+            if is_first_comment && self.buffer.is_empty() {
+                // Don't add leading newline for first comment when buffer is empty
+            } else if !self.buffer.ends_with('\n') {
                 self.buffer.push('\n');
             }
+
             // Preserve blank lines between code/comments
-            if let Some(prev_line) = last_end_line {
-                let gap = comment.location.start_line.saturating_sub(prev_line);
-                for _ in 1..gap {
-                    self.buffer.push('\n');
+            // But only if this is not the first comment in an empty buffer
+            if !(is_first_comment && self.buffer.is_empty()) {
+                if let Some(prev_line) = last_end_line {
+                    let gap = comment.location.start_line.saturating_sub(prev_line);
+                    for _ in 1..gap {
+                        self.buffer.push('\n');
+                    }
                 }
             }
+
             writeln!(self.buffer, "{}", comment.text)?;
             self.emitted_comment_indices.push(idx);
             last_end_line = Some(comment.location.end_line);
+            is_first_comment = false;
         }
         Ok(())
     }

data/lib/rfmt/cache.rb

@@ -1,12 +1,11 @@
 # frozen_string_literal: true
 
-require 'openssl'
 require 'json'
 require 'fileutils'
 
 module Rfmt
   # Cache system for formatted files
-  # Uses SHA256 hash of file content to determine if formatting is needed
+  # Uses mtime (modification time) to determine if formatting is needed
   class Cache
     class CacheError < StandardError; end
 
@@ -23,22 +22,22 @@ module Rfmt
     end
 
     # Check if file needs formatting
-    # Returns true if file content has changed or not in cache
+    # Returns true if file mtime has changed or not in cache
     def needs_formatting?(file_path)
       return true unless File.exist?(file_path)
 
-      current_hash = file_hash(file_path)
-      cached_hash = @cache_data.dig(file_path, 'hash')
+      current_mtime = File.mtime(file_path).to_i
+      cached_mtime = @cache_data.dig(file_path, 'mtime')
 
-      current_hash != cached_hash
+      current_mtime != cached_mtime
     end
 
-    # Mark file as formatted with current content
+    # Mark file as formatted with current mtime
     def mark_formatted(file_path)
       return unless File.exist?(file_path)
 
       @cache_data[file_path] = {
-        'hash' => file_hash(file_path),
+        'mtime' => File.mtime(file_path).to_i,
         'formatted_at' => Time.now.to_i,
         'version' => CACHE_VERSION
       }
@@ -103,13 +102,6 @@ module Rfmt
       @cache_data = {}
     end
 
-    def file_hash(file_path)
-      content = File.read(file_path)
-      OpenSSL::Digest::SHA256.hexdigest(content)
-    rescue StandardError => e
-      raise CacheError, "Failed to read file #{file_path}: #{e.message}"
-    end
-
     def cache_size
       cache_file = File.join(@cache_dir, 'cache.json')
       return 0 unless File.exist?(cache_file)

data/lib/rfmt/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rfmt
 version: !ruby/object:Gem::Version
-  version: 1.2.5
+  version: 1.2.7
 platform: ruby
 authors:
 - fujitani sora