Add React comparison guide and link it in the menu#102
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
WalkthroughThis PR adds framework-specific "Comparison" documentation pages (React, Solid, Svelte, Vue), inserts "Comparison" links into the guides menus and introduction pages, and updates the top-level README and per-framework READMEs/introduction copy to emphasize the framework-agnostic core with framework-native reactivity blocks, a modular methods API (~2.5 kB starting), and end-to-end type-safety for nested paths/field arrays. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ 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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
website/src/routes/(docs)/react/guides/(get-started)/comparison/index.mdx (1)
27-27: ⚡ Quick winAdd version/date context for bundle-size numbers.
This row is likely to drift over time. Add a short qualifier (e.g., “as of May 2026”) and measurement basis/toolchain so readers don’t treat these as timeless constants.
Proposed doc tweak
-| Bundle size (min+gzip) | From ~2.5 kB | ~12 kB | ~15 kB | +| Bundle size (min+gzip, as of May 2026) | From ~2.5 kB | ~12 kB | ~15 kB |🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@website/src/routes/`(docs)/react/guides/(get-started)/comparison/index.mdx at line 27, Update the "Bundle size (min+gzip)" table row in index.mdx to include a date/version qualifier and measurement basis so the numbers are not treated as permanent; e.g., append " (min+gzip, measured with [toolchain] as of [Month Year])" to the header or each cell — locate the row containing the text "Bundle size (min+gzip)" and modify it to include the qualifier (for example: "Bundle size (min+gzip, measured with webpack 5 + terser, as of May 2026)").
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@website/src/routes/`(docs)/react/guides/(get-started)/comparison/index.mdx:
- Line 27: Update the "Bundle size (min+gzip)" table row in index.mdx to include
a date/version qualifier and measurement basis so the numbers are not treated as
permanent; e.g., append " (min+gzip, measured with [toolchain] as of [Month
Year])" to the header or each cell — locate the row containing the text "Bundle
size (min+gzip)" and modify it to include the qualifier (for example: "Bundle
size (min+gzip, measured with webpack 5 + terser, as of May 2026)").
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 90e7b2d3-3128-434f-88e8-4135d6edc0bc
📒 Files selected for processing (3)
website/src/routes/(docs)/react/guides/(get-started)/comparison/index.mdxwebsite/src/routes/(docs)/react/guides/(get-started)/introduction/index.mdxwebsite/src/routes/(docs)/react/guides/menu.md
There was a problem hiding this comment.
Pull request overview
Adds a new React “Comparison” guide and surfaces it in the React docs navigation to help users evaluate Formisch vs. React Hook Form and TanStack Form.
Changes:
- Add a new
/react/guides/comparison/page with a short comparison table and decision guidance. - Link the new guide from the React guides menu.
- Add a callout link to the comparison guide from the React introduction page.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| website/src/routes/(docs)/react/guides/menu.md | Adds “Comparison” entry to the React guides menu. |
| website/src/routes/(docs)/react/guides/(get-started)/introduction/index.mdx | Links readers to the new comparison guide from the introduction. |
| website/src/routes/(docs)/react/guides/(get-started)/comparison/index.mdx | Introduces the new comparison guide content and links to the long-form blog article. |
Comments suppressed due to low confidence (1)
website/src/routes/(docs)/react/guides/(get-started)/comparison/index.mdx:44
- This is an internal link to an existing blog route but uses Markdown link syntax (plain
<a>). Prefer theLinkcomponent for internal navigation (consistent with other guides and avoids full reloads).
For a side-by-side mapping of common React Hook Form APIs to their Formisch equivalents, see the migration section of the [comparison article](/blog/react-form-library-comparison/).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
website/src/routes/(docs)/react/guides/(get-started)/comparison/index.mdx (1)
27-27:⚠️ Potential issue | 🟠 Major | ⚡ Quick winTanStack Form bundle size claim needs clarification.
The comparison table states TanStack Form at "~15 kB," but current documentation reveals the library unconditionally imports devtools dependencies in production builds, adding approximately 11.6 kB of code that doesn't tree-shake away. The actual production bundle footprint is framework-dependent and more complex than a single figure suggests. Update the claim to reflect the production overhead or link to bundler analysis recommendations, as the ~15 kB figure doesn't capture the current documented behavior.
React Hook Form's "~12 kB" is accurate (current: 11.2 kB minified+gzipped). Formisch's "~2.5 kB" is confirmed across all framework documentation.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@website/src/routes/`(docs)/react/guides/(get-started)/comparison/index.mdx at line 27, Update the bundle-size cell for "TanStack Form" in the comparison table row containing "Bundle size (min+gzip)" to clarify that the ~15 kB figure is misleading: state that TanStack Form currently unconditionally imports devtools in production (adding ~11.6 kB) so actual production footprint is framework- and bundler-dependent, and either replace "~15 kB" with a clearer note like "varies; +~11.6 kB devtools included in prod" or add a footnote/link to bundler analysis recommendations and the library docs; keep the existing "~12 kB" for React Hook Form and "~2.5 kB" for Formisch unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@website/src/routes/`(docs)/react/guides/(get-started)/comparison/index.mdx:
- Line 27: Update the bundle-size cell for "TanStack Form" in the comparison
table row containing "Bundle size (min+gzip)" to clarify that the ~15 kB figure
is misleading: state that TanStack Form currently unconditionally imports
devtools in production (adding ~11.6 kB) so actual production footprint is
framework- and bundler-dependent, and either replace "~15 kB" with a clearer
note like "varies; +~11.6 kB devtools included in prod" or add a footnote/link
to bundler analysis recommendations and the library docs; keep the existing "~12
kB" for React Hook Form and "~2.5 kB" for Formisch unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9fa2e76d-37ff-4c54-95bd-bec9018945d7
📒 Files selected for processing (1)
website/src/routes/(docs)/react/guides/(get-started)/comparison/index.mdx
Fix #101