-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
docs(solid-start): sync supabase example to react #5828
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(solid-start): sync supabase example to react #5828
Conversation
WalkthroughConfiguration update to Solid framework examples: renamed "Basic + Supabase" entry to "Basic + Supabase Auth" with path correction, added new "Basic + Static rendering" example entry, and updated corresponding package name for consistency. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes
Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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 |
|
View your CI Pipeline Execution ↗ for commit c933d25
☁️ Nx Cloud last updated this comment at |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
examples/solid/start-supabase-basic/package.json (1)
18-20: Useworkspace:*protocol for internal TanStack dependencies.Lines 18–20 reference internal TanStack packages (
@tanstack/solid-router,@tanstack/solid-router-devtools,@tanstack/solid-start) with semver ranges instead of theworkspace:*protocol. Per coding guidelines, internal dependencies should useworkspace:*to ensure monorepo consistency.Apply this diff to use workspace protocol:
"dependencies": { - "@tanstack/solid-router": "^1.135.2", - "@tanstack/solid-router-devtools": "^1.135.2", - "@tanstack/solid-start": "^1.135.2", + "@tanstack/solid-router": "workspace:*", + "@tanstack/solid-router-devtools": "workspace:*", + "@tanstack/solid-start": "workspace:*",As per coding guidelines.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (7)
examples/solid/start-supabase-basic/public/android-chrome-192x192.pngis excluded by!**/*.pngexamples/solid/start-supabase-basic/public/android-chrome-512x512.pngis excluded by!**/*.pngexamples/solid/start-supabase-basic/public/apple-touch-icon.pngis excluded by!**/*.pngexamples/solid/start-supabase-basic/public/favicon-16x16.pngis excluded by!**/*.pngexamples/solid/start-supabase-basic/public/favicon-32x32.pngis excluded by!**/*.pngexamples/solid/start-supabase-basic/public/favicon.icois excluded by!**/*.icopnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (2)
docs/start/config.json(1 hunks)examples/solid/start-supabase-basic/package.json(1 hunks)
🧰 Additional context used
📓 Path-based instructions (3)
**/package.json
📄 CodeRabbit inference engine (AGENTS.md)
Use workspace:* protocol for internal dependencies in package.json files
Files:
examples/solid/start-supabase-basic/package.json
examples/{react,solid}/**
📄 CodeRabbit inference engine (AGENTS.md)
Keep example applications under examples/react/ and examples/solid/
Files:
examples/solid/start-supabase-basic/package.json
docs/{router,start}/**
📄 CodeRabbit inference engine (AGENTS.md)
Place router docs under docs/router/ and start framework docs under docs/start/
Files:
docs/start/config.json
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: TanStack/router PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-09-23T17:36:12.598Z
Learning: Applies to packages/{react-router,solid-router}/** : Implement React and Solid bindings/components only in packages/react-router/ and packages/solid-router/
Learnt from: CR
Repo: TanStack/router PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-09-23T17:36:12.598Z
Learning: Applies to examples/{react,solid}/** : Keep example applications under examples/react/ and examples/solid/
📚 Learning: 2025-09-23T17:36:12.598Z
Learnt from: CR
Repo: TanStack/router PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-09-23T17:36:12.598Z
Learning: Applies to examples/{react,solid}/** : Keep example applications under examples/react/ and examples/solid/
Applied to files:
docs/start/config.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: Test
- GitHub Check: Preview
🔇 Additional comments (3)
examples/solid/start-supabase-basic/package.json (1)
2-2: Package name change aligns with directory structure.The updated package name "tanstack-solid-start-example-supabase-basic" now matches the directory path structure (start-supabase-basic), improving consistency across the codebase.
docs/start/config.json (2)
334-335: Framework synchronization: Solid now matches React naming and paths.The label change to "Basic + Supabase Auth" and path update to
framework/solid/examples/start-supabase-basicalign the Solid example with the React equivalent (lines 285–286), ensuring consistency across frameworks. The path update also matches the actual directory name from the package.json change.
341-344: New "Basic + Static rendering" example entry added for Solid.Verification confirms the referenced example directory
examples/solid/start-basic-static/exists and the path format in config.json (framework/solid/examples/start-basic-static) is consistent with the React entry, maintaining feature parity between framework examples.
More templates
@tanstack/arktype-adapter
@tanstack/directive-functions-plugin
@tanstack/eslint-plugin-router
@tanstack/history
@tanstack/nitro-v2-vite-plugin
@tanstack/react-router
@tanstack/react-router-devtools
@tanstack/react-router-ssr-query
@tanstack/react-start
@tanstack/react-start-client
@tanstack/react-start-server
@tanstack/router-cli
@tanstack/router-core
@tanstack/router-devtools
@tanstack/router-devtools-core
@tanstack/router-generator
@tanstack/router-plugin
@tanstack/router-ssr-query-core
@tanstack/router-utils
@tanstack/router-vite-plugin
@tanstack/server-functions-plugin
@tanstack/solid-router
@tanstack/solid-router-devtools
@tanstack/solid-router-ssr-query
@tanstack/solid-start
@tanstack/solid-start-client
@tanstack/solid-start-server
@tanstack/start-client-core
@tanstack/start-plugin-core
@tanstack/start-server-core
@tanstack/start-static-server-functions
@tanstack/start-storage-context
@tanstack/valibot-adapter
@tanstack/virtual-file-routes
@tanstack/zod-adapter
commit: |
* docs(solid-start): sync supabase example to react * ci: apply automated fixes --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Summary by CodeRabbit
New Features
Documentation