Skip to content

##NO_HANDLER on app 484, and the thumbnail gaps re-measured - #69

Merged
oblomov-dev merged 2 commits into
mainfrom
claude/viele-kleine-verbesserungen-9lzh6d
Sep 4, 2026
Merged

##NO_HANDLER on app 484, and the thumbnail gaps re-measured#69
oblomov-dev merged 2 commits into
mainfrom
claude/viele-kleine-verbesserungen-9lzh6d

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

##NO_HANDLER on app 484

The TRY around the two component-parameter reads is there because the app runs with or without them — a missing PRODUCT/QUANTITY leaves the fields initial and the app carries on. SLIN reports an empty CATCH on every real system, and the pragma is how ABAP says the handler is meant to be empty; src/00/00's context class already writes it that way in four places. Found by running abap2UI5/linter's main over the corpus (its empty-catch-block rule is newer than the pinned 0.6.1).

The thumbnail gaps, re-measured

AGENTS.md carried 18 of 31 app views render from the 2026-08 measurement, and the corpus has grown since. Re-run with the same harness the deploy uses (screenshotFiles( ) over src at the pinned runtime): 19 of the 32 app classes photograph, 24 of the 37 documents they build.

The interesting half is that the failure set did not move at all — the same thirteen classes, in the same three categories the first measurement named, each still failing for the reason it named:

  • seven sap.ui.comp Smart Controls the harness's OpenUI5 runtime does not carry — SmartFilterBar (313, 477), SmartForm (475, 476), SmartVariantManagement (478, 493), SmartChart (479)
  • three z2ui5.cc controls that do not load headless — Websocket (489), MIME-audio (487), Smart Multi Input (319)
  • the three RAP samples whose ObjectStatus gets an empty state from the mock model (005, 006, 010)

They are written out per class now, so the next re-measurement can tell "the same gaps" from "different gaps that happen to count the same".

Checks

npm run check green: abaplint, abap2ui5lint (32 classes, 37 views, render gate on, 0 findings), overview/keywords/abapdoc/samples-md/catalogue/app-rules/prose/derived all up to date.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CD8XM2xmi9NHb17JHP67nZ


Generated by Claude Code

The TRY around the two component-parameter reads is there because the app
runs with or without them - a missing PRODUCT/QUANTITY leaves the fields
initial and the app carries on. SLIN reports an empty CATCH on every real
system, and the pragma is how ABAP says the handler is meant to be empty;
`src/00/00`'s context class already writes it that way in four places.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CD8XM2xmi9NHb17JHP67nZ
AGENTS.md carried "18 of 31 app views render" from the 2026-08 measurement,
and the corpus has grown since. Re-run with the same harness the deploy uses
(`screenshotFiles( )` over `src` at the pinned runtime): 19 of the 32 app
classes photograph, 24 of the 37 documents they build.

The interesting half is that the FAILURE set did not move at all - the same
thirteen classes, in the same three categories the first measurement named,
and every one of them still fails for the reason it named: seven sap.ui.comp
Smart Controls (SmartFilterBar 313/477, SmartForm 475/476,
SmartVariantManagement 478/493, SmartChart 479) that the harness's OpenUI5
runtime does not carry, three z2ui5.cc controls that do not load headless
(Websocket 489, MIME-audio 487, Smart Multi Input 319), and the three RAP
samples whose ObjectStatus gets an empty `state` from the mock model
(005/006/010). The categories are written out per class now, so the next
re-measurement can tell "the same gaps" from "different gaps that happen to
count the same".

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CD8XM2xmi9NHb17JHP67nZ
Copilot AI lite review requested due to automatic review settings September 4, 2026 22:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@oblomov-dev
oblomov-dev merged commit 857e971 into main Sep 4, 2026
22 checks passed
@oblomov-dev
oblomov-dev deleted the claude/viele-kleine-verbesserungen-9lzh6d branch September 4, 2026 22:23
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.

3 participants