fontisan 0.2.13 → 0.2.16

data/docs/package.json

@@ -0,0 +1,15 @@
+{
+  "scripts": {
+    "dev": "vitepress dev",
+    "build": "vitepress build",
+    "preview": "vitepress preview",
+    "format": "prettier -w ."
+  },
+  "type": "module",
+  "devDependencies": {
+    "@types/node": "^25.4.0",
+    "prettier": "^3.8.1",
+    "typescript": "^5.9.3",
+    "vitepress": "^1.6.4"
+  }
+}

data/docs/public/favicon.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" id="uuid-c6d7bb0b-ce20-4c4f-b0df-e25e79a561a7" viewBox="0 0 500 500" width="500" height="500"><path d="M64.43,140.29h67.44v13.2h-48.96v180h48.96v13.2h-67.44v-206.4Z" style="fill:#4d4b54;"></path><path d="M352.67,333.5h48.96v-180h-48.96v-13.2h67.44v206.4h-67.44v-13.2Z" style="fill:#4d4b54;"></path><path d="M237.77,169.8c4.32-20.16,8.64-39.6,16.92-54.36,9.36-16.92,24.12-33.48,55.08-33.48,21.96,0,34.2,10.8,34.2,26.28,0,10.44-6.84,15.84-14.4,15.84-9.72,0-13.32-6.48-14.4-14.04-1.44-9-2.16-17.64-10.8-17.64-18.36,0-27.36,21.24-32.76,51.84l-4.32,25.56h34.2l-2.16,14.76h-33.84l-20.52,122.76c-5.04,30.96-21.6,111.6-70.92,111.6-23.04,0-32.04-13.32-32.04-25.92,0-10.08,5.04-17.64,14.4-17.64,21.6,0,9.72,32.04,23.4,32.04,19.08,0,28.44-59.04,32.4-82.8l23.4-140.04h-30.24l1.44-14.76h30.96Z" style="fill:#bf4e6a;"></path></svg>

data/docs/public/logo-full.svg

@@ -0,0 +1 @@
+<?xml version="1.0" encoding="UTF-8"?><svg id="uuid-5b0c2658-028c-4702-accc-5ed4e2e91676" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 2100.48 500"><path d="M257.61,451.33c-14.44,6.96-31.78,5.55-41.96-7.84L67.48,248.75c-5.62-7.38-6.35-18.76-.56-25.73l75.41-90.8c3.17-3.82,7.58-5.34,12.06-7.47l160.97-76.62c14.33-6.82,30.68-3.53,42.04,7.07l105.12,142.11c2.69,5.58,5.13,10.27,8.53,15.28l55.82,82.24c5.37,7.91,6.05,23.69-4.06,28.56l-265.21,127.93ZM287.03,310.96l161.6-76.96c11.16-5.32,21.54-22.67,13.21-34.01l-105.49-143.66c-11.5-10.31-27.05-13.14-41.14-6.4l-166.89,79.87c-13.23,6.33-14.59,25.94-6.48,36.81l97.77,131.09c11.25,15.09,29.74,21.66,47.42,13.24ZM232.38,452.15c6.57,2.92,14.18,2.5,21.22-.89l267.88-128.95c8.75-4.21,9.43-18.42,4.56-25.6l-59.66-88.07c-1.3,13.05-7.4,22.31-18.52,27.6l-157.65,75.1c-9.63,4.59-18.53,5.83-29.47,4.16-6.26-3.25-13.52-5.43-18.2-11.67l-100.41-133.93c-6.12-8.16-7.99-15.97-6.35-27.06l-67.77,81.52c-5.01,6.03-4.38,16.57.54,23.04l148.59,195.3c3.88,5.09,9.36,7.96,15.25,9.45Z" style="fill:#676565;"/><path d="M287.03,310.96c-17.68,8.42-36.17,1.85-47.42-13.24l-97.77-131.09c-8.11-10.88-6.75-30.48,6.48-36.81l166.89-79.87c14.09-6.74,29.64-3.91,41.14,6.4l105.49,143.66c8.33,11.34-2.05,28.69-13.21,34.01l-161.6,76.96Z" style="fill:#e1dfd2;"/><path d="M232.38,452.15c1.96-4.82,3.23-9.57,4.66-15.14l27.96-108.44c1.32-5.12-3.24-8.92-4.26-13.08,10.94,1.67,19.84.42,29.47-4.16l157.65-75.1c11.12-5.3,17.22-14.55,18.52-27.6l59.66,88.07c4.87,7.19,4.2,21.39-4.56,25.6l-267.88,128.95c-7.05,3.39-14.65,3.81-21.22.89Z" style="fill:#bebbac;"/><path d="M260.74,315.5c1.02,4.16,5.58,7.95,4.26,13.08l-27.96,108.44c-1.44,5.57-2.71,10.31-4.66,15.14-5.89-1.49-11.37-4.36-15.25-9.45L68.54,247.4c-4.92-6.47-5.55-17.01-.54-23.04l67.77-81.52c-1.64,11.09.23,18.9,6.35,27.06l100.41,133.93c4.68,6.24,11.94,8.42,18.2,11.67Z" style="fill:#dddac8;"/><polygon points="282.54 238.11 289.49 249.09 263.7 260.65 194.99 159.94 222.35 148.43 228.68 158.31 214.22 164.38 269.97 243.75 282.54 238.11" style="fill:#4d4b54;"/><polygon points="369.09 198.8 383.44 192.08 329.33 114.13 315.75 120.26 309.88 111.47 336 98.94 403.17 194.97 375.96 207.94 369.09 198.8" style="fill:#4d4b54;"/><path d="M292.11,160.41c9.03,15.98,17.59,31.55,25.28,48.2,7.43,16.09,16.44,39,4.36,54.53-5.6,7.2-13.93,10.74-22.32,9.15-3.42-.65-5.8-2.78-5.93-5.92-.12-2.78,1.59-5.75,4.93-6.61,2.7-.69,6.2-.29,8.92.64,5.17,1.76,9.75-.71,11.18-6.18,1.14-4.4.26-9.46-1.57-13.97-4.76-11.73-10.73-22.33-17-33.4l-21.8-38.5c-.25-.44-6.36-11.42-6.36-11.42-2.87-6.21-16.16-25.52-5.57-44.8,5.23-9.53,15.89-15.13,26.35-12.59,3.15.76,5.82,3.09,6.27,5.63.57,3.22-1.12,6.43-4.3,7.81-8,3.48-12.19-4.65-17.66-2.08-7.12,3.35-3.91,13.49-1.03,19.02l10.48,20.15c2.29,4.41,1.97,3.58,5.79,10.33Z" style="fill:#bf4e6a;"/><polygon points="266.05 179.81 260.64 171.75 310.91 148.47 315.92 155.96 266.05 179.81" style="fill:#bf4e6a;"/><path d="M703.28,220.77v144.67h-27.95v-144.67h-31.82v-24.08h31.99v-20.92c0-38.32,24.43-61.35,63.11-61.35,15.12,0,24.79,4.22,27.77,5.45l-3.34,22.68c-4.75-2.11-13.54-4.22-23.2-4.22-21.45,0-36.91,10.55-36.91,39.38v18.98h52.38v24.08h-52.03Z" style="fill:#bf4e6a;"/><path d="M863.6,193.17c51.33,0,89.65,38.5,89.65,87.89s-38.32,87.89-89.65,87.89-89.65-38.5-89.65-87.89,38.32-87.89,89.65-87.89ZM863.6,342.94c35.51,0,61.7-27.07,61.7-61.88s-26.19-61.88-61.7-61.88-61.7,27.07-61.7,61.88,26.19,61.88,61.7,61.88Z" style="fill:#bf4e6a;"/><path d="M1154.16,262.78v102.66h-27.77v-95.8c0-32.87-17.23-50.1-47.81-50.1-28.83,0-52.21,19.86-52.21,50.62v95.27h-27.95v-168.75h28.12v25.31c9.49-15.64,29.53-28.83,56.43-28.83,44.47,0,71.19,26.19,71.19,69.61Z" style="fill:#bf4e6a;"/><path d="M1246.27,220.77v91.23c0,22.15,11.95,31.82,29.18,31.82,9.14,0,20.74-1.76,28.83-4.75l3.69,24.08c-7.21,3.16-21.8,5.8-35.51,5.8-32.52,0-54.14-17.93-54.14-52.38v-95.8h-31.11v-24.08h31.29v-54.14h27.6v54.14h53.96v24.08h-53.79Z" style="fill:#bf4e6a;"/><path d="M1339.26,140.09c0-10.2,7.03-17.23,17.23-17.23s17.58,7.03,17.58,17.23-7.38,17.4-17.58,17.4-17.23-7.21-17.23-17.4ZM1342.6,196.69h27.95v168.75h-27.95v-168.75Z" style="fill:#4d4b54;"/><path d="M1496.76,270.34c33.57,6.68,51.33,21.8,51.33,48.16,0,32.87-26.19,50.45-67.15,50.45-23.91,0-47.46-4.75-63.28-12.66l3.87-24.96c15.47,7.91,37.09,12.66,57.48,12.66,27.07,0,41.66-8.44,41.66-24.08s-13.01-21.45-36.74-26.02l-21.97-4.39c-27.95-5.45-45.88-20.21-45.88-46.23,0-30.06,26.54-50.1,68.55-50.1,20.39,0,40.96,3.87,54.14,9.32l-3.52,24.43c-13.71-5.45-32.34-8.96-48.87-8.96-27.42,0-42.54,9.14-42.54,24.61,0,14.59,11.6,19.69,35.33,24.26l17.58,3.52Z" style="fill:#4d4b54;"/><path d="M1757.09,365.44h-27.95v-27.77c-12.66,18.11-34.8,31.29-65.04,31.29-48.34,0-85.08-37.62-85.08-88.42s36.74-87.36,85.08-87.36c30.23,0,52.38,13.01,65.04,31.11v-27.6h27.95v168.75ZM1606.8,280.54c0,34.98,24.96,63.11,60.12,63.11s62.58-28.12,62.58-63.11-26.54-62.05-62.58-62.05-60.12,27.25-60.12,62.05Z" style="fill:#4d4b54;"/><path d="M1971.19,262.78v102.66h-27.77v-95.8c0-32.87-17.23-50.1-47.81-50.1-28.83,0-52.21,19.86-52.21,50.62v95.27h-27.95v-168.75h28.12v25.31c9.49-15.64,29.53-28.83,56.43-28.83,44.47,0,71.19,26.19,71.19,69.61Z" style="fill:#4d4b54;"/></svg>

data/docs/public/logo.svg

@@ -0,0 +1 @@
+<?xml version="1.0" encoding="UTF-8"?><svg id="uuid-bbbd5b2f-b286-4c12-becd-414fca1ebde9" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 500 500"><path d="M210.8,451.33c-14.44,6.96-31.78,5.55-41.96-7.84L20.67,248.75c-5.62-7.38-6.35-18.76-.56-25.73l75.41-90.8c3.17-3.82,7.58-5.34,12.06-7.47l160.97-76.62c14.33-6.82,30.68-3.53,42.04,7.07l105.12,142.11c2.69,5.58,5.13,10.27,8.53,15.28l55.82,82.24c5.37,7.91,6.05,23.69-4.06,28.56l-265.21,127.93ZM240.21,310.96l161.6-76.96c11.16-5.32,21.54-22.67,13.21-34.01l-105.49-143.66c-11.5-10.31-27.05-13.14-41.14-6.4l-166.89,79.87c-13.23,6.33-14.59,25.94-6.48,36.81l97.77,131.09c11.25,15.09,29.74,21.66,47.42,13.24ZM185.57,452.15c6.57,2.92,14.18,2.5,21.22-.89l267.88-128.95c8.75-4.21,9.43-18.42,4.56-25.6l-59.66-88.07c-1.3,13.05-7.4,22.31-18.52,27.6l-157.65,75.1c-9.63,4.59-18.53,5.83-29.47,4.16-6.26-3.25-13.52-5.43-18.2-11.67l-100.41-133.93c-6.12-8.16-7.99-15.97-6.35-27.06L21.19,224.36c-5.01,6.03-4.38,16.57.54,23.04l148.59,195.3c3.88,5.09,9.36,7.96,15.25,9.45Z" style="fill:#676565;"/><path d="M240.21,310.96c-17.68,8.42-36.17,1.85-47.42-13.24l-97.77-131.09c-8.11-10.88-6.75-30.48,6.48-36.81l166.89-79.87c14.09-6.74,29.64-3.91,41.14,6.4l105.49,143.66c8.33,11.34-2.05,28.69-13.21,34.01l-161.6,76.96Z" style="fill:#e1dfd2;"/><path d="M185.57,452.15c1.96-4.82,3.23-9.57,4.66-15.14l27.96-108.44c1.32-5.12-3.24-8.92-4.26-13.08,10.94,1.67,19.84.42,29.47-4.16l157.65-75.1c11.12-5.3,17.22-14.55,18.52-27.6l59.66,88.07c4.87,7.19,4.2,21.39-4.56,25.6l-267.88,128.95c-7.05,3.39-14.65,3.81-21.22.89Z" style="fill:#bebbac;"/><path d="M213.93,315.5c1.02,4.16,5.58,7.95,4.26,13.08l-27.96,108.44c-1.44,5.57-2.71,10.31-4.66,15.14-5.89-1.49-11.37-4.36-15.25-9.45L21.73,247.4c-4.92-6.47-5.55-17.01-.54-23.04l67.77-81.52c-1.64,11.09.23,18.9,6.35,27.06l100.41,133.93c4.68,6.24,11.94,8.42,18.2,11.67Z" style="fill:#dddac8;"/><polygon points="235.73 238.11 242.68 249.09 216.89 260.65 148.18 159.94 175.53 148.43 181.86 158.31 167.41 164.38 223.16 243.75 235.73 238.11" style="fill:#4d4b54;"/><polygon points="322.28 198.8 336.63 192.08 282.52 114.13 268.93 120.26 263.07 111.47 289.19 98.94 356.36 194.97 329.15 207.94 322.28 198.8" style="fill:#4d4b54;"/><path d="M245.3,160.41c9.03,15.98,17.59,31.55,25.28,48.2,7.43,16.09,16.44,39,4.36,54.53-5.6,7.2-13.93,10.74-22.32,9.15-3.42-.65-5.8-2.78-5.93-5.92-.12-2.78,1.59-5.75,4.93-6.61,2.7-.69,6.2-.29,8.92.64,5.17,1.76,9.75-.71,11.18-6.18,1.14-4.4.26-9.46-1.57-13.97-4.76-11.73-10.73-22.33-17-33.4l-21.8-38.5c-.25-.44-6.36-11.42-6.36-11.42-2.87-6.21-16.16-25.52-5.57-44.8,5.23-9.53,15.89-15.13,26.35-12.59,3.15.76,5.82,3.09,6.27,5.63.57,3.22-1.12,6.43-4.3,7.81-8,3.48-12.19-4.65-17.66-2.08-7.12,3.35-3.91,13.49-1.03,19.02l10.48,20.15c2.29,4.41,1.97,3.58,5.79,10.33Z" style="fill:#bf4e6a;"/><polygon points="219.23 179.81 213.82 171.75 264.09 148.47 269.11 155.96 219.23 179.81" style="fill:#bf4e6a;"/></svg>

data/docs/public/site.webmanifest

@@ -0,0 +1,21 @@
+{
+  "name": "MyWebSite",
+  "short_name": "MySite",
+  "icons": [
+    {
+      "src": "/web-app-manifest-192x192.png",
+      "sizes": "192x192",
+      "type": "image/png",
+      "purpose": "maskable"
+    },
+    {
+      "src": "/web-app-manifest-512x512.png",
+      "sizes": "512x512",
+      "type": "image/png",
+      "purpose": "maskable"
+    }
+  ],
+  "theme_color": "#ffffff",
+  "background_color": "#ffffff",
+  "display": "standalone"
+}

data/fontisan.gemspec

@@ -41,7 +41,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "base64"
   spec.add_dependency "bindata", "~> 2.5"
   spec.add_dependency "brotli", "~> 0.5"
-  spec.add_dependency "lutaml-model", "~> 0.7"
+  spec.add_dependency "lutaml-model", "~> 0.8"
   spec.add_dependency "nokogiri", "~> 1.16"
   spec.add_dependency "thor", "~> 1.3"
 end

data/lib/fontisan/commands/features_command.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "set"
 require "yaml"
 require_relative "base_command"
 require_relative "../models/features_info"

data/lib/fontisan/commands/info_command.rb

@@ -161,10 +161,10 @@ module Fontisan
 
         # Check if variable font (Type1 fonts are never variable)
         info.is_variable = if font.is_a?(Type1Font)
-                            false
-                          else
-                            font.has_table?(Constants::FVAR_TAG)
-                          end
+                             false
+                           else
+                             font.has_table?(Constants::FVAR_TAG)
+                           end
       end
 
       # Populate essential fields for brief mode (metadata tables only).
@@ -273,7 +273,7 @@ module Fontisan
         info.version = font_info.version
         info.copyright = font_info.copyright
         info.description = font_info.notice
-        info.designer = nil  # Type 1 fonts may not have designer info
+        info.designer = nil # Type 1 fonts may not have designer info
 
         # Metrics
         if font_dict&.font_b_box

data/lib/fontisan/commands/scripts_command.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "set"
 require "yaml"
 require_relative "base_command"
 require_relative "../models/scripts_info"

data/lib/fontisan/converters/format_converter.rb

@@ -351,7 +351,8 @@ _options)
         is_type1 = font.is_a?(Type1Font)
 
         unless is_type1 || font.respond_to?(:table)
-          raise ArgumentError, "Font must respond to :table method or be a Type1Font"
+          raise ArgumentError,
+                "Font must respond to :table method or be a Type1Font"
         end
 
         unless target_format.is_a?(Symbol)

data/lib/fontisan/converters/type1_converter.rb

@@ -327,7 +327,10 @@ module Fontisan
 
         # Get CharStrings INDEX from CFF
         charstrings_index = cff_table.charstrings_index(0)
-        raise Fontisan::Error, "CharStrings INDEX not found" unless charstrings_index
+        unless charstrings_index
+          raise Fontisan::Error,
+                "CharStrings INDEX not found"
+        end
 
         # Get Private DICT for context
         private_dict = cff_table.private_dict(0)
@@ -335,7 +338,7 @@ module Fontisan
         # Create CFF to Type 1 converter
         converter = Type1::CffToType1Converter.new(
           nominal_width: private_dict&.nominal_width || 0,
-          default_width: private_dict&.default_width || 0
+          default_width: private_dict&.default_width || 0,
         )
 
         # Convert each CFF CharString to Type 1 format
@@ -354,7 +357,7 @@ module Fontisan
           private_dict_hash = build_private_dict_hash(private_dict)
           type1_charstrings[glyph_name] = converter.convert(
             cff_charstring,
-            private_dict: private_dict_hash
+            private_dict: private_dict_hash,
           )
         end
 
@@ -551,7 +554,7 @@ module Fontisan
         # Parse version (e.g., "001.000" => 1.0)
         version_parts = version_str.split(".")
         major = version_parts[0].to_i
-        minor = version_parts[1]&.to_i || 0
+        minor = version_parts[1].to_i
         version = major + (minor / 1000.0)
 
         # Version (Fixed 16.16) - stored as int32
@@ -626,24 +629,24 @@ module Fontisan
 
         # Ascent (int16) - Distance from baseline to highest ascender
         # Use BlueValues[2] or [3] if available, otherwise font_bbox[3]
-        if blue_values.length >= 4
-          ascent = blue_values[3] # Top zone top
-        elsif blue_values.length >= 3
-          ascent = blue_values[2] # Top zone bottom
-        else
-          ascent = font_bbox[3] # y_max
-        end
+        ascent = if blue_values.length >= 4
+                   blue_values[3] # Top zone top
+                 elsif blue_values.length >= 3
+                   blue_values[2] # Top zone bottom
+                 else
+                   font_bbox[3] # y_max
+                 end
         data << [ascent].pack("s>")
 
         # Descent (int16) - Distance from baseline to lowest descender (negative)
         # Use BlueValues[0] or [1] if available, otherwise font_bbox[1]
-        if blue_values.length >= 2
-          descent = blue_values[0] # Bottom zone bottom (negative)
-        elsif blue_values.length >= 1
-          descent = blue_values[0]
-        else
-          descent = font_bbox[1] # y_min (should be negative)
-        end
+        descent = if blue_values.length >= 2
+                    blue_values[0] # Bottom zone bottom (negative)
+                  elsif blue_values.length >= 1
+                    blue_values[0]
+                  else
+                    font_bbox[1] # y_min (should be negative)
+                  end
         data << [descent].pack("s>")
 
         # Line Gap (int16) - Additional space between lines
@@ -728,33 +731,33 @@ module Fontisan
 
         # Extract font names with fallbacks
         font_name = font.font_name || font_dict&.font_name || "Unnamed"
-        family_name = if font_info&.respond_to?(:family_name)
+        family_name = if font_info.respond_to?(:family_name)
                         font_info.family_name || font_dict&.family_name || font_name
                       else
                         font_dict&.family_name || font_name
                       end
-        full_name = if font_info&.respond_to?(:full_name)
+        full_name = if font_info.respond_to?(:full_name)
                       font_info.full_name || font_dict&.full_name || family_name
                     else
                       font_dict&.full_name || family_name
                     end
-        version = if font_info&.respond_to?(:version)
+        version = if font_info.respond_to?(:version)
                     font_info.version || font.version || "001.000"
                   else
                     font.version || "001.000"
                   end
-        copyright = if font_info&.respond_to?(:copyright)
-                       font_info.copyright || font_dict&.raw_data&.dig(:copyright) || ""
-                     else
-                       font_dict&.raw_data&.dig(:copyright) || ""
-                     end
+        copyright = if font_info.respond_to?(:copyright)
+                      font_info.copyright || font_dict&.raw_data&.dig(:copyright) || ""
+                    else
+                      font_dict&.raw_data&.dig(:copyright) || ""
+                    end
         postscript_name = font_name
-        weight = if font_info&.respond_to?(:weight)
+        weight = if font_info.respond_to?(:weight)
                    font_info.weight
                  else
                    "Regular"
                  end
-        notice = if font_info&.respond_to?(:notice)
+        notice = if font_info.respond_to?(:notice)
                    font_info.notice
                  else
                    ""
@@ -782,7 +785,9 @@ module Fontisan
         ]
 
         # Filter out empty strings and build string storage
-        name_records = name_records.select { |r| !r[:string].nil? && !r[:string].empty? }
+        name_records = name_records.select do |r|
+          !r[:string].nil? && !r[:string].empty?
+        end
 
         # Build string storage (UTF-16BE encoded for Windows platform)
         string_storage = (+"").b
@@ -810,15 +815,15 @@ module Fontisan
         # Write name records
         platform_id = 3  # Windows
         encoding_id = 1  # Unicode BMP
-        language_id = 0x0409  # US English
+        language_id = 0x0409 # US English
 
         name_records.each do |record|
           data << [platform_id].pack("n")           # platform ID
           data << [encoding_id].pack("n")           # encoding ID
           data << [language_id].pack("n")           # language ID
           data << [record[:name_id]].pack("n")      # name ID
-          data << [record[:encoded].bytesize].pack("n")  # string length
-          data << [record[:offset]].pack("n")       # string offset
+          data << [record[:encoded].bytesize].pack("n") # string length
+          data << [record[:offset]].pack("n") # string offset
         end
 
         # Write string storage
@@ -926,13 +931,13 @@ module Fontisan
 
         # Unicode ranges (4 x uint32) - Basic Latin + Latin-1
         # Bits 0-31: Basic Latin, Latin-1, Latin Extended-A/B, etc.
-        data << [0x00000001].pack("N")  # Basic Latin (0-7F)
+        data << [0x00000001].pack("N") # Basic Latin (0-7F)
         data << [0x00000000].pack("N")
         data << [0x00000000].pack("N")
         data << [0x00000000].pack("N")
 
         # achVendID (4 bytes) - Vendor ID
-        data << "UKWN"  # Unknown
+        data << "UKWN" # Unknown
 
         # fsSelection (uint16) - Font selection flags
         # Bit 6 (0x40) = Regular weight if 400-500
@@ -946,25 +951,25 @@ module Fontisan
         data << [fs_selection].pack("n")
 
         # usFirstCharIndex (uint16) - First Unicode character
-        data << [32].pack("n")  # Space
+        data << [32].pack("n") # Space
 
         # usLastCharIndex (uint16) - Last Unicode character
-        data << [0xFFFD].pack("n")  # Replacement character
+        data << [0xFFFD].pack("n") # Replacement character
 
         # sTypoAscender (int16) - Use BlueValues or font bbox
-        if blue_values.length >= 4
-          typo_ascender = blue_values[3]
-        else
-          typo_ascender = font_bbox[3]
-        end
+        typo_ascender = if blue_values.length >= 4
+                          blue_values[3]
+                        else
+                          font_bbox[3]
+                        end
         data << [typo_ascender].pack("s>")
 
         # sTypoDescender (int16) - Use BlueValues or font bbox (negative)
-        if blue_values.length >= 2
-          typo_descender = blue_values[0]
-        else
-          typo_descender = font_bbox[1]
-        end
+        typo_descender = if blue_values.length >= 2
+                           blue_values[0]
+                         else
+                           font_bbox[1]
+                         end
         data << [typo_descender].pack("s>")
 
         # sTypoLineGap (int16)
@@ -1013,7 +1018,7 @@ module Fontisan
         # Version (Fixed 16.16) - Use version 3.0 for CFF fonts (no glyph names)
         # Version 2.0 would include glyph names, but for OTF output version 3.0 is fine
         # since CFF table contains the glyph names
-        data << [0x00030000].pack("N")  # Version 3.0
+        data << [0x00030000].pack("N") # Version 3.0
 
         # Italic Angle (Fixed 16.16)
         # Get from FontInfo if available, otherwise default to 0
@@ -1030,7 +1035,7 @@ module Fontisan
         data << [underline_thickness].pack("s>")
 
         # Fixed Pitch (uint32) - Boolean for monospace
-        is_fixed_pitch = (font_info.is_fixed_pitch || false) ? 1 : 0
+        is_fixed_pitch = font_info.is_fixed_pitch || false ? 1 : 0
         data << [is_fixed_pitch].pack("N")
 
         # Min/Max Memory for Type 42 (uint32 each) - Not used for CFF, set to 0
@@ -1066,10 +1071,10 @@ module Fontisan
           unicode = Type1::AGL.unicode_for_glyph_name(glyph_name)
 
           # If no Unicode mapping, try to derive from encoding position
-          if unicode.nil?
+          if unicode.nil? && (glyph_index < 128)
             # For standard encoding, try to map from position
             # This is a simplified approach - real implementation would be more robust
-            unicode = glyph_index if glyph_index < 128
+            unicode = glyph_index
           end
 
           # Map Unicode to glyph index
@@ -1088,21 +1093,21 @@ module Fontisan
         subtable_data = build_cmap_format_4(unicode_to_glyph)
 
         # Calculate offsets
-        encoding_records_offset = 4  # After version (2) + num_tables (2)
-        subtable_offset = encoding_records_offset + 8  # After one encoding record (8 bytes)
+        encoding_records_offset = 4 # After version (2) + num_tables (2)
+        subtable_offset = encoding_records_offset + 8 # After one encoding record (8 bytes)
 
         # Build cmap table header
         # Version (uint16)
         data << [0].pack("n")
 
         # Number of encoding records (uint16)
-        data << [1].pack("n")  # One encoding record
+        data << [1].pack("n") # One encoding record
 
         # Encoding record: Platform ID (uint16), Encoding ID (uint16), Subtable offset (uint32)
         # Platform 3 (Windows), Encoding 1 (Unicode BMP)
         data << [3].pack("n")           # Platform ID: Windows
         data << [1].pack("n")           # Encoding ID: Unicode BMP
-        data << [subtable_offset].pack("N")  # Subtable offset
+        data << [subtable_offset].pack("N") # Subtable offset
 
         # Append subtable data
         data << subtable_data
@@ -1160,13 +1165,13 @@ module Fontisan
         # Calculate segment count and related values
         seg_count = segments.length
         seg_count_x2 = seg_count * 2
-        search_range = 2 ** (Math.log2(seg_count).to_i) * 2
+        search_range = 2**Math.log2(seg_count).to_i * 2
         entry_selector = Math.log2(search_range / 2).to_i
         range_shift = (seg_count - search_range / 2) * 2
 
         # Build format 4 subtable header (14 bytes)
-        data << [4].pack("n")                    # Format
-        data << [calculate_cmap4_length(segments)].pack("n")  # Length (placeholder)
+        data << [4].pack("n") # Format
+        data << [calculate_cmap4_length(segments)].pack("n") # Length (placeholder)
         data << [0].pack("n")                    # Language (0 = independent)
         data << [seg_count_x2].pack("n")         # segCountX2
         data << [search_range].pack("n")         # searchRange
@@ -1203,9 +1208,9 @@ module Fontisan
 
         # Write arrays (padded to even length)
         end_codes.each { |code| data << [code].pack("n") }
-        data << [0].pack("n")  # Reserved padding
+        data << [0].pack("n") # Reserved padding
         start_codes.each { |code| data << [code].pack("n") }
-        id_deltas.each { |delta| data << [delta].pack("s>") }  # Signed
+        id_deltas.each { |delta| data << [delta].pack("s>") } # Signed
         id_range_offsets.each { |offset| data << [offset].pack("n") }
         glyph_id_array.each { |gid| data << [gid].pack("n") }
 
@@ -1227,7 +1232,7 @@ module Fontisan
         seg_count = segments.length
 
         # Rough estimate (actual calculation done during construction)
-        14 + (seg_count * 8) + (seg_count * 2) + 100  # 100 for glyph ID array estimate
+        14 + (seg_count * 8) + (seg_count * 2) + 100 # 100 for glyph ID array estimate
       end
     end
   end

data/lib/fontisan/hints/hint_converter.rb

@@ -276,7 +276,8 @@ module Fontisan
         # Merge all extracted hints (prep_hints and fpgm_hints override stem widths if present)
         # Note: fpgm_hints contains metadata (fpgm_size, has_functions, complexity)
         # which we must filter out before merging into PostScript dict hints
-        fpgm_dict_hints = fpgm_hints.reject { |k, _| %i[fpgm_size has_functions complexity].include?(k) }
+        fpgm_dict_hints = fpgm_hints.except(:fpgm_size, :has_functions,
+                                            :complexity)
         hints.merge!(prep_hints).merge!(fpgm_dict_hints).merge!(blue_zones)
 
         # Provide default blue_values if none were detected

data/lib/fontisan/loading_modes.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "set"
-
 module Fontisan
   # Loading modes module that defines which tables are loaded in each mode.
   #

data/lib/fontisan/open_type_font.rb

@@ -32,46 +32,6 @@ module Fontisan
     # Page size for lazy loading alignment (typical filesystem page size)
     PAGE_SIZE = 4096
 
-    # Read OpenType Font from a file
-    #
-    # @param path [String] Path to the OTF file
-    # @param mode [Symbol] Loading mode (:metadata or :full, default: :full)
-    # @param lazy [Boolean] If true, load tables on demand (default: false)
-    # @return [OpenTypeFont] A new instance
-    # @raise [ArgumentError] if path is nil or empty, or if mode is invalid
-    # @raise [Errno::ENOENT] if file does not exist
-    # @raise [RuntimeError] if file format is invalid
-    def self.from_file(path, mode: LoadingModes::FULL, lazy: false)
-      if path.nil? || path.to_s.empty?
-        raise ArgumentError,
-              "path cannot be nil or empty"
-      end
-      raise Errno::ENOENT, "File not found: #{path}" unless File.exist?(path)
-
-      # Validate mode
-      LoadingModes.validate_mode!(mode)
-
-      File.open(path, "rb") do |io|
-        font = read(io)
-        font.initialize_storage
-        font.loading_mode = mode
-        font.lazy_load_enabled = lazy
-
-        if lazy
-          # Keep file handle open for lazy loading
-          font.io_source = File.open(path, "rb")
-          font.setup_finalizer
-        else
-          # Read tables upfront
-          font.read_table_data(io)
-        end
-
-        font
-      end
-    rescue BinData::ValidityError, EOFError => e
-      raise "Invalid OTF file: #{e.message}"
-    end
-
     # Initialize storage hashes
     #
     # Extends base class to add page_cache for lazy loading.

data/lib/fontisan/sfnt_font.rb

@@ -149,34 +149,55 @@ module Fontisan
     # @raise [RuntimeError] if file format is invalid
     def self.from_file(path, mode: LoadingModes::FULL, lazy: false)
       if path.nil? || path.to_s.empty?
-        raise ArgumentError,
-              "path cannot be nil or empty"
+        raise ArgumentError, "path cannot be nil or empty"
       end
       raise Errno::ENOENT, "File not found: #{path}" unless File.exist?(path)
 
-      # Validate mode
       LoadingModes.validate_mode!(mode)
 
-      File.open(path, "rb") do |io|
-        font = read(io)
-        font.initialize_storage
-        font.loading_mode = mode
-        font.lazy_load_enabled = lazy
-
-        if lazy
-          # Keep file handle open for lazy loading
-          font.io_source = File.open(path, "rb")
-          font.setup_finalizer
-        else
-          # Read tables upfront
-          font.read_table_data(io)
-        end
+      font = new
+      font.initialize_storage
+      font.loading_mode = mode
+      font.lazy_load_enabled = lazy
+
+      lazy ? load_lazy(path, font) : load_eager(path, font)
+    end
+
+    # Load font with lazy loading (keeps file handle open)
+    #
+    # @param path [String] Path to the font file
+    # @param font [SfntFont] Font instance to populate
+    # @return [SfntFont] The populated font instance
+    def self.load_lazy(path, font)
+      # rubocop:disable Security/Open -- Intentionally kept open for lazy loading
+      # Finalizer handles cleanup when object is GC'd
+      font.io_source = File.open(path, "rb")
+      # rubocop:enable Security/Open
+      font.setup_finalizer
+      font.io_source.rewind
+      font.read(font.io_source)
+      font
+    rescue BinData::ValidityError, EOFError => e
+      font_type = name.split("::").last
+      raise "Invalid #{font_type} file: #{e.message}"
+    end
 
-        font
+    # Load font with eager loading (reads all data, closes file)
+    #
+    # @param path [String] Path to the font file
+    # @param font [SfntFont] Font instance to populate
+    # @return [SfntFont] The populated font instance
+    def self.load_eager(path, font)
+      File.open(path, "rb") do |io|
+        font.read(io)
+        font.read_table_data(io)
       end
+      font
     rescue BinData::ValidityError, EOFError => e
-      raise "Invalid font file: #{e.message}"
+      font_type = name.split("::").last
+      raise "Invalid #{font_type} file: #{e.message}"
     end
+    private_class_method :load_lazy, :load_eager
 
     # Read SFNT Font from collection at specific offset
     #
@@ -422,9 +443,7 @@ module Fontisan
     #
     # @return [Hash<String, SfntTable>] Hash mapping tag => SfntTable instance
     def all_sfnt_tables
-      table_names.each_with_object({}) do |tag, hash|
-        hash[tag] = sfnt_table(tag)
-      end
+      table_names.to_h { |tag| [tag, sfnt_table(tag)] }
     end
 
     # Get parsed table instance

data/lib/fontisan/tables/glyf/compound_glyph.rb

@@ -61,7 +61,6 @@ module Fontisan
         :scale_y,
         :scale_01,
         :scale_10,
-        keyword_init: true,
       ) do
         # Check if arguments are x,y offsets (vs point numbers)
         def args_are_xy?