-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Bug]: Missing database-first legacy-store guard script #91628
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.bugSomething isn't workingSomething isn't workingclawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PR
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.bugSomething isn't workingSomething isn't workingclawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PR
Type
Fields
Priority
None yet
Bug type
Behavior bug (incorrect output/state without crash)
Beta release blocker
No
Summary
The database-first validation plan references
pnpm check:database-first-legacy-stores, but the package script is missing, leaving no reliable CI guard against reintroducing legacy runtime state stores.Steps to reproduce
check:database-first-legacy-stores.Expected behavior
pnpm check:database-first-legacy-storesshould exist and fail new runtime source that writes legacy session/transcript/state/cache stores outside doctor, migration, or explicit fixture owners.Actual behavior
The script is missing. The SQLite plan execution recorded the proof as missing and noted that manual searches produced too much noise to serve as a regression guard.
OpenClaw version
2026.6.2 source checkout at
6fcc9457020e5a9cb21ba2dbceb010a765a3335dOperating system
Linux
Install method
Source checkout using
pnpmModel
NOT_ENOUGH_INFO
Provider / routing chain
NOT_ENOUGH_INFO
Additional provider/model setup details
NOT_ENOUGH_INFO
Logs, screenshots, and evidence
Impact and severity
Affected: database-first architecture validation for core and plugin runtime state changes.
Severity: High, because focused SQLite store tests can be green while runtime legacy file writes are reintroduced without an early CI failure.
Frequency: Always, until the package script and guard are added.
Consequence: Runtime
sessions.json, transcript JSONL, sidecar SQLite, and JSON/JSONL state/cache regressions can slip through normal check paths.Additional information
The observed local bug bundle contained no secrets.