@ftschopp/dynatable-core 2.2.0 → 2.2.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## @ftschopp/dynatable-core [2.2.1](https://github.com/ftschopp/dynatable/compare/@ftschopp/dynatable-core@2.2.0...@ftschopp/dynatable-core@2.2.1) (2026-06-03)
+
+
+### Bug Fixes
+
+* **core:** reject undefined in update actions with actionable error ([#56](https://github.com/ftschopp/dynatable/issues/56)) ([5b8bee9](https://github.com/ftschopp/dynatable/commit/5b8bee9e9c041abdbff1b2b97d1eaba16730f257)), closes [#55](https://github.com/ftschopp/dynatable/issues/55)
+
 # @ftschopp/dynatable-core [2.2.0](https://github.com/ftschopp/dynatable/compare/@ftschopp/dynatable-core@2.1.0...@ftschopp/dynatable-core@2.2.0) (2026-06-02)
 
 

package/README.md

@@ -265,6 +265,13 @@ await table.entities.User.update({ username: 'alice' })
   .where((attr, op) => op.gt(attr.followerCount, 0))
   .execute();
 
+// Note: `.set()`, `.setIfNotExists()`, `.add()` and `.delete()` reject
+// `undefined` values up front. DynamoDB can't encode `undefined` in
+// `ExpressionAttributeValues`, so the builder throws with the offending
+// keys instead of letting the request fail server-side. Use `.remove(attr)`
+// to clear an attribute, or filter `undefined` out of your payload before
+// calling `.set()`. `null` is allowed — it writes the DynamoDB `NULL` type.
+
 // DELETE - Remove item
 await table.entities.User.delete({ username: 'alice' })
   .returning('ALL_OLD')

package/dist/builders/update/create-update-builder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"create-update-builder.d.ts","sourceRoot":"","sources":["../../../src/builders/update/create-update-builder.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAE1D,OAAO,EAAgC,SAAS,EAA4B,MAAM,WAAW,CAAC;AAC9F,OAAO,EAAE,aAAa,EAAE,YAAY,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACpE,OAAO,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAC;AAyB7D;;;;;;;;GAQG;AACH,wBAAgB,mBAAmB,CAAC,KAAK,EACvC,SAAS,EAAE,MAAM,EACjB,GAAG,EAAE,OAAO,CAAC,KAAK,CAAC,EACnB,MAAM,EAAE,cAAc,EACtB,cAAc,GAAE,SAAS,EAAO,EAChC,aAAa,GAAE;IACb,GAAG,EAAE,YAAY,EAAE,CAAC;IACpB,MAAM,EAAE,YAAY,EAAE,CAAC;IACvB,GAAG,EAAE,YAAY,EAAE,CAAC;IACpB,MAAM,EAAE,YAAY,EAAE,CAAC;CACuB,EAChD,UAAU,GAAE,MAAM,GAAG,SAAS,GAAG,SAAS,GAAG,aAAa,GAAG,aAAsB,EACnF,YAAY,SAAI,EAChB,gBAAgB,UAAQ,EACxB,MAAM,CAAC,EAAE,cAAc,EACvB,YAAY,CAAC,EAAE,YAAY,EAC3B,SAAS,GAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAM,EACnC,gBAAgB,CAAC,EAAE,SAAS,GAAG,OAAO,GAAG,MAAM,EAO/C,oBAAoB,GAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAM,GAC7C,aAAa,CAAC,KAAK,CAAC,CA4jBtB"}
+{"version":3,"file":"create-update-builder.d.ts","sourceRoot":"","sources":["../../../src/builders/update/create-update-builder.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAE1D,OAAO,EAAgC,SAAS,EAA4B,MAAM,WAAW,CAAC;AAC9F,OAAO,EAAE,aAAa,EAAE,YAAY,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACpE,OAAO,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAC;AAkE7D;;;;;;;;GAQG;AACH,wBAAgB,mBAAmB,CAAC,KAAK,EACvC,SAAS,EAAE,MAAM,EACjB,GAAG,EAAE,OAAO,CAAC,KAAK,CAAC,EACnB,MAAM,EAAE,cAAc,EACtB,cAAc,GAAE,SAAS,EAAO,EAChC,aAAa,GAAE;IACb,GAAG,EAAE,YAAY,EAAE,CAAC;IACpB,MAAM,EAAE,YAAY,EAAE,CAAC;IACvB,GAAG,EAAE,YAAY,EAAE,CAAC;IACpB,MAAM,EAAE,YAAY,EAAE,CAAC;CACuB,EAChD,UAAU,GAAE,MAAM,GAAG,SAAS,GAAG,SAAS,GAAG,aAAa,GAAG,aAAsB,EACnF,YAAY,SAAI,EAChB,gBAAgB,UAAQ,EACxB,MAAM,CAAC,EAAE,cAAc,EACvB,YAAY,CAAC,EAAE,YAAY,EAC3B,SAAS,GAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAM,EACnC,gBAAgB,CAAC,EAAE,SAAS,GAAG,OAAO,GAAG,MAAM,EAO/C,oBAAoB,GAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAM,GAC7C,aAAa,CAAC,KAAK,CAAC,CAkkBtB"}

package/dist/builders/update/create-update-builder.js

@@ -25,6 +25,37 @@ function actionAttrName(action) {
     const values = Object.values(action.names);
     return values[0];
 }
+/**
+ * Reject `undefined` before it reaches DynamoDB. SET / ADD / DELETE /
+ * setIfNotExists all encode the value into `ExpressionAttributeValues` — an
+ * `undefined` there is either silently dropped by the marshaller (when
+ * `removeUndefinedValues` is on, which then fails server-side with an unused
+ * expression value error) or rejected outright. Catching it at the call site
+ * gives a useful error and a pointer to the right primitive.
+ *
+ * `null` is intentionally allowed — DynamoDB has a NULL attribute type and
+ * the caller may legitimately want to write it.
+ */
+function assertNoUndefined(method, payload) {
+    const undefinedKeys = 'attr' in payload
+        ? payload.value === undefined
+            ? [payload.attr]
+            : []
+        : Object.entries(payload)
+            .filter(([, v]) => v === undefined)
+            .map(([k]) => k);
+    if (undefinedKeys.length === 0)
+        return;
+    const guidance = method === 'set'
+        ? `Use .remove(attr) to clear an attribute, or filter undefined out ` +
+            `before calling .set().`
+        : method === 'setIfNotExists'
+            ? `setIfNotExists requires a concrete value; omit the key or filter ` +
+                `it out.`
+            : `.${method}() requires a concrete value for each attribute.`;
+    throw new Error(`.${method}() received undefined for key(s) [${undefinedKeys.join(', ')}]. ` +
+        `DynamoDB cannot encode undefined in ExpressionAttributeValues. ${guidance}`);
+}
 /**
  * Creates an UpdateBuilder for an item key and table.
  *
@@ -72,6 +103,7 @@ setIfNotExistsInputs = {}) {
                 attrOrUpdates !== null) {
                 // Multiple updates case
                 const updates = attrOrUpdates;
+                assertNoUndefined('set', updates);
                 const newActions = [];
                 const newSetInputs = { ...setInputs };
                 for (const [attr, val] of Object.entries(updates)) {
@@ -88,6 +120,7 @@ setIfNotExistsInputs = {}) {
             }
             // Single update case
             const attrName = normalizeAttr(attrOrUpdates);
+            assertNoUndefined('set', { attr: attrName, value });
             const valueName = getUniqueValueName(attrName);
             const action = {
                 expression: `#${attrName} = :${valueName}`,
@@ -103,6 +136,7 @@ setIfNotExistsInputs = {}) {
                 typeof attrOrUpdates === 'object' &&
                 attrOrUpdates !== null) {
                 const updates = attrOrUpdates;
+                assertNoUndefined('setIfNotExists', updates);
                 const newActions = [];
                 const newSetIfNotExistsInputs = { ...setIfNotExistsInputs };
                 for (const [attr, val] of Object.entries(updates)) {
@@ -119,6 +153,7 @@ setIfNotExistsInputs = {}) {
             }
             // Single update case
             const attrName = normalizeAttr(attrOrUpdates);
+            assertNoUndefined('setIfNotExists', { attr: attrName, value });
             const valueName = getUniqueValueName(attrName);
             const action = {
                 expression: `#${attrName} = if_not_exists(#${attrName}, :${valueName})`,
@@ -137,6 +172,7 @@ setIfNotExistsInputs = {}) {
         },
         add(attr, value) {
             const attrName = normalizeAttr(attr);
+            assertNoUndefined('add', { attr: attrName, value });
             const valueName = getUniqueValueName(attrName);
             const action = {
                 expression: `#${attrName} :${valueName}`,
@@ -147,6 +183,7 @@ setIfNotExistsInputs = {}) {
         },
         delete(attr, value) {
             const attrName = normalizeAttr(attr);
+            assertNoUndefined('delete', { attr: attrName, value });
             const valueName = getUniqueValueName(attrName);
             const action = {
                 expression: `#${attrName} :${valueName}`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ftschopp/dynatable-core",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "Core library for DynamoDB single table design",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",