@freestyle-sh/with-python 0.2.0 → 0.2.2

package/dist/index.d.ts

@@ -1,16 +1,19 @@
-import { VmWith, VmWithInstance, CreateVmOptions } from 'freestyle-sandboxes';
+import { VmWith, VmWithInstance, VmSpec } from 'freestyle-sandboxes';
 import { VmRunCodeInstance, JSONValue, RunCodeResponse, VmRunCode } from '@freestyle-sh/with-type-run-code';
 
 declare class VmPython extends VmWith<PythonRuntimeInstance> implements VmRunCode<VmRunCodeInstance> {
     constructor();
-    configure(existingConfig: CreateVmOptions): CreateVmOptions | Promise<CreateVmOptions>;
+    configureSnapshotSpec(spec: VmSpec): VmSpec;
     createInstance(): PythonRuntimeInstance;
 }
 declare class PythonRuntimeInstance extends VmWithInstance implements VmRunCodeInstance {
     builder: VmPython;
     constructor(builder: VmPython);
-    runCode<Result extends JSONValue = any>({ code, }: {
+    runCode<Result extends JSONValue = any>(args: string | {
         code: string;
+        argv?: string[];
+        env?: Record<string, string>;
+        workdir?: string;
     }): Promise<RunCodeResponse<Result>>;
 }
 

package/dist/index.js

@@ -1,16 +1,16 @@
-import { VmWith, VmTemplate, VmWithInstance } from 'freestyle-sandboxes';
+import { VmWith, VmSpec, VmWithInstance } from 'freestyle-sandboxes';
 
 class VmPython extends VmWith {
   constructor() {
     super();
   }
-  configure(existingConfig) {
-    const pythonConfig = {
-      template: new VmTemplate({
+  configureSnapshotSpec(spec) {
+    return this.composeSpecs(
+      spec,
+      new VmSpec({
         aptDeps: ["python3"]
       })
-    };
-    return this.compose(existingConfig, pythonConfig);
+    );
   }
   createInstance() {
     return new PythonRuntimeInstance(this);
@@ -22,19 +22,28 @@ class PythonRuntimeInstance extends VmWithInstance {
     super();
     this.builder = builder;
   }
-  async runCode({
-    code
-  }) {
+  async runCode(args) {
+    const options = typeof args === "string" ? { code: args } : args;
+    const { code, argv, env, workdir } = options;
+    const shellEscape = (value) => `'${value.replace(/'/g, "'\\''")}'`;
+    const argvArgs = argv?.map(shellEscape).join(" ");
+    const envPrefix = env ? `${Object.entries(env).map(([key, value]) => `${key}=${shellEscape(value)}`).join(" ")} ` : "";
+    const cdPrefix = workdir ? `cd ${shellEscape(workdir)} && ` : "";
+    const command = `${cdPrefix}${envPrefix}python3 -c "${code.replace(/"/g, '\\"')}"${argvArgs ? ` -- ${argvArgs}` : ""}`;
     const result = await this.vm.exec({
-      command: `python3 -c "${code.replace(/"/g, '\\"')}"`
+      command
     });
     let parsedResult = void 0;
     if (result.stdout) {
-      try {
-        parsedResult = JSON.parse(result.stdout);
-      } catch (e) {
-        if (result.stderr) {
-          `Failed to parse JSON output. Stderr: ${result.stderr}`;
+      const lines = result.stdout.split(/\r?\n/).map((line) => line.trim()).filter((line) => line.length > 0);
+      const lastLine = lines[lines.length - 1];
+      if (lastLine) {
+        try {
+          parsedResult = JSON.parse(lastLine);
+        } catch (e) {
+          if (result.stderr) {
+            `Failed to parse JSON output. Stderr: ${result.stderr}`;
+          }
         }
       }
     }

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@freestyle-sh/with-python",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "private": false,
   "dependencies": {
-    "freestyle-sandboxes": "^0.1.2",
-    "@freestyle-sh/with-type-run-code": "^0.2.0"
+    "freestyle-sandboxes": "^0.1.8",
+    "@freestyle-sh/with-type-run-code": "^0.2.2"
   },
   "type": "module",
   "main": "./dist/index.js",