From 0c02a412ccdb367659175639744fa84fe001cd35 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Tue, 22 Sep 2026 00:37:23 +0000 Subject: [PATCH] [Refactor] Use declarative map in calculateLocalChecksums Replace the forEach-and-push accumulation with a direct map over the theme file system entries. Behavior is unchanged: Map iteration order is preserved and the size computation is identical. Co-Authored-By: Claude Opus 4.8 --- .../theme/src/cli/utilities/theme-uploader.ts | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/packages/theme/src/cli/utilities/theme-uploader.ts b/packages/theme/src/cli/utilities/theme-uploader.ts index 6a4254cf248..ba7ace7365e 100644 --- a/packages/theme/src/cli/utilities/theme-uploader.ts +++ b/packages/theme/src/cli/utilities/theme-uploader.ts @@ -396,21 +396,13 @@ function createBatches(files: T[]): T[][] { } function calculateLocalChecksums(localThemeFileSystem: ThemeFileSystem): ChecksumWithSize[] { - const checksums: ChecksumWithSize[] = [] - - localThemeFileSystem.files.forEach((file, key) => { + return Array.from(localThemeFileSystem.files.entries()).map(([key, file]) => ({ + key, + checksum: file.checksum, // Text files: use UTF-8 byte count // Binary files: use base64 length - const size = file.value ? Buffer.byteLength(file.value, 'utf8') : (file.attachment?.length ?? 0) - - checksums.push({ - key, - checksum: file.checksum, - size, - }) - }) - - return checksums + size: file.value ? Buffer.byteLength(file.value, 'utf8') : (file.attachment?.length ?? 0), + })) } async function uploadBatch(