fix: make packages optional in pnpm-workspace.yaml (#124)

# fix: make `packages` optional in `pnpm-workspace.yaml`

Added the `#[serde(default)]` attribute to the `packages` field in the `PnpmWorkspace` struct to make it optional. This allows PNPM workspaces to function correctly even when the `packages` field is not specified in the `pnpm-workspace.yaml` file.

Added a test fixture with a `pnpm-workspace.yaml` that doesn't include a `packages` field to verify the fix works correctly.

Author: branchseer

crates/vite_task_plan/tests/plan_snapshots/fixtures/pnpm-workspace-packages-optional/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "additional-envs",
+  "private": true,
+  "scripts": {
+    "hello": "echo hello"
+  }
+}

crates/vite_task_plan/tests/plan_snapshots/fixtures/pnpm-workspace-packages-optional/pnpm-workspace.yaml

@@ -0,0 +1 @@
+# no `packages` field

crates/vite_task_plan/tests/plan_snapshots/fixtures/pnpm-workspace-packages-optional/snapshots.toml

@@ -0,0 +1,3 @@
+[[plan]]
+name = "allow `packages` in pnpm-workspace.yaml to be optional"
+args = ["run", "hello"]

crates/vite_task_plan/tests/plan_snapshots/fixtures/pnpm-workspace-packages-optional/snapshots/query - allow `packages` in pnpm-workspace.yaml to be optional.snap

@@ -0,0 +1,53 @@
+---
+source: crates/vite_task_plan/tests/plan_snapshots/main.rs
+expression: "&plan_json"
+input_file: crates/vite_task_plan/tests/plan_snapshots/fixtures/pnpm-workspace-packages-optional
+---
+{
+  "root_node": {
+    "Expanded": [
+      {
+        "key": [
+          "<workspace>/",
+          "hello"
+        ],
+        "node": {
+          "task_display": {
+            "package_name": "additional-envs",
+            "task_name": "hello",
+            "package_path": "<workspace>/"
+          },
+          "items": [
+            {
+              "execution_item_display": {
+                "task_display": {
+                  "package_name": "additional-envs",
+                  "task_name": "hello",
+                  "package_path": "<workspace>/"
+                },
+                "command": "echo hello",
+                "and_item_index": null,
+                "cwd": "<workspace>/"
+              },
+              "kind": {
+                "Leaf": {
+                  "InProcess": {
+                    "kind": {
+                      "Echo": {
+                        "strings": [
+                          "hello"
+                        ],
+                        "trailing_newline": true
+                      }
+                    }
+                  }
+                }
+              }
+            }
+          ]
+        },
+        "neighbors": []
+      }
+    ]
+  }
+}

crates/vite_task_plan/tests/plan_snapshots/fixtures/pnpm-workspace-packages-optional/snapshots/task graph.snap

@@ -0,0 +1,35 @@
+---
+source: crates/vite_task_plan/tests/plan_snapshots/main.rs
+expression: task_graph_json
+input_file: crates/vite_task_plan/tests/plan_snapshots/fixtures/pnpm-workspace-packages-optional
+---
+[
+  {
+    "key": [
+      "<workspace>/",
+      "hello"
+    ],
+    "node": {
+      "task_display": {
+        "package_name": "additional-envs",
+        "task_name": "hello",
+        "package_path": "<workspace>/"
+      },
+      "resolved_config": {
+        "command": "echo hello",
+        "resolved_options": {
+          "cwd": "<workspace>/",
+          "cache_config": {
+            "env_config": {
+              "fingerprinted_envs": [],
+              "pass_through_envs": [
+                "<default pass-through envs>"
+              ]
+            }
+          }
+        }
+      }
+    },
+    "neighbors": []
+  }
+]

crates/vite_workspace/src/lib.rs

@@ -27,6 +27,7 @@ struct PnpmWorkspace {
     /// The packages to include in the workspace.
     ///
     /// <https://pnpm.io/pnpm-workspace_yaml>
+    #[serde(default)]
     packages: Vec<Str>,
 }