smarter_csv 1.17.2 → 1.17.4

data/lib/smarter_csv/parser.rb

@@ -195,8 +195,10 @@ module SmarterCSV
           return [nil, n] if all_blank
         end
 
-        # Batch-strip using C-level map! — faster than per-element strip inside the loop
-        fields.map!(&:strip) if strip
+        # In-place strip! — allocation-free when there's no surrounding whitespace
+        # (matches the sister site in parse_csv_line_ruby; completes the
+        # "strip -> strip!" sweep documented in the 1.17.0 commit notes).
+        fields.each(&:strip!) if strip
 
         remove_empty = options[:remove_empty_values]
         hash = {}

data/lib/smarter_csv/reader.rb

@@ -32,13 +32,14 @@ module SmarterCSV
     # rubocop:disable Naming/MethodName
     def headerA
       record_warning(type: :deprecation, code: :header_a_method) do
-        "Deprecarion Warning: 'headerA' will be removed in future versions. Use 'headders'"
+        "Deprecation Warning: 'headerA' will be removed in future versions. Use 'headers'"
       end
       @headerA
     end
     # rubocop:enable Naming/MethodName
 
-    # first parameter: filename or input object which responds to readline method
+    # first parameter: a path (String or Pathname) to open, or an already-open readable IO
+    # (anything responding to #gets — File, StringIO, Tempfile, Zlib::GzipReader, pipes, ...)
     def initialize(input, given_options = {})
       @input = input
       @has_rails = !!defined?(Rails)
@@ -123,7 +124,14 @@ module SmarterCSV
       @verbose = options[:verbose]
 
       begin
-        fh = input.is_a?(String) ? File.open(input, "r:#{options[:file_encoding]}") : input
+        # Decide whether `input` is an already-open, readable stream or a path we must open.
+        # The reader reads lines via #gets (see file_io.rb and PeekableIO), so a public #gets
+        # is exactly what we need: real IOs (File, StringIO, Tempfile, Zlib::GzipReader, pipes,
+        # custom non-seekable streams) expose it, while path-like inputs (String, Pathname) do
+        # not — their only #gets is the private Kernel#gets. 1.17.0 narrowed this to
+        # input.is_a?(String), which sent Pathname down the IO branch and then called its
+        # private Kernel#gets, raising "private method 'gets' called" (issue #337).
+        fh = input.respond_to?(:gets) ? input : File.open(input, "r:#{options[:file_encoding]}")
 
         # Rewindable inputs (File, Tempfile, StringIO, Zlib::GzipReader, ...) use
         # native rewind for auto-detection — no wrapper overhead in the hot loop.
@@ -272,10 +280,14 @@ module SmarterCSV
         start_time  = Process.clock_gettime(Process::CLOCK_MONOTONIC) if on_start || on_complete
 
         if on_start
-          input_meta = if @input.is_a?(String)
-                         { input: @input, file_size: (File.size(@input) rescue nil) }
-                       else
+          # Same path-vs-IO distinction as the File.open above: an already-open IO responds
+          # to #gets and we can't know its on-disk size, so we report its class name. A
+          # path-like input (String, or a Pathname via #to_path) gets its path and file size.
+          input_meta = if @input.respond_to?(:gets)
                          { input: @input.class.name, file_size: nil }
+                       else
+                         path = @input.respond_to?(:to_path) ? @input.to_path : @input
+                         { input: path, file_size: (File.size(path) rescue nil) }
                        end
           on_start.call(input_meta.merge(col_sep: options[:col_sep], row_sep: options[:row_sep]))
         end

data/lib/smarter_csv/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SmarterCSV
-  VERSION = "1.17.2"
+  VERSION = "1.17.4"
 end

data/smarter_csv.gemspec

@@ -30,11 +30,13 @@ Gem::Specification.new do |spec|
   spec.homepage      = "https://github.com/tilo/smarter_csv"
   spec.license       = 'MIT'
 
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = spec.homepage
-  spec.metadata["changelog_uri"] = "https://github.com/tilo/smarter_csv/blob/main/CHANGELOG.md"
-  spec.metadata["documentation_uri"] = "https://github.com/tilo/smarter_csv/tree/main/docs"
-  spec.metadata["bug_tracker_uri"] = "https://github.com/tilo/smarter_csv/issues"
+  spec.metadata["homepage_uri"]       = spec.homepage
+  spec.metadata["source_code_uri"]    = spec.homepage
+  spec.metadata["changelog_uri"]      = "https://github.com/tilo/smarter_csv/blob/main/CHANGELOG.md"
+  spec.metadata["documentation_uri"]  = "https://github.com/tilo/smarter_csv/tree/main/docs"
+  spec.metadata["bug_tracker_uri"]    = "https://github.com/tilo/smarter_csv/issues"
+  spec.metadata["upgrade_uri"]        = "https://github.com/tilo/smarter_csv/blob/main/UPGRADING.md"
+  spec.metadata["upgrade_wizard_uri"] = "https://tilo.github.io/smarter_csv/upgrade_wizard.html"
 
   spec.required_ruby_version = ">= 2.6.0"
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: smarter_csv
 version: !ruby/object:Gem::Version
-  version: 1.17.2
+  version: 1.17.4
 platform: ruby
 authors:
 - Tilo Sloboda
 bindir: bin
 cert_chain: []
-date: 2026-05-21 00:00:00.000000000 Z
+date: 2026-06-03 00:00:00.000000000 Z
 dependencies: []
 description: |
   SmarterCSV is a high-performance CSV reader and writer for Ruby focused on
@@ -39,6 +39,8 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- UPGRADING.md
+- docs/.nojekyll
 - docs/_introduction.md
 - docs/bad_row_quarantine.md
 - docs/basic_read_api.md
@@ -63,6 +65,8 @@ files:
 - docs/releases/1.17.0/performance_notes.md
 - docs/row_col_sep.md
 - docs/ruby_csv_pitfalls.md
+- docs/upgrade_path.json
+- docs/upgrade_wizard.html
 - docs/value_converters.md
 - docs/warnings.md
 - ext/smarter_csv/extconf.rb
@@ -101,6 +105,8 @@ metadata:
   changelog_uri: https://github.com/tilo/smarter_csv/blob/main/CHANGELOG.md
   documentation_uri: https://github.com/tilo/smarter_csv/tree/main/docs
   bug_tracker_uri: https://github.com/tilo/smarter_csv/issues
+  upgrade_uri: https://github.com/tilo/smarter_csv/blob/main/UPGRADING.md
+  upgrade_wizard_uri: https://tilo.github.io/smarter_csv/upgrade_wizard.html
 rdoc_options: []
 require_paths:
 - lib