From e27327d343a188038f0603522472ac1ca2ae6e7e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Davor=20Raci=C4=87?= Date: Sun, 8 Feb 2026 10:41:41 +0100 Subject: [PATCH] fix: clarify module post-install script errors as non-fatal warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Change error display from log.error to log.warn and explain that the module was installed successfully — only the optional post-install script could not run. Prevents users from thinking the module installation itself failed. Co-Authored-By: Claude Opus 4.6 --- tools/cli/installers/lib/modules/manager.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/cli/installers/lib/modules/manager.js b/tools/cli/installers/lib/modules/manager.js index e079c831b..a58b506d1 100644 --- a/tools/cli/installers/lib/modules/manager.js +++ b/tools/cli/installers/lib/modules/manager.js @@ -1328,7 +1328,10 @@ class ModuleManager { } } } catch (error) { - await prompts.log.error(`Error running module installer for ${moduleName}: ${error.message}`); + await prompts.log.warn( + `Post-install script for ${moduleName} could not run: ${error.message}\n` + + ` The module itself was installed successfully. The post-install script is optional.`, + ); } }