@rytass/bpm-core-nestjs-module 0.11.0 → 0.12.0

package/src/lib/workflow-engine/workflow-condition-evaluator.js

@@ -21,6 +21,14 @@ function evaluateWorkflowEdgeCondition(edge, context) {
     return compareWorkflowValues(actualValue, operator, expectedValue);
 }
 function compareWorkflowValues(actualValue, operator, expectedValue) {
+    // A table value is a list of rows. Only IS_FILLED / IS_EMPTY (handled above,
+    // by row count) are defined for it; the designer offers nothing else, but a
+    // hand-written or imported definition can still carry a value comparison.
+    // Stringifying a row record would compare `[object Object]` and route the
+    // instance on a coincidence, so it never matches (ADR 16 §3.8).
+    if (isNonPrimitiveValue(actualValue)) {
+        return false;
+    }
     if (Array.isArray(actualValue)) {
         return compareArrayValue(actualValue, operator, expectedValue);
     }
@@ -67,6 +75,16 @@ function compareOrderedValue(actualValue, operator, expectedValue) {
     }
     return false;
 }
+/**
+ * True for a row record, or for a list holding any of them. Multi-select values
+ * (`readonly string[]`) stay comparable.
+ */
+function isNonPrimitiveValue(value) {
+    if (Array.isArray(value)) {
+        return value.some((entry) => isNonPrimitiveValue(entry));
+    }
+    return typeof value === 'object' && value !== null;
+}
 function isEmptyWorkflowValue(value) {
     if (value === null || typeof value === 'undefined') {
         return true;

package/src/lib/workflow-engine/workflow-condition-evaluator.js.map

@@ -1 +1 @@
-{"version":3,"file":"workflow-condition-evaluator.js","sourceRoot":"","sources":["../../../../../../libs/bpm-core/src/lib/workflow-engine/workflow-condition-evaluator.ts"],"names":[],"mappings":";;AASA,sEA4BC;AA5BD,SAAgB,6BAA6B,CAC3C,IAAkB,EAClB,OAAiC;IAEjC,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,iBAAiB,CAAC;IAC7C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,iBAAiB,CAAC;IAE7C,IAAI,CAAC,QAAQ,IAAI,CAAC,QAAQ,EAAE,CAAC;QAC3B,OAAO,KAAK,CAAC;IACf,CAAC;IAED,MAAM,WAAW,GAAG,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;IAE/C,IAAI,QAAQ,KAAK,WAAW,EAAE,CAAC;QAC7B,OAAO,CAAC,oBAAoB,CAAC,WAAW,CAAC,CAAC;IAC5C,CAAC;IAED,IAAI,QAAQ,KAAK,UAAU,EAAE,CAAC;QAC5B,OAAO,oBAAoB,CAAC,WAAW,CAAC,CAAC;IAC3C,CAAC;IAED,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC;IAE/C,IAAI,OAAO,aAAa,KAAK,WAAW,EAAE,CAAC;QACzC,OAAO,KAAK,CAAC;IACf,CAAC;IAED,OAAO,qBAAqB,CAAC,WAAW,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;AACrE,CAAC;AAED,SAAS,qBAAqB,CAC5B,WAAoB,EACpB,QAAuC,EACvC,aAAqB;IAErB,IAAI,KAAK,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,CAAC;QAC/B,OAAO,iBAAiB,CAAC,WAAW,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;IACjE,CAAC;IAED,IAAI,QAAQ,KAAK,QAAQ,EAAE,CAAC;QAC1B,OAAO,sBAAsB,CAAC,WAAW,CAAC;YACxC,sBAAsB,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC;IACvD,CAAC;IAED,IAAI,QAAQ,KAAK,YAAY,EAAE,CAAC;QAC9B,OAAO,sBAAsB,CAAC,WAAW,CAAC;YACxC,sBAAsB,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC;IACvD,CAAC;IAED,OAAO,mBAAmB,CAAC,WAAW,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;AACnE,CAAC;AAED,SAAS,iBAAiB,CACxB,WAA+B,EAC/B,QAAuC,EACvC,aAAqB;IAErB,MAAM,qBAAqB,GAAG,WAAW;SACtC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,sBAAsB,CAAC,KAAK,CAAC,CAAC;SAC7C,QAAQ,CAAC,aAAa,CAAC,CAAC;IAE3B,IAAI,QAAQ,KAAK,QAAQ,EAAE,CAAC;QAC1B,OAAO,qBAAqB,CAAC;IAC/B,CAAC;IAED,IAAI,QAAQ,KAAK,YAAY,EAAE,CAAC;QAC9B,OAAO,CAAC,qBAAqB,CAAC;IAChC,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,mBAAmB,CAC1B,WAAoB,EACpB,QAAuC,EACvC,aAAqB;IAErB,MAAM,YAAY,GAAG,gBAAgB,CAAC,WAAW,CAAC,CAAC;IACnD,MAAM,cAAc,GAAG,gBAAgB,CAAC,aAAa,CAAC,CAAC;IACvD,MAAM,gBAAgB,GACpB,YAAY,KAAK,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,sBAAsB,CAAC,WAAW,CAAC,CAAC;IAC7E,MAAM,kBAAkB,GACtB,YAAY,KAAK,IAAI,IAAI,cAAc,KAAK,IAAI;QAC9C,CAAC,CAAC,cAAc;QAChB,CAAC,CAAC,aAAa,CAAC;IAEpB,IAAI,QAAQ,KAAK,cAAc,EAAE,CAAC;QAChC,OAAO,gBAAgB,GAAG,kBAAkB,CAAC;IAC/C,CAAC;IAED,IAAI,QAAQ,KAAK,wBAAwB,EAAE,CAAC;QAC1C,OAAO,gBAAgB,IAAI,kBAAkB,CAAC;IAChD,CAAC;IAED,IAAI,QAAQ,KAAK,WAAW,EAAE,CAAC;QAC7B,OAAO,gBAAgB,GAAG,kBAAkB,CAAC;IAC/C,CAAC;IAED,IAAI,QAAQ,KAAK,qBAAqB,EAAE,CAAC;QACvC,OAAO,gBAAgB,IAAI,kBAAkB,CAAC;IAChD,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,oBAAoB,CAAC,KAAc;IAC1C,IAAI,KAAK,KAAK,IAAI,IAAI,OAAO,KAAK,KAAK,WAAW,EAAE,CAAC;QACnD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;QACzB,OAAO,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC;IAC5B,CAAC;IAED,OAAO,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC;AACjE,CAAC;AAED,SAAS,sBAAsB,CAAC,KAAc;IAC5C,IACE,OAAO,KAAK,KAAK,QAAQ;QACzB,OAAO,KAAK,KAAK,QAAQ;QACzB,OAAO,KAAK,KAAK,SAAS,EAC1B,CAAC;QACD,OAAO,KAAK,CAAC;IACf,CAAC;IAED,OAAO,MAAM,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;AAC7B,CAAC;AAED,SAAS,sBAAsB,CAC7B,WAAoB,EACpB,aAAqB;IAErB,IAAI,OAAO,WAAW,KAAK,SAAS,EAAE,CAAC;QACrC,OAAO,aAAa,KAAK,MAAM,CAAC;IAClC,CAAC;IAED,IAAI,OAAO,WAAW,KAAK,QAAQ,EAAE,CAAC;QACpC,MAAM,cAAc,GAAG,MAAM,CAAC,aAAa,CAAC,CAAC;QAE7C,OAAO,MAAM,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,aAAa,CAAC;IAC1E,CAAC;IAED,OAAO,aAAa,CAAC;AACvB,CAAC;AAED,SAAS,gBAAgB,CAAC,KAAc;IACtC,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,OAAO,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC;IAC/C,CAAC;IAED,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,EAAE,CAAC;QAC/C,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,WAAW,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC;IAElC,OAAO,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC;AAC3D,CAAC"}
+{"version":3,"file":"workflow-condition-evaluator.js","sourceRoot":"","sources":["../../../../../../libs/bpm-core/src/lib/workflow-engine/workflow-condition-evaluator.ts"],"names":[],"mappings":";;AASA,sEA4BC;AA5BD,SAAgB,6BAA6B,CAC3C,IAAkB,EAClB,OAAiC;IAEjC,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,iBAAiB,CAAC;IAC7C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,iBAAiB,CAAC;IAE7C,IAAI,CAAC,QAAQ,IAAI,CAAC,QAAQ,EAAE,CAAC;QAC3B,OAAO,KAAK,CAAC;IACf,CAAC;IAED,MAAM,WAAW,GAAG,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;IAE/C,IAAI,QAAQ,KAAK,WAAW,EAAE,CAAC;QAC7B,OAAO,CAAC,oBAAoB,CAAC,WAAW,CAAC,CAAC;IAC5C,CAAC;IAED,IAAI,QAAQ,KAAK,UAAU,EAAE,CAAC;QAC5B,OAAO,oBAAoB,CAAC,WAAW,CAAC,CAAC;IAC3C,CAAC;IAED,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC;IAE/C,IAAI,OAAO,aAAa,KAAK,WAAW,EAAE,CAAC;QACzC,OAAO,KAAK,CAAC;IACf,CAAC;IAED,OAAO,qBAAqB,CAAC,WAAW,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;AACrE,CAAC;AAED,SAAS,qBAAqB,CAC5B,WAAoB,EACpB,QAAuC,EACvC,aAAqB;IAErB,6EAA6E;IAC7E,4EAA4E;IAC5E,0EAA0E;IAC1E,0EAA0E;IAC1E,gEAAgE;IAChE,IAAI,mBAAmB,CAAC,WAAW,CAAC,EAAE,CAAC;QACrC,OAAO,KAAK,CAAC;IACf,CAAC;IAED,IAAI,KAAK,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,CAAC;QAC/B,OAAO,iBAAiB,CAAC,WAAW,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;IACjE,CAAC;IAED,IAAI,QAAQ,KAAK,QAAQ,EAAE,CAAC;QAC1B,OAAO,sBAAsB,CAAC,WAAW,CAAC;YACxC,sBAAsB,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC;IACvD,CAAC;IAED,IAAI,QAAQ,KAAK,YAAY,EAAE,CAAC;QAC9B,OAAO,sBAAsB,CAAC,WAAW,CAAC;YACxC,sBAAsB,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC;IACvD,CAAC;IAED,OAAO,mBAAmB,CAAC,WAAW,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;AACnE,CAAC;AAED,SAAS,iBAAiB,CACxB,WAA+B,EAC/B,QAAuC,EACvC,aAAqB;IAErB,MAAM,qBAAqB,GAAG,WAAW;SACtC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,sBAAsB,CAAC,KAAK,CAAC,CAAC;SAC7C,QAAQ,CAAC,aAAa,CAAC,CAAC;IAE3B,IAAI,QAAQ,KAAK,QAAQ,EAAE,CAAC;QAC1B,OAAO,qBAAqB,CAAC;IAC/B,CAAC;IAED,IAAI,QAAQ,KAAK,YAAY,EAAE,CAAC;QAC9B,OAAO,CAAC,qBAAqB,CAAC;IAChC,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAS,mBAAmB,CAC1B,WAAoB,EACpB,QAAuC,EACvC,aAAqB;IAErB,MAAM,YAAY,GAAG,gBAAgB,CAAC,WAAW,CAAC,CAAC;IACnD,MAAM,cAAc,GAAG,gBAAgB,CAAC,aAAa,CAAC,CAAC;IACvD,MAAM,gBAAgB,GACpB,YAAY,KAAK,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,sBAAsB,CAAC,WAAW,CAAC,CAAC;IAC7E,MAAM,kBAAkB,GACtB,YAAY,KAAK,IAAI,IAAI,cAAc,KAAK,IAAI;QAC9C,CAAC,CAAC,cAAc;QAChB,CAAC,CAAC,aAAa,CAAC;IAEpB,IAAI,QAAQ,KAAK,cAAc,EAAE,CAAC;QAChC,OAAO,gBAAgB,GAAG,kBAAkB,CAAC;IAC/C,CAAC;IAED,IAAI,QAAQ,KAAK,wBAAwB,EAAE,CAAC;QAC1C,OAAO,gBAAgB,IAAI,kBAAkB,CAAC;IAChD,CAAC;IAED,IAAI,QAAQ,KAAK,WAAW,EAAE,CAAC;QAC7B,OAAO,gBAAgB,GAAG,kBAAkB,CAAC;IAC/C,CAAC;IAED,IAAI,QAAQ,KAAK,qBAAqB,EAAE,CAAC;QACvC,OAAO,gBAAgB,IAAI,kBAAkB,CAAC;IAChD,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;GAGG;AACH,SAAS,mBAAmB,CAAC,KAAc;IACzC,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;QACzB,OAAO,KAAK,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,mBAAmB,CAAC,KAAK,CAAC,CAAC,CAAC;IAC3D,CAAC;IAED,OAAO,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI,CAAC;AACrD,CAAC;AAED,SAAS,oBAAoB,CAAC,KAAc;IAC1C,IAAI,KAAK,KAAK,IAAI,IAAI,OAAO,KAAK,KAAK,WAAW,EAAE,CAAC;QACnD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;QACzB,OAAO,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC;IAC5B,CAAC;IAED,OAAO,OAAO,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC;AACjE,CAAC;AAED,SAAS,sBAAsB,CAAC,KAAc;IAC5C,IACE,OAAO,KAAK,KAAK,QAAQ;QACzB,OAAO,KAAK,KAAK,QAAQ;QACzB,OAAO,KAAK,KAAK,SAAS,EAC1B,CAAC;QACD,OAAO,KAAK,CAAC;IACf,CAAC;IAED,OAAO,MAAM,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;AAC7B,CAAC;AAED,SAAS,sBAAsB,CAC7B,WAAoB,EACpB,aAAqB;IAErB,IAAI,OAAO,WAAW,KAAK,SAAS,EAAE,CAAC;QACrC,OAAO,aAAa,KAAK,MAAM,CAAC;IAClC,CAAC;IAED,IAAI,OAAO,WAAW,KAAK,QAAQ,EAAE,CAAC;QACpC,MAAM,cAAc,GAAG,MAAM,CAAC,aAAa,CAAC,CAAC;QAE7C,OAAO,MAAM,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,aAAa,CAAC;IAC1E,CAAC;IAED,OAAO,aAAa,CAAC;AACvB,CAAC;AAED,SAAS,gBAAgB,CAAC,KAAc;IACtC,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,OAAO,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC;IAC/C,CAAC;IAED,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,EAAE,CAAC;QAC/C,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,WAAW,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC;IAElC,OAAO,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC;AAC3D,CAAC"}

package/src/lib/workflow-engine/workflow-engine.service.js

@@ -3314,6 +3314,15 @@ function writeValueAtPath(value, path, nextValue) {
     if (segments.length === 0) {
         return value;
     }
+    // An index step is not understood by this writer — `items[0].qty` splits into
+    // the literal segment `items[0]`, which would be created as a brand new
+    // top-level key rather than reaching the row. That is not the no-op ADR 16
+    // §3.8 asks for: it pollutes `formData` (and therefore the CEL context) with
+    // a key no schema declares, and makes `readValueAtPath` start answering for
+    // a path it should never resolve.
+    if (segments.some((segment) => /[[\]]/u.test(segment))) {
+        return value;
+    }
     return writeNestedValue(value, segments, nextValue);
 }
 function writeNestedValue(value, segments, nextValue) {
@@ -3325,6 +3334,15 @@ function writeNestedValue(value, segments, nextValue) {
         return { ...value, [currentSegment]: nextValue };
     }
     const currentValue = value[currentSegment];
+    // Descending through something that already holds a non-record value must be
+    // a no-op, not an overwrite. `items.qty` against a table would otherwise
+    // replace the whole row array with `{ qty: ... }`; the same guard protects a
+    // scalar from being swallowed by a nested write (ADR 16 §3.8).
+    if (typeof currentValue !== 'undefined' &&
+        currentValue !== null &&
+        !isRecord(currentValue)) {
+        return value;
+    }
     const nestedValue = isRecord(currentValue) ? currentValue : {};
     return {
         ...value,
@@ -3616,17 +3634,105 @@ function readCandidateStatusForClosedTask(taskStatus) {
     return workflow_engine_enums_1.TaskCandidateStatusEnum.CANCELLED;
 }
 function validateSubmittedFormData(schema, formData) {
-    const missingFields = schema.fields.filter((field) => {
-        const fieldValue = formData[field.fieldKey];
-        return (isSubmittedFormFieldVisible(field, schema.fields, formData) &&
-            isSubmittedFormFieldRequired(field, schema.fields, formData) &&
-            !isSubmittedFieldValuePresent(fieldValue));
-    });
+    const visibleFields = schema.fields.filter((field) => isSubmittedFormFieldVisible(field, schema.fields, formData));
+    // Tables answer "is this filled?" by row count and report per-row problems
+    // with an instance path, so they get their own pass instead of the flat
+    // "missing required fields" list (ADR 16 §3.7).
+    const missingFields = visibleFields.filter((field) => !(0, form_1.isTableFieldDefinition)(field) &&
+        isSubmittedFormFieldRequired(field, schema.fields, formData) &&
+        !isSubmittedFieldValuePresent(formData[field.fieldKey]));
     if (missingFields.length > 0) {
         throw new common_1.BadRequestException(`Form data is missing required fields: ${missingFields
             .map((field) => field.label || field.fieldKey)
             .join(', ')}`);
     }
+    const tableErrors = visibleFields
+        .filter(form_1.isTableFieldDefinition)
+        .flatMap((field) => validateSubmittedTableValue(field, formData[field.fieldKey], isSubmittedFormFieldRequired(field, schema.fields, formData)));
+    if (tableErrors.length > 0) {
+        throw new common_1.BadRequestException(tableErrors.join('; '));
+    }
+}
+/**
+ * Authoritative shape, row-count and per-row required checks for one table
+ * value (ADR 16 §3.7). Depth stops here on purpose: like the flat fields, only
+ * required / shape / row count are enforced server side; `minimum`, `maxLength`
+ * and friends stay a frontend hint.
+ */
+function validateSubmittedTableValue(field, value, required) {
+    const path = `formData.${field.fieldKey}`;
+    if (typeof value === 'undefined' || value === null) {
+        return required || readTableMinRows(field) > 0
+            ? [`${path} must include at least ${readTableRequiredRowCount(field)} row(s)`]
+            : [];
+    }
+    if (!Array.isArray(value)) {
+        return [`${path} must be an array of rows`];
+    }
+    const rowErrors = value.flatMap((row, index) => validateSubmittedTableRow(field, row, `${path}[${index}]`));
+    // A malformed row cannot be counted against min/max meaningfully; report the
+    // shape problem first so the submitter fixes the real cause.
+    if (rowErrors.length > 0) {
+        return rowErrors;
+    }
+    return validateSubmittedTableRowCount(field, value.length, required, path);
+}
+function validateSubmittedTableRowCount(field, rowCount, required, path) {
+    const minRows = required
+        ? Math.max(readTableMinRows(field), 1)
+        : readTableMinRows(field);
+    const maxRows = readTableMaxRows(field);
+    return [
+        ...(rowCount < minRows
+            ? [`${path} must include at least ${minRows} row(s)`]
+            : []),
+        ...(rowCount > maxRows
+            ? [`${path} must include at most ${maxRows} row(s)`]
+            : []),
+    ];
+}
+function validateSubmittedTableRow(field, row, path) {
+    if (!isRecord(row)) {
+        return [`${path} must be an object`];
+    }
+    const columnsByKey = new Map(field.columns.map((column) => [column.fieldKey, column]));
+    const unknownKeyErrors = Object.keys(row)
+        .filter((columnKey) => !columnsByKey.has(columnKey))
+        .map((columnKey) => `${path}.${columnKey} is not a table column`);
+    const cellErrors = Object.entries(row)
+        .filter(([columnKey, cellValue]) => columnsByKey.has(columnKey) && !(0, form_1.isFormTableCellValue)(cellValue))
+        .map(([columnKey]) => `${path}.${columnKey} is not a valid cell value`);
+    const requiredErrors = field.columns
+        .filter((column) => isSubmittedTableColumnRequired(column) &&
+        !isSubmittedFieldValuePresent(readRowCellValue(row, column.fieldKey)))
+        .map((column) => `${path}.${column.fieldKey} is required`);
+    return [...unknownKeyErrors, ...cellErrors, ...requiredErrors];
+}
+/**
+ * Own-property read. `constructor`, `toString` and friends all satisfy the
+ * column key identifier rule, so a plain index would walk the prototype chain
+ * and report an empty row as filled.
+ */
+function readRowCellValue(row, columnKey) {
+    return Object.prototype.hasOwnProperty.call(row, columnKey)
+        ? row[columnKey]
+        : undefined;
+}
+/**
+ * Columns carry no `requiredWhen`: row-scoped conditions are out of scope for
+ * V1 and the structural lint rejects them (ADR 16 §3.1).
+ */
+function isSubmittedTableColumnRequired(column) {
+    return column.required === true;
+}
+function readTableMinRows(field) {
+    return typeof field.minRows === 'number' ? field.minRows : 0;
+}
+function readTableMaxRows(field) {
+    return typeof field.maxRows === 'number' ? field.maxRows : form_1.FORM_TABLE_MAX_ROWS;
+}
+function readTableRequiredRowCount(field) {
+    return Math.max(readTableMinRows(field), 1);
 }
 function readFormDefinitionSnapshotSchema(snapshot) {
     const schema = snapshot.schema;
@@ -3663,6 +3769,13 @@ function evaluateFormConditionExpression(expression, fields, values, fallback) {
     if (!field) {
         return fallback;
     }
+    // A table value is a list of rows; comparing it with `==` / `>` has no
+    // meaning, and the structural lint already rejects cell-level references.
+    // Falling back keeps a table operand from silently deciding visibility
+    // through a string comparison against row records (ADR 16 §3.8).
+    if ((0, form_1.isTableFieldDefinition)(field)) {
+        return fallback;
+    }
     return evaluateFormConditionRule(rule, field, values[field.fieldKey]);
 }
 function parseFormConditionRule(expression) {