solidjs / solid / 34726253392

72%

main: 89%

push

github

docs: correct createContext guidance for server rendering (#3395)

The createContext JSDoc told readers to reach for a module-scope
signal or store for app-wide state. With ssr: true a module is
evaluated once per process, so that state is shared by every request:
a store derived from a server function leaks one user's data into
another's render, and a module-scope signal has no owner to dispose
it. The 2.0 docs say the opposite, and the reference page is generated
from this JSDoc, so the site contradicted itself.

Also fix two smaller items in the same examples:

- The createContext and useContext @example blocks render <TodoList />,
  whose stub returns nothing, so the examples fail tsc as written.
- The handleServerFunctionRequest @example imported from
  "@solidjs/web/server-functions", which resolves to the client entry
  under the browser condition; the function is exported from
  "@solidjs/web/server-functions/server".

519 of 790 branches covered (65.7%)

Branch coverage included in aggregate %.

772 of 1007 relevant lines covered (76.66%)

15.01 hits per line