@bitstillery/mithril 3.2.0 → 3.3.0

package/api/router.ts

@@ -105,7 +105,7 @@ export default function router($window: any, mountRedraw: MountRedraw) {
                 // (e.g. Stock <-> TBO) forces full remount. Key must change when path changes.
                 // Always wrap when we have a path so structure is consistent; key drives remount.
                 const fragmentKey = remountPath ?? currentPath ?? undefined
-                if (fragmentKey != null) {
+                if (remountPath && fragmentKey != null) {
                     return hyperscript.fragment({key: fragmentKey}, result)
                 }
                 return result

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@bitstillery/mithril",
-    "version": "3.2.0",
+    "version": "3.3.0",
     "description": "Mithril + Signals, Store and SSR",
     "license": "MIT",
     "author": "Bitstillery",
@@ -56,6 +56,7 @@
         "test": "bun test",
         "prepublishOnly": "bun run lint:ts",
         "pack:dry-run": "npm pack --dry-run",
+        "publish": "./scripts/release.sh",
         "release": "./scripts/release.sh",
         "release:patch": "./scripts/release.sh --release-as patch",
         "release:minor": "./scripts/release.sh --release-as minor",

package/signal.ts

@@ -100,32 +100,40 @@ export class Signal<T> {
     set value(newValue: T) {
         if (this._value !== newValue) {
             this._value = newValue
-            // Ensure _subscribers is initialized (defensive check)
-            if (!this._subscribers) {
-                this._subscribers = new Set()
-            }
-            // Notify all subscribers
-            const context = getSSRContext()
-            this._subscribers.forEach((fn) => {
-                try {
-                    // Always run watchers - wrap in SSR context if available
-                    if (context) {
-                        // Run watcher inside SSR context, similar to events
-                        runWithContext(context, () => {
-                            fn()
-                        })
-                    } else {
+            this.trigger()
+        }
+    }
+
+    /**
+     * Notify subscribers and trigger redraws without changing the value.
+     * Use when the value is an object/array that was mutated in place (e.g. keys added/removed).
+     */
+    trigger(): void {
+        // Ensure _subscribers is initialized (defensive check)
+        if (!this._subscribers) {
+            this._subscribers = new Set()
+        }
+        // Notify all subscribers
+        const context = getSSRContext()
+        this._subscribers.forEach((fn) => {
+            try {
+                // Always run watchers - wrap in SSR context if available
+                if (context) {
+                    // Run watcher inside SSR context, similar to events
+                    runWithContext(context, () => {
                         fn()
-                    }
-                } catch (e) {
-                    console.error('Error in signal subscriber:', e)
+                    })
+                } else {
+                    fn()
                 }
-            })
-            // Trigger component redraws for affected components
-            // This is set up in index.ts after m.redraw is created
-            if ((signal as any).__redrawCallback) {
-                ;(signal as any).__redrawCallback(this)
+            } catch (e) {
+                console.error('Error in signal subscriber:', e)
             }
+        })
+        // Trigger component redraws for affected components
+        // This is set up in index.ts after m.redraw is created
+        if ((signal as any).__redrawCallback) {
+            ;(signal as any).__redrawCallback(this)
         }
     }
 

package/state.ts

@@ -211,6 +211,9 @@ export function state<T extends Record<string, any>>(initial: T, name?: string,
                 ;(value as any)._parentSignal = sig as Signal<any>
             } else if (Array.isArray(value)) {
                 arrayParentSignalMap.set(value, sig as Signal<any>)
+            } else if ((value as any).__isState === true) {
+                // Nested object proxies: notify parent when keys are added/removed
+                arrayParentSignalMap.set(value, sig as Signal<any>)
             }
         }
 
@@ -738,8 +741,13 @@ export function state<T extends Record<string, any>>(initial: T, name?: string,
                         nestedSignalMap.set(key, createPropertySignal(value))
                     }
                 } else {
-                    // Create new signal
+                    // Create new signal (new key added to object)
                     nestedSignalMap.set(key, createPropertySignal(value))
+                    // Notify parent so subscribers see the key addition
+                    const parentSignal = arrayParentSignalMap.get(wrapped) || (wrapped as any)._parentSignal
+                    if (parentSignal && typeof (parentSignal as any).trigger === 'function') {
+                        ;(parentSignal as Signal<any>).trigger()
+                    }
                 }
 
                 return true
@@ -794,6 +802,11 @@ export function state<T extends Record<string, any>>(initial: T, name?: string,
                     }
                     // Remove from the signal map
                     nestedSignalMap.delete(key)
+                    // Notify parent so subscribers see the key removal
+                    const parentSignal = arrayParentSignalMap.get(wrapped) || (wrapped as any)._parentSignal
+                    if (parentSignal && typeof (parentSignal as any).trigger === 'function') {
+                        ;(parentSignal as Signal<any>).trigger()
+                    }
                 }
 
                 // Delete from target