Skip to content

feat: add SQLiteMC encryption for JVM#640

Closed
softartdev wants to merge 3 commits into
devfrom
codex/implement-jvm-encryption-with-sqlite-mc
Closed

feat: add SQLiteMC encryption for JVM#640
softartdev wants to merge 3 commits into
devfrom
codex/implement-jvm-encryption-with-sqlite-mc

Conversation

@softartdev
Copy link
Copy Markdown
Owner

Summary

  • use sqlite-mc driver for encrypted SQLite on JVM
  • enable previously ignored desktop UI tests
  • configure build to use sqlite-mc Gradle plugin

Testing

  • ./gradlew --no-daemon :desktop-compose-app:cleanJvmTest :desktop-compose-app:jvmTest --tests "com.softartdev.notedelight.ui.DesktopUiTests" (fails: Unknown function last_insert_rowid)

https://chatgpt.com/codex/tasks/task_e_68c4fe27acac83309a1c68a99167e2b0

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment on lines 55 to 58
jvmMain.dependencies {
implementation(libs.sqlDelight.jvm)
implementation(libs.appdirs)
implementation(libs.napier)
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P1] Declare SQLiteMC driver dependency for JVM sources

The Kotlin classes now import io.toxicity.sqlite.mc.driver.SQLiteMCDriver and related configuration, but the jvmMain source set no longer pulls in any driver dependency—implementation(libs.sqlDelight.jvm) was removed and nothing replaced it. Without a runtime such as io.toxicity.sqlite-mc:sqlite-driver the JVM module will not compile (unresolved reference) or will crash at runtime with ClassNotFoundException. Add the SQLiteMC driver dependency to the jvmMain (and corresponding test) dependencies to match the new imports.

Useful? React with 👍 / 👎.

This commit removes the `lastInsertRowId` query from the `Note.sq` SQLDelight file.
This commit replaces the SQLDelight database library with SQLiteMC.

The changes include:
- Updating plugin aliases in `build.gradle.kts` and `core/data/db-sqldelight/build.gradle.kts`.
- Replacing SQLDelight dependencies with SQLiteMC dependencies in `gradle/libs.versions.toml`, `android-compose-app/build.gradle.kts`, and `core/data/db-sqldelight/build.gradle.kts`.
- Updating the version catalog (`gradle/libs.versions.toml`) to reflect the new SQLiteMC version and remove the old SQLDelight version.
- Commenting out SQLDelight specific dependencies and configurations.
@softartdev softartdev closed this Sep 14, 2025
@softartdev softartdev deleted the codex/implement-jvm-encryption-with-sqlite-mc branch September 14, 2025 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant