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

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 11a4ddf9e18d4a0e84b7e1e0bc4a97b34fa21b0006ba5eee34cff79761e51ead
-  data.tar.gz: 5a7195f12faeab4e91015fcb1d758d13f3b76490f9ae599cd0afbdab87b63e00
+  metadata.gz: bbaa1085da775617ff05dd56b98b9596a269ce6dbdcbf2022298b805ba60d9d1
+  data.tar.gz: 12cba17bddc0b42ef9e9352a816711ab4dca1ee8b2c1090c95774aaf32772c1f
 SHA512:
-  metadata.gz: ce2ee1789cb6035d1bd647c3cc638d233713b5d756533b420a154058e8c0974ecabcb4ee2f0172ee8d7ac4e073e5a697af885f3a1ae273bbef4679f31bf97c16
-  data.tar.gz: 552ebbd157858654fe2c3f0c03e94351c44c21037f9450a1911bd4d0107e6f234f8ebd2ecad8d757401f96400d01eb1d2a69705f4111d0a92ac46b8667bd141d
+  metadata.gz: c8f0ca2517304db1a98e07f86b6f170de1a798e73044914974dc9997b8ef9475f71926466ed40f035b02bafba2b35206dcbefb66573c82def82e6c36ffb9a8e1
+  data.tar.gz: 5c575c1788cd624ed78d358e4d5a01158a532f6e16676d86fa267aadd7eb6300684f6a9cbf9038e43dd849af81de6b0459c533323def4cd9cdbc3a354c457bad

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)
@@ -75,10 +77,14 @@ module Proscenium
 
         if extract_lazy_scripts
           content_for :proscenium_lazy_scripts do
-            javascript_tag "window.prosceniumLazyScripts = #{scripts.to_json}"
+            tag.script type: 'application/json', id: 'prosceniumLazyScripts' do
+              raw scripts.to_json
+            end
           end
         else
-          out << javascript_tag("window.prosceniumLazyScripts = #{scripts.to_json}")
+          out << tag.script(type: 'application/json', id: 'prosceniumLazyScripts') do
+            raw scripts.to_json
+          end
         end
       else
         Importer.each_javascript(delete: true) do |path, _|

data/lib/proscenium/libs/react-manager/index.jsx

@@ -1,7 +1,16 @@
-const elements = document.querySelectorAll("[data-proscenium-component-path]");
+window.Proscenium = window.Proscenium || { lazyScripts: {} };
+
+const element = document.querySelector("#prosceniumLazyScripts");
+if (element) {
+  const scripts = JSON.parse(element.text);
+  window.Proscenium.lazyScripts = {
+    ...window.Proscenium.lazyScripts,
+    ...scripts,
+  };
+}
 
-// Initialize only if there are components.
-elements.length > 0 && init();
+const elements = document.querySelectorAll("[data-proscenium-component-path]");
+elements.length > 0 && init(elements);
 
 function init() {
   /**
@@ -20,12 +29,12 @@ function init() {
     // For testing and simulation of slow connections.
     // const sim = new Promise((resolve) => setTimeout(resolve, 5000));
 
-    if (!(path in window.prosceniumLazyScripts)) {
-      throw `[proscenium/react/manager] Cannot load component ${path} (not found in prosceniumLazyScripts)`;
+    if (!window.Proscenium.lazyScripts[path]) {
+      throw `[proscenium/react/manager] Cannot load component ${path} (not found in Proscenium.lazyScripts)`;
     }
 
     const react = import("@proscenium/react-manager/react");
-    const Component = import(window.prosceniumLazyScripts[path].outpath);
+    const Component = import(window.Proscenium.lazyScripts[path].outpath);
 
     const forwardChildren =
       "prosceniumComponentForwardChildren" in element.dataset &&

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.11'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: proscenium
 version: !ruby/object:Gem::Version
-  version: 0.11.0.pre.9
+  version: 0.11.0.pre.11
 platform: x86_64-darwin
 authors:
 - Joel Moss
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-05 00:00:00.000000000 Z
+date: 2023-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -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