refactor(builder): Run `generateVersionInfo` by default by maxreichmann · Pull Request #1453 · UI5/cli

maxreichmann · 2026-07-16T14:20:07Z

JIRA: CPOUI5FOUNDATION-1157

What has been done?

Exclusion of task generateVersionInforemoved
- now it will be executed by default for every build of project type application (default, jsdoc, self-contained)
Tests refactored accordingly
- +new test verifying exclusion/inclusion of generateVersionInfo: ProjectBuilder.integration.js:2758
Docs
- migrate-v5.md: new section explaining new behavior
- Builder.md: changed generateVersionInfo from disabled to enabled in Standard Tasks table

Info

for Server:
- task generateVersionInfo is explicitly NOT executed because middleware versionInfo IS executed (which creates sap-ui-version.json anyways)
- Test verifying exclusion of generateVersionInfo: BuildServer.integration.js:1030
- (existing) Test verifying sap-ui-version.json is generated: server/test/lib/server/main.js:204

matz3 · 2026-07-20T11:15:52Z

We should also check how this now relates to the versionInfo middleware. My assumption would be that we need to keep the middleware to serve the file for non-application projects (and when the task was skipped?). For application projects, we should be able to just serve the file from the application build output. This should at least be tested, confirmed and documented.

maxreichmann · 2026-07-22T10:14:36Z

Update:

generateVersionInfo is now excluded from server builds (for normal builds, it's executed by default).
Tests are added and existing ones are adjusted.
Docs are adjusted.
--> Main part is done and ready for review.

"Side tasks done": I'm also fine to move those changes out to another PR if this eases the review process.

matz3

"Side tasks done": I'm also fine to move those changes out to another PR if this eases the review process.

Yes, I think that would help with reviewing the docs changes. Thanks for looking into that!

maxreichmann · 2026-07-24T07:41:28Z

~~Removing the Draft state of this PR to check merge conflicts~~

~~❗️ Pls don't start a review yet :)~~

`+` Fix some related tests

`+` Adjust Standard Tasks table (TODO: check `transformBootstrapHTML` link!!)

`+` docs: Replace hardcoded URLs (ui5.github.io/..) with dynamic links `+` docs: Add links to all Builder tasks in Standard Tasks table `+` refactor: Remove "jsdoc" npm script (+remove unused dep `open-cli`)

(versionInfo middleware still runs -> sap-ui-version.json is generated) `+` add test verifying exclusion `+` adjust docs stating this exclusion

(from ProjectGraph.js to server.js) This logic will affect the `ui5 serve` command but not the internal BuildServer now.

maxreichmann · 2026-07-24T09:16:52Z

✅ Update: Rebased. CI problems now fixed. PR ready for review

This PR includes the outsourced "Side tasks done" from original PR #1453. Following changes have been made: * docs: Replace hardcoded URLs (ui5.github.io/..) with dynamic links * docs: Replace dead "API Reference" links * Guide the reader to use the new sidebar menu * Add "📚" emoji to "API" in sidebar to mimic depr. API Reference button * docs: Add links to all Builder tasks in Standard Tasks table * refactor(internal/documentation): Remove of npm script `jsdoc` (deprecated & not used in CI) * Remove now unused dep `open-cli`

Co-authored-by: Merlin Beutlberger <m.beutlberger@sap.com>

maxreichmann changed the title ~~refactor(builder): Remove default exclusion of "generateVersionInfo"~~ refactor(builder): Remove default exclusion of generateVersionInfo Jul 16, 2026

maxreichmann changed the title ~~refactor(builder): Remove default exclusion of generateVersionInfo~~ refactor(builder): Run generateVersionInfo by default Jul 20, 2026

maxreichmann force-pushed the run-generateVersionInfo-by-default branch 5 times, most recently from f354410 to b6ca355 Compare July 22, 2026 10:11

matz3 reviewed Jul 22, 2026

View reviewed changes

Comment thread internal/documentation/docs/pages/Builder.md Outdated

Comment thread internal/documentation/docs/pages/Builder.md Outdated

Comment thread packages/project/lib/graph/ProjectGraph.js Outdated

maxreichmann mentioned this pull request Jul 22, 2026

docs: Several refactorings #1473

Draft

maxreichmann force-pushed the run-generateVersionInfo-by-default branch 6 times, most recently from fe552c6 to da4d1f4 Compare July 23, 2026 10:20

maxreichmann mentioned this pull request Jul 24, 2026

refactor(builder): Run generateVersionInfo by default #1476

Closed

maxreichmann marked this pull request as ready for review July 24, 2026 07:41

maxreichmann added 10 commits July 24, 2026 10:34

refactor(builder): Remove default exclusion of "generateVersionInfo"

46e62a4

`+` Fix some related tests

test(project): Adjust ProjectBuilder tests

5d68006

test(project): Adjust BuildServer tests

3e635e3

docs: Add section to migrate-v5

3629c80

`+` Adjust Standard Tasks table (TODO: check `transformBootstrapHTML` link!!)

docs: Refactor migration text

42bdb07

docs: Replace dead "API ref" links with sidebar menu guide

f55dfec

`+` docs: Replace hardcoded URLs (ui5.github.io/..) with dynamic links `+` docs: Add links to all Builder tasks in Standard Tasks table `+` refactor: Remove "jsdoc" npm script (+remove unused dep `open-cli`)

test(project): Add test with exclusion of generateVersionInfo

05280f0

refactor(project): Exclude task generateVersionInfo from server builds

54cf77e

(versionInfo middleware still runs -> sap-ui-version.json is generated) `+` add test verifying exclusion `+` adjust docs stating this exclusion

refactor: Move "Side tasks done" out to dedicated PR (#1473)

1e086f5

docs: Add sentence for ui5 serve

ecd0b68

maxreichmann force-pushed the run-generateVersionInfo-by-default branch 2 times, most recently from 7c276e1 to 0a42367 Compare July 24, 2026 08:45

refactor: Move exclusion logic of generateVersionInfo

8827f4e

(from ProjectGraph.js to server.js) This logic will affect the `ui5 serve` command but not the internal BuildServer now.

maxreichmann force-pushed the run-generateVersionInfo-by-default branch from 0a42367 to 8827f4e Compare July 24, 2026 08:54

maxreichmann requested a review from a team July 24, 2026 09:17

RandomByte reviewed Jul 24, 2026

View reviewed changes

Comment thread packages/server/lib/server.js Outdated

refactor: Apply suggestion

59b0286

Co-authored-by: Merlin Beutlberger <m.beutlberger@sap.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

refactor(builder): Run `generateVersionInfo` by default#1453

refactor(builder): Run `generateVersionInfo` by default#1453
maxreichmann wants to merge 12 commits into
mainfrom
run-generateVersionInfo-by-default

maxreichmann commented Jul 16, 2026 •

edited

Loading

Uh oh!

matz3 commented Jul 20, 2026

Uh oh!

maxreichmann commented Jul 22, 2026

Uh oh!

matz3 left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

maxreichmann commented Jul 24, 2026 •

edited

Loading

Uh oh!

maxreichmann commented Jul 24, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

maxreichmann commented Jul 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

matz3 commented Jul 20, 2026

Uh oh!

maxreichmann commented Jul 22, 2026

Uh oh!

matz3 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

maxreichmann commented Jul 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

maxreichmann commented Jul 24, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

maxreichmann commented Jul 16, 2026 •

edited

Loading

maxreichmann commented Jul 24, 2026 •

edited

Loading