@hypen-space/core 0.4.31 → 0.4.35

package/src/router.ts

@@ -4,6 +4,7 @@
  */
 
 import { createObservableState, getStateSnapshot } from "./state.js";
+import { disposableListener, type Disposable } from "./disposable.js";
 import { frameworkLoggers } from "./logger.js";
 
 const log = frameworkLoggers.router;
@@ -31,6 +32,7 @@ export class HypenRouter {
   private subscribers = new Set<RouteChangeCallback>();
   private isInitialized = false;
   private isUpdating = false;
+  private browserListeners: Disposable[] = [];
 
   constructor() {
     // Create observable state for reactivity
@@ -65,18 +67,22 @@ export class HypenRouter {
     this.state.query = this.parseQuery();
 
     // Listen for browser back/forward
-    window.addEventListener("popstate", () => {
-      const newPath = this.getPathFromBrowser();
-      this.updatePath(newPath, false); // Don't push to history again
-    });
+    this.browserListeners.push(
+      disposableListener(window, "popstate", () => {
+        const newPath = this.getPathFromBrowser();
+        this.updatePath(newPath, false); // Don't push to history again
+      })
+    );
 
     // Listen for hash changes
-    window.addEventListener("hashchange", () => {
-      // Don't respond to hashchange events we triggered ourselves
-      if (this.isUpdating) return;
-      const newPath = this.getPathFromBrowser();
-      this.updatePath(newPath, false);
-    });
+    this.browserListeners.push(
+      disposableListener(window, "hashchange", () => {
+        // Don't respond to hashchange events we triggered ourselves
+        if (this.isUpdating) return;
+        const newPath = this.getPathFromBrowser();
+        this.updatePath(newPath, false);
+      })
+    );
 
     this.isInitialized = true;
     log.debug("Router initialized at:", initialPath);
@@ -333,4 +339,24 @@ export class HypenRouter {
     const queryString = new URLSearchParams(query).toString();
     return `${path}?${queryString}`;
   }
+
+  /**
+   * Clean up browser event listeners and subscriptions
+   */
+  dispose(): void {
+    for (const listener of this.browserListeners) {
+      listener.dispose();
+    }
+    this.browserListeners = [];
+    this.subscribers.clear();
+    this.isInitialized = false;
+    log.debug("Router disposed");
+  }
+
+  /**
+   * Support `using` syntax for automatic cleanup
+   */
+  [Symbol.dispose](): void {
+    this.dispose();
+  }
 }

package/src/state.ts

@@ -279,9 +279,15 @@ export function createObservableState<T extends object>(
           notificationPending = false;
           if (batchDepth === 0) {
             notifyChange();
+          } else {
+            // A batch started after we scheduled; the batch's __endBatch will
+            // call notifyChange() when it completes, so nothing to do here.
           }
         });
       }
+    } else {
+      // Inside a batch — mark that changes occurred so __endBatch knows to notify
+      pendingChange = pendingChange || { paths: [], newValues: {} };
     }
   }
 
@@ -377,11 +383,12 @@ export function createObservableState<T extends object>(
       },
 
       deleteProperty(obj, prop) {
-        if (prop in obj) {
-          delete obj[prop];
+        const existed = Object.prototype.hasOwnProperty.call(obj, prop);
+        const result = delete obj[prop];
+        if (existed) {
           scheduleBatch();
         }
-        return true;
+        return result;
       },
     });
 

package/wasm-browser/README.md

@@ -11,7 +11,7 @@ The core reactive rendering engine for Hypen, written in Rust. Compiles to WASM
 ### Rust
 
 ```rust
-use hypen_engine::{Engine, ast_to_ir};
+use hypen_engine::{Engine, ast_to_ir_node};
 use hypen_parser::parse_component;
 use serde_json::json;
 
@@ -21,7 +21,7 @@ engine.set_render_callback(|patches| {
 });
 
 let ast = parse_component(r#"Column { Text("Hello") }"#)?;
-engine.render(&ast_to_ir(&ast));
+engine.render_ir_node(&ast_to_ir_node(&ast));
 engine.update_state(json!({ "count": 42 }));
 ```
 
@@ -120,7 +120,7 @@ use serde_json::json;
 let mut engine = Engine::new();
 
 // Register a custom component
-// Note: In practice, you'd typically parse Hypen DSL with ast_to_ir
+// Note: In practice, you'd typically parse Hypen DSL with ast_to_ir_node
 engine.register_component(Component::new("Greeting", |props: IndexMap<String, serde_json::Value>| {
     Element::new("Text")
         .with_prop("text", Value::Binding(
@@ -400,7 +400,7 @@ The engine integrates with the Hypen parser from `../parser`:
 
 ```rust
 use hypen_parser::parse_component;
-use hypen_engine::ast_to_ir;
+use hypen_engine::ast_to_ir_node;
 
 let source = r#"
     Column {
@@ -410,14 +410,14 @@ let source = r#"
 "#;
 
 let ast = parse_component(source)?;
-let element = ast_to_ir(&ast); // Convert AST → IR
-engine.render(&element);
+let ir_node = ast_to_ir_node(&ast); // Convert AST → IR
+engine.render_ir_node(&ir_node);
 ```
 
 ### Full Example with Parser
 
 ```rust
-use hypen_engine::{Engine, ast_to_ir};
+use hypen_engine::{Engine, ast_to_ir_node};
 use hypen_parser::parse_component;
 use serde_json::json;
 
@@ -438,10 +438,10 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
     "#;
 
     let ast = parse_component(source)?;
-    let element = ast_to_ir(&ast);
+    let ir_node = ast_to_ir_node(&ast);
 
     // Render
-    engine.render(&element);
+    engine.render_ir_node(&ir_node);
 
     // Update state
     engine.update_state(json!({"count": 42}));
@@ -557,7 +557,7 @@ impl Engine {
 
 ```rust
 pub use engine::Engine;
-pub use ir::{ast_to_ir, Element, Value};
+pub use ir::{ast_to_ir_node, Element, IRNode, Value};
 pub use lifecycle::{Module, ModuleInstance};
 pub use reconcile::Patch;
 pub use state::StateChange;