fontisan 0.4.34 → 0.4.36

data/lib/fontisan/cli.rb

@@ -38,10 +38,14 @@ module Fontisan
                            desc: "Omit the codepoint list from the report"
     option :font_index, type: :numeric, default: nil,
                         desc: "Audit only the given face of a collection"
+    option :validate, type: :string, default: nil,
+                      desc: "Run validation checks. Use 'true' for all, or a " \
+                            "profile name: default, structural, ots, layout"
     # Produce a structured font audit report (YAML or JSON).
     def audit(font_file)
       cmd_options = options.dup
       cmd_options[:include_codepoints] = !options[:no_codepoints]
+      cmd_options[:validate] = parse_validate_option(options[:validate])
       command = Commands::AuditCommand.new(font_file, cmd_options)
       result = command.run
       write_audit_result(result, options[:output])
@@ -797,6 +801,15 @@ module Fontisan
       format("%<idx>02d-%<name>s.%<suffix>s", idx: idx, name: safe_name, suffix: suffix)
     end
 
+    # The --validate option is either nil (no validation), "true" (run all
+    # checks), or a profile name (e.g. "ots", "structural").
+    def parse_validate_option(raw)
+      return nil if raw.nil? || raw.empty?
+      return true if raw == "true"
+
+      raw.to_sym
+    end
+
     def serialize_report(report, format)
       format == :json ? report.to_json : report.to_yaml
     end

data/lib/fontisan/commands/audit_command.rb

@@ -75,6 +75,7 @@ module Fontisan
           populate_style(r, face)
           populate_coverage(r, face)
           populate_layout(r, face)
+          populate_validation(r, face) if validate?
         end
       end
 
@@ -246,11 +247,44 @@ module Fontisan
       end
 
       def features_for_script(layout_table, script)
-        
           layout_table.features(script_tag: script)
       rescue StandardError
           []
-        
+      end
+
+      # ------------------------------------------------------------------
+      # Validation (audit checks)
+      # ------------------------------------------------------------------
+
+      def populate_validation(report, face)
+        report.validation_issues = run_validation_checks(face)
+      end
+
+      def run_validation_checks(face)
+        checks = Audit::CheckRegistry.for(validation_profile)
+        checks.flat_map { |check| safe_run_check(check, face) }
+      end
+
+      def safe_run_check(check, face)
+        check.call(face)
+      rescue StandardError => e
+        [Models::ValidationReport::Issue.new(
+          severity: "fatal",
+          category: check.code.to_s,
+          message: "Check #{check.code} failed to execute: #{e.class}: #{e.message}",
+          location: nil,
+        )]
+      end
+
+      def validate?
+        @options.fetch(:validate, false)
+      end
+
+      def validation_profile
+        profile = @options[:validate]
+        return :default if profile == true
+
+        profile&.to_sym || :default
       end
     end
   end

data/lib/fontisan/models/audit_report.rb

@@ -73,6 +73,9 @@ module Fontisan
       attribute :opentype_scripts, :string, collection: true
       attribute :features, :string, collection: true
 
+      # Validation (populated only with --validate)
+      attribute :validation_issues, Models::ValidationReport::Issue, collection: true
+
       json do
         map "generated_at", to: :generated_at
         map "fontisan_version", to: :fontisan_version
@@ -104,6 +107,7 @@ module Fontisan
 
         map "opentype_scripts", to: :opentype_scripts
         map "features", to: :features
+        map "validation_issues", to: :validation_issues
       end
 
       yaml do
@@ -137,6 +141,7 @@ module Fontisan
 
         map "opentype_scripts", to: :opentype_scripts
         map "features", to: :features
+        map "validation_issues", to: :validation_issues
       end
     end
   end

data/lib/fontisan/version.rb

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

data/lib/fontisan.rb

@@ -77,6 +77,7 @@ module Fontisan
 
   # Namespace hubs (each hub declares its own child autoloads)
   autoload :Binary, "fontisan/binary"
+  autoload :Audit, "fontisan/audit"
   autoload :Collection, "fontisan/collection"
   autoload :Commands, "fontisan/commands"
   autoload :Converters, "fontisan/converters"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fontisan
 version: !ruby/object:Gem::Version
-  version: 0.4.34
+  version: 0.4.36
 platform: ruby
 authors:
 - Ribose Inc.
@@ -293,6 +293,20 @@ files:
 - exe/fontisan
 - fontisan.gemspec
 - lib/fontisan.rb
+- lib/fontisan/audit.rb
+- lib/fontisan/audit/check.rb
+- lib/fontisan/audit/check_registry.rb
+- lib/fontisan/audit/checks.rb
+- lib/fontisan/audit/checks/cmap_check.rb
+- lib/fontisan/audit/checks/collection_integrity_check.rb
+- lib/fontisan/audit/checks/format_round_trip_check.rb
+- lib/fontisan/audit/checks/glyph_name_check.rb
+- lib/fontisan/audit/checks/hinting_check.rb
+- lib/fontisan/audit/checks/opentype_conformance_check.rb
+- lib/fontisan/audit/checks/ots_compatibility_check.rb
+- lib/fontisan/audit/checks/table_directory_check.rb
+- lib/fontisan/audit/checks/variable_font_check.rb
+- lib/fontisan/audit/checks/woff2_validation_check.rb
 - lib/fontisan/base_collection.rb
 - lib/fontisan/binary.rb
 - lib/fontisan/binary/base_record.rb