Skip to content

refactor: store sandbox registry in sqlite#91127

Merged
steipete merged 1 commit into
mainfrom
refactor/sandbox-registry-sqlite
Jun 7, 2026
Merged

refactor: store sandbox registry in sqlite#91127
steipete merged 1 commit into
mainfrom
refactor/sandbox-registry-sqlite

Conversation

@steipete

@steipete steipete commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Move sandbox runtime/browser registry reads and writes from sharded JSON files into the shared SQLite state DB.
  • Keep doctor-owned migration for legacy monolithic JSON files and current sharded registry directories, preserving newer SQLite/sharded rows and quarantining invalid sources.
  • Update sandbox/doctor docs for SQLite as the canonical registry store.

Verification

  • node scripts/run-vitest.mjs src/agents/sandbox/registry.test.ts src/commands/doctor-sandbox.warns-sandbox-enabled-without-docker.test.ts
  • pnpm tsgo:core
  • pnpm tsgo:core:test
  • pnpm lint:kysely
  • pnpm format:docs:check
  • git diff --check
  • .agents/skills/autoreview/scripts/autoreview --mode local: clean, no accepted/actionable findings

@steipete
steipete requested a review from a team as a code owner June 7, 2026 09:05
@steipete

steipete commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

Land-ready proof for this PR:

Work done:

  • Sandbox runtime/browser registry now reads/writes sandbox_registry_entries in the shared SQLite state DB.
  • openclaw doctor --fix imports legacy monolithic JSON files and sharded registry directories into SQLite, preserves newer SQLite/sharded rows, and quarantines invalid legacy sources.
  • Sandbox and doctor docs now describe SQLite as the canonical registry target.

Verification run locally:

  • node scripts/run-vitest.mjs src/agents/sandbox/registry.test.ts src/commands/doctor-sandbox.warns-sandbox-enabled-without-docker.test.ts
  • pnpm tsgo:core
  • pnpm tsgo:core:test
  • pnpm lint:kysely
  • pnpm format:docs:check
  • git diff --check
  • .agents/skills/autoreview/scripts/autoreview --mode local clean, no accepted/actionable findings

Known proof gaps:

  • No live Docker sandbox run; change is storage/doctor migration path and covered by focused registry/doctor tests.

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation commands Command implementations docker Docker and sandbox tooling agents Agent runtime and tooling size: L maintainer Maintainer-authored PR labels Jun 7, 2026
@steipete
steipete merged commit ce015ce into main Jun 7, 2026
74 of 77 checks passed
@steipete
steipete deleted the refactor/sandbox-registry-sqlite branch June 7, 2026 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling commands Command implementations docker Docker and sandbox tooling docs Improvements or additions to documentation maintainer Maintainer-authored PR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant