fix: skip name checks for private packages

Author: skirtles-code

packages/app/e2e.test.ts

@@ -153,6 +153,9 @@ describe.sequential.each([
     expect(process.stderr).toContain("pkg-pr-new:");
     expect(process.stderr).toContain("playground-a:");
     expect(process.stderr).toContain("playground-b:");
+    expect(process.stderr).toContain(
+      "private-playground/ because the package is private",
+    );
   }, 20_000);
 
   it(`serves and installs playground-a for ${mode}`, async () => {

packages/cli/index.ts

@@ -274,16 +274,13 @@ const main = defineCommand({
             const pJsonPath = path.resolve(p, "package.json");
             const pJson = await readPackageJson(pJsonPath);
 
-            if (!pJson) {
+            if (!pJson || pJson.private) {
               continue;
             }
 
             if (!pJson.name) {
               throw new Error(`"name" field in ${pJsonPath} should be defined`);
             }
-            if (pJson.private) {
-              continue;
-            }
 
             const packageName = pJson.name;
             packageInfos.push({ packageName, pJson });
@@ -471,15 +468,15 @@ const main = defineCommand({
             }
 
             try {
+              if (pJson.private) {
+                console.warn(`skipping ${p} because the package is private`);
+                continue;
+              }
               if (!pJson.name) {
                 throw new Error(
                   `"name" field in ${pJsonPath} should be defined`,
                 );
               }
-              if (pJson.private) {
-                console.warn(`skipping ${p} because the package is private`);
-                continue;
-              }
 
               const { filename, shasum } = await resolveTarball(
                 packMethod,

playgrounds/private-playground/package.json

@@ -0,0 +1,3 @@
+{
+  "private": true
+}