activemail 1.2.3 → 2.1.0

data/lib/activemail/tokens.rb

@@ -2,6 +2,7 @@
 # frozen_string_literal: true
 
 require 'sorbet-runtime'
+require_relative 'tokens/defaults'
 require_relative 'tokens/button_style'
 require_relative 'tokens/scss_serializer'
 
@@ -9,61 +10,6 @@ module ActiveMail
   class Tokens
     extend T::Sig
 
-    TokenMap = T.type_alias { T::Hash[Symbol, String] }
-
-    DEFAULT_COLORS = T.let(
-      {
-        primary: '#2a9d8f',
-        secondary: '#264653',
-        text: '#1a1a1a',
-        background: '#ffffff',
-        muted: '#6b7280',
-        border: '#e5e7eb',
-        button_text: '#ffffff'
-      }.freeze,
-      TokenMap
-    )
-
-    SYSTEM_FONT_STACK = '-apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Helvetica, Arial, sans-serif'
-
-    DEFAULT_FONTS = T.let(
-      {
-        body: SYSTEM_FONT_STACK,
-        heading: SYSTEM_FONT_STACK
-      }.freeze,
-      TokenMap
-    )
-
-    DEFAULT_SPACINGS = T.let(
-      {
-        xs: '4px',
-        sm: '8px',
-        md: '16px',
-        lg: '24px',
-        xl: '40px'
-      }.freeze,
-      TokenMap
-    )
-
-    DEFAULT_RADII = T.let(
-      {
-        button: '4px',
-        box: '4px'
-      }.freeze,
-      TokenMap
-    )
-
-    # Open/Closed: adding a group here is the only change needed to wire it everywhere.
-    GROUPS = T.let(
-      {
-        color: DEFAULT_COLORS,
-        font: DEFAULT_FONTS,
-        spacing: DEFAULT_SPACINGS,
-        radius: DEFAULT_RADII
-      }.freeze,
-      T::Hash[Symbol, TokenMap]
-    )
-
     sig { void }
     def initialize
       @stores = T.let(GROUPS.transform_values(&:dup), T::Hash[Symbol, TokenMap])
@@ -123,6 +69,13 @@ module ActiveMail
       end
     end
 
+    # Every token with derived colors filled in; nil marks a derived token left unset.
+    sig { returns(T::Hash[Symbol, T::Hash[Symbol, T.nilable(String)]]) }
+    def resolved
+      colors = DERIVED_COLORS.keys.to_h { |name| [name, color(name)] }
+      @stores.merge(color: store_for(:color).merge(colors))
+    end
+
     sig { params(variant: T.any(String, Symbol)).returns(ButtonStyle) }
     def button_style(variant)
       ButtonStyle.from(self, variant)
@@ -130,7 +83,7 @@ module ActiveMail
 
     sig { returns(String) }
     def to_scss
-      ScssSerializer.call(@stores)
+      ScssSerializer.call(resolved)
     end
 
     private
@@ -142,14 +95,22 @@ module ActiveMail
 
     sig { params(group: Symbol, name: T.any(String, Symbol)).returns(String) }
     def fetch!(group, name)
-      store_for(group).fetch(name.to_sym) { raise KeyError, "unknown #{group} token #{name.inspect}" }
+      access(group, name, nil) || raise(KeyError, "unknown or unset #{group} token #{name.inspect}")
+    end
+
+    sig { params(group: Symbol, key: Symbol).returns(T.nilable(String)) }
+    def derived(group, key)
+      return unless group == :color
+
+      source = DERIVED_COLORS[key]
+      source && store_for(:color)[source]
     end
 
     sig { params(group: Symbol, name: T.any(String, Symbol), value: T.nilable(String)).returns(T.nilable(String)) }
     def access(group, name, value)
       store = store_for(group)
       key = name.to_sym
-      return store[key] if value.nil?
+      return store.fetch(key) { derived(group, key) } if value.nil?
       # Emitted verbatim into SCSS — reject blanks that would yield broken CSS.
       raise ArgumentError, "token #{key} value must not be blank" if value.strip.empty?
 

data/lib/activemail/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module ActiveMail
-  VERSION = '1.2.3'
+  VERSION = '2.1.0'
 end

data/lib/activemail.rb

@@ -2,15 +2,17 @@
 # frozen_string_literal: true
 
 require 'nokogiri'
+require 'securerandom'
 require 'sorbet-runtime'
 
 require_relative 'activemail/version'
+require_relative 'activemail/errors'
 require_relative 'activemail/components/base'
+require_relative 'activemail/components/button_renderer'
 require_relative 'activemail/components/button'
 require_relative 'activemail/components/row'
 require_relative 'activemail/components/columns'
 require_relative 'activemail/components/container'
-require_relative 'activemail/components/inky'
 require_relative 'activemail/components/block_grid'
 require_relative 'activemail/components/menu'
 require_relative 'activemail/components/menu_item'
@@ -22,19 +24,32 @@ require_relative 'activemail/components/wrapper'
 require_relative 'activemail/components/cta'
 require_relative 'activemail/components/info_box'
 require_relative 'activemail/configuration'
-require_relative 'activemail/inliner/interceptor'
+require_relative 'activemail/pipeline'
 require_relative 'activemail/parse_error_reporter'
 
 module ActiveMail
-  class ParseError < StandardError; end
-
   class Core
     extend T::Sig
 
     # Nokogiri cannot parse a bare <th> outside a <tr>; components that emit
     # <th> use this placeholder, swapped back at the end.
     INTERIM_TH_TAG = 'activemail-interim-th'
-    INTERIM_TH_TAG_REGEX = T.let(%r{(?<=<|</)#{Regexp.escape(INTERIM_TH_TAG)}}, Regexp)
+
+    # Comment placeholder handed to components as `inner`; the transformed children
+    # are moved there without being re-serialized.
+    SLOT_TEXT = 'activemail-slot'
+    SLOT = T.let("<!--#{SLOT_TEXT}-->".freeze, String)
+
+    # The component markup repeats the id of the node it replaces, which libxml2 still has registered.
+    LIBXML_ID_REDEFINED_CODE = 513
+
+    RAW_REGEX = T.let(%r{< *raw *>([\s\S]*?)</ *raw *>}i, Regexp)
+
+    # One pass over the serialized output; <style>/<script> bodies are matched first so they are left alone.
+    POSTPROCESS_REGEX = T.let(
+      %r{(?<verbatim><(?<tag>style|script)\b[\s\S]*?</\k<tag>\s*>)|(?<=<|</)#{Regexp.escape(INTERIM_TH_TAG)}(?![\w-])|\u00A0}i,
+      Regexp
+    )
 
     DEFAULT_COMPONENTS = T.let(
       {
@@ -44,7 +59,6 @@ module ActiveMail
         'row' => ActiveMail::Components::Row,
         'columns' => ActiveMail::Components::Columns,
         'container' => ActiveMail::Components::Container,
-        'inky' => ActiveMail::Components::Inky,
         'block-grid' => ActiveMail::Components::BlockGrid,
         'menu' => ActiveMail::Components::Menu,
         'item' => ActiveMail::Components::MenuItem,
@@ -60,85 +74,99 @@ module ActiveMail
     sig { returns(Integer) }
     attr_reader :column_count, :container_width
 
-    sig { returns(T::Hash[String, ActiveMail::Components::Base]) }
-    attr_reader :component_instances
+    sig { returns(ActiveMail::Configuration) }
+    attr_reader :config
 
-    sig { params(options: T::Hash[Symbol, T.untyped]).void }
-    def initialize(options = {})
-      config = ::ActiveMail.configuration
-      @component_instances = T.let(build_components(config, options[:components]), T::Hash[String, ActiveMail::Components::Base])
-      @column_count = T.let(ActiveMail.assert_positive_dimension!(:column_count, options[:column_count] || config.column_count), Integer)
-      @container_width = T.let(ActiveMail.assert_positive_dimension!(:container_width, options[:container_width] || config.container_width), Integer)
+    sig do
+      params(
+        config: ActiveMail::Configuration,
+        components: T.untyped,
+        column_count: T.untyped,
+        container_width: T.untyped
+      ).void
+    end
+    def initialize(config: ::ActiveMail.configuration, components: nil, column_count: nil, container_width: nil)
+      @config = T.let(config, ActiveMail::Configuration)
+      @component_instances = T.let(build_components(config, components), T::Hash[String, ActiveMail::Components::Base])
+      @column_count = T.let(ActiveMail.positive_integer!(:column_count, column_count || config.column_count), Integer)
+      @container_width = T.let(ActiveMail.positive_integer!(:container_width, container_width || config.container_width), Integer)
+      @error_reporter = T.let(ParseErrorReporter.new(@component_instances.keys + [INTERIM_TH_TAG], config), ParseErrorReporter)
     end
 
     # Object, not String: ActionView::OutputBuffer is no longer a String since Rails 7.1.
+    # Holds no per-call state: the instance is shared across threads.
     sig { params(html_string: Object).returns(String) }
     def transpile(html_string)
-      raws, str = extract_raws(normalize_input(html_string))
+      marker = "activemail-raw-#{SecureRandom.hex(8)}-"
+      raws = {}
+      str = extract_raws(normalize_input(html_string), marker, raws)
       parse_cmd = ::ActiveMail.full_document?(str) ? :parse : :fragment
       html = Nokogiri::HTML.public_send(parse_cmd, str)
-      ParseErrorReporter.new(component_instances.keys).call(html.errors)
+      @error_reporter.call(html.errors)
       transform_doc(html)
-      string = html.to_html
-      string = string.gsub(INTERIM_TH_TAG_REGEX, 'th')
-      # Needle is a literal U+00A0 (Nokogiri decodes the nbsp entity to one); re-encode
-      # it to the entity for email clients that mishandle raw NBSP bytes.
-      string = string.gsub(' ', '&nbsp;')
-      re_inject_raws(string, raws)
+      re_inject_raws(postprocess(html.to_html), marker, raws)
     end
 
-    sig { params(elem: Nokogiri::XML::Node).returns(Nokogiri::XML::Node) }
-    def transform_doc(elem)
-      if elem.respond_to?(:children)
-        elem.children.each { |child| transform_doc(child) }
-        markup = component_factory(elem)
-        elem.replace(markup) if markup
-      end
-      elem
-    end
+    private
 
-    sig { params(node: Nokogiri::XML::Node).returns(T.nilable(String)) }
-    def component_factory(node)
+    sig { returns(T::Hash[String, ActiveMail::Components::Base]) }
+    attr_reader :component_instances
+
+    sig { params(node: Nokogiri::XML::Node).void }
+    def transform_doc(node)
+      node.children.each { |child| transform_doc(child) }
       component = component_instances[node.name]
-      return unless component
+      replace_with_component(node, component, component.transform(node, SLOT)) if component
+    end
+
+    # Only the component's own markup is parsed; the node's children are moved into
+    # its slot, never re-serialized.
+    sig { params(node: Nokogiri::XML::Node, component: ActiveMail::Components::Base, markup: String).void }
+    def replace_with_component(node, component, markup)
+      fragment = Nokogiri::HTML::DocumentFragment.new(node.document, markup, node)
+      @error_reporter.call(fragment.errors.reject { |error| error.code == LIBXML_ID_REDEFINED_CODE })
+      slots = fragment.xpath('.//comment()').select { |comment| comment.content == SLOT_TEXT }
+      ensure_slot_usage!(node, component, slots.size)
+      slots.first&.replace(node.children)
+      node.replace(fragment)
+    end
+
+    sig { params(node: Nokogiri::XML::Node, component: ActiveMail::Components::Base, count: Integer).void }
+    def ensure_slot_usage!(node, component, count)
+      return if count == 1 || (count.zero? && node.children.all? { |child| child.text? && child.content.strip.empty? })
 
-      # Nokogiri::NodeSet has no #join; map to String first.
-      inner = node.children.map(&:to_s).join # rubocop:disable Style/MapJoin
-      component.transform(node, inner)
+      raise ActiveMail::Error,
+            "<#{node.name}> (#{component.class}) must output `inner` exactly once, found #{count}: its children would be lost or duplicated"
     end
 
-    sig { params(string: String).returns([T::Array[String], String]) }
-    def extract_raws(string)
-      raws = []
-      i = 0
+    sig { params(string: String, marker: String, raws: T::Hash[String, String]).returns(String) }
+    def extract_raws(string, marker, raws)
       # Only the tags + content, across lines: surrounding whitespace is left in
       # place (true pass-through; eating adjacent newlines corrupts <pre>/inline text).
-      regex = %r{< *raw *>([\s\S]*?)</ *raw *>}i
-      str = string
-      while (raw = str.match(regex))
-        raws[i] = T.must(raw[1])
-        str = str.sub(regex, "###RAW#{i}###")
-        i += 1
+      string.gsub(RAW_REGEX) do
+        placeholder = "#{marker}#{raws.size}~"
+        raws[placeholder] = T.must(Regexp.last_match(1))
+        placeholder
       end
-      [raws, str]
     end
 
-    sig { params(string: String, raws: T::Array[String]).returns(String) }
-    def re_inject_raws(string, raws)
-      str = string
-      raws.each_with_index do |val, i|
-        # Block form: the 2-arg String#sub would expand \0/\1/\& in val.
-        str = str.sub("###RAW#{i}###") { val }
+    # U+00A0 is what Nokogiri decodes the nbsp entity to; re-encoded for email clients that mishandle raw NBSP bytes.
+    sig { params(string: String).returns(String) }
+    def postprocess(string)
+      string.gsub(POSTPROCESS_REGEX) do |match|
+        next match if Regexp.last_match(:verbatim)
+
+        match == "\u00A0" ? '&nbsp;' : 'th'
       end
+    end
+
+    sig { params(string: String, marker: String, raws: T::Hash[String, String]).returns(String) }
+    def re_inject_raws(string, marker, raws)
+      str = raws.empty? ? string : string.gsub(/#{Regexp.escape(marker)}\d+~/) { |token| raws.fetch(token) }
       str = str.html_safe if str.respond_to?(:html_safe)
       str
     end
 
-    private
-
-    # Internal transpilation details, not public surface (column_count/container_width stay public).
-    private :component_instances, :transform_doc, :component_factory, :extract_raws, :re_inject_raws
-
     sig { params(config: ActiveMail::Configuration, overrides: T.untyped).returns(T::Hash[String, ActiveMail::Components::Base]) }
     def build_components(config, overrides)
       # Lookup is by node name (String); a Symbol key would never match.
@@ -151,8 +179,8 @@ module ActiveMail
     def normalize_input(html_string)
       html_string = html_string.to_s
       html_string = html_string.dup.force_encoding(Encoding::UTF_8) if html_string.encoding == Encoding::BINARY
-      html_string = ::ActiveMail.scrub_invalid_bytes(html_string) unless html_string.valid_encoding?
-      html_string.gsub(/doctype/i, 'DOCTYPE')
+      html_string = ::ActiveMail.scrub_invalid_bytes(html_string, config.on_parse_error) unless html_string.valid_encoding?
+      html_string
     end
   end
 end

data/lib/generators/activemail/templates/component.rb.tt

@@ -4,10 +4,9 @@
 module Components
   # Register in config/initializers/activemail.rb:
   #   config.register_component "<%= tag_name %>", Components::<%= class_name %>
-  class <%= class_name %> < ActiveMail::Components::Base
+  class <%= class_name %> < ActiveMail::Components::TableComponent
     def transform(node, inner)
-      classes = combine_classes(node, "<%= tag_name %>")
-      %(<table class="#{classes}" #{TABLE_RESET}><tbody><tr><td>#{inner}</td></tr></tbody></table>)
+      %(<table#{root_attributes(node, classes: "<%= tag_name %>")} #{TABLE_RESET}><tbody><tr><td>#{inner}</td></tr></tbody></table>)
     end
   end
 end

data/lib/generators/activemail/templates/initializer.rb

@@ -14,9 +14,6 @@ ActiveMail.configure do |config|
   # ActiveMail::Inliner::Base subclass/instance.
   # config.inliner = :premailer
 
-  # Set false if another inliner (e.g. premailer-rails) already runs on mailers.
-  # config.register_inline_interceptor = true
-
   # How malformed markup is handled: :warn (default), :ignore, or :raise.
   # config.on_parse_error = :warn
 

data/lib/generators/activemail/templates/mailer_layout.html.inky-erb

@@ -1,29 +1,13 @@
 <!DOCTYPE html>
 <html lang="<%%= I18n.locale %>" xmlns="http://www.w3.org/1999/xhtml" xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office">
-  <head>
-    <meta charset="utf-8">
-    <meta name="viewport" content="width=device-width, initial-scale=1">
-    <meta name="x-apple-disable-message-reformatting">
-    <meta name="color-scheme" content="light dark">
-    <meta name="supported-color-schemes" content="light dark">
-    <!--[if mso]>
-    <noscript>
-      <xml>
-        <o:OfficeDocumentSettings>
-          <o:PixelsPerInch>96</o:PixelsPerInch>
-        </o:OfficeDocumentSettings>
-      </xml>
-    </noscript>
-    <![endif]-->
-    <%%= activemail_inline_styles %>
-    <%%# Fallback for when activemail_inline_styles cannot read the compiled asset. %>
-    <%%= stylesheet_link_tag "activemail/activemail", media: "all" %>
-  </head>
+  <%%= render "layouts/activemail/document_head" %>
 
   <body style="margin:0;padding:0;word-spacing:normal;">
     <div role="article" aria-roledescription="email" lang="<%%= I18n.locale %>" style="width:100%;">
       <container>
+        <%%= render "layouts/activemail/head" %>
         <%%= yield %>
+        <%%= render "layouts/activemail/footer" %>
       </container>
     </div>
   </body>

data/lib/generators/activemail/templates/mailer_layout.html.inky-haml

@@ -1,17 +1,9 @@
 !!!
 %html{lang: I18n.locale, xmlns: "http://www.w3.org/1999/xhtml", "xmlns:v" => "urn:schemas-microsoft-com:vml", "xmlns:o" => "urn:schemas-microsoft-com:office:office"}
-  %head
-    %meta{charset: "utf-8"}
-    %meta{name: "viewport", content: "width=device-width, initial-scale=1"}
-    %meta{name: "x-apple-disable-message-reformatting"}
-    %meta{name: "color-scheme", content: "light dark"}
-    %meta{name: "supported-color-schemes", content: "light dark"}
-    :plain
-      <!--[if mso]><noscript><xml><o:OfficeDocumentSettings><o:PixelsPerInch>96</o:PixelsPerInch></o:OfficeDocumentSettings></xml></noscript><![endif]-->
-    = activemail_inline_styles
-    -# Fallback for when activemail_inline_styles cannot read the compiled asset.
-    = stylesheet_link_tag "activemail/activemail", media: "all"
+  = render "layouts/activemail/document_head"
   %body{style: "margin:0;padding:0;word-spacing:normal;"}
     %div{role: "article", "aria-roledescription" => "email", lang: I18n.locale, style: "width:100%;"}
       %container
+        = render "layouts/activemail/head"
         = yield
+        = render "layouts/activemail/footer"

data/lib/generators/activemail/templates/mailer_layout.html.inky-slim

@@ -1,16 +1,9 @@
 doctype html
 html lang=I18n.locale xmlns="http://www.w3.org/1999/xhtml" xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office"
-  head
-    meta charset="utf-8"
-    meta name="viewport" content="width=device-width, initial-scale=1"
-    meta name="x-apple-disable-message-reformatting"
-    meta name="color-scheme" content="light dark"
-    meta name="supported-color-schemes" content="light dark"
-    | <!--[if mso]><noscript><xml><o:OfficeDocumentSettings><o:PixelsPerInch>96</o:PixelsPerInch></o:OfficeDocumentSettings></xml></noscript><![endif]-->
-    = activemail_inline_styles
-    / Fallback for when activemail_inline_styles cannot read the compiled asset.
-    = stylesheet_link_tag "activemail/activemail", media: "all"
+  = render "layouts/activemail/document_head"
   body style="margin:0;padding:0;word-spacing:normal;"
     div role="article" aria-roledescription="email" lang=I18n.locale style="width:100%;"
       container
+        = render "layouts/activemail/head"
         = yield
+        = render "layouts/activemail/footer"

data/lib/tasks/activemail.rake

@@ -20,15 +20,14 @@ namespace :activemail do
     task render_all: :environment do
       require 'activemail/quality/render_all'
 
-      config = ActiveMail::Quality.config
-      output_root = defined?(Rails) && Rails.respond_to?(:root) ? Rails.root.join(config.output_dir) : Pathname(config.output_dir)
-      result = ActiveMail::Quality::RenderAll.new(output_root: output_root, config: config).call
-      puts "Rendered #{result.rendered} email(s) into #{output_root}"
+      root = defined?(Rails) && Rails.respond_to?(:root) && Rails.root ? Rails.root : Pathname.pwd
+      result = ActiveMail::Quality::RenderAll.new(root: root, config: ActiveMail::Quality.config).call
+      puts "Rendered #{result.rendered} email(s) into #{result.output_root}"
       # A green run on zero previews would silently verify nothing — make it visible.
       warn '[activemail] WARNING: no mailer previews were discovered — nothing was checked.' if result.discovered.zero?
       result.render_failures.each { |key, error| puts "  render failed: #{key}: #{error}" }
       result.guard_failures.each do |key, violations|
-        puts "  guard failed: #{key}", violations.map { |v| "    - [#{v.rule}] #{v.message}" }.join("\n")
+        puts "  guard failed: #{key}", violations.map { |v| "    - #{v}" }.join("\n")
       end
       abort "\n#{result.broken_required.size} required preview(s) failed: #{result.broken_required.join(', ')}" if result.broken_required.any?
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activemail
 version: !ruby/object:Gem::Version
-  version: 1.2.3
+  version: 2.1.0
 platform: ruby
 authors:
 - Advitam
@@ -81,6 +81,7 @@ files:
 - CHANGELOG.md
 - LICENSE.txt
 - README.md
+- UPGRADING.md
 - app/assets/stylesheets/activemail/_activemail_tokens.scss.erb
 - app/assets/stylesheets/activemail/_components.scss
 - app/assets/stylesheets/activemail/_dark.scss
@@ -89,13 +90,20 @@ files:
 - app/assets/stylesheets/activemail/_utilities.scss
 - app/assets/stylesheets/activemail/activemail.scss
 - app/helpers/active_mail/styles_helper.rb
+- app/views/layouts/activemail/_document_head.html.erb
 - app/views/layouts/activemail/_footer.html.inky-erb
 - app/views/layouts/activemail/_head.html.inky-erb
 - app/views/layouts/activemail/mailer.html.inky-erb
 - lib/activemail.rb
+- lib/activemail/asset_pipeline.rb
+- lib/activemail/asset_pipeline/base.rb
+- lib/activemail/asset_pipeline/null.rb
+- lib/activemail/asset_pipeline/propshaft.rb
+- lib/activemail/asset_pipeline/sprockets.rb
 - lib/activemail/components/base.rb
 - lib/activemail/components/block_grid.rb
 - lib/activemail/components/button.rb
+- lib/activemail/components/button_renderer.rb
 - lib/activemail/components/callout.rb
 - lib/activemail/components/center.rb
 - lib/activemail/components/columns.rb
@@ -103,32 +111,42 @@ files:
 - lib/activemail/components/cta.rb
 - lib/activemail/components/h_line.rb
 - lib/activemail/components/info_box.rb
-- lib/activemail/components/inky.rb
 - lib/activemail/components/menu.rb
 - lib/activemail/components/menu_item.rb
 - lib/activemail/components/row.rb
 - lib/activemail/components/spacer.rb
+- lib/activemail/components/table_component.rb
 - lib/activemail/components/wrapper.rb
 - lib/activemail/configuration.rb
+- lib/activemail/errors.rb
+- lib/activemail/inliner.rb
 - lib/activemail/inliner/base.rb
-- lib/activemail/inliner/interceptor.rb
 - lib/activemail/inliner/null.rb
 - lib/activemail/inliner/premailer.rb
+- lib/activemail/inliner/premailer/document.rb
 - lib/activemail/inliner/roadie.rb
 - lib/activemail/libxml.rb
 - lib/activemail/parse_error_reporter.rb
+- lib/activemail/pipeline.rb
 - lib/activemail/quality.rb
+- lib/activemail/quality/builtin_message_rules.rb
+- lib/activemail/quality/builtin_rules.rb
 - lib/activemail/quality/configuration.rb
 - lib/activemail/quality/guard.rb
 - lib/activemail/quality/minitest.rb
+- lib/activemail/quality/output_dir.rb
+- lib/activemail/quality/output_filter.rb
 - lib/activemail/quality/preview_renderer.rb
 - lib/activemail/quality/render_all.rb
 - lib/activemail/quality/rspec.rb
-- lib/activemail/rails/compiled_stylesheet.rb
+- lib/activemail/quality/rule.rb
+- lib/activemail/quality/rule_exemptions.rb
 - lib/activemail/rails/engine.rb
+- lib/activemail/rails/rendering.rb
 - lib/activemail/rails/template_handler.rb
 - lib/activemail/tokens.rb
 - lib/activemail/tokens/button_style.rb
+- lib/activemail/tokens/defaults.rb
 - lib/activemail/tokens/scss_serializer.rb
 - lib/activemail/version.rb
 - lib/generators/activemail/component_generator.rb

data/lib/activemail/components/inky.rb

@@ -1,18 +0,0 @@
-# typed: strict
-# frozen_string_literal: true
-
-require_relative 'base'
-
-module ActiveMail
-  module Components
-    # Renders a bare <tr>, useful inside hand-written tables.
-    class Inky < Base
-      extend T::Sig
-
-      sig { override.params(_node: Nokogiri::XML::Node, inner: String).returns(String) }
-      def transform(_node, inner)
-        %(<tr>#{inner}</tr>)
-      end
-    end
-  end
-end

data/lib/activemail/inliner/interceptor.rb

@@ -1,51 +0,0 @@
-# typed: strict
-# frozen_string_literal: true
-
-require 'sorbet-runtime'
-
-module ActiveMail
-  module Inliner
-    class Interceptor
-      class << self
-        extend T::Sig
-
-        sig { params(message: T.untyped).void }
-        def delivering_email(message)
-          config = ActiveMail.configuration
-          # Runtime check: an engine initializer runs before the host's, so a
-          # boot-time flag read would always see the default.
-          return unless config.register_inline_interceptor
-
-          inliner = config.resolved_inliner
-          return if inliner.noop?
-
-          html_parts(message).each do |part|
-            part.body = inliner.inline(part.body.to_s)
-          rescue StandardError => e
-            # Surface which inliner failed; do not swallow (a silently non-inlined
-            # mail is worse). Ruby sets e as #cause, preserving the original backtrace.
-            raise ActiveMail::Inliner::Error, "[#{inliner.class}] CSS inlining failed: #{e.message}"
-          end
-        end
-
-        private
-
-        # The html part(s) to inline: a single text/html message, or every text/html
-        # part of a multipart message. Attachments are never inlined.
-        sig { params(message: T.untyped).returns(T::Array[T.untyped]) }
-        def html_parts(message)
-          return message.all_parts.select { |part| html_body_part?(part) } if message.multipart?
-
-          html_body_part?(message) ? [message] : []
-        end
-
-        sig { params(part: T.untyped).returns(T::Boolean) }
-        def html_body_part?(part)
-          return false unless part.respond_to?(:mime_type) && part.mime_type == 'text/html'
-
-          !(part.respond_to?(:attachment?) && part.attachment?)
-        end
-      end
-    end
-  end
-end