@ronin/compiler 0.14.12 → 0.14.13-leo-ron-1099-1-experimental-354

package/dist/index.d.ts

@@ -95,7 +95,7 @@ type OrderedByInstruction = {
     ascending?: Array<string | Expression>;
     descending?: Array<string | Expression>;
 };
-type ForInstruction = Array<string> | Record<string, string>;
+type UsingInstruction = Array<string> | Record<string, string>;
 type CombinedInstructions = {
     with?: WithInstruction | Array<WithInstruction>;
     to?: FieldSelector;
@@ -105,14 +105,14 @@ type CombinedInstructions = {
     before?: string | null;
     after?: string | null;
     limitedTo?: number;
-    for?: ForInstruction;
+    using?: UsingInstruction;
 };
-type InstructionSchema = 'with' | 'to' | 'including' | 'selecting' | 'orderedBy' | 'orderedBy.ascending' | 'orderedBy.descending' | 'before' | 'after' | 'limitedTo' | 'for';
+type InstructionSchema = 'with' | 'to' | 'including' | 'selecting' | 'orderedBy' | 'orderedBy.ascending' | 'orderedBy.descending' | 'before' | 'after' | 'limitedTo' | 'using';
 type GetQuery = Record<string, Omit<CombinedInstructions, 'to'> | null>;
 type SetQuery = Record<string, Omit<CombinedInstructions, 'to'> & {
     to: FieldSelector;
 }>;
-type AddQuery = Record<string, Omit<CombinedInstructions, 'with' | 'for'> & {
+type AddQuery = Record<string, Omit<CombinedInstructions, 'with' | 'using'> & {
     to: FieldSelector;
 }>;
 type RemoveQuery = Record<string, Omit<CombinedInstructions, 'to'>>;
@@ -121,7 +121,7 @@ type GetInstructions = Omit<CombinedInstructions, 'to'>;
 type SetInstructions = Omit<CombinedInstructions, 'to'> & {
     to: FieldSelector;
 };
-type AddInstructions = Omit<CombinedInstructions, 'with' | 'for'> & {
+type AddInstructions = Omit<CombinedInstructions, 'with' | 'using'> & {
     to: FieldSelector;
 };
 type RemoveInstructions = Omit<CombinedInstructions, 'to'>;

package/dist/index.js

@@ -257,8 +257,8 @@ var handleBeforeOrAfter = (model, statementParams, instructions) => {
 };
 
 // src/instructions/for.ts
-var handleFor = (model, instructions) => {
-  const normalizedFor = Array.isArray(instructions.for) ? Object.fromEntries(instructions.for.map((presetSlug) => [presetSlug, null])) : instructions.for;
+var handleUsing = (model, instructions) => {
+  const normalizedFor = Array.isArray(instructions.using) ? Object.fromEntries(instructions.using.map((presetSlug) => [presetSlug, null])) : instructions.using;
   for (const presetSlug in normalizedFor) {
     if (!Object.hasOwn(normalizedFor, presetSlug)) continue;
     const arg = normalizedFor[presetSlug];
@@ -269,28 +269,28 @@ var handleFor = (model, instructions) => {
         code: "PRESET_NOT_FOUND"
       });
     }
-    const replacedForFilter = structuredClone(preset.instructions);
+    const replacedUsingFilter = structuredClone(preset.instructions);
     if (arg !== null) {
       findInObject(
-        replacedForFilter,
+        replacedUsingFilter,
         QUERY_SYMBOLS.VALUE,
         (match) => match.replace(QUERY_SYMBOLS.VALUE, arg)
       );
     }
-    for (const subInstruction in replacedForFilter) {
-      if (!Object.hasOwn(replacedForFilter, subInstruction)) continue;
+    for (const subInstruction in replacedUsingFilter) {
+      if (!Object.hasOwn(replacedUsingFilter, subInstruction)) continue;
       const instructionName = subInstruction;
       const currentValue = instructions[instructionName];
       if (currentValue) {
         let newValue;
         if (Array.isArray(currentValue)) {
           newValue = [
-            ...replacedForFilter[instructionName],
+            ...replacedUsingFilter[instructionName],
             ...currentValue
           ];
         } else if (isObject(currentValue)) {
           newValue = {
-            ...replacedForFilter[instructionName],
+            ...replacedUsingFilter[instructionName],
             ...currentValue
           };
         }
@@ -298,7 +298,7 @@ var handleFor = (model, instructions) => {
         continue;
       }
       Object.assign(instructions, {
-        [instructionName]: replacedForFilter[instructionName]
+        [instructionName]: replacedUsingFilter[instructionName]
       });
     }
   }
@@ -615,8 +615,8 @@ var compileQueryInput = (defaultQuery, models, statementParams, options) => {
   const single = queryModel !== model.pluralSlug;
   let instructions = formatIdentifiers(model, queryInstructions);
   const returning = options?.returning ?? true;
-  if (instructions && Object.hasOwn(instructions, "for")) {
-    instructions = handleFor(model, instructions);
+  if (instructions && typeof instructions.using !== "undefined") {
+    instructions = handleUsing(model, instructions);
   }
   if (queryType === "count") {
     if (!instructions) instructions = {};
@@ -1247,9 +1247,12 @@ var addDefaultModelAttributes = (model, isNew) => {
 };
 var addDefaultModelFields = (model, isNew) => {
   const copiedModel = { ...model };
-  const newFields = copiedModel.fields || [];
-  if (isNew || newFields.length > 0) {
-    copiedModel.fields = [...getSystemFields(copiedModel.idPrefix), ...newFields];
+  const existingFields = copiedModel.fields || [];
+  if (isNew || existingFields.length > 0) {
+    const additionalFields = getSystemFields(copiedModel.idPrefix).filter((newField) => {
+      return !existingFields.some(({ slug }) => slug === newField.slug);
+    });
+    copiedModel.fields = [...additionalFields, ...existingFields];
   }
   return copiedModel;
 };
@@ -1361,8 +1364,12 @@ var addDefaultModelPresets = (list, model) => {
       slug: presetSlug
     });
   }
-  if (Object.keys(defaultPresets).length > 0) {
-    model.presets = [...defaultPresets, ...model.presets || []];
+  if (defaultPresets.length > 0) {
+    const existingPresets = model.presets || [];
+    const additionalPresets = defaultPresets.filter((newPreset) => {
+      return !existingPresets.some(({ slug }) => slug === newPreset.slug);
+    });
+    model.presets = [...additionalPresets, ...existingPresets];
   }
   return model;
 };

package/package.json

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