hotwire-spark 0.1.9 → 0.1.11

data/app/javascript/hotwire/spark/reloaders/replace_html_reloader.js

@@ -0,0 +1,31 @@
+import { log } from "../logger.js"
+
+export class ReplaceHtmlReloader {
+  static async reload() {
+    return new ReplaceHtmlReloader().reload()
+  }
+
+  async reload() {
+    await this.#reloadHtml()
+  }
+
+  async #reloadHtml() {
+    log("Reload html with Turbo...")
+
+    this.#keepScrollPosition()
+    await this.#visitCurrentPage()
+  }
+
+  #keepScrollPosition() {
+    document.addEventListener("turbo:before-render", () => {
+      Turbo.navigator.currentVisit.scrolled = true
+    }, { once: true })
+  }
+
+  #visitCurrentPage() {
+    return new Promise(resolve => {
+      document.addEventListener("turbo:load", () => resolve(document), { once: true })
+      window.Turbo.visit(window.location)
+    })
+  }
+}

data/app/javascript/hotwire/spark/reloaders/stimulus_reloader.js

@@ -2,14 +2,23 @@ import { log } from "../logger.js"
 import { cacheBustedUrl, reloadHtmlDocument } from "../helpers.js"
 
 export class StimulusReloader {
-  static async reload(filePattern) {
+  static async reload(changedFilePath) {
     const document = await reloadHtmlDocument()
-    return new StimulusReloader(document, filePattern).reload()
+    return new StimulusReloader(document, changedFilePath).reload()
   }
 
-  constructor(document, filePattern = /./) {
+  static async reloadAll() {
+    Stimulus.controllers.forEach(controller => {
+      Stimulus.unload(controller.identifier)
+      Stimulus.register(controller.identifier, controller.constructor)
+    })
+
+    return Promise.resolve()
+  }
+
+  constructor(document, changedFilePath) {
     this.document = document
-    this.filePattern = filePattern
+    this.changedFilePath = changedFilePath
     this.application = window.Stimulus
   }
 
@@ -40,7 +49,12 @@ export class StimulusReloader {
   }
 
   #shouldReloadController(path) {
-    return this.filePattern.test(path)
+    return this.#extractControllerName(path) === this.#changedControllerIdentifier
+  }
+
+  get #changedControllerIdentifier() {
+    this.changedControllerIdentifier = this.changedControllerIdentifier || this.#extractControllerName(this.changedFilePath)
+    return this.changedControllerIdentifier
   }
 
   get #stimulusPathsByModule() {
@@ -72,7 +86,7 @@ export class StimulusReloader {
     if (this.#didChangeTriggerAReload) {
       return []
     } else {
-      return this.application.controllers.filter(controller => this.filePattern.test(`${controller.identifier}_controller`))
+      return this.application.controllers.filter(controller => this.#changedControllerIdentifier === controller.identifier)
     }
   }
 
@@ -86,10 +100,12 @@ export class StimulusReloader {
 
   #extractControllerName(path) {
     return path
-      .replace(/^.*\//, "")
+      .replace(/^\/+/, "")
+      .replace(/^controllers\//, "")
       .replace("_controller", "")
       .replace(/\//g, "--")
       .replace(/_/g, "-")
+      .replace(/\.js$/, "")
   }
 
   #registerController(name, module) {

data/config/routes.rb

@@ -1,2 +1,3 @@
 Hotwire::Spark::Engine.routes.draw do
+  get "/source_files", to: "source_files#show"
 end

data/lib/hotwire/spark/change.rb

@@ -0,0 +1,37 @@
+class Hotwire::Spark::Change
+  attr_reader :paths, :extensions, :changed_path, :action
+
+  def initialize(paths, extensions, changed_path, action)
+    @paths = paths
+    @extensions = extensions
+    @changed_path = changed_path
+    @action = action
+  end
+
+  def broadcast
+    broadcast_reload_action if should_broadcast?
+  end
+
+  private
+    def broadcast_reload_action
+      Hotwire::Spark.cable_server.broadcast "hotwire_spark", reload_message
+    end
+
+    def reload_message
+      { action: action, path: canonical_changed_path }
+    end
+
+    def canonical_changed_path
+      canonical_changed_path = changed_path
+      paths.each { |path| canonical_changed_path = canonical_changed_path.to_s.gsub(/^#{path}/, "") }
+      canonical_changed_path
+    end
+
+    def should_broadcast?
+      changed_path.to_s =~ extension_regexp
+    end
+
+    def extension_regexp
+      /#{extensions.map { |ext| "\\." + ext }.join("|")}$/
+    end
+end

data/lib/hotwire/spark/default_options.rb

@@ -0,0 +1,36 @@
+class Hotwire::Spark::DefaultOptions
+  def initialize
+    @config = base_options
+
+    build
+  end
+
+  def to_h
+    @config
+  end
+
+  private
+    def base_options
+      {
+        enabled: Rails.env.development?,
+        css_paths: File.directory?("app/assets/builds") ? %w[ app/assets/builds ] : %w[ app/assets/stylesheets ],
+        css_extensions: %w[ css ],
+        html_paths: %w[ app/controllers app/helpers app/models app/views ],
+        html_extensions: %w[ rb erb ],
+        stimulus_paths: %w[ app/javascript/controllers ],
+        stimulus_extensions: %w[ js ],
+        html_reload_method: :morph
+      }
+    end
+
+    def build
+      configure_jsbundling if defined?(Jsbundling)
+    end
+
+    def configure_jsbundling
+      @config[:stimulus_paths] = []
+      @config[:html_paths] << "app/assets/builds"
+      @config[:html_extensions] << "js"
+      @config[:html_reload_method] = :replace
+    end
+end

data/lib/hotwire/spark/engine.rb

@@ -1,4 +1,5 @@
 require "action_cable/server/base"
+require "hotwire/spark/default_options"
 
 module Hotwire::Spark
   class Engine < ::Rails::Engine
@@ -6,11 +7,7 @@ module Hotwire::Spark
 
     config.hotwire = ActiveSupport::OrderedOptions.new unless config.respond_to?(:hotwire)
     config.hotwire.spark = ActiveSupport::OrderedOptions.new
-    config.hotwire.spark.merge! \
-      enabled: Rails.env.development?,
-      css_paths: File.directory?("app/assets/builds") ? %w[ app/assets/builds ] : %w[ app/assets/stylesheets ],
-      html_paths: %w[ app/controllers app/helpers app/models app/views ],
-      stimulus_paths: %w[ app/javascript/controllers ]
+    config.hotwire.spark.merge! Hotwire::Spark::DefaultOptions.new.to_h
 
     initializer "hotwire_spark.config" do |application|
       config.hotwire.spark.each do |key, value|

data/lib/hotwire/spark/file_watcher.rb

@@ -36,9 +36,13 @@ class Hotwire::Spark::FileWatcher
       changed_files.each do |file|
         @callbacks_by_path.each do |path, callbacks|
           if file.to_s.start_with?(path.to_s)
-            callbacks.each { |callback| callback.call(file) }
+            callbacks.each { |callback| callback.call(as_relative_path(file)) }
           end
         end
       end
     end
+
+    def as_relative_path(path)
+      Pathname.new(path).relative_path_from(Rails.application.root)
+    end
 end

data/lib/hotwire/spark/installer.rb

@@ -29,25 +29,20 @@ class Hotwire::Spark::Installer
     end
 
     def register_monitored_paths
-      monitor :css_paths, action: :reload_css
-      monitor :html_paths, action: :reload_html
-      monitor :stimulus_paths, action: :reload_stimulus
+      monitor :css_paths, action: :reload_css, extensions: Hotwire::Spark.css_extensions
+      monitor :html_paths, action: :reload_html, extensions: Hotwire::Spark.html_extensions
+      monitor :stimulus_paths, action: :reload_stimulus, extensions: Hotwire::Spark.stimulus_extensions
     end
 
-    def monitor(paths_name, action:)
-      file_watcher.monitor Hotwire::Spark.public_send(paths_name) do |file_path|
-        broadcast_reload_action(action, file_path)
+    def monitor(paths_name, action:, extensions:)
+      paths = Hotwire::Spark.public_send(paths_name)
+      if paths.present?
+        file_watcher.monitor paths do |file_path|
+          Hotwire::Spark::Change.new(paths, extensions, file_path, action).broadcast
+        end
       end
     end
 
-    def broadcast_reload_action(action, file_path)
-      Hotwire::Spark.cable_server.broadcast "hotwire_spark", reload_message_for(action, file_path)
-    end
-
-    def reload_message_for(action, file_path)
-      { action: action, path: file_path }
-    end
-
     def file_watcher
       @file_watches ||= Hotwire::Spark::FileWatcher.new
     end

data/lib/hotwire/spark/middleware.rb

@@ -7,6 +7,7 @@ class Hotwire::Spark::Middleware
     status, headers, response = @app.call(env)
 
     if html_response?(headers)
+      @request = ActionDispatch::Request.new(env)
       html = html_from(response)
       html = inject_javascript(html)
       html = inject_options(html)
@@ -38,18 +39,22 @@ class Hotwire::Spark::Middleware
     end
 
     def view_helpers
-      ActionController::Base.helpers
+      @request.controller_instance.helpers
     end
 
     def inject_options(html)
-      if Hotwire::Spark.logging
-        html.sub("</head>", "#{logging_option}</head>")
-      else
-        html
-      end
+      html.sub("</head>", "#{options}</head>")
+    end
+
+    def options
+      [ logging_option, html_reload_method_option ].compact.join("\n")
     end
 
     def logging_option
-      view_helpers.tag.meta(name: "hotwire-spark:logging", content: "true")
+      view_helpers.tag.meta(name: "hotwire-spark:logging", content: "true") if Hotwire::Spark.logging
+    end
+
+    def html_reload_method_option
+      view_helpers.tag.meta(name: "hotwire-spark:html-reload-method", content: Hotwire::Spark.html_reload_method)
     end
 end

data/lib/hotwire/spark/version.rb

@@ -1,5 +1,5 @@
 module Hotwire
   module Spark
-    VERSION = "0.1.9"
+    VERSION = "0.1.11"
   end
 end

data/lib/hotwire-spark.rb

@@ -8,10 +8,13 @@ loader.ignore("#{__dir__}/hotwire/spark/version.rb")
 loader.setup
 
 module Hotwire::Spark
-  mattr_accessor :css_paths, default: []
-  mattr_accessor :html_paths, default: []
-  mattr_accessor :stimulus_paths, default: []
+  %i[ css html stimulus ].each do |type|
+    mattr_accessor "#{type}_paths".to_sym, default: []
+    mattr_accessor "#{type}_extensions".to_sym, default: []
+  end
+
   mattr_accessor :logging, default: false
+  mattr_accessor :html_reload_method, default: :morph
 
   mattr_accessor :enabled, default: Rails.env.development?
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hotwire-spark
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.1.11
 platform: ruby
 authors:
 - Jorge Manrubia
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-22 00:00:00.000000000 Z
+date: 2024-12-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 8.0.0
+        version: 7.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 8.0.0
+        version: 7.0.0
 - !ruby/object:Gem::Dependency
   name: zeitwerk
   requirement: !ruby/object:Gem::Requirement
@@ -131,11 +131,14 @@ files:
 - app/javascript/hotwire/spark/index.js
 - app/javascript/hotwire/spark/logger.js
 - app/javascript/hotwire/spark/reloaders/css_reloader.js
-- app/javascript/hotwire/spark/reloaders/html_reloader.js
+- app/javascript/hotwire/spark/reloaders/morph_html_reloader.js
+- app/javascript/hotwire/spark/reloaders/replace_html_reloader.js
 - app/javascript/hotwire/spark/reloaders/stimulus_reloader.js
 - config/routes.rb
 - lib/hotwire-spark.rb
 - lib/hotwire/spark/action_cable/server.rb
+- lib/hotwire/spark/change.rb
+- lib/hotwire/spark/default_options.rb
 - lib/hotwire/spark/engine.rb
 - lib/hotwire/spark/file_watcher.rb
 - lib/hotwire/spark/installer.rb