feat: Add Standard Schema validator to qwik-city

[adrian-budau]

What is it?

• Feature / enhancement

Description

Based on #7281

Adds a schema$ validator accepting any Standard Schema confirming validation library

Checklist

• My code follows the developer guidelines of this project
• I performed a self-review of my own code
• I added a changeset with pnpm change
• I made corresponding changes to the Qwik docs
• I added new tests to cover the fix / functionality

[changeset-bot]

🦋 Changeset detected

Latest commit: 3ab676c

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

This PR includes changesets to release 4 packages

Name	Type
@builder.io/qwik-city	Minor
eslint-plugin-qwik	Minor
@builder.io/qwik	Minor
create-qwik	Minor

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

[wmertens]

Interesting! If it doesn't increase build size when not in use, that sounds great. Can you run npx syncpack-fix-mismatches? And you also have to run pnpm api.update

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
qwik-docs	✅ Ready (View Log)	Visit Preview	69d9253

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@builder.io/qwik@7518

npm i https://pkg.pr.new/@builder.io/qwik-city@7518

npm i https://pkg.pr.new/eslint-plugin-qwik@7518

npm i https://pkg.pr.new/create-qwik@7518

commit: 69d9253

[wmertens]

Looking at the build sizes, qwik-city grows with 100 bytes or so and the qwik docs with a few thousand but it seems negligeable. LGTM!

[wmertens]

I'd like @fabian-hiller to have a quick look at this before merging.

[wmertens]

@adrian-budau actually, I forgot that Fabian already worked on this in #7281, with specific attention to the types of error returns, which are not handled correctly here.
Do you think you could continue that PR? Fabian has no time right now...

[adrian-budau]

Well I started the work based on that PR so sure, but I'd like a little more context on what exactly needs to be changed about the error returns?

[fabian-hiller]

Yes, the problem is with the dot-path error type. It would be great if you could finish it and get it merged!

[fabian-hiller]

If I remember correctly, my PR missed testing and updating the docs. Also, there was an idea to include the types directly into the repo to avoid adding a dependency. Feel free to copy the code from my PR.

[gioboa]

is this PR still valid?

[fabian-hiller]

It would be great if we could get this or my PR merged at some pint.

[thejackshelton]

@adrian-budau hey any plans on continuing this PR?

[maiieul]

Closing as stale. We'll do it in v2. We have a PR open we may be able to merge soon.