You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
maintain CI for the actual Linux glibc x86_64 and macOS ARM64 targets, with a patch inventory
Dropped old Guion patches include reader busy_timeout, extra BEGIN IMMEDIATE sites, the Reader release sender, From<io::Error>, rusqlite 0.32 alignment, broad lint allowances, and fork-specific musl/no-rusqlite/Windows checks.
Why
Official v0.0.7 already contains upload retry behavior and newer PowerSync core/rusqlite dependencies. Carrying the old concurrency workarounds would hide bugs and make upstream synchronization harder. Each retained behavioral patch has a deterministic regression test. The fork intentionally follows upstream's reqwest defaults and default rusqlite consumer path.
FlickNote disposable canary compiled on rusqlite 0.39 after removing its remaining SQLx SQLite dependency
FlickNote actor canary passed live write, pre-connect backlog, and transient retry scenarios
Downstream migration note
FlickNote must remove its remaining SQLx SQLite dependency before pinning this release because SQLx 0.8 and rusqlite 0.39 link incompatible libsqlite3-sys versions. That cleanup is already part of FlickNote plan #1783. Its generic connector errors should use upstream PowerSyncError::upload_error rather than restoring the old From<io::Error> patch.
CI bootstrap note: this PR introduces the repository's first pull-request workflow and adds pull_request to the existing CI workflow. GitHub does not register a new PR trigger until the workflow exists on the default branch, so PR #3 currently reports no checks. The full local matrix and FlickNote downstream canary are recorded in the PR body. After merge, the main-branch push will run the newly installed CI; subsequent PRs will receive normal checks.
Resolved the reported merge conflict with merge commit 88cd55b. This uses Git's ours merge strategy only to reconcile the old Guion 0.0.5 history with the intentionally rebuilt upstream 0.0.7 history. The candidate file tree is byte-for-byte unchanged from verified commit 4af5094 (same tree 8a6245a9…; git diff 4af5094..88cd55b is empty), so no retired patches were reintroduced and no force-push was used.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Rebuild the Guion fork on official PowerSync Native v0.0.7 and retain only evidence-backed downstream patches.
Retained fixes:
ConnectionEstablishedonly after a successful HTTP responseDropped old Guion patches include reader
busy_timeout, extraBEGIN IMMEDIATEsites, the Reader release sender,From<io::Error>, rusqlite 0.32 alignment, broad lint allowances, and fork-specific musl/no-rusqlite/Windows checks.Why
Official v0.0.7 already contains upload retry behavior and newer PowerSync core/rusqlite dependencies. Carrying the old concurrency workarounds would hide bugs and make upstream synchronization harder. Each retained behavioral patch has a deterministic regression test. The fork intentionally follows upstream's reqwest defaults and default rusqlite consumer path.
Verification
cargo test --workspace --all-features— 59 passedcargo fmt --all --check— passedcargo deny check— passedDownstream migration note
FlickNote must remove its remaining SQLx SQLite dependency before pinning this release because SQLx 0.8 and rusqlite 0.39 link incompatible
libsqlite3-sysversions. That cleanup is already part of FlickNote plan #1783. Its generic connector errors should use upstreamPowerSyncError::upload_errorrather than restoring the oldFrom<io::Error>patch.Target release after merge:
v0.0.7-guion.1.