[PR #1094] [MERGED] feat(prerender): log server startup #1091

Closed
opened 2026-05-06 13:11:56 +02:00 by BreizhHardware · 0 comments

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/1094
Author: @NathanDrake2406
Created: 5/6/2026
Status: Merged
Merged: 5/6/2026
Merged by: @james-elicx

Base: mainHead: nathan/prerender-server-logs


📝 Commits (1)

  • 29842aa feat(prerender): log server startup

📊 Changes

4 files changed (+155 additions, -6 deletions)

View changed files

📝 packages/vinext/src/build/prerender.ts (+2 -0)
📝 packages/vinext/src/build/run-prerender.ts (+1 -0)
📝 packages/vinext/src/server/prod-server.ts (+25 -6)
tests/prod-server-logs.test.ts (+127 -0)

📄 Description

Summary

  • Add a prerender-specific production server startup log that includes the actual local URL/port.
  • Keep the default production server startup log unchanged for normal vinext start.
  • Thread the narrow purpose: "prerender" option through app-only, pages-only, and hybrid/shared prerender server startup paths.
  • Add focused coverage for Pages Router, App Router, and default production server startup logs.

Part of #563.

Validation

  • vp test run tests/prod-server-logs.test.ts
  • vp test run tests/prerender.test.ts -t "runPrerender — output"
  • vp check tests/prod-server-logs.test.ts packages/vinext/src/server/prod-server.ts packages/vinext/src/build/prerender.ts packages/vinext/src/build/run-prerender.ts

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/vinext/pull/1094 **Author:** [@NathanDrake2406](https://github.com/NathanDrake2406) **Created:** 5/6/2026 **Status:** ✅ Merged **Merged:** 5/6/2026 **Merged by:** [@james-elicx](https://github.com/james-elicx) **Base:** `main` ← **Head:** `nathan/prerender-server-logs` --- ### 📝 Commits (1) - [`29842aa`](https://github.com/cloudflare/vinext/commit/29842aa95d61f237e1eadbc66ee89948999213ff) feat(prerender): log server startup ### 📊 Changes **4 files changed** (+155 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `packages/vinext/src/build/prerender.ts` (+2 -0) 📝 `packages/vinext/src/build/run-prerender.ts` (+1 -0) 📝 `packages/vinext/src/server/prod-server.ts` (+25 -6) ➕ `tests/prod-server-logs.test.ts` (+127 -0) </details> ### 📄 Description ## Summary - Add a prerender-specific production server startup log that includes the actual local URL/port. - Keep the default production server startup log unchanged for normal `vinext start`. - Thread the narrow `purpose: "prerender"` option through app-only, pages-only, and hybrid/shared prerender server startup paths. - Add focused coverage for Pages Router, App Router, and default production server startup logs. Part of #563. ## Validation - `vp test run tests/prod-server-logs.test.ts` - `vp test run tests/prerender.test.ts -t "runPrerender — output"` - `vp check tests/prod-server-logs.test.ts packages/vinext/src/server/prod-server.ts packages/vinext/src/build/prerender.ts packages/vinext/src/build/run-prerender.ts` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
BreizhHardware 2026-05-06 13:11:56 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/vinext#1091
No description provided.