Skip to content

fix: don't destroy partial-line app output with the prerender progress line - #16750

Open
Nic-Polumeyv wants to merge 1 commit into
version-3from
progress-partial-line
Open

fix: don't destroy partial-line app output with the prerender progress line#16750
Nic-Polumeyv wants to merge 1 commit into
version-3from
progress-partial-line

Conversation

@Nic-Polumeyv

@Nic-Polumeyv Nic-Polumeyv commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

If app code writes to stdout without a trailing newline during prerendering on a TTY, the next progress line lands on the same row and the update after that clears the row:

rendering /partial...
rendering /dynamic/foo...   <- app output destroyed

The write proxies now track whether the last chunk ended mid-line, and the progress line starts a fresh row instead:

rendering /partial...
IMPORTANT APP OUTPUT (no newline)
rendering /dynamic/foo...

surfaced while reviewing #16748

@pkg-svelte-dev

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from 8941485:

pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/8941485b12648db2a78022bdd96ae121f82b1016

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/kit/pr/16750

@changeset-bot

changeset-bot Bot commented Aug 12, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8941485

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sveltejs/kit Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@svelte-docs-bot

Copy link
Copy Markdown

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