@praxisjs/store 2.0.3 → 2.0.5

package/CHANGELOG.md

@@ -1,5 +1,46 @@
 # @praxisjs/store
 
+## 2.0.5
+
+### Patch Changes
+
+- 75115a9: Decorated fields are defined by their innermost decorator. With `@praxisjs/vite-plugin`, `@State()`, `@Prop()`, every built-in field decorator and `createFieldDecorator` bindings install their property directly with the field's initial value instead of replacing a plain value, which kept every instance in V8's slow dictionary mode. New `addClaimableInitializer(context, init)` export for field decorators written against the TC39 API.
+- 75115a9: Field decorators throw a `TypeError` when the class is defined if they decorate an `accessor`, naming the member to fix. Covers `@State()`, `@Prop()`, every built-in field decorator and `createFieldDecorator` bindings. New `assertFieldContext(context, label?)` export for field decorators written against the TC39 API directly.
+- 75115a9: Published packages no longer contain their tests: package builds compile `src` without `src/**/__tests__` (through a `tsconfig.build.json`), and each package's `files` leaves test sources and stale compiled tests out of the tarball. `pnpm typecheck` still type-checks the tests.
+- 75115a9: Performance pass and bug fixes across the feature packages.
+  - `@praxisjs/router`: static paths are matched through an index and dynamic ones with a single regex run; `RouterView` keeps a shared layout mounted across navigations; new `router.isActive(path)` (used by `Link`, so a navigation re-evaluates only the links whose state changed) and `router.destroy()`. Fixes regex characters in route paths (`.`, `(`, `+`...) being treated as patterns, and a router replaced by `createRouter()` still reacting to `popstate`. `@Route` no longer wraps the class, and the field decorators share one accessor.
+  - `@praxisjs/store`: plugin-wrapped actions keep their identity across reads, extensions are bound once, and writing a field inside an effect no longer subscribes the effect to it. `constructor` and inherited `Object.prototype` members are no longer reported as actions. `@Store` caches the singleton.
+  - `@praxisjs/motion`: `@Tween` and `@Spring` share one accessor per field and no longer start a frame loop at construction; `tween()`/`spring()` react to target writes without an effect. Fixes `reset()` leaving a tween marked as playing.
+  - `@praxisjs/di`: cached singletons and tokens resolve without allocating, `@Injectable` registers the class itself instead of a subclass, and a dependency cycle through `inject()` calls in constructors now reports `[DI] Circular dependency detected` instead of overflowing the stack.
+  - `@praxisjs/fsm`: `send()` and `reset()` no longer subscribe the calling effect to the machine state, transitions allocate nothing per send, and `@StateMachine` shares one accessor.
+  - `@praxisjs/head`: tag diffing without `JSON.stringify` or a full `<head>` query per update; updating or removing an entry below the top one leaves the document alone, and removing an unknown id no longer resets the title.
+  - `@praxisjs/content`: each entry is loaded, parsed and rendered once and shared by `getCollection`, `getPage` and `getEntry`; `getEntry` looks the slug up in an index; schema defaults are built once per class.
+
+- 75115a9: Smaller production bundles. The framework and feature packages are marked `"sideEffects": false`, so bundlers leave out the modules an app does not use (`For`/`Index` and `selector()` were kept in every bundle by their shape keepers). `@praxisjs/core`'s graph gates its observer instrumentation on a boolean that minifiers fold away when no observer is ever installed. `@praxisjs/runtime` passes `Index`'s positional update to the list view, so `For`-only apps leave it out, and builds its event prop table from camel-case names. `@praxisjs/devtools` is built one file per module and its decorators check `process.env.NODE_ENV` inline, so `@Trace`/`@Debug` in production code no longer bundle the registry and the panel. `@praxisjs/vite-plugin` serves TypeScript's decorator helpers and the claimed fields' helper from one virtual module (`virtual:praxisjs/decorators`) instead of writing them into every decorated file, and replaces the identical copies `tsc` wrote into ES module libraries.
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+- Updated dependencies [75115a9]
+  - @praxisjs/decorators@2.0.0
+  - @praxisjs/core@2.3.0
+
+## 2.0.4
+
+### Patch Changes
+
+- 055d5b3: `createStore` actions keep a stable wrapper identity, `$patch()` and `$reset()` run in a single batch, and the previous value passed to `onMutation` is read untracked.
+- Updated dependencies [a97a37c]
+- Updated dependencies [c7a5950]
+  - @praxisjs/core@2.2.0
+  - @praxisjs/decorators@1.7.0
+
 ## 2.0.3
 
 ### Patch Changes

package/dist/apply-plugins.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"apply-plugins.d.ts","sourceRoot":"","sources":["../src/apply-plugins.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAErD,wBAAgB,mBAAmB,CAAC,CAAC,SAAS,MAAM,EAClD,QAAQ,EAAE,CAAC,EACX,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,WAAW,EAAE,GACrB,CAAC,CAsEH"}
+{"version":3,"file":"apply-plugins.d.ts","sourceRoot":"","sources":["../src/apply-plugins.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAUrD,wBAAgB,mBAAmB,CAAC,CAAC,SAAS,MAAM,EAClD,QAAQ,EAAE,CAAC,EACX,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,WAAW,EAAE,GACrB,CAAC,CA8FH"}

package/dist/apply-plugins.js

@@ -1,55 +1,85 @@
+import { pauseTracking, resumeTracking } from "@praxisjs/core/internal";
+// `constructor` and the members every object inherits (`toString`, `hasOwnProperty`...) are not
+// store actions: wrapping them would report them to plugins and break `new store.constructor()`.
+function isBuiltin(prop, val) {
+    return prop === "constructor" || Object.prototype[prop] === val;
+}
 export function applyPluginsToStore(instance, storeName, plugins) {
     if (plugins.length === 0)
         return instance;
-    const extensions = {};
+    const actionPlugins = plugins.filter((p) => "onAction" in p || "onActionDone" in p);
+    const mutationPlugins = plugins.filter((p) => "onMutation" in p);
+    // Extensions are stored ready to return (functions already bound), so a read does no work.
+    const extensions = new Map();
+    // One wrapper per method, so `store.action` keeps its identity across reads; rebuilt only when
+    // the method itself is replaced.
+    const wrappers = new Map();
+    function wrap(name, fn) {
+        if (actionPlugins.length === 0) {
+            return (...args) => fn.apply(proxy, args);
+        }
+        return (...args) => {
+            for (const p of actionPlugins)
+                p.onAction?.({ name, args, storeName });
+            const result = fn.apply(proxy, args);
+            if (result instanceof Promise) {
+                return result
+                    .then((resolved) => {
+                    for (const p of actionPlugins)
+                        p.onActionDone?.({ name, args, storeName, result: resolved });
+                    return resolved;
+                })
+                    .catch((err) => {
+                    for (const p of actionPlugins)
+                        p.onActionDone?.({ name, args, storeName, result: undefined, error: err });
+                    throw err;
+                });
+            }
+            for (const p of actionPlugins)
+                p.onActionDone?.({ name, args, storeName, result });
+            return result;
+        };
+    }
     const proxy = new Proxy(instance, {
         get(target, prop) {
             if (typeof prop === "symbol")
                 return Reflect.get(target, prop);
-            if (prop in extensions) {
-                const ext = extensions[prop];
-                return typeof ext === "function"
-                    ? ext.bind(proxy)
-                    : ext;
-            }
+            if (extensions.size > 0 && extensions.has(prop))
+                return extensions.get(prop);
             const val = Reflect.get(target, prop, proxy);
-            if (typeof val === "function" && !prop.startsWith("$")) {
-                return (...args) => {
-                    for (const p of plugins)
-                        p.onAction?.({ name: prop, args, storeName });
-                    const result = val.apply(proxy, args);
-                    if (result instanceof Promise) {
-                        return result
-                            .then((resolved) => {
-                            for (const p of plugins)
-                                p.onActionDone?.({ name: prop, args, storeName, result: resolved });
-                            return resolved;
-                        })
-                            .catch((err) => {
-                            for (const p of plugins)
-                                p.onActionDone?.({ name: prop, args, storeName, result: undefined, error: err });
-                            throw err;
-                        });
-                    }
-                    for (const p of plugins)
-                        p.onActionDone?.({ name: prop, args, storeName, result });
-                    return result;
-                };
-            }
-            return val;
+            if (typeof val !== "function")
+                return val;
+            const cached = wrappers.get(prop);
+            if (cached?.fn === val)
+                return cached.wrapped;
+            if (prop.startsWith("$") || isBuiltin(prop, val))
+                return val;
+            const wrapped = wrap(prop, val);
+            wrappers.set(prop, { fn: val, wrapped });
+            return wrapped;
         },
         set(target, prop, value) {
             if (typeof prop === "symbol") {
                 target[prop] = value;
                 return true;
             }
-            const prevValue = Reflect.get(target, prop, proxy);
-            const ok = Reflect.set(target, prop, value, proxy);
-            if (ok) {
-                for (const p of plugins)
-                    p.onMutation?.({ key: prop, value, prevValue, storeName });
+            if (mutationPlugins.length === 0)
+                return Reflect.set(target, prop, value, proxy);
+            // A write is not a read: assigning a field inside an effect must not subscribe the effect to
+            // that field (through the previous value) or to whatever the plugins read.
+            const prevSub = pauseTracking();
+            try {
+                const prevValue = Reflect.get(target, prop, proxy);
+                const ok = Reflect.set(target, prop, value, proxy);
+                if (ok) {
+                    for (const p of mutationPlugins)
+                        p.onMutation?.({ key: prop, value, prevValue, storeName });
+                }
+                return ok;
+            }
+            finally {
+                resumeTracking(prevSub);
             }
-            return ok;
         },
     });
     for (const plugin of plugins) {
@@ -57,7 +87,10 @@ export function applyPluginsToStore(instance, storeName, plugins) {
             store: proxy,
             storeName,
             extend(props) {
-                Object.assign(extensions, props);
+                for (const key of Object.keys(props)) {
+                    const ext = props[key];
+                    extensions.set(key, typeof ext === "function" ? ext.bind(proxy) : ext);
+                }
             },
         });
     }

package/dist/apply-plugins.js.map

@@ -1 +1 @@
-{"version":3,"file":"apply-plugins.js","sourceRoot":"","sources":["../src/apply-plugins.ts"],"names":[],"mappings":"AAEA,MAAM,UAAU,mBAAmB,CACjC,QAAW,EACX,SAAiB,EACjB,OAAsB;IAEtB,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,QAAQ,CAAC;IAE1C,MAAM,UAAU,GAA4B,EAAE,CAAC;IAE/C,MAAM,KAAK,GAAM,IAAI,KAAK,CAAC,QAAQ,EAAE;QACnC,GAAG,CAAC,MAAM,EAAE,IAAqB;YAC/B,IAAI,OAAO,IAAI,KAAK,QAAQ;gBAAE,OAAO,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,CAAY,CAAC;YAE1E,IAAI,IAAI,IAAI,UAAU,EAAE,CAAC;gBACvB,MAAM,GAAG,GAAG,UAAU,CAAC,IAAI,CAAC,CAAC;gBAC7B,OAAO,OAAO,GAAG,KAAK,UAAU;oBAC9B,CAAC,CAAE,GAAoC,CAAC,IAAI,CAAC,KAAK,CAAC;oBACnD,CAAC,CAAC,GAAG,CAAC;YACV,CAAC;YAED,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,CAAY,CAAC;YAExD,IAAI,OAAO,GAAG,KAAK,UAAU,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;gBACvD,OAAO,CAAC,GAAG,IAAe,EAAW,EAAE;oBACrC,KAAK,MAAM,CAAC,IAAI,OAAO;wBAAE,CAAC,CAAC,QAAQ,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;oBACvE,MAAM,MAAM,GAAI,GAAoC,CAAC,KAAK,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;oBACxE,IAAI,MAAM,YAAY,OAAO,EAAE,CAAC;wBAC9B,OAAO,MAAM;6BACV,IAAI,CAAC,CAAC,QAAiB,EAAE,EAAE;4BAC1B,KAAK,MAAM,CAAC,IAAI,OAAO;gCACrB,CAAC,CAAC,YAAY,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;4BACtE,OAAO,QAAQ,CAAC;wBAClB,CAAC,CAAC;6BACD,KAAK,CAAC,CAAC,GAAY,EAAE,EAAE;4BACtB,KAAK,MAAM,CAAC,IAAI,OAAO;gCACrB,CAAC,CAAC,YAAY,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC,CAAC;4BACnF,MAAM,GAAG,CAAC;wBACZ,CAAC,CAAC,CAAC;oBACP,CAAC;oBACD,KAAK,MAAM,CAAC,IAAI,OAAO;wBACrB,CAAC,CAAC,YAAY,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,CAAC,CAAC;oBAC5D,OAAO,MAAM,CAAC;gBAChB,CAAC,CAAC;YACJ,CAAC;YAED,OAAO,GAAG,CAAC;QACb,CAAC;QAED,GAAG,CAAC,MAAM,EAAE,IAAqB,EAAE,KAAc;YAC/C,IAAI,OAAO,IAAI,KAAK,QAAQ,EAAE,CAAC;gBAC5B,MAAkC,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC;gBAClD,OAAO,IAAI,CAAC;YACd,CAAC;YACD,MAAM,SAAS,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,CAAY,CAAC;YAC9D,MAAM,EAAE,GAAY,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC;YAC5D,IAAI,EAAE,EAAE,CAAC;gBACP,KAAK,MAAM,CAAC,IAAI,OAAO;oBACrB,CAAC,CAAC,UAAU,EAAE,CAAC,EAAE,GAAG,EAAE,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;YAC/D,CAAC;YACD,OAAO,EAAE,CAAC;QACZ,CAAC;KACF,CAAC,CAAC;IAEH,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,MAAM,CAAC,MAAM,EAAE,CAAC;YACd,KAAK,EAAE,KAA2C;YAClD,SAAS;YACT,MAAM,CAAC,KAAK;gBACV,MAAM,CAAC,MAAM,CAAC,UAAU,EAAE,KAAK,CAAC,CAAC;YACnC,CAAC;SACF,CAAC,CAAC;IACL,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC"}
+{"version":3,"file":"apply-plugins.js","sourceRoot":"","sources":["../src/apply-plugins.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,aAAa,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAMxE,gGAAgG;AAChG,iGAAiG;AACjG,SAAS,SAAS,CAAC,IAAY,EAAE,GAAY;IAC3C,OAAO,IAAI,KAAK,aAAa,IAAK,MAAM,CAAC,SAAgD,CAAC,IAAI,CAAC,KAAK,GAAG,CAAC;AAC1G,CAAC;AAED,MAAM,UAAU,mBAAmB,CACjC,QAAW,EACX,SAAiB,EACjB,OAAsB;IAEtB,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,QAAQ,CAAC;IAE1C,MAAM,aAAa,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,IAAI,CAAC,IAAI,cAAc,IAAI,CAAC,CAAC,CAAC;IACpF,MAAM,eAAe,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,IAAI,CAAC,CAAC,CAAC;IACjE,2FAA2F;IAC3F,MAAM,UAAU,GAAG,IAAI,GAAG,EAAmB,CAAC;IAC9C,+FAA+F;IAC/F,iCAAiC;IACjC,MAAM,QAAQ,GAAG,IAAI,GAAG,EAA2C,CAAC;IAEpE,SAAS,IAAI,CAAC,IAAY,EAAE,EAAU;QACpC,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC/B,OAAO,CAAC,GAAG,IAAI,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;QAC5C,CAAC;QACD,OAAO,CAAC,GAAG,IAAI,EAAE,EAAE;YACjB,KAAK,MAAM,CAAC,IAAI,aAAa;gBAAE,CAAC,CAAC,QAAQ,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;YACvE,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;YACrC,IAAI,MAAM,YAAY,OAAO,EAAE,CAAC;gBAC9B,OAAO,MAAM;qBACV,IAAI,CAAC,CAAC,QAAiB,EAAE,EAAE;oBAC1B,KAAK,MAAM,CAAC,IAAI,aAAa;wBAC3B,CAAC,CAAC,YAAY,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;oBAChE,OAAO,QAAQ,CAAC;gBAClB,CAAC,CAAC;qBACD,KAAK,CAAC,CAAC,GAAY,EAAE,EAAE;oBACtB,KAAK,MAAM,CAAC,IAAI,aAAa;wBAC3B,CAAC,CAAC,YAAY,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC,CAAC;oBAC7E,MAAM,GAAG,CAAC;gBACZ,CAAC,CAAC,CAAC;YACP,CAAC;YACD,KAAK,MAAM,CAAC,IAAI,aAAa;gBAC3B,CAAC,CAAC,YAAY,EAAE,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,CAAC,CAAC;YACtD,OAAO,MAAM,CAAC;QAChB,CAAC,CAAC;IACJ,CAAC;IAED,MAAM,KAAK,GAAM,IAAI,KAAK,CAAC,QAAQ,EAAE;QACnC,GAAG,CAAC,MAAM,EAAE,IAAqB;YAC/B,IAAI,OAAO,IAAI,KAAK,QAAQ;gBAAE,OAAO,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,CAAY,CAAC;YAE1E,IAAI,UAAU,CAAC,IAAI,GAAG,CAAC,IAAI,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC;gBAAE,OAAO,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAE7E,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,CAAY,CAAC;YAExD,IAAI,OAAO,GAAG,KAAK,UAAU;gBAAE,OAAO,GAAG,CAAC;YAC1C,MAAM,MAAM,GAAG,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAClC,IAAI,MAAM,EAAE,EAAE,KAAK,GAAG;gBAAE,OAAO,MAAM,CAAC,OAAO,CAAC;YAC9C,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,SAAS,CAAC,IAAI,EAAE,GAAG,CAAC;gBAAE,OAAO,GAAG,CAAC;YAC7D,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,GAAa,CAAC,CAAC;YAC1C,QAAQ,CAAC,GAAG,CAAC,IAAI,EAAE,EAAE,EAAE,EAAE,GAAa,EAAE,OAAO,EAAE,CAAC,CAAC;YACnD,OAAO,OAAO,CAAC;QACjB,CAAC;QAED,GAAG,CAAC,MAAM,EAAE,IAAqB,EAAE,KAAc;YAC/C,IAAI,OAAO,IAAI,KAAK,QAAQ,EAAE,CAAC;gBAC5B,MAAkC,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC;gBAClD,OAAO,IAAI,CAAC;YACd,CAAC;YACD,IAAI,eAAe,CAAC,MAAM,KAAK,CAAC;gBAAE,OAAO,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC;YACjF,6FAA6F;YAC7F,2EAA2E;YAC3E,MAAM,OAAO,GAAG,aAAa,EAAE,CAAC;YAChC,IAAI,CAAC;gBACH,MAAM,SAAS,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,CAAY,CAAC;gBAC9D,MAAM,EAAE,GAAY,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC;gBAC5D,IAAI,EAAE,EAAE,CAAC;oBACP,KAAK,MAAM,CAAC,IAAI,eAAe;wBAC7B,CAAC,CAAC,UAAU,EAAE,CAAC,EAAE,GAAG,EAAE,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;gBAC/D,CAAC;gBACD,OAAO,EAAE,CAAC;YACZ,CAAC;oBAAS,CAAC;gBACT,cAAc,CAAC,OAAO,CAAC,CAAC;YAC1B,CAAC;QACH,CAAC;KACF,CAAC,CAAC;IAEH,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,MAAM,CAAC,MAAM,EAAE,CAAC;YACd,KAAK,EAAE,KAA2C;YAClD,SAAS;YACT,MAAM,CAAC,KAAK;gBACV,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;oBACrC,MAAM,GAAG,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC;oBACvB,UAAU,CAAC,GAAG,CACZ,GAAG,EACH,OAAO,GAAG,KAAK,UAAU,CAAC,CAAC,CAAE,GAAc,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,GAAG,CAC9D,CAAC;gBACJ,CAAC;YACH,CAAC;SACF,CAAC,CAAC;IACL,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC"}

package/dist/decorators.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"decorators.d.ts","sourceRoot":"","sources":["../src/decorators.ts"],"names":[],"mappings":"AAAA,OAAO,EAEL,gBAAgB,EAEjB,MAAM,sBAAsB,CAAC;AAK9B,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAQrD,qGAAqG;AACrG,qBAAa,aAAa;IACxB,SAAiB,CAAC,gBAAgB,CAAC,EAAE,IAAI,CAAC;CAC3C;AAED,MAAM,WAAW,eAAe;IAC9B,OAAO,CAAC,EAAE,WAAW,EAAE,CAAC;CACzB;AAED,wBAAgB,QAAQ,CAAC,OAAO,GAAE,eAAoB,IAElD,aAAa,KAAK,GAAG,IAAI,EAAE,OAAO,EAAE,KAAK,aAAa,EACtD,UAAU,qBAAqB,KAC9B,IAAI,CAMR;AAED,wBAAgB,KAAK,CAAC,CAAC,SAAS,aAAa,EAAE,UAAU,EAAE,UAAU,CAAC,GAAG,CAAC,CAUzE;AAED,wBAAgB,KAAK,CAAC,gBAAgB,EAAE,UAAU,OAAO,GAYtC,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,0BAA0B,CAAC,GAAG,CAAC,KAAK,IAAI,CACvF"}
+{"version":3,"file":"decorators.d.ts","sourceRoot":"","sources":["../src/decorators.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAA+C,MAAM,sBAAsB,CAAC;AAKrG,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAQrD,qGAAqG;AACrG,qBAAa,aAAa;IACxB,SAAiB,CAAC,gBAAgB,CAAC,EAAE,IAAI,CAAC;CAC3C;AAED,MAAM,WAAW,eAAe;IAC9B,OAAO,CAAC,EAAE,WAAW,EAAE,CAAC;CACzB;AAED,wBAAgB,QAAQ,CAAC,OAAO,GAAE,eAAoB,IAElD,aAAa,KAAK,GAAG,IAAI,EAAE,OAAO,EAAE,KAAK,aAAa,EACtD,UAAU,qBAAqB,KAC9B,IAAI,CAMR;AAED,wBAAgB,KAAK,CAAC,CAAC,SAAS,aAAa,EAAE,UAAU,EAAE,UAAU,CAAC,GAAG,CAAC,CAUzE;AAED,wBAAgB,KAAK,CAAC,gBAAgB,EAAE,UAAU,OAAO,IAWtC,QAAQ,SAAS,EAAE,SAAS,0BAA0B,CAAC,GAAG,CAAC,KAAG,IAAI,CAOpF"}

package/dist/decorators.js

@@ -1,4 +1,4 @@
-import { createFieldDecorator, } from "@praxisjs/decorators";
+import { addClaimableInitializer, assertFieldContext } from "@praxisjs/decorators";
 import { applyPluginsToStore } from "./apply-plugins.js";
 import { getGlobalPlugins } from "./plugin-registry.js";
 const storeRegistry = new Map();
@@ -15,27 +15,33 @@ export function Storable(options = {}) {
     };
 }
 export function store(StoreClass) {
-    if (!storeRegistry.has(StoreClass) || storeRegistry.get(StoreClass) === null) {
-        const instance = new StoreClass();
-        const perStore = storePluginsMap.get(StoreClass) ?? [];
-        const plugins = [...getGlobalPlugins(), ...perStore];
-        const wrapped = applyPluginsToStore(instance, StoreClass.name, plugins);
-        storeRegistry.set(StoreClass, wrapped);
-    }
-    return storeRegistry.get(StoreClass);
+    const existing = storeRegistry.get(StoreClass);
+    if (existing != null)
+        return existing;
+    const instance = new StoreClass();
+    const perStore = storePluginsMap.get(StoreClass) ?? [];
+    const plugins = [...getGlobalPlugins(), ...perStore];
+    const wrapped = applyPluginsToStore(instance, StoreClass.name, plugins);
+    storeRegistry.set(StoreClass, wrapped);
+    return wrapped;
 }
 export function Store(StoreConstructor) {
-    return createFieldDecorator({
-        bind(_instance, _name, _initialValue) {
-            return {
-                descriptor: {
-                    get() {
-                        return store(StoreConstructor);
-                    },
-                },
-            };
+    // A store is a singleton that is never replaced, so the first lookup is kept for every later read.
+    let cached;
+    const descriptor = {
+        get() {
+            return (cached ??= store(StoreConstructor));
         },
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    });
+        enumerable: true,
+        configurable: true,
+    };
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    return function (_value, context) {
+        assertFieldContext(context);
+        const name = context.name;
+        addClaimableInitializer(context, function () {
+            Object.defineProperty(this, name, descriptor);
+        });
+    };
 }
 //# sourceMappingURL=decorators.js.map

package/dist/decorators.js.map

@@ -1 +1 @@
-{"version":3,"file":"decorators.js","sourceRoot":"","sources":["../src/decorators.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,oBAAoB,GAGrB,MAAM,sBAAsB,CAAC;AAE9B,OAAO,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AACzD,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAIxD,MAAM,aAAa,GAAG,IAAI,GAAG,EAAgD,CAAC;AAC9E,MAAM,eAAe,GAAG,IAAI,OAAO,EAGhC,CAAC;AAEJ,qGAAqG;AACrG,MAAM,OAAO,aAAa;CAEzB;AAMD,MAAM,UAAU,QAAQ,CAAC,UAA2B,EAAE;IACpD,OAAO,UACL,WAAsD,EACtD,QAA+B;QAE/B,aAAa,CAAC,GAAG,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;QACrC,IAAI,OAAO,CAAC,OAAO,EAAE,MAAM,EAAE,CAAC;YAC5B,eAAe,CAAC,GAAG,CAAC,WAAW,EAAE,OAAO,CAAC,OAAO,CAAC,CAAC;QACpD,CAAC;IACH,CAAC,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,KAAK,CAA0B,UAAuB;IACpE,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,UAAU,CAAC,IAAI,aAAa,CAAC,GAAG,CAAC,UAAU,CAAC,KAAK,IAAI,EAAE,CAAC;QAC7E,MAAM,QAAQ,GAAG,IAAI,UAAU,EAAE,CAAC;QAClC,MAAM,QAAQ,GACZ,eAAe,CAAC,GAAG,CAAC,UAAuD,CAAC,IAAI,EAAE,CAAC;QACrF,MAAM,OAAO,GAAG,CAAC,GAAG,gBAAgB,EAAE,EAAE,GAAG,QAAQ,CAAC,CAAC;QACrD,MAAM,OAAO,GAAG,mBAAmB,CAAC,QAAQ,EAAE,UAAU,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;QACxE,aAAa,CAAC,GAAG,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;IACzC,CAAC;IACD,OAAO,aAAa,CAAC,GAAG,CAAC,UAAU,CAAM,CAAC;AAC5C,CAAC;AAED,MAAM,UAAU,KAAK,CAAC,gBAAmC;IACvD,OAAO,oBAAoB,CAAC;QAC1B,IAAI,CAAC,SAAS,EAAE,KAAK,EAAE,aAAa;YAClC,OAAO;gBACL,UAAU,EAAE;oBACV,GAAG;wBACD,OAAO,KAAK,CAAC,gBAA2C,CAAC,CAAC;oBAC5D,CAAC;iBACF;aACF,CAAC;QACJ,CAAC;QACH,8DAA8D;KAC7D,CAAqF,CAAC;AACzF,CAAC"}
+{"version":3,"file":"decorators.js","sourceRoot":"","sources":["../src/decorators.ts"],"names":[],"mappings":"AAAA,OAAO,EAAoB,uBAAuB,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC;AAErG,OAAO,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AACzD,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAIxD,MAAM,aAAa,GAAG,IAAI,GAAG,EAAgD,CAAC;AAC9E,MAAM,eAAe,GAAG,IAAI,OAAO,EAGhC,CAAC;AAEJ,qGAAqG;AACrG,MAAM,OAAO,aAAa;CAEzB;AAMD,MAAM,UAAU,QAAQ,CAAC,UAA2B,EAAE;IACpD,OAAO,UACL,WAAsD,EACtD,QAA+B;QAE/B,aAAa,CAAC,GAAG,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;QACrC,IAAI,OAAO,CAAC,OAAO,EAAE,MAAM,EAAE,CAAC;YAC5B,eAAe,CAAC,GAAG,CAAC,WAAW,EAAE,OAAO,CAAC,OAAO,CAAC,CAAC;QACpD,CAAC;IACH,CAAC,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,KAAK,CAA0B,UAAuB;IACpE,MAAM,QAAQ,GAAG,aAAa,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;IAC/C,IAAI,QAAQ,IAAI,IAAI;QAAE,OAAO,QAAa,CAAC;IAC3C,MAAM,QAAQ,GAAG,IAAI,UAAU,EAAE,CAAC;IAClC,MAAM,QAAQ,GACZ,eAAe,CAAC,GAAG,CAAC,UAAuD,CAAC,IAAI,EAAE,CAAC;IACrF,MAAM,OAAO,GAAG,CAAC,GAAG,gBAAgB,EAAE,EAAE,GAAG,QAAQ,CAAC,CAAC;IACrD,MAAM,OAAO,GAAG,mBAAmB,CAAC,QAAQ,EAAE,UAAU,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;IACxE,aAAa,CAAC,GAAG,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;IACvC,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,MAAM,UAAU,KAAK,CAAC,gBAAmC;IACvD,mGAAmG;IACnG,IAAI,MAAe,CAAC;IACpB,MAAM,UAAU,GAAuB;QACrC,GAAG;YACD,OAAO,CAAC,MAAM,KAAK,KAAK,CAAC,gBAA2C,CAAC,CAAC,CAAC;QACzE,CAAC;QACD,UAAU,EAAE,IAAI;QAChB,YAAY,EAAE,IAAI;KACnB,CAAC;IACF,8DAA8D;IAC9D,OAAO,UAAU,MAAiB,EAAE,OAAwC;QAC1E,kBAAkB,CAAC,OAAO,CAAC,CAAC;QAC5B,MAAM,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC;QAC1B,uBAAuB,CAAC,OAAO,EAAE;YAC/B,MAAM,CAAC,cAAc,CAAC,IAAI,EAAE,IAAI,EAAE,UAAU,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC;AACJ,CAAC"}

package/package.json

@@ -1,7 +1,8 @@
 {
   "name": "@praxisjs/store",
-  "version": "2.0.3",
+  "version": "2.0.5",
   "type": "module",
+  "sideEffects": false,
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "exports": {
@@ -10,15 +11,23 @@
       "types": "./dist/index.d.ts"
     }
   },
+  "files": [
+    "dist",
+    "src",
+    "!dist/**/__tests__",
+    "!src/**/__tests__",
+    "CHANGELOG.md"
+  ],
   "devDependencies": {
     "typescript": "^6.0.3"
   },
   "dependencies": {
-    "@praxisjs/core": "2.1.0",
-    "@praxisjs/decorators": "1.6.1"
+    "@praxisjs/decorators": "2.0.0",
+    "@praxisjs/core": "2.3.0"
   },
   "scripts": {
-    "build": "tsc",
-    "dev": "tsc --watch"
+    "build": "tsc -p tsconfig.build.json",
+    "dev": "tsc -p tsconfig.build.json --watch",
+    "typecheck": "tsc --noEmit"
   }
 }

package/src/apply-plugins.ts

@@ -1,5 +1,15 @@
+import { pauseTracking, resumeTracking } from "@praxisjs/core/internal";
+
 import type { StorePlugin } from "./plugin-types.js";
 
+type Action = (...args: unknown[]) => unknown;
+
+// `constructor` and the members every object inherits (`toString`, `hasOwnProperty`...) are not
+// store actions: wrapping them would report them to plugins and break `new store.constructor()`.
+function isBuiltin(prop: string, val: unknown): boolean {
+  return prop === "constructor" || (Object.prototype as unknown as Record<string, unknown>)[prop] === val;
+}
+
 export function applyPluginsToStore<T extends object>(
   instance: T,
   storeName: string,
@@ -7,45 +17,55 @@ export function applyPluginsToStore<T extends object>(
 ): T {
   if (plugins.length === 0) return instance;
 
-  const extensions: Record<string, unknown> = {};
+  const actionPlugins = plugins.filter((p) => "onAction" in p || "onActionDone" in p);
+  const mutationPlugins = plugins.filter((p) => "onMutation" in p);
+  // Extensions are stored ready to return (functions already bound), so a read does no work.
+  const extensions = new Map<string, unknown>();
+  // One wrapper per method, so `store.action` keeps its identity across reads; rebuilt only when
+  // the method itself is replaced.
+  const wrappers = new Map<string, { fn: Action; wrapped: Action }>();
+
+  function wrap(name: string, fn: Action): Action {
+    if (actionPlugins.length === 0) {
+      return (...args) => fn.apply(proxy, args);
+    }
+    return (...args) => {
+      for (const p of actionPlugins) p.onAction?.({ name, args, storeName });
+      const result = fn.apply(proxy, args);
+      if (result instanceof Promise) {
+        return result
+          .then((resolved: unknown) => {
+            for (const p of actionPlugins)
+              p.onActionDone?.({ name, args, storeName, result: resolved });
+            return resolved;
+          })
+          .catch((err: unknown) => {
+            for (const p of actionPlugins)
+              p.onActionDone?.({ name, args, storeName, result: undefined, error: err });
+            throw err;
+          });
+      }
+      for (const p of actionPlugins)
+        p.onActionDone?.({ name, args, storeName, result });
+      return result;
+    };
+  }
 
   const proxy: T = new Proxy(instance, {
     get(target, prop: string | symbol): unknown {
       if (typeof prop === "symbol") return Reflect.get(target, prop) as unknown;
 
-      if (prop in extensions) {
-        const ext = extensions[prop];
-        return typeof ext === "function"
-          ? (ext as (...a: unknown[]) => unknown).bind(proxy)
-          : ext;
-      }
+      if (extensions.size > 0 && extensions.has(prop)) return extensions.get(prop);
 
       const val = Reflect.get(target, prop, proxy) as unknown;
 
-      if (typeof val === "function" && !prop.startsWith("$")) {
-        return (...args: unknown[]): unknown => {
-          for (const p of plugins) p.onAction?.({ name: prop, args, storeName });
-          const result = (val as (...a: unknown[]) => unknown).apply(proxy, args);
-          if (result instanceof Promise) {
-            return result
-              .then((resolved: unknown) => {
-                for (const p of plugins)
-                  p.onActionDone?.({ name: prop, args, storeName, result: resolved });
-                return resolved;
-              })
-              .catch((err: unknown) => {
-                for (const p of plugins)
-                  p.onActionDone?.({ name: prop, args, storeName, result: undefined, error: err });
-                throw err;
-              });
-          }
-          for (const p of plugins)
-            p.onActionDone?.({ name: prop, args, storeName, result });
-          return result;
-        };
-      }
-
-      return val;
+      if (typeof val !== "function") return val;
+      const cached = wrappers.get(prop);
+      if (cached?.fn === val) return cached.wrapped;
+      if (prop.startsWith("$") || isBuiltin(prop, val)) return val;
+      const wrapped = wrap(prop, val as Action);
+      wrappers.set(prop, { fn: val as Action, wrapped });
+      return wrapped;
     },
 
     set(target, prop: string | symbol, value: unknown): boolean {
@@ -53,13 +73,21 @@ export function applyPluginsToStore<T extends object>(
         (target as Record<symbol, unknown>)[prop] = value;
         return true;
       }
-      const prevValue = Reflect.get(target, prop, proxy) as unknown;
-      const ok: boolean = Reflect.set(target, prop, value, proxy);
-      if (ok) {
-        for (const p of plugins)
-          p.onMutation?.({ key: prop, value, prevValue, storeName });
+      if (mutationPlugins.length === 0) return Reflect.set(target, prop, value, proxy);
+      // A write is not a read: assigning a field inside an effect must not subscribe the effect to
+      // that field (through the previous value) or to whatever the plugins read.
+      const prevSub = pauseTracking();
+      try {
+        const prevValue = Reflect.get(target, prop, proxy) as unknown;
+        const ok: boolean = Reflect.set(target, prop, value, proxy);
+        if (ok) {
+          for (const p of mutationPlugins)
+            p.onMutation?.({ key: prop, value, prevValue, storeName });
+        }
+        return ok;
+      } finally {
+        resumeTracking(prevSub);
       }
-      return ok;
     },
   });
 
@@ -68,7 +96,13 @@ export function applyPluginsToStore<T extends object>(
       store: proxy as unknown as Record<string, unknown>,
       storeName,
       extend(props) {
-        Object.assign(extensions, props);
+        for (const key of Object.keys(props)) {
+          const ext = props[key];
+          extensions.set(
+            key,
+            typeof ext === "function" ? (ext as Action).bind(proxy) : ext,
+          );
+        }
       },
     });
   }

package/src/decorators.ts

@@ -1,8 +1,4 @@
-import {
-  createFieldDecorator,
-  reactiveHostType,
-  type FieldBinding,
-} from "@praxisjs/decorators";
+import { reactiveHostType, addClaimableInitializer, assertFieldContext } from "@praxisjs/decorators";
 
 import { applyPluginsToStore } from "./apply-plugins.js";
 import { getGlobalPlugins } from "./plugin-registry.js";
@@ -37,28 +33,33 @@ export function Storable(options: StorableOptions = {}) {
 }
 
 export function store<T extends ReactiveStore>(StoreClass: new () => T): T {
-  if (!storeRegistry.has(StoreClass) || storeRegistry.get(StoreClass) === null) {
-    const instance = new StoreClass();
-    const perStore =
-      storePluginsMap.get(StoreClass as new (...args: unknown[]) => ReactiveStore) ?? [];
-    const plugins = [...getGlobalPlugins(), ...perStore];
-    const wrapped = applyPluginsToStore(instance, StoreClass.name, plugins);
-    storeRegistry.set(StoreClass, wrapped);
-  }
-  return storeRegistry.get(StoreClass) as T;
+  const existing = storeRegistry.get(StoreClass);
+  if (existing != null) return existing as T;
+  const instance = new StoreClass();
+  const perStore =
+    storePluginsMap.get(StoreClass as new (...args: unknown[]) => ReactiveStore) ?? [];
+  const plugins = [...getGlobalPlugins(), ...perStore];
+  const wrapped = applyPluginsToStore(instance, StoreClass.name, plugins);
+  storeRegistry.set(StoreClass, wrapped);
+  return wrapped;
 }
 
 export function Store(StoreConstructor: new () => unknown) {
-  return createFieldDecorator({
-    bind(_instance, _name, _initialValue): FieldBinding {
-      return {
-        descriptor: {
-          get(this: object): unknown {
-            return store(StoreConstructor as new () => ReactiveStore);
-          },
-        },
-      };
+  // A store is a singleton that is never replaced, so the first lookup is kept for every later read.
+  let cached: unknown;
+  const descriptor: PropertyDescriptor = {
+    get(): unknown {
+      return (cached ??= store(StoreConstructor as new () => ReactiveStore));
     },
+    enumerable: true,
+    configurable: true,
+  };
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  }) as unknown as (_value: undefined, context: ClassFieldDecoratorContext<any>) => void;
+  return function (_value: undefined, context: ClassFieldDecoratorContext<any>): void {
+    assertFieldContext(context);
+    const name = context.name;
+    addClaimableInitializer(context, function () {
+      Object.defineProperty(this, name, descriptor);
+    });
+  };
 }

package/dist/__tests__/create-store.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=create-store.test.d.ts.map

package/dist/__tests__/create-store.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"create-store.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/create-store.test.ts"],"names":[],"mappings":""}