Skip to content

docs: codify generated entry refactor guidance#640

Merged
southpolesteve merged 1 commit intomainfrom
codex/agents-generated-entry-guidance
Mar 22, 2026
Merged

docs: codify generated entry refactor guidance#640
southpolesteve merged 1 commit intomainfrom
codex/agents-generated-entry-guidance

Conversation

@southpolesteve
Copy link
Copy Markdown
Collaborator

Summary

Add repo guidance for the refactor pattern we've been using to make generated entry code more maintainable:

  • prefer vp for local check/test/build loops
  • keep packages/vinext/src/entries/* as thin codegen glue
  • move request lifecycle logic into typed helpers under packages/vinext/src/server/*
  • prefer helper unit tests plus delegation assertions over giant generated-string assertions
  • document the small stacked-PR + /bigbonk review loop for high-risk refactors

Why

We've been steadily extracting App Router runtime logic out of large generated template strings. This codifies the broad strokes of that pattern in AGENTS.md so future agents keep moving in the same direction instead of reintroducing large inline runtime subsystems.

Testing

  • vp check AGENTS.md

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 22, 2026

Open in StackBlitz

npm i https://pkg.pr.new/vinext@640

commit: fd576d0

@github-actions
Copy link
Copy Markdown

Example Preview Production Original
app-router-cloudflare preview production
pages-router-cloudflare preview production
app-router-playground preview production original
realworld-api-rest preview production
nextra-docs-template preview production
benchmarks preview production
hackernews preview production original

@southpolesteve southpolesteve merged commit 3c5365a into main Mar 22, 2026
30 checks passed
@southpolesteve southpolesteve deleted the codex/agents-generated-entry-guidance branch March 22, 2026 04:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant