#38 Phase 4: FsStore implements Store (move VariableStore to fs) #42

New Issue

2026-06-02T05:30:43Z

xiaoju commented

2026-06-02 05:30:43 +00:00

Parent

Part of #38 — unified Store type refactor.

Depends on

Phase 3 (core functions use Store)

What

Move variable-store.ts from core to fs
createFsStore() / openStore() returns Store with:
- cas: existing FsStore (SQLite CAS)
- var: SQLite VarStore (migrated from core)
- tag: SQLite TagStore (extracted from variable tag tables)
Delete variable-store.ts from core
Ensure @ocas/core has zero bun:sqlite imports

Acceptance

@ocas/core is pure TypeScript, no runtime deps
@ocas/fs exports openStore(): Promise<Store>
All fs tests pass
Variable/tag tests migrated to fs package

小橘 🍊（NEKO Team）

## Parent Part of #38 — unified Store type refactor. ## Depends on Phase 3 (core functions use Store) ## What - Move `variable-store.ts` from `core` to `fs` - `createFsStore()` / `openStore()` returns `Store` with: - `cas`: existing FsStore (SQLite CAS) - `var`: SQLite VarStore (migrated from core) - `tag`: SQLite TagStore (extracted from variable tag tables) - Delete `variable-store.ts` from core - Ensure `@ocas/core` has zero `bun:sqlite` imports ## Acceptance - `@ocas/core` is pure TypeScript, no runtime deps - `@ocas/fs` exports `openStore(): Promise<Store>` - All fs tests pass - Variable/tag tests migrated to fs package --- 小橘 🍊（NEKO Team）

xiaoju referenced this issue

2026-06-02 09:05:51 +00:00

chore: Phase 3 review cleanup items #47

xiaoju referenced this issue from a commit

2026-06-02 09:27:53 +00:00

feat(fs): add FsStore var/tag test coverage and share validateName

xiaoju referenced a pull request that will close this issue

2026-06-02 09:34:46 +00:00

fix(fs): add FsStore var/tag test coverage and share validateName #48

xiaoju closed this issue

2026-06-02 09:35:09 +00:00

xiaoju referenced this issue