klenod-build 0.0.7 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 408f4aa3b1757ef9a4900f2941bdff0cd13b6938cf41b122bca598e4701a1c78
-  data.tar.gz: 21b414e2b4c827b9d63c5f52f50563511f4cd6baa3393c6f19469a5e9654955d
+  metadata.gz: a357752604a0766c4448f82142e21ae3a9d51863839c538aa3fe04d30719a21b
+  data.tar.gz: ab3ffc5a88004014fe159b6ae3690c6e3f61ed0a0ba73cdac1f34499cc474262
 SHA512:
-  metadata.gz: 672880c84e22c78a458a47412156bf1531707f2d71e1df74313ec1d99c2ca83ab5ee101f888ca4cc2a62974d9fc77e59af47c0aa4555cbbb8d2c3980c84b3feb
-  data.tar.gz: 444c25581df5391724b91dddfd6727f3c8571be0ac0d4713ebbb2ab7031728b071c7a8d8dd8190fbaed883febd9ca3d0c0e9df8ab8bde671336d1860079a9763
+  metadata.gz: 28bcd4ee5726533dffa7ed0dac3b57e0a677fb050e067bc230a14e74895c164bd6caceec598c3847823cd421f8b7b18d6f4e30a0d62eef07883b30fc5615b95b
+  data.tar.gz: bd7a69a04c303d7719f44ca3eba9f00e6296e435b1970748781095427a525632d141e2506d06e19d2c6a1d0a6c9a340b1cc310beb18fa052db7491f1bc275f9e

data/lib/klenod/build/context.rb

@@ -97,6 +97,7 @@ module Klenod
           Plugins::GemImportPlugin.new,
           Plugins::SvgPlugin.new,
           Plugins::ImagePlugin.new,
+          Plugins::StaticAssetPlugin.new,
           Plugins::JsonPlugin.new,
           Plugins::YamlPlugin.new,
           Plugins::TomlPlugin.new,

data/lib/klenod/build/graph.rb

@@ -56,7 +56,7 @@ module Klenod
         end
       end
 
-      attr_reader :records, :mods, :asset_generation_queue, :mode, :profiler, :base
+      attr_reader :records, :mods, :asset_generation_queue, :mode, :profiler, :base, :namespace
       attr_reader :plugins
 
       def initialize(
@@ -80,6 +80,7 @@ module Klenod
           )
         @records = {}
         @mods = {}
+        @namespace = Module.new
         @virtual_sources = {}
         @virtual_metadata = {}
         @virtual_owners = {}
@@ -358,7 +359,8 @@ module Klenod
             imports: imports_for(record.resolved_dependencies, dependency_records),
             source_map: record.source_map,
             version: record.version,
-            eval_path: eval_path_for(module_id)
+            eval_path: eval_path_for(module_id),
+            namespace: namespace
           )
 
         @mods[module_id] = mod
@@ -540,7 +542,8 @@ module Klenod
           imports: imports_for(resolved_dependencies, dependency_records),
           source_map: transform.source_map,
           version: cached ? cached.version + 1 : 0,
-          eval_path: eval_path_for(module_id)
+          eval_path: eval_path_for(module_id),
+          namespace: namespace
         )
       end
 

data/lib/klenod/build/plugins/asset_javascript_metadata.rb

@@ -33,9 +33,10 @@ module Klenod
           }
 
           export default class ImageMetadata extends ImageBase {
-            constructor({ src, width, height, contentType, aspectRatio, variants = [] }) {
+            constructor({ src, width, height, contentType, aspectRatio, variants = [], placeholder = null }) {
               super({ src, width, height, contentType, aspectRatio });
               this.variants = Object.freeze([...variants]);
+              this.placeholder = placeholder;
               Object.freeze(this);
             }
 

data/lib/klenod/build/plugins/haml_plugin/transformer/ruby_builder.rb

@@ -442,6 +442,15 @@ module Klenod
               ast_silent_script(source) || raise(ArgumentError, "Could not build Ruby begin block from Haml script: #{source.inspect}")
             end
 
+            def silent_script_with_children(source, body)
+              source = rewrite_ruby_source(source, nil)
+              statements = parse_statements(source)
+              return raise(ArgumentError, "Could not parse Haml silent script: #{source.inspect}") unless statements
+
+              node = ast_begin([*statement_body_for(statements), *statement_body_for(body)])
+              Fragment.new(["begin", indent(source, 2), indent(to_source(body), 2), "end"].join("\n"), node)
+            end
+
             def branches(branches)
               ast_branches(branches) || raise(ArgumentError, "Could not build Ruby branch from Haml scripts: #{branches.map(&:first).inspect}")
             end
@@ -550,11 +559,12 @@ module Klenod
                 .filter_map do |(line, column), type, token, _state|
                   kind, prefix, pattern = VARIABLE_TOKEN_KINDS[type]
                   receiver = @variables[kind]
-                  next unless receiver && token.match?(pattern)
-
-                  name = token.delete_prefix(prefix)
-                  offset = line_offsets.fetch(line - 1) + column
-                  [offset, token.length, "(#{receiver})[#{symbol_source(name)}]"]
+                  if type == :on_gvar && token == "$*" && receiver
+                    [offset_for(line_offsets, line, column), token.length, receiver]
+                  elsif receiver && token.match?(pattern)
+                    name = token.delete_prefix(prefix)
+                    [offset_for(line_offsets, line, column), token.length, "(#{receiver})[#{symbol_source(name)}]"]
+                  end
                 end
                 .reverse_each
                 .each_with_object(source.dup) do |(offset, length, replacement), rewritten|
@@ -562,6 +572,10 @@ module Klenod
                 end
             end
 
+            def offset_for(line_offsets, line, column)
+              line_offsets.fetch(line - 1) + column
+            end
+
             def source_expressions(expressions)
               case expressions.length
               when 0
@@ -579,11 +593,14 @@ module Klenod
               factory = expression_fragment(factory)
               tag = expression_fragment(tag)
               children = children.map { |child| expression_fragment(child) }
+              props = props.dup
+              attribute_splats = props.delete(:__klenod_attribute_splats__) || []
 
               source_parts = [
                 to_source(tag),
                 *children.map { |child| argument_source(child) },
-                *keyword_props_source(props, mark: mark)
+                *keyword_props_source(props, mark: mark),
+                *attribute_splats.map { |value| "**#{argument_source(value)}" }
               ].compact
               Fragment.new("#{to_source(factory)}[#{source_parts.join(", ")}]", nil)
             end
@@ -712,6 +729,7 @@ module Klenod
                 case source
                 when /\Aif\s+(.+)\z/ then $1
                 when /\Aelsif\s+(.+)\z/ then $1
+                when /\Aunless\s+(.+)\z/ then "!(#{$1})"
                 else return nil
                 end
               predicate = parse_expression(predicate_source, context: :branch_predicate)

data/lib/klenod/build/plugins/haml_plugin/transformer.rb

@@ -12,6 +12,8 @@ module Klenod
     module Plugins
       module HamlPlugin
         class Transformer
+          ATTRIBUTE_SPLATS_KEY = :__klenod_attribute_splats__
+
           VALID_CONST_PATH = /\A[A-Z]\w*(?:::[A-Z]\w*)*\z/
 
           ConstPath = Data.define(:value) do
@@ -43,11 +45,13 @@ module Klenod
             import_rewriter: nil,
             markdown_components_source: "{}",
             variables: nil,
+            event_handler: nil,
             haml_helper_source: nil,
             cache_static_subtrees: false
           )
             component_base_class = ConstPath.parse(component_base_class, name: "component_base_class")
             factory = ConstPath.parse(factory, name: "factory")
+            event_handler = ConstPath.parse(event_handler, name: "event_handler") if event_handler
             validate_component_children(component_children)
             builder = RubyBuilder.new(profiler: profiler, variables: variables)
             haml_helper_source ||= builder.constant_assignment("HamlHelper", "Object") if styleable || cache_static_subtrees
@@ -56,11 +60,13 @@ module Klenod
             previous_static_constants = @static_constants
             previous_cache_static_subtrees = @cache_static_subtrees
             previous_component_children = @component_children
+            previous_event_handler = @event_handler
             @profiler = profiler
             @module_id = module_id
             @static_constants = []
             @cache_static_subtrees = cache_static_subtrees
             @component_children = component_children
+            @event_handler = event_handler
             template =
               if profiler
                 profiler.measure(:haml_compile_template, module_id: module_id.to_s) do
@@ -136,6 +142,7 @@ module Klenod
             @static_constants = previous_static_constants
             @cache_static_subtrees = previous_cache_static_subtrees
             @component_children = previous_component_children
+            @event_handler = previous_event_handler
           end
 
           private
@@ -369,6 +376,13 @@ module Klenod
               )
             end
 
+            unless source.start_with?("if ", "unless ", "case ")
+              return builder.silent_script_with_children(
+                source,
+                compile_nodes(node.children, factory: factory, styleable: styleable, builder: builder, markdown_compiler: markdown_compiler)
+              )
+            end
+
             compile_silent_branches(
               split_silent_script_branches(node, builder: builder),
               factory: factory,
@@ -507,7 +521,7 @@ module Klenod
           end
 
           def compile_ruby_filter(node, builder:, source: "", import_rewriter: nil)
-            text = node.value.fetch(:text)
+            text = node.value.fetch(:text) || ""
             if import_rewriter && text.include?("import")
               source_column_offset = source.lines.fetch(node.line, "").match(/\A\s*/).to_s.length
               text = import_rewriter.call(text, source_line_offset: node.line, source_column_offset: source_column_offset)
@@ -552,8 +566,9 @@ module Klenod
           def compile_filter_node(node, builder:, markdown_compiler:)
             return builder.render_ruby_filter(compile_ruby_filter(node, builder: builder)) if ruby_filter?(node)
             return builder.expression(markdown_compiler.compile(node.value.fetch(:text), interpolate: true)) if markdown_filter?(node)
+            return builder.literal(node.value.fetch(:text).to_s) if plain_filter?(node)
 
-            raise ArgumentError, "Only :ruby and :markdown Haml filters are supported"
+            raise ArgumentError, "Only :ruby, :markdown, and :plain Haml filters are supported"
           end
 
           def ruby_filter?(node)
@@ -564,6 +579,10 @@ module Klenod
             node.type == :filter && node.value.fetch(:name) == "markdown"
           end
 
+          def plain_filter?(node)
+            node.type == :filter && node.value.fetch(:name) == "plain"
+          end
+
           def css_filter?(node)
             node.type == :filter && node.value.fetch(:name) == "css"
           end
@@ -588,7 +607,7 @@ module Klenod
 
             measure_compile(:haml_compile_dynamic_attributes) do
               source = builder.line_rewritten_source(source, node.line)
-              simple = simple_dynamic_attributes(source, builder: builder)
+              simple = simple_dynamic_attributes(source, builder: builder) unless @event_handler
               if simple
                 simple.each { |key, value| props[key] = value }
                 return simple
@@ -601,8 +620,15 @@ module Klenod
 
               dynamic = {}
               hash.node.assocs.each do |assoc|
+                if assoc.is_a?(SyntaxTree::AssocSplat)
+                  splat_source = hash.source[assoc.value.location.start_char...assoc.value.location.end_char]
+                  props[ATTRIBUTE_SPLATS_KEY] ||= []
+                  props[ATTRIBUTE_SPLATS_KEY] << builder.expression(splat_source)
+                  next
+                end
+
                 key = attribute_key(assoc.key, builder: builder)
-                value = attribute_value(assoc, source, builder: builder)
+                value = event_handler_value(key, assoc.value, builder: builder) || attribute_value(assoc, builder: builder)
                 dynamic[key] = value
                 props[key] = value
               end
@@ -722,16 +748,9 @@ module Klenod
             end
           end
 
-          def node_source(source, node)
-            location = node.location
-            return source unless location
-
-            source[location.start_char...location.end_char]
-          end
-
-          def attribute_value(assoc, source, builder:)
+          def attribute_value(assoc, builder:)
             value = assoc.value
-            return builder.node_fragment(node_source(source, value), value) if value
+            return builder.fragment(value) if value
 
             key = omitted_attribute_value_name(assoc.key)
             return builder.expression(key) if key
@@ -739,6 +758,16 @@ module Klenod
             builder.ruby_parse_error(source, line_no: assoc.key.location&.start_line, context: "Could not parse Haml dynamic attributes")
           end
 
+          def event_handler_value(key, value, builder:)
+            return unless @event_handler && key.to_s.match?(/\Aon[a-z]/)
+            return unless value
+
+            method_name = builder.fragment(value).source
+            return unless method_name.match?(/\A[a-zA-Z_]\w*[!?=]?\z/)
+
+            builder.expression("#{@event_handler}.callback(self, :#{method_name})")
+          end
+
           def omitted_attribute_value_name(node)
             case node
             when SyntaxTree::Label

data/lib/klenod/build/plugins/haml_plugin.rb

@@ -66,6 +66,7 @@ module Klenod
             factory: DEFAULT_FACTORY,
             component_children: DEFAULT_COMPONENT_CHILDREN,
             variables: nil,
+            event_handler: nil,
             i18n: nil,
             cache_static_subtrees: false
           )
@@ -73,6 +74,7 @@ module Klenod
             @factory = factory
             @component_children = validate_component_children(component_children)
             @variables = validate_variables(variables)
+            @event_handler = event_handler
             @i18n_class, @i18n_constant = validate_i18n_options(i18n)
             @cache_static_subtrees = cache_static_subtrees
             @transformer = Transformer.new
@@ -208,6 +210,7 @@ module Klenod
                 import_rewriter: import_rewriter,
                 markdown_components_source: markdown_components_dependency ? "__klenod_import__(#{markdown_components_dependency.id.inspect})::Default" : "{}",
                 variables: @variables,
+                event_handler: @event_handler,
                 cache_static_subtrees: @cache_static_subtrees
               )
             import_rewrite =

data/lib/klenod/build/plugins/image_plugin.rb

@@ -3,6 +3,7 @@
 require "image_size"
 require "rmagick"
 require "uri"
+require "base64"
 
 require_relative "../asset"
 require_relative "../dependency"
@@ -27,12 +28,15 @@ module Klenod
 
           ImageDefaultKey = Data.define(:source_path, :source_hash, :format, :quality)
           ImageVariantKey = Data.define(:source_path, :source_hash, :width, :format, :quality)
+          ImagePlaceholderKey = Data.define(:source_path, :source_hash, :width, :format, :quality)
 
-          def initialize(widths: [], formats: nil)
+          def initialize(widths: [], formats: nil, placeholder: nil)
             @widths = widths
             @formats = formats
+            @placeholder = normalize_placeholder(placeholder)
             @default_asset_cache = {}
             @variant_cache = {}
+            @placeholder_cache = {}
           end
 
           def resolve(dependency, _context)
@@ -51,8 +55,9 @@ module Klenod
             image_options = image_options_for(module_id)
             asset = default_image_asset(module_id, source_path, source_hash, dimensions, image_options, context.asset_generation_queue)
             variant_assets = generate_variant_assets(module_id, source_path, source_hash, dimensions, image_options, context.asset_generation_queue)
+            placeholder = image_placeholder(source_path, source_hash)
             [asset, *variant_assets].each { |image_asset| image_asset.url = context.asset_url(image_asset.output_path) }
-            javascript_asset = javascript_image_asset(module_id, asset, variant_assets, context)
+            javascript_asset = javascript_image_asset(module_id, asset, variant_assets, context, placeholder:)
             javascript_asset.url = context.asset_url(javascript_asset.output_path)
             assets = [asset, *variant_assets]
             image_runtime_dependency =
@@ -66,7 +71,7 @@ module Klenod
 
             transform =
               TransformResult.new(
-                image_module_source(module_id, assets, image_runtime_dependency, context),
+                image_module_source(module_id, assets, image_runtime_dependency, context, placeholder:),
                 [image_runtime_dependency],
                 nil,
                 assets,
@@ -184,13 +189,14 @@ module Klenod
           end
 
           ImageOptions = Data.define(:widths, :formats, :explicit_formats, :quality)
+          PlaceholderOptions = Data.define(:width, :format, :quality)
 
           def image_runtime_source
             <<~RUBY
               ImageMetadata =
-                Data.define(:src, :width, :height, :content_type, :aspect_ratio, :variants) do
-                  def initialize(src:, width:, height:, content_type:, aspect_ratio:, variants:)
-                    super(src:, width:, height:, content_type:, aspect_ratio:, variants: variants.dup.freeze)
+                Data.define(:src, :width, :height, :content_type, :aspect_ratio, :variants, :placeholder) do
+                  def initialize(src:, width:, height:, content_type:, aspect_ratio:, variants:, placeholder: nil)
+                    super(src:, width:, height:, content_type:, aspect_ratio:, variants: variants.dup.freeze, placeholder:)
                   end
 
                   alias to_s src
@@ -219,7 +225,7 @@ module Klenod
             "# image asset: #{module_id}\n"
           end
 
-          def image_module_source(module_id, assets, image_runtime_dependency, context)
+          def image_module_source(module_id, assets, image_runtime_dependency, context, placeholder:)
             asset = image_asset(assets)
             variants = image_variant_assets(assets).map { |variant_asset| image_variant_source(variant_asset, context) }
 
@@ -232,6 +238,7 @@ module Klenod
                   height: #{asset.metadata[:height].inspect},
                   content_type: #{asset.content_type.inspect},
                   aspect_ratio: #{AssetJavaScriptMetadata.aspect_ratio(asset.metadata[:width], asset.metadata[:height]).inspect},
+                  placeholder: #{placeholder.inspect},
                   variants: [
                     #{variants.join(",\n    ")}
                   ]
@@ -255,8 +262,36 @@ module Klenod
             RUBY
           end
 
-          def javascript_image_asset(module_id, asset, variant_assets, context)
-            code = javascript_image_module_source(asset, variant_assets, context)
+          def image_placeholder(source_path, source_hash)
+            return nil unless @placeholder
+
+            key = ImagePlaceholderKey.new(source_path.to_s, source_hash, @placeholder.width, @placeholder.format, @placeholder.quality)
+            @placeholder_cache[key] ||= begin
+              bytes = generate_image_bytes(source_path, @placeholder.format, width: @placeholder.width, quality: @placeholder.quality)
+              "data:image/#{@placeholder.format};base64,#{Base64.strict_encode64(bytes)}"
+            end
+          end
+
+          def normalize_placeholder(value)
+            return nil if value.nil? || value == false
+
+            value = {width: value} if value.is_a?(Integer)
+            unless value.is_a?(Hash)
+              raise ArgumentError, "placeholder must be nil, false, an Integer width, or a Hash"
+            end
+
+            width = Integer(value.fetch(:width, 16))
+            raise ArgumentError, "placeholder width must be positive" unless width.positive?
+
+            format = value.fetch(:format, "webp").to_s.downcase
+            quality = Integer(value.fetch(:quality, 80))
+            raise ArgumentError, "placeholder quality must be between 0 and 100" unless (0..100).cover?(quality)
+
+            PlaceholderOptions.new(width, format, quality)
+          end
+
+          def javascript_image_asset(module_id, asset, variant_assets, context, placeholder:)
+            code = javascript_image_module_source(asset, variant_assets, context, placeholder:)
             hash = Hashing.short(code)
             Asset.new(
               module_id.path,
@@ -269,14 +304,15 @@ module Klenod
             )
           end
 
-          def javascript_image_module_source(asset, variant_assets, context)
+          def javascript_image_module_source(asset, variant_assets, context, placeholder:)
             variants = variant_assets.map { "new ImageVariant(#{AssetJavaScriptMetadata.object_literal(javascript_image_variant(it, context))})" }
             properties = {
               src: asset.url,
               width: asset.metadata[:width],
               height: asset.metadata[:height],
               contentType: asset.content_type,
-              aspectRatio: AssetJavaScriptMetadata.aspect_ratio(asset.metadata[:width], asset.metadata[:height])
+              aspectRatio: AssetJavaScriptMetadata.aspect_ratio(asset.metadata[:width], asset.metadata[:height]),
+              placeholder: placeholder
             }.map { |key, value| AssetJavaScriptMetadata.property(key, value) }
             properties << %(variants:[#{variants.join(",")}])
 

data/lib/klenod/build/plugins/static_asset_plugin.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+require_relative "../asset"
+require_relative "../hashing"
+require_relative "../plugin"
+require_relative "../transform_result"
+
+module Klenod
+  module Build
+    module Plugins
+      module StaticAssetPlugin
+        def self.new(...)
+          Plugin.new(...)
+        end
+
+        class Plugin < Klenod::Build::Plugin
+          CONTENT_TYPES = {
+            ".otf" => "font/otf",
+            ".ttf" => "font/ttf",
+            ".woff" => "font/woff",
+            ".woff2" => "font/woff2"
+          }.freeze
+
+          def transform(module_id, code, context)
+            return super unless CONTENT_TYPES.key?(module_id.extname)
+
+            hash = Hashing.short(code)
+            output_path = "/#{asset_name(module_id)}.#{hash}#{module_id.extname}"
+            asset = Asset.new(module_id.path, hash, output_path, nil, code, CONTENT_TYPES.fetch(module_id.extname), {type: :static_asset})
+            asset.url = context.asset_url(output_path)
+
+            TransformResult.new("Default = #{asset.url.inspect}\n", [], nil, [asset], [], {})
+          end
+
+          def import_value(_resolved_dependency, record, context)
+            return nil unless CONTENT_TYPES.key?(record.id.extname)
+
+            context.mods.fetch(record.id).const_get(:Exports)::Default
+          end
+
+          def runtime_import_value(_resolved_dependency, record, _context)
+            return nil unless CONTENT_TYPES.key?(record.id.extname)
+
+            Runtime::DefaultImport.new(:Default)
+          end
+
+          private
+
+          def asset_name(module_id)
+            File.basename(module_id.path, module_id.extname).gsub(/[^A-Za-z0-9]+/, "_")
+          end
+        end
+      end
+    end
+  end
+end

data/lib/klenod/build/plugins.rb

@@ -11,6 +11,7 @@ module Klenod
       autoload :GoogleFontsPlugin, File.expand_path("plugins/google_fonts_plugin", __dir__)
       autoload :SvgPlugin, File.expand_path("plugins/svg_plugin", __dir__)
       autoload :ImagePlugin, File.expand_path("plugins/image_plugin", __dir__)
+      autoload :StaticAssetPlugin, File.expand_path("plugins/static_asset_plugin", __dir__)
       autoload :DataPlugin, File.expand_path("plugins/data_plugin", __dir__)
       autoload :JsonPlugin, File.expand_path("plugins/data_plugin", __dir__)
       autoload :YamlPlugin, File.expand_path("plugins/data_plugin", __dir__)

data/lib/klenod/build/version.rb

@@ -2,6 +2,6 @@
 
 module Klenod
   module Build
-    VERSION = "0.0.7"
+    VERSION = "0.0.8"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: klenod-build
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors:
 - Andrés Alin
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.7
+        version: 0.0.8
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.7
+        version: 0.0.8
 - !ruby/object:Gem::Dependency
   name: async
   requirement: !ruby/object:Gem::Requirement
@@ -57,6 +57,20 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: 0.104.0
+- !ruby/object:Gem::Dependency
+  name: base64
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.3'
 - !ruby/object:Gem::Dependency
   name: brotli
   requirement: !ruby/object:Gem::Requirement
@@ -277,6 +291,7 @@ files:
 - lib/klenod/build/plugins/markdown_plugin.rb
 - lib/klenod/build/plugins/router_plugin.rb
 - lib/klenod/build/plugins/ruby_plugin.rb
+- lib/klenod/build/plugins/static_asset_plugin.rb
 - lib/klenod/build/plugins/svg_plugin.rb
 - lib/klenod/build/profiler.rb
 - lib/klenod/build/resolution_error_formatter.rb