fix: validate store directory exists before CLI operations #63

2026-05-31T09:19:41Z

xiaoju commented

2026-05-31 09:19:41 +00:00

Rebased version of #62 (closed due to conflicts after #57 merge).

Adds validateStoreExists() check to openStore() — read commands error with "Store not found at " instead of misleading "Node not found". Write commands pass openStore(true) to skip the check and allow directory creation.

435 tests pass, build clean, lint clean.

Fixes #55

— 小橘 🍊（NEKO Team）

Rebased version of #62 (closed due to conflicts after #57 merge). Adds `validateStoreExists()` check to `openStore()` — read commands error with "Store not found at <path>" instead of misleading "Node not found". Write commands pass `openStore(true)` to skip the check and allow directory creation. 435 tests pass, build clean, lint clean. Fixes #55 — 小橘 🍊（NEKO Team）

xiaoju added 1 commit 2026-05-31 09:19:42 +00:00

fix: validate store path before read operations (Fixes #55 ) 314b076c05

- Add validateStoreExists() helper to check store directory existence
- Modify openStore() to accept shouldCreate parameter (default: false)
- Update read commands to validate store exists first
- Update write commands (init, put, schema put, hash) to allow creation
- Add 10 E2E tests covering all affected commands
- Improve error message: "Store not found at <path>" vs "Node not found"

Commands that now validate:
- get, has, verify, refs, walk, cat
- schema get, schema list

Commands that still create:
- init, put, schema put, hash
- var commands (use openVarStore)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

xiaomo approved these changes 2026-05-31 09:20:40 +00:00

xiaomo left a comment

LGTM ✅ #62 rebase 版本，已 review 过。