Skip to content

Conversation

@devcrocod
Copy link
Contributor

workaround

Motivation and Context

How Has This Been Tested?

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

Copy link
Contributor

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 adds a workaround for WASM server support by including the kotlinx-coroutines-core-wasm dependency specifically for the wasmJsMain source set. This addresses compatibility issues when running coroutines in WASM JavaScript environments.

  • Adds kotlinx-coroutines-core-wasm-js dependency to the version catalog
  • Configures wasmJsMain source set to use the WASM-specific coroutines dependency

Reviewed Changes

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

File Description
gradle/libs.versions.toml Adds kotlinx-coroutines-core-wasm library definition using existing coroutines version
kotlin-sdk-core/build.gradle.kts Configures wasmJsMain dependencies to include the WASM coroutines workaround

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

@devcrocod devcrocod force-pushed the devcrocod/wasm-workaround branch from bf7fc2f to c749649 Compare September 11, 2025 14:02
Copy link
Contributor

@kpavlov kpavlov left a comment

Choose a reason for hiding this comment

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

Thank you, @devcrocod
This should fix missing dependency

@devcrocod devcrocod force-pushed the devcrocod/wasm-workaround branch from c749649 to 3baa696 Compare September 11, 2025 15:17
@kpavlov kpavlov force-pushed the devcrocod/wasm-workaround branch from 3baa696 to ad0cf2e Compare September 11, 2025 16:38
@devcrocod devcrocod merged commit 2137f6a into main Sep 11, 2025
4 checks passed
@devcrocod devcrocod deleted the devcrocod/wasm-workaround branch September 11, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants