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

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b3287d39998953706dc485e4e1fce3deb82ba6e0fe4b24a458c072e6ea37a0b
-  data.tar.gz: 47768ea66abc4a04d9e4b537e0be7a524b6ba0d674721b67f8a5094977862018
+  metadata.gz: edf70bf926665cd639fde327c74349c49ea85db20c607cc82c34129a37263262
+  data.tar.gz: 22e4a0ac95c58f8ce6e5e08199f0be4e746397789462c411f3b8da7c66cf234f
 SHA512:
-  metadata.gz: 6202ef97089d6021f74a5ccf38dbd6fcb2b615e620cea70d409fa21556a26ee8c025ceb649c29006e9c93cbbd47bb72aa8b87768edcba744b43d0eb039ada867
-  data.tar.gz: 4de4d272ab59e2f41407e3a1e1c7cdc8585e7256f33be967ceb798d5f8e90abc517fb0408f6fa28ca1cd6fa764137e9686fd97be343bd4e8bc34996098e3e210
+  metadata.gz: 6532d587e386f4f99ffc4fcc7f2873a9471694fbc0a90dc05e50ff1bdee053276852c69067375995015a29a6ffe4b6ecf329f78c158f99fcd997b19fb8afd386
+  data.tar.gz: f98efacb7bb59d2f565284f5cde6e2acdabe773c5c4627f07d31ab228750d1fb39a73b717a0c1f24d88861fe0273f966f6390bdd0950f914ee0521a872294c2c

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-linux
 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