Skip to content

fix(sv-utils): bump esrap to 2.3.6 to fix type dropping during migrations - #1294

Merged
dummdidumm merged 2 commits into
sveltejs:version-1from
osamajvd:fix/bump-esrap
Aug 21, 2026
Merged

fix(sv-utils): bump esrap to 2.3.6 to fix type dropping during migrations#1294
dummdidumm merged 2 commits into
sveltejs:version-1from
osamajvd:fix/bump-esrap

Conversation

@osamajvd

@osamajvd osamajvd commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Description

Bumps esrap to ^2.3.6, which fixed certain type annotations and arguments from being dropped: sveltejs/esrap#179

Example

Original source code:

const results = await db.select({
  name: clients.firstName,
  score: sql<number>`phc.similarity(${clients.firstName}, ${searchTermPhrase}::text)`
});

Previous vs new output

  const results = await db.select({
    name: clients.firstName,
-   score: sql`phc.similarity(${clients.firstName}, ${searchTermPhrase}::text)`
+   score: sql<number>`phc.similarity(${clients.firstName}, ${searchTermPhrase}::text)`
  });

Checklist

  • Update snapshots (if applicable)
  • Add a changeset (if applicable)
  • Allow maintainers to edit this PR
  • I care about what I'm doing, no matter the tool I use (Notepad, Sublime, VSCode, AI...)

@pkg-svelte-dev

pkg-svelte-dev Bot commented Aug 21, 2026

Copy link
Copy Markdown

Install the latest version of sv from 157bfd6:

pnx https://pkg.svelte.dev/sv/c/157bfd6f3dc051a485b665b949d3a9cc4e21f70e create

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/cli/pr/1294

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

@changeset-bot

changeset-bot Bot commented Aug 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 157bfd6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@sveltejs/sv-utils Patch
sv 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

@dummdidumm dummdidumm left a comment

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.

thank you!

Comment thread .changeset/wide-meals-bake.md Outdated
@dummdidumm
dummdidumm merged commit 25acc4c into sveltejs:version-1 Aug 21, 2026
1 check was pending
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