@rplx/core 0.2.0 → 0.2.1

package/dist/index.d.mts

@@ -79,6 +79,12 @@ interface StoreConfig<State, Cofx = {}> {
         enabled?: boolean;
         debounceTime?: number;
     };
+    /**
+     * Optional scheduler for batching state updates
+     * Defaults to requestAnimationFrame
+     * @internal - For testing purposes only
+     */
+    __scheduler?: (callback: FrameRequestCallback) => number;
 }
 interface QueuedEvent {
     eventKey: string;

package/dist/index.d.ts

@@ -79,6 +79,12 @@ interface StoreConfig<State, Cofx = {}> {
         enabled?: boolean;
         debounceTime?: number;
     };
+    /**
+     * Optional scheduler for batching state updates
+     * Defaults to requestAnimationFrame
+     * @internal - For testing purposes only
+     */
+    __scheduler?: (callback: FrameRequestCallback) => number;
 }
 interface QueuedEvent {
     eventKey: string;

package/dist/index.js

@@ -77,6 +77,8 @@ function createRegistrar(options) {
           } else {
             console.warn(`re-frame: can't clear ${kind} handler for ${id}. Handler not found.`);
           }
+        } else {
+          console.warn(`re-frame: can't clear ${kind} handler for ${id}. Handler not found.`);
         }
       }
     }
@@ -84,13 +86,13 @@ function createRegistrar(options) {
 }
 
 // src/modules/state.ts
-function createStateManager(initialState, onStateChange, onStateChangeForSubscriptions) {
+function createStateManager(initialState, onStateChange, onStateChangeForSubscriptions, scheduler = requestAnimationFrame) {
   let state = initialState;
   const stateChanges = [];
   let rafId = null;
   const scheduleNotification = () => {
     if (rafId !== null) return;
-    rafId = requestAnimationFrame(() => {
+    rafId = scheduler(() => {
       rafId = null;
       if (stateChanges.length > 0) {
         const latestState = stateChanges[stateChanges.length - 1];
@@ -375,21 +377,21 @@ function createEffectExecutor(deps) {
     registerEffect("db", (newState) => {
       stateManager.setState(newState);
     });
-    registerEffect("dispatch", async (config) => {
+    registerEffect("dispatch", (config) => {
       if (!config || typeof config.event !== "string") {
         console.error("re-frame: ignoring bad :dispatch value. Expected {event: string, payload: any}, but got:", config);
         return;
       }
-      await dispatch(config.event, config.payload);
+      dispatch(config.event, config.payload);
     });
-    registerEffect("dispatch-n", async (configs) => {
+    registerEffect("dispatch-n", (configs) => {
       if (!Array.isArray(configs)) {
         console.error("re-frame: ignoring bad :dispatch-n value. Expected an array, but got:", configs);
         return;
       }
       for (const config of configs) {
         if (config && config.event) {
-          await dispatch(config.event, config.payload);
+          dispatch(config.event, config.payload);
         }
       }
     });
@@ -676,33 +678,54 @@ function createRouter(deps) {
   const { eventManager } = deps;
   const eventQueue = [];
   let isProcessing = false;
+  let processingPromise = null;
   async function processEvent(eventKey, payload) {
     await eventManager.handleEvent(eventKey, payload);
   }
   async function processQueue() {
     if (isProcessing) return;
     isProcessing = true;
-    try {
-      while (eventQueue.length > 0) {
-        const event = eventQueue.shift();
-        await processEvent(event.eventKey, event.payload);
+    const promise = (async () => {
+      try {
+        while (eventQueue.length > 0) {
+          const event = eventQueue.shift();
+          try {
+            await processEvent(event.eventKey, event.payload);
+            event.resolve();
+          } catch (error) {
+            event.reject(error instanceof Error ? error : new Error(String(error)));
+          }
+        }
+      } finally {
+        isProcessing = false;
+        processingPromise = null;
       }
-    } finally {
-      isProcessing = false;
-    }
+    })();
+    processingPromise = promise;
+    await promise;
   }
   function dispatch(eventKey, payload) {
     return new Promise((resolve, reject) => {
-      eventQueue.push({ eventKey, payload });
+      eventQueue.push({
+        eventKey,
+        payload,
+        resolve,
+        reject
+      });
       if (!isProcessing) {
-        processQueue().then(resolve).catch(reject);
-      } else {
-        resolve();
+        processQueue().catch((error) => {
+          console.error("Unexpected error in processQueue:", error);
+        });
       }
     });
   }
   async function flush() {
-    await processQueue();
+    if (processingPromise) {
+      await processingPromise;
+    }
+    if (eventQueue.length > 0) {
+      await processQueue();
+    }
   }
   return {
     dispatch,
@@ -898,7 +921,9 @@ function createStore(config) {
       if (subscriptionManagerRef) {
         subscriptionManagerRef.notifyListeners(state);
       }
-    }
+    },
+    config.__scheduler ?? requestAnimationFrame
+    // Use scheduler if provided, otherwise default to RAF
   );
   const subscriptionManager = createSubscriptionManager({
     stateManager,
@@ -1075,7 +1100,8 @@ function debug() {
       return context;
     },
     after: (context) => {
-      console.log("New State:", context.coeffects.db);
+      const newState = context.effects.db !== void 0 ? context.effects.db : context.coeffects.db;
+      console.log("New State:", newState);
       console.log("Effects:", context.effects);
       console.groupEnd();
       return context;
@@ -1086,7 +1112,8 @@ function after(fn) {
   return {
     id: "after",
     after: (context) => {
-      fn(context.coeffects.db, context.effects);
+      const newState = context.effects.db !== void 0 ? context.effects.db : context.coeffects.db;
+      fn(newState, context.effects);
       return context;
     }
   };
@@ -1109,7 +1136,8 @@ function validate(schema) {
   return {
     id: "validate",
     after: (context) => {
-      const result = schema(context.coeffects.db);
+      const stateToValidate = context.effects.db !== void 0 ? context.effects.db : context.coeffects.db;
+      const result = schema(stateToValidate);
       if (result !== true) {
         console.error("State validation failed:", result);
       }

package/dist/index.mjs

@@ -42,6 +42,8 @@ function createRegistrar(options) {
           } else {
             console.warn(`re-frame: can't clear ${kind} handler for ${id}. Handler not found.`);
           }
+        } else {
+          console.warn(`re-frame: can't clear ${kind} handler for ${id}. Handler not found.`);
         }
       }
     }
@@ -49,13 +51,13 @@ function createRegistrar(options) {
 }
 
 // src/modules/state.ts
-function createStateManager(initialState, onStateChange, onStateChangeForSubscriptions) {
+function createStateManager(initialState, onStateChange, onStateChangeForSubscriptions, scheduler = requestAnimationFrame) {
   let state = initialState;
   const stateChanges = [];
   let rafId = null;
   const scheduleNotification = () => {
     if (rafId !== null) return;
-    rafId = requestAnimationFrame(() => {
+    rafId = scheduler(() => {
       rafId = null;
       if (stateChanges.length > 0) {
         const latestState = stateChanges[stateChanges.length - 1];
@@ -340,21 +342,21 @@ function createEffectExecutor(deps) {
     registerEffect("db", (newState) => {
       stateManager.setState(newState);
     });
-    registerEffect("dispatch", async (config) => {
+    registerEffect("dispatch", (config) => {
       if (!config || typeof config.event !== "string") {
         console.error("re-frame: ignoring bad :dispatch value. Expected {event: string, payload: any}, but got:", config);
         return;
       }
-      await dispatch(config.event, config.payload);
+      dispatch(config.event, config.payload);
     });
-    registerEffect("dispatch-n", async (configs) => {
+    registerEffect("dispatch-n", (configs) => {
       if (!Array.isArray(configs)) {
         console.error("re-frame: ignoring bad :dispatch-n value. Expected an array, but got:", configs);
         return;
       }
       for (const config of configs) {
         if (config && config.event) {
-          await dispatch(config.event, config.payload);
+          dispatch(config.event, config.payload);
         }
       }
     });
@@ -641,33 +643,54 @@ function createRouter(deps) {
   const { eventManager } = deps;
   const eventQueue = [];
   let isProcessing = false;
+  let processingPromise = null;
   async function processEvent(eventKey, payload) {
     await eventManager.handleEvent(eventKey, payload);
   }
   async function processQueue() {
     if (isProcessing) return;
     isProcessing = true;
-    try {
-      while (eventQueue.length > 0) {
-        const event = eventQueue.shift();
-        await processEvent(event.eventKey, event.payload);
+    const promise = (async () => {
+      try {
+        while (eventQueue.length > 0) {
+          const event = eventQueue.shift();
+          try {
+            await processEvent(event.eventKey, event.payload);
+            event.resolve();
+          } catch (error) {
+            event.reject(error instanceof Error ? error : new Error(String(error)));
+          }
+        }
+      } finally {
+        isProcessing = false;
+        processingPromise = null;
       }
-    } finally {
-      isProcessing = false;
-    }
+    })();
+    processingPromise = promise;
+    await promise;
   }
   function dispatch(eventKey, payload) {
     return new Promise((resolve, reject) => {
-      eventQueue.push({ eventKey, payload });
+      eventQueue.push({
+        eventKey,
+        payload,
+        resolve,
+        reject
+      });
       if (!isProcessing) {
-        processQueue().then(resolve).catch(reject);
-      } else {
-        resolve();
+        processQueue().catch((error) => {
+          console.error("Unexpected error in processQueue:", error);
+        });
       }
     });
   }
   async function flush() {
-    await processQueue();
+    if (processingPromise) {
+      await processingPromise;
+    }
+    if (eventQueue.length > 0) {
+      await processQueue();
+    }
   }
   return {
     dispatch,
@@ -863,7 +886,9 @@ function createStore(config) {
       if (subscriptionManagerRef) {
         subscriptionManagerRef.notifyListeners(state);
       }
-    }
+    },
+    config.__scheduler ?? requestAnimationFrame
+    // Use scheduler if provided, otherwise default to RAF
   );
   const subscriptionManager = createSubscriptionManager({
     stateManager,
@@ -1040,7 +1065,8 @@ function debug() {
       return context;
     },
     after: (context) => {
-      console.log("New State:", context.coeffects.db);
+      const newState = context.effects.db !== void 0 ? context.effects.db : context.coeffects.db;
+      console.log("New State:", newState);
       console.log("Effects:", context.effects);
       console.groupEnd();
       return context;
@@ -1051,7 +1077,8 @@ function after(fn) {
   return {
     id: "after",
     after: (context) => {
-      fn(context.coeffects.db, context.effects);
+      const newState = context.effects.db !== void 0 ? context.effects.db : context.coeffects.db;
+      fn(newState, context.effects);
       return context;
     }
   };
@@ -1074,7 +1101,8 @@ function validate(schema) {
   return {
     id: "validate",
     after: (context) => {
-      const result = schema(context.coeffects.db);
+      const stateToValidate = context.effects.db !== void 0 ? context.effects.db : context.coeffects.db;
+      const result = schema(stateToValidate);
       if (result !== true) {
         console.error("State validation failed:", result);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rplx/core",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "A re-frame inspired state management library for TypeScript",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -19,9 +19,18 @@
   "scripts": {
     "build": "tsup src/index.ts --format cjs,esm --dts",
     "dev": "tsup src/index.ts --format cjs,esm --dts --watch",
-    "clean": "rm -rf dist"
+    "clean": "rm -rf dist",
+    "test": "jest",
+    "test:watch": "jest --watch",
+    "test:coverage": "jest --coverage",
+    "test:types": "jest __tests__/type-tests"
   },
   "devDependencies": {
+    "@jest/test-sequencer": "^30.2.0",
+    "@types/jest": "^30.0.0",
+    "expect-type": "^1.3.0",
+    "jest": "^30.2.0",
+    "ts-jest": "^29.4.6",
     "tsup": "^8.0.0",
     "typescript": "^5.2.2"
   },
@@ -34,13 +43,13 @@
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "https://github.com/anonymeye/ripple.git",
-    "directory": "packages/ripple"
+    "url": "https://github.com/anonymeye/ripplex.git",
+    "directory": "packages/ripplex"
   },
   "bugs": {
-    "url": "https://github.com/anonymeye/ripple/issues"
+    "url": "https://github.com/anonymeye/ripplex/issues"
   },
-  "homepage": "https://github.com/anonymeye/ripple#readme",
+  "homepage": "https://github.com/anonymeye/ripplex#readme",
   "publishConfig": {
     "access": "public"
   }