fix: builder and installer bugs pt2 #1695

sgammon · 2025-10-10T18:02:35Z

Summary

Fixes a whole bunch of bugs related to Elide's builder and installer, especially with regard to Kotlin and Java.

New feature: Build Flags
New feature: Build State
New feature: Minimum Elide version enforcement
Fixes: Elide injects -H:+UnlockExperimentalVMOptions improperly #1692
Fixes: Native image reports are broken #1691
Fixes: Can't use -include-runtime with elide kotlinc #1688
Fixes: Can't use -Werror from kotlinc #1686

codecov · 2025-10-10T21:13:47Z

Codecov Report

❌ Patch coverage is 3.60360% with 749 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.94%. Comparing base (e2e3abc) to head (5707092).
⚠️ Report is 33 commits behind head on main.

Files with missing lines	Patch %	Lines
.../elide/tooling/jvm/resolver/MavenAetherResolver.kt	0.00%	111 Missing ⚠️
...ooling/project/codecs/ElidePackageManifestCodec.kt	0.00%	91 Missing ⚠️
...n/kotlin/elide/tooling/jvm/JvmBuildConfigurator.kt	0.00%	84 Missing ⚠️
...ng/gvm/nativeImage/NativeImageBuildConfigurator.kt	0.00%	83 Missing ⚠️
...e/tooling/project/manifest/ElidePackageManifest.kt	8.95%	61 Missing ⚠️
...n/kotlin/elide/runtime/version/ElideVersion.jvm.kt	0.00%	46 Missing ⚠️
...n/elide/runtime/gvm/kotlin/KotlinCompilerConfig.kt	8.88%	40 Missing and 1 partial ⚠️
...ide/tooling/project/flags/ProjectFlagDefinition.kt	0.00%	33 Missing ⚠️
...n/elide/tool/cli/cmd/project/ToolProjectCommand.kt	0.00%	22 Missing ⚠️
...tooling/src/main/kotlin/elide/tooling/MultiPath.kt	0.00%	18 Missing ⚠️
... and 22 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1695      +/-   ##
==========================================
- Coverage   40.57%   39.94%   -0.64%     
==========================================
  Files         867      874       +7     
  Lines       40059    40743     +684     
  Branches     5697     5797     +100     
==========================================
+ Hits        16255    16275      +20     
- Misses      21815    22478     +663     
- Partials     1989     1990       +1

Flag	Coverage Δ
jvm	`39.94% <3.60%> (-0.64%)`	⬇️
lib	`39.94% <3.60%> (-0.64%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...main/kotlin/elide/tooling/kotlin/KotlinCompiler.kt	`0.00% <ø> (ø)`
.../main/kotlin/elide/tool/cli/cmd/deps/AddCommand.kt	`50.00% <ø> (ø)`
...n/kotlin/elide/tool/cli/cmd/deps/InstallCommand.kt	`4.71% <ø> (ø)`
...c/main/kotlin/elide/tool/cli/cmd/dev/LspCommand.kt	`1.54% <ø> (ø)`
...c/main/kotlin/elide/tool/cli/cmd/dev/McpCommand.kt	`12.00% <ø> (ø)`
.../kotlin/elide/tool/cli/cmd/info/ToolInfoCommand.kt	`72.85% <ø> (ø)`
...elide/tool/cli/cmd/project/ProjectAdviceCommand.kt	`8.33% <ø> (ø)`
...kotlin/elide/tool/cli/cmd/repl/ToolShellCommand.kt	`31.04% <ø> (ø)`
...otlin/elide/tool/cli/cmd/runner/ToolTestCommand.kt	`0.00% <ø> (ø)`
...n/elide/tool/cli/cmd/secrets/ToolSecretsCommand.kt	`1.35% <ø> (ø)`
... and 40 more

... and 1 file with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2e3abc...5707092. Read the comment docs.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot

Pull Request Overview

This pull request implements a comprehensive set of builder and installer improvements for Elide, focusing on Kotlin/Java tooling enhancements and project flag support. The changes add build state management, toolchain configuration, minimum version enforcement, and extensive new features for Maven dependency handling.

Key changes:

New Build Features: Adds project flags, build state management, and toolchain settings
Enhanced Maven Support: Implements modules, dev dependencies, compile-only dependencies, runtime-only dependencies, and exclusions
Version Management: Adds minimum Elide version enforcement and version comparison utilities
API Updates: Updates all manifest codecs to accept build state parameters for context-aware parsing

Reviewed Changes

Copilot reviewed 78 out of 79 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
packages/tooling/src/main/pkl/*.pkl	New Pkl modules for toolchain, state, flags, and builder configuration
packages/tooling/src/main/kotlin/elide/tooling/project/flags/*.kt	Complete project flags system implementation
packages/tooling/src/main/kotlin/elide/tooling/project/codecs/*.kt	Updated all manifest codecs to accept build state context
packages/base/src/commonMain/kotlin/elide/runtime/version/*.kt	New version management and comparison system
packages/cli/src/main/kotlin/elide/tool/cli/cmd/project/ToolProjectCommand.kt	Enhanced project info display with flags rendering
packages/builder/src/main/kotlin/elide/tooling/jvm/resolver/MavenAetherResolver.kt	Extended Maven resolver with new dependency types and local JARs

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

packages/tooling/src/main/kotlin/elide/tooling/project/manifest/ElidePackageManifest.kt

packages/cli/src/main/kotlin/elide/tool/cli/cmd/project/ToolProjectCommand.kt

packages/tooling/src/main/kotlin/elide/tooling/project/manifest/ElidePackageManifest.kt

packages/tooling/src/main/kotlin/elide/tooling/project/PackageManifestService.kt

Signed-off-by: Sam Gammon <sam@elide.dev>

feat(tooling): add `includeRuntime` / `noStdlib` support for kotlin feat(tooling): add `reflection` control for kotlin feat(tooling): add initial `automodules` and jpms support infra feat(tooling): add `devPackages` support to `maven` feat(tooling): add `modules` support to `maven` feat(tooling): add `compileOnly` support to `maven` feat(tooling): add `runtimeOnly` support to `maven` feat(tooling): add support for global `exclusions` in maven block feat(tooling): add initial native image support for `exclusions` feat(tooling): add native image `defs` block feat(tooling): add native image support for `ldflags` and `cflags` blocks feat(tooling): add initial support for dynamic build state in `elide.pkl` fix(builder): exclude graalvm jars from native-image classpath fix(builder): don't include optimization settings when pgo is active fix(builder): runtime-native exports for graal reporter module fix(builder): relock experimental native image hosted options fix(builder): mounted codegen and main inputs for kotlinc and javac fix(builder): wrong count of java/kotlin sources in build messages fix(builder): kotlinc isn't receiving `-jvm-target` flag fix(cli): ship svm macros with final dist fix(cli): missing native image reflective types for kotlinc fix: incorrect snakeyaml dependency Fixes: #1692 Fixes: #1691 Fixes: #1688 Signed-off-by: Sam Gammon <sam@elide.dev>

Signed-off-by: Sam Gammon <sam@elide.dev>

… minimum Signed-off-by: Sam Gammon <sam@elide.dev>

Signed-off-by: Sam Gammon <sam@elide.dev>

sgammon added this to the Release R18: Beta milestone Oct 10, 2025

sgammon self-assigned this Oct 10, 2025

sgammon added bug Something isn't working lang:java Issues relating to Java language support lang:kotlin Related to Kotlin lang support tools:builder Stuff relating to Elide's builder labels Oct 10, 2025

sgammon added this to Elide Oct 10, 2025

sgammon added tools:install Things related to Elide's dependency installer module:tooling Changes and issues relating to the Elide `tooling` module labels Oct 10, 2025

sgammon moved this to In Progress in Elide Oct 10, 2025

sgammon force-pushed the fix/kotlinc-issues-beta9-pt2 branch from 2cb5918 to bf3a49d Compare October 10, 2025 20:51

sgammon force-pushed the fix/kotlinc-issues-beta9-pt2 branch from abfb74e to 73fd45c Compare October 13, 2025 21:29

sgammon marked this pull request as ready for review October 14, 2025 19:26

sgammon requested review from a team, Copilot and melodicore October 15, 2025 00:50

Copilot AI reviewed Oct 15, 2025

View reviewed changes

sgammon added 12 commits October 17, 2025 10:05

chore: issue 1.0.0-beta10-rc6

49d5341

Signed-off-by: Sam Gammon <sam@elide.dev>

fix(tooling): opt level s

38c279d

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: issue 1.0.0-beta10-rc7

d073298

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: reformat reachability metadata

7992bc9

Signed-off-by: Sam Gammon <sam@elide.dev>

fix(graalvm-kt): invoke kotlinc instead of javac with kt sources present

15daad5

Signed-off-by: Sam Gammon <sam@elide.dev>

fix(builder): never include poms on kotlinc classpath

5998471

Signed-off-by: Sam Gammon <sam@elide.dev>

feat(builder): local jar overrides

c717b6b

Signed-off-by: Sam Gammon <sam@elide.dev>

fix(tooling): kotlin opt-in flags

4dcbb5f

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: update reachability metadata

1b3dc3e

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: update api pins

48783d3

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: issue 1.0.0-beta10-rc8

70afb68

Signed-off-by: Sam Gammon <sam@elide.dev>

sgammon added 20 commits October 17, 2025 10:05

fix(secrets): add state param for manifest render

9f7a354

Signed-off-by: Sam Gammon <sam@elide.dev>

feat(tooling): initial support for project flags

cb80c0c

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: update reachability metadata

f67ee38

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: fmt

8c9f615

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: update api pins

a93e095

Signed-off-by: Sam Gammon <sam@elide.dev>

feat(base): add static elide version symbol

9f9af46

Signed-off-by: Sam Gammon <sam@elide.dev>

feat(tooling): engines block in toolchains; enforce elide version…

6ae4f60

… minimum Signed-off-by: Sam Gammon <sam@elide.dev>

feat(tooling): shorthand for maven dependency with custom repo

331b30d

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: update reachability metadata

fade0e8

Signed-off-by: Sam Gammon <sam@elide.dev>

chore(cli): prettier manifest enforcement errors

7f644e0

Signed-off-by: Sam Gammon <sam@elide.dev>

fix: exit handling for invalid manifest

b08ef00

Signed-off-by: Sam Gammon <sam@elide.dev>

fix: cosmetic/nit fixes

6305c3e

Signed-off-by: Sam Gammon <sam@elide.dev>

feat: gvm features in elide native image builder

f730261

Signed-off-by: Sam Gammon <sam@elide.dev>

fix: resolver bug with injected jars

8dd1159

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: issue 1.0.0-beta10-rc9

ca56132

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: kapt deps

e94c76e

Signed-off-by: Sam Gammon <sam@elide.dev>

feat(builder): don't fetch global exclusions

fc4d6fe

Signed-off-by: Sam Gammon <sam@elide.dev>

feat(tooling): early support wiring for ksp/kapt

7568194

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: experimental flags for svm wasm/obfuscation

a42c14b

Signed-off-by: Sam Gammon <sam@elide.dev>

chore: update api pins

c1845f6

Signed-off-by: Sam Gammon <sam@elide.dev>

sgammon force-pushed the fix/kotlinc-issues-beta9-pt2 branch from 2df7b12 to c1845f6 Compare October 17, 2025 17:09

fix: native target family should be native

5707092

Signed-off-by: Sam Gammon <sam@elide.dev>

darvld approved these changes Oct 17, 2025

View reviewed changes

sgammon merged commit ed7eaf7 into main Oct 18, 2025
18 of 19 checks passed

github-project-automation bot moved this from In Progress to Done in Elide Oct 18, 2025

sgammon mentioned this pull request Oct 30, 2025

release: 1.0.0-beta10 #1701

Merged

32 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: builder and installer bugs pt2 #1695

fix: builder and installer bugs pt2 #1695

Uh oh!

sgammon commented Oct 10, 2025 •

edited

Loading

Uh oh!

codecov bot commented Oct 10, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

fix: builder and installer bugs pt2 #1695

fix: builder and installer bugs pt2 #1695

Uh oh!

Conversation

sgammon commented Oct 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

codecov bot commented Oct 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

sgammon commented Oct 10, 2025 •

edited

Loading

codecov bot commented Oct 10, 2025 •

edited

Loading