fix(add-clidash): mkdir -p tools in install step; require Node >=22.5

Address review feedback on PR #2795:
- SKILL.md Step 1 now creates tools/ before cp -R (cp does not create
  intermediate parents; tools/ is not a standard NanoClaw dir, so the copy
  failed on a fresh checkout).
- package.json engines bumped to >=22.5 — server.js statically imports
  activity.js -> node:sqlite (DatabaseSync), which lands in Node 22.5, so
  the server crashed at module load on Node 20/21 LTS.
- Doc references (SKILL.md, README.md) updated to Node >=22.5 for consistency.

Verified 87/87 tests pass on Node 22.14.
This commit is contained in:
leetwito
2026-07-04 15:38:38 +03:00
parent 8ee7915418
commit 6c2d26836f
3 changed files with 8 additions and 3 deletions
+5 -1
View File
@@ -41,7 +41,11 @@ build step, no push pipeline, and no edits to NanoClaw source — it just reads
clidash is fully self-contained — copy the whole directory in:
`tools/` is not a standard NanoClaw directory and `cp -R` won't create it, so
make it first:
```bash
mkdir -p tools
cp -R .claude/skills/add-clidash/add/tools/clidash tools/clidash
```
@@ -76,7 +80,7 @@ Tests use a stub CLI — no real `ncl` or `docker` needed:
npm test
```
All tests should pass (Node ≥ 20, `node:test`, zero dependencies).
All tests should pass (Node ≥ 22.5, `node:test`, zero dependencies).
### 4. Run and verify
@@ -9,7 +9,8 @@ It ships pre-wired for NanoClaw's `ncl` CLI (agent groups, sessions, messaging
groups, wirings, users, roles, …) plus `docker`, but the same config shape
works for any list-as-JSON CLI.
- **Zero dependencies** — Node built-ins only (Node ≥ 20), no build step,
- **Zero dependencies** — Node built-ins only (Node ≥ 22.5, for `node:sqlite`),
no build step,
vanilla-JS frontend.
- **Read-only by construction** — the server can only `execFile` the configured
argv templates; `{resource}` is the sole substitution and is validated
@@ -9,6 +9,6 @@
"test": "node --test 'test/*.test.js'"
},
"engines": {
"node": ">=20"
"node": ">=22.5"
}
}