Skip to content

Harden vnode recognition - #997

Merged
JoviDeCroock merged 1 commit into
mainfrom
harden-vnode-recognition
Oct 27, 2026
Merged

JoviDeCroock merged 1 commit into
mainfrom
harden-vnode-recognition

Conversation

@JoviDeCroock

@JoviDeCroock JoviDeCroock commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

this makes use of the fact that v11 can use symbols

The challenge becomes versioning them together.

@netlify

netlify Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for preact-signals-demo ready!

Name Link
🔨 Latest commit 0aceb40
🔍 Latest deploy log https://app.netlify.com/projects/preact-signals-demo/deploys/6ae046e24265f1eee191a67b
😎 Deploy Preview https://deploy-preview-997--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.

@changeset-bot

changeset-bot Bot commented Oct 27, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0aceb40

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 Patch
preact-signals-devtools 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

@github-actions

Copy link
Copy Markdown
Contributor

Size Change: +67 B (+0.03%)

Total Size: 208 kB

📦 View Changed
Filename Size Change
docs/dist/assets/bench-********.js 1.6 kB -1 B (-0.06%)
docs/dist/assets/EmbeddedDevtools-********.js 23.2 kB -3 B (-0.01%)
docs/dist/assets/index-********.js 8.51 kB -1 B (-0.01%)
docs/dist/assets/signals.module-********.js 2.68 kB +11 B (+0.41%)
docs/dist/assets/Unmount-********.js 653 B +3 B (+0.46%)
docs/dist/assets/utils.module-********.js 519 B -1 B (-0.19%)
docs/dist/basic-********.js 246 B -2 B (-0.81%)
packages/devtools-ui/dist/devtools-ui.js 21.4 kB +10 B (+0.05%)
packages/devtools-ui/dist/devtools-ui.mjs 20.8 kB +17 B (+0.08%)
packages/preact/dist/signals.js 1.84 kB +10 B (+0.55%)
packages/preact/dist/signals.mjs 1.8 kB +24 B (+1.35%)
ℹ️ View Unchanged
Filename Size
docs/dist/assets/client-********.js 46.6 kB
docs/dist/assets/devtools-********.js 912 B
docs/dist/assets/jsxRuntime.module-********.js 300 B
docs/dist/assets/preact.module-********.js 4.74 kB
docs/dist/assets/signals-core.module-********.js 2.09 kB
docs/dist/assets/style-********.css 7.19 kB
docs/dist/nesting-********.js 1.14 kB
docs/dist/react-********.js 243 B
packages/core/dist/signals-core.js 2.09 kB
packages/core/dist/signals-core.mjs 2.09 kB
packages/debug/dist/debug.js 4.72 kB
packages/debug/dist/debug.mjs 4.24 kB
packages/devtools-adapter/dist/devtools-adapter.js 2.36 kB
packages/devtools-adapter/dist/devtools-adapter.mjs 2.07 kB
packages/preact-transform/dist/signals-transform.js 1.65 kB
packages/preact-transform/dist/signals-transform.mjs 1.61 kB
packages/preact-transform/dist/signals-transform.umd.js 1.77 kB
packages/react-transform/dist/signals-transform.js 7.63 kB
packages/react-transform/dist/signals-transform.mjs 6.77 kB
packages/react-transform/dist/signals-transform.umd.js 7.75 kB
packages/react/dist/signals.js 214 B
packages/react/dist/signals.mjs 165 B
packages/vite-plugin/dist/vite-plugin.js 8.87 kB
packages/vite-plugin/dist/vite-plugin.mjs 7.86 kB

compressed-size-action

@JoviDeCroock
JoviDeCroock marked this pull request as ready for review October 27, 2026 07:53
constructor: { configurable: true, value: undefined },
constructor: {
configurable: true,
value: createElement(Fragment, null).constructor,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@JoviDeCroock
JoviDeCroock merged commit 42d2c5f into main Oct 27, 2026
6 checks passed
@JoviDeCroock
JoviDeCroock deleted the harden-vnode-recognition branch October 27, 2026 08:11
@github-actions github-actions Bot mentioned this pull request Oct 27, 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