From 7c827e63e134dcf80647d5b55977bc8c427f1919 Mon Sep 17 00:00:00 2001 From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com> Date: Sun, 31 May 2026 00:17:45 +0000 Subject: [PATCH] [Refactor] Use uniq helper in copyConfigKeyEntry Follows the repository's convention of using the `uniq` helper from `@shopify/cli-kit/common/array` for array deduplication instead of manual `Set` spreads. --- .../build/steps/include-assets/copy-config-key-entry.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/app/src/cli/services/build/steps/include-assets/copy-config-key-entry.ts b/packages/app/src/cli/services/build/steps/include-assets/copy-config-key-entry.ts index c952cd8abf1..245466096d3 100644 --- a/packages/app/src/cli/services/build/steps/include-assets/copy-config-key-entry.ts +++ b/packages/app/src/cli/services/build/steps/include-assets/copy-config-key-entry.ts @@ -1,4 +1,5 @@ import {assertPathWithinAppDir} from './assert-path-within-app.js' +import {uniq} from '@shopify/cli-kit/common/array' import {joinPath, basename, relativePath, extname} from '@shopify/cli-kit/node/path' import {glob, copyFile, copyDirectoryContents, fileExists, mkdir, isDirectory} from '@shopify/cli-kit/node/fs' import {outputContent, outputDebug, outputToken} from '@shopify/cli-kit/node/output' @@ -66,7 +67,7 @@ export async function copyConfigKeyEntry(config: { // Deduplicate: the same source path shared across multiple targets // should only be copied once; the pathMap entry is reused for all references. - const uniquePaths = [...new Set(paths)] + const uniquePaths = uniq(paths) // Process sequentially to avoid filesystem race conditions on shared output paths. const pathMap = new Map()