Thanks to visit codestin.com
Credit goes to github.com

Skip to content

docs: add skills - #3362

Draft
lxsmnsyc wants to merge 1 commit into
solidjs:nextfrom
lxsmnsyc:solid-usage-skills
Draft

docs: add skills#3362
lxsmnsyc wants to merge 1 commit into
solidjs:nextfrom
lxsmnsyc:solid-usage-skills

Conversation

@lxsmnsyc

Copy link
Copy Markdown
Member

Three skills under .claude/skills/, each a self-contained SKILL.md that an agent picks up on this checkout and that can be copied into another project. They cover usage patterns for people writing Solid apps, not workflows for working on this repository.

  • solid-reactivity: the primitive signatures that changed from Solid 1.x. createEffect takes a compute and an effect, a function argument to createSignal or createStore makes it derived, async is a computation that throws while pending, and the 1.x names that are gone map to their replacements.
  • solid-control-flow: Show, For, Repeat, Switch, Match, Loading, Errored, Dynamic, Portal, and Reveal, with the callback-child and keyed semantics that decide what remounts.
  • solid-server-functions: the closure rule the compiler enforces, why the boundary is not authentication, what declaring GET gives up, error sanitization, and the deployment secret the no-JS form path needs.

Every claim is taken from the source in this repository rather than from Solid 1.x, including the removed-API map in packages/solid/src/index.ts.

Three skills under `.claude/skills/`, each a self-contained SKILL.md that an
agent picks up on this checkout and that can be copied into another project.
They cover usage patterns for people writing Solid apps, not workflows for
working on this repository.

- `solid-reactivity`: the primitive signatures that changed from Solid 1.x.
  `createEffect` takes a compute and an effect, a function argument to
  `createSignal` or `createStore` makes it derived, async is a computation
  that throws while pending, and the 1.x names that are gone map to their
  replacements.
- `solid-control-flow`: Show, For, Repeat, Switch, Match, Loading, Errored,
  Dynamic, Portal, and Reveal, with the callback-child and keyed semantics
  that decide what remounts.
- `solid-server-functions`: the closure rule the compiler enforces, why the
  boundary is not authentication, what declaring GET gives up, error
  sanitization, and the deployment secret the no-JS form path needs.

Every claim is taken from the source in this repository rather than from
Solid 1.x, including the removed-API map in `packages/solid/src/index.ts`.

Co-Authored-By: Claude Opus 5 <[email protected]>
@changeset-bot

changeset-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 4a24b42

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@lxsmnsyc
lxsmnsyc marked this pull request as draft September 11, 2026 04:49
@codspeed-hq

codspeed-hq Bot commented Sep 11, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 54.69%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 1 regressed benchmark
✅ 157 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
projection derive: write one NESTED field (reference) 262.3 µs 579 µs -54.69%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing lxsmnsyc:solid-usage-skills (4a24b42) with next (27aee36)

Open in CodSpeed

@nickshiro

Copy link
Copy Markdown

Coming from the PR discussion #3343 - I think this hits the same issue, just with higher stakes.

.claude/skills/ is vendor-specific tooling integration, not project content, the same way .vscode/ or .cursor/* are — except it's worse, because it's not something you can ignore and forget. It's living content that has to stay in sync with the actual API, duplicating what should already live in the docs, just reformatted for one vendor's schema.

And it won't stop at one. Cursor has its own rules format, Copilot has .github/copilot-instructions.md, and more will show up. If this repo takes SKILL.md, there's no principled way to say no to the next vendor's format - and in a year there's a shelf of overlapping, vendor-specific files that all encode the same knowledge about the same API, none of which the maintainers necessarily have the tooling expertise to review for correctness.

If the underlying knowledge (signatures, control-flow semantics, server-function rules) is accurate, it belongs in the project's actual docs - vendor-neutral, human-readable, maintained by people who know the internals. Turning that into a SKILL.md, .cursorrules, or anything else is a mechanical format conversion that doesn't need core-repo review or maintenance - it's a good fit for a separate community project (or something the vendor generates itself from existing docs), not something baked into solidjs/solid.

@nickshiro

Copy link
Copy Markdown

Speaking of my understanding of the field - I use Claude Code, too.

@lxsmnsyc

Copy link
Copy Markdown
Member Author

@nickshiro no worries, this is still a draft and isn't final yet. I'm still exploring the recipes before moving this to plugin form. Take note that the skills were meant to be plugins to be used by users, not a guide for the repo structure.

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