Skip to content

Stop linking the retired pipelines, and document the view builder as it really is reached - #28

Merged
oblomov-dev merged 4 commits into
mainfrom
claude/vigilant-sagan-d48gch-records
Sep 27, 2026
Merged

oblomov-dev merged 4 commits into
mainfrom
claude/vigilant-sagan-d48gch-records

Conversation

@oblomov-dev

@oblomov-dev oblomov-dev commented Sep 27, 2026 •

Copy link
Copy Markdown
Member

What changes

Links to the retired repositories. builder-abap2UI5-js is being archived; builder-cap2UI5, builder-cap2UI5-web and web-cap2UI5-build are archived and may be deleted. Nothing consumes their output: cap2ui5 depends on @abap2ui5/node-runtime from npm, which abap2UI5 builds itself.

  • guide/ecosystem.md, guide/where-it-comes-from.md: link cap2UI5's docs/adr/ and say what is true: archived or being archived, nothing consumes them.
  • AGENTS.md: the ground-truth table was split by a paragraph (the rows below it rendered as text). It is one table again, the builder row is gone, and a note says not to link the archived repositories.
  • HANDOVER.md, ROADMAP.md: history kept, dead links removed. ADR-006 and ADR-007 link their copies in cap2UI5/cap2UI5:docs/adr/, which exist on main today.
  • README.md: the old playground ran the retired port and may go offline; it now points at the Quickstart.

"Not importable" (AGENTS.md): importing @abap2ui5/node-runtime/output/… works (the package exports ./output/*), but it is unsupported and couples to transpiler output.

abap2UI5/abap2UI5#2802 is merged (2026-09-27, 5a1bd70) but not in 1.145.0 or any release, so the approuter still needs the extra route. Corrected in HANDOVER.md, guide/roadmap.md and reference/deployment.md, which all said "open".

The view builder is not reachable through c.raw. z2ui5_if_client has no reference to it. guide/views.md now says so and documents the supported path with an example class: an ABAP app transpiled against the hosted runtime (transpiler pinned with --save-exact, deps/open-abap-core pre-cloned, only your class copied into srv/abap/, loaded from one app module). It also covers the unsupported JS route through abap.Classes and the c.event() NUL-placeholder issue. guide/roadmap.md links to it. The zcl_my_app entry in .verify-refs-ignore has a wider reason, because the page's ?app_start=ZCL_MY_APP names an ABAP class rather than a defineApp app.

Verified

  • npm run check:ci is green against cap2UI5 main 003d293 and abap2UI5 main.
  • Checked against the published @abap2ui5/node-runtime@1.145.0 and cap2ui5@0.1.0: the exports map, abap2ui5.transpiler = 2.13.91, the README's abap_transpile.json, no builder reference in z2ui5_if_client, Z2UI5_CL_UI5_APP_HI_WORLD built with the builder, and the \u0000 event token in define-app.js.
  • Re-ran the ABAP recipe live in a fresh cds init --nodejs project (@sap/cds 9.9.3, Node 22):
    • --save-exact writes 2.13.91; without it, npm writes ^2.13.91.
    • The transpile reuses deps/open-abap-core and writes 747 files to output/. The app's file has no imports.
    • GET and POST on ?app_start=ZCL_MY_APP (and in lowercase) return 200. The edited value survives into a later roundtrip.
    • A misspelled builder method fails the transpile.
    • cds build --production carries srv/abap/ but not output/.
    • The startup hints do not list the app.
  • 5a1bd70 is not an ancestor of tag 1.145.0.
  • Not verified here: the c.event() NUL issue live through the builder. It is inferred from the placeholder in define-app.js.

The cap2UI5 PR that copies ADR-001 to ADR-004 and transpiler-roadmap.md is not merged yet. This PR names only paths that exist on cap2UI5 main, and HANDOVER.md says those records are "being copied".

🤖 Generated with Claude Code

https://claude.ai/code/session_01PWGynttXkP96LTv3YJKS4W

The builder repositories are archived or being archived and nothing consumes
their output. Link the decision records in cap2UI5/cap2UI5 docs/adr/ instead,
or keep the history as plain text. Fix the AGENTS.md ground-truth table that a
paragraph had split, and say that importing the runtime's output/ works but is
unsupported. abap2UI5/abap2UI5#2802 is merged (5a1bd70) but in no release yet,
so the approuter still needs the extra route.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PWGynttXkP96LTv3YJKS4W
z2ui5_if_client has no reference to z2ui5_cl_ui5_view_builder. Document the
supported way to use it — an ABAP app transpiled against the hosted runtime and
loaded from one app module — and why a JS app keeps template literals.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PWGynttXkP96LTv3YJKS4W
The class is the one the recipe was re-run with on cap2ui5 0.1.0 and
@abap2ui5/node-runtime 1.145.0: transpiled, loaded from srv/apps, state kept
across roundtrips.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PWGynttXkP96LTv3YJKS4W
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PWGynttXkP96LTv3YJKS4W
@oblomov-dev
oblomov-dev marked this pull request as ready for review September 27, 2026 20:08
@oblomov-dev
oblomov-dev merged commit fdee719 into main Sep 27, 2026
1 check passed
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.

2 participants