Skip to content

feat: support path-aware exists and delete APIs#1070

Open
mrousavy wants to merge 1 commit into
mainfrom
codex/path-aware-exists-delete
Open

feat: support path-aware exists and delete APIs#1070
mrousavy wants to merge 1 commit into
mainfrom
codex/path-aware-exists-delete

Conversation

@mrousavy

@mrousavy mrousavy commented Jun 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • add optional path arguments to existsMMKV(id, path?) and deleteMMKV(id, path?)
  • pass root paths through to native MMKV::checkExist/MMKV::removeStorage
  • expose public getBaseDirectory() so examples/tests do not import internal platform-context helpers
  • keep custom-path Harness coverage native-only and remove the explicit Web negative-path test

Testing

  • bun run specs
  • bun run typecheck
  • bun run lint
  • bun run lint-ci
  • bun run lint-cpp
  • bun run test
  • HARNESS_WEB_URL=http://localhost:3001 bun run test:harness --harnessRunner web
  • ./gradlew :react-native-mmkv:assembleRelease --no-daemon

Upstream reference: Tencent MMKV exposes removeStorage/checkExist with an optional root path: https://github.com/Tencent/MMKV/blob/master/Core/MMKV.h#L749-L752

@mrousavy mrousavy force-pushed the codex/path-aware-exists-delete branch from 3059697 to e9ce7d3 Compare June 4, 2026 08:26
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