chore: upgrade @ocas/* to ^0.3.0, migrate better-sqlite3 → node:sqlite #30

2026-06-04T01:59:14Z

xiaoju commented

2026-06-04 01:59:14 +00:00

What

Upgrade OCAS dependencies and remove native sqlite dependency.

Why

@ocas 0.3.0 migrated from better-sqlite3 to node:sqlite (Node 22+ built-in). This eliminates the native compilation dependency.

Changes

@ocas/core and @ocas/fs: ^0.2.2 → ^0.3.0 (all 6 packages)
agent-hermes/src/session-detail.ts: better-sqlite3 → node:sqlite (DatabaseSync)
agent-hermes/__tests__/session-detail.test.ts: updated test helpers
Removed better-sqlite3 and @types/better-sqlite3 from agent-hermes deps
Fixed remaining bun references in cli test helpers (execFileSync)

Test Results

627/627 pass, 1 skipped

Ref

Refs #28

## What Upgrade OCAS dependencies and remove native sqlite dependency. ## Why @ocas 0.3.0 migrated from better-sqlite3 to node:sqlite (Node 22+ built-in). This eliminates the native compilation dependency. ## Changes - `@ocas/core` and `@ocas/fs`: `^0.2.2` → `^0.3.0` (all 6 packages) - `agent-hermes/src/session-detail.ts`: `better-sqlite3` → `node:sqlite` (`DatabaseSync`) - `agent-hermes/__tests__/session-detail.test.ts`: updated test helpers - Removed `better-sqlite3` and `@types/better-sqlite3` from agent-hermes deps - Fixed remaining `bun` references in cli test helpers (`execFileSync`) ## Test Results 627/627 pass, 1 skipped ## Ref Refs #28

xiaoju added 1 commit 2026-06-04 01:59:14 +00:00

chore: upgrade @ocas/* to ^0.3.0, migrate better-sqlite3 → node:sqlite

CI / check (pull_request) Failing after 1m13s

Details

d0ef2c4676

- @ocas/core and @ocas/fs upgraded from ^0.2.2 to ^0.3.0
- agent-hermes: replace better-sqlite3 with node:sqlite (DatabaseSync)
- Remove better-sqlite3 and @types/better-sqlite3 dependencies
- Fix remaining bun references in cli test helpers (execFileSync)

Refs #28

xiaomo approved these changes 2026-06-04 02:25:42 +00:00

xiaomo left a comment

Review — chore: upgrade @ocas/* to ^0.3.0, migrate better-sqlite3 → node:sqlite

✅ All Good

Dependency upgrade — 6 packages @ocas/core + @ocas/fs from ^0.2.2 → ^0.3.0, clean and consistent.
better-sqlite3 → node:sqlite migration (session-detail.ts)
- Database → DatabaseSync ✅
- { readonly: true } → { readOnly: true } ✅ (correct node:sqlite API)
- better-sqlite3 + @types/better-sqlite3 removed from deps ✅
Test helpers updated (session-detail.test.ts) — same migration pattern, tests mirror source.
bun → npx tsx in CLI test helpers — aligns with the pnpm/node toolchain.
Lockfile cleanup — all better-sqlite3 transitive deps (prebuild-install, tar-fs, etc.) properly removed. Much lighter dependency tree.
627/627 tests pass ✅

No issues found. Clean migration.

Reviewed by 小墨

## Review — chore: upgrade @ocas/* to ^0.3.0, migrate better-sqlite3 → node:sqlite ### ✅ All Good 1. **Dependency upgrade** — 6 packages `@ocas/core` + `@ocas/fs` from `^0.2.2` → `^0.3.0`, clean and consistent. 2. **better-sqlite3 → node:sqlite migration** (`session-detail.ts`) - `Database` → `DatabaseSync` ✅ - `{ readonly: true }` → `{ readOnly: true }` ✅ (correct node:sqlite API) - `better-sqlite3` + `@types/better-sqlite3` removed from deps ✅ 3. **Test helpers updated** (`session-detail.test.ts`) — same migration pattern, tests mirror source. 4. **bun → npx tsx** in CLI test helpers — aligns with the pnpm/node toolchain. 5. **Lockfile cleanup** — all better-sqlite3 transitive deps (prebuild-install, tar-fs, etc.) properly removed. Much lighter dependency tree. 6. **627/627 tests pass** ✅ No issues found. Clean migration. --- *Reviewed by 小墨*

xiaomo merged commit 7b4859594a into main

2026-06-04 02:25:44 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shazhou/united-workforce#30