fontisan 0.4.43 → 0.4.45

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.43"
+  VERSION = "0.4.45"
 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,21 +42,20 @@ 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
+    # Methods below were private; made public for testability (TODO 30)
 
     def tag_index
       @flags & 0x3F
@@ -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).
     #
@@ -725,7 +726,7 @@ module Fontisan
       sfnt_data[head_offset + 8, 4] = [adjustment].pack("N")
     end
 
-    private
+    # Methods below were private; made public for testability (TODO 30)
 
     # Read variable-length UIntBase128 integer from IO
     def read_uint_base128(io)

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).
     #

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.43
+  version: 0.4.45
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-07-12 00:00:00.000000000 Z
+date: 2026-07-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -144,6 +144,25 @@ files:
 - 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/30-eliminate-send-in-specs.md
 - TODO.bug-fixes/README.md
 - TODO.improvements/01-cbdt-cblc-gid-stable-propagation.md
 - TODO.improvements/02-collection-outline-priority.md
@@ -490,6 +509,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
@@ -645,6 +665,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
@@ -678,6 +699,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
@@ -800,6 +822,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