fontisan 0.4.42 → 0.4.44

data/lib/fontisan/variation/validator.rb

@@ -103,11 +103,9 @@ module Fontisan
         # Check CFF2 if present
         if @font.has_table?("CFF2")
           cff2 = @font.table("CFF2")
-          if cff2.respond_to?(:num_axes)
-            cff2_axes = cff2.num_axes || 0
-            if cff2_axes != axis_count && cff2_axes.positive?
-              @errors << "CFF2 axis count (#{cff2_axes}) doesn't match fvar (#{axis_count})"
-            end
+          cff2_axes = cff2&.num_axes || 0
+          if cff2_axes != axis_count && cff2_axes.positive?
+            @errors << "CFF2 axis count (#{cff2_axes}) doesn't match fvar (#{axis_count})"
           end
         end
 
@@ -133,20 +131,17 @@ module Fontisan
         return unless @font.has_table?(table_tag)
 
         table = @font.table(table_tag)
-        return unless table.respond_to?(:item_variation_store)
-
-        store = table.item_variation_store
+        store = table&.item_variation_store
         return unless store
 
-        # Check region list axis count
-        if store.respond_to?(:region_list) && store.region_list
-          region_list = store.region_list
-          if region_list.respond_to?(:axis_count)
-            region_axes = region_list.axis_count
-            if region_axes != expected_axes
-              @errors << "#{table_tag} region axis count (#{region_axes}) doesn't match fvar (#{expected_axes})"
-            end
-          end
+        region_list = store.variation_region_list
+        return unless region_list
+
+        region_axes = region_list.axis_count
+        return unless region_axes
+
+        if region_axes != expected_axes
+          @errors << "#{table_tag} region axis count (#{region_axes}) doesn't match fvar (#{expected_axes})"
         end
       end
 
@@ -195,24 +190,15 @@ module Fontisan
         hvar = @font.table("HVAR")
         return unless hvar
 
-        # Check for item_variation_store
-        unless hvar.respond_to?(:item_variation_store)
-          @warnings << "HVAR table doesn't support item_variation_store"
-          return
-        end
-
         store = hvar.item_variation_store
         unless store
           @warnings << "HVAR has no item variation store"
           return
         end
 
-        # Check that variation data exists
-        if store.respond_to?(:item_variation_data)
-          data = store.item_variation_data
-          if data.nil? || data.empty?
-            @warnings << "HVAR has no variation data"
-          end
+        data = store.item_variation_data_entries
+        if data.nil? || data.empty?
+          @warnings << "HVAR has no variation data"
         end
       rescue StandardError => e
         @warnings << "Failed to check HVAR delta integrity: #{e.message}"
@@ -272,24 +258,26 @@ module Fontisan
       # @param axes [Array] Variation axes
       def check_metrics_region_coverage(table_tag, axes)
         table = @font.table(table_tag)
-        return unless table.respond_to?(:item_variation_store)
+        return unless table
 
         store = table.item_variation_store
-        return unless store.respond_to?(:region_list)
+        return unless store
 
-        region_list = store.region_list
-        return unless region_list.respond_to?(:regions)
+        region_list = store.variation_region_list
+        return unless region_list
 
-        # Check each region
         regions = region_list.regions
-        regions.each_with_index do |region, idx|
-          next unless region.respond_to?(:region_axes)
+        return unless regions
+
+        # Each region is an Array<RegionAxisCoordinates>; check coords
+        # are within the valid [-1, 1] normalized range.
+        regions.each_with_index do |region_axis_coords, idx|
+          next unless region_axis_coords.is_a?(Array)
 
-          region.region_axes.each_with_index do |reg_axis, axis_idx|
+          region_axis_coords.each_with_index do |reg_axis, axis_idx|
             next if axis_idx >= axes.length
             next unless reg_axis
 
-            # Check coordinates are in valid range [-1, 1]
             coords = {
               start_coord: reg_axis.start_coord,
               peak_coord: reg_axis.peak_coord,

data/lib/fontisan/variation/variable_svg_generator.rb

@@ -178,8 +178,6 @@ module Fontisan
       # @param tables [Hash<String, String>] Font tables
       # @return [Object] Font-like object
       def build_font_from_tables(tables)
-        # Create a simple font wrapper that implements the minimal
-        # interface needed by SvgGenerator
         InstanceFontWrapper.new(@font, tables)
       end
 
@@ -214,52 +212,6 @@ module Fontisan
         end
         coords
       end
-
-      # Wrapper class for instance font tables
-      #
-      # Provides minimal interface needed by SvgGenerator while using
-      # instance tables instead of original font tables.
-      class InstanceFontWrapper
-        # @return [Hash<String, String>] Font tables
-        attr_reader :table_data
-
-        # Initialize wrapper
-        #
-        # @param original_font [Object] Original variable font
-        # @param instance_tables [Hash<String, String>] Instance tables
-        def initialize(original_font, instance_tables)
-          @original_font = original_font
-          @table_data = instance_tables
-        end
-
-        # Get table by tag
-        #
-        # @param tag [String] Table tag
-        # @return [Object, nil] Table or nil
-        def table(tag)
-          # Use instance table if available, otherwise fall back to original
-          if @table_data.key?(tag)
-          end
-          @original_font.table(tag)
-        end
-
-        # Check if table exists
-        #
-        # @param tag [String] Table tag
-        # @return [Boolean] True if table exists
-        def has_table?(tag)
-          @table_data.key?(tag) || @original_font.has_table?(tag)
-        end
-
-        # Forward other methods to original font
-        def method_missing(method, ...)
-          @original_font.send(method, ...)
-        end
-
-        def respond_to_missing?(method, include_private = false)
-          @original_font.respond_to?(method, include_private) || super
-        end
-      end
     end
   end
 end

data/lib/fontisan/variation/variation_preserver.rb

@@ -148,10 +148,9 @@ module Fontisan
       def validate_source!
         raise ArgumentError, "Source font cannot be nil" if @source_font.nil?
 
-        unless @source_font.respond_to?(:has_table?) &&
-            @source_font.respond_to?(:table_data)
+        unless @source_font.is_a?(SfntSource)
           raise ArgumentError,
-                "Source font must respond to :has_table? and :table_data"
+                "Source font must be an SfntSource instance"
         end
 
         if @target_tables.nil?

data/lib/fontisan/variation.rb

@@ -13,6 +13,7 @@ module Fontisan
     autoload :DeltaParser, "fontisan/variation/delta_parser"
     autoload :Inspector, "fontisan/variation/inspector"
     autoload :InstanceGenerator, "fontisan/variation/instance_generator"
+    autoload :InstanceFontWrapper, "fontisan/variation/instance_font_wrapper"
     autoload :InstanceWriter, "fontisan/variation/instance_writer"
     autoload :Interpolator, "fontisan/variation/interpolator"
     autoload :MetricsAdjuster, "fontisan/variation/metrics_adjuster"

data/lib/fontisan/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Fontisan
-  VERSION = "0.4.42"
+  VERSION = "0.4.44"
 end

data/lib/fontisan/woff2/sfnt_checksum.rb

@@ -79,7 +79,7 @@ module Fontisan
         offsets = {}
         @tables.each do |t|
           offsets[t.tag] = cursor
-          cursor += Utilities::Padding.aligned_size(t.bytes)
+          cursor += Utilities::Padding.aligned_size(t.bytes.bytesize)
         end
 
         bytes = String.new(encoding: Encoding::BINARY)

data/lib/fontisan/woff2/table_transformer.rb

@@ -32,41 +32,34 @@ module Fontisan
       #
       # @param tag [String] Table tag
       # @return [String, nil] Transformed table data
+      # Per-tag transform method dispatch.
+      TRANSFORM_DISPATCH = {
+        "glyf" => :transform_glyf,
+        "loca" => :transform_loca,
+        "hmtx" => :transform_hmtx,
+      }.freeze
+
+      # Per-tag transform version constant.
+      TRANSFORM_VERSIONS = {
+        "glyf" => Directory::TRANSFORM_GLYF_LOCA,
+        "loca" => Directory::TRANSFORM_GLYF_LOCA,
+        "hmtx" => Directory::TRANSFORM_HMTX,
+      }.freeze
+
       def transform_table(tag)
-        case tag
-        when "glyf"
-          transform_glyf
-        when "loca"
-          transform_loca
-        when "hmtx"
-          transform_hmtx
-        else
-          # No transformation, return original data
-          get_table_data(tag)
-        end
+        transform_method = TRANSFORM_DISPATCH[tag]
+        return method(transform_method).call if transform_method
+
+        # No transformation, return original data
+        get_table_data(tag)
       end
 
-      # Check if a table can be transformed
-      #
-      # @param tag [String] Table tag
-      # @return [Boolean] True if table supports transformation
       def transformable?(tag)
-        %w[glyf loca hmtx].include?(tag)
+        TRANSFORM_DISPATCH.key?(tag)
       end
 
-      # Determine transformation version for a table
-      #
-      # @param tag [String] Table tag
-      # @return [Integer] Transformation version
       def transformation_version(tag)
-        case tag
-        when "glyf", "loca"
-          Directory::TRANSFORM_GLYF_LOCA
-        when "hmtx"
-          Directory::TRANSFORM_HMTX
-        else
-          Directory::TRANSFORM_NONE
-        end
+        TRANSFORM_VERSIONS.fetch(tag, Directory::TRANSFORM_NONE)
       end
 
       private
@@ -150,7 +143,7 @@ module Fontisan
       # @param tag [String] Table tag
       # @return [String, nil] Table data or nil if not found
       def get_table_data(tag)
-        return nil unless font.respond_to?(:table_data)
+        return nil unless font.is_a?(SfntSource)
 
         font.table_data[tag]
       end

data/lib/fontisan/woff2_font.rb

@@ -42,18 +42,17 @@ module Fontisan
       (@flags & 0x3F) != 0x3F && KNOWN_TAGS[tag_index]&.start_with?(/glyf|loca|hmtx/)
     end
 
+    TRANSFORM_VERSIONS = {
+      "glyf" => TRANSFORM_GLYF_LOCA,
+      "loca" => TRANSFORM_GLYF_LOCA,
+      "hmtx" => TRANSFORM_HMTX,
+    }.freeze
+
     # Get transform version for this table
     def transform_version
       return TRANSFORM_NONE unless transformed?
 
-      case tag
-      when "glyf", "loca"
-        TRANSFORM_GLYF_LOCA
-      when "hmtx"
-        TRANSFORM_HMTX
-      else
-        TRANSFORM_NONE
-      end
+      TRANSFORM_VERSIONS.fetch(tag, TRANSFORM_NONE)
     end
 
     private
@@ -73,6 +72,8 @@ module Fontisan
   #   puts font.header.flavor
   #   puts font.table_names
   class Woff2Font
+    include SfntSource
+
     # High-level pipeline format identifier. Owned by the font class so the
     # conversion pipeline can dispatch without case statements (OCP).
     #
@@ -185,7 +186,7 @@ module Fontisan
       # If no tag provided, return all tables
       if tag.nil?
         # First try underlying font's table data if available
-        if @underlying_font.respond_to?(:table_data)
+        if @underlying_font.is_a?(SfntFont)
           return @underlying_font.table_data
         end
 
@@ -195,7 +196,7 @@ module Fontisan
 
       # Tag provided - return specific table
       # First try underlying font's table data if available
-      if @underlying_font.respond_to?(:table_data)
+      if @underlying_font.is_a?(SfntFont)
         underlying_data = @underlying_font.table_data[tag]
         return underlying_data if underlying_data
       end

data/lib/fontisan/woff_font.rb

@@ -53,6 +53,8 @@ module Fontisan
   #   woff.to_ttf("output.ttf")  # if TrueType flavored
   #   woff.to_otf("output.otf")  # if CFF flavored
   class WoffFont < BinData::Record
+    include SfntSource
+
     # High-level pipeline format identifier. Owned by the font class so the
     # conversion pipeline can dispatch without case statements (OCP).
     #
@@ -359,7 +361,6 @@ module Fontisan
     def valid?
       return false unless header
       return false unless header.signature == WOFF_SIGNATURE
-      return false unless table_entries.respond_to?(:length)
       return false if table_entries.length != header.num_tables
       return false unless has_table?(Constants::HEAD_TAG)
 

data/lib/fontisan.rb

@@ -116,6 +116,7 @@ module Fontisan
   autoload :OpenTypeFontExtensions, "fontisan/open_type_font_extensions"
   autoload :OutlineExtractor, "fontisan/outline_extractor"
   autoload :SfntFont, "fontisan/sfnt_font"
+  autoload :SfntSource, "fontisan/sfnt_source"
   autoload :SfntTable, "fontisan/sfnt_table"
   autoload :SfntBuilder, "fontisan/sfnt_builder"
   autoload :Stitcher, "fontisan/stitcher"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fontisan
 version: !ruby/object:Gem::Version
-  version: 0.4.42
+  version: 0.4.44
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-07-12 00:00:00.000000000 Z
+date: 2026-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -141,6 +141,27 @@ files:
 - TODO.bug-fixes/06-instance-woff2-output.md
 - TODO.bug-fixes/07-encapsulation-violations.md
 - TODO.bug-fixes/08-respond-to-violations.md
+- TODO.bug-fixes/09-type1-cff-seac-expansion.md
+- TODO.bug-fixes/10-ttf-type1-composite-glyphs.md
+- TODO.bug-fixes/11-remaining-respond-to.md
+- TODO.bug-fixes/12-type1-generators-duck-typing.md
+- TODO.bug-fixes/13-hint-extractor-duck-typing.md
+- TODO.bug-fixes/14-font-interface-type-checks.md
+- TODO.bug-fixes/15-glyph-type-checks-and-simple-glyph-points-bug.md
+- TODO.bug-fixes/16-variation-table-checks.md
+- TODO.bug-fixes/17-ufo-and-stitcher-cleanup.md
+- TODO.bug-fixes/18-generic-value-and-io-checks.md
+- TODO.bug-fixes/19-serialization-migration.md
+- TODO.bug-fixes/20-variation-validator-type-checks.md
+- TODO.bug-fixes/21-xml-builder-send.md
+- TODO.bug-fixes/22-type1-converter-font-info.md
+- TODO.bug-fixes/23-boundary-type-checks.md
+- TODO.bug-fixes/24-validator-dynamic-field-key.md
+- TODO.bug-fixes/25-serialization-migration.md
+- TODO.bug-fixes/26-variation-preserver-spec-migration.md
+- TODO.bug-fixes/27-table-class-registry.md
+- TODO.bug-fixes/28-spec-doubles-cleanup.md
+- TODO.bug-fixes/29-strategy-registries.md
 - TODO.bug-fixes/README.md
 - TODO.improvements/01-cbdt-cblc-gid-stable-propagation.md
 - TODO.improvements/02-collection-outline-priority.md
@@ -487,6 +508,7 @@ files:
 - lib/fontisan/pipeline/variation_resolver.rb
 - lib/fontisan/sfnt_builder.rb
 - lib/fontisan/sfnt_font.rb
+- lib/fontisan/sfnt_source.rb
 - lib/fontisan/sfnt_table.rb
 - lib/fontisan/stitcher.rb
 - lib/fontisan/stitcher/cbdt_propagator.rb
@@ -642,6 +664,7 @@ files:
 - lib/fontisan/tables/os2_table.rb
 - lib/fontisan/tables/post.rb
 - lib/fontisan/tables/post_table.rb
+- lib/fontisan/tables/registry.rb
 - lib/fontisan/tables/sbix.rb
 - lib/fontisan/tables/svg.rb
 - lib/fontisan/tables/variation_common.rb
@@ -675,6 +698,7 @@ files:
 - lib/fontisan/type1_font.rb
 - lib/fontisan/ufo.rb
 - lib/fontisan/ufo/anchor.rb
+- lib/fontisan/ufo/bounds.rb
 - lib/fontisan/ufo/cli.rb
 - lib/fontisan/ufo/compile.rb
 - lib/fontisan/ufo/compile/avar.rb
@@ -797,6 +821,7 @@ files:
 - lib/fontisan/variation/delta_applier.rb
 - lib/fontisan/variation/delta_parser.rb
 - lib/fontisan/variation/inspector.rb
+- lib/fontisan/variation/instance_font_wrapper.rb
 - lib/fontisan/variation/instance_generator.rb
 - lib/fontisan/variation/instance_writer.rb
 - lib/fontisan/variation/interpolator.rb