11451 Commits (svelte-custom-renderer)

 

All Branches   

Search

Author	SHA1	Message	Date
Simon H	97d45f85c6	fix: don't override `$destroy/set/on` instance methods in dev mode (#18034) ... Don't think we need a test for this Fixes #17988	5 months ago
paoloricciuti	e0a5e4fa35	fix: pass all the args to custom renderer events	5 months ago
paoloricciuti	e2cedcd93e	fix: don't lowercase attributes with custom renderers	5 months ago
paoloricciuti	a0650d0611	fix: events don't go through propagation	5 months ago
Rich Harris	aba4ab8737	chore: fix editorconfig (#18033) ... I don't know why exactly, but VS Code suddenly started paying attention to this line. An indent size of 2 inside an IDE is for masochists and barbarians. If that's your preference then that's up to you but there's certainly no reason to impose it on everyone who clones this repo.	5 months ago
paoloricciuti	700454d77c	fix: lint	5 months ago
paoloricciuti	c573720d39	chore: better types	5 months ago
paoloricciuti	1270d5d434	chore: cleanup	5 months ago
paoloricciuti	206bdf42ed	fix: allow usage of `$props.id`	5 months ago
paoloricciuti	61ea8e9f19	chore: cleanup	5 months ago
paoloricciuti	a0b60cb137	fix: handle context withing `render`	5 months ago
paoloricciuti	8270a2fd55	fix: make sure returned elements are not primitives	5 months ago
paoloricciuti	13ae09b861	fix: throw descriptive error if using `createRawSnippet` using custom renderer	5 months ago
paoloricciuti	c322fdff25	fix: `validate_snippet_arguments`	5 months ago
paoloricciuti	1886dc9aab	chore: cleanup	5 months ago
paoloricciuti	38dc4aee42	chore: cleanup	5 months ago
paoloricciuti	22420042d4	chore: cleanup	5 months ago
paoloricciuti	9167c8df4d	fix: disallow `bind` and `transitions`	5 months ago
paoloricciuti	df42e4babd	fix: add validations for `svelte:dom` elements	5 months ago
paoloricciuti	098cdebe95	fix: disallow css injected with custom renderer	5 months ago
paoloricciuti	100fbbf05f	fix: conditional invocation	5 months ago
paoloricciuti	c189bdcfe3	chore: make option accept a function	5 months ago
paoloricciuti	b4cf3d0a71	chore: move option under experimental	5 months ago
paoloricciuti	f91c2fd6d8	fix: use operations functions everywhere	5 months ago
paoloricciuti	ff5e835bf0	fix: handle autofocus and muted	5 months ago
paoloricciuti	2dc8fd19a3	chore: add full object in output	5 months ago
paoloricciuti	a0d0f013da	fix: un-special case customizable-select	5 months ago
paoloricciuti	6830052254	fix: style and class special case	5 months ago
paoloricciuti	37bd61d3f1	fix: attributes dom access	5 months ago
paoloricciuti	7dca98564a	chore: add each reactive and key tests	5 months ago
paoloricciuti	5456125bab	Merge remote-tracking branch 'origin/main' into svelte-custom-renderer	5 months ago
paoloricciuti	beddf45690	chore: render into a fragment in tests	5 months ago
paoloricciuti	0f44733183	chore: boundary test	5 months ago
paoloricciuti	f828ea3c6a	chore: better test renderer	5 months ago
paoloricciuti	e86da84c79	fix: handle compiled `nodeValue`/`textContent`	5 months ago
paoloricciuti	ecffd81c22	fix: types	5 months ago
paoloricciuti	49cff549c7	chore: initial test suite	5 months ago
paoloricciuti	bd70ab621c	fix: events dom access	5 months ago
paoloricciuti	cf874c0ab9	fix: generate types	5 months ago
paoloricciuti	c459824a88	fix: don't clone node if a renderer is available	5 months ago
paoloricciuti	9ae3faa09a	chore: centralize methods in `operations.js`	5 months ago
paoloricciuti	432c0235c9	fix: push and pop renderer when needed	5 months ago
paoloricciuti	a59efbf610	fix: wrap component in boundary in `render`	5 months ago
github-actions[bot]	37ab33c0a3	Version Packages (#18006) ... 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.55.1 ### Patch Changes - fix: correctly handle bindings on the server ([#18009](https://github.com/sveltejs/svelte/pull/18009)) - fix: prevent hydration error on async `{@html ...}` ([#17999](https://github.com/sveltejs/svelte/pull/17999)) - fix: cleanup `superTypeParameters` in `ClassDeclarations`/`ClassExpression` ([#18015](https://github.com/sveltejs/svelte/pull/18015)) - fix: improve duplicate module import error message ([#18016](https://github.com/sveltejs/svelte/pull/18016)) - fix: reschedule new effects in prior batches ([#18021](https://github.com/sveltejs/svelte/pull/18021)) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>	5 months ago
Rohit Nair P	4879f9da99	fix: improve duplicate module import error message (#18016) ... ## Description **While investigating the compiler analysis phase for errors or missing lines, I located an explicit `TODO fix the message here` comment in the source code (`packages/svelte/src/compiler/phases/2-analyze/visitors/shared/utils.js`).** ### Triggering Code ```svelte <script module> import { foo } from './somewhere.js'; </script> <script> let foo = 'conflict'; // Triggers the duplicate module import error </script> ``` ## The Bug When a component author creates a local `let` declaration in the instance `<script>` block that shadows a variable imported in the `<script module>` block, the compiler emits a `declaration_duplicate_module_import` error. However, the error message text was misleading. It stated: ```diff - Cannot declare a variable with the same name as an import inside `<script module>` + Cannot declare a variable with the same name as an import from `<script module>` ``` This incorrectly implies that the duplicate declaration itself is happening *inside* the module script block, rather than colliding with an import *from* that block. ## Changes Made | Action | Target (File / Function) | Description | | :--- | :--- | :--- | | **Template Fix** | `packages/svelte/messages/compile-errors/script.md` | Changed "inside" to "from" to accurately reflect the nature of the shadow collision. | | **Regenerated Errors** | `packages/svelte/src/compiler/errors.js` | Ran `pnpm generate` to apply the updated markdown template into the dictionary list. | | **Removed TODO** | `ensure_no_module_import_conflict` function | Removed the original `// TODO fix the message here` notation from the validator. | | **Test Adjustments** | `illegal-variable-declaration/errors.json` | Updated the validator test suite snapshots to assert against the correct error message format. | ## Validation Results * The error correctly throws when a user replicates this component structure. * Vitest suite `tests/validator/test.ts` successfully asserts the new, clearer message. --- ### Before submitting the PR, please make sure you do the following - [x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs - [x] Prefix your PR title with `feat:`, `fix:`, `chore:`, or `docs:`. - [x] This message body should clearly illustrate what problems it solves. - [x] Ideally, include a test that fails without this PR but passes with it. - [x] If this PR changes code within `packages/svelte/src`, add a changeset (`npx changeset`). ### Tests and linting - [x] Run the tests with `pnpm test` and lint the project with `pnpm lint` # THANK YOU	5 months ago
Simon H	04eadbc8a9	fix: correctly handle bindings on the server (#18009) ... `item.subsume(item)` did nothing, honestly kinda weird how this only turned up now Fixes #17981	5 months ago
Paolo Ricciuti	957f2755fa	fix: cleanup `superTypeParameters` in `ClassDeclarations`/`ClassExpression` (#18015) ... Closes #18012 We were removing `superTypeArguments` but the AST showed `superTypeParameters`. Luckily, I was able to pinpoint `esrap@2.2.4` as the cause... I've only bumped `esrap` and that made the test fail (so that I could fix it) And of course it was that...there's literally a commit that explicitly print them lol f9137c4101	5 months ago
paoloricciuti	6d13e5a1d0	fix: store current renderer in effect and push it before executing it	5 months ago
paoloricciuti	4eb2c0bd80	fix: import correctly	5 months ago
paoloricciuti	0a0212c86e	chore: new operations functions to replace direct DOM access	5 months ago