get test packages#7519
Conversation
The server `Register` augmentation (`server.requestContext`) was not seen by
`createMiddleware().server` context nor by `handler.fetch`, leaving `context`
typed as `undefined`/`BaseContext`.
The default server entry typed `handler.fetch` via `RequestHandler<Register>`
with `Register` from `@tanstack/{react,solid}-router`. Once an app augments the
router's `Register` (e.g. the universal `router` registration in router.tsx),
that interface becomes a separately-merged declaration that no longer sees the
start `server.requestContext` augmentation. Read `Register` from
`@tanstack/{react,solid}-start` instead so the handler context aligns with the
registered context and the server middleware chain.
Also align docs to augment `@tanstack/react-start` (matching the generated
route-tree footer) and add an e2e repro.
Co-authored-by: opencode <noreply@opencode.ai>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 fb72d53
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
Bundle Size Benchmarks
Current gzip tracks all emitted client JS chunks. Initial gzip tracks only the entry/import graph. Trend sparkline is historical current gzip ending with this PR measurement; lower is better. |
Merging this PR will not alter performance
Comparing Footnotes
|
There was a problem hiding this comment.
Nx Cloud has identified a possible root cause for your failed CI:
We classified these failures as environment_state rather than code_change because neither failing task is touched by this PR's changes (TypeScript type fixes for server request context), and the primary blocker is a deterministic SyntaxError in a stale e2e-utils dist artifact that has a 0% flakiness rate. The second failure is a non-deterministic CSS assertion with a 15.86% flakiness rate, also unrelated to the PR's scope.
No code changes were suggested for this issue.
Trigger a rerun:
🎓 Learn more about Self-Healing CI on nx.dev
No description provided.