rbcsv 0.1.2 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ff6e4c9990f873fd5bfd280b944b39ba3c06507dad792391ea91d5348bb9924
-  data.tar.gz: efa2c4f924e4470aaf3a68ac0243b607f83e62cfed1131b82fa3982c6c80e4d5
+  metadata.gz: 13df1c56f1b7a646d461ae0d1623ffb1e4f0a6ee6746da0e39fd9b680626a935
+  data.tar.gz: 54e9d6ea0e7ec1446479a87fb02ff1614bc693c3c8f8168f3c575589da09a8cd
 SHA512:
-  metadata.gz: 639beb1512112987e6b63a1efc48d4e6bcb9058495c42f1e3e9c7e9778bcd2dabfb202f1118eab0d76d78d2000a0fe75b5629d995a289c83f5c78caeb0634605
-  data.tar.gz: b756b47fdb7f59bcfd9a437467909b69e89aa8c8264f128e3c93f687d5c5c9d71db3df748f6ddeb6f4fe3604ec8fc5eb701a09b11c98be4cade7563ba14851a4
+  metadata.gz: c27195b66c70740de593fdbe4bf3c69415dbc9332c708c3dcccdc93a1c455df639c8563d819760acf62dcc151bc057b39fd744118befd683ded3cde3fe461b1f
+  data.tar.gz: f27373dcfa45e8aa7f3142a9f2c4a6c968d1c2f92731eb1b01fed3776f135b528b16d4cc236af2bb49c9d8f3a94bb03a0e834937041a8dbbbe4d89e628578424

data/.ruby-version

@@ -1 +1 @@
-3.3.6
+3.4.5

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 ## [Unreleased]
 
+## [0.1.4] - 2025-09-20
+
+- **Fixed**: CSV parse method returning empty array due to incorrect header handling
+- **Fixed**: CSV reader now processes all rows instead of skipping header row
+- **Improved**: Added proper test coverage for simple CSV parsing
+
+## [0.1.3] - 2025-09-20
+
+- Internal version bump
+
+## [0.1.2] - 2025-09-20
+
+- Internal version bump
+
 ## [0.1.1] - 2025-09-20
 
 - Fixed Cargo workspace configuration

data/ext/rbcsv/extconf.rb

@@ -4,3 +4,16 @@ require "mkmf"
 require "rb_sys/mkmf"
 
 create_rust_makefile("rbcsv/rbcsv")
+
+# After Makefile generation, patch it to handle platform-specific output paths
+makefile_path = "Makefile"
+if File.exist?(makefile_path)
+  content = File.read(makefile_path)
+
+  # Find the RUSTLIB target and add copy command to handle platform-specific paths
+  content.gsub!(/(\$\(RUSTLIB\): FORCE\n\s+\$\(ECHO\) generating.*\n\s+\$\(CARGO\) rustc.*-l pthread)/,
+    '\1' + "\n\t@mkdir -p $(RB_SYS_CARGO_TARGET_DIR)/$(RB_SYS_CARGO_PROFILE_DIR)" +
+    "\n\t@if [ -f $(RB_SYS_CARGO_TARGET_DIR)/*/$(RB_SYS_CARGO_PROFILE_DIR)/$(SOEXT_PREFIX)$(TARGET_NAME).$(SOEXT) ]; then cp $(RB_SYS_CARGO_TARGET_DIR)/*/$(RB_SYS_CARGO_PROFILE_DIR)/$(SOEXT_PREFIX)$(TARGET_NAME).$(SOEXT) $(RB_SYS_CARGO_TARGET_DIR)/$(RB_SYS_CARGO_PROFILE_DIR)/$(SOEXT_PREFIX)$(TARGET_NAME).$(SOEXT); fi")
+
+  File.write(makefile_path, content)
+end

data/ext/rbcsv/src/lib.rs

@@ -5,7 +5,10 @@ use magnus::{Error, exception, function, prelude::*, Ruby};
 type Error = Box<dyn std::error::Error>;
 
 fn parse(s: String) -> Result<Vec<Vec<String>>, Error> {
-    let mut reader = csv::Reader::from_reader(s.as_bytes());
+    let mut reader = csv::ReaderBuilder::new()
+        .has_headers(false)  // ヘッダーを無効にして、すべての行を処理
+        .from_reader(s.as_bytes());
+
     let mut records = Vec::new();
 
     for result in reader.records() {
@@ -44,9 +47,23 @@ mod tests {
 
         assert!(result.is_ok());
 
+        let records = result.unwrap();
+        assert_eq!(records.len(), 3);  // ヘッダー行も含むため3行
+        assert_eq!(records[0], vec!["name", "age", "city"]);
+        assert_eq!(records[1], vec!["Alice", "25", "Tokyo"]);
+        assert_eq!(records[2], vec!["Bob", "30", "Osaka"]);
+    }
+
+    #[test]
+    fn test_parse_simple() {
+        let csv_data = "a,b\n1,2";
+        let result = parse(csv_data.to_string());
+
+        assert!(result.is_ok());
+
         let records = result.unwrap();
         assert_eq!(records.len(), 2);
-        assert_eq!(records[0], vec!["Alice", "25", "Tokyo"]);
-        assert_eq!(records[1], vec!["Bob", "30", "Osaka"]);
+        assert_eq!(records[0], vec!["a", "b"]);
+        assert_eq!(records[1], vec!["1", "2"]);
     }
 }

data/lib/rbcsv/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RbCsv
-  VERSION = "0.1.2"
+  VERSION = "0.1.4"
 end

data/quick_test.rb

@@ -0,0 +1,23 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+# Gemが利用可能か確認するためのワンライナーテスト
+
+puts "Testing rbcsv gem installation..."
+
+# gemコマンドでrbcsvが利用可能か確認
+system_result = system("ruby -e \"require 'rbcsv'; puts 'rbcsv loaded successfully: version ' + RbCsv::VERSION\"")
+
+if system_result
+  puts "\n✅ rbcsv gem is working!"
+else
+  puts "\n❌ rbcsv gem is not working properly"
+
+  # デバッグ情報
+  puts "\nDebugging information:"
+  puts "Current Ruby: #{`ruby -v`.strip}"
+  puts "Gem list:"
+  system("gem list rbcsv")
+  puts "\nGem paths:"
+  system("ruby -e \"puts $LOAD_PATH.grep(/gem/)\"")
+end

data/test_install.rb

@@ -0,0 +1,38 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require 'rbcsv'
+
+puts "rbcsv gem version: #{RbCsv::VERSION}"
+puts "Testing CSV parsing..."
+
+# Simple CSV data for testing
+csv_data = <<~CSV
+  name,age,city
+  Alice,30,Tokyo
+  Bob,25,Osaka
+  Carol,35,Kyoto
+CSV
+
+begin
+  result = RbCsv.parse(csv_data)
+
+  puts "\nParsing successful!"
+  puts "Number of records: #{result.length}"
+  puts "\nParsed data:"
+  result.each_with_index do |row, index|
+    puts "Row #{index}: #{row.inspect}"
+  end
+
+  puts "\nVerifying structure:"
+  puts "First row: #{result[0]}"
+  puts "Second row: #{result[1]}"
+  puts "Name from first record: #{result[0][0]}"
+  puts "Age from first record: #{result[0][1]}"
+
+  puts "\n✅ rbcsv gem is working correctly!"
+rescue => e
+  puts "❌ Error: #{e.message}"
+  puts e.backtrace
+  exit 1
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rbcsv
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.4
 platform: ruby
 authors:
 - fujitani sora
@@ -53,10 +53,12 @@ files:
 - lib/rbcsv.rb
 - lib/rbcsv/version.rb
 - output_comparison.rb
+- quick_test.rb
 - sample.csv
 - sig/r_csv.rbs
 - test.rb
 - test_fixed.rb
+- test_install.rb
 homepage: https://github.com/fs0414/rbcsv
 licenses:
 - MIT