Skip to content

[BUG FIX]: preserve field names without whitespace on decision table column… - #242

Open
cmurphy-navixio wants to merge 1 commit into
gorules:masterfrom
cmurphy-navixio:charlesm/fix-decision-table-import
Open

[BUG FIX]: preserve field names without whitespace on decision table column…#242
cmurphy-navixio wants to merge 1 commit into
gorules:masterfrom
cmurphy-navixio:charlesm/fix-decision-table-import

Conversation

@cmurphy-navixio

Copy link
Copy Markdown

When importing into a decision table node from Excel, the column field mapping in DecisionTableCommandBar.handleDataMapping it strips any characters outside the regex [a-zA-Z0-9\s._] and then lowercases the field mappings. In the case the field already has camel casing and has no spaces the result for input columns it breaks mappings for between the column and an input json path. A similar thing happens for the output columns

For example:

  • customer.riskScore → customer.riskscore (casing lost, so it no longer matches the actual input field)
  • Expressions containing characters like $ or [0] have those characters stripped entirely

As a result, re-importing a sheet could silently break the binding between columns and the fields they reference.

This fix only applies the normalization when the mapped value actually contains whitespace (i.e., it's a human-readable label that needs converting into a field name). If the trimmed value has nowhitespace, treat it as an already-valid field expression and use it as-is.

This applies to both input and output column mapping.

… import

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cmurphy-navixio cmurphy-navixio changed the title fix: preserve field names without whitespace on decision table column… [BUG FIX]: preserve field names without whitespace on decision table column… Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant