Test(cli): should not include private packages with a version in the …

…prompt

__fixtures__/private-package-with-version-field/.changeset/config.json

@@ -0,0 +1 @@
+{}

__fixtures__/private-package-with-version-field/package.json

@@ -0,0 +1,9 @@
+{
+  "private": true,
+  "name": "private-package-with-version-field",
+  "description": "Base yarn workspace work",
+  "version": "1.0.0",
+  "workspaces": [
+    "packages/*"
+  ]
+}

__fixtures__/private-package-with-version-field/packages/pkg-a/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "pkg-a",
+  "version": "1.0.0"
+}

__fixtures__/private-package-with-version-field/packages/pkg-b/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "pkg-b",
+  "version": "1.0.0",
+  "private": true
+}

__fixtures__/private-package-with-version-field/packages/pkg-c/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "pkg-c",
+  "version": "1.0.0"
+}

packages/cli/src/commands/add/__tests__/add.ts

@@ -230,4 +230,25 @@ describe("Changesets", () => {
     const { choices } = askCheckboxPlus.mock.calls[0][1][0];
     expect(choices).toEqual(["pkg-a", "pkg-c"]);
   });
+
+  it("should not include private packages with a version in the prompt", async () => {
+    const cwd = f.copy("private-package-with-version-field");
+
+    mockUserResponses({ releases: { "pkg-a": "patch" } });
+    await addChangeset(
+      cwd,
+      { empty: false },
+      {
+        ...defaultConfig,
+        privatePackages: {
+          version: false,
+          tag: false,
+        },
+      }
+    );
+
+    // @ts-ignore
+    const { choices } = askCheckboxPlus.mock.calls[0][1][0];
+    expect(choices).toEqual(["pkg-a", "pkg-c"]);
+  });
 });

packages/cli/src/commands/add/isListablePackage.ts

@@ -8,7 +8,7 @@ export function isListablePackage(config: Config, packageJson: PackageJSON) {
     return false;
   }
 
-  if (!config.privatePackages?.version && packageJson.private) {
+  if (!config.privatePackages.version && packageJson.private) {
     return false;
   }