xote 4.16.1 → 6.0.0

package/src/{Xote__Component.res → Node.res}

@@ -1,4 +1,3 @@
-open Signals
 
 /* ============================================================================
  * DOM Bindings
@@ -246,7 +245,7 @@ module Render = {
         setOwner(textNode, owner)
 
         runWithOwner(owner, () => {
-          let disposer = Effect.run(() => {
+          let disposer = Effect.runWithDisposer(() => {
             DOM.setTextContent(textNode, Signal.get(signal))
             None
           })
@@ -272,7 +271,7 @@ module Render = {
         setOwner(container, owner)
 
         runWithOwner(owner, () => {
-          let disposer = Effect.run(() => {
+          let disposer = Effect.runWithDisposer(() => {
             let children = Signal.get(signal)
 
             /* Dispose existing children */
@@ -315,7 +314,7 @@ module Render = {
             | Static(v) => DOM.setAttrOrProp(el, key, v)
             | SignalValue(signal) => {
                 DOM.setAttrOrProp(el, key, Signal.peek(signal))
-                let disposer = Effect.run(
+                let disposer = Effect.runWithDisposer(
                   () => {
                     DOM.setAttrOrProp(el, key, Signal.get(signal))
                     None
@@ -324,7 +323,7 @@ module Render = {
                 addDisposer(owner, disposer)
               }
             | Compute(compute) => {
-                let disposer = Effect.run(
+                let disposer = Effect.runWithDisposer(
                   () => {
                     DOM.setAttrOrProp(el, key, compute())
                     None
@@ -479,7 +478,7 @@ module Render = {
         fragment->DOM.appendChild(endAnchor)
 
         runWithOwner(owner, () => {
-          let disposer = Effect.run(() => {
+          let disposer = Effect.runWithDisposer(() => {
             reconcile()
             None
           })
@@ -499,23 +498,17 @@ module Render = {
 /* Text nodes */
 let text = (content: string): node => Text(content)
 
-let textSignal = (compute: unit => string): node => {
+let signalText = (compute: unit => string): node => {
   let signal = Computed.make(compute)
   SignalText(signal)
 }
 
-/* Reactive text nodes with type-specific helpers */
-let reactiveString = (compute: unit => string): node => {
-  let signal = Computed.make(compute)
-  SignalText(signal)
-}
-
-let reactiveInt = (compute: unit => int): node => {
+let signalInt = (compute: unit => int): node => {
   let signal = Computed.make(() => compute()->Int.toString)
   SignalText(signal)
 }
 
-let reactiveFloat = (compute: unit => float): node => {
+let signalFloat = (compute: unit => float): node => {
   let signal = Computed.make(() => compute()->Float.toString)
   SignalText(signal)
 }
@@ -559,22 +552,6 @@ let element = (
   (),
 ): node => Element({tag, attrs, events, children})
 
-/* Common elements */
-let div = (~attrs=?, ~events=?, ~children=?, ()) =>
-  element("div", ~attrs?, ~events?, ~children?, ())
-let span = (~attrs=?, ~events=?, ~children=?, ()) =>
-  element("span", ~attrs?, ~events?, ~children?, ())
-let button = (~attrs=?, ~events=?, ~children=?, ()) =>
-  element("button", ~attrs?, ~events?, ~children?, ())
-let input = (~attrs=?, ~events=?, ()) => element("input", ~attrs?, ~events?, ())
-let h1 = (~attrs=?, ~events=?, ~children=?, ()) => element("h1", ~attrs?, ~events?, ~children?, ())
-let h2 = (~attrs=?, ~events=?, ~children=?, ()) => element("h2", ~attrs?, ~events?, ~children?, ())
-let h3 = (~attrs=?, ~events=?, ~children=?, ()) => element("h3", ~attrs?, ~events?, ~children?, ())
-let p = (~attrs=?, ~events=?, ~children=?, ()) => element("p", ~attrs?, ~events?, ~children?, ())
-let ul = (~attrs=?, ~events=?, ~children=?, ()) => element("ul", ~attrs?, ~events?, ~children?, ())
-let li = (~attrs=?, ~events=?, ~children=?, ()) => element("li", ~attrs?, ~events?, ~children?, ())
-let a = (~attrs=?, ~events=?, ~children=?, ()) => element("a", ~attrs?, ~events?, ~children?, ())
-
 /* Null representation */
 let null = () => text("")
 

package/src/{Xote__Component.res.mjs → Node.res.mjs}

@@ -1,7 +1,9 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
-import * as Signals from "rescript-signals/src/Signals.res.mjs";
+import * as Effect$Xote from "./Effect.res.mjs";
+import * as Signal$Xote from "./Signal.res.mjs";
 import * as Stdlib_Dict from "@rescript/runtime/lib/es6/Stdlib_Dict.js";
+import * as Computed$Xote from "./Computed.res.mjs";
 import * as Stdlib_Option from "@rescript/runtime/lib/es6/Stdlib_Option.js";
 import * as Primitive_option from "@rescript/runtime/lib/es6/Primitive_option.js";
 
@@ -115,7 +117,7 @@ function addDisposer(owner, disposer) {
 
 function disposeOwner(owner) {
   owner.disposers.forEach(disposer => disposer.dispose());
-  owner.computeds.forEach(computed => Signals.Computed.dispose(computed));
+  owner.computeds.forEach(Computed$Xote.dispose);
 }
 
 function setOwner(element, owner) {
@@ -203,14 +205,14 @@ function render(node) {
               return setAttrOrProp(el, key, value._0);
             case "SignalValue" :
               let signal = value._0;
-              setAttrOrProp(el, key, Signals.Signal.peek(signal));
-              let disposer = Signals.Effect.run(() => {
-                setAttrOrProp(el, key, Signals.Signal.get(signal));
+              setAttrOrProp(el, key, Signal$Xote.peek(signal));
+              let disposer = Effect$Xote.runWithDisposer(() => {
+                setAttrOrProp(el, key, Signal$Xote.get(signal));
               }, undefined);
               return addDisposer(owner, disposer);
             case "Compute" :
               let compute = value._0;
-              let disposer$1 = Signals.Effect.run(() => {
+              let disposer$1 = Effect$Xote.runWithDisposer(() => {
                 setAttrOrProp(el, key, compute());
               }, undefined);
               return addDisposer(owner, disposer$1);
@@ -229,15 +231,15 @@ function render(node) {
       return document.createTextNode(node._0);
     case "SignalText" :
       let signal = node._0;
-      let textNode = document.createTextNode(Signals.Signal.peek(signal));
+      let textNode = document.createTextNode(Signal$Xote.peek(signal));
       let owner$1 = {
         disposers: [],
         computeds: []
       };
       setOwner(textNode, owner$1);
       runWithOwner(owner$1, () => {
-        let disposer = Signals.Effect.run(() => {
-          textNode.textContent = Signals.Signal.get(signal);
+        let disposer = Effect$Xote.runWithDisposer(() => {
+          textNode.textContent = Signal$Xote.get(signal);
         }, undefined);
         addDisposer(owner$1, disposer);
       });
@@ -259,8 +261,8 @@ function render(node) {
       container.setAttribute("style", "display: contents");
       setOwner(container, owner$2);
       runWithOwner(owner$2, () => {
-        let disposer = Signals.Effect.run(() => {
-          let children = Signals.Signal.get(signal$1);
+        let disposer = Effect$Xote.runWithDisposer(() => {
+          let children = Signal$Xote.get(signal$1);
           let childNodes = (Array.from(container.childNodes || []));
           childNodes.forEach(disposeElement);
           ((container.innerHTML = ''));
@@ -298,7 +300,7 @@ function render(node) {
         if (parentOpt == null) {
           return;
         }
-        let newItems = Signals.Signal.get(signal$2);
+        let newItems = Signal$Xote.get(signal$2);
         let newKeyMap = {};
         newItems.forEach(item => {
           newKeyMap[keyFn(item)] = item;
@@ -384,7 +386,7 @@ function render(node) {
       };
       let fragment$1 = document.createDocumentFragment();
       fragment$1.appendChild(startAnchor);
-      let initialItems = Signals.Signal.peek(signal$2);
+      let initialItems = Signal$Xote.peek(signal$2);
       initialItems.forEach(item => {
         let key = keyFn(item);
         let node = renderItem(item);
@@ -399,7 +401,7 @@ function render(node) {
       });
       fragment$1.appendChild(endAnchor);
       runWithOwner(owner$4, () => {
-        let disposer = Signals.Effect.run(() => {
+        let disposer = Effect$Xote.runWithDisposer(() => {
           reconcile();
         }, undefined);
         addDisposer(owner$4, disposer);
@@ -420,32 +422,24 @@ function text(content) {
   };
 }
 
-function textSignal(compute) {
-  let signal = Signals.Computed.make(compute, undefined);
+function signalText(compute) {
+  let signal = Computed$Xote.make(compute, undefined);
   return {
     TAG: "SignalText",
     _0: signal
   };
 }
 
-function reactiveString(compute) {
-  let signal = Signals.Computed.make(compute, undefined);
+function signalInt(compute) {
+  let signal = Computed$Xote.make(() => compute().toString(), undefined);
   return {
     TAG: "SignalText",
     _0: signal
   };
 }
 
-function reactiveInt(compute) {
-  let signal = Signals.Computed.make(() => compute().toString(), undefined);
-  return {
-    TAG: "SignalText",
-    _0: signal
-  };
-}
-
-function reactiveFloat(compute) {
-  let signal = Signals.Computed.make(() => compute().toString(), undefined);
+function signalFloat(compute) {
+  let signal = Computed$Xote.make(() => compute().toString(), undefined);
   return {
     TAG: "SignalText",
     _0: signal
@@ -481,7 +475,7 @@ function signalFragment(signal) {
 }
 
 function list(signal, renderItem) {
-  let nodesSignal = Signals.Computed.make(() => Signals.Signal.get(signal).map(renderItem), undefined);
+  let nodesSignal = Computed$Xote.make(() => Signal$Xote.get(signal).map(renderItem), undefined);
   return {
     TAG: "SignalFragment",
     _0: nodesSignal
@@ -510,50 +504,6 @@ function element(tag, attrsOpt, eventsOpt, childrenOpt, param) {
   };
 }
 
-function div(attrs, events, children, param) {
-  return element("div", attrs, events, children, undefined);
-}
-
-function span(attrs, events, children, param) {
-  return element("span", attrs, events, children, undefined);
-}
-
-function button(attrs, events, children, param) {
-  return element("button", attrs, events, children, undefined);
-}
-
-function input(attrs, events, param) {
-  return element("input", attrs, events, undefined, undefined);
-}
-
-function h1(attrs, events, children, param) {
-  return element("h1", attrs, events, children, undefined);
-}
-
-function h2(attrs, events, children, param) {
-  return element("h2", attrs, events, children, undefined);
-}
-
-function h3(attrs, events, children, param) {
-  return element("h3", attrs, events, children, undefined);
-}
-
-function p(attrs, events, children, param) {
-  return element("p", attrs, events, children, undefined);
-}
-
-function ul(attrs, events, children, param) {
-  return element("ul", attrs, events, children, undefined);
-}
-
-function li(attrs, events, children, param) {
-  return element("li", attrs, events, children, undefined);
-}
-
-function a(attrs, events, children, param) {
-  return element("a", attrs, events, children, undefined);
-}
-
 function $$null() {
   return {
     TAG: "Text",
@@ -597,10 +547,9 @@ export {
   computedAttr,
   Render,
   text,
-  textSignal,
-  reactiveString,
-  reactiveInt,
-  reactiveFloat,
+  signalText,
+  signalInt,
+  signalFloat,
   int,
   float,
   fragment,
@@ -608,17 +557,6 @@ export {
   list,
   keyedList,
   element,
-  div,
-  span,
-  button,
-  input,
-  h1,
-  h2,
-  h3,
-  p,
-  ul,
-  li,
-  a,
   $$null,
   mount,
   mountById,

package/src/{Xote__ReactiveProp.res → ReactiveProp.res}

@@ -1,4 +1,3 @@
-open Signals
 
 /* Defines a property that can either be a signal (Reactive) or a static value
  (Static) */

package/src/{Xote__ReactiveProp.res.mjs → ReactiveProp.res.mjs}

@@ -1,10 +1,10 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
-import * as Signals from "rescript-signals/src/Signals.res.mjs";
+import * as Signal$Xote from "./Signal.res.mjs";
 
 function get(value) {
   if (value.TAG === "Reactive") {
-    return Signals.Signal.get(value._0);
+    return Signal$Xote.get(value._0);
   } else {
     return value._0;
   }

package/src/{Xote__Router.res → Router.res}

@@ -1,6 +1,4 @@
-open Signals
-module Component = Xote__Component
-module Route = Xote__Route
+module Route = Route
 
 // Browser location type
 type location = {
@@ -57,7 +55,8 @@ let getGlobalState = (): globalRouterState => {
       }
 
       // Store in globalThis using the same symbol
-      %raw(`globalThis[Symbol.for("xote.router.state")] = state`)
+      ignore(state)
+      let _: unit = %raw(`globalThis[Symbol.for("xote.router.state")] = state`)
       state
     }
   }
@@ -72,7 +71,8 @@ let basePath = (): ref<string> => getGlobalState().basePath
 let warnIfNotInitialized = (methodName: string): unit => {
   let state = getGlobalState()
   if !state.initialized {
-    %raw(`console.warn(
+    ignore(methodName)
+    let _: unit = %raw(`console.warn(
       '[Xote Router] ' + methodName + ' called before Router.init(). ' +
       'Make sure to call Router.init() at your app entry point. ' +
       'This may cause incorrect routing behavior.'
@@ -147,11 +147,15 @@ let getScrollPosition = (): (float, float) => {
 }
 
 let scrollTo = (x: float, y: float): unit => {
-  %raw(`window.scrollTo(x, y)`)
+  ignore(x)
+  ignore(y)
+  let _: unit = %raw(`window.scrollTo(x, y)`)
 }
 
 // Create history state with scroll position
 let makeHistoryState = (scrollX: float, scrollY: float): historyState => {
+  ignore(scrollX)
+  ignore(scrollY)
   %raw(`({ scrollX: scrollX, scrollY: scrollY })`)
 }
 
@@ -161,6 +165,7 @@ let emptyHistoryState = (): historyState => {
 
 // Extract scroll position from history state
 let getScrollFromState = (state: historyState): option<(float, float)> => {
+  ignore(state)
   let scrollX: Nullable.t<float> = %raw(`state && state.scrollX`)
   let scrollY: Nullable.t<float> = %raw(`state && state.scrollY`)
   switch (Nullable.toOption(scrollX), Nullable.toOption(scrollY)) {
@@ -298,11 +303,11 @@ let replace = (pathname: string, ~search: string="", ~hash: string="", ()): unit
 // Route definition for routes() component
 type routeConfig = {
   pattern: string,
-  render: Route.params => Component.node,
+  render: Route.params => Node.node,
 }
 
 // Single route component - renders if pattern matches
-let route = (pattern: string, render: Route.params => Component.node): Component.node => {
+let route = (pattern: string, render: Route.params => Node.node): Node.node => {
   warnIfNotInitialized("Router.route()")
 
   let signal = Computed.make(() => {
@@ -312,11 +317,11 @@ let route = (pattern: string, render: Route.params => Component.node): Component
     | NoMatch => []
     }
   })
-  Component.signalFragment(signal)
+  Node.signalFragment(signal)
 }
 
 // Routes component - renders first matching route
-let routes = (configs: array<routeConfig>): Component.node => {
+let routes = (configs: array<routeConfig>): Node.node => {
   warnIfNotInitialized("Router.routes()")
 
   let signal = Computed.make(() => {
@@ -333,25 +338,25 @@ let routes = (configs: array<routeConfig>): Component.node => {
     | None => [] // No matching route - render nothing
     }
   })
-  Component.signalFragment(signal)
+  Node.signalFragment(signal)
 }
 
 // Link component - handles navigation without page reload
 let link = (
   ~to: string,
-  ~attrs: array<(string, Component.attrValue)>=[],
-  ~children: array<Component.node>=[],
+  ~attrs: array<(string, Node.attrValue)>=[],
+  ~children: array<Node.node>=[],
   (),
-): Component.node => {
+): Node.node => {
   warnIfNotInitialized("Router.link()")
 
   let handleClick = (_evt: Dom.event) => {
-    %raw(`_evt.preventDefault()`)
+    let _: unit = %raw(`_evt.preventDefault()`)
     push(to, ())
   }
 
-  Component.a(
-    ~attrs=Array.concat(attrs, [Component.attr("href", addBasePath(to))]),
+  Html.a(
+    ~attrs=Array.concat(attrs, [Node.attr("href", addBasePath(to))]),
     ~events=[("click", handleClick)],
     ~children,
     (),
@@ -360,7 +365,7 @@ let link = (
 
 // JSX Link component
 module Link = {
-  module ReactiveProp = Xote__ReactiveProp
+  module ReactiveProp = ReactiveProp
 
   type props<'class, 'id, 'style, 'target, 'ariaLabel> = {
     /* Required navigation prop */
@@ -374,36 +379,37 @@ module Link = {
     /* Event handlers */
     onClick?: Dom.event => unit,
     /* Children */
-    children?: Component.node,
+    children?: Node.node,
   }
 
   /* Helper to detect if a value is a ReactiveProp variant */
   let isReactiveProp = (value: 'a): bool => {
+    ignore(value)
     %raw(`value && typeof value === 'object' && ('TAG' in value) && (value.TAG === 'Static' || value.TAG === 'Reactive')`)
   }
 
   /* Helper to convert string attribute value */
-  let convertAttrValue = (key: string, value: 'a): (string, Component.attrValue) => {
+  let convertAttrValue = (key: string, value: 'a): (string, Node.attrValue) => {
     if isReactiveProp(value) {
       let rp: ReactiveProp.t<string> = Obj.magic(value)
       switch rp {
-      | Static(s) => Component.attr(key, s)
-      | Reactive(signal) => Component.signalAttr(key, signal)
+      | Static(s) => Node.attr(key, s)
+      | Reactive(signal) => Node.signalAttr(key, signal)
       }
     } else if typeof(value) == #function {
       let f: unit => string = Obj.magic(value)
-      Component.computedAttr(key, f)
+      Node.computedAttr(key, f)
     } else if typeof(value) == #object {
       let sig: Signal.t<string> = Obj.magic(value)
-      Component.signalAttr(key, sig)
+      Node.signalAttr(key, sig)
     } else {
       let s: string = Obj.magic(value)
-      Component.attr(key, s)
+      Node.attr(key, s)
     }
   }
 
   /* Convert props to attrs array */
-  let propsToAttrs = (props: props<_, _, _, _, _>): array<(string, Component.attrValue)> => {
+  let propsToAttrs = (props: props<_, _, _, _, _>): array<(string, Node.attrValue)> => {
     let attrs = []
 
     switch props.class {
@@ -435,20 +441,20 @@ module Link = {
   }
 
   /* Extract children from props */
-  let getChildren = (props: props<_, _, _, _, _>): array<Component.node> => {
+  let getChildren = (props: props<_, _, _, _, _>): array<Node.node> => {
     switch props.children {
-    | Some(Component.Fragment(children)) => children
+    | Some(Node.Fragment(children)) => children
     | Some(child) => [child]
     | None => []
     }
   }
 
   /* JSX component function */
-  let make = (props: props<_, _, _, _, _>): Component.node => {
+  let make = (props: props<_, _, _, _, _>): Node.node => {
     warnIfNotInitialized("Router.Link")
 
     let handleClick = (evt: Dom.event) => {
-      %raw(`evt.preventDefault()`)
+      let _: unit = %raw(`evt.preventDefault()`)
       push(props.to, ())
 
       // Call user's onClick if provided
@@ -458,8 +464,8 @@ module Link = {
       }
     }
 
-    Component.a(
-      ~attrs=Array.concat(propsToAttrs(props), [Component.attr("href", addBasePath(props.to))]),
+    Html.a(
+      ~attrs=Array.concat(propsToAttrs(props), [Node.attr("href", addBasePath(props.to))]),
       ~events=[("click", handleClick)],
       ~children=getChildren(props),
       (),