Skip to content

Conversation

@utsmannn
Copy link
Owner

@utsmannn utsmannn commented Oct 5, 2025

Summary

This PR resolves multiple build errors related to Kotlin version compatibility and deprecated Gradle APIs.

Changes

1. Kotlin Version Compatibility

  • Fixed kotlin-stdlib version conflict by downgrading dependencies:
    • kotlinx-serialization: 1.9.0 → 1.7.3
    • kotlinx-datetime: 0.7.1 → 0.6.1
    • Added resolution strategy in root build.gradle.kts to force kotlin-stdlib 2.1.0
    • Resolves KLIB ABI incompatibility errors with Kotlin 2.1.0 compiler

2. Deprecated API Removals

  • Removed deprecated outputModuleName from wasmJs configuration
    • Replaced deprecated buildDir with layout.buildDirectory.get().asFile

3. CocoaPods Configuration Fix

  • Added source 'https://cdn.cocoapods.org' to Firebase sample Podfile
    • Added missing FirebaseCoreInternal pod dependency

Testing

  • ✅ Build successful on main module
    • ✅ CocoaPods installation successful
    • ✅ All compilation tasks pass
      🤖 Generated with Claude Code

…cated APIs

- Fix kotlin-stdlib version conflict by downgrading dependencies:
  - kotlinx-serialization: 1.9.0 → 1.7.3
  - kotlinx-datetime: 0.7.1 → 0.6.1
  - Add resolution strategy to force kotlin-stdlib 2.1.0

- Remove deprecated outputModuleName from wasmJs configuration in:
  - jsonBuilderWeb/build.gradle.kts
  - samples/sharedCompose/build.gradle.kts
  - samples/routerApp/composeApp/build.gradle.kts

- Replace deprecated buildDir with layout.buildDirectory.get().asFile in:
  - jsonBuilderWeb/build.gradle.kts
  - samples/routerApp/composeApp/build.gradle.kts

- Fix CocoaPods configuration for Firebase sample:
  - Add source 'https://cdn.cocoapods.org' to Podfile
  - Add missing FirebaseCoreInternal pod dependency

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@utsmannn utsmannn requested a review from Copilot October 5, 2025 14:58
@utsmannn utsmannn added the enhancement New feature or request label Oct 5, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes build errors related to Kotlin version compatibility and deprecated Gradle APIs. The changes ensure compatibility with Kotlin 2.1.0 by addressing KLIB ABI incompatibility issues and removing deprecated API usage.

  • Removed deprecated moduleName configuration from wasmJs targets
  • Replaced deprecated buildDir with layout.buildDirectory.get().asFile
  • Updated CocoaPods configuration with source repository and missing dependency

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
samples/sharedCompose/build.gradle.kts Removed deprecated moduleName from wasmJs configuration
samples/routerApp/composeApp/build.gradle.kts Removed deprecated moduleName and replaced buildDir with modern API
samples/firebaseApp/iosApp/Podfile Added CocoaPods source repository and missing FirebaseCoreInternal dependency
samples/firebaseApp/composeApp/build.gradle.kts Added explicit project.dependencies.platform() call for Firebase BOM
jsonBuilderWeb/build.gradle.kts Removed deprecated moduleName and replaced buildDir with modern API
gradle/libs.versions.toml Updated library versions for compatibility

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

utsmannn and others added 2 commits October 5, 2025 22:11
Update compileSdk from 35 to 36 across all Android modules to meet
the minimum requirement for androidx.activity:activity-compose:1.11.0.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Upgrade Gradle wrapper from 8.12 to 8.13
- Add compose.materialIconsExtended dependency to routerApp to fix unresolved icons
- Update Podfile.lock checksums after Podfile changes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@utsmannn utsmannn merged commit 9977c5b into main Oct 5, 2025
1 check passed
@utsmannn utsmannn deleted the fix/build-errors-kotlin-version branch October 5, 2025 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants