Skip to content

Replace :core:proto with org.meshtastic:protobufs:2.7.25-SNAPSHOT#5675

Draft
jamesarich wants to merge 1 commit into
mainfrom
jamesarich/migrate-to-protobufs-sdk
Draft

Replace :core:proto with org.meshtastic:protobufs:2.7.25-SNAPSHOT#5675
jamesarich wants to merge 1 commit into
mainfrom
jamesarich/migrate-to-protobufs-sdk

Conversation

@jamesarich
Copy link
Copy Markdown
Collaborator

The meshtastic/protobufs KMP SDK PR landed, so we can now consume the Wire-generated models as a regular Maven dependency instead of running local codegen from a git submodule.

What this does

Replaces the :core:proto module (local Wire codegen + git submodule pointing at meshtastic/protobufs) with the published org.meshtastic:protobufs:2.7.25-SNAPSHOT artifact. This eliminates:

  • The git submodule (core/proto/src/main/proto)
  • The Wire Gradle plugin dependency
  • All local .proto -> Kotlin code generation
  • The wire-runtime catalog entry (now transitive from the SDK)

Approach

  • All 17 modules that depended on projects.core.proto now use libs.meshtastic.protobufs
  • The TAKPacket-SDK api() export (with zstd-jni/xpp3 exclusions) moves to :core:model, which already served as the primary transitive provider of proto types
  • The Sonatype snapshots repository was already configured in settings.gradle.kts
  • kmpSmokeCompile passes across JVM, Android, and iOS targets

Notes

  • This depends on snapshot 2.7.25-SNAPSHOT being available on https://central.sonatype.com/repository/maven-snapshots/
  • Once the SDK cuts a stable release, we should pin to that version
  • The SDK uses the same Wire options (boxOneOfsMinSize = 5000, makeImmutableCopies = false) so generated types are ABI-compatible

Drop the local Wire codegen module in favor of the published KMP
protobufs SDK from meshtastic/protobufs. This removes:

- The git submodule (core/proto/src/main/proto)
- The Wire plugin dependency
- All local .proto → Kotlin code generation

The TAKPacket-SDK api() export moves to :core:model which already
served as the primary transitive provider of proto types.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Build system changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant