Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
c92adf4 to
197c158
Compare
|
@coderabbitai review |
|
197c158 to
6c41f37
Compare

Chained PRs
Motivation
Large browser databases need bounded-memory writes and path-level activation primitives before SQLite Web can safely install streamed snapshots. Review of the unchanged vendor baseline also exposed upstream VFS and time-conversion defects that belong in this modification layer.
Solution
OpfsSAHPoolImport, which appends chunks directly to an OPFS SAH-pool slot.begin_import_db,has_path, andrename_pathoperations for staging, backup, activation, and rollback.This layer remains application-agnostic and does not add the SQLite Web snapshot API.
Review-loop fixes
Codex review hardened failure cleanup, rename/import atomicity, metadata compatibility, path validation, temporary-file handling, time conversion, and deterministic legacy/canonical path resolution. Generated build artifacts were removed from the commit.
Verification
git diff --checkpassed