EXPERIMENT: convert back to object syntax if no explicit columns were defined

Author: NarrowsProjects

lib/database/utilities/QueryBuilder.js

@@ -626,23 +626,12 @@ class QueryBuilder {
      */
     toImplementation(model) {
         const options = { ...this.options };
-        const hasColumns = options.attributes.some((attribute) => typeof attribute === 'string');
+        const hasExplicitColumns = options.attributes.some((attribute) => typeof attribute === 'string');
 
         if (!this._selectOnly) {
-            if (!hasColumns) {
-                this.setModel(model);
-                const attributes = Object.keys(this.model.rawAttributes);
-                options.attributes = options.attributes.concat(attributes);
-            }
-        }
-
-        const primaryKey = model.primaryKeyAttribute;
-
-        if (primaryKey) {
-            const alreadyOrdered = options.order?.some(([column]) => column === primaryKey);
-
-            if (!alreadyOrdered) {
-                this.orderBy(primaryKey, 'ASC');
+            if (!hasExplicitColumns) {
+                // Convert to object syntax — Sequelize will include all model columns by default
+                options.attributes = { include: options.attributes };
             }
         }