@ronin/compiler 0.17.19-corny-pagination-bug-experimental-416 → 0.17.19-leo-ron-1119-experimental-417

package/dist/index.d.ts

@@ -258,8 +258,6 @@ type InternalModelField = ModelField & {
     mountingPath: string;
     /** A custom value that was provided in the query, which is not stored in the DB. */
     mountedValue?: unknown;
-    /** Indicator if the field should be excluded from the final records. */
-    excluded?: boolean;
 };
 type ModelIndexField<T extends ModelEntityList<ModelField> = ModelEntityList<ModelField>> = {
     /** The collating sequence used for text placed inside the field. */

package/dist/index.js

@@ -379,7 +379,7 @@ var handleOrderedBy = (model, instruction) => {
 };
 
 // src/instructions/selecting.ts
-var handleSelecting = (models, model, statementParams, single, instructions, queryType, options = { inlineDefaults: false }) => {
+var handleSelecting = (models, model, statementParams, single, instructions, options = { inlineDefaults: false }) => {
   let isJoining = false;
   const selectedFields = filterSelectedFields(
     model,
@@ -403,9 +403,9 @@ var handleSelecting = (models, model, statementParams, single, instructions, que
     for (const [key, value] of Object.entries(flatObject)) {
       const symbol2 = getQuerySymbol(value);
       if (symbol2?.type === "query") {
-        const { queryType: queryType2, queryModel, queryInstructions } = splitQuery(symbol2.value);
+        const { queryType, queryModel, queryInstructions } = splitQuery(symbol2.value);
         const subQueryModel = getModelBySlug(models, queryModel);
-        if (queryType2 === "count") {
+        if (queryType === "count") {
           const subSelect = compileQueryInput(symbol2.value, models, statementParams, {
             parentModel: { ...model, tableAlias: model.table },
             inlineDefaults: options.inlineDefaults
@@ -430,10 +430,8 @@ var handleSelecting = (models, model, statementParams, single, instructions, que
           subSingle,
           {
             selecting: queryInstructions?.selecting,
-            including: queryInstructions?.including,
-            limitedTo: instructions.limitedTo
+            including: queryInstructions?.including
           },
-          queryType2,
           { ...options, mountingPath: subMountingPath }
         );
         if (nestedColumns !== "*") joinedColumns.push(nestedColumns);
@@ -456,14 +454,6 @@ var handleSelecting = (models, model, statementParams, single, instructions, que
       });
     }
   }
-  if (queryType === "get" && !single && !selectedFields.some((field) => field.slug === "ronin.createdAt") && instructions.limitedTo) {
-    selectedFields.push({
-      ...getSystemFields(model.idPrefix)["ronin.createdAt"],
-      slug: "ronin.createdAt",
-      excluded: true,
-      mountingPath: "ronin.createdAt"
-    });
-  }
   const columns = selectedFields.map((selectedField) => {
     if (selectedField.mountedValue) {
       return `${selectedField.mountedValue} as "${selectedField.slug}"`;
@@ -1087,18 +1077,6 @@ var compileQueryInput = (defaultQuery, models, statementParams, options) => {
       }
     });
   }
-  if (!single && (queryType === "get" && instructions?.limitedTo || queryType === "count" && (instructions?.before || instructions?.after))) {
-    instructions = instructions || {};
-    instructions.orderedBy = instructions.orderedBy || {};
-    instructions.orderedBy.ascending = instructions.orderedBy.ascending || [];
-    instructions.orderedBy.descending = instructions.orderedBy.descending || [];
-    if (![
-      ...instructions.orderedBy.ascending,
-      ...instructions.orderedBy.descending
-    ].includes("ronin.createdAt")) {
-      instructions.orderedBy.descending.push("ronin.createdAt");
-    }
-  }
   const { columns, isJoining, selectedFields } = handleSelecting(
     models,
     model,
@@ -1106,10 +1084,8 @@ var compileQueryInput = (defaultQuery, models, statementParams, options) => {
     single,
     {
       selecting: instructions?.selecting,
-      including: instructions?.including,
-      limitedTo: instructions?.limitedTo
+      including: instructions?.including
     },
-    queryType,
     // biome-ignore lint/complexity/useSimplifiedLogicExpression: This is needed.
     { inlineDefaults: options?.inlineDefaults || false }
   );
@@ -1180,6 +1156,18 @@ var compileQueryInput = (defaultQuery, models, statementParams, options) => {
     );
     if (withStatement.length > 0) conditions.push(withStatement);
   }
+  if (!single && (queryType === "get" && instructions?.limitedTo || queryType === "count" && (instructions?.before || instructions?.after))) {
+    instructions = instructions || {};
+    instructions.orderedBy = instructions.orderedBy || {};
+    instructions.orderedBy.ascending = instructions.orderedBy.ascending || [];
+    instructions.orderedBy.descending = instructions.orderedBy.descending || [];
+    if (![
+      ...instructions.orderedBy.ascending,
+      ...instructions.orderedBy.descending
+    ].includes("ronin.createdAt")) {
+      instructions.orderedBy.descending.push("ronin.createdAt");
+    }
+  }
   if (instructions && (typeof instructions.before !== "undefined" || typeof instructions.after !== "undefined")) {
     if (single) {
       throw new RoninError({
@@ -2054,7 +2042,7 @@ var transformMetaQuery = (models, dependencyStatements, statementParams, query,
       const value = prepareStatementValue(statementParams, entityValue);
       json = `json_insert(${field}, '$.${slug2}', json(${value}))`;
       if (!existingModel[pluralType]) existingModel[pluralType] = {};
-      existingModel[pluralType][slug2] = jsonValue;
+      existingModel[pluralType][slug2] = entityValue;
       break;
     }
     case "alter": {
@@ -2320,38 +2308,6 @@ var Transaction = class {
         );
       }
     }
-    const fieldsToDrop = selectedFields.filter((field) => field.excluded === true);
-    const deleteNestedProperty = (obj, path) => {
-      const parts = path.split(".");
-      const lastPart = parts.pop();
-      let current = obj;
-      for (const part of parts) {
-        if (!current || typeof current !== "object") return;
-        const currentAsRecord = current;
-        if (!(part in currentAsRecord)) return;
-        current = currentAsRecord[part];
-      }
-      if (typeof current === "object" && current !== null) {
-        delete current[lastPart];
-      }
-      if (parts.length > 0 && typeof current === "object" && current !== null && Object.keys(current).length === 0) {
-        let temp = obj;
-        for (let i = 0; i < parts.length - 1; i++) {
-          temp = temp[parts[i]];
-        }
-        const lastPart2 = parts.at(-1);
-        if (lastPart2) {
-          delete temp[lastPart2];
-        }
-      }
-    };
-    if (fieldsToDrop.length > 0) {
-      for (const record of result.records) {
-        for (const field of fieldsToDrop) {
-          deleteNestedProperty(record, field.slug);
-        }
-      }
-    }
     return result;
   }
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ronin/compiler",
-  "version": "0.17.19-corny-pagination-bug-experimental-416",
+  "version": "0.17.19-leo-ron-1119-experimental-417",
   "type": "module",
   "description": "Compiles RONIN queries to SQL statements.",
   "publishConfig": {