Update all dependencies#109
Open
krzema12 wants to merge 14 commits into
Open
Conversation
Collaborator
Author
|
White screen in running - TODO investigate and fix. |
- Add binaries.executable() to enable webpack distribution tasks - Set distribution outputDirectory to build/distributions - Add html-webpack-plugin npm dev dependency (5.6.3) - Create webpack.config.d/apply-html-webpack-plugin.js with custom template - Set duplicatesStrategy to INCLUDE on jsBrowserDistribution Sync task
Kotlin 2.3.21 requires JDK 17+. Switch all CI jobs from JDK 8 to JDK 17. Also remove redundant JDK 8 override in android job (was set after JDK 17).
- Fix HTTP server to serve JS with correct Content-Type header
- Fix main.kt FC pattern: use Player { ... } invoke operator instead of
Player.create { ... } which discarded the returned ReactElement
- Fix Player.kt infinite re-render: use this.synthesisParameters to
reference PlaybackCustomizationProps property instead of shadowed
Player state variable causing setState during render
- Remove ws.init() call - WaveSurfer.js v7 auto-initializes in constructor - Remove init() from WaveSurfer external type declarations - Fix copy task dependency to use jsBrowserDistribution instead of assemble - Use project-relative paths for copy task source directories - Restore --ignore-engines flag for KotlinNpmInstallTask
- Remove ws.init() call - WaveSurfer.js v7 auto-initializes in constructor - Remove init() from WaveSurfer external type declarations - Fix copy task dependency to use jsBrowserDistribution instead of assemble - Use buildDir File paths for copy tasks instead of layout.buildDirectory - Restore --ignore-engines flag for KotlinNpmInstallTask in root build.gradle.kts - Move worker/serviceworker sources from src/main/kotlin to src/jsMain/kotlin - Add binaries.executable() to worker/serviceworker build.gradle.kts
- Replace useEffectOnce + scopeRef with CoroutineScope(EmptyCoroutineContext).launch - Fix property name mangling mismatch between worker and main bundles by using JS object literals for postMessage - Restore service worker registration in main.kt
Make copyWorkerDistributionFiles and copyServiceWorkerDistributionFiles part of the :web:assemble task so worker.js and serviceworker.js are present in the distribution after a normal :web:build.
krzema12
force-pushed
the
update-all-dependencies
branch
from
fad9a92 to
16ae3c4
Compare
Collaborator
Author
|
Do not merge - multiple player components appear when clicking buttons1 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates all major dependencies to latest stable versions:
Also includes necessary migration changes:
kotlinOptions→compilerOptions,kotlin-android/kotlin-android-extensions→ AGP built-in Kotlin, JS modules →kotlin("multiplatform"),mainsource sets →jsMain, clikt import fix, generated source dir relocation to avoid Gradle 9.x shared-output validation.