@yopem-ui/cli 0.1.1 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yopem-ui/cli",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "CLI for installing Yopem UI source components and StyleX setup",
   "keywords": [
     "cli",

package/src/init.ts

@@ -1519,10 +1519,11 @@ function nextScripts(value: JsonValue | undefined) {
 }
 
 async function chooseFile(root: string, names: string[], fallback?: string) {
-  const matches: string[] = []
+  const existing = await Promise.all(
+    names.map((name) => existingFile(root, name)),
+  )
 
-  for (const name of names)
-    if (await existingFile(root, name)) matches.push(name)
+  const matches = names.filter((_, index) => existing[index])
 
   if (matches.length > 1)
     throw new Error(`Ambiguous configuration: ${matches.join(", ")}`)
@@ -1619,8 +1620,11 @@ export async function initProject(
     }
   }
 
-  const packageRun = await packageRunner(root, options.run)
-  const uiRun = await packageRunner(uiRoot, options.run)
+  const [packageRun, uiRun] = await Promise.all([
+    packageRunner(root, options.run),
+    packageRunner(uiRoot, options.run),
+  ])
+
   const edits = new Map<string, { before: string | null; after: string }>()
 
   const plannedFiles: {

package/src/install.ts

@@ -490,26 +490,34 @@ export async function writeFiles(
   }
 
   try {
-    for (const [path, change] of changes) {
-      const exists = await existingFile(root, path)
-      const before = exists ? await readFile(join(root, path)) : null
-
-      if ((before?.toString("utf8") ?? null) !== change.before) {
-        throw new Error(`File changed before commit: ${path}`)
-      }
-
-      const mode = exists ? (await lstat(join(root, path))).mode & 0o7777 : null
-      const after = change.after === null ? null : Buffer.from(change.after)
-      staged.push({
-        path,
-        before,
-        after,
-        mode,
-        afterMode: mode,
-        changed: change.before !== change.after,
-        directory: null,
-      })
-    }
+    staged.push(
+      ...(await Promise.all(
+        [...changes].map(async ([path, change]): Promise<StagedFile> => {
+          const exists = await existingFile(root, path)
+          const before = exists ? await readFile(join(root, path)) : null
+
+          if ((before?.toString("utf8") ?? null) !== change.before) {
+            throw new Error(`File changed before commit: ${path}`)
+          }
+
+          const mode = exists
+            ? (await lstat(join(root, path))).mode & 0o7777
+            : null
+
+          const after = change.after === null ? null : Buffer.from(change.after)
+
+          return {
+            path,
+            before,
+            after,
+            mode,
+            afterMode: mode,
+            changed: change.before !== change.after,
+            directory: null,
+          }
+        }),
+      )),
+    )
 
     for (const file of staged) {
       if (!file.changed) continue
@@ -535,7 +543,9 @@ export async function writeFiles(
 
     await prepare?.()
 
-    for (const file of staged) await verify(file, file.before, file.mode)
+    await Promise.all(
+      staged.map((file) => verify(file, file.before, file.mode)),
+    )
 
     for (const file of staged) {
       if (!file.changed) continue
@@ -737,10 +747,16 @@ export async function planInstall(name: string, options: InstallOptions = {}) {
     manifest.provenance[path] = { registryUrl, items: ownership }
   }
 
-  for (const [path, file] of files) {
-    const exists = await existingFile(root, path)
+  const currentFiles = await Promise.all(
+    [...files].map(async ([path, file]) => {
+      const exists = await existingFile(root, path)
+      const before = exists ? await readFile(join(root, path), "utf8") : null
+
+      return { path, file, before }
+    }),
+  )
 
-    const before = exists ? await readFile(join(root, path), "utf8") : null
+  for (const { path, file, before } of currentFiles) {
     const current = before === null ? null : hash(before)
     changes.set(path, { before, after: before })
     const previous = manifest.files[path]

package/src/project.ts

@@ -166,8 +166,12 @@ async function packageManager(root: string) {
     ["yarn", "yarn.lock"],
   ]
 
-  for (const [manager, path] of locks) {
-    if (await existingFile(root, path)) found.add(manager)
+  const existingLocks = await Promise.all(
+    locks.map(([, path]) => existingFile(root, path)),
+  )
+
+  for (const [index, [manager]] of locks.entries()) {
+    if (existingLocks[index]) found.add(manager)
   }
 
   if (found.size > 1) {
@@ -184,8 +188,12 @@ export async function packageRunner(
   run?: InstallOptions["run"],
 ): Promise<NonNullable<InstallOptions["run"]>> {
   const target = await realpath(root)
-  const owner = await workspaceRoot(target)
-  const local = await packageManager(target)
+
+  const [owner, local] = await Promise.all([
+    workspaceRoot(target),
+    packageManager(target),
+  ])
+
   const workspace = owner === target ? local : await packageManager(owner)
 
   if (local && workspace && local !== workspace) {