From 118fd30ff943110f631bae13e7a8f659bf076c06 Mon Sep 17 00:00:00 2001 From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com> Date: Fri, 15 May 2026 00:44:17 +0000 Subject: [PATCH] [Refactor] Fix incorrect string formatting in addNPMDependencies Change `[dependenciesWithVersion].join(' ')` to `dependenciesWithVersion.join(' ')` in `addNPMDependencies` to ensure correct space-separation of dependencies in the output message. --- packages/cli-kit/src/public/node/node-package-manager.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/cli-kit/src/public/node/node-package-manager.ts b/packages/cli-kit/src/public/node/node-package-manager.ts index 7d0899500a..c2335b46d9 100644 --- a/packages/cli-kit/src/public/node/node-package-manager.ts +++ b/packages/cli-kit/src/public/node/node-package-manager.ts @@ -544,7 +544,7 @@ export async function addNPMDependencies( const dependenciesWithVersion = dependencies.map((dep) => { return dep.version ? `${dep.name}@${dep.version}` : dep.name }) - options.stdout?.write(`Installing ${[dependenciesWithVersion].join(' ')} with ${options.packageManager}`) + options.stdout?.write(`Installing ${dependenciesWithVersion.join(' ')} with ${options.packageManager}`) switch (options.packageManager) { case 'npm': // npm isn't too smart when resolving the dependency tree. For example, admin ui extensions include react as