@esportsplus/reactivity 0.38.0 → 0.38.1

package/build/compiler/array.js

@@ -79,7 +79,9 @@ function length(access, replacements) {
                 : `${expr.getText(sf)}.$length = ${expr.getText(sf)}.length ${op} ${parent.right.getText(sf)}`
         });
     }
-    else if (parent && (ts.isPostfixUnaryExpression(parent) || ts.isPrefixUnaryExpression(parent))) {
+    else if (parent &&
+        (ts.isPostfixUnaryExpression(parent) || ts.isPrefixUnaryExpression(parent)) &&
+        (parent.operator === ts.SyntaxKind.PlusPlusToken || parent.operator === ts.SyntaxKind.MinusMinusToken)) {
         let op = parent.operator === ts.SyntaxKind.PlusPlusToken ? '+' : '-';
         replacements.push({
             node: parent,

package/package.json

@@ -36,7 +36,7 @@
   },
   "type": "module",
   "types": "build/index.d.ts",
-  "version": "0.38.0",
+  "version": "0.38.1",
   "scripts": {
     "agent:bench": "vitest bench --run",
     "agent:test": "tsc --noEmit && vitest run",

package/src/compiler/array.ts

@@ -133,7 +133,11 @@ function length(access: ts.PropertyAccessExpression, replacements: ReplacementIn
         });
     }
     // arr.length++ / arr.length-- / ++arr.length / --arr.length
-    else if (parent && (ts.isPostfixUnaryExpression(parent) || ts.isPrefixUnaryExpression(parent))) {
+    else if (
+        parent &&
+        (ts.isPostfixUnaryExpression(parent) || ts.isPrefixUnaryExpression(parent)) &&
+        (parent.operator === ts.SyntaxKind.PlusPlusToken || parent.operator === ts.SyntaxKind.MinusMinusToken)
+    ) {
         let op = parent.operator === ts.SyntaxKind.PlusPlusToken ? '+' : '-';
 
         replacements.push({

package/test/compiler/compiler.test.ts

@@ -347,6 +347,20 @@ describe('array transform', () => {
         expect(output).toContain('arr.$length = arr.length + 3');
     });
 
+    it('transforms arr.length++ and --arr.length to $length writes', () => {
+        expect(transformArray('let arr = reactive([1]); arr.length++;').output).toContain('arr.$length = arr.length + 1');
+        expect(transformArray('let arr = reactive([1]); --arr.length;').output).toContain('arr.$length = arr.length - 1');
+    });
+
+    it('transforms non-mutating unary operators on arr.length as reads', () => {
+        for (let op of ['!', '-', '+', '~']) {
+            let { output } = transformArray(`let arr = reactive([1]); let x = ${op}arr.length;`);
+
+            expect(output).toContain(`${op}arr.$length`);
+            expect(output).not.toContain('arr.$length =');
+        }
+    });
+
     it('transforms every compound operator on arr.length with its own token', () => {
         for (let op of ['<<', '>>', '>>>', '??', '||', '&&', '-', '*', '/', '%', '**', '&', '|', '^']) {
             let { output } = transformArray(`let arr = reactive([1]); arr.length ${op}= 2;`);