[oblomov-dev]

Four rounds of bug hunting in the layout editor. The bugs were found by:

• reading the code
• fuzzers over the generator, the XML and class import, and the model operations
• an independent review of the remaining files
• a random walk through the UI in Chromium (UI5 current and 1.71)
• random designs run through the real abap2UI5 runtime

Every fix has a test that fails without it.

Opening, importing, exporting

• A saved .design.json was read as an ABAP class: its indented lines start with ", which is an ABAP comment.
• A view or class that does not parse closed the dialog, and the pasted text was lost.
• "Create attributes" after an import looped forever on a binding path longer than 30 characters.
• A class renamed in the editor:
  • kept its old CLASS name;
  • combined with a new attribute, its declaration was written into the middle of a line.
• 7.02 classes:
  • a builder variable other than view was taken for 7.40;
  • the chain head was overwritten with an undeclared view =.
• A view built on after an IF was half imported, and the export called a variable it no longer declared. Such a class is now refused with the reason.
• A new WHEN could land in a nested CASE, and WHEN \A` OR `B`` counted only A.
• VALUE \Walldorf, DE`, VALUE 5 READ-ONLYand=>` in seed rows lost their values.
• Initial dates and times in seed rows came back as 0000-00-00 / 00:00:00.
• The command palette's Copy ABAP regenerated an imported class and dropped the author's code.
• Share link: a large design overflowed the stack.

Generator and canvas

• Texts of backticks gave lines over 255 characters, and emoji could be torn in two at a cut.
• Line breaks and tabs showed as spaces on the canvas. escapeXml now escapes as c_escape_xml does.
• The xmlns order depended on the order the view was built in, so the same view could give different classes.
• Things the canvas showed differently from what abap2UI5 sends, all confirmed on the transpiled runtime:
  • midnight;
  • decimals like .5 and 1e3;
  • -0;
  • control characters, which are now also never written into the class.
• After a failed render, the next view failed with "duplicate id". Each view now gets an id that is never reused.
• The preview accepted render messages from any window. It now takes them only from the editor that frames it.
• A view with two root elements dropped the second one silently.

Editing

• Controls could be dropped into the grid table's rows, which UI5 refuses.
• A new Wizard had fewer than the 3 steps UI5 requires.
• Static values UI5 cannot parse are now reported in Checks:
  • an error for int, float and enum, because they fail the view;
  • a warning for a CSS size (checked with the 1.71 pattern) or a flag;
  • the property field turns red as it is typed.
• Binding a table again, or "+ Table", replaced hand-built columns without asking.
• A table of only table fields bound a text to a table field.
• An edit that threw half way left the design half changed.
• Focusing an event field with Tab named the event.
• A field switched between table and scalar kept stale rows.
• Attribute names on_event, view_display, model_init and main clashed with the class's methods.
• Undo merged the first keystrokes after New or Open.
• parseClassName accepted sap.m.; tools/serve.mjs served sibling folders.

Small improvements

• Attributes named in ƒx expressions count as used and are renamed with them.
• Ctrl+S saves the design.
• Enter submits in several fields, and Escape closes menus.
• A pasted #d= link opens in an editor that is already open.
• The outline shows #field bindings.

Checks

• npm test: 377 tests.
• npm run check:abap: 23 files, 0 failing.
• npm run check:abap702: 0 issues.
• npm run verify:abap: 23/23 classes.
• npm run test:e2e:local: 55/55 tests on UI5 1.144 and 1.71.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NJdCHJULfqVBdruCtKNUm2

---

Generated by Claude Code