Skip to content

build(deps): merge Pkl 0.32.2 preserving Elide engine support - #7

Merged
Sam Gammon (sgammon) merged 7 commits into
elide/0.32.xfrom
elide/local-0.32.2-security
Oct 27, 2026
Merged

Sam Gammon (sgammon) merged 7 commits into
elide/0.32.xfrom
elide/local-0.32.2-security

Conversation

@darvld

@darvld Dario Valdespino (darvld) commented Oct 24, 2026 •

Copy link
Copy Markdown
Member

Merge Pkl 0.32.2 into Elide's 0.32.x fork

Repository: elide-dev/pkl
Head: elide/local-0.32.2-security
Base: elide/0.32.x
Commit: f7682a9

Scope

  • Merge upstream Pkl 0.32.2 while retaining Elide's engine factory and lazy-engine initialization support.
  • Preserve org.pkl-lang.lazy-engine-init; a stock replacement can consume the Truffle preinitialization engine during class initialization.
  • The upstream fallback-CA correction is in pkl-commons-cli, not pkl-core. Do not describe this core-JAR update as fixing Elide's TLS trust issue.

Verification

  • Built pkl-core JAR and sources JAR with the fork's Gradle wrapper.
  • Elide PklEngineSharingTest: 5 passed.
  • Native Elide evaluated a computed Pkl manifest successfully.
  • Portable lazy-engine patch reverse-application check passed.
  • PR range: target is an ancestor; 6 commits ahead, zero target-only commits; diff whitespace check passed.

Integration

The runtime submission includes the rebuilt pkl-core.jar and the portable custom patch. No binary publication is required to review that runtime change.

Related runtime submission: https://github.com/elide-dev/runtime/pull/2659.

…e#1793)

The expression preamble length was subtracted from endColumn even
when endColumn came from the already-stripped sourceLine.length(),
so multi-line frames got a negative caret width and String.repeat
threw. Apply the offset only to the frame-derived columns.

---------

Co-authored-by: Dan Chao <dan.chao@apple.com>
@sgammon
Sam Gammon (sgammon) merged commit ddd67d0 into elide/0.32.x Oct 27, 2026
36 checks passed
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.

4 participants