Bring back global setup

Author: zastrowm

.github/workflows/ci.yml

@@ -26,9 +26,6 @@ jobs:
       - name: Install dependencies
         run: npm install
 
-      - name: Sync content
-        run: npx astro sync
-
       - name: Build
         run: npm run cms:build
 

AGENTS.md

@@ -116,8 +116,6 @@ Pre-commit hooks automatically run:
 
 All checks must pass before commit is allowed.
 
-**Test prerequisite:** Tests use `astro:content` via a mock that reads `.astro/data-store.json`. This file is generated by `npm run build` or `npx astro dev`. If it doesn't exist yet, run one of those first.
-
 ## Coding Patterns and Best Practices
 
 ### Code Style Guidelines (for Typescript)

SITE-ARCHITECTURE.md

@@ -903,25 +903,3 @@ This package is pinned to an exact version (`1.0.6`) rather than using a semver
 **Known bug:** The upstream plugin does not account for Astro's `base` path configuration, causing it to incorrectly flag all internal links as broken when the site is deployed under a sub-path. See [imazen/astro-broken-link-checker#16](https://github.com/imazen/astro-broken-link-checker/issues/16).
 
 **Local fix:** Rather than waiting for an upstream fix, the plugin source has been inlined into `scripts/astro-broken-links-checker-index.js` and `scripts/astro-broken-links-checker-check-links.js`. The fix captures `config.base` in the `astro:config:setup` hook and strips the base prefix from internal links before resolving them against the `dist/` directory. `astro.config.mjs` imports from the local copy instead of the npm package.
-
-## Testing
-
-Tests run with [Vitest](https://vitest.dev/) via `npm test`. Because many tests call `getCollection()` from `astro:content`, and that virtual module only exists inside Astro's Vite pipeline, we alias it to a lightweight mock.
-
-### `astro:content` Mock (`test/__mocks__/astro-content.ts`)
-
-The mock reads `.astro/data-store.json` directly at module init time using `devalue.unflatten` — the same serialization format Astro uses internally. It implements `getCollection` and `getEntry` against that in-memory map, bypassing the `astro:data-layer-content` virtual module entirely.
-
-The alias is registered in `vitest.config.ts`:
-
-```typescript
-alias: {
-  'astro:content': resolve('./test/__mocks__/astro-content.ts'),
-}
-```
-
-**Prerequisite:** `.astro/data-store.json` must exist and be populated before running tests. Run `npm run build` or `npx astro dev` once to generate it. If the file is missing, `readFileSync` will throw with a clear error.
-
-### Why not use the real `astro:content`?
-
-`astro:content` is a Vite virtual module resolved by Astro's `vite-plugin-content-virtual-mod` plugin. It works fine during `astro build` / `astro dev`, but vitest's module resolution can't satisfy the `astro:data-layer-content` dynamic import that the real runtime depends on — even with `getViteConfig` from `astro/config` wiring up Astro's plugins. The mock sidesteps this by reading the pre-built data store directly.

test/global-setup.ts

@@ -0,0 +1,59 @@
+import { existsSync, readFileSync } from 'node:fs'
+import { spawn } from 'node:child_process'
+
+const DATA_STORE_PATH = '.astro/data-store.json'
+const TIMEOUT_MS = 60_000
+const SYNCED_MARKER = 'Synced content'
+
+function isDataStorePopulated(): boolean {
+  if (!existsSync(DATA_STORE_PATH)) return false
+  try {
+    return readFileSync(DATA_STORE_PATH, 'utf-8').trim().length > 100
+  } catch {
+    return false
+  }
+}
+
+export async function setup() {
+  if (isDataStorePopulated()) return
+
+  console.log('[global-setup] Data store not populated — starting astro dev temporarily...')
+
+  const server = spawn('npx', ['astro', 'dev'], {
+    stdio: ['ignore', 'pipe', 'pipe'],
+    detached: true,
+  })
+
+  try {
+    await new Promise<void>((resolve, reject) => {
+      const timer = setTimeout(() => reject(new Error('Timed out waiting for "Synced content"')), TIMEOUT_MS)
+
+      const onData = (chunk: Buffer) => {
+        process.stdout.write(chunk)
+        if (chunk.toString().includes(SYNCED_MARKER)) {
+          clearTimeout(timer)
+          resolve()
+        }
+      }
+
+      server.stdout.on('data', onData)
+      server.stderr.on('data', onData)
+      server.on('error', (err) => {
+        clearTimeout(timer)
+        reject(err)
+      })
+      server.on('close', (code) => {
+        if (code !== null) {
+          clearTimeout(timer)
+          reject(new Error(`Astro dev server exited with code ${code}`))
+        }
+      })
+    })
+  } finally {
+    if (server.pid != null) {
+      process.kill(-server.pid, 'SIGTERM')
+    }
+  }
+
+  console.log('[global-setup] Data store ready.')
+}

vitest.config.ts

@@ -4,6 +4,7 @@ import { resolve } from 'node:path'
 export default getViteConfig({
   test: {
     include: ['test/**/*.test.ts'],
+    globalSetup: ['test/global-setup.ts'],
     alias: {
       'astro:content': resolve('./test/__mocks__/astro-content.ts'),
     },