fix theme package listings issue

Author: EvilGenius13

packages/cli-kit/src/public/node/archiver.integration.test.ts

@@ -26,7 +26,8 @@ describe('zip', () => {
 
       // Then
       const archiveEntries = await readArchiveFiles(zipPath)
-      expect(structure.sort()).toEqual(archiveEntries.sort())
+      const expectedEntries = withExplicitDirectoryEntries(structure)
+      expect(expectedEntries.sort()).toEqual(archiveEntries.sort())
     })
   })
 
@@ -49,7 +50,8 @@ describe('zip', () => {
 
       // Then
       const archiveEntries = await readArchiveFiles(zipPath)
-      expect([`extensions/first/main.js`]).toEqual(archiveEntries)
+      const expectedEntries = withExplicitDirectoryEntries([`extensions/first/main.js`])
+      expect(expectedEntries.sort()).toEqual(archiveEntries.sort())
     })
   })
 })
@@ -157,3 +159,19 @@ async function readArchiveFiles(zipPath: string) {
 
   return archiveEntries
 }
+
+function withExplicitDirectoryEntries(files: string[]): string[] {
+  const entries = new Set<string>()
+  for (const file of files) {
+    entries.add(file)
+    let currentDir = dirname(file)
+    while (currentDir && currentDir !== '.' && currentDir !== '/') {
+      const dirEntry = currentDir.endsWith('/') ? currentDir : `${currentDir}/`
+      entries.add(dirEntry)
+      const parent = dirname(currentDir)
+      if (parent === currentDir) break
+      currentDir = parent
+    }
+  }
+  return Array.from(entries)
+}

packages/cli-kit/src/public/node/archiver.ts

@@ -1,4 +1,4 @@
-import {relativePath, joinPath} from './path.js'
+import {relativePath, joinPath, dirname} from './path.js'
 import {glob, removeFile} from './fs.js'
 import {outputDebug, outputContent, outputToken} from '../../public/node/output.js'
 import archiver from 'archiver'
@@ -52,16 +52,41 @@ export async function zip(options: ZipOptions): Promise<void> {
     })
     archive.pipe(output)
 
+    // Find parent directories to add explicitly to the archive
+    const directoriesToAdd = new Set<string>()
     for (const filePath of pathsToZip) {
-      const fileRelativePath = relativePath(inputDirectory, filePath)
-      archive.file(filePath, {name: fileRelativePath})
+      const relPath = relativePath(inputDirectory, filePath)
+      collectParentDirectories(relPath, directoriesToAdd)
+    }
+
+    // Add directories, parents before children
+    const sortedDirs = Array.from(directoriesToAdd).sort((left, right) => left.localeCompare(right))
+    for (const dir of sortedDirs) {
+      const dirName = dir.endsWith('/') ? dir : `${dir}/`
+      archive.append(Buffer.alloc(0), {name: dirName})
+    }
+
+    // Add files
+    for (const filePath of pathsToZip) {
+      const rel = relativePath(inputDirectory, filePath)
+      if (filePath && rel) archive.file(filePath, {name: rel})
     }
 
     // eslint-disable-next-line @typescript-eslint/no-floating-promises
     archive.finalize()
   })
 }
 
+function collectParentDirectories(fileRelativePath: string, accumulator: Set<string>): void {
+  let currentDir = dirname(fileRelativePath)
+  while (currentDir && currentDir !== '.' && currentDir !== '/') {
+    accumulator.add(currentDir)
+    const parent = dirname(currentDir)
+    if (parent === currentDir) break
+    currentDir = parent
+  }
+}
+
 export interface BrotliOptions {
   /**
    * The directory to compress.
@@ -133,7 +158,7 @@ export async function brotliCompress(options: BrotliOptions): Promise<void> {
           // eslint-disable-next-line @typescript-eslint/no-floating-promises
           archive.finalize()
         })
-        .catch((error) => reject(error))
+        .catch((error) => reject(error instanceof Error ? error : new Error(String(error))))
     })
 
     const tarContent = readFileSync(tempTarPath)

packages/theme/src/cli/services/package.test.ts

@@ -191,7 +191,10 @@ async function readArchiveFiles(zipPath: string) {
   await expect(fileExists(zipPath)).resolves.toBeTruthy()
   // eslint-disable-next-line @babel/new-cap
   const archive = new StreamZip.async({file: zipPath})
-  const archiveEntries = Object.keys(await archive.entries())
+  const entries = await archive.entries()
+  const archiveEntries = Object.values(entries)
+    .filter((entry: any) => !entry.isDirectory)
+    .map((entry: any) => entry.name)
   await archive.close()
 
   return archiveEntries