quidproquo-actionprocessor-awslambda 0.1.22 → 0.1.23

package/lib/commonjs/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.d.ts

@@ -3,5 +3,13 @@ import { KeyValueStoreQPQConfigSetting, KvsQueryOperation, Nullable } from 'quid
  * Pick the GSI that can serve the query, or null to query the primary table.
  * Returns the GSI's partition key name, which is also its index name (the CDK
  * construct names each GSI after its partition key).
+ *
+ * A Query MUST constrain the partition key of whatever it runs against. So: if
+ * the query names the primary partition key, the primary table can serve it
+ * (strongly consistent, and its sort key is available) - prefer it over any GSI.
+ * Otherwise route to the GSI whose partition key the query names. Deciding on
+ * the primary SORT key instead is wrong: a GSI commonly has the primary sort key
+ * as its partition key (pk userId / sk tenantId with a GSI on tenantId), and a
+ * query on that key alone can only be served by the GSI.
  */
 export declare const getDynamoTableIndexByConfigAndQuery: (setting: KeyValueStoreQPQConfigSetting, query: KvsQueryOperation) => Nullable<string>;

package/lib/commonjs/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.js

@@ -6,15 +6,20 @@ const flattenKvsQueryConditions_1 = require("./flattenKvsQueryConditions");
  * Pick the GSI that can serve the query, or null to query the primary table.
  * Returns the GSI's partition key name, which is also its index name (the CDK
  * construct names each GSI after its partition key).
+ *
+ * A Query MUST constrain the partition key of whatever it runs against. So: if
+ * the query names the primary partition key, the primary table can serve it
+ * (strongly consistent, and its sort key is available) - prefer it over any GSI.
+ * Otherwise route to the GSI whose partition key the query names. Deciding on
+ * the primary SORT key instead is wrong: a GSI commonly has the primary sort key
+ * as its partition key (pk userId / sk tenantId with a GSI on tenantId), and a
+ * query on that key alone can only be served by the GSI.
  */
 const getDynamoTableIndexByConfigAndQuery = (setting, query) => {
     var _a;
     const queriedKeys = (0, flattenKvsQueryConditions_1.flattenKvsQueryConditions)(query).map((condition) => condition.key);
-    // If the query includes the primary sort key, the primary table is a direct
-    // match: prefer it over any GSI so we never land on a GSI with a non-SK
-    // condition.
-    const primarySortKey = (_a = setting.sortKeys[0]) === null || _a === void 0 ? void 0 : _a.key;
-    if (primarySortKey && queriedKeys.includes(primarySortKey)) {
+    const primaryPartitionKey = (_a = setting.partitionKey) === null || _a === void 0 ? void 0 : _a.key;
+    if (primaryPartitionKey && queriedKeys.includes(primaryPartitionKey)) {
         return null;
     }
     const matchingIndex = setting.indexes.find((index) => queriedKeys.includes(index.partitionKey.key));

package/lib/commonjs/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.js.map

@@ -1 +1 @@
-{"version":3,"file":"getDynamoTableIndexByConfigAndQuery.js","sourceRoot":"","sources":["../../../../../src/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.ts"],"names":[],"mappings":";;;AAEA,2EAAwE;AAExE;;;;GAIG;AACI,MAAM,mCAAmC,GAAG,CAAC,OAAsC,EAAE,KAAwB,EAAoB,EAAE;;IACxI,MAAM,WAAW,GAAG,IAAA,qDAAyB,EAAC,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,SAAS,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAEvF,4EAA4E;IAC5E,wEAAwE;IACxE,aAAa;IACb,MAAM,cAAc,GAAG,MAAA,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,0CAAE,GAAG,CAAC;IAChD,IAAI,cAAc,IAAI,WAAW,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC;QAC3D,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,aAAa,GAAG,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC;IAEpG,OAAO,aAAa,CAAC,CAAC,CAAC,aAAa,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/D,CAAC,CAAC;AAdW,QAAA,mCAAmC,uCAc9C","sourcesContent":["import { KeyValueStoreQPQConfigSetting, KvsQueryOperation, Nullable } from 'quidproquo-core';\n\nimport { flattenKvsQueryConditions } from './flattenKvsQueryConditions';\n\n/**\n * Pick the GSI that can serve the query, or null to query the primary table.\n * Returns the GSI's partition key name, which is also its index name (the CDK\n * construct names each GSI after its partition key).\n */\nexport const getDynamoTableIndexByConfigAndQuery = (setting: KeyValueStoreQPQConfigSetting, query: KvsQueryOperation): Nullable<string> => {\n  const queriedKeys = flattenKvsQueryConditions(query).map((condition) => condition.key);\n\n  // If the query includes the primary sort key, the primary table is a direct\n  // match: prefer it over any GSI so we never land on a GSI with a non-SK\n  // condition.\n  const primarySortKey = setting.sortKeys[0]?.key;\n  if (primarySortKey && queriedKeys.includes(primarySortKey)) {\n    return null;\n  }\n\n  const matchingIndex = setting.indexes.find((index) => queriedKeys.includes(index.partitionKey.key));\n\n  return matchingIndex ? matchingIndex.partitionKey.key : null;\n};\n"]}
+{"version":3,"file":"getDynamoTableIndexByConfigAndQuery.js","sourceRoot":"","sources":["../../../../../src/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.ts"],"names":[],"mappings":";;;AAEA,2EAAwE;AAExE;;;;;;;;;;;;GAYG;AACI,MAAM,mCAAmC,GAAG,CAAC,OAAsC,EAAE,KAAwB,EAAoB,EAAE;;IACxI,MAAM,WAAW,GAAG,IAAA,qDAAyB,EAAC,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,SAAS,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAEvF,MAAM,mBAAmB,GAAG,MAAA,OAAO,CAAC,YAAY,0CAAE,GAAG,CAAC;IACtD,IAAI,mBAAmB,IAAI,WAAW,CAAC,QAAQ,CAAC,mBAAmB,CAAC,EAAE,CAAC;QACrE,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,aAAa,GAAG,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC;IAEpG,OAAO,aAAa,CAAC,CAAC,CAAC,aAAa,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/D,CAAC,CAAC;AAXW,QAAA,mCAAmC,uCAW9C","sourcesContent":["import { KeyValueStoreQPQConfigSetting, KvsQueryOperation, Nullable } from 'quidproquo-core';\n\nimport { flattenKvsQueryConditions } from './flattenKvsQueryConditions';\n\n/**\n * Pick the GSI that can serve the query, or null to query the primary table.\n * Returns the GSI's partition key name, which is also its index name (the CDK\n * construct names each GSI after its partition key).\n *\n * A Query MUST constrain the partition key of whatever it runs against. So: if\n * the query names the primary partition key, the primary table can serve it\n * (strongly consistent, and its sort key is available) - prefer it over any GSI.\n * Otherwise route to the GSI whose partition key the query names. Deciding on\n * the primary SORT key instead is wrong: a GSI commonly has the primary sort key\n * as its partition key (pk userId / sk tenantId with a GSI on tenantId), and a\n * query on that key alone can only be served by the GSI.\n */\nexport const getDynamoTableIndexByConfigAndQuery = (setting: KeyValueStoreQPQConfigSetting, query: KvsQueryOperation): Nullable<string> => {\n  const queriedKeys = flattenKvsQueryConditions(query).map((condition) => condition.key);\n\n  const primaryPartitionKey = setting.partitionKey?.key;\n  if (primaryPartitionKey && queriedKeys.includes(primaryPartitionKey)) {\n    return null;\n  }\n\n  const matchingIndex = setting.indexes.find((index) => queriedKeys.includes(index.partitionKey.key));\n\n  return matchingIndex ? matchingIndex.partitionKey.key : null;\n};\n"]}

package/lib/esm/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.d.ts

@@ -3,5 +3,13 @@ import { KeyValueStoreQPQConfigSetting, KvsQueryOperation, Nullable } from 'quid
  * Pick the GSI that can serve the query, or null to query the primary table.
  * Returns the GSI's partition key name, which is also its index name (the CDK
  * construct names each GSI after its partition key).
+ *
+ * A Query MUST constrain the partition key of whatever it runs against. So: if
+ * the query names the primary partition key, the primary table can serve it
+ * (strongly consistent, and its sort key is available) - prefer it over any GSI.
+ * Otherwise route to the GSI whose partition key the query names. Deciding on
+ * the primary SORT key instead is wrong: a GSI commonly has the primary sort key
+ * as its partition key (pk userId / sk tenantId with a GSI on tenantId), and a
+ * query on that key alone can only be served by the GSI.
  */
 export declare const getDynamoTableIndexByConfigAndQuery: (setting: KeyValueStoreQPQConfigSetting, query: KvsQueryOperation) => Nullable<string>;

package/lib/esm/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.js

@@ -3,14 +3,19 @@ import { flattenKvsQueryConditions } from './flattenKvsQueryConditions';
  * Pick the GSI that can serve the query, or null to query the primary table.
  * Returns the GSI's partition key name, which is also its index name (the CDK
  * construct names each GSI after its partition key).
+ *
+ * A Query MUST constrain the partition key of whatever it runs against. So: if
+ * the query names the primary partition key, the primary table can serve it
+ * (strongly consistent, and its sort key is available) - prefer it over any GSI.
+ * Otherwise route to the GSI whose partition key the query names. Deciding on
+ * the primary SORT key instead is wrong: a GSI commonly has the primary sort key
+ * as its partition key (pk userId / sk tenantId with a GSI on tenantId), and a
+ * query on that key alone can only be served by the GSI.
  */
 export const getDynamoTableIndexByConfigAndQuery = (setting, query) => {
     const queriedKeys = flattenKvsQueryConditions(query).map((condition) => condition.key);
-    // If the query includes the primary sort key, the primary table is a direct
-    // match: prefer it over any GSI so we never land on a GSI with a non-SK
-    // condition.
-    const primarySortKey = setting.sortKeys[0]?.key;
-    if (primarySortKey && queriedKeys.includes(primarySortKey)) {
+    const primaryPartitionKey = setting.partitionKey?.key;
+    if (primaryPartitionKey && queriedKeys.includes(primaryPartitionKey)) {
         return null;
     }
     const matchingIndex = setting.indexes.find((index) => queriedKeys.includes(index.partitionKey.key));

package/lib/esm/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.js.map

@@ -1 +1 @@
-{"version":3,"file":"getDynamoTableIndexByConfigAndQuery.js","sourceRoot":"","sources":["../../../../../src/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,yBAAyB,EAAE,MAAM,6BAA6B,CAAC;AAExE;;;;GAIG;AACH,MAAM,CAAC,MAAM,mCAAmC,GAAG,CAAC,OAAsC,EAAE,KAAwB,EAAoB,EAAE;IACxI,MAAM,WAAW,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,SAAS,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAEvF,4EAA4E;IAC5E,wEAAwE;IACxE,aAAa;IACb,MAAM,cAAc,GAAG,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC;IAChD,IAAI,cAAc,IAAI,WAAW,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC;QAC3D,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,aAAa,GAAG,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC;IAEpG,OAAO,aAAa,CAAC,CAAC,CAAC,aAAa,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/D,CAAC,CAAC","sourcesContent":["import { KeyValueStoreQPQConfigSetting, KvsQueryOperation, Nullable } from 'quidproquo-core';\n\nimport { flattenKvsQueryConditions } from './flattenKvsQueryConditions';\n\n/**\n * Pick the GSI that can serve the query, or null to query the primary table.\n * Returns the GSI's partition key name, which is also its index name (the CDK\n * construct names each GSI after its partition key).\n */\nexport const getDynamoTableIndexByConfigAndQuery = (setting: KeyValueStoreQPQConfigSetting, query: KvsQueryOperation): Nullable<string> => {\n  const queriedKeys = flattenKvsQueryConditions(query).map((condition) => condition.key);\n\n  // If the query includes the primary sort key, the primary table is a direct\n  // match: prefer it over any GSI so we never land on a GSI with a non-SK\n  // condition.\n  const primarySortKey = setting.sortKeys[0]?.key;\n  if (primarySortKey && queriedKeys.includes(primarySortKey)) {\n    return null;\n  }\n\n  const matchingIndex = setting.indexes.find((index) => queriedKeys.includes(index.partitionKey.key));\n\n  return matchingIndex ? matchingIndex.partitionKey.key : null;\n};\n"]}
+{"version":3,"file":"getDynamoTableIndexByConfigAndQuery.js","sourceRoot":"","sources":["../../../../../src/logic/dynamo/qpqDynamoOrm/getDynamoTableIndexByConfigAndQuery.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,yBAAyB,EAAE,MAAM,6BAA6B,CAAC;AAExE;;;;;;;;;;;;GAYG;AACH,MAAM,CAAC,MAAM,mCAAmC,GAAG,CAAC,OAAsC,EAAE,KAAwB,EAAoB,EAAE;IACxI,MAAM,WAAW,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,SAAS,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;IAEvF,MAAM,mBAAmB,GAAG,OAAO,CAAC,YAAY,EAAE,GAAG,CAAC;IACtD,IAAI,mBAAmB,IAAI,WAAW,CAAC,QAAQ,CAAC,mBAAmB,CAAC,EAAE,CAAC;QACrE,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,aAAa,GAAG,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC;IAEpG,OAAO,aAAa,CAAC,CAAC,CAAC,aAAa,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/D,CAAC,CAAC","sourcesContent":["import { KeyValueStoreQPQConfigSetting, KvsQueryOperation, Nullable } from 'quidproquo-core';\n\nimport { flattenKvsQueryConditions } from './flattenKvsQueryConditions';\n\n/**\n * Pick the GSI that can serve the query, or null to query the primary table.\n * Returns the GSI's partition key name, which is also its index name (the CDK\n * construct names each GSI after its partition key).\n *\n * A Query MUST constrain the partition key of whatever it runs against. So: if\n * the query names the primary partition key, the primary table can serve it\n * (strongly consistent, and its sort key is available) - prefer it over any GSI.\n * Otherwise route to the GSI whose partition key the query names. Deciding on\n * the primary SORT key instead is wrong: a GSI commonly has the primary sort key\n * as its partition key (pk userId / sk tenantId with a GSI on tenantId), and a\n * query on that key alone can only be served by the GSI.\n */\nexport const getDynamoTableIndexByConfigAndQuery = (setting: KeyValueStoreQPQConfigSetting, query: KvsQueryOperation): Nullable<string> => {\n  const queriedKeys = flattenKvsQueryConditions(query).map((condition) => condition.key);\n\n  const primaryPartitionKey = setting.partitionKey?.key;\n  if (primaryPartitionKey && queriedKeys.includes(primaryPartitionKey)) {\n    return null;\n  }\n\n  const matchingIndex = setting.indexes.find((index) => queriedKeys.includes(index.partitionKey.key));\n\n  return matchingIndex ? matchingIndex.partitionKey.key : null;\n};\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "quidproquo-actionprocessor-awslambda",
-  "version": "0.1.22",
+  "version": "0.1.23",
   "description": "",
   "main": "./lib/commonjs/index.js",
   "module": "./lib/esm/index.js",
@@ -66,11 +66,11 @@
     "lodash": "^4.17.21",
     "node-cache": "^5.1.2",
     "node-match-path": "^0.6.3",
-    "quidproquo-actionprocessor-js": "0.1.22",
-    "quidproquo-actionprocessor-node": "0.1.22",
-    "quidproquo-config-aws": "0.1.22",
-    "quidproquo-core": "0.1.22",
-    "quidproquo-webserver": "0.1.22"
+    "quidproquo-actionprocessor-js": "0.1.23",
+    "quidproquo-actionprocessor-node": "0.1.23",
+    "quidproquo-config-aws": "0.1.23",
+    "quidproquo-core": "0.1.23",
+    "quidproquo-webserver": "0.1.23"
   },
   "devDependencies": {
     "@types/busboy": "^1.5.0",
@@ -78,7 +78,7 @@
     "@types/lodash": "^4.14.194",
     "@types/node": "^22.13.13",
     "esbuild": "^0.25.8",
-    "quidproquo-tsconfig": "0.1.22",
+    "quidproquo-tsconfig": "0.1.23",
     "typescript": "^5.8.2"
   }
 }