fontisan 0.2.7 → 0.2.9

data/lib/fontisan/woff2/table_transformer.rb

@@ -138,7 +138,8 @@ module Fontisan
         num_glyphs = maxp_table.num_glyphs
 
         # Parse hmtx table
-        advance_widths, lsbs = parse_hmtx_table(hmtx_data, num_h_metrics, num_glyphs)
+        advance_widths, lsbs = parse_hmtx_table(hmtx_data, num_h_metrics,
+                                                num_glyphs)
 
         # Build transformed hmtx table
         build_transformed_hmtx(advance_widths, lsbs, num_h_metrics, num_glyphs)
@@ -388,7 +389,8 @@ module Fontisan
       # @param num_h_metrics [Integer] Number of hMetric entries
       # @param num_glyphs [Integer] Total number of glyphs
       # @return [String] Transformed hmtx data
-      def build_transformed_hmtx(advance_widths, lsbs, num_h_metrics, num_glyphs)
+      def build_transformed_hmtx(advance_widths, lsbs, num_h_metrics,
+num_glyphs)
         data = String.new(encoding: Encoding::BINARY)
 
         # Flags: Use proportional encoding (not explicit) and explicit LSBs

data/lib/fontisan/woff2_font.rb

@@ -666,7 +666,8 @@ module Fontisan
         offset += padding
       end
 
-      # Write table directory
+      # Write table directory (all entries first)
+      # rubocop:disable Style/CombinableLoops - Must write directory entries first, then data
       table_records.each do |record|
         sfnt_data << record[:tag].ljust(4, "\x00")
         sfnt_data << [record[:checksum]].pack("N")
@@ -674,7 +675,7 @@ module Fontisan
         sfnt_data << [record[:length]].pack("N")
       end
 
-      # Write table data with padding
+      # Then write all table data with padding
       table_records.each do |record|
         sfnt_data << record[:data]
 
@@ -683,6 +684,7 @@ module Fontisan
           Constants::TABLE_ALIGNMENT
         sfnt_data << ("\x00" * padding) if padding.positive?
       end
+      # rubocop:enable Style/CombinableLoops
 
       # Update checksumAdjustment in head table
       update_checksum_in_memory(sfnt_data, table_records)

data/lib/fontisan/woff_font.rb

@@ -438,12 +438,13 @@ module Fontisan
           io.write("\x00" * padding) if padding.positive?
         end
 
+        # Update checksum adjustment in head table BEFORE closing file
+        # This avoids Windows file locking issues when Tempfiles are used
+        update_checksum_adjustment_in_io(io)
+
         io.pos
       end
 
-      # Update checksum adjustment in head table
-      update_checksum_adjustment_in_file(output_path)
-
       File.size(output_path)
     end
 
@@ -458,43 +459,58 @@ module Fontisan
       [search_range, entry_selector, range_shift]
     end
 
-    # Update checksumAdjustment field in head table
+    # Update checksumAdjustment field in head table using an open IO object
     #
-    # @param path [String] Path to the font file
+    # @param io [IO] Open IO object positioned at start of file
     # @return [void]
-    def update_checksum_adjustment_in_file(path)
-      # Find head table position in output file
+    def update_checksum_adjustment_in_io(io)
+      # Save current position
+      current_pos = io.pos
+
+      # Find head table position in the file
       head_offset = nil
-      File.open(path, "rb") do |io|
-        io.seek(4) # Skip sfnt_version
-        num_tables = io.read(2).unpack1("n")
-        io.seek(12) # Start of table directory
-
-        num_tables.times do
-          tag = io.read(4)
-          io.read(4) # checksum
-          offset = io.read(4).unpack1("N")
-          io.read(4) # length
-
-          if tag == Constants::HEAD_TAG
-            head_offset = offset
-            break
-          end
+      io.seek(4) # Skip sfnt_version
+      num_tables = io.read(2).unpack1("n")
+      io.seek(12) # Start of table directory
+
+      num_tables.times do
+        tag = io.read(4)
+        io.read(4) # checksum
+        offset = io.read(4).unpack1("N")
+        io.read(4) # length
+
+        if tag == Constants::HEAD_TAG
+          head_offset = offset
+          break
         end
       end
 
       return unless head_offset
 
-      # Use tempfile-based checksum calculation for Windows compatibility
-      File.open(path, "r+b") do |io|
-        checksum, _tmpfile = Utilities::ChecksumCalculator.calculate_checksum_from_io_with_tempfile(io)
+      # Rewind to calculate checksum from the beginning
+      io.rewind
+
+      # Calculate checksum directly from IO to avoid Windows Tempfile issues
+      checksum = Utilities::ChecksumCalculator.calculate_checksum_from_io(io)
 
-        # Calculate adjustment
-        adjustment = Utilities::ChecksumCalculator.calculate_adjustment(checksum)
+      # Calculate adjustment
+      adjustment = Utilities::ChecksumCalculator.calculate_adjustment(checksum)
 
-        # Write adjustment to head table (offset 8 within head table)
-        io.seek(head_offset + 8)
-        io.write([adjustment].pack("N"))
+      # Write adjustment to head table (offset 8 within head table)
+      io.seek(head_offset + 8)
+      io.write([adjustment].pack("N"))
+
+      # Restore original position
+      io.seek(current_pos)
+    end
+
+    # Update checksumAdjustment field in head table
+    #
+    # @param path [String] Path to the font file
+    # @return [void]
+    def update_checksum_adjustment_in_file(path)
+      File.open(path, "r+b") do |io|
+        update_checksum_adjustment_in_io(io)
       end
     end
   end

data/lib/fontisan.rb

@@ -112,6 +112,8 @@ require_relative "fontisan/models/scripts_info"
 require_relative "fontisan/models/features_info"
 require_relative "fontisan/models/all_scripts_features_info"
 require_relative "fontisan/models/validation_report"
+require_relative "fontisan/models/font_report"
+require_relative "fontisan/models/collection_validation_report"
 require_relative "fontisan/models/font_export"
 require_relative "fontisan/models/collection_font_summary"
 require_relative "fontisan/models/collection_info"
@@ -319,8 +321,6 @@ module Fontisan
   end
 
   class << self
-    private
-
     # Get loading mode for validation profile
     #
     # Temporarily disabled - will be reimplemented with new DSL framework

data/scripts/compare_stack_aware.rb

@@ -153,7 +153,7 @@ def main
     puts "\nStack-Aware Efficiency: #{efficiency_ratio}% of normal optimization"
   end
 
-  time_overhead = ((stack_aware[:time] / unoptimized[:time] - 1) * 100).round(2)
+  time_overhead = (((stack_aware[:time] / unoptimized[:time]) - 1) * 100).round(2)
   puts "\nProcessing Time Overhead:"
   puts "  Stack-Aware vs Unoptimized: +#{time_overhead}%"
 

data/scripts/measure_optimization.rb

@@ -96,8 +96,7 @@ def print_summary(results)
   puts "\n\n#{'=' * 80}"
   puts "SUMMARY TABLE"
   puts "=" * 80
-  puts format("%-30s %8s %12s %12s %10s %8s",
-              "Font", "Glyphs", "Before", "After", "Saved", "Reduction")
+  puts "Font                             Glyphs       Before        After      Saved Reduction"
   puts "-" * 80
 
   results.each do |r|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fontisan
 version: !ruby/object:Gem::Version
-  version: 0.2.7
+  version: 0.2.9
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-01-06 00:00:00.000000000 Z
+date: 2026-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -111,14 +111,21 @@ files:
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
+- CHANGELOG.md
 - Gemfile
 - LICENSE
 - README.adoc
 - Rakefile
 - benchmark/variation_quick_bench.rb
+- docs/APPLE_LEGACY_FONTS.adoc
+- docs/COLLECTION_VALIDATION.adoc
+- docs/COLOR_FONTS.adoc
+- docs/DOCUMENTATION_SUMMARY.md
 - docs/EXTRACT_TTC_MIGRATION.md
 - docs/FONT_HINTING.adoc
+- docs/VALIDATION.adoc
 - docs/VARIABLE_FONT_OPERATIONS.adoc
+- docs/WOFF_WOFF2_FORMATS.adoc
 - exe/fontisan
 - fontisan.gemspec
 - lib/fontisan.rb
@@ -203,12 +210,14 @@ files:
 - lib/fontisan/models/collection_font_summary.rb
 - lib/fontisan/models/collection_info.rb
 - lib/fontisan/models/collection_list_info.rb
+- lib/fontisan/models/collection_validation_report.rb
 - lib/fontisan/models/color_glyph.rb
 - lib/fontisan/models/color_layer.rb
 - lib/fontisan/models/color_palette.rb
 - lib/fontisan/models/features_info.rb
 - lib/fontisan/models/font_export.rb
 - lib/fontisan/models/font_info.rb
+- lib/fontisan/models/font_report.rb
 - lib/fontisan/models/font_summary.rb
 - lib/fontisan/models/glyph_info.rb
 - lib/fontisan/models/glyph_outline.rb
@@ -251,6 +260,8 @@ files:
 - lib/fontisan/pipeline/strategies/preserve_strategy.rb
 - lib/fontisan/pipeline/transformation_pipeline.rb
 - lib/fontisan/pipeline/variation_resolver.rb
+- lib/fontisan/sfnt_font.rb
+- lib/fontisan/sfnt_table.rb
 - lib/fontisan/subset/builder.rb
 - lib/fontisan/subset/glyph_mapping.rb
 - lib/fontisan/subset/options.rb
@@ -292,6 +303,7 @@ files:
 - lib/fontisan/tables/cff2/table_reader.rb
 - lib/fontisan/tables/cff2/variation_data_extractor.rb
 - lib/fontisan/tables/cmap.rb
+- lib/fontisan/tables/cmap_table.rb
 - lib/fontisan/tables/colr.rb
 - lib/fontisan/tables/cpal.rb
 - lib/fontisan/tables/cvar.rb
@@ -302,20 +314,29 @@ files:
 - lib/fontisan/tables/glyf/curve_converter.rb
 - lib/fontisan/tables/glyf/glyph_builder.rb
 - lib/fontisan/tables/glyf/simple_glyph.rb
+- lib/fontisan/tables/glyf_table.rb
 - lib/fontisan/tables/gpos.rb
 - lib/fontisan/tables/gsub.rb
 - lib/fontisan/tables/gvar.rb
 - lib/fontisan/tables/head.rb
+- lib/fontisan/tables/head_table.rb
 - lib/fontisan/tables/hhea.rb
+- lib/fontisan/tables/hhea_table.rb
 - lib/fontisan/tables/hmtx.rb
+- lib/fontisan/tables/hmtx_table.rb
 - lib/fontisan/tables/hvar.rb
 - lib/fontisan/tables/layout_common.rb
 - lib/fontisan/tables/loca.rb
+- lib/fontisan/tables/loca_table.rb
 - lib/fontisan/tables/maxp.rb
+- lib/fontisan/tables/maxp_table.rb
 - lib/fontisan/tables/mvar.rb
 - lib/fontisan/tables/name.rb
+- lib/fontisan/tables/name_table.rb
 - lib/fontisan/tables/os2.rb
+- lib/fontisan/tables/os2_table.rb
 - lib/fontisan/tables/post.rb
+- lib/fontisan/tables/post_table.rb
 - lib/fontisan/tables/sbix.rb
 - lib/fontisan/tables/svg.rb
 - lib/fontisan/tables/variation_common.rb