rbcsv 0.1.3 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c7642f2718a3887f7f1c93d81555b354f80b9324822a6da7efc1f877f7381f66
-  data.tar.gz: 2a704520e1dcccb552d378708ac27b8bd99a22a8afa28ac372bb35512ba26d43
+  metadata.gz: 502b63a87094fd5315834ce7772b29e6debaf6b18b48a88f60eae202ffd008ec
+  data.tar.gz: d56609a5352b69b365e87cc0bc6106a579d34927e919db0dfa8bc22228d14620
 SHA512:
-  metadata.gz: 254312f4b7239085910f4ef853586ce5fc42ec80b5bb435819f68dc5c8e143511a9db80227b970f153d019bc02facf5f2043b97747b55d312794a1c91bd71d26
-  data.tar.gz: 74fbec2b486933f924f480cc6c3dc06d748616cdadc12b371d5e48f428346b6b4562b63f5b9c53c32c0bfb09df9091a8e50a2e9d73b09c433e322e496ef40a52
+  metadata.gz: b1e2e790b9a546cad7dfa343bb32feacc590f369f4f16fa736ea63147ec1781a6107a3bf1120bcd360af3fa907459409bd3b92b0d55b7af39cd027e81aaee49d
+  data.tar.gz: 2ffea00a8972c26615e7bc58da5f77c6b2d7b6c1de15f31d0b064b0a2e1b3994b933c2b23297a219e2cf9c78a95ecde7694d67aa4369b4785dd091c9a64b8ba8

data/.ruby-version

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

data/CHANGELOG.md

@@ -1,5 +1,40 @@
 ## [Unreleased]
 
+## [0.1.6] - 2025-09-27
+
+### Changed
+- Version bump for gem release
+
+## [0.1.5] - 2025-09-27
+
+### Fixed
+- Fixed Magnus init function compatibility with version 0.6
+- Added missing rb-sys dependency for proper Ruby header linking
+- Fixed module name inconsistency in RSpec tests (RCsv → RbCsv)
+
+### Changed
+- Updated Magnus initialization to use `#[magnus::init]` attribute
+- Improved error handling with Magnus::Error type
+- Cleaned up unused imports
+
+### Added
+- Documentation for AI-driven gem version upgrade process
+- Tests for parse_with_trim functionality
+
+## [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/Cargo.lock

@@ -238,6 +238,7 @@ version = "0.1.0"
 dependencies = [
  "csv",
  "magnus",
+ "rb-sys",
 ]
 
 [[package]]

data/DEVELOPMENT.md

@@ -0,0 +1,247 @@
+# 開発ガイド
+
+このドキュメントでは、rbcsvの開発環境のセットアップ、ビルド方法、リリース手順について説明します。
+
+## 必要な環境
+
+- Ruby 3.0以降
+- Rust（最新の安定版を推奨）
+- Bundler gem
+- Git
+
+## 開発環境のセットアップ
+
+### 1. リポジトリのクローン
+
+```bash
+git clone <repository-url>
+cd r_csv
+```
+
+### 2. 依存関係のインストール
+
+```bash
+bundle install
+```
+
+### 3. ネイティブ拡張のビルド
+
+```bash
+rake compile
+```
+
+### 4. テストの実行
+
+#### Rubyテスト
+```bash
+bundle exec rspec
+```
+
+#### Rustテスト
+```bash
+cd ext/rbcsv
+cargo test
+cd ../..
+```
+
+### 5. 動作確認テスト
+
+```bash
+ruby -I lib -e "require 'rbcsv'; p RbCsv.parse('a,b\n1,2')"
+```
+
+期待される出力: `[["a", "b"], ["1", "2"]]`
+
+## プロジェクト構成
+
+```
+r_csv/
+├── lib/
+│   ├── rbcsv.rb              # メインのRubyモジュール
+│   └── rbcsv/
+│       ├── version.rb        # バージョン定義
+│       └── rbcsv.bundle      # コンパイル済みネイティブ拡張
+├── ext/
+│   └── rbcsv/
+│       ├── src/
+│       │   └── lib.rs        # Rust実装
+│       ├── Cargo.toml        # Rust依存関係
+│       └── extconf.rb        # Ruby拡張の設定
+├── spec/                     # Rubyテスト
+├── rbcsv.gemspec            # Gem仕様
+├── Rakefile                 # ビルドタスク
+└── DEVELOPMENT.md           # このファイル
+```
+
+## ビルドプロセス
+
+このプロジェクトは、`rb_sys`クレートとRubyの拡張メカニズムを通じてコンパイルされるRustベースのネイティブ拡張を使用しています。
+
+### 手動ビルド手順
+
+1. **前回のビルドをクリーン**（必要に応じて）:
+   ```bash
+   rm -rf lib/rbcsv/rbcsv.bundle tmp/
+   ```
+
+2. **拡張のコンパイル**:
+   ```bash
+   rake compile
+   ```
+
+3. **代替ビルド方法**（rakeが失敗する場合）:
+   ```bash
+   cd ext/rbcsv
+   cargo build --release
+   cp ../../target/release/librbcsv.dylib ../../lib/rbcsv/rbcsv.bundle
+   cd ../..
+   ```
+
+### ビルドのトラブルシューティング
+
+#### ABIバージョンの不一致
+「incompatible ABI version」エラーが発生した場合:
+
+1. ビルドと実行で同じRubyバージョンを使用していることを確認
+2. クリーンして再ビルド:
+   ```bash
+   rm -rf lib/rbcsv/rbcsv.bundle tmp/
+   rake compile
+   ```
+
+#### Rubyバージョンの競合
+開発版Ruby（3.5.0devなど）を使用する場合、プリコンパイル済みのgemは動作しません。必ずソースから再ビルドしてください。
+
+## リリース手順
+
+### 1. バージョンの更新
+
+`lib/rbcsv/version.rb`を編集:
+```ruby
+module RbCsv
+  VERSION = "x.y.z"  # バージョン番号を更新
+end
+```
+
+### 2. CHANGELOG.mdの更新
+
+リリース用の新しいセクションを追加:
+```markdown
+## [x.y.z] - YYYY-MM-DD
+
+- **修正**: バグ修正の説明
+- **追加**: 新機能の説明
+- **変更**: 変更点の説明
+- **削除**: 削除された機能の説明
+```
+
+### 3. ビルドとテスト
+
+```bash
+# クリーンビルド
+rm -rf lib/rbcsv/rbcsv.bundle tmp/
+
+# 拡張の再ビルド
+rake compile
+
+# 機能テスト
+ruby -I lib -e "require 'rbcsv'; p RbCsv.parse('a,b\n1,2')"
+
+# テストスイートの実行
+bundle exec rspec
+```
+
+### 4. Gemのビルド
+
+```bash
+gem build rbcsv.gemspec
+```
+
+これにより、現在のディレクトリに`rbcsv-x.y.z.gem`が作成されます。
+
+### 5. 変更のコミット
+
+```bash
+git add -A
+git commit -m "Release vx.y.z
+
+- 主な変更点の簡潔な説明
+- 重要な改善や修正のリスト"
+```
+
+### 6. Gitタグの作成
+
+```bash
+git tag vx.y.z
+```
+
+### 7. リポジトリへのプッシュ
+
+```bash
+git push origin main
+git push origin vx.y.z
+```
+
+### 8. Gemの公開（オプション）
+
+RubyGems.orgに公開する場合:
+
+```bash
+gem push rbcsv-x.y.z.gem
+```
+
+**注意**: RubyGems.orgの適切な認証情報が設定されていることを確認してください。
+
+## 開発のヒント
+
+### コードスタイル
+
+- 標準的なRubyとRustのフォーマット規約に従う
+- Rustコードのフォーマットには`cargo fmt`を使用
+- 適切なRubyリンティングツールを使用
+
+### テスト
+
+- 新機能のテストを`spec/`に追加
+- Rustユニットテストを`ext/rbcsv/src/lib.rs`に追加
+- コミット前にすべてのテストが通ることを確認
+
+### デバッグ
+
+Rustコードにデバッグ出力を追加する（開発ビルド用）:
+
+```rust
+#[cfg(not(test))]
+eprintln!("デバッグ情報: {:?}", variable);
+```
+
+これはテスト以外のビルドでのみ出力され、本番環境には影響しません。
+
+## よくある問題
+
+### 拡張が読み込めない
+
+1. 正しいRubyバージョン用に拡張がビルドされているか確認
+2. 拡張ファイルの存在を確認: `lib/rbcsv/rbcsv.bundle`
+3. 再ビルドを試す: `rake compile`
+
+### 空配列が返される
+
+これはv0.1.4で修正された既知の問題です。CSVリーダーで`has_headers(false)`設定を使用した最新バージョンを使用していることを確認してください。
+
+### ビルドの失敗
+
+1. Rustがインストールされ、最新であることを確認
+2. Ruby開発ヘッダーが利用可能であることを確認
+3. ビルドキャッシュをクリア: `rm -rf tmp/`
+
+## コントリビューション
+
+1. リポジトリをフォーク
+2. 機能ブランチを作成
+3. 変更を実施
+4. 新機能のテストを追加
+5. すべてのテストが通ることを確認
+6. プルリクエストを送信
+
+コードは既存のスタイルに従い、適切なテストを含めてください。

data/ext/rbcsv/Cargo.toml

@@ -12,3 +12,4 @@ crate-type = ["cdylib"]
 [dependencies]
 csv = "1.3.1"
 magnus = { version = "0.6.2" }
+rb-sys = { version = "0.9", features = ["link-ruby"] }

data/ext/rbcsv/src/lib.rs

@@ -1,35 +1,92 @@
-#[cfg(not(test))]
-use magnus::{Error, exception, function, prelude::*, Ruby};
+use core::fmt;
 
-#[cfg(test)]
-type Error = Box<dyn std::error::Error>;
+use std::error::Error as StdError;
+
+use magnus::{Error as MagnusError, Object};
+
+#[derive(Debug)]
+struct CustomError {
+    message: String,
+    kind: ErrorKind,
+}
+
+#[derive(Debug)]
+enum ErrorKind {
+    Io,
+    _Parse,
+    _Network,
+    _Other,
+}
+
+impl fmt::Display for CustomError {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        write!(f, "{:?}: {}", self.kind, self.message)
+    }
+}
+
+impl StdError for CustomError {}
+
+impl CustomError {
+    fn new(kind: ErrorKind, message: impl Into<String>) -> Self {
+        CustomError {
+            message: message.into(),
+            kind,
+        }
+    }
+}
+
+type Error = CustomError;
+
+fn escape_sanitize(s: String) -> String {
+    s.replace("\\n", "\n")
+        .replace("\\r", "\r")
+        .replace("\\t", "\t")
+        .replace("\\\"", "\"")
+        .replace("\\\\", "\\")
+}
+
+fn parse(s: String) -> Result<Vec<Vec<String>>, MagnusError> {
+    parse_with_options(s, false)  // デフォルトはトリムなし
+        .map_err(|e| MagnusError::new(magnus::exception::runtime_error(), e.to_string()))
+}
+
+fn parse_with_trim(s: String, trim_whitespace: bool) -> Result<Vec<Vec<String>>, MagnusError> {
+    parse_with_options(s, trim_whitespace)
+        .map_err(|e| MagnusError::new(magnus::exception::runtime_error(), e.to_string()))
+}
+
+fn parse_with_options(s: String, trim_whitespace: bool) -> Result<Vec<Vec<String>>, Error> {
+    // エスケープシーケンスを実際の文字に変換
+    let processed = escape_sanitize(s);
+
+    let mut reader = csv::ReaderBuilder::new()
+        .has_headers(false) // ヘッダーを無効にして、すべての行 を読み込む
+        .trim(if trim_whitespace { csv::Trim::All } else { csv::Trim::None })
+        .from_reader(processed.as_bytes());
 
-fn parse(s: String) -> Result<Vec<Vec<String>>, Error> {
-    let mut reader = csv::Reader::from_reader(s.as_bytes());
     let mut records = Vec::new();
 
     for result in reader.records() {
         match result {
             Ok(record) => {
                 let row: Vec<String> = record.iter().map(|field| field.to_string()).collect();
+                println!("row {:?}", row);
                 records.push(row);
             }
-            #[cfg(not(test))]
-            Err(e) => return Err(Error::new(exception::runtime_error(), format!("CSV parse error: {}", e))),
-            #[cfg(test)]
-            Err(e) => return Err(Box::new(e)),
+            Err(_e) => {
+                return Err(Error::new(ErrorKind::Io, "File not found"));
+            }
         }
     }
 
     Ok(records)
 }
 
-#[cfg(not(test))]
 #[magnus::init]
-fn init(ruby: &Ruby) -> Result<(), Error> {
-    let module = ruby.define_module("RbCsv")?;
-    module.define_singleton_method("parse", function!(parse, 1))?;
-    Ok(())
+fn init() {
+    let module = magnus::define_module("RbCsv").expect("Failed to define module");
+    module.define_singleton_method("parse", magnus::function!(parse, 1)).expect("Failed to define parse method");
+    module.define_singleton_method("parse_with_trim", magnus::function!(parse_with_trim, 2)).expect("Failed to define parse_with_trim method");
 }
 
 #[cfg(test)]
@@ -38,15 +95,55 @@ mod tests {
     use super::*;
 
     #[test]
-    fn test_parse() {
-        let csv_data = "name,age,city\nAlice,25,Tokyo\nBob,30,Osaka";
+    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"]);
+    }
+
+    #[test]
+    /// -  シングルクォートの文字列のようなエスケープされた改行をテスト
+    fn test_parse_with_escaped_newline() {
+        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!["a", "b"]);
+        assert_eq!(records[1], vec!["1", "2"]);
+    }
+
+    #[test]
+    fn test_parse_with_trim() {
+        let csv_data = " a , b \n 1 , 2 ";
+        let result = parse_with_trim(csv_data.to_string(), true);
+
+        assert!(result.is_ok());
+
+        let records = result.unwrap();
+        assert_eq!(records.len(), 2);
+        assert_eq!(records[0], vec!["a", "b"]);
+        assert_eq!(records[1], vec!["1", "2"]);
+    }
+
+    #[test]
+    fn test_parse_without_trim() {
+        let csv_data = " a , b \n 1 , 2 ";
+        let result = parse_with_trim(csv_data.to_string(), false);
+
+        assert!(result.is_ok());
+
+        let records = result.unwrap();
+        assert_eq!(records.len(), 2);
+        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.3"
+  VERSION = "0.1.6"
 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.3
+  version: 0.1.6
 platform: ruby
 authors:
 - fujitani sora
@@ -43,6 +43,7 @@ files:
 - CODE_OF_CONDUCT.md
 - Cargo.lock
 - Cargo.toml
+- DEVELOPMENT.md
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -53,10 +54,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
@@ -79,7 +82,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 3.3.11
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 3.7.2
 specification_version: 4
 summary: High-performance CSV processing library with Rust extensions
 test_files: []