@playcraft/cli 0.1.38-beta → 0.1.40-beta

package/dist/workspace-runtime/workspaces/files.js

@@ -55,45 +55,63 @@ export async function readWorkspaceFile(root, relativePath, revisions) {
 export async function listWorkspaceFiles(root, revisions) {
     const rootReal = await ensureWorkspaceRoot(root);
     const files = [];
-    for (const relativePath of await walkWorkspaceFiles(rootReal, rootReal)) {
+    const unreadablePaths = new Set();
+    for (const relativePath of await walkWorkspaceFiles(rootReal, rootReal, unreadablePaths)) {
         const target = join(rootReal, ...relativePath.split('/'));
-        if ((await lstat(target)).size > LARGE_FILE_STREAM_THRESHOLD) {
-            const sampled = await sampleDiskFile(target);
-            files.push({ path: relativePath, revision: revisions[relativePath]?.revision ?? 0, contentHash: sampled.hash, size: sampled.size });
-            continue;
+        try {
+            if ((await lstat(target)).size > LARGE_FILE_STREAM_THRESHOLD) {
+                const sampled = await sampleDiskFile(target);
+                files.push({ path: relativePath, revision: revisions[relativePath]?.revision ?? 0, contentHash: sampled.hash, size: sampled.size });
+                continue;
+            }
+            const content = await readFile(target, 'utf8');
+            files.push({
+                path: relativePath,
+                revision: revisions[relativePath]?.revision ?? 0,
+                contentHash: hashWorkspaceFileContent(content),
+                size: Buffer.byteLength(content, 'utf8'),
+            });
+        }
+        catch (error) {
+            if (!isPermissionDenied(error))
+                throw error;
+            unreadablePaths.add(relativePath);
         }
-        const content = await readFile(target, 'utf8');
-        files.push({
-            path: relativePath,
-            revision: revisions[relativePath]?.revision ?? 0,
-            contentHash: hashWorkspaceFileContent(content),
-            size: Buffer.byteLength(content, 'utf8'),
-        });
     }
     files.sort((left, right) => left.path.localeCompare(right.path));
-    return workspaceFileListSchema.parse({ files });
+    return workspaceFileListSchema.parse({ files, ...(unreadablePaths.size ? { unreadablePaths: [...unreadablePaths].sort() } : {}) });
 }
-async function walkWorkspaceFiles(rootReal, current) {
+async function walkWorkspaceFiles(rootReal, current, unreadablePaths) {
     const entries = await readdir(current, { withFileTypes: true });
     const paths = [];
     for (const entry of entries) {
-        if (entry.isDirectory()) {
-            if (SKIP_DIRECTORY_NAMES.has(entry.name) || entry.name === '.' || entry.name === '..')
+        const child = join(current, entry.name);
+        try {
+            if (entry.isDirectory()) {
+                if (SKIP_DIRECTORY_NAMES.has(entry.name) || entry.name === '.' || entry.name === '..')
+                    continue;
+                const childReal = await realpath(child);
+                assertInsideRoot(rootReal, childReal);
+                paths.push(...(await walkWorkspaceFiles(rootReal, childReal, unreadablePaths)));
+                continue;
+            }
+            if (!entry.isFile())
                 continue;
-            const child = join(current, entry.name);
             const childReal = await realpath(child);
             assertInsideRoot(rootReal, childReal);
-            paths.push(...(await walkWorkspaceFiles(rootReal, childReal)));
-            continue;
+            paths.push(toPosixRelative(rootReal, childReal));
+        }
+        catch (error) {
+            if (!isPermissionDenied(error))
+                throw error;
+            unreadablePaths.add(toPosixRelative(rootReal, child));
         }
-        if (!entry.isFile())
-            continue;
-        const childReal = await realpath(join(current, entry.name));
-        assertInsideRoot(rootReal, childReal);
-        paths.push(toPosixRelative(rootReal, childReal));
     }
     return paths;
 }
+function isPermissionDenied(error) {
+    return ['EACCES', 'EPERM'].includes(String(error?.code));
+}
 async function assertParentInsideRoot(rootReal, parent) {
     if (parent === rootReal)
         return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@playcraft/cli",
-  "version": "0.1.38-beta",
+  "version": "0.1.40-beta",
   "private": false,
   "type": "module",
   "engines": {
@@ -51,8 +51,8 @@
     "@gltf-transform/core": "^4.3.0",
     "@gltf-transform/extensions": "^4.3.0",
     "@gltf-transform/functions": "^4.3.0",
-    "@playcraft/build": "^0.1.38-beta",
-    "@playcraft/common": "^0.1.38-beta",
+    "@playcraft/build": "^0.1.40-beta",
+    "@playcraft/common": "^0.1.40-beta",
     "better-sqlite3-pinned": "npm:better-sqlite3@13.0.3",
     "chokidar": "^4.0.3",
     "commander": "^13.1.0",