Skip to content

feat(sv-utils): add text.add and text.set, deprecate text.upsert - #1424

Open
jycouet wants to merge 8 commits into
mainfrom
f/text-defaults-override
Open

jycouet wants to merge 8 commits into
mainfrom
f/text-defaults-override

Conversation

@jycouet

@jycouet jycouet commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #1344

Description

text.upsert never updates an existing key, so it's really "add if missing". Renamed to text.defaults (same behavior), added text.override (replaces an existing KEY= value), and kept upsert as a deprecated alias.

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 Oct 16, 2026 •

Copy link
Copy Markdown

Install the latest version of sv from 8d933f6:

pnx https://pkg.svelte.dev/sv/c/8d933f6c7b41e7f57e958cceb73e72ae54cd0853 create

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

@changeset-bot

changeset-bot Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8d933f6

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 Minor
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

@svelte-docs-bot

Copy link
Copy Markdown

Comment thread documentation/docs/50-api/20-sv-utils.md Outdated
@sacrosanctic sacrosanctic changed the title feat(sv-utils): add text.defaults and text.override, deprecate text.upsert feat(sv-utils): add text.add and text.set, deprecate text.upsert Oct 23, 2026
@manuel3108 manuel3108 added the needs-addon-integration-tests Run the addon integration test suite label Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-addon-integration-tests Run the addon integration test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants