mirror of https://github.com/vercel/next.js.git synced 2026-09-20 02:25:18 +08:00

Files

T

Marcos Hernanz 4344b83a6b Flag newly disabled deploy tests (#96505 )

## What changed

Added scoped review guidance under `test/` so Vercel Agent flags changes
that newly disable deploy-mode tests or assertions.

## Why

Deploy coverage should only be disabled for assertions that are
inherently self-hosted-specific. When deployed behavior differs, the
test should prefer documenting those semantics instead of skipping
deploy mode.

## Validation

- `pnpm install --frozen-lockfile`
- `pnpm build-all`
- Hosted Vercel Agent E2E: an intentional `skipDeployment: true` probe
was flagged inline, then removed
- Hosted Vercel Agent Review passed on the final diff

2026-08-03 19:29:36 +02:00

386 B

Raw Permalink Blame History

Test review instructions

During code review, flag any change that newly disables, skips, or omits deploy-mode tests or assertions, including skipDeployment: true and isNextDeploy conditionals. Only allow this when the assertion is inherently specific to self-hosted behavior; even then, prefer an explicit deploy-mode assertion that documents the different deployed semantics.

386 B Raw Permalink Blame History

Test review instructions

386 B

Raw Permalink Blame History