Refuse a sample no catalogue can reach, and seven samples for capabilities it never showed

[oblomov-dev]

Two topics in one branch, against AGENTS.md's "one topic per PR". They were meant to be two pull requests, and I only have one branch here. Say the word and I will split the second commit onto a branch of its own — the two are independent and the split is a git reset away.

Both come out of the same audit: the catalogue was measured against z2ui5_if_client, once for what it lists and once for what it teaches.

---

1 — 1186f83 the gate: every sample is a tile

Every gate here counts tiles, and they all read the same scan — so a sample scanSamples( ) cannot place is invisible to all of them at once. That is how the fourteen samples of #843 sat in the src/ root package for two commits: in no overview app, in no SAMPLES.md row, with thirteen green gates.

The two skips that produce an unplaceable class were bare continues (a class directly in src/, a class in a top-level package that is not an area). They now push onto orphans, and scripts/check-orphan-samples.mjs refuses them.

Measured — a class planted in the root package, nothing removed, so no catalogue goes stale:

Gate
check:launchpad / check:catalogue / check:keywords / check:agents	green — blind to it
check:orphans	red — caught it

Being skipped by a rule stays fine: the overview app is dropped by name, a ZZZ helper comes back under hidden, a src/00/98 app is scanned and listed. What is refused is a class skipped by where its file happens to sit.

2 — a7e3763 seven samples

513 Inline CSS on a Control	the css pseudo-method of control_by_id — one whitelisted declaration onto the control's own DOM node, for a value the control has no property for
514 Aggregation Item by Index	<id>/<aggregation>/<index> — the only way to reach a control cloned from a bound template, since UI5 mints its id from the runtime view prefix
515 The Global Busy Indicator	control_global BUSY_INDICATOR — a singleton renders nothing and has no id, so a global target is the only wire that reaches it
516 Keyboard Layout of an Input	the bound inputMode property of z2ui5.cc.InputExt — the keyboard is model data, not an action
517 Upload with an UploadSet	z2ui5.cc.UploadSetExt reads each added file into bound properties, so sap.m.upload.UploadSet works without the upload endpoint it expects
518 Register an Icon Font	control_global ICON_POOL registerFont. Without it the URI renders no glyph and logs nothing, which is the whole symptom
519 Basics VII — Translatable Texts	ABAP's own translation instead of an i18n file, with a real TPOOL in the abapGit sidecar

Three capabilities on the audit list are deliberately NOT here. INVISIBLE_MESSAGE (@SInCE 1.78), POPUP setWithinArea (1.89) and FORMATTING set/addCustomCurrencies (1.120) are all above this repository's 1.71 floor, and the packages that used to hold a release-restricted sample are gone (§1). They belong in samples-stack or nowhere — not in src/01 with a disclaimer.

The three render-error excludes

516, 517 and 518 join that list with reasons, in the two groups the file already draws:

• 516/517 name a companion control the linter's harness does not mirror — only MultiInputExt and MessageManager are, in lib/cc-controls.mjs. The real fix is upstream in @abap2ui5/linter.
• 518 hits the same boundary one step out: the harness reconstructs the view and never executes a frontend action, so the font registration cannot have happened while it renders. abap2UI5/samples-controls carries the identical case as its app 350.

Measured rather than assumed: swapping the control name inside app 074, which is already on the list, shows the harness is indifferent to which cc control it is — and even the mirrored MultiInputExt fails from a file that is not excluded.

Verified

abaplint 0 issues at v750 and Cloud; the 702 downport 0 issues in a throwaway worktree (the text pool and the text symbols survive it unchanged); abap2ui5-linter 166 files, 0 failing; npm run check green; and every new file checked for the abapGit round trip — no BOM on .abap, BOM on .xml, LF, final newline, no trailing whitespace, no line over 255.

Catalogue 113 → 120 tiles; README, SAMPLES.md, catalogue.json, catalogue-derived.json and the overview app regenerated, and AGENTS.md's category table now reads Basics I … VII.

Merge order

abap2UI5/docs#322 declares the samples: frontmatter these seven " @docs lines pair with. It has to merge first — until it does, check-docs-links fails here, because it reads the docs checkout at main.

🤖 Generated with Claude Code

https://claude.ai/code/session_019rmo8XKoBEpBRZ2TZMUQrF