Skip to content

docs: sync diagnostics pages and the Vite plugin's test environment - #1665

Open
ryansolid wants to merge 1 commit into
v2-rebuildfrom
docs/sync-diagnostics-vite-test-env
Open

ryansolid wants to merge 1 commit into
v2-rebuildfrom
docs/sync-diagnostics-vite-test-env

Conversation

@ryansolid

Copy link
Copy Markdown
Member

Requested by Ryan · project thread

  • I have read the Contribution guide
  • This PR references an issue (sync with upstream changes, no issue)

Description(required)

Diagnostics reference. I regenerated it with scripts/extract-diagnostics.mjs against solid next at 6cca8b57.

  • Adds STORE_WRITE_OUTSIDE_SETTER (solid #4194) and UNCOMMITTED_INPUT_OVERWRITTEN (solid #4205).
  • Removes BINDING_SLOT_POSITION, since binding slots were removed upstream.
  • Picks up the skill's text changes to EFFECT_RELAY_TEAR, LOWERCASE_EVENT_ATTRIBUTE, REACTIVITY_HALTED, SSR_BOUNDARY_WATERFALL and SSR_LIVE_HOLE_LATCHED.
  • Generator changes: the FIX_STARTS entry for SSR_BOUNDARY_WATERFALL now matches the skill's new capitalization (the run failed without this), and I deleted the dead BINDING_SLOT_POSITION entry.

Vite plugin test environment. @solidjs/vite-plugin no longer defaults test.environment to jsdom (vite-plugin-solid #459).

  • Testing guide: removed the "Browser mode still needs jsdom installed" note and its troubleshooting entry, and removed jsdom from the browser-mode install line. A new troubleshooting entry covers document is not defined.
  • vite-plugin-solid/options: the Vitest bullet now says the plugin doesn't set the environment.
  • Migration from Solid 1, "Update tests": adds "The Vite plugin no longer sets test.environment" and says to set it.

These pages are what the From React additions in #1664 need to link to (both new codes). lint:tone (only a pre-existing generated warning remains), check:lint and prettier all pass.

Related issues & labels


Generated by Claude Code

- Regenerate the diagnostics reference from solid next 6cca8b57: adds
  STORE_WRITE_OUTSIDE_SETTER (#4194) and UNCOMMITTED_INPUT_OVERWRITTEN
  (#4205), drops BINDING_SLOT_POSITION (binding slots were removed), and
  picks up skill edits to EFFECT_RELAY_TEAR, LOWERCASE_EVENT_ATTRIBUTE,
  REACTIVITY_HALTED, SSR_BOUNDARY_WATERFALL and SSR_LIVE_HOLE_LATCHED.
  The generator's FIX_STARTS entry for SSR_BOUNDARY_WATERFALL follows the
  skill's new capitalization, and the dead BINDING_SLOT_POSITION entry goes.
- The Vite plugin no longer defaults test.environment to jsdom
  (vite-plugin-solid #459): the testing guide drops the "browser mode still
  needs jsdom" note and troubleshooting entry, browser mode no longer
  installs jsdom, and a new entry covers `document is not defined`. The
  plugin options reference and the Solid 1 migration guide say to set the
  environment.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01R9d4iekBWmtH343gNAGJpd
@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@ryansolid ryansolid self-assigned this Oct 21, 2026
@netlify

netlify Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for solid-docs-v2 ready!

Name Link
🔨 Latest commit c68b968
🔍 Latest deploy log https://app.netlify.com/projects/solid-docs-v2/deploys/6ad8f7f390b6a71b79d52552
😎 Deploy Preview https://deploy-preview-1665--solid-docs-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants