From 3c3566258b41f9ca71cc34d2dfaf6504250f6370 Mon Sep 17 00:00:00 2001
From: Merlin Beutlberger <m.beutlberger@sap.com>
Date: Wed, 22 Jul 2026 15:07:31 +0200
Subject: [PATCH] refactor(builder): Include component path in
 versionInfoGenerator logs

Several verbose and error logs in isBundledWithLibrary reported that a
component didn't declare 'sap.app/embeddedBy' or that its reference didn't
point to the library, without naming which component was meant. When a build
lists multiple embedded components, the message alone doesn't say which one.

Include the component path in the remaining log messages, matching the format
already used by the error branches. Update the affected assertions in the
processor and task tests.
---
 packages/builder/lib/processors/versionInfoGenerator.js   | 8 +++++---
 .../builder/test/lib/processors/versionInfoGenerator.js   | 8 ++++++++
 packages/builder/test/lib/tasks/generateVersionInfo.js    | 6 ++++--
 3 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/packages/builder/lib/processors/versionInfoGenerator.js b/packages/builder/lib/processors/versionInfoGenerator.js
index f7161ec399e..3edc80e8427 100644
--- a/packages/builder/lib/processors/versionInfoGenerator.js
+++ b/packages/builder/lib/processors/versionInfoGenerator.js
@@ -101,7 +101,7 @@ const processManifest = async (manifestResource) => {
  */
 const isBundledWithLibrary = (embeddedBy, componentPath, libraryPathPrefix) => {
 	if (typeof embeddedBy === "undefined") {
-		log.verbose("  Component doesn't declare 'sap.app/embeddedBy', don't list it as 'embedded'");
+		log.verbose(`  Component '${componentPath}' doesn't declare 'sap.app/embeddedBy', don't list it as 'embedded'`);
 		return false;
 	}
 	if (typeof embeddedBy !== "string") {
@@ -122,11 +122,13 @@ const isBundledWithLibrary = (embeddedBy, componentPath, libraryPathPrefix) => {
 		resolvedEmbeddedBy = resolvedEmbeddedBy + "/";
 	}
 	if ( libraryPathPrefix === resolvedEmbeddedBy ) {
-		log.verbose("  Component's 'sap.app/embeddedBy' property points to library, list it as 'embedded'");
+		log.verbose(
+			`  Component '${componentPath}': property 'sap.app/embeddedBy' points to library, list it as 'embedded'`);
 		return true;
 	} else {
 		log.verbose(
-			`  Component's 'sap.app/embeddedBy' points to '${resolvedEmbeddedBy}', don't list it as 'embedded'`);
+			`  Component '${componentPath}': property 'sap.app/embeddedBy' points to '${resolvedEmbeddedBy}', ` +
+			`don't list it as 'embedded'`);
 		return false;
 	}
 };
diff --git a/packages/builder/test/lib/processors/versionInfoGenerator.js b/packages/builder/test/lib/processors/versionInfoGenerator.js
index ff88226818d..5a69ac6c72d 100644
--- a/packages/builder/test/lib/processors/versionInfoGenerator.js
+++ b/packages/builder/test/lib/processors/versionInfoGenerator.js
@@ -355,6 +355,10 @@ test.serial("versionInfoGenerator library infos with embeds", async (t) => {
 	assertVersionInfoContent(t, oExpected, result);
 	t.is(t.context.infoLogStub.callCount, 0);
 	t.is(t.context.warnLogStub.callCount, 0);
+	t.is(t.context.verboseLogStub.callCount, 1);
+	t.is(t.context.verboseLogStub.getCall(0).args[0],
+		"  Component '/resources/lib/a/sub' doesn't declare 'sap.app/embeddedBy', " +
+		"don't list it as 'embedded'");
 });
 
 test.serial("versionInfoGenerator library infos with no embeds", async (t) => {
@@ -512,4 +516,8 @@ test.serial("versionInfoGenerator library infos with embeds and embeddedBy (hasO
 	assertVersionInfoContent(t, oExpected, result);
 	t.is(t.context.infoLogStub.callCount, 0);
 	t.is(t.context.warnLogStub.callCount, 0);
+	t.is(t.context.verboseLogStub.callCount, 1);
+	t.is(t.context.verboseLogStub.getCall(0).args[0],
+		"  Component '/resources/lib/a/sub': property 'sap.app/embeddedBy' points to library, " +
+		"list it as 'embedded'");
 });
diff --git a/packages/builder/test/lib/tasks/generateVersionInfo.js b/packages/builder/test/lib/tasks/generateVersionInfo.js
index edd608328e6..bedb63baf19 100644
--- a/packages/builder/test/lib/tasks/generateVersionInfo.js
+++ b/packages/builder/test/lib/tasks/generateVersionInfo.js
@@ -955,7 +955,8 @@ test.serial("integration: Library without dependencies and embeddedBy undefined"
 
 	t.is(verboseLogStub.callCount, 1);
 	t.is(verboseLogStub.firstCall.args[0],
-		"  Component doesn't declare 'sap.app/embeddedBy', don't list it as 'embedded'");
+		"  Component '/resources/lib/a/sub/fold' doesn't declare 'sap.app/embeddedBy', " +
+		"don't list it as 'embedded'");
 });
 
 test.serial("integration: Library without dependencies and embeddedBy not a string", async (t) => {
@@ -1098,7 +1099,8 @@ test.serial("integration: Library without dependencies and embeddedBy path not c
 
 	t.is(verboseLogStub.callCount, 1);
 	t.is(verboseLogStub.firstCall.args[0],
-		"  Component's 'sap.app/embeddedBy' points to '/resources/lib/a/sub/', don't list it as 'embedded'");
+		"  Component '/resources/lib/a/sub/fold': property 'sap.app/embeddedBy' points to " +
+		"'/resources/lib/a/sub/', don't list it as 'embedded'");
 });
 
 test.serial("integration: Library with manifest with invalid dependency", async (t) => {