Skip to content

Commit 2d9350c

Browse files
committed
docs(skills): correct the install boundary policy and drop the false naming rule
1 parent 54d2ce5 commit 2d9350c

4 files changed

Lines changed: 33 additions & 9 deletions

File tree

‎packages/skills/README.md‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,9 +82,12 @@ as plain relative paths, and a symlink anywhere *inside* the destination — a
8282
managed file or one of its parent directories — is refused rather than written
8383
through, so nothing outside can be created or deleted.
8484

85-
The destination root itself is followed if it is a symlink, because a shared
86-
skills directory is a reasonable layout. If you do not want that, pass an
87-
explicit `--dir`.
85+
An auto-detected destination is confined to your project: if `.claude/skills`
86+
(or any parent) is a symlink pointing outside it, the install is refused rather
87+
than followed, so a repository cannot redirect it somewhere else.
88+
89+
Installing outside the project is opt-in — pass `--dir <path>` or `--global`
90+
and that boundary is lifted, because you named the destination yourself.
8891

8992
## Fetching instead of installing
9093

‎packages/skills/skill/SKILL.md‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,15 @@ import {
9898
`createPolyPerspectiveCamera` for depth foreshortening.
9999
- The camera is the **outer** node; the scene nests inside it. CSS `perspective`
100100
only applies to descendants.
101-
- Every public export is `Poly`-prefixed, except generic math types (`Vec2`,
102-
`Vec3`, `Polygon`, `PolyMaterial`) and the `*/three` subpaths.
101+
- **Do not infer names from a prefix rule.** `Poly` prefixing is a convention
102+
for newer renderer-facing components, hooks and types — not a description of
103+
the export inventory. Plenty of public names have no prefix: `loadMesh`,
104+
`parseObj` / `parseStl` / `parseGltf` / `parseVox`, every `*Polygons`
105+
generator, `BASE_TILE`, `LoopOnce` / `LoopRepeat` / `LoopPingPong`, the
106+
generic math types (`Vec2`, `Vec3`, `Polygon`), and the vanilla factories
107+
`createSelect` and `createTransformControls`. The `*/three` subpaths use
108+
Three-compatible names deliberately. Check
109+
[docs/api-index.md](docs/api-index.md) rather than guessing.
103110

104111
## Authoring polygons — the five silent failures
105112

‎website/public/skill.md‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,15 @@ import {
9898
`createPolyPerspectiveCamera` for depth foreshortening.
9999
- The camera is the **outer** node; the scene nests inside it. CSS `perspective`
100100
only applies to descendants.
101-
- Every public export is `Poly`-prefixed, except generic math types (`Vec2`,
102-
`Vec3`, `Polygon`, `PolyMaterial`) and the `*/three` subpaths.
101+
- **Do not infer names from a prefix rule.** `Poly` prefixing is a convention
102+
for newer renderer-facing components, hooks and types — not a description of
103+
the export inventory. Plenty of public names have no prefix: `loadMesh`,
104+
`parseObj` / `parseStl` / `parseGltf` / `parseVox`, every `*Polygons`
105+
generator, `BASE_TILE`, `LoopOnce` / `LoopRepeat` / `LoopPingPong`, the
106+
generic math types (`Vec2`, `Vec3`, `Polygon`), and the vanilla factories
107+
`createSelect` and `createTransformControls`. The `*/three` subpaths use
108+
Three-compatible names deliberately. Check
109+
[docs/api-index.md](/skill/docs/api-index.md) rather than guessing.
103110

104111
## Authoring polygons — the five silent failures
105112

‎website/public/skill/SKILL.md‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,15 @@ import {
9898
`createPolyPerspectiveCamera` for depth foreshortening.
9999
- The camera is the **outer** node; the scene nests inside it. CSS `perspective`
100100
only applies to descendants.
101-
- Every public export is `Poly`-prefixed, except generic math types (`Vec2`,
102-
`Vec3`, `Polygon`, `PolyMaterial`) and the `*/three` subpaths.
101+
- **Do not infer names from a prefix rule.** `Poly` prefixing is a convention
102+
for newer renderer-facing components, hooks and types — not a description of
103+
the export inventory. Plenty of public names have no prefix: `loadMesh`,
104+
`parseObj` / `parseStl` / `parseGltf` / `parseVox`, every `*Polygons`
105+
generator, `BASE_TILE`, `LoopOnce` / `LoopRepeat` / `LoopPingPong`, the
106+
generic math types (`Vec2`, `Vec3`, `Polygon`), and the vanilla factories
107+
`createSelect` and `createTransformControls`. The `*/three` subpaths use
108+
Three-compatible names deliberately. Check
109+
[docs/api-index.md](docs/api-index.md) rather than guessing.
103110

104111
## Authoring polygons — the five silent failures
105112

0 commit comments

Comments
 (0)