Skip to content

Gate the release numbers the docs name, regenerate the sample gallery - #18

Merged
oblomov-dev merged 3 commits into
mainfrom
claude/repo-structure-analysis-6t4t6x
Aug 31, 2026
Merged

Gate the release numbers the docs name, regenerate the sample gallery#18
oblomov-dev merged 3 commits into
mainfrom
claude/repo-structure-analysis-6t4t6x

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

Three commits from the 2026-08-30 ecosystem review:

  • verify-refs check 5: every three-part release number the docs mention must be the framework release the checkout actually pins (static version on z2ui5_if_app — the same place the prose points readers at), or an allowlisted entry with a reason (1.143.0 as upstream history, 1.113.0 as an OpenUI5 number). The pin is named in nine places across four pages; the day the port moves it, every one goes stale at once — now as a failing check with file and line. The pin itself is read from the checkout and deliberately never allowlisted.
  • samples.md regenerated and its check wired into CI: the page had already rotted the same way (upstream gained sample 497; the page said 104 apps / 97 in the gallery) because gen-samples --check hung in no workflow. Regenerated (105/98), and the check joins check:ci.
  • The architecture diagram's two crooked boxes are 31 columns like every other; timeout ceilings on the deploy jobs.

Validated: verify-refs green against the cap2UI5 checkout (positive and negative path), gen-samples --check green, full site build green.

🤖 Generated with Claude Code

https://claude.ai/code/session_01BuzK8EC2CdQ5PJ6YCdfKJc


Generated by Claude Code

claude added 3 commits August 30, 2026 21:34
…allery

Two rots, one pattern - a number in prose that nothing re-measures:

verify-refs learns check 5: every three-part release number the docs
mention must be the framework release the checkout actually pins
(static version on z2ui5_if_app - the same place the prose points its
readers at), or an allowlisted entry with a reason. The pin is named
in nine places across four pages today; the day the port moves it,
every one of them goes stale at once, and this makes that a failing
check with file and line instead of a discovery. 1.143.0 (the upstream
release where the bindings merged - history) and 1.113.0 (an OpenUI5
number) are allowlisted with their reasons; the pin itself is read
from the checkout and deliberately never listed.

docs/guide/samples.md had already rotted the same way: upstream gained
sample 497 and the page still said 104 apps / 97 in the gallery.
Regenerated (105/98), and gen-samples --check joins check:ci so the
next new sample fails the workflow instead of aging the page - the
generator existed, nothing ran its check in CI.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BuzK8EC2CdQ5PJ6YCdfKJc
abap2UI5 sets timeout-minutes on all of its jobs and most repositories
follow; this one had jobs without a ceiling, so a hung run could hold
its slot (and in the publish workflows, the downstream pipeline) for
GitHub's six-hour default. Values follow the job's real budget.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BuzK8EC2CdQ5PJ6YCdfKJc
Every box in the component diagram is 31 columns wide except the two
handler headers (35 and 30), whose right edges did not meet their own
bodies. Both are 31 now.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BuzK8EC2CdQ5PJ6YCdfKJc
Copilot AI lite review requested due to automatic review settings August 31, 2026 01:18

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 218b521 into main Aug 31, 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.

3 participants