scala/scala3

Scala 3 ScriptEngine not supporting bindings or attributes

Open

#14,262 opened on Jan 12, 2022

View on GitHub
 (1 comment) (1 reaction) (0 assignees)Scala (6,247 stars) (1,159 forks)batch import
area:replhelp wanteditype:enhancement

Description

This hello world example works fine in Scala 2: https://github.com/invesdwin/invesdwin-context/tree/master/invesdwin-context-parent/invesdwin-context-scala/src/test/java/de/invesdwin/context/scala/tests/hello Which internally uses this wrapper for the ScriptEngine: https://github.com/invesdwin/invesdwin-context/blob/master/invesdwin-context-parent/invesdwin-context-scala/src/main/java/de/invesdwin/context/scala/pool/WrappedScalaScriptEngine.java

But it seems to fail in Scala3 because its ScriptEngine does not use the bindings or context attributes: https://github.com/lampepfl/dotty/blob/8e1054e9fcc91cba8fc3910051377b4a019b9589/compiler/src/dotty/tools/repl/ScriptEngine.scala

Contributor guide

Tech stack
scala
Domain
api
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Familiarity with Scala 2 vs Scala 3Understanding of Java ScriptEngine API
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
Examine the linked Scala 2 wrapper (WrappedScalaScriptEngine.java) and the current Scala 3 ScriptEngine implementation in dotty/tools/repl/ScriptEngine.scala. Identify where bindings and context attributes are ignored in the Scala 3 version and adapt the Scala 2 approach to add support.