@ronin/compiler 0.14.2-leo-ron-1099-1-experimental-321 → 0.14.3-leo-ron-1099-1-experimental-322

package/dist/index.js

@@ -26,6 +26,10 @@ var CURRENT_TIME_EXPRESSION = {
 var composeIncludedTableAlias = (fieldSlug) => {
   return `including_${fieldSlug}`;
 };
+var MOUNTING_PATH_SUFFIX = /(.*?)(\{(\d+)\})?$/;
+var composeMountingPath = (input) => {
+  return input.replace(MOUNTING_PATH_SUFFIX, (_, p, __, n) => `${p}{${n ? +n + 1 : 1}}`);
+};
 var MODEL_ENTITY_ERROR_CODES = {
   field: "FIELD_NOT_FOUND",
   index: "INDEX_NOT_FOUND",
@@ -299,7 +303,7 @@ var handleFor = (model, instructions) => {
 };
 
 // src/instructions/including.ts
-var handleIncluding = (models, model, statementParams, single, instruction) => {
+var handleIncluding = (models, model, statementParams, single, instruction, options = {}) => {
   let statement = "";
   let tableSubQuery;
   for (const ephemeralFieldSlug in instruction) {
@@ -311,8 +315,9 @@ var handleIncluding = (models, model, statementParams, single, instruction) => {
     const relatedModel = getModelBySlug(models, queryModel);
     let joinType = "LEFT";
     let relatedTableSelector = `"${relatedModel.table}"`;
-    const tableAlias = composeIncludedTableAlias(ephemeralFieldSlug);
     const subSingle = queryModel !== relatedModel.pluralSlug;
+    const subMountingPath = ephemeralFieldSlug === "ronin_root" ? options.mountingPath ? composeMountingPath(options.mountingPath) : void 0 : `${options?.mountingPath ? `${options?.mountingPath}.` : ""}${subSingle ? ephemeralFieldSlug : `${ephemeralFieldSlug}[0]`}`;
+    const tableAlias = composeIncludedTableAlias(subMountingPath || ephemeralFieldSlug);
     if (!modifiableQueryInstructions?.with) {
       joinType = "CROSS";
       if (subSingle) {
@@ -332,7 +337,7 @@ var handleIncluding = (models, model, statementParams, single, instruction) => {
       );
       relatedTableSelector = `(${subSelect.main.statement})`;
     }
-    statement += `${joinType} JOIN ${relatedTableSelector} as ${tableAlias}`;
+    statement += `${joinType} JOIN ${relatedTableSelector} as "${tableAlias}"`;
     model.tableAlias = model.tableAlias || model.table;
     if (joinType === "LEFT") {
       const subStatement = composeConditions(
@@ -356,7 +361,8 @@ var handleIncluding = (models, model, statementParams, single, instruction) => {
         { ...relatedModel, tableAlias },
         statementParams,
         subSingle,
-        modifiableQueryInstructions.including
+        modifiableQueryInstructions.including,
+        { mountingPath: subMountingPath }
       );
       statement += ` ${subIncluding.statement}`;
     }
@@ -409,7 +415,7 @@ var handleSelecting = (models, model, statementParams, single, instructions, opt
   ).filter((field) => !(field.type === "link" && field.kind === "many")).map((field) => {
     const newField = { ...field, mountingPath: field.slug };
     if (options.mountingPath) {
-      newField.mountingPath = `${options.mountingPath}.${field.slug}`;
+      newField.mountingPath = `${options.mountingPath.replace(/\{\d+\}/g, "")}.${field.slug}`;
     }
     return newField;
   });
@@ -430,11 +436,11 @@ var handleSelecting = (models, model, statementParams, single, instructions, opt
         const subQueryModel = getModelBySlug(models, queryModel);
         isJoining = true;
         if (queryInstructions?.selecting) options.expandColumns = true;
-        const tableAlias = composeIncludedTableAlias(key);
         const subSingle = queryModel !== subQueryModel.pluralSlug;
         if (!model.tableAlias)
           model.tableAlias = single && !subSingle ? `sub_${model.table}` : model.table;
-        const subMountingPath = key === "ronin_root" ? options.mountingPath : `${options?.mountingPath ? `${options?.mountingPath}.` : ""}${subSingle ? key : `${key}[0]`}`;
+        const subMountingPath = key === "ronin_root" ? options.mountingPath ? composeMountingPath(options.mountingPath) : void 0 : `${options?.mountingPath ? `${options?.mountingPath}.` : ""}${subSingle ? key : `${key}[0]`}`;
+        const tableAlias = composeIncludedTableAlias(subMountingPath || key);
         const { columns: nestedColumns, selectedFields: nestedSelectedFields } = handleSelecting(
           models,
           { ...subQueryModel, tableAlias },
@@ -2022,7 +2028,18 @@ var Transaction = class {
       for (const arrayField of joinFields.values()) {
         const currentValue = existingRecord[arrayField];
         const newValue = record[arrayField];
-        currentValue.push(...newValue);
+        for (const newRecord of newValue) {
+          if ("id" in newRecord) {
+            const existingIndex = currentValue.findIndex((value) => {
+              return value.id === newRecord.id;
+            });
+            if (existingIndex > -1) {
+              Object.assign(currentValue[existingIndex], newRecord);
+              continue;
+            }
+          }
+          currentValue.push(newRecord);
+        }
       }
     }
     return single ? records[0] : records;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ronin/compiler",
-  "version": "0.14.2-leo-ron-1099-1-experimental-321",
+  "version": "0.14.3-leo-ron-1099-1-experimental-322",
   "type": "module",
   "description": "Compiles RONIN queries to SQL statements.",
   "publishConfig": {