Skip to content

Add fallback names to signal transforms - #960

Merged
JoviDeCroock merged 1 commit into
mainfrom
jdv/fallback-reactive-names
Jul 23, 2026
Merged

Add fallback names to signal transforms#960
JoviDeCroock merged 1 commit into
mainfrom
jdv/fallback-reactive-names

Conversation

@JoviDeCroock

Copy link
Copy Markdown
Member

Unnamed effects and reactive primitives are difficult to identify in debugging tools when the transform cannot derive a contextual name.

Transform effects and signal effects too, and fall back to file:line while preserving explicit names.

Assisted-By: devx/633e51ab-4160-4b2c-a196-017418800531
@changeset-bot

changeset-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 64e040e

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

This PR includes changesets to release 2 packages
Name Type
@preact/signals-preact-transform Patch
@preact/signals-react-transform 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

@netlify

netlify Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploy Preview for preact-signals-demo ready!

Name Link
🔨 Latest commit 64e040e
🔍 Latest deploy log https://app.netlify.com/projects/preact-signals-demo/deploys/6a60c3b4fc925000091daf58
😎 Deploy Preview https://deploy-preview-960--preact-signals-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

Copy link
Copy Markdown
Contributor

Size Change: -7.75 kB (-4.08%)

Total Size: 182 kB

📦 View Changed
Filename Size Change
docs/dist/assets/index-********.js 838 B -7.59 kB (-90.06%) 🏆
packages/preact-transform/dist/signals-transform.js 1.66 kB -29 B (-1.72%)
packages/preact-transform/dist/signals-transform.mjs 1.61 kB -27 B (-1.65%)
packages/preact-transform/dist/signals-transform.umd.js 1.77 kB -33 B (-1.83%)
packages/react-transform/dist/signals-transform.js 7.63 kB -24 B (-0.31%)
packages/react-transform/dist/signals-transform.mjs 6.78 kB -26 B (-0.38%)
packages/react-transform/dist/signals-transform.umd.js 7.75 kB -24 B (-0.31%)
ℹ️ View Unchanged
Filename Size
docs/dist/assets/bench-********.js 1.6 kB
docs/dist/assets/client-********.js 46.6 kB
docs/dist/assets/devtools-********.js 912 B
docs/dist/assets/EmbeddedDevtools-********.js 17.8 kB
docs/dist/assets/jsxRuntime.module-********.js 300 B
docs/dist/assets/preact.module-********.js 4.74 kB
docs/dist/assets/signals-core.module-********.js 1.92 kB
docs/dist/assets/signals.module-********.js 2.65 kB
docs/dist/assets/style-********.css 5.26 kB
docs/dist/assets/Unmount-********.js 652 B
docs/dist/assets/utils.module-********.js 516 B
docs/dist/basic-********.js 250 B
docs/dist/nesting-********.js 1.15 kB
docs/dist/react-********.js 242 B
packages/core/dist/signals-core.js 1.94 kB
packages/core/dist/signals-core.mjs 1.93 kB
packages/debug/dist/debug.js 4.64 kB
packages/debug/dist/debug.mjs 4.15 kB
packages/devtools-adapter/dist/devtools-adapter.js 2.36 kB
packages/devtools-adapter/dist/devtools-adapter.mjs 2.07 kB
packages/devtools-ui/dist/devtools-ui.js 16.3 kB
packages/devtools-ui/dist/devtools-ui.mjs 15.7 kB
packages/preact/dist/signals.js 1.81 kB
packages/preact/dist/signals.mjs 1.74 kB
packages/react/dist/signals.js 214 B
packages/react/dist/signals.mjs 165 B
packages/vite-plugin/dist/vite-plugin.js 8.86 kB
packages/vite-plugin/dist/vite-plugin.mjs 7.86 kB

compressed-size-action

@JoviDeCroock
JoviDeCroock merged commit 3d0f647 into main Jul 23, 2026
6 checks passed
@JoviDeCroock
JoviDeCroock deleted the jdv/fallback-reactive-names branch July 23, 2026 07:53
@github-actions github-actions Bot mentioned this pull request Jul 23, 2026
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.

2 participants