11398 Commits (e5e5f4d40fa39f7868fa16d7c33ba6e322c2cd11)

 

All Branches   

Search

Author	SHA1	Message	Date
Nic	e5e5f4d40f	script references resolved by the parser	24 hours ago
Nic	3bcf7b6a73	scopes and declarations of scripts from the parser's tables	24 hours ago
Nic	2abd971127	references from the parser: is-reference goes, what the compiler builds is judged by position	1 day ago
Nic	77f9de99aa	template expressions always go through the parser	1 day ago
Nic	ebe8f5e827	accessor fields reported through the parser's manifest	1 day ago
Nic	052ee6325f	parse errors carry a code, no message regex	1 day ago
Nic	f3b399f9b1	scripts parse in place, typescript erased by the parser	2 days ago
Nic	0a61385aa9	parse with teasel instead of acorn	2 days ago
Bao Nguyen	5895c637b0	fix: recognise `aria-braillelabel` and `aria-brailleroledescription` as known ARIA attributes (#18765)	4 days ago
JY Wey	9e69544238	fix: in non-async mode, only push variable to current_sources when active_reaction is updating (#18550) ... fix: #16814 in non-async mode, when the derived inside `SvelteDate` is created and evaluated, the `#reaction` is from the `active_reaction` when the `SvelteDate` instance is created, which could possible in NON_UPDATING state, so the derived should not be added to `current_sources` to prevent re-run. --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com> Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	4 days ago
Simon H	6f7857af59	chore: compiler profiling + parser bench (#18767) ... Used these to speed up parsing/find compilation bottlenecks. Useful for later analysis therefore committing it.	4 days ago
Bao Nguyen	4bf15ae6f3	fix: throw when `setContext` is called after an `await` during SSR (#18739) ... Fixes #17233.	7 days ago
Simon H	9d91a40faf	chore: bump packages (#18752) ... esrap bump ensures a bugfix, devalue silences vulnerability reports	1 week ago
Simon H	b580455c9c	fix: serialize input default values during SSR (#18733) ... Alternative to #18452 --------- Co-authored-by: svelte-triage-bot <team@svelte.com>	1 week ago
Simon H	b2c22ab66d	chore: add regression test for teardown value behavior (#18751) ... #18746 would've introduced a bug that makes this test fail. Add it now so we don't regress in the future	1 week ago
Simon H	b1142efe96	perf: speed up parser a little bit (#18736) ... ~5% in local benchmarks by avoiding regex in some places	1 week ago
Nic Polumeyv	34142af3a1	perf: build identifier and member chain expressions without acorn (#18740) ... After #18738 the biggest thing left in the parser profile was acorn's constructor, `getOptions` and `wordsRegexp` alone were around 12% of parse time. `parseExpressionAt` is `new Parser()` followed by `parseExpression()`, and we call it for every expression in the template. For something like `{name}` the constructor (normalizing options, four keyword regexes, scope state) is about 2 µs while the actual parse is under 1 µs, and components have dozens of these. The fix seemed to belong in acorn, cache the options and regexes so repeated `parseExpressionAt` calls stop redoing them. I lasted about twenty minutes in acorn's constructor... I was way in over my head. The Svelte alternative was one parser per component, reset between expressions, but that means re-initializing acorn's and acorn-typescript's internals by hand and breaking whenever either adds a field. I realized most of the expressions in the test corpus are an identifier or an `a.b.c` chain, and you don't need a JS parser for those. So that's this fix. `read_expression` now scans for that shape, builds the nodes itself in the form acorn would, and hands everything else to acorn like before.No behavior change, verified byte-for-byte against acorn on the test corpus. About 18% faster parsing on typical components on top of #18738. I guess this should be a fix for acorn, rather than here, so I'm not against closing this. --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com> Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	1 week ago
Nic Polumeyv	504a7536c6	perf: stop scanning the whole template for every expression (#18738) ... Parsing a 90 KB component drops from 460 ms to 35 ms; typical components parse ~30% faster. `read_pattern` blanked out the entire template before the pattern with `replace(/[^\n]/g, ' ')` to keep positions aligned, and `acorn.parseExpressionAt` re-counted lines from the top of the file on every call. Acorn never looks before `pos`, so the real template can be the prefix, and acorn 8.18's `startLocation` takes the line we already have from the locator. The blanking also had `loc.column` off by one for every node inside a destructured pattern (noted in #18087), hence the snapshot updates.	1 week ago
svelte-triage-bot[bot]	214dd3d005	fix: cancel deferred event listener attachment (#18749) ... Fixes #18748	1 week ago
github-actions[bot]	7bc0a70fe6	Version Packages (#18682) ... This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## svelte@5.57.0 ### Minor Changes - feat: export `RenderOutput`, `SyncRenderOutput`, `Csp` and `Sha256Source` from `svelte/server` ([#18648](https://github.com/sveltejs/svelte/pull/18648)) - feat: add `has` function to `createContext` ([#18472](https://github.com/sveltejs/svelte/pull/18472)) - feat: support `defaultValue` on `<select>` ([#18591](https://github.com/sveltejs/svelte/pull/18591)) - feat: add getOrInsert/getOrInsertComputed to SvelteMap ([#18728](https://github.com/sveltejs/svelte/pull/18728)) ### Patch Changes - fix: block template store subscriptions on the promise that assigns the store ([#18582](https://github.com/sveltejs/svelte/pull/18582)) - fix: route $derived teardown errors through invoke_error_boundary ([#18486](https://github.com/sveltejs/svelte/pull/18486)) - fix: track SvelteDate snapshots in reactions ([#18700](https://github.com/sveltejs/svelte/pull/18700)) - fix: remove `<svelte:head>` anchors on unmount ([#18697](https://github.com/sveltejs/svelte/pull/18697)) - fix: warn on undeclared shorthand event handlers on `<svelte:window>`, `<svelte:document>` and `<svelte:body>` ([#18480](https://github.com/sveltejs/svelte/pull/18480)) - perf: reuse the cached value in the `<option>`/`<select>` value guard ([#18713](https://github.com/sveltejs/svelte/pull/18713)) - fix: prevent malformed AST output for `<select>` with static `value` attribute ([#18449](https://github.com/sveltejs/svelte/pull/18449)) - fix: apply ownership mutation ignores to binding assignments ([#18718](https://github.com/sveltejs/svelte/pull/18718)) - fix: prevent onoutroend from firing twice when compilerOptions.hmr is true ([#18655](https://github.com/sveltejs/svelte/pull/18655)) - fix: preserve whitespace after inline elements when printing ([#18685](https://github.com/sveltejs/svelte/pull/18685)) - perf: fold SSR block-open markers into the branch's first push ([#18712](https://github.com/sveltejs/svelte/pull/18712)) - fix: run `onDestroy` callbacks when a server render throws ([#18585](https://github.com/sveltejs/svelte/pull/18585)) - fix: report `derived_invalid_export` for `export let x = $derived(...)` in runes mode ([#18692](https://github.com/sveltejs/svelte/pull/18692)) - fix: never apply class hash to elements inside `<svelte:head>` ([#18160](https://github.com/sveltejs/svelte/pull/18160)) - fix: keep `defaultChecked` on hydrated radio inputs with spread attributes ([#18701](https://github.com/sveltejs/svelte/pull/18701)) - fix: accept `onfocusin`/`onfocusout` in `a11y_mouse_events_have_key_events` ([#18689](https://github.com/sveltejs/svelte/pull/18689)) - perf: O(n²)→O(n) Map lookups for legacy `$:` reactive statement ordering ([#18602](https://github.com/sveltejs/svelte/pull/18602)) - fix: distinct memoizer on style/class directives ([#18466](https://github.com/sveltejs/svelte/pull/18466)) - fix: measure nested transitions before applying their starting styles ([#18647](https://github.com/sveltejs/svelte/pull/18647)) - fix: don't turn component instances stored in `$state` into state proxies ([#18646](https://github.com/sveltejs/svelte/pull/18646)) - perf: emit `$.only_child` for elements with a single child ([#18717](https://github.com/sveltejs/svelte/pull/18717)) - fix: omit `bind:focused` from SSR output (it has no HTML attribute) ([#18724](https://github.com/sveltejs/svelte/pull/18724)) - fix: more robust rendering of Svelte custom element slots ([#18710](https://github.com/sveltejs/svelte/pull/18710)) - perf: optimize simple object destructuring in `@const` tags ([#18390](https://github.com/sveltejs/svelte/pull/18390)) - fix: properly apply static textarea value attribute during CSR ([#18727](https://github.com/sveltejs/svelte/pull/18727)) - fix: end a restored reaction context at the end of its synchronous segment ([#18694](https://github.com/sveltejs/svelte/pull/18694)) - fix: keep the dependencies of a reaction that throws, so deriveds it read are neither leaked nor stuck in their error ([#18703](https://github.com/sveltejs/svelte/pull/18703)) - fix: don't resurrect outroing elements when an ancestor block is paused and resumed ([#18431](https://github.com/sveltejs/svelte/pull/18431)) - perf: use `$.comment()` for single-comment templates ([#18714](https://github.com/sveltejs/svelte/pull/18714)) - chore: move `@types/trusted-types` to devDependencies ([#18730](https://github.com/sveltejs/svelte/pull/18730)) - perf: store setters cache as `Set` instead of `Array` ([#18251](https://github.com/sveltejs/svelte/pull/18251)) - fix: transform derived assignments and select function bindings correctly during server-side rendering ([#18669](https://github.com/sveltejs/svelte/pull/18669)) - fix: keep boolean attributes with an empty string value when rendering attribute objects on the server ([#18721](https://github.com/sveltejs/svelte/pull/18721)) - fix: sync `SvelteURL` port signal when the protocol setter clears the port ([#18705](https://github.com/sveltejs/svelte/pull/18705)) - fix: block declaration tags and `{@const}` on async values read inside closures ([#18533](https://github.com/sveltejs/svelte/pull/18533)) - fix: avoid css tree-shaking for exported Snippet ([#18540](https://github.com/sveltejs/svelte/pull/18540)) - perf: treat `<img loading>` as a static element again ([#18711](https://github.com/sveltejs/svelte/pull/18711)) - fix: prevent `selectedcontent` mutation from changing the selected option ([#18495](https://github.com/sveltejs/svelte/pull/18495)) - fix: avoid `NaN` keyframe values in `slide` transition for elements without a layout box ([#18430](https://github.com/sveltejs/svelte/pull/18430)) - fix: preserve line feed character references in attribute values ([#18691](https://github.com/sveltejs/svelte/pull/18691)) - fix: decode uppercase-`X` hex numeric character references (`&#X...;`) ([#18708](https://github.com/sveltejs/svelte/pull/18708)) - chore: clarify when `$effect.pre` runs relative to DOM updates ([#18534](https://github.com/sveltejs/svelte/pull/18534)) - fix: scope SSR boundary failed snippets to their boundary ([#18593](https://github.com/sveltejs/svelte/pull/18593)) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>	1 week ago
Nic Polumeyv	6374a2a0db	chore: move `@types/trusted-types` to devDependencies (#18730) ... Only referenced by a JSDoc `@import` in `src/internal/client/dom/blocks/html.js`; nothing in the published `types/index.d.ts` uses it, so consumers do not need it installed.	1 week ago
Adnane Taghi	4ae5eb3f23	feat: add has function to createContext (#18472) ... closes #18471. --------- Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	2 weeks ago
Simon H	5034b59e9e	chore: bump playwright (#18729) ... supports later ubuntu versions	2 weeks ago
Simon H	74197ccbe1	feat: add getOrInsert/getOrInsertComputed to SvelteMap (#18728) ... Closes #18014	2 weeks ago
Simon H	edbe11ebc9	fix: properly apply static textarea value attribute during CSR (#18727) ... Fixes #18574 Simpler alternative to #18627 - we just special-case this in the "should this be treated as a dynamic attribute" branches so `$.set_value(...)` is always generated. Given how edge-case-y this is (why would you not just write it into the body of the textarea?) this is fine.	2 weeks ago
Xia Chao	05b69164d1	fix: omit bind:focused from SSR output (#18724) ... --------- Co-authored-by: Xia Chao <236466140+bun-unsafe@users.noreply.github.com> Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	2 weeks ago
Simon H	08c2a72907	chore: set Vitest reporter for github actions (#18722) ... test failures show up really bad now, minimal is way better for github actions	2 weeks ago
Nic Polumeyv	7c6f36aa92	fix: keep boolean attributes with an empty string value when rendering attribute objects on the server (#18721) ... `<input disabled="" {...props}>` and any other boolean attribute written as `""` now renders on the server the way it does on the client and in plain markup.	2 weeks ago
Nic Polumeyv	2e8b57bc58	fix: render `selected` options for `<select multiple>` with an array `value` on the server (#18720) ... Follow-up to #18591. --------- Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	2 weeks ago
Nic Polumeyv	864de81bb6	fix: keep the current selection of a `<select>` when its `defaultValue` is applied (#18719) ... Also ensure init_select isn't called multiple times Follow-up to #18591.	2 weeks ago
Nic Polumeyv	4d5139552d	fix: block declaration tags on async values read inside closures (#18533) ... Fixes #18469 Declaration tags collect blockers from `metadata.expression.dependencies`, which only sees eager reads, so a `$derived` reading an async declaration inside a closure gets no blocker and evaluates while the value is still `undefined`. Align with template expressions which already collect blockers from `references` --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com> Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	2 weeks ago
Rich Harris	aadc97ce1b	fix: simplify boundary error handling and handle hydration errors correctly (#18539) ... remove the `catch` clause in `this.#run` - in two locations there's already a catch, and in the hydration-location we have to add special handling for hydration errors. --------- Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	2 weeks ago
Sushanth012	bfc9b65b3d	docs: clarify mounting components with context (#18515) ... Closes #18462.	2 weeks ago
Bao Nguyen	0f458d5aba	fix: report `derived_invalid_export` for `export let x = $derived(...)` (#18692) ... Fixes #15842 Run the more targeted validations first --------- Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	2 weeks ago
Khaled Waleed	955b701df2	fix: measure nested transitions before applying their starting styles (#18647) ... Fixes #18421 (regression from #16035, which re-broke the #4784 scenario). Since #16035 the dummy delay animation runs with `fill: 'forwards'`, so an intro's `t = 0` styles (for `slide`, `height: 0`) apply the moment the transition is created and stay applied until the real animation starts a frame later. Transition functions of other elements in the same batch run after that and measure a collapsed DOM. With nested intros (the TreeView from the issue), intro effects run deepest-first, so every ancestor's `slide` captures a height where its children are pinned at 0. On the issue's REPL every `<ul>` animates `0px -> 36px` (the height of a single row) and then jumps to its real size when the animation finishes, which is the "children pop out" behavior from #4784. The fix waits one microtask before applying the initial styles and creating the dummy, the same way deferred transitions (`crossfade`) already work in `animate()`. Everything in the batch measures the DOM first; the microtask still runs before the next paint, so elements never render unstyled and the #14732 behavior is preserved. Verified in a real browser that a delayed intro (`delay: 300`) stays held at `height: 0` for the whole delay, including the first painted frame. Most of the diff is the existing dummy/onfinish block moving into `queue_micro_task`; hiding whitespace shows the actual change.	2 weeks ago
svelte-triage-bot[bot]	c7d8233a5c	fix: apply ownership mutation ignores to bindings (#18718) ... Fixes #18715. Copy ignore comment info to the generated assignment so it's recognized later --------- Co-authored-by: svelte-triage-bot <team@svelte.com> Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	2 weeks ago
Simon H	68e7c9b521	fix: transform derived assignments and select function bindings correctly during server-side rendering (#18669) ... follow-up to the fix in #18607 - there are a two more bugs related to writeable deriveds in that area	2 weeks ago
Mathias Picker	14ec75c5fe	perf: fold SSR block-open markers into the branch's first push (#18712) ... One fewer `$$renderer.push` per rendered branch	2 weeks ago
Ion	c575b07bca	feat: support `defaultValue` on `<select>` (#18591) ... Closes #18447 --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com> Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com>	2 weeks ago
Bao Nguyen	b433f3cd87	fix: preserve line feed character references in attribute values (#18691) ... Fixes #15555 Pass on whether we're in an attribute to not replace the charcode with the whitespace charcode in that case	2 weeks ago
Mathias Picker	34489c10df	perf: emit `$.only_child` for elements with a single child (#18717) ... `<p>{text}</p>` and friends compile to `var x = $.child(p, true); $.reset(p);`. Across the 4,020 components in `packages/svelte/tests`, that exact pair occurs 1,885 times, more than any other adjacent statement pair. `$.only_child` does both, reducing the number of statements in the compiled output	2 weeks ago
Mathias Picker	6b99f9fb52	perf: reuse the cached value in the `<option>`/`<select>` value guard (#18713) ... One fewer evaluation of the value expression per update. --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	2 weeks ago
Mathias Picker	8299cbf11b	perf: use `$.comment()` for single comment templates (#18714) ... Move the "single comment, we can make this faster" optimization into `transform_template` so all usage locations can benefit (e.g. select/option elements) --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	2 weeks ago
Mathias Picker	359413f512	perf: treat `<img loading>` as a static element again (#18711) ... Emits less code and skips two live DOM traversals per instance. `is_static_element` treats `<img>` with a `loading` attribute as non-static. That was added in #14237 so `RegularElement` could emit `$.handle_lazy_img(node_id)`, which needed a reference to the element. #15272 deleted `handle_lazy_img` (replaced by `importNode`) and removed the call site, but left this check behind. --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	2 weeks ago
Simon H	135f1ec004	fix: more robust rendering of Svelte custom element slots (#18710) ... Instead of checking the given nodes at startup and only injecting those into the inner Svelte component constructor, we are now detecting (via the `$$host` property) whether or not the component is rendered as a custom element, and in that case create all slots as real slots right away. Fixes #13638 Fixes #8997 (therefore closes #8999)	2 weeks ago
Jawad Ali	c894afd8c7	fix: sync SvelteURL port when protocol setter clears it (#18705) ... Per the WHATWG URL spec, assigning a new protocol can clear the URL's port when the current port equals the new scheme's default port. Therefore also `set` the port when the protocol is updated.	2 weeks ago
Nic Polumeyv	6b16b7f73f	fix: keep defaultChecked on hydrated radio inputs with spread attributes (#18701) ... radio buttons were missing in the type check - remove the check instead (aligns with how we do it in the compiler, see `has_default_value_attribute`). Strictly speaking not fully correct but pragmatic solution that doesn't have any impact in practise Needed by sveltejs/kit#16926.	2 weeks ago
Jawad Ali	1b02aa28c8	fix: decode uppercase-X hex numeric character references (#18708) ... uppercase X is also valid per the HTML spec	2 weeks ago
Nic Polumeyv	2b740e496e	fix: commit a reaction's dependencies even when it throws (#18703) ... Fixes #18414 and a second bug with the same cause. `update_reaction` in `runtime.js` commits a reaction's dependencies after `fn()` returns: it swaps in `new_deps`, removes stale reactions and registers the reaction into each new dep's `reactions`. When `fn()` throws, the whole block is skipped. Two things go wrong from that. A derived that the run read for the first time was already set `CONNECTED` in `get` and registered itself into its own deps' `reactions` during `update_derived`, but the throwing reaction never registers as its reader. It sits in its deps' `reactions` with `reactions === null` of its own, `remove_reaction`'s disconnect cascade can never reach it, and it retains its `ctx`, closures and DOM past component destruction and `unmount()`. A derived that throws during its own run ends up with no dependencies at all. It is not in its sources' `reactions`, so a later change to them never re-runs it, and for an unowned derived `update_derived` then marks the error value CLEAN because the derived has no deps. Anything reading it keeps getting the first error forever. The commit block is now `update_dependencies` and the `catch` calls it before `handle_error`, so a failing reaction keeps the deps it read up to the throw. That subscribes it to the inputs that produced the error, and lets the normal disconnect cascade clean up when it is destroyed (`Boundary.#handle_error` destroys the failed effect). Co-authored-by: Randy Murphy <randalmurphal@users.noreply.github.com>	2 weeks ago
Mathias Picker	fdb3aa8ba8	perf: store setters cache as Set instead of Array (#18251) ... --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>	2 weeks ago