fix(cloudflare): skip config file validation during svelte check & sync (#1336)

Co-authored-by: John Royal <34844819+john-royal@users.noreply.github.com>

Author: yeoularu

alchemy/src/cloudflare/sveltekit/plugin.ts

@@ -2,17 +2,11 @@ import adapter, { type AdapterOptions } from "@sveltejs/adapter-cloudflare";
 import type { Adapter } from "@sveltejs/kit";
 import { getPlatformProxyOptions } from "../cloudflare-env-proxy.ts";
 
-const isSvelteLanguageServer = !!process.argv.find((arg) =>
-  arg.includes("svelte-language-server"),
-);
-
 export default (options?: AdapterOptions): Adapter => {
-  if (isSvelteLanguageServer) {
-    // the svelte language server runs from the root and therefore may be running in a different cwd than the `Website.cwd`
-    // for now, we work around this by returning a noop adapter to avoid breaking svelte intellisense
+  if (shouldDisable()) {
     return {
-      adapt() {},
       name: "alchemy-noop",
+      adapt() {},
     };
   }
   const { platformProxy: proxyOptions, ...config } = options ?? {};
@@ -23,3 +17,18 @@ export default (options?: AdapterOptions): Adapter => {
     ...config,
   });
 };
+
+/**
+ * Return true for `svelte-language-server`, `svelte-check`, and `svelte-kit sync`. This is because:
+ * - The svelte language server runs from the root and therefore may be running in a different cwd than the `Website.cwd`, so disable to avoid breaking intellisense
+ * - The `svelte-check` and `svelte-kit sync` commands do not require Cloudflare-specific configuration, so disable to avoid breaking the commands
+ */
+function shouldDisable(argv = process.argv): boolean {
+  return (
+    argv.some(
+      (arg) =>
+        arg.includes("svelte-check") || arg.includes("svelte-language-server"),
+    ) ||
+    (argv.some((arg) => arg.includes("svelte-kit")) && argv.includes("sync"))
+  );
+}

alchemy/src/cloudflare/vite/vite.ts

@@ -2,6 +2,7 @@ import path from "pathe";
 import { getPackageManagerRunner } from "../../util/detect-package-manager.ts";
 import type { Assets } from "../assets.ts";
 import type { Bindings } from "../bindings.ts";
+import { withSkipPathValidation } from "../miniflare/paths.ts";
 import {
   spreadBuildProps,
   spreadDevProps,
@@ -34,8 +35,9 @@ export async function Vite<B extends Bindings>(
       port = args[index + 1];
     } else {
       try {
-        const config = await import(
-          path.resolve(props.cwd ?? process.cwd(), "vite.config.ts")
+        const config = await withSkipPathValidation(
+          () =>
+            import(path.resolve(props.cwd ?? process.cwd(), "vite.config.ts")),
         );
         port = config.default?.server?.port ?? 5173;
       } catch {}