feat(sv-utils): color.command now also accepts string[]

[sacrosanctic]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: ab8afc2

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

[svelte-docs-bot]

Preview: https://svelte-dev-git-preview-cli-1024-svelte.vercel.app/

[pkg-pr-new]

Open in StackBlitz

npx https://pkg.pr.new/svelte-migrate@1024

npx https://pkg.pr.new/sv@1024

npx https://pkg.pr.new/@sveltejs/sv-utils@1024

commit: ab8afc2

[jycouet]

@sacrosanctic what do you think about this? 👀
experiment!

[sacrosanctic]

looks good, wish the fn name wasnt so long

[jycouet]

looks good, wish the fn name wasnt so long

Same here !
What about formatCommand ? (and a small JSDoc)?

I'm not sure about your last 2 commits? 👀
Is it ok with intellisense? (I didn't check yet)

[sacrosanctic]

What about formatCommand ? (and a small JSDoc)?

I feel like we have to keep it as resolveCommandArray. To discourage similar types of interface changing behaviour.

I'm not sure about your last 2 commits? 👀

They're non functional changes, feel free to reverse it, I like that the logic and data is now cleanly separated.

Is it ok with intellisense? (I didn't check yet)

Yes? What the problem?

[jycouet]

With Record<string, ..., I could do color.hello("") and types will be happy.
I tried with as const or with satisfies, but none are working as we want.

Let's just go back to a simpler solution (adding a few characters)