3508 Commits (goodbye-acorn)

Author	SHA1	Message	Date
Nic	f6980e14b0	teasel 0.0.14	6 days ago
Nic	e9ca5ecb22	teasel 0.0.13: a sequence before as in an each block	1 week ago
Nic	b6cf850197	svelte:element's this is promoted here, as upstream does it: a mixed value warns and takes its first chunk	1 week ago
Nic	4a143a3011	teasel 0.0.12	1 week ago
Nic	2dd7f7824f	a var declared again: its declarator has the binding, and an initializer there reassigns	1 week ago
Nic	bc75f67b03	teasel 0.0.11: one plan per parse	1 week ago
Nic	092ada80c2	release the source once it has parsed	2 weeks ago
Nic	415e67170f	teasel 0.0.10	2 weeks ago
Nic	0a2d6ee487	a declaration is a reference	2 weeks ago
Nic	461d4e940d	teasel 0.0.9	2 weeks ago
Nic	283650614f	teasel 0.0.8	2 weeks ago
Nic	65aec0fa1a	teasel 0.0.6	2 weeks ago
Nic	6ca0605435	fmt	2 weeks ago
Nic	35e1e0baed	a render tag needs a call, a duplicate clause and an elseif keep their own errors	2 weeks ago
Nic	b77026cf26	the parser at 0.0.5: a host's nodes carry no comments, a parse keeps its buffers	2 weeks ago
Nic	0f44bb0caa	comments in a tag stay in the list, the tag's nodes carry none	2 weeks ago
Nic	33f847ec4c	a js parse error points at its token, as before	2 weeks ago
Nic	969c6a9234	parenthesized stays the compiler's, off the public tree	2 weeks ago
Nic	b2d7d57b00	a this given as text warns again, at the attribute as before	2 weeks ago
Nic	9fb5350b2b	the parser at 0.0.4: the scope analysis declares first and resolves after, and a root's tables are whole again; teasel is ours for the release-age rule	2 weeks ago
Nic	b5099802b8	the parser comes from the registry	2 weeks ago
Nic	45e48602f6	the tables a root keeps are typed as a root's; the module's answer is cast where its option is known on	2 weeks ago
Nic	05bba23563	Merge remote-tracking branch 'origin/main' into goodbye-acorn ... # Conflicts: # packages/svelte/src/compiler/phases/1-parse/index.js # packages/svelte/src/compiler/phases/1-parse/state/element.js # packages/svelte/src/compiler/phases/1-parse/state/tag.js # packages/svelte/src/compiler/phases/2-analyze/visitors/Identifier.js # packages/svelte/src/compiler/phases/3-transform/utils.js # packages/svelte/src/compiler/phases/scope.js	2 weeks ago
Nic Polumeyv	6eb720a1b7	chore: scope.js stops importing from the analysis visitors (#18787) ... `phases/scope.js` imported `validate_identifier_name` from `2-analyze/visitors/shared/utils.js`, which imports `get_rune` back from `scope.js`. The check reads a `Binding` and `Scope.declare` is its main caller, so it now lives in `scope.js`. No behaviour change.	2 weeks ago
Nic Polumeyv	e6ae460caa	chore: analysis and transform stop importing each other (#18786) ... Analysis imported `should_proxy` from `3-transform/client/utils.js` and transform imported `check_graph_for_cycles` from `2-analyze/utils`. The graph search now lives in `compiler/utils`, `should_proxy` in `phases/scope.js` next to the bindings it reads. No behaviour change.	2 weeks ago
Nic Polumeyv	5981c060d2	chore: move parser helpers next to their users (#18785) ... Phase 1 imported `disallow_children` from phase 2, `fuzzymatch` sat in the parser's utils while only analysis and `extract_svelte_ignore` used it, `create_fragment` had a file to itself although `create_attribute` lives in `phases/nodes.js`, and `push_array` had a single caller. Each now sits next to its users: `create_fragment` and `disallow_children` in `phases/nodes.js`, `fuzzymatch` in `compiler/utils`, `push_array` inside `mapped_code.js`. Four files fewer, no behaviour change.	2 weeks ago
svelte-triage-bot[bot]	f2ad10ecd2	fix: preserve global CSS without scopable elements (#18793) ... ## Summary - mark fully global selectors as used without requiring a scopable component element - preserve exact ICSS `:export` rules under the same conditions - keep elements inside `<svelte:head>` excluded from CSS hash injection - add a regression test covering both reported selector forms Fixes #18792	2 weeks ago
svelte-triage-bot[bot]	a8a9b02e38	fix: reduce SSR render result garbage collection (#18798) ... Fixes #18797. ### Problem SSR `render()` is lazy, but the wrapper benchmark discarded its result and therefore measured almost no rendering work. When `.body` was consumed, each render result's own accessor properties caused substantial garbage-collection overhead. ### Fix - Consume `render(App).body` in both the warmup and measured SSR benchmark loops. - Move lazy render-result properties onto a shared `RenderResult` prototype. - Memoize synchronous output and the asynchronous render promise while preserving lazy sync/async API behavior. - Add a regression test ensuring exposed render-result properties are inherited. - Add a patch changeset for `svelte`. The benchmark now deliberately selects the synchronous lazy-render path by reading `.body`, so the process-wide async flag imported by the reactivity benchmarks does not determine this benchmark's render mode. ### Performance on this host | Case | Time | GC time | | --- | ---: | ---: | | main, output discarded | 0.20 ms | 0.00 ms | | main, `.body` consumed before fix | 163.04 ms | 62.68 ms | | `svelte@5.38.10` | 110.05 ms | 1.39 ms | | fixed, `.body` consumed | 104.79 ms | 2.43 ms | ### Tests - `pnpm test packages/svelte/src/internal/server/renderer.test.ts` — 53 passed - `pnpm test server-side-rendering` — 234 passed, 2 skipped - `pnpm test` — 7,771 passed, 55 skipped - `pnpm lint` - `pnpm check` - `pnpm format` - `git diff --check` <!-- svelte-triage-bot:feedback-baseline:116724e3b641c84ed56b2a60d494b6cb2298bbb3 --> Co-authored-by: svelte-triage-bot <team@svelte.com>	2 weeks ago
Nic Polumeyv	6be176df2f	fix: resolve the fallback of an each block in the enclosing scope (#18803) ... In `phases/scope.js`, `EachBlock` visits `node.fallback` with the block's own scope, so `{:else}` resolves the loop's name to the each context. With `let item = $state('outer')` outside and `{#each items as item}…{:else}<button onclick={() => (item = 'changed')}>` the compiler reports "Cannot reassign or bind to each block argument" for a variable that is not the block's argument. The fallback renders when there is nothing to iterate, so it is visited in the enclosing scope, as an await block's `pending` is.	2 weeks ago
Xia Chao	34b13ac3e4	fix: keep `$state.eager` when used as a variable initializer (#18809) ... fixes #18808 If you write `let x = $state.eager(...)`, the client compiler used to delete that line and then still read `x`. The page crashed with `x is not defined`. Server rendering kept the binding. Using `$state.eager(...)` directly in the markup already worked. This puts the `let` / `const` form on the same path as `$state.snapshot`. Added a runtime-runes sample that fails without this change. Co-authored-by: Xia Chao <236466140+bun-unsafe@users.noreply.github.com>	2 weeks ago
Nic	2cb85375aa	the parser says which tables each piece of JavaScript has, so the partition and the root collection go	2 weeks ago
Nic	a7f4c5c5bb	the template is read by the parser: one call per component, the compiler's shape put on the tree after it, the scope tables cut per expression for the analysis, the parser's errors mapped to ours; phase 1's own reader goes	3 weeks ago
Nic	57c7de58a5	one parser call: js.parse(entry, offset, { end, stopAt }) replaces the five parse-at methods, the answer is kept whole for its tables, and a scope's bindings are grouped here	3 weeks ago
Simon H	ce89035ecb	fix: prevent effect tree of batches from interfering with each other (#18508) ... While working on another Svelte feature I noticed a bug that boils down to batches interfering with each other through the effect tree: If batch A schedules an effect, it is walked up to the root (possibly). Now if in the meantime batch B also wants to schedule effects, it can have unintended consequences. Normally this does not happen, since it's extremely hard to run into this situation. There's basically two cases: Either an unfortunate timing of microtasks, or during flushing effects are scheduling new effects which messes with a `#commit()` right after (the test case does this). To fix this we now defer walking up the tree until the batch is actually processed. That way we set + unset the markers on the branches synchronously so there's no chance of another batch interfering.	3 weeks ago
Simon H	d16099ef0f	fix: remove `WAS_MARKED` flag in favor of `Set` (#18127) ... Removes the `WAS_MARKED` logic in favor of a simple `Set` heuristic: If `mark_reactions` goes beyond a certain count, initialize it, otherwise keep it `null`. Balances the common case of not having many transitive dependencies with the edge case of cyclic dependencies as was seen in #16658 (I rechecked that reproduction and it remains fast with this change). I ran the benchmark against this and it doesn't hit the `Seen` heuristic once (i.e. even the benchmark doesn't have this extreme level of dependencies where it becomes noticeable). Fixes #18123, supersedes #18124 --------- Co-authored-by: Rich Harris <rich.harris@vercel.com>	3 weeks ago
Nic	5d7a91864c	a generic snippet's type parameters are parsed, not scanned: <T extends () => void> works and bracket.js is gone	3 weeks ago
Nic	6632b93c8b	the parser says what was in parens: the four scans back through the source for a ( go	3 weeks ago
Nic	394fa2c588	a parse error reaches the host as the span the parser saw, and the end of the input is the host's own error whatever was being read	3 weeks ago
Nic	3ee5cc64af	loose mode asks the parser to recover: the bracket scanner, the synthetic declaration and the paren lookahead go, a placeholder spans what could not be read	3 weeks ago
Nic	76c64bd0ce	stop sets instead of rewinds: the parser ends the list at as or the comma, the promise at then or catch, a value at />	3 weeks ago
Nic	9bc9689083	the parser's tables sit on the root they belong to, and a reference is one record: what resolves it and what the parser saw ride along	3 weeks ago
Nic	ea1ef3a067	a template expression that declares nothing takes the short way through the tables	3 weeks ago
Nic	69b0bcd0c8	scopes come from the parser's tables: a script is no longer walked, a template's javascript neither, and a reference finds its ancestors through the parser's parent links	3 weeks ago
Nic	b7451891c2	scopes take a declaration's initial value, a write's value and top-level await from the parser instead of walking for them	3 weeks ago
Nic	0bc2e55cae	fmt	3 weeks ago
Nic	03ee8b51bf	create_fragment and disallow_children live in nodes.js, fuzzymatch in utils; no parse imports from analyze	3 weeks ago
Nic	652da2c81b	css, script and options readers at the folder root; read/ is gone	3 weeks ago
Nic	803476f793	readers move the cursor themselves; read/context and read/expression fold into js.js	3 weeks ago
Nic	e5e5f4d40f	script references resolved by the parser	3 weeks ago
Nic	3bcf7b6a73	scopes and declarations of scripts from the parser's tables	3 weeks ago