@valbuild/server 0.62.3 → 0.62.4

package/dist/valbuild-server.cjs.dev.js

@@ -1516,6 +1516,7 @@ class ValOps {
 
   // #region getTree
   async getTree(analysis) {
+    console.log("analysis", analysis);
     if (!analysis) {
       const {
         sources
@@ -1541,7 +1542,7 @@ class ValOps {
           error: new patch.PatchError(`Module at path: '${path}' not found`)
         });
       }
-      const source = sources[path];
+      patchedSources[path] = sources[path];
       for (const {
         patchId
       } of patches) {
@@ -1576,7 +1577,9 @@ class ValOps {
               applicableOps.push(op);
             }
           }
-          const patchRes = patch.applyPatch(source, jsonOps, applicableOps.concat(...Object.values(fileFixOps)));
+          const patchRes = patch.applyPatch(patch.deepClone(patchedSources[path]),
+          // applyPatch mutates the source. On add operations it will add multiple items? There is something strange going on. DeepClone seems to fix, but is that the right?
+          jsonOps, applicableOps.concat(...Object.values(fileFixOps)));
           if (fp.result.isErr(patchRes)) {
             if (!errors[path]) {
               errors[path] = [];

package/dist/valbuild-server.cjs.prod.js

@@ -1516,6 +1516,7 @@ class ValOps {
 
   // #region getTree
   async getTree(analysis) {
+    console.log("analysis", analysis);
     if (!analysis) {
       const {
         sources
@@ -1541,7 +1542,7 @@ class ValOps {
           error: new patch.PatchError(`Module at path: '${path}' not found`)
         });
       }
-      const source = sources[path];
+      patchedSources[path] = sources[path];
       for (const {
         patchId
       } of patches) {
@@ -1576,7 +1577,9 @@ class ValOps {
               applicableOps.push(op);
             }
           }
-          const patchRes = patch.applyPatch(source, jsonOps, applicableOps.concat(...Object.values(fileFixOps)));
+          const patchRes = patch.applyPatch(patch.deepClone(patchedSources[path]),
+          // applyPatch mutates the source. On add operations it will add multiple items? There is something strange going on. DeepClone seems to fix, but is that the right?
+          jsonOps, applicableOps.concat(...Object.values(fileFixOps)));
           if (fp.result.isErr(patchRes)) {
             if (!errors[path]) {
               errors[path] = [];

package/dist/valbuild-server.esm.js

@@ -2,7 +2,7 @@ import { newQuickJSWASMModule } from 'quickjs-emscripten';
 import ts from 'typescript';
 import { result, pipe } from '@valbuild/core/fp';
 import { FILE_REF_PROP, FILE_REF_SUBTYPE_TAG, RT_IMAGE_TAG, VAL_EXTENSION, derefPatch, Internal, Schema, ImageSchema, FileSchema } from '@valbuild/core';
-import { deepEqual, isNotRoot, PatchError, parseAndValidateArrayIndex, applyPatch, JSONOps, sourceToPatchPath } from '@valbuild/core/patch';
+import { deepEqual, isNotRoot, PatchError, parseAndValidateArrayIndex, applyPatch, JSONOps, deepClone, sourceToPatchPath } from '@valbuild/core/patch';
 import * as fsPath from 'path';
 import fsPath__default from 'path';
 import fs, { promises } from 'fs';
@@ -1486,6 +1486,7 @@ class ValOps {
 
   // #region getTree
   async getTree(analysis) {
+    console.log("analysis", analysis);
     if (!analysis) {
       const {
         sources
@@ -1511,7 +1512,7 @@ class ValOps {
           error: new PatchError(`Module at path: '${path}' not found`)
         });
       }
-      const source = sources[path];
+      patchedSources[path] = sources[path];
       for (const {
         patchId
       } of patches) {
@@ -1546,7 +1547,9 @@ class ValOps {
               applicableOps.push(op);
             }
           }
-          const patchRes = applyPatch(source, jsonOps, applicableOps.concat(...Object.values(fileFixOps)));
+          const patchRes = applyPatch(deepClone(patchedSources[path]),
+          // applyPatch mutates the source. On add operations it will add multiple items? There is something strange going on. DeepClone seems to fix, but is that the right?
+          jsonOps, applicableOps.concat(...Object.values(fileFixOps)));
           if (result.isErr(patchRes)) {
             if (!errors[path]) {
               errors[path] = [];

package/package.json

@@ -12,7 +12,7 @@
     "./package.json": "./package.json"
   },
   "types": "dist/valbuild-server.cjs.d.ts",
-  "version": "0.62.3",
+  "version": "0.62.4",
   "scripts": {
     "typecheck": "tsc --noEmit",
     "test": "jest",
@@ -22,9 +22,9 @@
     "@types/jest": "^29.2.5"
   },
   "dependencies": {
-    "@valbuild/core": "~0.62.3",
-    "@valbuild/shared": "~0.62.3",
-    "@valbuild/ui": "~0.62.3",
+    "@valbuild/core": "~0.62.4",
+    "@valbuild/shared": "~0.62.4",
+    "@valbuild/ui": "~0.62.4",
     "image-size": "^1.0.2",
     "minimatch": "^3.0.4",
     "quickjs-emscripten": "^0.21.1",