@esportsplus/reactivity 0.31.1 → 0.31.2

package/build/reactive/array.js

@@ -28,8 +28,9 @@ class ReactiveArray extends Array {
         if (prev === value) {
             return;
         }
+        let length = this.length;
         this[i] = value;
-        if (i >= this.length) {
+        if (i >= length) {
             write(this._length, i + 1);
         }
         this.dispatch('set', { index: i, item: value });

package/package.json

@@ -1,13 +1,13 @@
 {
   "author": "ICJR",
   "dependencies": {
-    "@esportsplus/utilities": "^0.27.2"
+    "@esportsplus/utilities": "^0.27.3"
   },
   "devDependencies": {
-    "@esportsplus/typescript": "^0.29.0",
-    "@types/node": "^25.0.9",
-    "vite": "^7.3.1",
-    "vitest": "^4.0.18"
+    "@esportsplus/typescript": "^0.29.1",
+    "@types/node": "^25.6.0",
+    "vite": "^8.0.8",
+    "vitest": "^4.1.4"
   },
   "exports": {
     ".": {
@@ -36,7 +36,7 @@
   },
   "type": "module",
   "types": "build/index.d.ts",
-  "version": "0.31.1",
+  "version": "0.31.2",
   "scripts": {
     "build": "tsc",
     "build:test": "pnpm build && vite build --config test/vite.config.ts",

package/src/reactive/array.ts

@@ -84,9 +84,11 @@ class ReactiveArray<T> extends Array<T> {
             return;
         }
 
+        let length = this.length;
+
         this[i] = value;
 
-        if (i >= this.length) {
+        if (i >= length) {
             write(this._length, i + 1);
         }
 

package/tests/array.ts

@@ -91,14 +91,12 @@ describe('ReactiveArray', () => {
             arr.$set(5, 99);
             await Promise.resolve();
 
-            // Native .length is 6, but reactive _length check runs after
-            // this[i] = value so i >= this.length is false — _length not updated
             expect(arr.length).toBe(6);
             expect(arr[5]).toBe(99);
-            expect(lengths).toEqual([3]);
+            expect(lengths).toEqual([3, 6]);
         });
 
-        it('$set creates sparse array without updating reactive $length', async () => {
+        it('$set creates sparse array and updates reactive $length', async () => {
             let arr = new ReactiveArray<number>(),
                 lengths: number[] = [];
 
@@ -117,15 +115,41 @@ describe('ReactiveArray', () => {
             // Native length becomes 101 via Array behavior
             expect(arr.length).toBe(101);
 
-            // Reactive $length NOT updated: this[100] = value sets native .length
-            // to 101 before the check, so 100 >= 101 is false
-            expect(lengths).toEqual([0]);
+            // Reactive $length updated correctly
+            expect(lengths).toEqual([0, 101]);
 
             // Intermediate indices are empty (sparse)
             expect(arr[0]).toBe(undefined);
             expect(arr[50]).toBe(undefined);
             expect(arr[99]).toBe(undefined);
         });
+
+        it('$set with negative index does not affect length', () => {
+            let arr = new ReactiveArray(1, 2, 3);
+
+            arr.$set(-1 as any, 42);
+
+            expect(arr.length).toBe(3);
+            expect(arr.$length).toBe(3);
+            expect((arr as any)[-1]).toBe(42);
+        });
+
+        it('$set at large index updates $length', async () => {
+            let arr = new ReactiveArray(1, 2, 3),
+                lengths: number[] = [];
+
+            effect(() => {
+                lengths.push(arr.$length);
+            });
+
+            expect(lengths).toEqual([3]);
+
+            arr.$set(10000, 99);
+            await Promise.resolve();
+
+            expect(arr.length).toBe(10001);
+            expect(lengths).toEqual([3, 10001]);
+        });
     });
 
 

package/tests/tsconfig.json

@@ -1,11 +1,10 @@
 {
     "extends": "../tsconfig.json",
     "compilerOptions": {
-        "baseUrl": "..",
         "noEmit": true,
         "paths": {
             "~/*": [
-                "src/*"
+                "../src/*"
             ]
         },
         "rootDir": ".."