Fix up pnpm dev (#7424)

* feat: use persistent per turbo recs

* feat: preserve types yay

* fix: *actually* solve problem

* fix: type failures from content types template

* fix: bad JS doc comments

* fix: bad src/content/template path

* chore: changeset

---------

Co-authored-by: Nate Moore <natemoo-re@users.noreply.github.com>

.changeset/fluffy-pandas-breathe.md

@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Update internal types for more stable builds for Astro maintainers.

package.json

@@ -11,7 +11,7 @@
     "build": "turbo run build --filter=astro --filter=create-astro --filter=\"@astrojs/*\" --filter=\"@benchmark/*\"",
     "build:ci": "turbo run build:ci --filter=astro --filter=create-astro --filter=\"@astrojs/*\" --filter=\"@benchmark/*\"",
     "build:examples": "turbo run build --filter=\"@example/*\"",
-    "dev": "turbo run dev --parallel --filter=astro --filter=create-astro --filter=\"@astrojs/*\" --filter=\"@benchmark/*\"",
+    "dev": "turbo run dev --concurrency=40 --parallel --filter=astro --filter=create-astro --filter=\"@astrojs/*\" --filter=\"@benchmark/*\"",
     "format": "pnpm run format:code",
     "format:ci": "pnpm run format:imports && pnpm run format:code",
     "format:code": "prettier -w . --cache --plugin-search-dir=.",

packages/astro/package.json

@@ -91,12 +91,13 @@
     "client.d.ts",
     "client-base.d.ts",
     "client-image.d.ts",
+    "content-types.template.d.ts",
+    "content-module.template.mjs",
     "import-meta.d.ts",
     "astro-jsx.d.ts",
     "types.d.ts",
     "README.md",
-    "vendor",
-    "src/content/template"
+    "vendor"
   ],
   "scripts": {
     "prebuild": "astro-scripts prebuild --to-string \"src/runtime/server/astro-island.ts\" \"src/runtime/client/{idle,load,media,only,visible}.ts\"",

packages/astro/src/content/utils.ts

@@ -24,7 +24,7 @@ import { createImage } from './runtime-assets.js';
 /**
  * Amap from a collection + slug to the local file path.
  * This is used internally to resolve entry imports when using `getEntry()`.
- * @see `src/content/virtual-mod.mjs`
+ * @see `content-module.template.mjs`
  */
 export type ContentLookupMap = {
 	[collectionName: string]: { type: 'content' | 'data'; entries: { [lookupId: string]: string } };
@@ -432,7 +432,6 @@ export type ContentPaths = {
 	cacheDir: URL;
 	typesTemplate: URL;
 	virtualModTemplate: URL;
-	virtualAssetsModTemplate: URL;
 	config: {
 		exists: boolean;
 		url: URL;
@@ -444,14 +443,13 @@ export function getContentPaths(
 	fs: typeof fsMod = fsMod
 ): ContentPaths {
 	const configStats = search(fs, srcDir);
-	const templateDir = new URL('../../src/content/template/', import.meta.url);
+	const pkgBase = new URL('../../', import.meta.url);
 	return {
 		cacheDir: new URL('.astro/', root),
 		contentDir: new URL('./content/', srcDir),
 		assetsDir: new URL('./assets/', srcDir),
-		typesTemplate: new URL('types.d.ts', templateDir),
-		virtualModTemplate: new URL('virtual-mod.mjs', templateDir),
-		virtualAssetsModTemplate: new URL('virtual-mod-assets.mjs', templateDir),
+		typesTemplate: new URL('content-types.template.d.ts', pkgBase),
+		virtualModTemplate: new URL('content-module.template.mjs', pkgBase),
 		config: configStats,
 	};
 }

packages/astro/src/content/vite-plugin-content-virtual-mod.ts

@@ -94,7 +94,7 @@ export function astroContentVirtualModPlugin({
 /**
  * Generate a map from a collection + slug to the local file path.
  * This is used internally to resolve entry imports when using `getEntry()`.
- * @see `src/content/virtual-mod.mjs`
+ * @see `content-module.template.mjs`
  */
 export async function getStringifiedLookupMap({
 	contentPaths,

packages/astro/test/units/content-collections/frontmatter.test.js

@@ -10,7 +10,7 @@ import { createFs, triggerFSEvent } from '../test-utils.js';
 const root = new URL('../../fixtures/alias/', import.meta.url);
 
 function getTypesDts() {
-	const typesdtsURL = new URL('../../../src/content/template/types.d.ts', import.meta.url);
+	const typesdtsURL = new URL('../../../content-types.template.d.ts', import.meta.url);
 	const relpath = slash(path.relative(fileURLToPath(root), fileURLToPath(typesdtsURL)));
 	return {
 		[relpath]: nodeFS.readFileSync(typesdtsURL, 'utf-8'),

scripts/cmd/build.js

@@ -133,5 +133,7 @@ export default async function build(...args) {
 }
 
 async function clean(outdir) {
-	return deleteAsync([`${outdir}/**`, `!${outdir}/**/*.d.ts`]);
+	await deleteAsync([`${outdir}/**`, `!${outdir}/**/*.d.ts`], {
+		onlyFiles: true,
+	});
 }

turbo.json

@@ -32,7 +32,8 @@
       "outputMode": "new-only"
     },
     "dev": {
-      "cache": false
+      "cache": false,
+      "persistent": true
     },
     "test": {
       "env": ["RUNNER_OS", "NODE_VERSION"],