fontisan 0.1.0 → 0.2.1

data/Rakefile

@@ -9,67 +9,89 @@ require "rubocop/rake_task"
 RuboCop::RakeTask.new
 
 namespace :fixtures do
-  fixtures_dir = "spec/fixtures/fonts"
+  # Load centralized fixture configuration
+  require_relative "spec/support/fixture_fonts"
+
+  # Helper method to download a single file
+  def download_single_file(name, url, target_path)
+    require "open-uri"
+
+    puts "[fixtures:download] Downloading #{name}..."
+    FileUtils.mkdir_p(File.dirname(target_path))
+
+    URI.open(url) do |remote|
+      File.binwrite(target_path, remote.read)
+    end
+
+    puts "[fixtures:download] #{name} downloaded successfully"
+  end
 
   # Helper method to download and extract a font archive
   def download_font(name, url, target_dir)
     require "open-uri"
     require "zip"
 
-    zip_file = "#{target_dir}/#{name}.zip"
-
     puts "[fixtures:download] Downloading #{name}..."
     FileUtils.mkdir_p(target_dir)
 
-    URI.open(url) do |remote|
-      File.binwrite(zip_file, remote.read)
+    # Create a manual temp file path - OS will clean up temp files automatically
+    temp_path = File.join(Dir.tmpdir, "fontisan_#{name}_#{Process.pid}_#{rand(10000)}.zip")
+
+    # Download using IO.copy_stream for better Windows compatibility
+    URI.open(url, "rb") do |remote|
+      File.open(temp_path, "wb") do |file|
+        IO.copy_stream(remote, file)
+      end
     end
 
     puts "[fixtures:download] Extracting #{name}..."
-    Zip::File.open(zip_file) do |zip|
-      zip.each do |entry|
-        dest_path = File.join(target_dir, entry.name)
+
+    # Open zip file and ensure it's fully closed before we're done
+    zip_file = Zip::File.open(temp_path)
+    begin
+      zip_file.each do |entry|
+        # Skip macOS metadata files and directories
+        next if entry.name.start_with?("__MACOSX/") || entry.name.include?("/._")
+        next if entry.directory?
+
+        # Ensure entry.name is relative by stripping leading slashes
+        relative_name = entry.name.sub(%r{^/+}, "")
+
+        dest_path = File.join(target_dir, relative_name)
         FileUtils.mkdir_p(File.dirname(dest_path))
-        entry.extract(dest_path) unless File.exist?(dest_path)
+
+        # Skip if file already exists
+        next if File.exist?(dest_path)
+
+        # Write the file content directly using binary mode
+        File.open(dest_path, "wb") do |file|
+          IO.copy_stream(entry.get_input_stream, file)
+        end
       end
+    ensure
+      # Explicitly close the zip file to release file handle on Windows
+      zip_file.close if zip_file
     end
 
-    FileUtils.rm(zip_file)
+    # Temp file left in Dir.tmpdir - OS will clean it up automatically
+
     puts "[fixtures:download] #{name} downloaded successfully"
   rescue LoadError => e
     warn "[fixtures:download] Error: Required gem not installed. Please run: gem install rubyzip"
     raise e
   end
 
-  # Font configurations with target directories and marker files
-  # All fonts are downloaded via Rake
-  fonts = {
-    "Libertinus" => {
-      url: "https://github.com/alerque/libertinus/releases/download/v7.051/Libertinus-7.051.zip",
-      target_dir: "#{fixtures_dir}/libertinus",
-      marker: "#{fixtures_dir}/libertinus/Libertinus-7.051/static/OTF/LibertinusSerif-Regular.otf",
-    },
-    "MonaSans" => {
-      url: "https://github.com/github/mona-sans/releases/download/v2.0/MonaSans.zip",
-      target_dir: "#{fixtures_dir}/MonaSans",
-      marker: "#{fixtures_dir}/MonaSans/MonaSans/variable/MonaSans[wdth,wght].ttf",
-    },
-    "NotoSerifCJK" => {
-      url: "https://github.com/notofonts/noto-cjk/releases/download/Serif2.003/01_NotoSerifCJK.ttc.zip",
-      target_dir: "#{fixtures_dir}/NotoSerifCJK",
-      marker: "#{fixtures_dir}/NotoSerifCJK/NotoSerifCJK.ttc",
-    },
-    "NotoSerifCJK-VF" => {
-      url: "https://github.com/notofonts/noto-cjk/releases/download/Serif2.003/02_NotoSerifCJK-OTF-VF.zip",
-      target_dir: "#{fixtures_dir}/NotoSerifCJK-VF",
-      marker: "#{fixtures_dir}/NotoSerifCJK-VF/Variable/OTC/NotoSerifCJK-VF.otf.ttc",
-    },
-  }
+  # Get font configurations from centralized source
+  fonts = FixtureFonts.rakefile_config
 
   # Create file tasks for each font
   fonts.each do |name, config|
     file config[:marker] do
-      download_font(name, config[:url], config[:target_dir])
+      if config[:single_file]
+        download_single_file(name, config[:url], config[:marker])
+      else
+        download_font(name, config[:url], config[:target_dir])
+      end
     end
   end
 
@@ -78,12 +100,12 @@ namespace :fixtures do
 
   desc "Clean downloaded fixtures"
   task :clean do
-    %w[libertinus MonaSans NotoSerifCJK NotoSerifCJK-VF].each do |dir|
-      path = File.join(fixtures_dir, dir)
-      if File.exist?(path)
-        FileUtils.rm_rf(path)
-        puts "[fixtures:clean] Removed #{path}"
-      end
+    fonts.values.map { |config| config[:target_dir] }.each do |path|
+      next unless File.exist?(path)
+
+      puts "[fixtures:clean] Removing #{path}..."
+      FileUtils.rm_rf(path)
+      puts "[fixtures:clean] Removed #{path}"
     end
   end
 end

data/benchmark/variation_quick_bench.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require 'benchmark'
+require 'fontisan'
+
+# Load variable font
+FONT_PATH = 'spec/fixtures/SourceSans3VF-Roman.otf'
+
+unless File.exist?(FONT_PATH)
+  puts "Error: Test font not found at #{FONT_PATH}"
+  exit 1
+end
+
+font = Fontisan::FontLoader.load(FONT_PATH)
+
+puts "=== Variable Font Parallel Performance ==="
+puts "Font: #{FONT_PATH}"
+puts
+
+# Test coordinates
+coords = 4.times.map { |i| { "wght" => 300 + i * 200 } }
+puts "Generating #{coords.size} instances"
+puts
+
+# Sequential (1 thread)
+seq_gen = Fontisan::Variation::ParallelGenerator.new(font, threads: 1)
+seq_time = Benchmark.realtime do
+  seq_gen.generate_batch(coords)
+end
+
+# Parallel (4 threads)
+par_gen = Fontisan::Variation::ParallelGenerator.new(font, threads: 4)
+par_time = Benchmark.realtime do
+  par_gen.generate_batch(coords)
+end
+
+puts "Sequential (1 thread): #{format('%.3f', seq_time)}s"
+puts "Parallel (4 threads):  #{format('%.3f', par_time)}s"
+puts "Speedup: #{format('%.2f', seq_time / par_time)}x"
+puts
+
+# Cache stats
+cache = Fontisan::Variation::ThreadSafeCache.new
+5.times { cache.fetch("test_#{rand(100)}") { rand } }
+
+puts "Cache Statistics:"
+puts cache.statistics.inspect