Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 0 additions & 17 deletions .changeset/atproto-comments-external-svelte.md

This file was deleted.

18 changes: 18 additions & 0 deletions components/atproto-comments/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,23 @@
# @svebcomponents/atproto.comments

## 0.2.1

### Patch Changes

- a8212eb: Ship a Svelte-external build and route bundler consumers to it via the `svelte`
export condition, fixing a hydration crash (`Cannot read properties of null
(reading 'f')`) that wiped a correctly server-rendered thread on the client.

The client build bundled its own Svelte runtime. In a SvelteKit/Vite app —
which has its own Svelte — that put **two** Svelte runtimes on the page, and
under `experimental.async` the app's async-boundary hydration would cross into
the component's separate runtime and dereference a null effect. The component
now also emits `dist/client-svelte`/`dist/server-svelte` (Svelte marked
external), and its package `exports` declare a `svelte` condition pointing at
them, so bundlers dedupe the component onto the app's single Svelte runtime.
The default `dist/client` (Svelte bundled) is unchanged and still serves the
CDN / bare-`import` drop-in, which has no app Svelte to dedupe against.

## 0.2.0

### Minor Changes
Expand Down
2 changes: 1 addition & 1 deletion components/atproto-comments/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@svebcomponents/atproto.comments",
"version": "0.2.0",
"version": "0.2.1",
"type": "module",
"jsdelivr": "./dist/client/index.js",
"description": "Web component rendering ATProto/Bluesky threads as blog comments",
Expand Down