CodeVetter · Repo Unpack · Contributor handoff
withastro/astro
Begin with the pinned manifests, current core architecture README, and the exact owning modules for the requested change. Tests and CI were inspected, not executed, and this brief supplies no passing baseline. Preserve runtime portability, built-output test imports, request finalization, and deterministic content persistence. Documentation contains drift, so confirm names and behavior against source before planning edits. The inspection is bounded; widen searches only where the task requires more evidence.
- Read core/README.md before request-runtime work: SSRManifest, owning-module memoization, RenderEnvironment, and FetchState are the documented current architecture.
- Use package.json as the package-manager authority: it pins pnpm@11.27.0, while CONTRIBUTING.md contains older pnpm guidance.
- AGENTS.md calls for explicit assumptions, simple solutions, surgical changes, and verifiable goals; unrelated cleanup should remain outside the change.
- Runtime files and generated runtime virtual modules must avoid Node APIs to support environments such as Cloudflare Workers; Node APIs are permitted in Vite plugin implementation code.
- For future authorized verification, rebuild before running tests that import dist, then select a specific astro-scripts test file before broad package suites.
- Reuse the documented createTestApp(), createPage(), createEndpoint(), createMockFetchState(), and domain-specific helpers instead of duplicating setup.
- Request-chain changes should preserve dispatch-level origin checks and provider finalization; dedicated tests exercise actions-before-middleware, pages-without-middleware, and rejecting finalizers.
- Content changes should preserve deterministic serialization, atomic rename, chunk manifest-last commits, and explicit onFileWritten notifications used for dev invalidation.
- AstroIsland source is prebuilt; the package prebuild script also serializes client directive implementations, so browser-runtime edits must account for generated artifacts.
- Formatting uses tabs, single quotes, and semicolons for JavaScript/TypeScript; AGENTS.md identifies pnpm format and pnpm lint:ai, whose execution remains outside this read-only investigation.
- Treat CI definitions as intended checks, not observed results; the Linux ts-plugin command explicitly tolerates failure, and Playwright has CI retries and one worker.