Skip to content

feat: add bundled Remote Config defaults - #858

Closed
shameondev wants to merge 1 commit into
codex/rc-lkg-durable-commitfrom
codex/rc-v2-bundled-fallback
Closed

feat: add bundled Remote Config defaults#858
shameondev wants to merge 1 commit into
codex/rc-lkg-durable-commitfrom
codex/rc-v2-bundled-fallback

Conversation

@shameondev

Copy link
Copy Markdown
Contributor

Summary

  • add a synchronous @JvmStatic pre-initialization getter taking only Context and logical key
  • read only qonversion_remote_config_defaults.json from app assets
  • validate the exact configurator schema-1 artifact, framed digest, canonical bytes, limits, portable numbers and Unicode surrogate pairs
  • expose immutable JSON-compatible values in a nullable wrapper so present JSON null differs from missing/corrupt
  • cache one fail-closed process document and pin exact configurator goldens

Safety boundary

This is an additive bundled-default getter stacked on persistent LKG #857. It does not add Fetched/Active orchestration, a new network endpoint, identity transitions, or change legacy fallback behavior. It never reads SharedPreferences, singleton state, identity, Documents, or network.

Verification

  • 14 focused reader/public API tests pass
  • detektAll passes
  • :sdk:assembleDebug passes
  • independent agent review approved after portable header-number and Unicode surrogate fixes

lintDebug still reports the pre-existing untouched min-API issue in internal/storage/Cache.kt:33; this PR introduces no new lint finding.

@shameondev
shameondev force-pushed the codex/rc-v2-bundled-fallback branch from 2a68be0 to 97d6dd0 Compare August 5, 2026 21:02
@shameondev
shameondev changed the base branch from codex/rc-persistent-lkg to codex/rc-lkg-durable-commit August 5, 2026 21:02
@shameondev

Copy link
Copy Markdown
Contributor Author

Консолидировано в release-train PR (один PR на репозиторий). Ветка сохранена.

@shameondev shameondev closed this Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant