Skip to content

Conversation

@bgw
Copy link
Member

@bgw bgw commented Nov 29, 2025

This is currently broken.

Removing the automatic serde derives from #[turbo_tasks::value] should remove most of the macro expansion. I'll follow up with removing all the manual derives.

@bgw bgw changed the title Turbopack: Remove automatic serde derives from #[turbo_tasks::value] macro WIP: Turbopack: Remove automatic serde derives from #[turbo_tasks::value] macro Nov 29, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 29, 2025

CodSpeed Performance Report

Merging #86634 will not alter performance

Comparing bgw/rm-value-serde (8a3f543) with bgw/bincode-task-inputs (7002c09)

Summary

✅ 17 untouched
⏩ 3 skipped1

Footnotes

  1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@bgw bgw force-pushed the bgw/rm-value-serde branch from 99ed5bc to 8a3f543 Compare December 1, 2025 05:16
@bgw bgw force-pushed the bgw/bincode-task-inputs branch from b68b533 to 7002c09 Compare December 1, 2025 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

created-by: Turbopack team PRs by the Turbopack team. Turbopack Related to Turbopack with Next.js.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants