fontist 3.0.30 → 3.0.31

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d858c57f16d09454000112d1047f120b1a752a76331e3788f65cb441fe7a668c
-  data.tar.gz: 60c5f0e33198e6b7b8ad24db123280a5fff7c4eaf8b56581d3ab8c9108114b44
+  metadata.gz: 2e9016d35c6100b87e2a7585f7193a8fbac76a1a32ab01e178a2d95bef017fa5
+  data.tar.gz: 6f02798721c34a5402bf413c9e1c003c3e968d1e01de44ae3562b726cf65bdb4
 SHA512:
-  metadata.gz: 581ebfd99f20a966cea6574294b4a3de879cdcea6051f916645e84ac9bd5e9ee16d4d0bfcbbd82f0291b6d6ae3e72dcf1c5524d02f6e1d67d3fbcfd880b03d73
-  data.tar.gz: 7188a89be063579e76ffd351c5ddd0c7296836a4c6162c1974ca54f79ca5cddd6134a71b3d48d04455fbc1ac8debbe1e6f2922d414a16aea2e85c83a62742fc2
+  metadata.gz: 7b2a062de85a9d66e0ec4e6e6d726c1d6716f8653c4ee96313eea8ba379941476064121d014851ce15bc6e7cf59ee4dcf3b77d68cd43fef7863238b9bbb55a68
+  data.tar.gz: 4d92357d6f58a7a1588bf4e6fb7793c94bbd8e47ecd1106cf9bbab87ef4321a4dd7fd172cbafab873f10c27c2563224fb7634fc0ad652baeda59e2ec433597b7

data/lib/fontist/font_file.rb

@@ -63,14 +63,30 @@ module Fontist
           font = validate_and_load_single_font(path)
         end
         names = extract_names_from_font(font)
-        # WOFF and WOFF2 fonts from Fontisan hold no open file to close.
+        # `has_table?` consults the table directory, which survives the
+        # metadata-mode load, so the fvar check is cheap; only an actual
+        # variable font pays for the follow-up full read. WOFF and WOFF2
+        # fonts from Fontisan hold no open file to close.
+        variable = font.has_table?("fvar")
         font.close if font.respond_to?(:close)
+        names[:variable_instances] = extract_variable_instances_from_path(path) if
+          variable && names[:family_name]
         names
       rescue StandardError => e
         raise_font_file_error(e)
       end
       # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
 
+      def extract_variable_instances_from_path(path)
+        # Full mode: the metadata-mode loader gates fvar data out, so the
+        # instance pass needs the real table read; this runs only for
+        # fonts already identified as variable by the cheap directory check.
+        font = Fontisan::FontLoader.load(path, mode: :full, lazy: true)
+        extract_variable_instances(font)
+      ensure
+        font.close if font.respond_to?(:close)
+      end
+
       # rubocop:disable Metrics/MethodLength
       def validate_and_load_single_font(path)
         # Load font first (we need to validate it, not the file path)
@@ -183,6 +199,62 @@ module Fontist
       end
       # rubocop:enable Metrics/MethodLength
 
+      # Named instances of a variable font, as `{family:, style:, full_name:}`
+      # triples. A variable file carries every named instance in one fvar
+      # table, but its name table exposes only the default instance's
+      # family, so an index built from the name table alone registers the
+      # file under a single family ("Fira Code Light") and every lookup of
+      # the other families/styles misses it. The RIBBI mapping follows the
+      # convention static families use: Regular/Bold live on the base
+      # family, every other weight becomes its own "Base Weight" family.
+      #
+      # @return [Array<Hash>] empty for a static font
+      # rubocop:disable Metrics/MethodLength
+      def extract_variable_instances(font)
+        return [] unless font.has_table?("fvar")
+
+        fvar = font.table("fvar")
+        name_table = font.table(Fontisan::Constants::NAME_TAG)
+        return [] unless name_table
+
+        base = name_table.english_name(Fontisan::Tables::Name::PREFERRED_FAMILY) ||
+          extract_any_name(name_table, Fontisan::Tables::Name::PREFERRED_FAMILY) ||
+          name_table.english_name(Fontisan::Tables::Name::FAMILY)
+        return [] unless base
+
+        ribbi = ["regular", "bold", "italic", "bold italic"]
+
+        fvar.instances.filter_map do |instance|
+          # fvar yields plain hashes ({name_id:, flags:, coordinates:,
+          # postscript_name_id:}), not message objects.
+          style = name_table.english_name(instance[:name_id]) ||
+            extract_any_name(name_table, instance[:name_id])
+          next unless style
+
+          if ribbi.include?(style.downcase)
+            family = base
+            type = style
+            full_name = "#{family} #{style}"
+          else
+            family = "#{base} #{style}"
+            type = "Regular"
+            full_name = family
+          end
+
+          {
+            family: family,
+            style: type,
+            full_name: full_name,
+          }
+        end
+      rescue StandardError => e
+        Fontist.ui.debug(
+          "Could not read variable instances: #{e.message}",
+        )
+        []
+      end
+      # rubocop:enable Metrics/MethodLength
+
       # Extract any available name for a given name ID by manually decoding the name record
       def extract_any_name(name_table, name_id)
         # Find any name record with the matching name_id
@@ -256,5 +328,11 @@ module Fontist
     def preferred_subfamily_name
       @info[:preferred_subfamily]
     end
+
+    # Named instances of a variable font (`{family:, style:, full_name:}`
+    # triples); empty for a static font.
+    def variable_instances
+      @info[:variable_instances] || []
+    end
   end
 end

data/lib/fontist/system_index.rb

@@ -729,9 +729,48 @@ spinner_index = nil)
     def detect_file_font(path)
       font_file = FontFile.from_path(path)
 
+      entries = variable_font_entries(font_file, path)
+      return entries unless entries.empty?
+
       parse_font(font_file, path)
     end
 
+    # One index entry per named instance of a variable font: a variable
+    # file answers every lookup only if each family/style pair it carries
+    # is indexed separately — the name table alone registers just the
+    # default instance ("Fira Code Light"), leaving find_styles blind to
+    # the rest of the set.
+    def variable_font_entries(font_file, path)
+      instances = font_file.variable_instances
+      return [] if instances.empty?
+
+      file_size = begin
+        File.size(path)
+      rescue StandardError
+        0
+      end
+      file_mtime = begin
+        File.mtime(path).to_i
+      rescue StandardError
+        0
+      end
+
+      base = font_file.preferred_family_name || font_file.family
+      instances.map do |instance|
+        family = instance[:family]
+        SystemIndexFont.new(
+          path: path,
+          full_name: instance[:full_name],
+          family_name: family,
+          subfamily: instance[:style],
+          preferred_family_name: base,
+          preferred_subfamily_name: instance[:style],
+          file_size: file_size,
+          file_mtime: file_mtime,
+        )
+      end
+    end
+
     def detect_collection_fonts(path)
       CollectionFile.from_path(path) do |collection|
         collection.map do |font_file|

data/lib/fontist/version.rb

@@ -1,3 +1,3 @@
 module Fontist
-  VERSION = "3.0.30".freeze
+  VERSION = "3.0.31".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fontist
 version: !ruby/object:Gem::Version
-  version: 3.0.30
+  version: 3.0.31
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-11-03 00:00:00.000000000 Z
+date: 2026-11-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: down