proscenium 0.11.0.pre.9-x86_64-darwin → 0.11.0.pre.10-x86_64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 11a4ddf9e18d4a0e84b7e1e0bc4a97b34fa21b0006ba5eee34cff79761e51ead
-  data.tar.gz: 5a7195f12faeab4e91015fcb1d758d13f3b76490f9ae599cd0afbdab87b63e00
+  metadata.gz: 2830fab8a04207b16ce26439994e8cb5ae6882a2f78e90f49faca58f1c3db2b4
+  data.tar.gz: 87549366f8decdda2fdcb5e63461fac3cff46e75fa4f1bc00661cf5d24b4552c
 SHA512:
-  metadata.gz: ce2ee1789cb6035d1bd647c3cc638d233713b5d756533b420a154058e8c0974ecabcb4ee2f0172ee8d7ac4e073e5a697af885f3a1ae273bbef4679f31bf97c16
-  data.tar.gz: 552ebbd157858654fe2c3f0c03e94351c44c21037f9450a1911bd4d0107e6f234f8ebd2ecad8d757401f96400d01eb1d2a69705f4111d0a92ac46b8667bd141d
+  metadata.gz: f134a7364fdb029e1c5f2195934ee57fd2111eefd0d3dcebd84651ce593be69d40ca50a3feca9104a3540947eadf6648160c19bb0976e1d68847fd5e91d43a76
+  data.tar.gz: fef4d6c37eefac4acc926793abb61ffff76936056ef8c9b619107b13034440900dceb5454bf59824886b5ee7988bfc2820d59023940f968d1cfb557699ad1a09

data/lib/proscenium/css_module/transformer.rb

@@ -9,7 +9,8 @@ module Proscenium
     end
 
     def initialize(source_path)
-      @source_path = source_path
+      return unless (@source_path = source_path)
+
       @source_path = Pathname.new(@source_path) unless @source_path.is_a?(Pathname)
       @source_path = @source_path.sub_ext(FILE_EXT) unless @source_path.to_s.end_with?(FILE_EXT)
     end
@@ -37,6 +38,7 @@ module Proscenium
     # @return [Array<String>] the transformed CSS module names.
     def class_names(*names, require_prefix: true)
       names.map do |name|
+        original_name = name.dup
         name = name.to_s if name.is_a?(Symbol)
 
         if name.include?('/')
@@ -50,25 +52,31 @@ module Proscenium
             path, name = name.split('@')
           end
 
-          class_name! name, path: "#{path}#{FILE_EXT}"
+          class_name! name, original_name, path: "#{path}#{FILE_EXT}"
         elsif name.start_with?('@')
-          class_name! name[1..]
+          class_name! name[1..], original_name
         else
-          require_prefix ? name : class_name!(name)
+          require_prefix ? name : class_name!(name, original_name)
         end
       end
     end
 
-    def class_name!(name, path: @source_path)
+    def class_name!(name, original_name, path: @source_path)
+      unless path
+        raise Proscenium::CssModule::TransformError.new(original_name, 'CSS module path not given')
+      end
+
       resolved_path = Resolver.resolve(path.to_s)
       digest = Importer.import(resolved_path)
 
-      sname = name.to_s
-      if sname.start_with?('_')
-        "_#{sname[1..]}-#{digest}"
-      else
-        "#{sname}-#{digest}"
-      end
+      transformed_name = name.to_s
+      transformed_name = if transformed_name.start_with?('_')
+                           "_#{transformed_name[1..]}-#{digest}"
+                         else
+                           "#{transformed_name}-#{digest}"
+                         end
+
+      [transformed_name, resolved_path]
     end
   end
 end

data/lib/proscenium/css_module.rb

@@ -6,11 +6,20 @@ module Proscenium::CssModule
   autoload :Path
   autoload :Transformer
 
+  class TransformError < StandardError
+    def initialize(name, additional_msg = nil)
+      msg = "Failed to transform CSS module `#{name}`"
+      msg << ' - ' << additional_msg if additional_msg
+
+      super msg
+    end
+  end
+
   # Accepts one or more CSS class names, and transforms them into CSS module names.
   #
   # @param name [String,Symbol,Array<String,Symbol>]
   def css_module(*names)
-    cssm.class_names(*names, require_prefix: false).join ' '
+    cssm.class_names(*names, require_prefix: false).map { |name, _| name }.join(' ')
   end
 
   private

data/lib/proscenium/helper.rb

@@ -22,7 +22,9 @@ module Proscenium
     # @param name [String,Symbol,Array<String,Symbol>]
     def css_module(*names)
       path = Pathname.new(@lookup_context.find(@virtual_path).identifier).sub_ext('')
-      CssModule::Transformer.new(path).class_names(*names, require_prefix: false).join ' '
+      CssModule::Transformer.new(path).class_names(*names, require_prefix: false).map do |name, _|
+        name
+      end.join(' ')
     end
 
     def include_stylesheets(**options)

data/lib/proscenium/phlex/css_modules.rb

@@ -6,6 +6,28 @@ module Proscenium
 
     def self.included(base)
       base.extend CssModule::Path
+      base.extend ClassMethods
+    end
+
+    module ClassMethods
+      # Set of CSS module paths that have been resolved after being transformed from 'class' HTML
+      # attributes. See #process_attributes. This is here because Phlex caches attributes. Which
+      # means while the CSS class names will be transformed, any resolved paths will be lost in
+      # subsequent requests.
+      attr_accessor :resolved_css_module_paths
+    end
+
+    def before_template
+      self.class.resolved_css_module_paths ||= Concurrent::Set.new
+      super
+    end
+
+    def after_template
+      self.class.resolved_css_module_paths.each do |path|
+        Proscenium::Importer.import path
+      end
+
+      super
     end
 
     # Resolve and side load any CSS modules in the "class" attributes, where a CSS module is a class
@@ -44,7 +66,11 @@ module Proscenium
     def process_attributes(**attributes)
       if attributes.key?(:class) && (attributes[:class] = tokens(attributes[:class])).include?('@')
         names = attributes[:class].is_a?(Array) ? attributes[:class] : attributes[:class].split
-        attributes[:class] = cssm.class_names(*names)
+
+        attributes[:class] = cssm.class_names(*names).map do |name, path|
+          self.class.resolved_css_module_paths << path if path
+          name
+        end
       end
 
       attributes

data/lib/proscenium/phlex.rb

@@ -6,7 +6,6 @@ module Proscenium
   class Phlex < ::Phlex::HTML
     extend ActiveSupport::Autoload
 
-    autoload :Page
     autoload :CssModules
     autoload :ReactComponent
 

data/lib/proscenium/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Proscenium
-  VERSION = '0.11.0.pre.9'
+  VERSION = '0.11.0.pre.10'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: proscenium
 version: !ruby/object:Gem::Version
-  version: 0.11.0.pre.9
+  version: 0.11.0.pre.10
 platform: x86_64-darwin
 authors:
 - Joel Moss
@@ -112,7 +112,6 @@ files:
 - lib/proscenium/monkey.rb
 - lib/proscenium/phlex.rb
 - lib/proscenium/phlex/css_modules.rb
-- lib/proscenium/phlex/page.rb
 - lib/proscenium/phlex/react_component.rb
 - lib/proscenium/railtie.rb
 - lib/proscenium/react_componentable.rb
@@ -149,7 +148,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.1
 requirements: []
-rubygems_version: 3.4.19
+rubygems_version: 3.4.20
 signing_key:
 specification_version: 4
 summary: The engine powering your Rails frontend

data/lib/proscenium/phlex/page.rb

@@ -1,62 +0,0 @@
-# frozen_string_literal: true
-
-require 'phlex/rails'
-
-# Include this in your view for additional logic for rendering a full HTML page, usually from a
-# controller.
-module Proscenium::Phlex::Page
-  include Phlex::Rails::Helpers::CSPMetaTag
-  include Phlex::Rails::Helpers::CSRFMetaTags
-  include Phlex::Rails::Helpers::FaviconLinkTag
-  include Phlex::Rails::Helpers::PreloadLinkTag
-  include Phlex::Rails::Helpers::StyleSheetLinkTag
-  include Phlex::Rails::Helpers::ActionCableMetaTag
-  include Phlex::Rails::Helpers::AutoDiscoveryLinkTag
-  include Phlex::Rails::Helpers::JavaScriptIncludeTag
-  include Phlex::Rails::Helpers::JavaScriptImportMapTags
-  include Phlex::Rails::Helpers::JavaScriptImportModuleTag
-
-  def self.included(klass)
-    klass.extend(Phlex::Rails::Layout::Interface)
-  end
-
-  def template(&block)
-    doctype
-    html do
-      head
-      body(&block)
-    end
-  end
-
-  private
-
-  def after_template
-    super
-    @_buffer.gsub!('<!-- [SIDE_LOAD_STYLESHEETS] -->', capture { include_stylesheets })
-  end
-
-  def page_title
-    Rails.application.class.name.deconstantize
-  end
-
-  def head
-    super do
-      title { page_title }
-
-      yield if block_given?
-
-      csp_meta_tag
-      csrf_meta_tags
-
-      comment { '[SIDE_LOAD_STYLESHEETS]' }
-    end
-  end
-
-  def body
-    super do
-      yield if block_given?
-
-      include_javascripts type: :module, defer: true
-    end
-  end
-end