hotwire-spark 0.1.6 → 0.1.7

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

@@ -18,18 +18,26 @@ export class StimulusReloader {
     log("Reload Stimulus controllers...")
 
     this.application.stop()
-    await this.#reloadStimulusControllers()
+
+    await this.#reloadChangedStimulusControllers()
+    this.#unloadDeletedStimulusControllers()
+
     this.application.start()
   }
 
-  async #reloadStimulusControllers() {
+  async #reloadChangedStimulusControllers() {
     await Promise.all(
-      this.#stimulusControllerPaths.map(async moduleName => this.#reloadStimulusController(moduleName))
+      this.#stimulusControllerPathsToReload.map(async moduleName => this.#reloadStimulusController(moduleName))
     )
   }
 
+  get #stimulusControllerPathsToReload() {
+    this.controllerPathsToReload = this.controllerPathsToReload || this.#stimulusControllerPaths.filter(path => this.#shouldReloadController(path))
+    return this.controllerPathsToReload
+  }
+
   get #stimulusControllerPaths() {
-    return Object.keys(this.#stimulusPathsByModule).filter(path => path.endsWith("_controller") && this.#shouldReloadController(path))
+    return Object.keys(this.#stimulusPathsByModule).filter(path => path.endsWith("_controller"))
   }
 
   #shouldReloadController(path) {
@@ -57,6 +65,22 @@ export class StimulusReloader {
     this.#registerController(controllerName, module)
   }
 
+  #unloadDeletedStimulusControllers() {
+    this.#controllersToUnload.forEach(controller => this.#deregisterController(controller.identifier))
+  }
+
+  get #controllersToUnload() {
+    if (this.#didChangeTriggerAReload) {
+      return []
+    } else {
+      return this.application.controllers.filter(controller => this.filePattern.test(`${controller.identifier}_controller`))
+    }
+  }
+
+  get #didChangeTriggerAReload() {
+    return this.#stimulusControllerPathsToReload.length > 0
+  }
+
   #pathForModuleName(moduleName) {
     return this.#stimulusPathsByModule[moduleName]
   }
@@ -73,4 +97,9 @@ export class StimulusReloader {
     this.application.unload(name)
     this.application.register(name, module.default)
   }
+
+  #deregisterController(name) {
+    log(`\tRemoving controller ${name}`)
+    this.application.unload(name)
+  }
 }

data/lib/hotwire/spark/version.rb

@@ -1,5 +1,5 @@
 module Hotwire
   module Spark
-    VERSION = "0.1.6"
+    VERSION = "0.1.7"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hotwire-spark
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors:
 - Jorge Manrubia