From 50de3baa10bb46d9ae6a2ae4eab54ee12ce27d62 Mon Sep 17 00:00:00 2001
From: Will Chen <willchen90@gmail.com>
Date: Mon, 21 Apr 2025 15:01:34 -0700
Subject: [PATCH] Make pnpm optional for setup flow

---
 src/components/SetupBanner.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/components/SetupBanner.tsx b/src/components/SetupBanner.tsx
index 4d21aab..b742658 100644
--- a/src/components/SetupBanner.tsx
+++ b/src/components/SetupBanner.tsx
@@ -83,9 +83,8 @@ export function SetupBanner() {
     await checkNode();
   }, [checkNode, setNodeInstallStep]);
 
-  const isNodeSetupComplete = Boolean(
-    nodeSystemInfo?.nodeVersion && nodeSystemInfo?.pnpmVersion
-  );
+  // We only check for node version because pnpm is not required for the app to run.
+  const isNodeSetupComplete = Boolean(nodeSystemInfo?.nodeVersion);
 
   const itemsNeedAction: string[] = [];
   if (!isNodeSetupComplete && nodeSystemInfo) {
@@ -161,7 +160,8 @@ export function SetupBanner() {
                   Node.js ({nodeSystemInfo!.nodeVersion}) installed.{" "}
                   {nodeSystemInfo!.pnpmVersion && (
                     <span className="text-xs text-gray-500">
-                      pnpm ({nodeSystemInfo!.pnpmVersion}) installed.
+                      {" "}
+                      (optional) pnpm ({nodeSystemInfo!.pnpmVersion}) installed.
                     </span>
                   )}
                 </p>