Skip to content

Branch remote runner plugin fixe #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
proto-aiken-13 wants to merge 26 commits into
base: main
Choose a base branch
from
Draft

Branch remote runner plugin fixe #46

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
fddcd14
Setup of the remoteRunner plugin
proto-aiken-13 Jun 15, 2026
bfcea05
Update directory
proto-aiken-13 Jun 17, 2026
d0883bd
Modify the remoteRunnerPlugin structure and add ev3Engine usage
proto-aiken-13 Jun 22, 2026
04bb38f
fix: migrate remoteExecution to vitest, remove abstract, fix rollup c…
Akshay-2007-1 Jun 22, 2026
e9a67bd
chore: add py-slang and transitive deps to yarn.lock
Akshay-2007-1 Jun 22, 2026
806e6c2
chore: fix yarn.lock — merge @babel/runtime range, add @sourceacademy…
Akshay-2007-1 Jun 22, 2026
b713b50
chore: fix yarn.lock ordering and missing entries
Akshay-2007-1 Jun 22, 2026
75af89b
chore: fix extra blank lines in yarn.lock
Akshay-2007-1 Jun 22, 2026
3c206eb
chore: remove trailing blank line from yarn.lock
Akshay-2007-1 Jun 22, 2026
ec51966
fix: disable verbatimModuleSyntax for remoteExecution package
Akshay-2007-1 Jun 22, 2026
549c5b3
fix: set es2020 target and noImplicitAny:false for py-slang compatibi…
Akshay-2007-1 Jun 22, 2026
90026d3
fix: use noCheck:true to skip type-checking py-slang source
Akshay-2007-1 Jun 22, 2026
19fd917
fix: remove unnecessary re-export of remoteRunnerPlugin from runner-test
Akshay-2007-1 Jun 22, 2026
3d467f1
fix: resolve ESLint errors in remoteExecution plugin
Akshay-2007-1 Jun 22, 2026
de8bd92
fix: add .ts extensions to nodeResolve in remoteExecution rollup config
Akshay-2007-1 Jun 22, 2026
08b2202
fix: include py-slang source in TypeScript program for rollup bundling
Akshay-2007-1 Jun 22, 2026
00bce74
fix: add inline plugin to transpile node_modules TypeScript via trans…
Akshay-2007-1 Jun 22, 2026
a04d4be
fix: add @rollup/plugin-commonjs to handle moo/nearley CJS modules
Akshay-2007-1 Jun 22, 2026
2b1b555
fix: add @rollup/plugin-commonjs dep and update yarn.lock workspace e…
Akshay-2007-1 Jun 22, 2026
5a62899
fix: make py-slang external in bundle, inline it via vitest
Akshay-2007-1 Jun 22, 2026
575669d
style: fix Prettier formatting in remoteExecution rollup.config.mjs
Akshay-2007-1 Jun 22, 2026
f979647
style: drop arrow function parens per Prettier arrowParens:avoid setting
Akshay-2007-1 Jun 22, 2026
c318438
test: mock py-slang in runner test to avoid TypeScript source loading
Akshay-2007-1 Jun 22, 2026
572ea84
fix: use Promise.resolve instead of async to satisfy require-await li…
Akshay-2007-1 Jun 22, 2026
5a257b0
feat(ev3): add conductor-based EV3 evaluator bundle
proto-aiken-13 Jun 29, 2026
764ee5b
Merge branch 'main' into branch-remoteRunnerPlugin-fixed
proto-aiken-13 Jun 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 20 additions & 17 deletions src/runner/remoteExecution/rollup.config.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,21 +2,24 @@ import nodeResolve from "@rollup/plugin-node-resolve";
import terser from "@rollup/plugin-terser";
import typescript from "@rollup/plugin-typescript";

/**
* @type {import('rollup').RollupOptions}
*/
export default {
input: "src/index.ts",
external: id => id.includes("py-slang"),
output: [
{
file: "dist/index.cjs",
format: "cjs",
export default [
// existing build
{
input: "src/index.ts",
external: id => id.includes("py-slang"),
output: [
{ file: "dist/index.cjs", format: "cjs" },
{ file: "dist/index.mjs", format: "esm" },
],
plugins: [nodeResolve(), typescript(), terser()],
},
// new EV3 conductor worker bundle
{
input: "src/entry.ts",

@gemini-code-assist gemini-code-assist Bot Jun 29, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

The file src/entry.ts specified as the input for the EV3 conductor worker bundle does not exist in the repository. This will cause the Rollup build to fail. Please correct the input path or add the missing src/entry.ts file.

output: {
file: "dist/ev3-pyslang.js",
format: "iife", // self-contained for Web Worker
},
{
file: "dist/index.mjs",
format: "esm",
},
],
plugins: [nodeResolve(), typescript(), terser()],
};
plugins: [nodeResolve(), typescript(), terser()],
},
];
Empty file added src/runner/remoteExecution/src/Ev3Evaluator.ts
View file
Open in desktop
Empty file.
Empty file added src/runner/remoteExecution/src/entry.ts
View file
Open in desktop
Empty file.
17 changes: 17 additions & 0 deletions yarn.lock
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5987,6 +5987,23 @@ __metadata:
languageName: node
linkType: hard

"railroad-diagrams@npm:^1.0.0":
version: 1.0.0
resolution: "railroad-diagrams@npm:1.0.0"
checksum: 10c0/81bf8f86870a69fb9ed243102db9ad6416d09c4cb83964490d44717690e07dd982f671503236a1f8af28f4cb79d5d7a87613930f10ac08defa845ceb6764e364
languageName: node
linkType: hard

"randexp@npm:0.4.6":
version: 0.4.6
resolution: "randexp@npm:0.4.6"
dependencies:
discontinuous-range: "npm:1.0.0"
ret: "npm:~0.1.10"
checksum: 10c0/14ee14b6d7f5ce69609b51cc914fb7a7c82ad337820a141c5f762c5ad1fe868f5191ea6e82359aee019b625ee1359486628fa833909d12c3b5dd9571908c3345
languageName: node
linkType: hard

"read-yaml-file@npm:^1.1.0":
version: 1.1.0
resolution: "read-yaml-file@npm:1.1.0"
Expand Down
Loading