@graffy/pg 0.16.3-alpha.4 → 0.16.3-alpha.5

package/index.cjs

@@ -442,12 +442,7 @@ function getArgSql({ $first, $last, $after, $before, $since, $until, $all, $curs
   if (!common.isEmpty(filter))
     where.push(getSql(filter, options));
   if (!hasRangeArg)
-    return {
-      meta: meta(baseKey),
-      where,
-      limit: 1,
-      ensureSingleRow: $group === true
-    };
+    return { meta: meta(baseKey), where, limit: 1 };
   const groupCols = Array.isArray($group) && $group.length && $group.map((prop) => lookup(prop, options));
   const group = groupCols ? join(groupCols, ", ") : void 0;
   const orderCols = ($order || [idCol]).map(
@@ -474,8 +469,7 @@ function getArgSql({ $first, $last, $after, $before, $since, $until, $all, $curs
     where,
     order,
     group,
-    limit: $first || $last,
-    ensureSingleRow: true
+    limit: $first || $last
   };
 }
 function getBoundCond(orderCols, bound, kind) {
@@ -509,23 +503,9 @@ function getBoundCond(orderCols, bound, kind) {
 }
 const MAX_LIMIT = 4096;
 function selectByArgs(args, projection, options) {
-  const { table, idCol } = options;
-  const { where, order, group, limit, meta, ensureSingleRow } = getArgSql(
-    args,
-    options
-  );
+  const { table } = options;
+  const { where, order, group, limit, meta } = getArgSql(args, options);
   const clampedLimit = Math.min(MAX_LIMIT, limit || MAX_LIMIT);
-  if (!ensureSingleRow) {
-    return sql`
-      SELECT
-      ${getSelectCols(options, projection)}, ${meta}
-      FROM "${raw(table)}" WHERE "${raw(idCol)}" = (
-        SELECT "${raw(idCol)}" FROM "${raw(table)}"
-        ${where.length ? sql`WHERE ${join(where, " AND ")}` : empty}
-        LIMIT 2
-      )
-    `;
-  }
   return sql`
     SELECT
     ${getSelectCols(options, projection)}, ${meta}
@@ -561,7 +541,7 @@ function getSingleSql(arg, options) {
         SELECT "${raw(idCol)}"
         FROM "${raw(table)}"
         WHERE ${join(where, " AND ")}
-        LIMIT 2
+        LIMIT 1
       )`,
     meta
   };

package/index.mjs

@@ -440,12 +440,7 @@ function getArgSql({ $first, $last, $after, $before, $since, $until, $all, $curs
   if (!isEmpty(filter))
     where.push(getSql(filter, options));
   if (!hasRangeArg)
-    return {
-      meta: meta(baseKey),
-      where,
-      limit: 1,
-      ensureSingleRow: $group === true
-    };
+    return { meta: meta(baseKey), where, limit: 1 };
   const groupCols = Array.isArray($group) && $group.length && $group.map((prop) => lookup(prop, options));
   const group = groupCols ? join(groupCols, ", ") : void 0;
   const orderCols = ($order || [idCol]).map(
@@ -472,8 +467,7 @@ function getArgSql({ $first, $last, $after, $before, $since, $until, $all, $curs
     where,
     order,
     group,
-    limit: $first || $last,
-    ensureSingleRow: true
+    limit: $first || $last
   };
 }
 function getBoundCond(orderCols, bound, kind) {
@@ -507,23 +501,9 @@ function getBoundCond(orderCols, bound, kind) {
 }
 const MAX_LIMIT = 4096;
 function selectByArgs(args, projection, options) {
-  const { table, idCol } = options;
-  const { where, order, group, limit, meta, ensureSingleRow } = getArgSql(
-    args,
-    options
-  );
+  const { table } = options;
+  const { where, order, group, limit, meta } = getArgSql(args, options);
   const clampedLimit = Math.min(MAX_LIMIT, limit || MAX_LIMIT);
-  if (!ensureSingleRow) {
-    return sql`
-      SELECT
-      ${getSelectCols(options, projection)}, ${meta}
-      FROM "${raw(table)}" WHERE "${raw(idCol)}" = (
-        SELECT "${raw(idCol)}" FROM "${raw(table)}"
-        ${where.length ? sql`WHERE ${join(where, " AND ")}` : empty}
-        LIMIT 2
-      )
-    `;
-  }
   return sql`
     SELECT
     ${getSelectCols(options, projection)}, ${meta}
@@ -559,7 +539,7 @@ function getSingleSql(arg, options) {
         SELECT "${raw(idCol)}"
         FROM "${raw(table)}"
         WHERE ${join(where, " AND ")}
-        LIMIT 2
+        LIMIT 1
       )`,
     meta
   };

package/package.json

@@ -2,7 +2,7 @@
   "name": "@graffy/pg",
   "description": "The standard Postgres module for Graffy. Each instance this module mounts a Postgres table as a Graffy subtree.",
   "author": "aravind (https://github.com/aravindet)",
-  "version": "0.16.3-alpha.4",
+  "version": "0.16.3-alpha.5",
   "main": "./index.cjs",
   "exports": {
     "import": "./index.mjs",
@@ -16,7 +16,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@graffy/common": "0.16.3-alpha.4",
+    "@graffy/common": "0.16.3-alpha.5",
     "debug": "^4.3.3"
   },
   "peerDependencies": {

package/types/sql/getArgSql.d.ts

@@ -5,7 +5,7 @@
   @param {{prefix: string, idCol: string, verDefault: string}} options
 
   @typedef { import('sql-template-tag').Sql } Sql
-  @return {{ meta: Sql, where: Sql[], order?: Sql, group?: Sql, limit: number, ensureSingleRow: boolean }}
+  @return {{ meta: Sql, where: Sql[], order?: Sql, group?: Sql, limit: number }}
 */
 export default function getArgSql({ $first, $last, $after, $before, $since, $until, $all, $cursor: _, ...rest }: object, options: {
     prefix: string;
@@ -17,7 +17,6 @@ export default function getArgSql({ $first, $last, $after, $before, $since, $unt
     order?: Sql;
     group?: Sql;
     limit: number;
-    ensureSingleRow: boolean;
 };
 /**
  * Uses the args object (typically passed in the $key attribute)