fontisan 0.4.33 → 0.4.34

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b16e0b97aeb7431384ce5f50ff3e4dfd8977b863ec76655ef09e6d53fefddc3c
-  data.tar.gz: b60547f43a45309f700f4a11bb67d36788cfde286de179b697a7fbfc3d43a897
+  metadata.gz: 1c278ba38670bfc65e41805d14d6bc175ae451f0e5400e4b25036bf3ae6a84f6
+  data.tar.gz: 59ec85b42b6879f589c13429e8e490a41eb32257530207647203c010ee461182
 SHA512:
-  metadata.gz: c5b05277f8c003e64537cfeb57eb933b8e34f26449acc86de639b1ed90e8b344e502f0a8181417c92de1b8d4c51ffbd98ed9ccacf04ae38b4ff5e0deeb6e80b4
-  data.tar.gz: ee1631c92ebf6a5fd7ba66c4f40bef089a8725560f08d013fd16dfafed46e6c8c3890d35128ded48d03c996f6a547bac1f2d8c17939a12bf5a429bd6ac9a1f58
+  metadata.gz: 17b9aeaeeb0589379b2ff65413aabb3db15c4d0dd323bd4891e4863a9dca2fdb7190fd795ebcd62fa16d6d570e7f7bb50c740b9947c0b42a5e673380e0164cd2
+  data.tar.gz: c9b35dd44d43fc634b1bf07aa84d2bd99527e4695636424cbf46aca757615b09d6dc27f3512e701ab8676d1111f51f980dd97d0d0fd82c47f4177592f9b94c95

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2026-07-09 14:55:09 UTC using RuboCop version 1.86.1.
+# on 2026-07-11 04:55:54 UTC using RuboCop version 1.86.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -12,182 +12,39 @@ Gemspec/RequiredRubyVersion:
     - 'docs/node_modules/speakingurl/speakingurl-rails.gemspec'
     - 'fontisan.gemspec'
 
-# Offense count: 276
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: with_first_argument, with_fixed_indentation
-Layout/ArgumentAlignment:
-  Enabled: false
-
-# Offense count: 22
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: with_first_element, with_fixed_indentation
-Layout/ArrayAlignment:
-  Exclude:
-    - 'lib/fontisan/collection/writer.rb'
-    - 'lib/fontisan/tables/cff2/index_builder.rb'
-    - 'lib/fontisan/ufo/compile/cbdt_cblc.rb'
-    - 'lib/fontisan/ufo/compile/cff.rb'
-    - 'lib/fontisan/ufo/compile/cmap.rb'
-    - 'lib/fontisan/ufo/compile/cpal.rb'
-    - 'lib/fontisan/ufo/compile/gvar.rb'
-    - 'lib/fontisan/ufo/compile/item_variation_store.rb'
-    - 'lib/fontisan/ufo/compile/sbix.rb'
-    - 'lib/fontisan/ufo/compile/stat.rb'
-    - 'lib/fontisan/ufo/info.rb'
-    - 'spec/fontisan/stitcher/glyph_signature_spec.rb'
-    - 'spec/fontisan/svg_to_glyf/contour_builder_spec.rb'
-    - 'spec/fontisan/ufo/compile/feature_writers/mkmk_spec.rb'
-    - 'spec/fontisan/ufo/compile/filters/transformations_spec.rb'
-
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: IndentationWidth.
-Layout/AssignmentIndentation:
-  Exclude:
-    - 'lib/fontisan/woff2/glyf_loca_reconstruct.rb'
-
-# Offense count: 58
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyleAlignWith.
-# SupportedStylesAlignWith: either, start_of_block, start_of_line
-Layout/BlockAlignment:
-  Enabled: false
-
-# Offense count: 55
-# This cop supports safe autocorrection (--autocorrect).
-Layout/BlockEndNewline:
-  Enabled: false
-
-# Offense count: 29
-# This cop supports safe autocorrection (--autocorrect).
-Layout/ClosingParenthesisIndentation:
-  Exclude:
-    - 'benchmark/compile_benchmark.rb'
-    - 'lib/fontisan/ufo/compile/fvar.rb'
-    - 'lib/fontisan/ufo/writer.rb'
-    - 'spec/fontisan/stitcher/deduplicator_spec.rb'
-    - 'spec/fontisan/stitcher/include_codepoints_map_spec.rb'
-    - 'spec/fontisan/svg_to_glyf/affine_transform_spec.rb'
-    - 'spec/fontisan/ufo/compile/filters/propagate_anchors_spec.rb'
-    - 'spec/fontisan/ufo/compile_gvar_spec.rb'
-    - 'spec/fontisan/ufo/compile_item_variation_store_spec.rb'
-    - 'spec/fontisan/ufo/filters_spec.rb'
-
-# Offense count: 2
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowForAlignment.
-Layout/CommentIndentation:
-  Exclude:
-    - 'lib/fontisan/ufo/compile/cff.rb'
-
 # Offense count: 4
 # This cop supports safe autocorrection (--autocorrect).
-Layout/ElseAlignment:
-  Exclude:
-    - 'lib/fontisan/commands/validate_collection_command.rb'
-    - 'lib/fontisan/converters/collection_converter.rb'
-    - 'spec/fontisan/woff2/sfnt_checksum_spec.rb'
-
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyleAlignWith.
-# SupportedStylesAlignWith: keyword, variable, start_of_line
-Layout/EndAlignment:
-  Exclude:
-    - 'lib/fontisan/commands/validate_collection_command.rb'
-    - 'lib/fontisan/converters/collection_converter.rb'
-    - 'spec/fontisan/woff2/sfnt_checksum_spec.rb'
-
-# Offense count: 29
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: consistent, consistent_relative_to_receiver, special_for_inner_method_call, special_for_inner_method_call_in_parentheses
-Layout/FirstArgumentIndentation:
-  Exclude:
-    - 'benchmark/compile_benchmark.rb'
-    - 'lib/fontisan/ufo/compile/fvar.rb'
-    - 'lib/fontisan/ufo/writer.rb'
-    - 'spec/fontisan/stitcher/deduplicator_spec.rb'
-    - 'spec/fontisan/stitcher/include_codepoints_map_spec.rb'
-    - 'spec/fontisan/svg_to_glyf/affine_transform_spec.rb'
-    - 'spec/fontisan/ufo/compile/filters/propagate_anchors_spec.rb'
-    - 'spec/fontisan/ufo/compile_gvar_spec.rb'
-    - 'spec/fontisan/ufo/compile_item_variation_store_spec.rb'
-    - 'spec/fontisan/ufo/filters_spec.rb'
-
-# Offense count: 14
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: special_inside_parentheses, consistent, align_brackets
-Layout/FirstArrayElementIndentation:
-  Exclude:
-    - 'spec/fontisan/stitcher/glyph_signature_spec.rb'
-    - 'spec/fontisan/ufo/compile/feature_writers/mark_spec.rb'
-
-# Offense count: 194
-# This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowMultipleStyles, EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle.
 # SupportedHashRocketStyles: key, separator, table
 # SupportedColonStyles: key, separator, table
 # SupportedLastArgumentHashStyles: always_inspect, always_ignore, ignore_implicit, ignore_explicit
 Layout/HashAlignment:
-  Enabled: false
-
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: normal, indented_internal_methods
-Layout/IndentationConsistency:
   Exclude:
-    - 'lib/fontisan/ufo/compile/cff.rb'
+    - 'lib/fontisan/ufo/compile/variable_otf.rb'
+    - 'spec/fontisan/ufo/compile/cff2_spec.rb'
 
-# Offense count: 121
+# Offense count: 9
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: Width, EnforcedStyleAlignWith, AllowedPatterns.
 # SupportedStylesAlignWith: start_of_line, relative_to_receiver
 Layout/IndentationWidth:
-  Enabled: false
+  Exclude:
+    - 'lib/fontisan/commands/audit_command.rb'
+    - 'lib/fontisan/ufo/compile/glyf_loca.rb'
 
-# Offense count: 2235
+# Offense count: 1969
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: Max, AllowHeredoc, AllowURI, AllowQualifiedName, URISchemes, AllowRBSInlineAnnotation, AllowCopDirectives, AllowedPatterns, SplitStrings.
 # URISchemes: http, https
 Layout/LineLength:
   Enabled: false
 
-# Offense count: 7
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: symmetrical, new_line, same_line
-Layout/MultilineArrayBraceLayout:
-  Exclude:
-    - 'spec/fontisan/stitcher/glyph_signature_spec.rb'
-    - 'spec/fontisan/ufo/compile/feature_writers/mark_spec.rb'
-
-# Offense count: 29
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: symmetrical, new_line, same_line
-Layout/MultilineMethodCallBraceLayout:
-  Exclude:
-    - 'benchmark/compile_benchmark.rb'
-    - 'lib/fontisan/ufo/compile/fvar.rb'
-    - 'lib/fontisan/ufo/writer.rb'
-    - 'spec/fontisan/stitcher/deduplicator_spec.rb'
-    - 'spec/fontisan/stitcher/include_codepoints_map_spec.rb'
-    - 'spec/fontisan/svg_to_glyf/affine_transform_spec.rb'
-    - 'spec/fontisan/ufo/compile/filters/propagate_anchors_spec.rb'
-    - 'spec/fontisan/ufo/compile_gvar_spec.rb'
-    - 'spec/fontisan/ufo/compile_item_variation_store_spec.rb'
-    - 'spec/fontisan/ufo/filters_spec.rb'
-
-# Offense count: 225
+# Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowInHeredoc.
 Layout/TrailingWhitespace:
-  Enabled: false
+  Exclude:
+    - 'lib/fontisan/commands/audit_command.rb'
 
 # Offense count: 2
 Lint/CopDirectiveSyntax:
@@ -248,7 +105,7 @@ Lint/StructNewOverride:
   Exclude:
     - 'lib/fontisan/optimizers/pattern_analyzer.rb'
 
-# Offense count: 24
+# Offense count: 33
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods, IgnoreNotImplementedMethods, NotImplementedExceptions.
 # NotImplementedExceptions: NotImplementedError
@@ -261,12 +118,12 @@ Lint/UselessConstantScoping:
     - 'lib/fontisan/conversion_options.rb'
     - 'lib/fontisan/type1/charstrings.rb'
 
-# Offense count: 715
+# Offense count: 752
 # Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes, Max.
 Metrics/AbcSize:
   Enabled: false
 
-# Offense count: 34
+# Offense count: 37
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns, inherit_mode.
 # AllowedMethods: refine
 Metrics/BlockLength:
@@ -284,28 +141,28 @@ Metrics/CollectionLiteralLength:
     - 'lib/fontisan/type1/agl.rb'
     - 'lib/fontisan/type1/encodings.rb'
 
-# Offense count: 367
+# Offense count: 375
 # Configuration parameters: AllowedMethods, AllowedPatterns, Max.
 Metrics/CyclomaticComplexity:
   Enabled: false
 
-# Offense count: 990
+# Offense count: 1019
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 Metrics/MethodLength:
   Max: 135
 
-# Offense count: 30
+# Offense count: 31
 # Configuration parameters: CountKeywordArgs.
 Metrics/ParameterLists:
   Max: 39
   MaxOptionalParameters: 4
 
-# Offense count: 279
+# Offense count: 289
 # Configuration parameters: AllowedMethods, AllowedPatterns, Max.
 Metrics/PerceivedComplexity:
   Enabled: false
 
-# Offense count: 202
+# Offense count: 209
 # Configuration parameters: MinNameLength, AllowNamesEndingInNumbers, AllowedNames, ForbiddenNames.
 # AllowedNames: as, at, by, cc, db, id, if, in, io, ip, of, on, os, pp, to
 Naming/MethodParameterName:
@@ -371,7 +228,7 @@ RSpec/AnyInstance:
 RSpec/ContextWording:
   Enabled: false
 
-# Offense count: 39
+# Offense count: 40
 # Configuration parameters: IgnoredMetadata.
 RSpec/DescribeClass:
   Enabled: false
@@ -387,7 +244,7 @@ RSpec/DescribeMethod:
     - 'spec/fontisan/type1/seac_expander_spec.rb'
     - 'spec/fontisan/type1_real_fonts_spec.rb'
 
-# Offense count: 1710
+# Offense count: 1726
 # Configuration parameters: CountAsOne.
 RSpec/ExampleLength:
   Max: 66
@@ -465,7 +322,7 @@ RSpec/MultipleDescribes:
     - 'spec/fontisan/utils/thread_pool_spec.rb'
     - 'spec/fontisan/variation/cache_spec.rb'
 
-# Offense count: 2126
+# Offense count: 2149
 RSpec/MultipleExpectations:
   Max: 19
 
@@ -529,16 +386,6 @@ Rake/MethodDefinitionInTask:
   Exclude:
     - 'Rakefile'
 
-# Offense count: 68
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, ProceduralMethods, FunctionalMethods, AllowedMethods, AllowedPatterns, AllowBracesOnProceduralOneLiners, BracesRequiredMethods.
-# SupportedStyles: line_count_based, semantic, braces_for_chaining, always_braces
-# ProceduralMethods: benchmark, bm, bmbm, create, each_with_object, measure, new, realtime, tap, with_object
-# FunctionalMethods: let, let!, subject, watch
-# AllowedMethods: lambda, proc, it
-Style/BlockDelimiters:
-  Enabled: false
-
 # Offense count: 6
 # This cop supports safe autocorrection (--autocorrect).
 Style/ComparableClamp:
@@ -573,47 +420,9 @@ Style/HashLikeCase:
     - 'lib/fontisan/commands/unpack_command.rb'
     - 'lib/fontisan/models/validation_report.rb'
 
-# Offense count: 16
-# This cop supports safe autocorrection (--autocorrect).
-Style/MultilineIfModifier:
-  Exclude:
-    - 'lib/fontisan/svg/standalone_glyph.rb'
-    - 'lib/fontisan/svg_to_glyf/geometry/transform_parser.rb'
-    - 'lib/fontisan/ufo/cli.rb'
-    - 'lib/fontisan/ufo/convert.rb'
-    - 'lib/fontisan/ufo/convert/from_bin_data.rb'
-    - 'lib/fontisan/ufo/convert/to_dfont.rb'
-    - 'lib/fontisan/ufo/convert/to_otc.rb'
-    - 'lib/fontisan/ufo/convert/to_postscript.rb'
-    - 'lib/fontisan/ufo/convert/to_woff.rb'
-    - 'lib/fontisan/ufo/convert/to_woff2.rb'
-    - 'lib/fontisan/ufo/plist.rb'
-    - 'lib/fontisan/woff2/collection_decoder.rb'
-    - 'lib/fontisan/woff2/collection_encoder.rb'
-    - 'spec/support/fixture_downloader.rb'
-
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-Style/MultilineTernaryOperator:
-  Exclude:
-    - 'lib/fontisan/commands/validate_collection_command.rb'
-    - 'lib/fontisan/converters/collection_converter.rb'
-    - 'spec/fontisan/woff2/sfnt_checksum_spec.rb'
-
 # Offense count: 1
 # Configuration parameters: AllowedMethods.
 # AllowedMethods: respond_to_missing?
 Style/OptionalBooleanParameter:
   Exclude:
     - 'lib/fontisan/conversion_options.rb'
-
-# Offense count: 22
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyleForMultiline.
-# SupportedStylesForMultiline: comma, consistent_comma, diff_comma, no_comma
-Style/TrailingCommaInArguments:
-  Exclude:
-    - 'benchmark/compile_benchmark.rb'
-    - 'spec/fontisan/ufo/compile/filters/propagate_anchors_spec.rb'
-    - 'spec/fontisan/ufo/compile_gvar_spec.rb'
-    - 'spec/fontisan/ufo/filters_spec.rb'

data/TODO.improvements/README.md

@@ -11,7 +11,7 @@ Each file is `NN-short-name.md` where `NN` is the priority order.
 - [x] ~~[15 — CFF/CFF2 subsetter strategy](15-cff-cff2-subsetter-strategy.md)~~ ✓ Done — CFF via UFO round-trip, CFF2 via standalone INDEX filter with VStore preservation
 
 ### P1 — High-value features
-- [03 — `fontisan audit` command (identity+style+features lens)](03-fontisan-audit-command.md)
+- [x] ~~[03 — `fontisan audit` command (identity+style+features lens)](03-fontisan-audit-command.md)~~ ✓ Done (Phase 1: identity, style, coverage, OpenType layout, provenance; CLI `fontisan audit` with YAML/JSON output)
 - [x] ~~[04 — UFO composite glyph encoding](04-ufo-composite-glyph-encoding.md)~~ ✓ Done
 - [x] ~~[05 — OTF compiler real CFF charstrings](05-otf-compiler-real-cff.md)~~ ✓ Already working (discovered in PR #117 — Cff.build produces real Type 2 charstrings; stale TODO marker removed)
 
@@ -19,14 +19,14 @@ Each file is `NN-short-name.md` where `NN` is the priority order.
 - [x] ~~[07 — CPAL v1 header fields](07-cpal-v1-header-fields.md)~~ ✓ Done (v0.4.25)
 - [x] ~~[08 — CFF standard string table](08-cff-standard-string-table.md)~~ ✓ Done (full 391 SIDs per Adobe TN 5176)
 - [x] ~~[06 — CFF2 blend/vsindex operators](06-cff2-blend-vsindex-operators.md)~~ ✓ Done — `Cff2.build_variable` emits blend operators in charstrings; `VariableOtf` passes masters through; VStore embedded with regions
-- [09 — Type 1 seac expansion](09-type1-seac-expansion.md)
-- [10 — UFO image set + feature writers](10-ufo-image-set-feature-writers.md)
+- [x] ~~[09 — Type 1 seac expansion](09-type1-seac-expansion.md)~~ ✓ Done — recursive seac expansion with cycle detection; `CharStrings.from_hash` factory added for clean test construction
+- [x] ~~[10 — UFO image set + feature writers](10-ufo-image-set-feature-writers.md)~~ ✓ Done — `ImageSet` loads/writes UFO 3 images directory; `FeatureCompiler` parses liga/kern FEA constructs; `Compile::Gsub` builds LigatureSubst GSUB
 - [x] ~~[11 — kern groups.plist support](11-kern-groups-plist.md)~~ ✓ Done (Groups model in PR #116 + GPOS group resolution in this PR)
 
 ### P3 — Code-quality cleanup
 - [x] ~~[13 — Split `OctokitFetcher` out of `fixture_downloader.rb`](13-split-octokit-fetcher.md)~~ ✓ Done (v0.4.24)
 - [x] ~~[12 — `cbdt_fixture.rb` full BinData conversion](12-cbdt-fixture-bindata-conversion.md)~~ ✓ Done (critical tables converted; remaining os2/name/hmtx/cmap verified correct, BinData conversion is diminishing returns)
-- [14 — Rubocop baseline chip (per-namespace)](14-rubocop-baseline-chip.md)
+- [x] ~~[14 — Rubocop baseline chip (per-namespace)](14-rubocop-baseline-chip.md)~~ ✓ Partially done — baseline reduced from 619 to 428 lines; all namespaces pass `rubocop` with 0 offenses; remaining baseline offenses are non-autocorrectable manual fixes (~7h total per TODO estimate)
 
 ## Convention
 

data/lib/fontisan/cli.rb

@@ -31,6 +31,25 @@ module Fontisan
     desc "stitch", "Stitch glyphs from multiple source fonts into one output"
     subcommand "stitch", StitcherCli
 
+    desc "audit FONT_FILE", "Produce a structured audit report (identity, style, coverage, layout)"
+    option :output, type: :string,
+                    desc: "Output file path. For collections, use a directory path."
+    option :no_codepoints, type: :boolean, default: false,
+                           desc: "Omit the codepoint list from the report"
+    option :font_index, type: :numeric, default: nil,
+                        desc: "Audit only the given face of a collection"
+    # Produce a structured font audit report (YAML or JSON).
+    def audit(font_file)
+      cmd_options = options.dup
+      cmd_options[:include_codepoints] = !options[:no_codepoints]
+      command = Commands::AuditCommand.new(font_file, cmd_options)
+      result = command.run
+      write_audit_result(result, options[:output])
+    rescue Errno::ENOENT
+      warn "File not found: #{font_file}" unless options[:quiet]
+      exit 1
+    end
+
     desc "info PATH", "Display font information"
     option :brief, type: :boolean, default: false,
                    desc: "Brief mode - only essential info (5x faster, uses metadata loading)",
@@ -732,6 +751,52 @@ module Fontisan
       puts output unless options[:quiet]
     end
 
+    # Write the audit result to stdout or a file. For a single report,
+    # the output path (if given) is the file path. For a collection,
+    # the output path is a directory; each face is written as
+    # `{font_index:02d}-{postscript_name}.yaml`.
+    def write_audit_result(result, output_path)
+      if result.is_a?(Array)
+        write_audit_collection(result, output_path)
+      else
+        write_audit_single(result, output_path)
+      end
+    end
+
+    def write_audit_single(report, output_path)
+      content = serialize_audit(report)
+      if output_path
+        FileUtils.mkpath(File.dirname(output_path)) unless File.dirname(output_path) == "."
+        File.write(output_path, content)
+      else
+        puts content unless options[:quiet]
+      end
+    end
+
+    def write_audit_collection(reports, output_path)
+      if output_path
+        FileUtils.mkpath(output_path)
+        reports.each do |report|
+          name = audit_face_filename(report)
+          File.write(File.join(output_path, name), serialize_audit(report))
+        end
+        say("Wrote #{reports.size} reports to #{output_path}") unless options[:quiet]
+      else
+        puts serialize_audit(reports) unless options[:quiet]
+      end
+    end
+
+    def serialize_audit(report)
+      options[:format] == "json" ? report.to_json : report.to_yaml
+    end
+
+    def audit_face_filename(report)
+      safe_name = report.postscript_name&.gsub(%r{[^A-Za-z0-9._-]}, "_") || "font"
+      suffix = options[:format] == "json" ? "json" : "yaml"
+      idx = report.font_index || 0
+      format("%<idx>02d-%<name>s.%<suffix>s", idx: idx, name: safe_name, suffix: suffix)
+    end
+
     def serialize_report(report, format)
       format == :json ? report.to_json : report.to_yaml
     end