[Alpha] Kotlin Multiplatform SDK #1

Manrich121 · 2024-01-30T12:28:39Z

PowerSync Kotlin SDK

Alpha Release paper doc

Alpha version

PowerSync support for Android and iOS platforms in KMP
Support for async/Kotlin coroutines DB queries
Support watch queries
- Platform-specific native bindings to sqlite3_update_hook and sqlite3_commit_hook
Convenience SupabaseConnector published under com.powersync:connector-supabase namespace
Custom gradle plugin for publishing to Sonatype Central
Initial support for building a Swift Framework with SKIE + KMMBridge and published via GitHub action

Todo's

Finish GitHub action to publish to Sonatype Central
Make repo and packages public
~~Remove GitHub auth for maven registry (here) in supabase-todolite app~~

Future work/ideas

…lds root project

… pod and static framework

…S issue.

core/src/commonMain/kotlin/com/powersync/connectors/PowerSyncCredentials.kt

rkistner · 2024-02-27T13:36:28Z

core/src/commonMain/kotlin/com/powersync/connectors/PowerSyncCredentials.kt

+            val parts = jwt.split('.')
+            check(parts.size == 3) { "Invalid JWT" }
+
+            val payload = parts[1].decodeBase64String()


Make sure this works with base64-url encoding, and without padding. See discussion:
powersync-ja/powersync.dart#58 (review)

Alternatively, just remove getExpiryDate and decodeJwt for the alpha.

From ktor's implementation, it does look like they handle padding and byte masks, normalizing the input before decoding.

https://github.com/ktorio/ktor/blob/main/ktor-utils/common/src/io/ktor/util/Base64.kt

Looks like padding is fine, but I don't see anything handling base64url characters (uses - and _ instead of + and /).

This functionality is unlikely to be used much so it's not a massive blocker, but I'd prefer to remove it rather than have a broken implementation

stevensJourney

Looks good from my side.

connectors/build.gradle.kts

connectors/src/commonMain/kotlin/com/powersync/connectors/SupabaseConnector.kt

core/src/commonMain/kotlin/com/powersync/bucket/BucketStorage.kt

core/src/commonMain/kotlin/com/powersync/db/internal/PsInternalDatabase.kt

demos/supabase-todolist/settings.gradle.kts

+        maven {
+            url = uri("https://maven.pkg.github.com/powersync-ja/powersync-kotlin")
+            credentials {
+                username = localProperties.getProperty("GITHUB_USERNAME")
+                password = localProperties.getProperty("GITHUB_TOKEN")
+            }
+        }
+    }


core/src/commonMain/sqldelight/com/powersync/Powersync.sq

…kotlin into init-kmp

…or.supabase` namespace and update demo apps accordingly

…of the coroutine

core/src/commonMain/sqldelight/com/powersync/Powersync.sq

rkistner · 2024-02-29T16:22:04Z

core/src/commonMain/kotlin/com/powersync/connectors/PowerSyncCredentials.kt

+            val parts = jwt.split('.')
+            check(parts.size == 3) { "Invalid JWT" }
+
+            val payload = parts[1].decodeBase64String()


This functionality is unlikely to be used much so it's not a massive blocker, but I'd prefer to remove it rather than have a broken implementation

…from PowerSyncCredentials

Manrich121 added 30 commits December 3, 2023 15:12

init: kmm with shiftUI

a36a7f0

init: kmm with composeApp and shared module

a6f0574

init: kmm with composeApp and shared library module

9dfd2c6

init: ktor in kmm

641182b

init: sqldelight android working

d9f8649

init: sqldelight android working

0802bb3

wip: iosApp depends on two frameworks

2c38a84

wip: create samples directory with complete gradle project includeBui…

687735b

…lds root project

wip: loading powersync slqite extension

1fdcb53

wip: try linking local framework

3adb675

wip: try pod linkOnly

42f978e

works: loading powersync-sqlite-core extension in iOS, using linkOnly…

c6a5b23

… pod and static framework

chore: add cache properties for native iosSim

fffe5c4

chore: clean up and rename namespace

5f6c3a7

wip: implement common API

56594d4

wip: crud classses and interfaces

de0aed8

wip: sync stream implementation

e03513b

wip: powersync DB and connector

20dfe48

wip: Connecting to powersync instance using SupabaseConnector

52c6b80

wip: bucketStorage implementation

1f00ae2

wip: bucketStorage replace JsonNames with SerialName

dd32bfb

wip: Enable async DB ops. Update iOS sqlite version to 3.38.5. Fix iO…

024e629

…S issue.

wip: crud upload impl

00c1302

wip: crud upload impl

6e0a437

wip: watch queries and flows

cf5f585

init: watched queries

d7b2569

clean-up demo app PS integration

8333b3a

polish: Compose UI

f999581

fix: sqlite version on iOS. Add retry logic for streamRequest

cff85b9

wip: PowerSync dialect

408c8ac

Manrich121 added 8 commits February 27, 2024 14:04

PowerSyncCredentials.kt user_id is optional

bc3d412

chore: use actions/checkout@4

ea2cf1d

chore: use actions/cache@4

d88aee9

update readme.md

4fa5f1f

fix: null user_id when steam is connected

a18943d

fix: add try/catch around login to supabase in demo apps

34f749b

remove convenience tables for 'sqlite_master' and 'sqlite_sequence'

6f7b3b3

chore: use newer version of Sqlite 3.45

2b8f7b1

Manrich121 requested review from rkistner, DominicGBauer and stevensJourney February 27, 2024 13:16

rkistner reviewed Feb 27, 2024

View reviewed changes

Update readme: limitations and future work

0669106

stevensJourney previously approved these changes Feb 28, 2024

View reviewed changes

Manrich121 requested review from rkistner and removed request for DominicGBauer February 29, 2024 08:20

rkistner reviewed Feb 29, 2024

View reviewed changes

Manrich121 added 2 commits February 29, 2024 12:23

fix: add null check and warning when loading local.properties

4428bef

Merge branch 'init-kmp' of https://github.com/powersync-ja/powersync-…

cfa009f

…kotlin into init-kmp

Manrich121 dismissed stevensJourney’s stale review via cfa009f February 29, 2024 10:23

Manrich121 added 3 commits February 29, 2024 13:39

review feedback: Refactor SupabaseConnector to `com.powersync.connect…

61e2196

…or.supabase` namespace and update demo apps accordingly

feature: add reporting of table updates as flow to PsInternalDatabase

1274e68

fix: PsSqlDriver.fireTableUpdates should copy pendingUpdates outside …

0127046

…of the coroutine

Manrich121 requested a review from rkistner February 29, 2024 15:09

rkistner reviewed Feb 29, 2024

View reviewed changes

CR feedback: remove sqlite_sequence from Powersync.sq and JWT decode …

7389d4c

…from PowerSyncCredentials

Manrich121 requested a review from rkistner March 4, 2024 08:10

rkistner approved these changes Mar 4, 2024

View reviewed changes

Manrich121 merged commit 6ef581a into main Mar 4, 2024
3 checks passed

Manrich121 deleted the init-kmp branch March 4, 2024 09:12

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Alpha] Kotlin Multiplatform SDK #1

[Alpha] Kotlin Multiplatform SDK #1

Manrich121 commented Jan 30, 2024 •

edited

Loading

rkistner Feb 27, 2024

Manrich121 Feb 29, 2024 •

edited

Loading

rkistner Feb 29, 2024

rkistner Feb 29, 2024

Manrich121 Mar 4, 2024

stevensJourney left a comment

This comment was marked as resolved.

rkistner Feb 29, 2024

[Alpha] Kotlin Multiplatform SDK #1

[Alpha] Kotlin Multiplatform SDK #1

Conversation

Manrich121 commented Jan 30, 2024 • edited Loading

PowerSync Kotlin SDK

Alpha version

Todo's

Future work/ideas

rkistner Feb 27, 2024

Choose a reason for hiding this comment

Manrich121 Feb 29, 2024 • edited Loading

Choose a reason for hiding this comment

rkistner Feb 29, 2024

Choose a reason for hiding this comment

rkistner Feb 29, 2024

Choose a reason for hiding this comment

Manrich121 Mar 4, 2024

Choose a reason for hiding this comment

stevensJourney left a comment

Choose a reason for hiding this comment

This comment was marked as resolved.

rkistner Feb 29, 2024

Choose a reason for hiding this comment

Manrich121 commented Jan 30, 2024 •

edited

Loading

Manrich121 Feb 29, 2024 •

edited

Loading