Closed juliusmarminge closed 2 hours ago
Latest commit: 8b92ab01234e922a35c25c17e2cdcd37172df857
The changes in this PR will be included in the next version bump.
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
The latest updates on your projects. Learn more about Vercel for Git βοΈ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
docs-uploadthing | β Ready (Inspect) | Visit Preview | π¬ Add feedback | Nov 22, 2024 9:21am |
The pull request introduces several modifications to the uploadthing
package, primarily focusing on dependency updates and changes to input parsing logic. New dependencies, @standard-schema/spec
and valibot
, are added to the package.json
. The input parsing functionality is updated to support asynchronous operations, including changes to method signatures in various files, enhancing type safety and error handling. Additionally, the test suite is expanded to incorporate validation using the new valibot
library alongside existing tests.
File Path | Change Summary |
---|---|
packages/uploadthing/package.json |
Added dependencies: @standard-schema/spec (v1.0.0-beta.3) and valibot (v1.0.0-beta.7). |
packages/uploadthing/src/internal/handler.ts |
Modified handleUploadAction to change input parsing from asynchronous to synchronous, simplifying error handling. |
packages/uploadthing/src/internal/parser.ts |
Updated ParseFn to return Promise<TType> , renamed parse to parseAsync , added JsonParser type. |
packages/uploadthing/src/internal/types.ts |
Updated UploadBuilder interface to enhance type safety for input method signatures. |
packages/uploadthing/src/internal/upload-builder.ts |
Changed parse method to parseAsync , allowing for asynchronous input parsing. |
packages/uploadthing/test/upload-builder.test.ts |
Expanded tests to include valibot for validation, added new test cases, and renamed existing tests for clarity. |
docs/src/app/(docs)/file-routes/page.mdx |
Updated documentation to reflect support for multiple schema validators, including valibot and Effect/Schema . |
packages/uploadthing/test/input.test.ts |
Introduced new tests for input validation using Zod, Valibot, and Effect Schema, covering various input scenarios. |
uploadthing
package's functionality.package.json
file, which may indirectly relate to the main PR's dependency updates, although it does not directly affect the core functionality.@uploadthing/mime-types
package, which may relate to the overall upload functionality but does not directly connect to the changes in the main PR.package.json
file.uploadthing
package, which aligns with the changes made in the main PR.uploadthing
package but does not directly connect to the main PR's changes.sdk
, examples
π° In the land of code, where rabbits hop,
New dependencies added, we won't stop!
Parsing's now async, oh what a delight,
Withvalibot
in tests, we'll code through the night!
Type safety's enhanced, our inputs are clear,
Hopping through changes, we have nothing to fear! π
[!WARNING]
Rate limit exceeded
@juliusmarminge has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 19 minutes and 35 seconds before requesting another review.
β How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.π¦ How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://docs.coderabbit.ai/faq) for further information.π₯ Commits
Reviewing files that changed from the base of the PR and between 36acdf3448d8f3bdad35ecb5da3cc1525574af56 and 8b92ab01234e922a35c25c17e2cdcd37172df857.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Bundle | Size (gzip) | Visualization |
---|---|---|
Main | 27.59KB | See Treemap π |
PR (b93fc4af895ba5fac9e4ce5d5bf3836bba4758bd) | 27.59KB | See Treemap π |
Diff | No change |
- [@example/minimal-astro-react](https://pkg.pr.new/template/209e68f3-e357-4488-92a1-32d45dcf1365) - [@example/minimal-appdir](https://pkg.pr.new/template/67a2f679-4f18-4854-b327-ec7b683b47b7) - [@example/minimal-nuxt](https://pkg.pr.new/template/328c3d6f-a09e-4204-96de-b7bb37312014) - [@example/minimal-pagedir](https://pkg.pr.new/template/c98901de-0e44-4197-9388-9087cae384b1) - [@example/minimal-expo](https://pkg.pr.new/template/4166c481-d722-474b-a952-5616b1fffe01) - [@example/minimal-solidstart](https://pkg.pr.new/template/d8799a69-6532-41db-bf9c-504280a1f6f0) - [@example/minimal-sveltekit](https://pkg.pr.new/template/958d47db-f21a-4030-a264-e7e21be7c0e7) - [@example/minimal-tanstack-start](https://pkg.pr.new/template/5458732b-e9d9-4f83-9c08-932dfb3f80ef)
pnpm add https://pkg.pr.new/pingdotgg/uploadthing@1061
commit: 8b92ab0
Add support for
@standard-schema/spec
as input parsers. This will support Valibot >=1, Zod >=3.24, ArkType and more I believe. Also adds (parital) support foreffect/Schema
Summary by CodeRabbit
Release Notes
New Features
valibot
library for enhanced input validation alongside existing validation methods.@standard-schema/spec
for improved schema handling.Effect/Schema
andValibot
.Bug Fixes
Tests
valibot
andeffect/Schema
.