react-on-rails 16.3.0-rc.2 → 16.4.0-rc.0

package/lib/ComponentRegistry.js

@@ -6,13 +6,17 @@ export default {
      */
     register(components) {
         Object.keys(components).forEach((name) => {
-            if (registeredComponents.has(name)) {
-                console.warn('Called register for component that is already registered', name);
-            }
             const component = components[name];
             if (!component) {
                 throw new Error(`Called register with null component named ${name}`);
             }
+            // Reference comparison lets HMR re-register the same component silently
+            // while still catching bugs where different components share a name.
+            const existing = registeredComponents.get(name);
+            if (existing && existing.component !== component) {
+                console.error(`ReactOnRails: Component "${name}" was registered with a different component than previously. ` +
+                    'This is likely a bug — ensure each component has a unique registration name.');
+            }
             const renderFunction = isRenderFunction(component);
             const isRenderer = renderFunction && component.length === 3;
             registeredComponents.set(name, {

package/lib/StoreRegistry.js

@@ -7,14 +7,18 @@ export default {
      */
     register(storeGenerators) {
         Object.keys(storeGenerators).forEach((name) => {
-            if (registeredStoreGenerators.has(name)) {
-                console.warn('Called registerStore for store that is already registered', name);
-            }
             const store = storeGenerators[name];
             if (!store) {
                 throw new Error('Called ReactOnRails.registerStores with a null or undefined as a value ' +
                     `for the store generator with key ${name}.`);
             }
+            // Reference comparison lets HMR re-register the same store silently
+            // while still catching bugs where different stores share a name.
+            const existing = registeredStoreGenerators.get(name);
+            if (existing && existing !== store) {
+                console.error(`ReactOnRails: Store "${name}" was registered with a different store generator than previously. ` +
+                    'This is likely a bug — ensure each store has a unique registration name.');
+            }
             registeredStoreGenerators.set(name, store);
         });
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-on-rails",
-  "version": "16.3.0-rc.2",
+  "version": "16.4.0-rc.0",
   "description": "react-on-rails JavaScript for react_on_rails Ruby gem",
   "main": "lib/ReactOnRails.full.js",
   "type": "module",
@@ -65,8 +65,6 @@
     "build-watch": "pnpm run clean && tsc --watch",
     "clean": "rm -rf ./lib",
     "test": "jest tests",
-    "type-check": "tsc --noEmit --noErrorTruncation",
-    "yalc:publish": "yalc publish",
-    "yalc": "yalc"
+    "type-check": "tsc --noEmit --noErrorTruncation"
   }
 }