fix: temporarily disable vp CLI caching due to Windows issue (#10)

Disable restoreVpCache and saveVpCache at the code level until a new
vite-plus release fixes the 'Cannot find module which' error on Windows.
Tests are skipped with describe.skip and FIXME comments.

Author: fengmk2

dist/index.mjs

@@ -83,7 +83,9 @@ describe("resolveVersion", () => {
   });
 });
 
-describe("restoreVpCache", () => {
+// FIXME: Re-enable these tests after vp CLI caching is re-enabled.
+// Caching is temporarily disabled due to Windows `Cannot find module 'which'` issue (#10).
+describe.skip("restoreVpCache", () => {
   beforeEach(() => {
     vi.stubEnv("RUNNER_OS", "Linux");
     vi.stubEnv("HOME", "/home/runner");
@@ -145,7 +147,9 @@ describe("restoreVpCache", () => {
   });
 });
 
-describe("saveVpCache", () => {
+// FIXME: Re-enable these tests after vp CLI caching is re-enabled.
+// Caching is temporarily disabled due to Windows `Cannot find module 'which'` issue (#10).
+describe.skip("saveVpCache", () => {
   beforeEach(() => {
     vi.stubEnv("HOME", "/home/runner");
   });

src/cache-vp.test.ts

@@ -1,8 +1,4 @@
-import { restoreCache, saveCache } from "@actions/cache";
-import { info, debug, saveState, getState, warning } from "@actions/core";
-import { arch, platform } from "node:os";
-import { State } from "./types.js";
-import { getVitePlusHome } from "./utils.js";
+import { info, warning } from "@actions/core";
 
 const SEMVER_RE = /^\d+\.\d+\.\d+/;
 
@@ -31,60 +27,15 @@ export async function resolveVersion(versionInput: string): Promise<string | und
   }
 }
 
-export async function restoreVpCache(version: string, nodeVersion: string): Promise<boolean> {
-  const vpHome = getVitePlusHome();
-  const runnerOS = process.env.RUNNER_OS || platform();
-  const runnerArch = arch();
-  const primaryKey = `setup-vp-${runnerOS}-${runnerArch}-${version}-node${nodeVersion}`;
-
-  debug(`Vp cache key: ${primaryKey}`);
-  debug(`Vp cache path: ${vpHome}`);
-  saveState(State.VpCachePrimaryKey, primaryKey);
-
-  try {
-    const matchedKey = await restoreCache([vpHome], primaryKey);
-    if (matchedKey) {
-      info(`Vite+ restored from cache (key: ${matchedKey})`);
-      saveState(State.VpCacheMatchedKey, matchedKey);
-      return true;
-    }
-  } catch (error) {
-    warning(`Failed to restore vp cache: ${error}`);
-  }
-
+// FIXME: Re-enable vp CLI caching after the new version of vite-plus is released
+// that fixes the Windows `Cannot find module 'which'` issue (#10).
+export async function restoreVpCache(_version: string, _nodeVersion: string): Promise<boolean> {
+  info("Vp CLI caching is temporarily disabled");
   return false;
 }
 
+// FIXME: Re-enable vp CLI caching after the new version of vite-plus is released
+// that fixes the Windows `Cannot find module 'which'` issue (#10).
 export async function saveVpCache(): Promise<void> {
-  const forceInstall =
-    process.env.SETUP_VP_FORCE_INSTALL === "true" || process.env.SETUP_VP_FORCE_INSTALL === "1";
-  if (forceInstall) {
-    info("SETUP_VP_FORCE_INSTALL is set, skipping vp cache save");
-    return;
-  }
-
-  const primaryKey = getState(State.VpCachePrimaryKey);
-  const matchedKey = getState(State.VpCacheMatchedKey);
-
-  if (!primaryKey) {
-    debug("No vp cache key found. Skipping save.");
-    return;
-  }
-
-  if (primaryKey === matchedKey) {
-    info(`Vp cache hit on primary key "${primaryKey}". Skipping save.`);
-    return;
-  }
-
-  try {
-    const vpHome = getVitePlusHome();
-    const cacheId = await saveCache([vpHome], primaryKey);
-    if (cacheId === -1) {
-      warning("Vp cache save failed or was skipped.");
-      return;
-    }
-    info(`Vp cache saved with key: ${primaryKey}`);
-  } catch (error) {
-    warning(`Failed to save vp cache: ${String(error)}`);
-  }
+  info("Vp CLI caching is temporarily disabled, skipping save");
 }

src/cache-vp.ts

@@ -11,12 +11,10 @@ const INSTALL_URL_PS1 = "https://staging.viteplus.dev/install.ps1";
 
 export async function installVitePlus(inputs: Inputs, nodeVersion: string): Promise<void> {
   const { version } = inputs;
-  const forceInstall =
-    process.env.SETUP_VP_FORCE_INSTALL === "true" || process.env.SETUP_VP_FORCE_INSTALL === "1";
 
   // Try to resolve version and restore from cache
   const resolvedVersion = await resolveVersion(version);
-  if (resolvedVersion && !forceInstall) {
+  if (resolvedVersion) {
     const cacheHit = await restoreVpCache(resolvedVersion, nodeVersion);
     if (cacheHit) {
       ensureVitePlusBinInPath();
@@ -25,10 +23,6 @@ export async function installVitePlus(inputs: Inputs, nodeVersion: string): Prom
     }
   }
 
-  if (forceInstall) {
-    info("SETUP_VP_FORCE_INSTALL is set, skipping cache and forcing fresh install");
-  }
-
   // Cache miss or resolution failed — install fresh
   const installVersion = resolvedVersion || version;
   info(`Installing ${DISPLAY_NAME}@${installVersion}...`);