Skip to content

feat: out of order streaming for Suspense#8611

Open
Varixo wants to merge 14 commits into
build/v2from
v2-out-of-order-streaming
Open

feat: out of order streaming for Suspense#8611
Varixo wants to merge 14 commits into
build/v2from
v2-out-of-order-streaming

Conversation

@Varixo
Copy link
Copy Markdown
Member

@Varixo Varixo commented May 5, 2026

No description provided.

@Varixo Varixo self-assigned this May 5, 2026
@Varixo Varixo requested review from a team as code owners May 5, 2026 16:49
@Varixo Varixo added the V2 label May 5, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 5, 2026

🦋 Changeset detected

Latest commit: d243c02

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

This PR includes changesets to release 5 packages
Name Type
@qwik.dev/core Patch
eslint-plugin-qwik Patch
@qwik.dev/react Patch
@qwik.dev/router Patch
create-qwik 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

@maiieul maiieul moved this from Backlog to Waiting For Review in Qwik Development May 5, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 5, 2026

Open in StackBlitz

@qwik.dev/core

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/core@8611

@qwik.dev/router

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/router@8611

eslint-plugin-qwik

npm i https://pkg.pr.new/QwikDev/qwik/eslint-plugin-qwik@8611

create-qwik

npm i https://pkg.pr.new/QwikDev/qwik/create-qwik@8611

@qwik.dev/optimizer

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/optimizer@8611

commit: d243c02

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview d243c02

@Varixo Varixo force-pushed the v2-out-of-order-streaming branch 6 times, most recently from a9b29e0 to d72b9f7 Compare May 9, 2026 12:14
@Varixo Varixo marked this pull request as draft May 9, 2026 17:07
@maiieul maiieul moved this from Waiting For Review to In progress in Qwik Development May 9, 2026
@Varixo Varixo marked this pull request as ready for review May 9, 2026 20:20
@maiieul maiieul moved this from In progress to Waiting For Review in Qwik Development May 9, 2026
@Varixo Varixo marked this pull request as draft May 10, 2026 06:07
@maiieul maiieul moved this from Waiting For Review to In progress in Qwik Development May 10, 2026
@Varixo Varixo marked this pull request as ready for review May 10, 2026 14:41
@maiieul maiieul moved this from In progress to Waiting For Review in Qwik Development May 10, 2026
@Varixo Varixo force-pushed the v2-out-of-order-streaming branch from ac0875f to 91119b6 Compare May 13, 2026 20:14
Copy link
Copy Markdown
Member

@wmertens wmertens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Initial comments

Comment thread packages/docs/src/routes/docs/labs/suspense/index.mdx
Comment thread packages/docs/src/routes/docs/labs/suspense/index.mdx Outdated
Comment thread packages/docs/src/routes/docs/labs/suspense/index.mdx Outdated
Comment thread packages/docs/src/routes/docs/labs/suspense/index.mdx
Comment thread packages/docs/src/routes/docs/labs/suspense/index.mdx Outdated
Comment thread packages/docs/src/routes/docs/labs/suspense/index.mdx
Comment thread packages/qwik/src/core/shared/serdes/serialize.ts
Copy link
Copy Markdown
Member

@wmertens wmertens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm worried about forwardRefs, can you write a test with Suspense and a Promise that only resolves after the first chunk, followed by a Promise that resolves before the first chunk?

Comment thread packages/qwik/src/core/shared/vnode-data-types.ts
@thejackshelton thejackshelton marked this pull request as draft May 19, 2026 05:59
@maiieul maiieul moved this from Waiting For Review to In progress in Qwik Development May 19, 2026
@wmertens wmertens force-pushed the v2-out-of-order-streaming branch from 97ec911 to 91119b6 Compare May 19, 2026 06:10
@Varixo Varixo force-pushed the v2-out-of-order-streaming branch 3 times, most recently from 4710287 to 102c52b Compare May 22, 2026 14:22
@Varixo Varixo marked this pull request as ready for review May 24, 2026 19:29
@maiieul maiieul moved this from In progress to Waiting For Review in Qwik Development May 24, 2026
@Varixo Varixo force-pushed the v2-out-of-order-streaming branch 5 times, most recently from dd781b6 to d7b43a3 Compare May 26, 2026 19:35
@Varixo Varixo force-pushed the v2-out-of-order-streaming branch from d7b43a3 to aef6bf0 Compare May 26, 2026 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Waiting For Review

Development

Successfully merging this pull request may close these issues.

3 participants