Skip to content

Conversation

@birkskyum
Copy link
Member

@birkskyum birkskyum commented Nov 12, 2025

Summary by CodeRabbit

  • New Features

    • Added a new "Basic + Static rendering" example for Solid Start.
  • Documentation

    • Renamed the Supabase example to "Basic + Supabase Auth" for improved clarity.

@github-actions github-actions bot added the documentation Everything documentation related label Nov 12, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 12, 2025

Walkthrough

Configuration 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

Cohort / File(s) Summary
Solid examples configuration
docs/start/config.json
Renamed "Basic + Supabase" example to "Basic + Supabase Auth" with path correction (start-basic-supabasestart-supabase-basic); added new "Basic + Static rendering" example entry (start-basic-static)
Package metadata
examples/solid/start-supabase-basic/package.json
Updated package name from tanstack-solid-start-example-basic-supabase to tanstack-solid-start-example-supabase-basic for naming consistency

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Verify the new example path start-basic-static exists or will be added separately
  • Confirm package name change aligns with directory structure and no other references need updating

Possibly related PRs

Poem

🐰 A hop and a skip through the config we go,
Renaming and ordering, letting examples flow,
Supabase shuffled to its rightful place,
Static rendering joins the race! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title claims to sync the supabase example to react, but the changes only update Solid framework examples and documentation configuration. No React-related changes are present in the changeset. Update the title to accurately reflect the changes, such as 'docs(solid-start): rename and add supabase examples' or 'docs(solid-start): reorganize supabase and static rendering examples'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs(solid-start)--sync-supabase-example-to-react

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.

@nx-cloud
Copy link

nx-cloud bot commented Nov 12, 2025

View your CI Pipeline Execution ↗ for commit c933d25

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded 6m 27s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 1m 42s View ↗

☁️ Nx Cloud last updated this comment at 2025-11-12 00:49:52 UTC

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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: Use workspace:* 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 the workspace:* protocol. Per coding guidelines, internal dependencies should use workspace:* 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

📥 Commits

Reviewing files that changed from the base of the PR and between cb4a646 and c933d25.

⛔ Files ignored due to path filters (7)
  • examples/solid/start-supabase-basic/public/android-chrome-192x192.png is excluded by !**/*.png
  • examples/solid/start-supabase-basic/public/android-chrome-512x512.png is excluded by !**/*.png
  • examples/solid/start-supabase-basic/public/apple-touch-icon.png is excluded by !**/*.png
  • examples/solid/start-supabase-basic/public/favicon-16x16.png is excluded by !**/*.png
  • examples/solid/start-supabase-basic/public/favicon-32x32.png is excluded by !**/*.png
  • examples/solid/start-supabase-basic/public/favicon.ico is excluded by !**/*.ico
  • pnpm-lock.yaml is 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-basic align 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.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 12, 2025

More templates

@tanstack/arktype-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/arktype-adapter@5828

@tanstack/directive-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/directive-functions-plugin@5828

@tanstack/eslint-plugin-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/eslint-plugin-router@5828

@tanstack/history

npm i https://pkg.pr.new/TanStack/router/@tanstack/history@5828

@tanstack/nitro-v2-vite-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/nitro-v2-vite-plugin@5828

@tanstack/react-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router@5828

@tanstack/react-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-devtools@5828

@tanstack/react-router-ssr-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-ssr-query@5828

@tanstack/react-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start@5828

@tanstack/react-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-client@5828

@tanstack/react-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-server@5828

@tanstack/router-cli

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-cli@5828

@tanstack/router-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-core@5828

@tanstack/router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools@5828

@tanstack/router-devtools-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools-core@5828

@tanstack/router-generator

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-generator@5828

@tanstack/router-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-plugin@5828

@tanstack/router-ssr-query-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-ssr-query-core@5828

@tanstack/router-utils

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-utils@5828

@tanstack/router-vite-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-vite-plugin@5828

@tanstack/server-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/server-functions-plugin@5828

@tanstack/solid-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router@5828

@tanstack/solid-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router-devtools@5828

@tanstack/solid-router-ssr-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router-ssr-query@5828

@tanstack/solid-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start@5828

@tanstack/solid-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-client@5828

@tanstack/solid-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-server@5828

@tanstack/start-client-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-client-core@5828

@tanstack/start-plugin-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-plugin-core@5828

@tanstack/start-server-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-core@5828

@tanstack/start-static-server-functions

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-static-server-functions@5828

@tanstack/start-storage-context

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-storage-context@5828

@tanstack/valibot-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/valibot-adapter@5828

@tanstack/virtual-file-routes

npm i https://pkg.pr.new/TanStack/router/@tanstack/virtual-file-routes@5828

@tanstack/zod-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/zod-adapter@5828

commit: c933d25

@birkskyum birkskyum merged commit 34f475e into main Nov 12, 2025
6 checks passed
@birkskyum birkskyum deleted the docs(solid-start)--sync-supabase-example-to-react branch November 12, 2025 01:23
roduyemi pushed a commit to roduyemi/oss-router that referenced this pull request Nov 19, 2025
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Everything documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants