Feat: support astro add without npm installing (#3183)

* deps: update to latest proload

* feat: avoid parsing config on resolveConfigUrl

* chore: changeset

* fix: bump proload

* fix: bump proload

* fix: bump proload

* fix: bump proload

* fix: bump proload

Co-authored-by: Nate Moore <nate@skypack.dev>

.changeset/giant-impalas-design.md

@@ -0,0 +1,6 @@
+---
+'astro': patch
+'@astrojs/tailwind': patch
+---
+
+Support "astro add" before installing project dependencies

packages/astro/package.json

@@ -84,8 +84,8 @@
     "@babel/generator": "^7.17.9",
     "@babel/parser": "^7.17.9",
     "@babel/traverse": "^7.17.9",
-    "@proload/core": "^0.2.2",
-    "@proload/plugin-tsm": "^0.1.1",
+    "@proload/core": "^0.3.2-next.4",
+    "@proload/plugin-tsm": "^0.2.1-next.0",
     "ast-types": "^0.14.2",
     "boxen": "^6.2.1",
     "ci-info": "^3.3.0",

packages/astro/src/cli/index.ts

@@ -104,15 +104,6 @@ export async function cli(args: string[]) {
 		logging.level = 'silent';
 	}
 
-	let config: AstroConfig;
-	try {
-		// Note: ideally, `loadConfig` would return the config AND its filePath
-		// For now, `add` has to resolve the config again internally
-		config = await loadConfig({ cwd: root, flags, cmd });
-	} catch (err) {
-		return throwAndExit(err);
-	}
-
 	switch (cmd) {
 		case 'add': {
 			try {
@@ -124,6 +115,7 @@ export async function cli(args: string[]) {
 		}
 		case 'dev': {
 			try {
+				const config = await loadConfig({ cwd: root, flags, cmd });
 				await devServer(config, { logging });
 				return await new Promise(() => {}); // lives forever
 			} catch (err) {
@@ -133,19 +125,22 @@ export async function cli(args: string[]) {
 
 		case 'build': {
 			try {
+				const config = await loadConfig({ cwd: root, flags, cmd });
 				return await build(config, { logging });
 			} catch (err) {
 				return throwAndExit(err);
 			}
 		}
 
 		case 'check': {
+			const config = await loadConfig({ cwd: root, flags, cmd });
 			const ret = await check(config);
 			return process.exit(ret);
 		}
 
 		case 'preview': {
 			try {
+				const config = await loadConfig({ cwd: root, flags, cmd });
 				const server = await preview(config, { logging });
 				return await server.closed(); // keep alive until the server is closed
 			} catch (err) {

packages/astro/src/core/config.ts

@@ -10,7 +10,7 @@ import { pathToFileURL, fileURLToPath } from 'url';
 import { mergeConfig as mergeViteConfig } from 'vite';
 import { BUNDLED_THEMES } from 'shiki';
 import { z } from 'zod';
-import load, { ProloadError } from '@proload/core';
+import load, { resolve, ProloadError } from '@proload/core';
 import loadTypeScript from '@proload/plugin-tsm';
 import postcssrc from 'postcss-load-config';
 import { arraify, isObject } from './util.js';
@@ -385,11 +385,15 @@ export async function resolveConfigURL(
 		userConfigPath = /^\.*\//.test(flags.config) ? flags.config : `./${flags.config}`;
 		userConfigPath = fileURLToPath(new URL(userConfigPath, `file://${root}/`));
 	}
-	// Automatically load config file using Proload
+	// Resolve config file path using Proload
 	// If `userConfigPath` is `undefined`, Proload will search for `astro.config.[cm]?[jt]s`
-	const config = await load('astro', { mustExist: false, cwd: root, filePath: userConfigPath });
-	if (config) {
-		return pathToFileURL(config.filePath);
+	const configPath = await resolve('astro', {
+		mustExist: false,
+		cwd: root,
+		filePath: userConfigPath,
+	});
+	if (configPath) {
+		return pathToFileURL(configPath);
 	}
 }
 

packages/integrations/tailwind/package.json

@@ -27,7 +27,7 @@
     "dev": "astro-scripts dev \"src/**/*.ts\""
   },
   "dependencies": {
-    "@proload/core": "^0.2.2",
+    "@proload/core": "^0.3.1",
     "autoprefixer": "^10.4.4",
     "postcss": "^8.4.12",
     "tailwindcss": "^3.0.24"