Skip to content

Conversation

@birkskyum
Copy link
Member

@birkskyum birkskyum commented Nov 12, 2025

Summary by CodeRabbit

  • Documentation

    • Added installation guide for TanStack Router with Webpack in Solid projects, covering configuration, Babel presets, and file-based routing setup with recommended linter/formatter settings.
  • New Features

    • Added a complete quickstart example project for Solid.js with Webpack integration, including file-based routing, navigation between pages, and developer tooling.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 12, 2025

Walkthrough

This PR adds a complete Solid.js Webpack file-based routing quickstart example with accompanying documentation. It includes configuration files, route definitions, a generated routeTree file, and step-by-step installation guidance for integrating TanStack Router with Webpack in Solid projects.

Changes

Cohort / File(s) Summary
Documentation
docs/router/framework/solid/installation/with-webpack.md
New guide covering TanStack Router installation with Webpack in Solid, including plugin setup, Babel presets, configuration options, and best practices for ignoring generated files.
Build Configuration
examples/solid/quickstart-webpack-file-based/webpack.config.js, examples/solid/quickstart-webpack-file-based/tsconfig.json, examples/solid/quickstart-webpack-file-based/.babelrc
New Webpack ES module configuration with HtmlWebpackPlugin and TanStack Router plugin for Solid; TypeScript config targeting ES2020 with Solid JSX support; Babel config with solid-preset and TypeScript support.
Project Metadata & Ignore Rules
examples/solid/quickstart-webpack-file-based/package.json, examples/solid/quickstart-webpack-file-based/README.md, examples/solid/quickstart-webpack-file-based/.gitignore, examples/solid/quickstart-webpack-file-based/.vscode/settings.json
Package configuration with dev/build scripts on port 3001; usage documentation; standard ignore patterns; VSCode settings to exclude generated routeTree file from watchers, search, and read-only markers.
HTML & App Bootstrap
examples/solid/quickstart-webpack-file-based/public/index.html, examples/solid/quickstart-webpack-file-based/src/index.tsx, examples/solid/quickstart-webpack-file-based/src/app.tsx
HTML entry point with Tailwind browser script; Solid render bootstrap checking for existing content; App component with RouterProvider, module augmentation for type-safety, and defaultPreload/scroll restoration configuration.
Route Definitions
examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx, examples/solid/quickstart-webpack-file-based/src/routes/index.tsx, examples/solid/quickstart-webpack-file-based/src/routes/about.tsx
Root route with header navigation and devtools; index (home) route; about route. All use createRootRoute or createFileRoute with component definitions.
Generated Routing Types
examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
Auto-generated file containing route tree construction, FileRoutes type mappings (FileRoutesByFullPath, FileRoutesByTo, FileRoutesById), module augmentation for @tanstack/solid-router type registry, and routeTree export.

Sequence Diagram

sequenceDiagram
    participant Browser
    participant Webpack
    participant App
    participant Router
    participant RouteTree as routeTree.gen.ts
    participant Component as Route Component

    Browser->>Webpack: Load index.html
    Webpack->>App: Execute src/index.tsx
    App->>App: render(App, root)
    App->>Router: Create router from routeTree
    Router->>RouteTree: Import route definitions
    RouteTree->>Router: Return route tree with types
    Router->>App: RouterProvider established
    
    Note over App: User navigates
    
    Browser->>Router: User clicks Link
    Router->>RouteTree: Match route path
    RouteTree->>Component: Get component for path
    Component->>Browser: Render matched component
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~35–45 minutes

Areas requiring attention:

  • webpack.config.js: Verify plugin configuration, loader setup, and devServer options follow best practices for Solid projects
  • src/routeTree.gen.ts: Validate generated type definitions, module augmentations, and route tree structure for correctness
  • src/app.tsx: Confirm module augmentation syntax and router type-safety setup are properly configured
  • package.json: Verify all dependency versions are compatible and aligned with project requirements

Possibly related PRs

Suggested reviewers

  • schiller-manuel
  • brenelz
  • birkskyum

Poem

🐰 A rabbit hops through webpack's maze,
With Solid routes now set ablaze!
File-based magic, types so clean,
Best router setup ever seen! 🎯✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately reflects the main changes: adding a webpack example and installation guide for Solid Router, which is demonstrated throughout the changeset.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch webpack-solid-router

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.

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

nx-cloud bot commented Nov 12, 2025

View your CI Pipeline Execution ↗ for commit 503f68a

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

☁️ Nx Cloud last updated this comment at 2025-11-12 16:09:02 UTC

@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@5833

@tanstack/directive-functions-plugin

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

@tanstack/eslint-plugin-router

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

@tanstack/history

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

@tanstack/nitro-v2-vite-plugin

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

@tanstack/react-router

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

@tanstack/react-router-devtools

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

@tanstack/react-router-ssr-query

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

@tanstack/react-start

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

@tanstack/react-start-client

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

@tanstack/react-start-server

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

@tanstack/router-cli

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

@tanstack/router-core

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

@tanstack/router-devtools

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

@tanstack/router-devtools-core

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

@tanstack/router-generator

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

@tanstack/router-plugin

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

@tanstack/router-ssr-query-core

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

@tanstack/router-utils

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

@tanstack/router-vite-plugin

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

@tanstack/server-functions-plugin

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

@tanstack/solid-router

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

@tanstack/solid-router-devtools

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

@tanstack/solid-router-ssr-query

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

@tanstack/solid-start

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

@tanstack/solid-start-client

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

@tanstack/solid-start-server

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

@tanstack/start-client-core

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

@tanstack/start-plugin-core

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

@tanstack/start-server-core

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

@tanstack/start-static-server-functions

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

@tanstack/start-storage-context

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

@tanstack/valibot-adapter

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

@tanstack/virtual-file-routes

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

@tanstack/zod-adapter

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

commit: 503f68a

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: 2

🧹 Nitpick comments (2)
examples/solid/quickstart-webpack-file-based/webpack.config.js (1)

43-44: Use an absolute path for devServer.static.

When this config runs from outside the example directory (e.g., via workspace-wide scripts), 'public' resolves relative to the current working directory and can miss the intended folder. Webpack’s guidance is to supply absolute static directories to avoid those resolution issues.(webpack.js.org)

-    static: ['public'],
+    static: [path.resolve(__dirname, './public')],
docs/router/framework/solid/installation/with-webpack.md (1)

29-36: Replace non-descriptive “here” link text

Markdown lint (MD059) warns about generic link text. Please swap “here” for descriptive wording so readers know what they’ll open.

-And in the .babelrc (SWC doesn't support solid-js, see [here](https://www.answeroverflow.com/m/1135200483116593182)), add these presets:
+And in the .babelrc (SWC doesn't support solid-js; see the [AnswerOverflow discussion](https://www.answeroverflow.com/m/1135200483116593182)), add these presets:
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d5ce04c and 503f68a.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (15)
  • docs/router/framework/solid/installation/with-webpack.md (1 hunks)
  • examples/solid/quickstart-webpack-file-based/.babelrc (1 hunks)
  • examples/solid/quickstart-webpack-file-based/.gitignore (1 hunks)
  • examples/solid/quickstart-webpack-file-based/.vscode/settings.json (1 hunks)
  • examples/solid/quickstart-webpack-file-based/README.md (1 hunks)
  • examples/solid/quickstart-webpack-file-based/package.json (1 hunks)
  • examples/solid/quickstart-webpack-file-based/public/index.html (1 hunks)
  • examples/solid/quickstart-webpack-file-based/src/app.tsx (1 hunks)
  • examples/solid/quickstart-webpack-file-based/src/index.tsx (1 hunks)
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts (1 hunks)
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx (1 hunks)
  • examples/solid/quickstart-webpack-file-based/src/routes/about.tsx (1 hunks)
  • examples/solid/quickstart-webpack-file-based/src/routes/index.tsx (1 hunks)
  • examples/solid/quickstart-webpack-file-based/tsconfig.json (1 hunks)
  • examples/solid/quickstart-webpack-file-based/webpack.config.js (1 hunks)
🧰 Additional context used
📓 Path-based instructions (6)
docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (AGENTS.md)

Use internal docs links relative to the docs/ folder (e.g., ./guide/data-loading)

Files:

  • docs/router/framework/solid/installation/with-webpack.md
docs/{router,start}/**

📄 CodeRabbit inference engine (AGENTS.md)

Place router docs under docs/router/ and start framework docs under docs/start/

Files:

  • docs/router/framework/solid/installation/with-webpack.md
**/*.{ts,tsx}

📄 CodeRabbit inference engine (AGENTS.md)

Use TypeScript in strict mode with extensive type safety across the codebase

Files:

  • examples/solid/quickstart-webpack-file-based/src/app.tsx
  • examples/solid/quickstart-webpack-file-based/src/index.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/about.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
  • examples/solid/quickstart-webpack-file-based/src/routes/index.tsx
examples/{react,solid}/**

📄 CodeRabbit inference engine (AGENTS.md)

Keep example applications under examples/react/ and examples/solid/

Files:

  • examples/solid/quickstart-webpack-file-based/src/app.tsx
  • examples/solid/quickstart-webpack-file-based/src/index.tsx
  • examples/solid/quickstart-webpack-file-based/README.md
  • examples/solid/quickstart-webpack-file-based/tsconfig.json
  • examples/solid/quickstart-webpack-file-based/src/routes/about.tsx
  • examples/solid/quickstart-webpack-file-based/public/index.html
  • examples/solid/quickstart-webpack-file-based/webpack.config.js
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
  • examples/solid/quickstart-webpack-file-based/src/routes/index.tsx
  • examples/solid/quickstart-webpack-file-based/package.json
**/src/routes/**

📄 CodeRabbit inference engine (AGENTS.md)

Place file-based routes under src/routes/ directories

Files:

  • examples/solid/quickstart-webpack-file-based/src/routes/about.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/index.tsx
**/package.json

📄 CodeRabbit inference engine (AGENTS.md)

Use workspace:* protocol for internal dependencies in package.json files

Files:

  • examples/solid/quickstart-webpack-file-based/package.json
🧠 Learnings (12)
📓 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 docs/{router,start}/** : Place router docs under docs/router/ and start framework docs under docs/start/
📚 Learning: 2025-10-01T18:31:35.420Z
Learnt from: schiller-manuel
Repo: TanStack/router PR: 5330
File: e2e/react-start/custom-basepath/src/routeTree.gen.ts:58-61
Timestamp: 2025-10-01T18:31:35.420Z
Learning: Do not review files named `routeTree.gen.ts` in TanStack Router repositories, as these are autogenerated files that should not be manually modified.

Applied to files:

  • examples/solid/quickstart-webpack-file-based/.vscode/settings.json
  • docs/router/framework/solid/installation/with-webpack.md
  • examples/solid/quickstart-webpack-file-based/src/app.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
📚 Learning: 2025-10-08T08:11:47.088Z
Learnt from: nlynzaad
Repo: TanStack/router PR: 5402
File: packages/router-generator/tests/generator/no-formatted-route-tree/routeTree.nonnested.snapshot.ts:19-21
Timestamp: 2025-10-08T08:11:47.088Z
Learning: Test snapshot files in the router-generator tests directory (e.g., files matching the pattern `packages/router-generator/tests/generator/**/routeTree*.snapshot.ts` or `routeTree*.snapshot.js`) should not be modified or have issues flagged, as they are fixtures used to verify the generator's output and are intentionally preserved as-is.

Applied to files:

  • examples/solid/quickstart-webpack-file-based/.vscode/settings.json
  • docs/router/framework/solid/installation/with-webpack.md
  • examples/solid/quickstart-webpack-file-based/src/app.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/about.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
  • examples/solid/quickstart-webpack-file-based/src/routes/index.tsx
📚 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 packages/{react-router,solid-router}/** : Implement React and Solid bindings/components only in packages/react-router/ and packages/solid-router/

Applied to files:

  • examples/solid/quickstart-webpack-file-based/.vscode/settings.json
  • docs/router/framework/solid/installation/with-webpack.md
  • examples/solid/quickstart-webpack-file-based/src/app.tsx
  • examples/solid/quickstart-webpack-file-based/src/index.tsx
  • examples/solid/quickstart-webpack-file-based/tsconfig.json
  • examples/solid/quickstart-webpack-file-based/.babelrc
  • examples/solid/quickstart-webpack-file-based/src/routes/about.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
  • examples/solid/quickstart-webpack-file-based/src/routes/index.tsx
  • examples/solid/quickstart-webpack-file-based/package.json
📚 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 packages/{router-cli,router-generator,router-plugin,virtual-file-routes}/** : Keep CLI, generators, bundler plugins, and virtual file routing utilities in their dedicated tooling package directories

Applied to files:

  • examples/solid/quickstart-webpack-file-based/.vscode/settings.json
  • docs/router/framework/solid/installation/with-webpack.md
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
  • examples/solid/quickstart-webpack-file-based/package.json
📚 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 **/*.{ts,tsx} : Use TypeScript in strict mode with extensive type safety across the codebase

Applied to files:

  • examples/solid/quickstart-webpack-file-based/.vscode/settings.json
  • examples/solid/quickstart-webpack-file-based/tsconfig.json
  • examples/solid/quickstart-webpack-file-based/.babelrc
📚 Learning: 2025-11-02T16:16:24.898Z
Learnt from: nlynzaad
Repo: TanStack/router PR: 5732
File: packages/start-client-core/src/client/hydrateStart.ts:6-9
Timestamp: 2025-11-02T16:16:24.898Z
Learning: In packages/start-client-core/src/client/hydrateStart.ts, the `import/no-duplicates` ESLint disable is necessary for imports from `#tanstack-router-entry` and `#tanstack-start-entry` because both aliases resolve to the same placeholder file (`fake-start-entry.js`) in package.json during static analysis, even though they resolve to different files at runtime.

Applied to files:

  • examples/solid/quickstart-webpack-file-based/.vscode/settings.json
  • examples/solid/quickstart-webpack-file-based/src/app.tsx
  • examples/solid/quickstart-webpack-file-based/src/index.tsx
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
  • examples/solid/quickstart-webpack-file-based/package.json
📚 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 docs/{router,start}/** : Place router docs under docs/router/ and start framework docs under docs/start/

Applied to files:

  • docs/router/framework/solid/installation/with-webpack.md
📚 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 packages/router-plugin/** : Use unplugin for universal bundler plugins in the router-plugin package

Applied to files:

  • docs/router/framework/solid/installation/with-webpack.md
  • examples/solid/quickstart-webpack-file-based/package.json
📚 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 packages/router-core/** : Keep framework-agnostic core router logic in packages/router-core/

Applied to files:

  • examples/solid/quickstart-webpack-file-based/src/app.tsx
  • examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx
📚 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 **/src/routes/** : Place file-based routes under src/routes/ directories

Applied to files:

  • examples/solid/quickstart-webpack-file-based/src/routes/about.tsx
  • examples/solid/quickstart-webpack-file-based/src/routeTree.gen.ts
  • examples/solid/quickstart-webpack-file-based/src/routes/index.tsx
📚 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 **/package.json : Use workspace:* protocol for internal dependencies in package.json files

Applied to files:

  • examples/solid/quickstart-webpack-file-based/package.json
🧬 Code graph analysis (3)
examples/solid/quickstart-webpack-file-based/src/routes/about.tsx (2)
examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx (1)
  • Route (4-6)
examples/solid/quickstart-webpack-file-based/src/routes/index.tsx (1)
  • Route (3-5)
examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx (3)
examples/solid/quickstart-webpack-file-based/src/routes/about.tsx (1)
  • Route (3-5)
examples/solid/quickstart-webpack-file-based/src/routes/index.tsx (1)
  • Route (3-5)
examples/solid/quickstart-file-based/src/routes/__root.tsx (1)
  • RootComponent (13-41)
examples/solid/quickstart-webpack-file-based/src/routes/index.tsx (2)
examples/solid/quickstart-webpack-file-based/src/routes/__root.tsx (1)
  • Route (4-6)
examples/solid/quickstart-webpack-file-based/src/routes/about.tsx (1)
  • Route (3-5)
🪛 markdownlint-cli2 (0.18.1)
docs/router/framework/solid/installation/with-webpack.md

29-29: Link text should be descriptive

(MD059, descriptive-link-text)

⏰ 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 (7)
examples/solid/quickstart-webpack-file-based/README.md (1)

1-6: LGTM!

Clear and concise instructions for running the example.

examples/solid/quickstart-webpack-file-based/public/index.html (1)

1-12: LGTM!

Standard HTML entry point with proper meta tags and Tailwind CSS CDN for styling. The structure is appropriate for an example application.

examples/solid/quickstart-webpack-file-based/.gitignore (1)

1-11: LGTM!

Standard ignore patterns covering local files, build artifacts, and IDE metadata.

examples/solid/quickstart-webpack-file-based/src/routes/index.tsx (1)

1-13: LGTM!

Properly structured file-based route using createFileRoute with a simple home component. Follows TanStack Router conventions for Solid.

examples/solid/quickstart-webpack-file-based/src/routes/about.tsx (1)

1-13: LGTM!

Properly structured file-based route for the about page, following the same pattern as the index route.

examples/solid/quickstart-webpack-file-based/tsconfig.json (1)

1-16: LGTM!

TypeScript configuration is properly set up for Solid.js with strict mode enabled and appropriate compiler options for the Webpack environment.

examples/solid/quickstart-webpack-file-based/.vscode/settings.json (1)

1-11: LGTM!

Properly configures VSCode to treat the autogenerated routeTree.gen.ts file as readonly and excludes it from watchers and search results. This aligns with best practices for handling generated files.

Comment on lines +10 to +12
"@tanstack/solid-router": "^1.135.2",
"@tanstack/solid-router-devtools": "^1.135.2",
"solid-js": "^1.9.10"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Use workspace protocol for internal TanStack dependencies.

Internal TanStack packages should use the workspace:* protocol instead of specific version numbers to ensure proper monorepo dependency resolution.

Apply this diff to fix the dependencies:

   "dependencies": {
-    "@tanstack/solid-router": "^1.135.2",
-    "@tanstack/solid-router-devtools": "^1.135.2",
+    "@tanstack/solid-router": "workspace:*",
+    "@tanstack/solid-router-devtools": "workspace:*",
     "solid-js": "^1.9.10"
   },

Based on learnings

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"@tanstack/solid-router": "^1.135.2",
"@tanstack/solid-router-devtools": "^1.135.2",
"solid-js": "^1.9.10"
"@tanstack/solid-router": "workspace:*",
"@tanstack/solid-router-devtools": "workspace:*",
"solid-js": "^1.9.10"
🤖 Prompt for AI Agents
In examples/solid/quickstart-webpack-file-based/package.json around lines 10 to
12, the TanStack internal packages are pinned to fixed versions; replace the
version strings for "@tanstack/solid-router" and
"@tanstack/solid-router-devtools" with the workspace protocol (e.g.,
"workspace:*") so the monorepo resolves internal dependencies correctly, leaving
"solid-js" as-is.

"@babel/core": "^7.28.5",
"@babel/preset-typescript": "^7.27.1",
"@swc/core": "^1.10.15",
"@tanstack/router-plugin": "^1.135.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Use workspace protocol for @tanstack/router-plugin.

The @tanstack/router-plugin is an internal package and should use the workspace:* protocol.

Apply this diff:

     "@swc/core": "^1.10.15",
-    "@tanstack/router-plugin": "^1.135.2",
+    "@tanstack/router-plugin": "workspace:*",
     "babel-loader": "^10.0.0",

Based on learnings

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"@tanstack/router-plugin": "^1.135.2",
"@swc/core": "^1.10.15",
"@tanstack/router-plugin": "workspace:*",
"babel-loader": "^10.0.0",
🤖 Prompt for AI Agents
In examples/solid/quickstart-webpack-file-based/package.json around line 18, the
dependency "@tanstack/router-plugin" uses a semantic version "^1.135.2" but it
should reference the local/internal package via the workspace protocol; update
the dependency entry to use "workspace:*" (i.e., replace the version string with
workspace:*), save package.json and run your package manager's install so the
workspace link is resolved.

@birkskyum birkskyum merged commit 71f9c7d into main Nov 12, 2025
6 checks passed
@birkskyum birkskyum deleted the webpack-solid-router branch November 12, 2025 16:09
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