Skip to content

fix(core): use bcode database filename#102

Merged
Alezander9 merged 1 commit into
mainfrom
db-filename
Jun 23, 2026
Merged

fix(core): use bcode database filename#102
Alezander9 merged 1 commit into
mainfrom
db-filename

Conversation

@Alezander9

@Alezander9 Alezander9 commented Jun 23, 2026

Copy link
Copy Markdown
Member

Summary:

  • Restore BrowserCode database filenames in the new core database path owner.
  • Add a focused Database.path test for bcode.db and bcode-.db behavior.

Context:

  • The upstream v1.17 sync moved DB path ownership from packages/opencode/src/storage/db.ts to packages/core/src/database/database.ts and accidentally reintroduced opencode.db.
  • BrowserCode memory documents bcode.db as the intended branding invariant.

Tests:

  • bun test test/database-path.test.ts (packages/core)
  • bun typecheck (packages/core)
  • pre-push filtered bun turbo typecheck

Summary by cubic

Restore BrowserCode database filenames in packages/core: use bcode.db and bcode-<channel>.db instead of opencode.db, fixing a regression from the v1.17 sync.

  • Bug Fixes
    • Updated Database.path() to return bcode.db or bcode-<channel>.db, and honor OPENCODE_DISABLE_CHANNEL_DB.
    • Added packages/core/test/database-path.test.ts to cover default and env-var behavior.

Written for commit 5da41a9. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Re-trigger cubic

@Alezander9 Alezander9 merged commit a3739c7 into main Jun 23, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant