feat: Incremental Build by RandomByte · Pull Request #1267 · UI5/cli

RandomByte · 2026-01-07T12:28:05Z

Implementation of RFC 0017 Incremental Build

This PR supersedes previous PoC: #1238

JIRA: CPOUI5FOUNDATION-1174

coveralls · 2026-01-07T12:30:25Z

coverage: 99.566% (+4.9%) from 94.658%
when pulling b727ebc on feat/incremental-build-4
into cb29ec1 on main.

…ters Cherry-picked from SAP/ui5-fs@5651627 JIRA: CPOUI5FOUNDATION-1174

Cherry-picked from: SAP/ui5-fs@5651627 JIRA: CPOUI5FOUNDATION-1174

Cherry-picked from: SAP/ui5-builder@ef5a3b2 JIRA: CPOUI5FOUNDATION-1174

Cherry-picked from: SAP/ui5-project@82b20ee JIRA: CPOUI5FOUNDATION-1174

Prerequisite for versioning support Cherry-picked from: SAP/ui5-project@83b5c4f JIRA: CPOUI5FOUNDATION-1174

Cherry-picked from: SAP/ui5-project@cb4e858 JIRA: CPOUI5FOUNDATION-1174

Cherry-picked from: SAP/ui5-cli@d29ead8 JIRA: CPOUI5FOUNDATION-1174

* Improve handling for concurrent resource access and modifications, especially when buffering streams. * Deprecate getStatInfo in favor of dedicated getSize, isDirectory, getLastModified methods. * Deprecate synchronous getStream in favor of getStreamAsync and modifyStream, allowing for atomic modification of resource content * Generate Resource hash using ssri

…d library resources `+` Adjust FixtureTester(BuildServer.integration.js)

`+` Add cases for OmitFromBuildResult tagging `+` Add cases for components (copied "node_modules" folder from application.a) `+` Adjust tests for BuildContext.js to work with new logic

…tures folders

`+` Mark module test as incorrect with comment

…tual fixtures folders" This reverts commit 806ab4d.

… 'supportsDifferentialBuilds' For better consistency

…-task project types (e.g. modules) `+` Fix module test with now new assertions

´+´ style: Apply same line padding: (within a build stage: 1 empty line; between two build stages: 2 empty lines)

RandomByte · 2026-02-18T10:07:32Z

packages/project/test/fixtures/application.a/custom-tasks/custom-task-2.js

+	const test2JS = await workspace.byPath(`/resources/${projectNamespace}/test2.js`);
+
+	const tag = taskUtil.getTag(testJS, taskUtil.STANDARD_TAGS.IsDebugVariant);
+	if (!test2JS && testJS) {


Why is this if necessary? Both bodies seem identical

Yeah, my only motive behind this was to have two seperate logic blocks (one for each build) and to have it consistent with custom-task-0.js (for easier understanding). But I'll refactor it to only one.

Done in 03d0f0e

RandomByte · 2026-02-18T10:08:22Z

packages/project/test/fixtures/application.a/custom-tasks/custom-task-0.js

+	} else {
+		// For #3 Build (NEW behavior expected as in #1):
+		if (tag) {
+			console.log("Tag set by custom-task-1 is present in custom-task-0 now, as EXPECTED.");


This is not expected. A task must never see tags only set by later tasks

The tag for test.js is set by custom-task-1 in #1 Build.
In, #3 Build we then only touch test2.js, which skips execution of custom-task-1 (it's not an input of it). But should custom-task-0 still not be able to see the tag? What's then the purpose of the cache?

We need to make sure that tasks always behave the same, no matter whether other tasks are skipped (by using a valid cache) or not.

This means that a custom-task-0, which can't find a tag for resource test.js in the first build, must never find a tag for that resource in any other build.

Otherwise you could end up with different build results, depending on whether you build your project with or without cache. This must to be avoided.

Got it now. That's definitely the right way.
I'll refactor the tests now.

Done in 03d0f0e

`+` Fix logic for preceding tasks being able to read tags `+` Clean-up custom-task code

RandomByte marked this pull request as draft January 7, 2026 12:28

RandomByte mentioned this pull request Jan 7, 2026

feat: Incremental Build #1238

Closed

RandomByte force-pushed the feat/incremental-build-4 branch 3 times, most recently from 5224cd2 to 4904c84 Compare January 9, 2026 09:22

RandomByte force-pushed the feat/incremental-build-4 branch from 950fc6d to 41eed91 Compare January 14, 2026 15:28

maxreichmann force-pushed the feat/incremental-build-4 branch from bb39565 to 2a21507 Compare January 20, 2026 10:01

RandomByte force-pushed the feat/incremental-build-4 branch 3 times, most recently from 6233816 to f858659 Compare January 20, 2026 16:58

RandomByte mentioned this pull request Jan 21, 2026

[RFC] 0017 Incremental Build #1036

Open

RandomByte force-pushed the feat/incremental-build-4 branch from 71db1d0 to a2c371f Compare January 26, 2026 09:54

RandomByte added 17 commits February 9, 2026 14:30

feat(fs): Enhance API for incremental build, add tracking readers/wri…

6633686

…ters Cherry-picked from SAP/ui5-fs@5651627 JIRA: CPOUI5FOUNDATION-1174

feat(server): Use incremental build in server

1ef1b89

Cherry-picked from: SAP/ui5-fs@5651627 JIRA: CPOUI5FOUNDATION-1174

feat(builder): Adapt tasks for incremental build

efc0d0d

Cherry-picked from: SAP/ui5-builder@ef5a3b2 JIRA: CPOUI5FOUNDATION-1174

refactor(project): Align getReader API internals with ComponentProjects

2ad5db1

Cherry-picked from: SAP/ui5-project@82b20ee JIRA: CPOUI5FOUNDATION-1174

refactor(project): Refactor specification-internal workspace handling

c1fc387

Prerequisite for versioning support Cherry-picked from: SAP/ui5-project@83b5c4f JIRA: CPOUI5FOUNDATION-1174

refactor(project): Implement basic incremental build functionality

a1e5512

Cherry-picked from: SAP/ui5-project@cb4e858 JIRA: CPOUI5FOUNDATION-1174

refactor(cli): Use cache in ui5 build

579963f

Cherry-picked from: SAP/ui5-cli@d29ead8 JIRA: CPOUI5FOUNDATION-1174

refactor(project): Use cacache

03c204c

refactor(project): Add cache manager

b13447f

refactor(fs): Provide createBuffer factory in FileSystem adapter

0cf61dc

refactor(project): Refactor cache classes

766be40

refactor(fs): Add Proxy reader

cc54a0e

refactor(project): API refactoring

375e18f

refactor(builder): Rename task param 'buildCache' to 'cacheUtil'

0aeddf2

refactor(project): Cleanup

a5e7dda

refactor(project): Move resource comparison to util

c9ade5e

RandomByte and others added 16 commits February 9, 2026 14:31

test(project): Update ProjectBuildCache and TaskBuildCache tests

002367c

test(project): Update ProjectBuilder tests and JSDoc

371bdb2

test(project): Update TaskRunner tests

85eac90

test(project): Update various tests

e4a526d

test(project): Add missing comma

66daff1

refactor(project): Improve abort signal handling

8ad4888

refactor(project): Fix additional tests

24254bf

refactor(project): Move dependency indice init into PBC

06c9df0

fix(project): Improve BuildServer stability on resource changes

dfa9a6b

test(project): Add case for BuildServer which requests application an…

d6f6fba

…d library resources `+` Adjust FixtureTester(BuildServer.integration.js)

test(project): Add cases

421b5ad

`+` Add cases for OmitFromBuildResult tagging `+` Add cases for components (copied "node_modules" folder from application.a) `+` Adjust tests for BuildContext.js to work with new logic

test(project): Add cases for project type "module" (ProjectBuilder)

5bddb1c

refactor(project): ProjectBuildCache state handling

6118277

test(project): Update node_modules deps to be aligned with actual fix…

806ab4d

…tures folders

test(project): Add "library" dependencies to "module" fixture

433f7ee

`+` Mark module test as incorrect with comment

test(project): Update ProjectBuildContext tests

cf43f0c

RandomByte force-pushed the feat/incremental-build-4 branch from 7364b4b to cf43f0c Compare February 9, 2026 13:32

maxreichmann and others added 6 commits February 10, 2026 10:37

Revert "test(project): Update node_modules deps to be aligned with ac…

f2061e7

…tual fixtures folders" This reverts commit 806ab4d.

refactor(project): Rename task param 'supportsDifferentialUpdates' =>…

56e6dd0

… 'supportsDifferentialBuilds' For better consistency

test(project): Extend FixtureTester (ProjectBuilder) to work with non…

590612e

…-task project types (e.g. modules) `+` Fix module test with now new assertions

test(project): Add cases for ui5.yaml path mapping (Modules)

549d577

test(project): Add race condition test

f1d4bff

test(project): Add modify file case for modules

a263757

´+´ style: Apply same line padding: (within a build stage: 1 empty line; between two build stages: 2 empty lines)

maxreichmann force-pushed the feat/incremental-build-4 branch from 7da465c to 252b966 Compare February 16, 2026 17:14

test(project): Add case for multiple custom tasks (tag handling)

874a943

maxreichmann force-pushed the feat/incremental-build-4 branch from 252b966 to 874a943 Compare February 16, 2026 17:17

RandomByte commented Feb 18, 2026

View reviewed changes

matz3 and others added 3 commits February 18, 2026 14:15

fix(project): Fix build manifest access

cd35589

test(project): Fix multiple-task tests (Address review)

03d0f0e

`+` Fix logic for preceding tasks being able to read tags `+` Clean-up custom-task code

test(project): Re-Add eslint rule (removed by accident)

ee89736

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Incremental Build#1267

feat: Incremental Build#1267
RandomByte wants to merge 153 commits intomainfrom
feat/incremental-build-4

RandomByte commented Jan 7, 2026

Uh oh!

coveralls commented Jan 7, 2026 •

edited

Loading

Uh oh!

RandomByte Feb 18, 2026

Uh oh!

maxreichmann Feb 18, 2026

Uh oh!

maxreichmann Feb 18, 2026

Uh oh!

RandomByte Feb 18, 2026

Uh oh!

maxreichmann Feb 18, 2026

Uh oh!

RandomByte Feb 18, 2026

Uh oh!

maxreichmann Feb 18, 2026

Uh oh!

maxreichmann Feb 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Comments

Conversation

RandomByte commented Jan 7, 2026

Uh oh!

coveralls commented Jan 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Comments

coveralls commented Jan 7, 2026 •

edited

Loading