Skip to content

fix: nested SpEL record when switching Kafka sink raw editor mode#9241

Closed
qoomak wants to merge 1 commit intoimprove/data-mapper-automap-fixfrom
improve/data-mapper-raw-editor-fix
Closed

fix: nested SpEL record when switching Kafka sink raw editor mode#9241
qoomak wants to merge 1 commit intoimprove/data-mapper-automap-fixfrom
improve/data-mapper-raw-editor-fix

Conversation

@qoomak
Copy link
Copy Markdown
Member

@qoomak qoomak commented Apr 7, 2026

Summary

  • Fixes switching between structured form and raw editor mode in Kafka sink node
  • Dynamic → raw editor: groups dotted param names (combinedState.key, combinedState.tool_record) into proper nested SpEL structure instead of flat dotted keys — flat keys are invalid SpEL and caused type mismatch / validation errors
  • Raw editor → dynamic: handles both old flat format and new nested Value expression when distributing back to individual params; normalizes indentation of embedded SpEL records

Test plan

  • Open a Kafka Avro sink with dotted params (e.g. combinedState.key, combinedState.tool_record)
  • Toggle Raw editor ON — Value expression should be nested: { combinedState: { key: ..., tool_record: {...} }, modelOutput: { value: ... } }
  • Toggle Raw editor back OFF — individual params should be restored with correct expressions and clean 2-space indentation
  • Verify no validation errors ("only string keys supported") on the Value expression in raw editor mode

…tor mode

Dynamic → raw editor: group dotted param names (e.g. combinedState.key,
combinedState.tool_record) into nested structure instead of flat dotted keys.
Flat keys are not valid SpEL map keys and caused type mismatch errors.

Raw editor → dynamic: handle both old flat and new nested Value expressions
when distributing back to individual params. Normalize indentation of embedded
SpEL records.
@github-actions github-actions bot added ui client client main fe labels Apr 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

🔴 Cypress snapshot PR created: #9243
Merge it into your branch to update snapshots.
⚠️ Be careful! Snapshot changes are not necessarily the cause of the error. Check the logs.

@qoomak
Copy link
Copy Markdown
Member Author

qoomak commented Apr 9, 2026

Zastąpiony przez #9261

@qoomak qoomak closed this Apr 9, 2026
@qoomak
Copy link
Copy Markdown
Member Author

qoomak commented Apr 9, 2026

Superseded by #9261

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client client main fe ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant