what-core 0.1.1 → 0.2.0

package/src/store.js

@@ -4,15 +4,32 @@
 
 import { signal, computed, batch } from './reactive.js';
 
+// --- storeComputed ---
+// Marker wrapper to explicitly tag a function as a computed in createStore.
+// Without this, createStore can't distinguish computed(state => ...) from action(item => ...).
+//
+// Usage:
+//   const useCounter = createStore({
+//     count: 0,
+//     doubled: storeComputed(state => state.count * 2),
+//     addItem(item) { /* this is an action */ },
+//   });
+
+export function storeComputed(fn) {
+  fn._storeComputed = true;
+  return fn;
+}
+
 // --- createStore ---
 // Creates a reactive store with actions. Each key becomes a signal.
 //
 // Usage:
 //   const useCounter = createStore({
 //     count: 0,
-//     doubled: (state) => state.count * 2,  // computed
-//     increment() { this.count++; },         // action
+//     doubled: storeComputed(state => state.count * 2),  // computed
+//     increment() { this.count++; },                      // action
 //     decrement() { this.count--; },
+//     addItem(item) { this.items.push(item); },           // action (not confused with computed)
 //   });
 //
 //   function Counter() {
@@ -27,12 +44,13 @@ export function createStore(definition) {
   const state = {};
 
   // Separate state, computeds, and actions
+  // Use explicit _storeComputed marker instead of function.length heuristic
   for (const [key, value] of Object.entries(definition)) {
-    if (typeof value === 'function' && value.length > 0 && key !== 'constructor') {
-      // Computed: function that takes state
+    if (typeof value === 'function' && value._storeComputed) {
+      // Computed: explicitly marked with storeComputed()
       computeds[key] = value;
     } else if (typeof value === 'function') {
-      // Action: function with no args that uses `this`
+      // Action: any other function
       actions[key] = value;
     } else {
       // State: initial value