feat: upgrade to Zod v4 by coji · Pull Request #104 · coji/upflow

coji · 2026-01-10T11:44:31Z

Summary

Upgrade zod from v3.25.64 to v4.3.5
Replace zodix with @coji/zodix for Zod v4 support
Update imports to use @conform-to/zod/v4 and @coji/zodix/v4
Update CLAUDE.md with new import paths

Breaking Changes

z.enum() no longer accepts required_error parameter (removed in Zod v4)

Test plan

pnpm typecheck passes
pnpm build passes
pnpm test passes (25 tests)

References

🤖 Generated with Claude Code

Summary by CodeRabbit

Chores
- Updated validation libraries to newer versions including Zod v4, Conform v4, and zodix package updates
Changes
- Simplified validation error messages in integration settings form to use default error handling

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2026-01-10T11:44:42Z

Warning

Rate limit exceeded

@coji has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 12 minutes and 28 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 for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between bec75a3 and 62711d9.

📒 Files selected for processing (1)

app/routes/admin+/$organization.settings/types.ts

📝 Walkthrough

Walkthrough

This PR upgrades dependencies and import paths across the codebase: replaces zodix with @coji/zodix/v4, updates @conform-to/zod to @conform-to/zod/v4, and bumps zod from v3 to v4. Additionally, custom required_error messages are removed from enum fields in the integration settings schema.

Changes

Cohort / File(s)	Summary
Dashboard Routes `app/routes/_dashboard+/$organization._index/route.tsx`, `app/routes/_dashboard+/$organization.ongoing/route.tsx`, `app/routes/_dashboard+/$organization/route.tsx`	Updated zodix import path from `'zodix'` to `'@coji/zodix/v4'`
Admin Core Routes `app/routes/admin+/$organization._layout/route.tsx`, `app/routes/admin+/$organization.members/route.tsx`, `app/routes/admin+/$organization.repositories/$repository.$pull/route.tsx`, `app/routes/admin+/$organization.repositories/$repository._index/route.tsx`, `app/routes/admin+/$organization.repositories/$repository.delete/route.tsx`, `app/routes/admin+/$organization.repositories._index/route.tsx`, `app/routes/admin+/$organization.repositories.tsx`, `app/routes/admin+/$organization.settings/route.tsx`	Updated zodix import path from `'zodix'` to `'@coji/zodix/v4'`
Admin Repository Configuration `app/routes/admin+/$organization.repositories.add/route.tsx`, `app/routes/admin+/$organization.repositories.$repository.settings/route.tsx`	Updated both `zx` import from `'zodix'` to `'@coji/zodix/v4'` and `parseWithZod` from `'@conform-to/zod'` to `'@conform-to/zod/v4'`
Admin Settings Forms `app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts`, `app/routes/admin+/$organization.settings/forms/delete-organization.tsx`, `app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts`, `app/routes/admin+/$organization.settings/forms/export-settings.tsx`, `app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts`, `app/routes/admin+/$organization.settings/forms/integration-settings.tsx`, `app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts`, `app/routes/admin+/$organization.settings/forms/organization-settings.tsx`	Updated `parseWithZod` import from `'@conform-to/zod'` to `'@conform-to/zod/v4'`
Admin Create Route `app/routes/admin+/create/route.tsx`	Updated `parseWithZod` import from `'@conform-to/zod'` to `'@conform-to/zod/v4'`
Settings Schema `app/routes/admin+/$organization.settings/types.ts`	Removed custom `required_error` messages from `provider` and `method` enum fields in `integrationSettingsSchema`
Package Dependencies `package.json`	Added `@coji/zodix: "0.7.0"`, removed `zodix: "0.4.4"`, and upgraded `zod` from `"3.25.64"` to `"4.3.5"`
Documentation `CLAUDE.md`	Updated code example imports to reflect Conform-to/Zod v4 and zodix v4 equivalents

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~20 minutes

Possibly related PRs

feat: add excluded users setting and recalculate feature #100: Modifies app/routes/admin+/$organization.settings/types.ts with overlapping scope; the main PR removes enum error messages while this PR adds the excludedUsers field to the schema.

Poem

🐰 Dependencies dance in v4's embrace,
Zodix and Conform now join the race,
Enums stripped clean of custom cries,
A cleaner schema beneath the skies,
Hop along, code hops with grace! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'feat: upgrade to Zod v4' accurately reflects the main change in the pull request, which is upgrading Zod from v3 to v4 across the entire codebase.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

- Upgrade zod from v3.25.64 to v4.3.5 - Replace zodix with @coji/zodix for Zod v4 support - Update imports to use @conform-to/zod/v4 and @coji/zodix/v4 - Update CLAUDE.md with new import paths - Remove deprecated required_error parameter from z.enum() Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (1)

app/routes/admin+/$organization.settings/types.ts (1)
16-17: LGTM! Correct removal of deprecated parameter.

The required_error parameter has been removed as it's deprecated in Zod v4. Error messages will now use Zod's default validation messages for enum fields.

If custom error messages are desired, you can migrate to Zod v4's unified error parameter:
♻️ Optional: Add custom error messages using v4 syntax
 export const integrationSettingsSchema = z.object({
   id: z.string().optional(),
-  provider: z.enum(['github']),
-  method: z.enum(['token']),
+  provider: z.enum(['github'], {
+    error: (issue) => {
+      if (issue.code === 'invalid_enum_value' || issue.code === 'invalid_type') {
+        return 'provider is required';
+      }
+    },
+  }),
+  method: z.enum(['token'], {
+    error: (issue) => {
+      if (issue.code === 'invalid_enum_value' || issue.code === 'invalid_type') {
+        return 'method is required';
+      }
+    },
+  }),
   privateToken: z.string().min(1, { message: 'private token is required' }),
 })

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5ecc1fb and bec75a3.

⛔ Files ignored due to path filters (1)

pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (25)

CLAUDE.md
app/routes/_dashboard+/$organization._index/route.tsx
app/routes/_dashboard+/$organization.ongoing/route.tsx
app/routes/_dashboard+/$organization/route.tsx
app/routes/admin+/$organization._layout/route.tsx
app/routes/admin+/$organization.members/route.tsx
app/routes/admin+/$organization.repositories.$repository.$pull/route.tsx
app/routes/admin+/$organization.repositories.$repository._index/route.tsx
app/routes/admin+/$organization.repositories.$repository.delete/route.tsx
app/routes/admin+/$organization.repositories.$repository.settings/route.tsx
app/routes/admin+/$organization.repositories._index/route.tsx
app/routes/admin+/$organization.repositories.add/route.tsx
app/routes/admin+/$organization.repositories.tsx
app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts
app/routes/admin+/$organization.settings/forms/delete-organization.tsx
app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/export-settings.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/integration-settings.tsx
app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/organization-settings.tsx
app/routes/admin+/$organization.settings/route.tsx
app/routes/admin+/$organization.settings/types.ts
app/routes/admin+/create/route.tsx
package.json

🧰 Additional context used

📓 Path-based instructions (7)

**/{routes,server,entry}/**/*.{ts,tsx}

📄 CodeRabbit inference engine (CLAUDE.md)

Use React Router v7 in SSR mode with Express server for the web framework

Files:

app/routes/_dashboard+/$organization._index/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.tsx
app/routes/admin+/create/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/delete-organization.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.tsx
app/routes/_dashboard+/$organization.ongoing/route.tsx
app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts
app/routes/admin+/$organization.repositories._index/route.tsx
app/routes/admin+/$organization._layout/route.tsx
app/routes/admin+/$organization.repositories.$repository._index/route.tsx
app/routes/admin+/$organization.settings/route.tsx
app/routes/admin+/$organization.repositories.$repository.settings/route.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts
app/routes/admin+/$organization.repositories.$repository.delete/route.tsx
app/routes/_dashboard+/$organization/route.tsx
app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts
app/routes/admin+/$organization.repositories.add/route.tsx
app/routes/admin+/$organization.repositories.tsx
app/routes/admin+/$organization.settings/types.ts
app/routes/admin+/$organization.settings/forms/organization-settings.tsx
app/routes/admin+/$organization.repositories.$repository.$pull/route.tsx
app/routes/admin+/$organization.members/route.tsx

app/routes/**/*.{ts,tsx}

📄 CodeRabbit inference engine (CLAUDE.md)

app/routes/**/*.{ts,tsx}: Use file-based routing with remix-flat-routes convention using the + folder syntax for organizing routes
Use intent-based form action dispatch with ts-pattern for routes with multiple form actions

Files:

app/routes/_dashboard+/$organization._index/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.tsx
app/routes/admin+/create/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/delete-organization.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.tsx
app/routes/_dashboard+/$organization.ongoing/route.tsx
app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts
app/routes/admin+/$organization.repositories._index/route.tsx
app/routes/admin+/$organization._layout/route.tsx
app/routes/admin+/$organization.repositories.$repository._index/route.tsx
app/routes/admin+/$organization.settings/route.tsx
app/routes/admin+/$organization.repositories.$repository.settings/route.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts
app/routes/admin+/$organization.repositories.$repository.delete/route.tsx
app/routes/_dashboard+/$organization/route.tsx
app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts
app/routes/admin+/$organization.repositories.add/route.tsx
app/routes/admin+/$organization.repositories.tsx
app/routes/admin+/$organization.settings/types.ts
app/routes/admin+/$organization.settings/forms/organization-settings.tsx
app/routes/admin+/$organization.repositories.$repository.$pull/route.tsx
app/routes/admin+/$organization.members/route.tsx

**/*.{ts,tsx}

📄 CodeRabbit inference engine (CLAUDE.md)

Use ~/ path alias prefix for imports from the app/ directory

Files:

app/routes/_dashboard+/$organization._index/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.tsx
app/routes/admin+/create/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/delete-organization.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.tsx
app/routes/_dashboard+/$organization.ongoing/route.tsx
app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts
app/routes/admin+/$organization.repositories._index/route.tsx
app/routes/admin+/$organization._layout/route.tsx
app/routes/admin+/$organization.repositories.$repository._index/route.tsx
app/routes/admin+/$organization.settings/route.tsx
app/routes/admin+/$organization.repositories.$repository.settings/route.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts
app/routes/admin+/$organization.repositories.$repository.delete/route.tsx
app/routes/_dashboard+/$organization/route.tsx
app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts
app/routes/admin+/$organization.repositories.add/route.tsx
app/routes/admin+/$organization.repositories.tsx
app/routes/admin+/$organization.settings/types.ts
app/routes/admin+/$organization.settings/forms/organization-settings.tsx
app/routes/admin+/$organization.repositories.$repository.$pull/route.tsx
app/routes/admin+/$organization.members/route.tsx

**/{routes,services}/**/*.{ts,tsx}

📄 CodeRabbit inference engine (CLAUDE.md)

Use Conform with Zod for type-safe form validation and parsing

Files:

app/routes/_dashboard+/$organization._index/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.tsx
app/routes/admin+/create/route.tsx
app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/delete-organization.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.tsx
app/routes/_dashboard+/$organization.ongoing/route.tsx
app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts
app/routes/admin+/$organization.repositories._index/route.tsx
app/routes/admin+/$organization._layout/route.tsx
app/routes/admin+/$organization.repositories.$repository._index/route.tsx
app/routes/admin+/$organization.settings/route.tsx
app/routes/admin+/$organization.repositories.$repository.settings/route.tsx
app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts
app/routes/admin+/$organization.repositories.$repository.delete/route.tsx
app/routes/_dashboard+/$organization/route.tsx
app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts
app/routes/admin+/$organization.repositories.add/route.tsx
app/routes/admin+/$organization.repositories.tsx
app/routes/admin+/$organization.settings/types.ts
app/routes/admin+/$organization.settings/forms/organization-settings.tsx
app/routes/admin+/$organization.repositories.$repository.$pull/route.tsx
app/routes/admin+/$organization.members/route.tsx

**/*.server.ts

📄 CodeRabbit inference engine (CLAUDE.md)

**/*.server.ts: Use .server.ts file suffix for server-only code that won't be bundled for the client
Use Kysely query builder for type-safe database operations with types generated from the database schema

Files:

app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts
app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts

**/*.action.server.ts

📄 CodeRabbit inference engine (CLAUDE.md)

Use .action.server.ts suffix for form action handler files in server-side code

Files:

app/routes/admin+/$organization.settings/forms/export-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/delete-organization.action.server.ts
app/routes/admin+/$organization.settings/forms/integration-settings.action.server.ts
app/routes/admin+/$organization.settings/forms/organization-settings.action.server.ts

app/routes/**/$*.tsx

📄 CodeRabbit inference engine (CLAUDE.md)

Use $param syntax for dynamic route segments and $organization.settings/ pattern for nested route folders

Files:

app/routes/admin+/$organization.repositories.tsx

🧠 Learnings (9)

📓 Common learnings

Learnt from: CR
Repo: coji/upflow PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-01-10T09:39:54.188Z
Learning: Applies to **/{routes,services}/**/*.{ts,tsx} : Use Conform with Zod for type-safe form validation and parsing