.github/ISSUE_TEMPLATE/language-request.md

@@ -4,6 +4,7 @@ about: Template for requesting language support
 title: Add [insert language name here]
 labels: package
 assignees: ''
+
 ---
 
 Provide links to different compilers/interpreters that could be used to implement this language, and discuss pros/cons of each.

.github/PULL_REQUEST_TEMPLATE/package.md

@@ -1,11 +1,10 @@
 Checklist:
-
+* [ ] The package builds locally with `./piston build-pkg [package] [version]`
--   [ ] The package builds locally with `./piston build-pkg [package] [version]`
+* [ ] The package installs with `./piston ppman install [package]=[version]`
--   [ ] The package installs with `./piston ppman install [package]=[version]`
+* [ ] The package runs the test code with `./piston run [package] -l [version] packages/[package]/[version]/test.*`
--   [ ] The package runs the test code with `./piston run [package] -l [version] packages/[package]/[version]/test.*`
+* [ ] Package files are placed in the correct directory
--   [ ] Package files are placed in the correct directory
+* [ ] No old package versions are removed
--   [ ] No old package versions are removed
+* [ ] All source files are deleted in the `build.sh` script
--   [ ] All source files are deleted in the `build.sh` script
+* [ ] `metadata.json`'s `language` and `version` fields match the directory path
--   [ ] `metadata.json`'s `language` and `version` fields match the directory path
+* [ ] Any extensions the language may use are set as aliases
--   [ ] Any extensions the language may use are set as aliases
+* [ ] Any alternative names the language is referred to are set as aliases.
--   [ ] Any alternative names the language is referred to are set as aliases.

.github/workflows/api-push.yaml

@@ -7,6 +7,7 @@ on:
     paths:
       - api/**
 
+  
 jobs:
   push_to_registry:
     runs-on: ubuntu-latest

.github/workflows/package-pr.yaml

@@ -1,4 +1,4 @@
-name: 'Package Pull Requests'
+name: "Package Pull Requests"
 
 on:
   pull_request:
@@ -8,7 +8,7 @@ on:
       - reopened
       - synchronize
     paths:
-            - 'packages/**'
+      - "packages/**"
 
 jobs:
   check-pkg:

.github/workflows/package-push.yaml

@@ -8,6 +8,7 @@ on:
     paths:
       - packages/**
 
+
 jobs:
   build-pkg:
     name: Build package
@@ -50,9 +51,9 @@ jobs:
     runs-on: ubuntu-latest
     needs: build-pkg
     steps:
-            - name: 'Download all release assets'
+      - name: "Download all release assets"
         run: curl -s https://api.github.com/repos/engineer-man/piston/releases/latest | jq '.assets[].browser_download_url' -r | xargs -L 1 curl -sLO
-            - name: 'Generate index file'
+      - name: "Generate index file"
         run: |
           echo "" > index
           BASEURL=https://github.com/engineer-man/piston/releases/download/pkgs/

.gitignore

@@ -1,3 +1,2 @@
 data/
 .piston_env
-node_modules

.prettierignore

@@ -1,12 +0,0 @@
-node_modules
-data/
-api/_piston
-repo/build
-packages/*/*/*
-packages/*.pkg.tar.gz
-!packages/*/*/metadata.json
-!packages/*/*/build.sh
-!packages/*/*/environment
-!packages/*/*/run
-!packages/*/*/compile
-!packages/*/*/test.*

api/.gitignore

@@ -1 +1,2 @@
+node_modules
 _piston

api/.prettierignore

@@ -0,0 +1 @@
+node_modules

api/package-lock.json

@@ -1,12 +1,12 @@
 {
     "name": "piston-api",
-    "version": "3.1.0",
+    "version": "3.0.0",
     "lockfileVersion": 2,
     "requires": true,
     "packages": {
         "": {
             "name": "piston-api",
-            "version": "3.1.0",
+            "version": "3.0.0",
             "license": "MIT",
             "dependencies": {
                 "body-parser": "^1.19.0",
@@ -20,6 +20,9 @@
                 "semver": "^7.3.4",
                 "uuid": "^8.3.2",
                 "waitpid": "git+https://github.com/HexF/node-waitpid.git"
+            },
+            "devDependencies": {
+                "prettier": "2.2.1"
             }
         },
         "node_modules/accepts": {
@@ -406,6 +409,18 @@
             "resolved": "https://registry.npmjs.org/path-to-regexp/-/path-to-regexp-0.1.7.tgz",
             "integrity": "sha1-32BBeABfUi8V60SQ5yR6G/qmf4w="
         },
+        "node_modules/prettier": {
+            "version": "2.2.1",
+            "resolved": "https://registry.npmjs.org/prettier/-/prettier-2.2.1.tgz",
+            "integrity": "sha512-PqyhM2yCjg/oKkFPtTGUojv7gnZAoG80ttl45O6x2Ug/rMJw4wcc9k6aaf2hibP7BGVCCM33gZoGjyvt9mm16Q==",
+            "dev": true,
+            "bin": {
+                "prettier": "bin-prettier.js"
+            },
+            "engines": {
+                "node": ">=10.13.0"
+            }
+        },
         "node_modules/proxy-addr": {
             "version": "2.0.6",
             "resolved": "https://registry.npmjs.org/proxy-addr/-/proxy-addr-2.0.6.tgz",
@@ -580,8 +595,7 @@
             }
         },
         "node_modules/waitpid": {
-            "resolved": "git+ssh://git@github.com/HexF/node-waitpid.git#a08d116a5d993a747624fe72ff890167be8c34aa",
+            "resolved": "git+ssh://git@github.com/HexF/node-waitpid.git#a08d116a5d993a747624fe72ff890167be8c34aa"
-            "hasInstallScript": true
         },
         "node_modules/ws": {
             "version": "7.5.3",
@@ -899,6 +913,12 @@
             "resolved": "https://registry.npmjs.org/path-to-regexp/-/path-to-regexp-0.1.7.tgz",
             "integrity": "sha1-32BBeABfUi8V60SQ5yR6G/qmf4w="
         },
+        "prettier": {
+            "version": "2.2.1",
+            "resolved": "https://registry.npmjs.org/prettier/-/prettier-2.2.1.tgz",
+            "integrity": "sha512-PqyhM2yCjg/oKkFPtTGUojv7gnZAoG80ttl45O6x2Ug/rMJw4wcc9k6aaf2hibP7BGVCCM33gZoGjyvt9mm16Q==",
+            "dev": true
+        },
         "proxy-addr": {
             "version": "2.0.6",
             "resolved": "https://registry.npmjs.org/proxy-addr/-/proxy-addr-2.0.6.tgz",

api/package.json

@@ -16,5 +16,11 @@
         "uuid": "^8.3.2",
         "waitpid": "git+https://github.com/HexF/node-waitpid.git"
     },
-    "license": "MIT"
+    "license": "MIT",
+    "scripts": {
+        "lint": "prettier . --write"
+    },
+    "devDependencies": {
+        "prettier": "2.2.1"
+    }
 }

api/src/api/v2.js

@@ -8,46 +8,7 @@ const { Job } = require('../job');
 const package = require('../package');
 const logger = require('logplease').create('api/v2');
 
-const SIGNALS = [
+const SIGNALS = ["SIGABRT","SIGALRM","SIGBUS","SIGCHLD","SIGCLD","SIGCONT","SIGEMT","SIGFPE","SIGHUP","SIGILL","SIGINFO","SIGINT","SIGIO","SIGIOT","SIGKILL","SIGLOST","SIGPIPE","SIGPOLL","SIGPROF","SIGPWR","SIGQUIT","SIGSEGV","SIGSTKFLT","SIGSTOP","SIGTSTP","SIGSYS","SIGTERM","SIGTRAP","SIGTTIN","SIGTTOU","SIGUNUSED","SIGURG","SIGUSR1","SIGUSR2","SIGVTALRM","SIGXCPU","SIGXFSZ","SIGWINCH"]
-    'SIGABRT',
-    'SIGALRM',
-    'SIGBUS',
-    'SIGCHLD',
-    'SIGCLD',
-    'SIGCONT',
-    'SIGEMT',
-    'SIGFPE',
-    'SIGHUP',
-    'SIGILL',
-    'SIGINFO',
-    'SIGINT',
-    'SIGIO',
-    'SIGIOT',
-    'SIGKILL',
-    'SIGLOST',
-    'SIGPIPE',
-    'SIGPOLL',
-    'SIGPROF',
-    'SIGPWR',
-    'SIGQUIT',
-    'SIGSEGV',
-    'SIGSTKFLT',
-    'SIGSTOP',
-    'SIGTSTP',
-    'SIGSYS',
-    'SIGTERM',
-    'SIGTRAP',
-    'SIGTTIN',
-    'SIGTTOU',
-    'SIGUNUSED',
-    'SIGURG',
-    'SIGUSR1',
-    'SIGUSR2',
-    'SIGVTALRM',
-    'SIGXCPU',
-    'SIGXFSZ',
-    'SIGWINCH',
-];
 // ref: https://man7.org/linux/man-pages/man7/signal.7.html
 
 function get_job(body){
@@ -60,7 +21,7 @@ function get_job(body) {
         compile_memory_limit,
         run_memory_limit,
         run_timeout,
-        compile_timeout,
+        compile_timeout
     } = body;
 
     return new Promise((resolve, reject) => {
@@ -107,7 +68,7 @@ function get_job(body) {
                 }
                 if (typeof constraint_value !== 'number') {
                     return reject({
-                        message: `If specified, ${constraint_name} must be a number`,
+                        message: `If specified, ${constraint_name} must be a number`
                     });
                 }
                 if (configured_limit <= 0) {
@@ -115,12 +76,12 @@ function get_job(body) {
                 }
                 if (constraint_value > configured_limit) {
                     return reject({
-                        message: `${constraint_name} cannot exceed the configured limit of ${configured_limit}`,
+                        message: `${constraint_name} cannot exceed the configured limit of ${configured_limit}`
                     });
                 }
                 if (constraint_value < 0) {
                     return reject({
-                        message: `${constraint_name} must be non-negative`,
+                        message: `${constraint_name} must be non-negative`
                     });
                 }
             }
@@ -130,11 +91,10 @@ function get_job(body) {
         run_timeout = run_timeout || rt.timeouts.run;
         compile_memory_limit = compile_memory_limit || rt.memory_limits.compile;
         run_timeout = run_timeout || rt.timeouts.run;
-        resolve(
+        resolve(new Job({
-            new Job({
             runtime: rt,
             args: args || [],
-                stdin: stdin || '',
+            stdin: stdin || "",
             files,
             timeouts: {
                 run: run_timeout,
@@ -143,9 +103,8 @@ function get_job(body) {
             memory_limits: {
                 run: run_memory_limit,
                 compile: compile_memory_limit,
-                },
+            }
-            })
+        }));
-        );
     });
 }
 
@@ -164,103 +123,87 @@ router.use((req, res, next) => {
 });
 
 router.ws('/connect', async (ws, req) => {
+
     let job = null;
     let eventBus = new events.EventEmitter();
 
-    eventBus.on('stdout', data =>
+    eventBus.on("stdout", (data) => ws.send(JSON.stringify({type: "data", stream: "stdout", data: data.toString()})))
-        ws.send(
+    eventBus.on("stderr", (data) => ws.send(JSON.stringify({type: "data", stream: "stderr", data: data.toString()})))
-            JSON.stringify({
+    eventBus.on("stage", (stage)=> ws.send(JSON.stringify({type: "stage", stage})))
-                type: 'data',
+    eventBus.on("exit", (stage, status) => ws.send(JSON.stringify({type: "exit", stage, ...status})))
-                stream: 'stdout',
+
-                data: data.toString(),
+    ws.on("message", async (data) => {
-            })
-        )
-    );
-    eventBus.on('stderr', data =>
-        ws.send(
-            JSON.stringify({
-                type: 'data',
-                stream: 'stderr',
-                data: data.toString(),
-            })
-        )
-    );
-    eventBus.on('stage', stage =>
-        ws.send(JSON.stringify({ type: 'stage', stage }))
-    );
-    eventBus.on('exit', (stage, status) =>
-        ws.send(JSON.stringify({ type: 'exit', stage, ...status }))
-    );
 
-    ws.on('message', async data => {
         try{
             const msg = JSON.parse(data);
 
             switch(msg.type){
-                case 'init':
+                case "init":
                     if(job === null){
                         job = await get_job(msg);
 
                         await job.prime();
 
-                        ws.send(
+                        ws.send(JSON.stringify({
-                            JSON.stringify({
+                            type: "runtime",
-                                type: 'runtime',
                             language: job.runtime.language,
-                                version: job.runtime.version.raw,
+                            version: job.runtime.version.raw
-                            })
+                        }))
-                        );
 
                         await job.execute_interactive(eventBus);
 
-                        ws.close(4999, 'Job Completed');
+                        ws.close(4999, "Job Completed");
+
                     }else{
-                        ws.close(4000, 'Already Initialized');
+                        ws.close(4000, "Already Initialized");
                     }
                     break;
-                case 'data':
+            case "data":
                 if(job !== null){
-                        if (msg.stream === 'stdin') {
+                    if(msg.stream === "stdin"){
-                            eventBus.emit('stdin', msg.data);
+                        eventBus.emit("stdin", msg.data)
                     }else{
-                            ws.close(4004, 'Can only write to stdin');
+                        ws.close(4004, "Can only write to stdin")
                     }
                 }else{
-                        ws.close(4003, 'Not yet initialized');
+                    ws.close(4003, "Not yet initialized")
                 }
                 break;
-                case 'signal':
+            case "signal":
                 if(job !== null){
                     if(SIGNALS.includes(msg.signal)){
-                            eventBus.emit('signal', msg.signal);
+                        eventBus.emit("signal", msg.signal)
                     }else{
-                            ws.close(4005, 'Invalid signal');
+                        ws.close(4005, "Invalid signal")
                     }
                 }else{
-                        ws.close(4003, 'Not yet initialized');
+                    ws.close(4003, "Not yet initialized")
                 }
                 break;
             }
+
         }catch(error){
-            ws.send(JSON.stringify({ type: 'error', message: error.message }));
+            ws.send(JSON.stringify({type: "error", message: error.message}))
-            ws.close(4002, 'Notified Error');
+            ws.close(4002, "Notified Error")
             // ws.close message is limited to 123 characters, so we notify over WS then close.
         }
-    });
+    })
 
-    ws.on('close', async () => {
+    ws.on("close", async ()=>{
         if(job !== null){
-            await job.cleanup();
+            await job.cleanup()
         }
-    });
+    })
 
     setTimeout(()=>{
         //Terminate the socket after 1 second, if not initialized.
-        if (job === null) ws.close(4001, 'Initialization Timeout');
+        if(job === null)
-    }, 1000);
+            ws.close(4001, "Initialization Timeout");
-});
+    }, 1000)
+})
 
 router.post('/execute', async (req, res) => {
+
     try{
         const job = await get_job(req.body);
 

api/src/config.js

@@ -5,7 +5,8 @@ const logger = Logger.create('config');
 function parse_overrides(overrides) {
     try {
         return JSON.parse(overrides);
-    } catch (e) {
+    }
+    catch (e) {
         return null;
     }
 }
@@ -15,20 +16,15 @@ function validate_overrides(overrides, options) {
         for (let key in overrides[language]) {
             if (
                 ![
-                    'max_process_count',
+                    'max_process_count', 'max_open_files', 'max_file_size',
-                    'max_open_files',
+                    'compile_memory_limit', 'run_memory_limit', 'compile_timeout',
-                    'max_file_size',
+                    'run_timeout', 'output_max_size'
-                    'compile_memory_limit',
-                    'run_memory_limit',
-                    'compile_timeout',
-                    'run_timeout',
-                    'output_max_size',
                 ].includes(key)
             ) {
                 logger.error(`Invalid overridden option: ${key}`);
                 return false;
             }
-            let option = options.find(o => o.key === key);
+            let option = options.find((o) => o.key === key);
             let parser = option.parser;
             let raw = overrides[language][key];
             let value = parser(raw);
@@ -36,19 +32,14 @@ function validate_overrides(overrides, options) {
             for (let validator of validators) {
                 let response = validator(value, raw);
                 if (response !== true) {
-                    logger.error(
+                    logger.error(`Failed to validate overridden option: ${key}`, response);
-                        `Failed to validate overridden option: ${key}`,
-                        response
-                    );
                     return false;
                 }
             }
             overrides[language][key] = value;
         }
         // Modifies the reference
-        options[
+        options[options.index_of(options.find((o) => o.key === 'limit_overrides'))] = overrides;
-            options.index_of(options.find(o => o.key === 'limit_overrides'))
-        ] = overrides;
     }
     return true;
 }
@@ -144,28 +135,32 @@ const options = [
     },
     {
         key: 'compile_timeout',
-        desc: 'Max time allowed for compile stage in milliseconds',
+        desc:
+            'Max time allowed for compile stage in milliseconds',
         default: 10000, // 10 seconds
         parser: parse_int,
         validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
     },
     {
         key: 'run_timeout',
-        desc: 'Max time allowed for run stage in milliseconds',
+        desc:
+            'Max time allowed for run stage in milliseconds',
         default: 3000, // 3 seconds
         parser: parse_int,
         validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
     },
     {
         key: 'compile_memory_limit',
-        desc: 'Max memory usage for compile stage in bytes (set to -1 for no limit)',
+        desc:
+            'Max memory usage for compile stage in bytes (set to -1 for no limit)',
         default: -1, // no limit
         parser: parse_int,
         validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
     },
     {
         key: 'run_memory_limit',
-        desc: 'Max memory usage for run stage in bytes (set to -1 for no limit)',
+        desc:
+            'Max memory usage for run stage in bytes (set to -1 for no limit)',
         default: -1, // no limit
         parser: parse_int,
         validators: [(x, raw) => !is_nan(x) || `${raw} is not a number`],
@@ -182,7 +177,7 @@ const options = [
         desc: 'Maximum number of concurrent jobs to run at one time',
         default: 64,
         parser: parse_int,
-        validators: [x => x > 0 || `${x} cannot be negative`],
+        validators: [(x) => x > 0 || `${x} cannot be negative`]
     },
     {
         key: 'limit_overrides',
@@ -192,12 +187,10 @@ const options = [
         default: {},
         parser: parse_overrides,
         validators: [
-            x => !!x || `Invalid JSON format for the overrides\n${x}`,
+            (x) => !!x || `Invalid JSON format for the overrides\n${x}`,
-            (overrides, _, options) =>
+            (overrides, _, options) => validate_overrides(overrides, options) || `Failed to validate the overrides`
-                validate_overrides(overrides, options) ||
+        ]
-                `Failed to validate the overrides`,
+    }
-        ],
-    },
 ];
 
 logger.info(`Loading Configuration from environment`);

api/src/index.js

@@ -15,6 +15,8 @@ const logger = Logger.create('index');
 const app = express();
 expressWs(app);
 
+
+
 (async () => {
     logger.info('Setting loglevel to', config.log_level);
     Logger.setLogLevel(config.log_level);

api/src/job.js

@@ -19,12 +19,15 @@ let gid = 0;
 let remainingJobSpaces = config.max_concurrent_jobs;
 let jobQueue = [];
 
+
 setInterval(()=>{
     // Every 10ms try resolve a new job, if there is an available slot
     if(jobQueue.length > 0 && remainingJobSpaces > 0){
-        jobQueue.shift()();
+        jobQueue.shift()()
     }
-}, 10);
+}, 10)
+
+
 
 class Job {
     constructor({ runtime, files, args, stdin }) {
@@ -57,10 +60,10 @@ class Job {
 
     async prime() {
         if(remainingJobSpaces < 1){
-            logger.info(`Awaiting job slot uuid=${this.uuid}`);
+            logger.info(`Awaiting job slot uuid=${this.uuid}`)
-            await new Promise(resolve => {
+            await new Promise((resolve)=>{
-                jobQueue.push(resolve);
+                jobQueue.push(resolve)
-            });
+            })
         }
 
         logger.info(`Priming job uuid=${this.uuid}`);
@@ -76,15 +79,10 @@ class Job {
             let file_path = path.join(this.dir, file.name);
             const rel = path.relative(this.dir, file_path);
 
-            if (rel.startsWith('..'))
+            if(rel.startsWith(".."))
-                throw Error(
+                throw Error(`File path "${file.name}" tries to escape parent directory: ${rel}`)
-                    `File path "${file.name}" tries to escape parent directory: ${rel}`
-                );
 
-            await fs.mkdir(path.dirname(file_path), {
+            await fs.mkdir(path.dirname(file_path), {recursive: true,  mode: 0o700})
-                recursive: true,
-                mode: 0o700,
-            });
             await fs.chown(path.dirname(file_path), this.uid, this.gid);
 
             await fs.write_file(file_path, file.content);
@@ -134,28 +132,29 @@ class Job {
                 proc.stdin.end();
                 proc.stdin.destroy();
             }else{
-                eventBus.on('stdin', data => {
+                eventBus.on("stdin", (data) => {
                     proc.stdin.write(data);
-                });
+                })
 
-                eventBus.on('kill', signal => {
+                eventBus.on("kill", (signal) => {
-                    proc.kill(signal);
+                    proc.kill(signal)
-                });
+                })
             }
 
-            const kill_timeout = set_timeout(async _ => {
+            const kill_timeout = set_timeout(
-                logger.info(
+                async _ => {
-                    `Timeout exceeded timeout=${timeout} uuid=${this.uuid}`
+                    logger.info(`Timeout exceeded timeout=${timeout} uuid=${this.uuid}`)
+                    process.kill(proc.pid, 'SIGKILL')
+                },
+                timeout
             );
-                process.kill(proc.pid, 'SIGKILL');
-            }, timeout);
 
             proc.stderr.on('data', async data => {
                 if(eventBus !== null) {
-                    eventBus.emit('stderr', data);
+                    eventBus.emit("stderr", data);
                 } else if (stderr.length > this.runtime.output_max_size) {
-                    logger.info(`stderr length exceeded uuid=${this.uuid}`);
+                    logger.info(`stderr length exceeded uuid=${this.uuid}`)
-                    process.kill(proc.pid, 'SIGKILL');
+                    process.kill(proc.pid, 'SIGKILL')
                 } else {
                     stderr += data;
                     output += data;
@@ -164,10 +163,10 @@ class Job {
 
             proc.stdout.on('data', async data => {
                 if(eventBus !== null){
-                    eventBus.emit('stdout', data);
+                    eventBus.emit("stdout", data);
                 } else if (stdout.length > this.runtime.output_max_size) {
-                    logger.info(`stdout length exceeded uuid=${this.uuid}`);
+                    logger.info(`stdout length exceeded uuid=${this.uuid}`)
-                    process.kill(proc.pid, 'SIGKILL');
+                    process.kill(proc.pid, 'SIGKILL')
                 } else {
                     stdout += data;
                     output += data;
@@ -180,8 +179,8 @@ class Job {
                 proc.stderr.destroy();
                 proc.stdout.destroy();
 
-                await this.cleanup_processes();
+                await this.cleanup_processes()
-                logger.debug(`Finished exit cleanup uuid=${this.uuid}`);
+                logger.debug(`Finished exit cleanup uuid=${this.uuid}`)
             };
 
             proc.on('exit', async (code, signal) => {
@@ -253,26 +252,26 @@ class Job {
         }
 
         logger.info(
-            `Interactively executing job uuid=${this.uuid} uid=${
+            `Interactively executing job uuid=${this.uuid} uid=${this.uid} gid=${
-                this.uid
+                this.gid
-            } gid=${this.gid} runtime=${this.runtime.toString()}`
+            } runtime=${this.runtime.toString()}`
         );
 
         if(this.runtime.compiled){
-            eventBus.emit('stage', 'compile');
+            eventBus.emit("stage", "compile")
             const {error, code, signal} = await this.safe_call(
                 path.join(this.runtime.pkgdir, 'compile'),
                 this.files.map(x => x.name),
                 this.runtime.timeouts.compile,
                 this.runtime.memory_limits.compile,
                 eventBus
-            );
+            )
 
-            eventBus.emit('exit', 'compile', { error, code, signal });
+            eventBus.emit("exit", "compile", {error, code, signal})
         }
 
         logger.debug('Running');
-        eventBus.emit('stage', 'run');
+        eventBus.emit("stage", "run")
         const {error, code, signal} = await this.safe_call(
             path.join(this.runtime.pkgdir, 'run'),
             [this.files[0].name, ...this.args],
@@ -281,50 +280,47 @@ class Job {
             eventBus
         );
 
-        eventBus.emit('exit', 'run', { error, code, signal });
+        eventBus.emit("exit", "run", {error, code, signal})
+
 
         this.state = job_states.EXECUTED;
     }
 
     async cleanup_processes(dont_wait = []) {
         let processes = [1];
-        logger.debug(`Cleaning up processes uuid=${this.uuid}`);
+        logger.debug(`Cleaning up processes uuid=${this.uuid}`)
 
         while (processes.length > 0) {
-            processes = [];
+            processes = []
 
-            const proc_ids = await fs.readdir('/proc');
 
-            processes = await Promise.all(
+            const proc_ids = await fs.readdir("/proc");
-                proc_ids.map(async proc_id => {
+
+
+            processes = await Promise.all(proc_ids.map(async (proc_id) => {
                 if(isNaN(proc_id)) return -1;
                 try{
-                        const proc_status = await fs.read_file(
+                    const proc_status = await fs.read_file(path.join("/proc",proc_id,"status"));
-                            path.join('/proc', proc_id, 'status')
+                    const proc_lines = proc_status.to_string().split("\n")
-                        );
+                    const uid_line = proc_lines.find(line=>line.starts_with("Uid:"))
-                        const proc_lines = proc_status.to_string().split('\n');
+                    const [_, ruid, euid, suid, fuid] = uid_line.split(/\s+/);
-                        const uid_line = proc_lines.find(line =>
-                            line.starts_with('Uid:')
-                        );
-                        const [_, ruid, euid, suid, fuid] =
-                            uid_line.split(/\s+/);
 
                     if(ruid == this.uid || euid == this.uid)
-                            return parse_int(proc_id);
+                        return parse_int(proc_id)
+
                 }catch{
-                        return -1;
+                    return -1
                 }
 
-                    return -1;
+                return -1
-                })
+            }))
-            );
 
-            processes = processes.filter(p => p > 0);
+            processes = processes.filter(p => p > 0)
 
             if(processes.length > 0)
-                logger.debug(
+                logger.debug(`Got processes to kill: ${processes} uuid=${this.uuid}`)
-                    `Got processes to kill: ${processes} uuid=${this.uuid}`
+
-                );
+
 
             for (const proc of processes) {
                 // First stop the processes, but keep their resources allocated so they cant re-fork
@@ -343,11 +339,12 @@ class Job {
                     // Could already be dead and just needs to be waited on
                 }
 
-                if (!dont_wait.includes(proc)) wait_pid(proc);
+                if(!dont_wait.includes(proc))
+                    wait_pid(proc);
             }
         }
 
-        logger.debug(`Cleaned up processes uuid=${this.uuid}`);
+        logger.debug(`Cleaned up processes uuid=${this.uuid}`)
     }
 
     async cleanup_filesystem() {
@@ -385,6 +382,7 @@ class Job {
     }
 }
 
+
 module.exports = {
     Job,
 };

api/src/package.js

@@ -74,9 +74,10 @@ class Package {
         await new Promise((resolve, reject) => {
             read_stream.on('data', chunk => hash.update(chunk));
             read_stream.on('end', () => resolve());
-            read_stream.on('error', error => reject(error));
+            read_stream.on('error', error => reject(error))
         });
         
+
         const cs = hash.digest('hex');
 
         if (cs !== this.checksum) {

api/src/runtime.js

@@ -9,17 +9,8 @@ const runtimes = [];
 
 class Runtime {
     constructor({
-        language,
+        language, version, aliases, pkgdir, runtime, timeouts, memory_limits, max_process_count,
-        version,
+        max_open_files, max_file_size, output_max_size
-        aliases,
-        pkgdir,
-        runtime,
-        timeouts,
-        memory_limits,
-        max_process_count,
-        max_open_files,
-        max_file_size,
-        output_max_size,
     }) {
         this.language = language;
         this.version = version;
@@ -34,67 +25,37 @@ class Runtime {
         this.output_max_size = output_max_size;
     }
 
-    static compute_single_limit(
+    static compute_single_limit(language_name, limit_name, language_limit_overrides) {
-        language_name,
-        limit_name,
-        language_limit_overrides
-    ) {
         return (
-            (config.limit_overrides[language_name] &&
+            config.limit_overrides[language_name] && config.limit_overrides[language_name][limit_name]
-                config.limit_overrides[language_name][limit_name]) ||
+            || language_limit_overrides && language_limit_overrides[limit_name]
-            (language_limit_overrides &&
+            || config[limit_name]
-                language_limit_overrides[limit_name]) ||
-            config[limit_name]
         );
     }
 
     static compute_all_limits(language_name, language_limit_overrides) {
         return {
             timeouts: {
-                compile: this.compute_single_limit(
+                compile:
-                    language_name,
+                    this.compute_single_limit(language_name, 'compile_timeout', language_limit_overrides),
-                    'compile_timeout',
+                run:
-                    language_limit_overrides
+                    this.compute_single_limit(language_name, 'run_timeout', language_limit_overrides)
-                ),
-                run: this.compute_single_limit(
-                    language_name,
-                    'run_timeout',
-                    language_limit_overrides
-                ),
             },
             memory_limits: {
-                compile: this.compute_single_limit(
+                compile:
-                    language_name,
+                    this.compute_single_limit(language_name, 'compile_memory_limit', language_limit_overrides),
-                    'compile_memory_limit',
+                run:
-                    language_limit_overrides
+                    this.compute_single_limit(language_name, 'run_memory_limit', language_limit_overrides)
-                ),
-                run: this.compute_single_limit(
-                    language_name,
-                    'run_memory_limit',
-                    language_limit_overrides
-                ),
             },
-            max_process_count: this.compute_single_limit(
+            max_process_count:
-                language_name,
+                this.compute_single_limit(language_name, 'max_process_count', language_limit_overrides),
-                'max_process_count',
+            max_open_files:
-                language_limit_overrides
+                this.compute_single_limit(language_name, 'max_open_files', language_limit_overrides),
-            ),
+            max_file_size:
-            max_open_files: this.compute_single_limit(
+                this.compute_single_limit(language_name, 'max_file_size', language_limit_overrides),
-                language_name,
+            output_max_size:
-                'max_open_files',
+                this.compute_single_limit(language_name, 'output_max_size', language_limit_overrides),
-                language_limit_overrides
+        }
-            ),
-            max_file_size: this.compute_single_limit(
-                language_name,
-                'max_file_size',
-                language_limit_overrides
-            ),
-            output_max_size: this.compute_single_limit(
-                language_name,
-                'output_max_size',
-                language_limit_overrides
-            ),
-        };
     }
 
     static load_package(package_dir) {
@@ -102,14 +63,7 @@ class Runtime {
             fss.read_file_sync(path.join(package_dir, 'pkg-info.json'))
         );
 
-        let {
+        let { language, version, build_platform, aliases, provides, limit_overrides } = info;
-            language,
-            version,
-            build_platform,
-            aliases,
-            provides,
-            limit_overrides,
-        } = info;
         version = semver.parse(version);
 
         if (build_platform !== globals.platform) {
@@ -129,10 +83,7 @@ class Runtime {
                         version,
                         pkgdir: package_dir,
                         runtime: language,
-                        ...Runtime.compute_all_limits(
+                        ...Runtime.compute_all_limits(lang.language, lang.limit_overrides)
-                            lang.language,
-                            lang.limit_overrides
-                        ),
                     })
                 );
             });
@@ -143,7 +94,7 @@ class Runtime {
                     version,
                     aliases,
                     pkgdir: package_dir,
-                    ...Runtime.compute_all_limits(language, limit_overrides),
+                    ...Runtime.compute_all_limits(language, limit_overrides)
                 })
             );
         }

cli/.gitignore

@@ -0,0 +1 @@
+node_modules

cli/commands/execute.js

@@ -3,44 +3,8 @@ const path = require('path');
 const chalk = require('chalk');
 const WebSocket = require('ws');
 
-const SIGNALS = [
+const SIGNALS = ["SIGABRT","SIGALRM","SIGBUS","SIGCHLD","SIGCLD","SIGCONT","SIGEMT","SIGFPE","SIGHUP","SIGILL","SIGINFO","SIGINT","SIGIO","SIGIOT","SIGLOST","SIGPIPE","SIGPOLL","SIGPROF","SIGPWR","SIGQUIT","SIGSEGV","SIGSTKFLT","SIGTSTP","SIGSYS","SIGTERM","SIGTRAP","SIGTTIN","SIGTTOU","SIGUNUSED","SIGURG","SIGUSR1","SIGUSR2","SIGVTALRM","SIGXCPU","SIGXFSZ","SIGWINCH"]
-    'SIGABRT',
+
-    'SIGALRM',
-    'SIGBUS',
-    'SIGCHLD',
-    'SIGCLD',
-    'SIGCONT',
-    'SIGEMT',
-    'SIGFPE',
-    'SIGHUP',
-    'SIGILL',
-    'SIGINFO',
-    'SIGINT',
-    'SIGIO',
-    'SIGIOT',
-    'SIGLOST',
-    'SIGPIPE',
-    'SIGPOLL',
-    'SIGPROF',
-    'SIGPWR',
-    'SIGQUIT',
-    'SIGSEGV',
-    'SIGSTKFLT',
-    'SIGTSTP',
-    'SIGSYS',
-    'SIGTERM',
-    'SIGTRAP',
-    'SIGTTIN',
-    'SIGTTOU',
-    'SIGUNUSED',
-    'SIGURG',
-    'SIGUSR1',
-    'SIGUSR2',
-    'SIGVTALRM',
-    'SIGXCPU',
-    'SIGXFSZ',
-    'SIGWINCH',
-];
 
 exports.command = ['execute <language> <file> [args..]'];
 exports.aliases = ['run'];
@@ -51,18 +15,18 @@ exports.builder = {
         string: true,
         desc: 'Set the version of the language to use',
         alias: ['l'],
-        default: '*',
+        default: '*'
     },
     stdin: {
         boolean: true,
         desc: 'Read input from stdin and pass to executor',
-        alias: ['i'],
+        alias: ['i']
     },
     run_timeout: {
         alias: ['rt', 'r'],
         number: true,
         desc: 'Milliseconds before killing run process',
-        default: 3000,
+        default: 3000
     },
     compile_timeout: {
         alias: ['ct', 'c'],
@@ -78,126 +42,117 @@ exports.builder = {
     interactive: {
         boolean: true,
         alias: ['t'],
-        desc: 'Run interactively using WebSocket transport',
+        desc: 'Run interactively using WebSocket transport'
     },
     status: {
         boolean: true,
         alias: ['s'],
-        desc: 'Output additional status to stderr',
+        desc: 'Output additional status to stderr'
-    },
+    }
 };
 
 async function handle_interactive(files, argv){
-    const ws = new WebSocket(
+    const ws = new WebSocket(argv.pistonUrl.replace("http", "ws") + "/api/v2/connect")
-        argv.pistonUrl.replace('http', 'ws') + '/api/v2/connect'
-    );
 
-    const log_message =
+    const log_message = (process.stderr.isTTY && argv.status) ? console.error : ()=>{};
-        process.stderr.isTTY && argv.status ? console.error : () => {};
 
-    process.on('exit', () => {
+    process.on("exit", ()=>{
         ws.close();
         process.stdin.end();
         process.stdin.destroy();
         process.exit();        
-    });
+    })
 
     for(const signal of SIGNALS){
         process.on(signal, ()=>{
-            ws.send(JSON.stringify({ type: 'signal', signal }));
+            ws.send(JSON.stringify({type: 'signal', signal}))
-        });
+        }) 
     }
 
+    
+
     ws.on('open', ()=>{
         const request = {
-            type: 'init',
+            type: "init",
             language: argv.language,
             version: argv['language_version'],
             files: files,
             args: argv.args,
             compile_timeout: argv.ct,
-            run_timeout: argv.rt,
+            run_timeout: argv.rt
-        };
+        }
 
-        ws.send(JSON.stringify(request));
+        ws.send(JSON.stringify(request))
-        log_message(chalk.white.bold('Connected'));
+        log_message(chalk.white.bold("Connected"))
 
         process.stdin.resume();
 
-        process.stdin.on('data', data => {
+        process.stdin.on("data", (data) => {
-            ws.send(
+            ws.send(JSON.stringify({
-                JSON.stringify({
+                type: "data",
-                    type: 'data',
+                stream: "stdin",
-                    stream: 'stdin',
+                data: data.toString()
-                    data: data.toString(),
+            }))
+        })
     })
-            );
-        });
-    });
 
-    ws.on('close', (code, reason) => {
+    ws.on("close", (code, reason)=>{
         log_message(
-            chalk.white.bold('Disconnected: '),
+            chalk.white.bold("Disconnected: "),
-            chalk.white.bold('Reason: '),
+            chalk.white.bold("Reason: "),
             chalk.yellow(`"${reason}"`),
-            chalk.white.bold('Code: '),
+            chalk.white.bold("Code: "),
-            chalk.yellow(`"${code}"`)
+            chalk.yellow(`"${code}"`),
-        );
+            )
-        process.stdin.pause();
+        process.stdin.pause()
-    });
+    })
 
     ws.on('message', function(data){
         const msg = JSON.parse(data);
         
         switch(msg.type){
-            case 'runtime':
+            case "runtime":
-                log_message(
+                log_message(chalk.bold.white("Runtime:"), chalk.yellow(`${msg.language} ${msg.version}`))
-                    chalk.bold.white('Runtime:'),
-                    chalk.yellow(`${msg.language} ${msg.version}`)
-                );
                 break;
-            case 'stage':
+            case "stage":
-                log_message(
+                log_message(chalk.bold.white("Stage:"), chalk.yellow(msg.stage))
-                    chalk.bold.white('Stage:'),
-                    chalk.yellow(msg.stage)
-                );
                 break;
-            case 'data':
+            case "data":
-                if (msg.stream == 'stdout') process.stdout.write(msg.data);
+                if(msg.stream == "stdout") process.stdout.write(msg.data)
-                else if (msg.stream == 'stderr') process.stderr.write(msg.data);
+                else if(msg.stream == "stderr") process.stderr.write(msg.data)
-                else log_message(chalk.bold.red(`(${msg.stream}) `), msg.data);
+                else log_message(chalk.bold.red(`(${msg.stream}) `), msg.data)
                 break;
-            case 'exit':
+            case "exit":
                 if(msg.signal === null)
                     log_message(
-                        chalk.white.bold('Stage'),
+                        chalk.white.bold("Stage"),
                         chalk.yellow(msg.stage),
-                        chalk.white.bold('exited with code'),
+                        chalk.white.bold("exited with code"),
                         chalk.yellow(msg.code)
-                    );
+                    )
                 else
                     log_message(
-                        chalk.white.bold('Stage'),
+                        chalk.white.bold("Stage"),
                         chalk.yellow(msg.stage),
-                        chalk.white.bold('exited with signal'),
+                        chalk.white.bold("exited with signal"),
                         chalk.yellow(msg.signal)
-                    );
+                    )
                     break;
             default:
-                log_message(chalk.red.bold('Unknown message:'), msg);
+                log_message(chalk.red.bold("Unknown message:"), msg)
         }
-    });
+    })
+
 }
 
 async function run_non_interactively(files, argv) {
-    const stdin =
+    
-        (argv.stdin &&
+
-            (await new Promise((resolve, _) => {
+    const stdin = (argv.stdin && await new Promise((resolve, _) => {
         let data = '';
-                process.stdin.on('data', d => (data += d));
+        process.stdin.on('data', d => data += d);
         process.stdin.on('end', _ => resolve(data));
-            }))) ||
+    })) || '';
-        '';
 
     const request = {
         language: argv.language,
@@ -206,7 +161,7 @@ async function run_non_interactively(files, argv) {
         args: argv.args,
         stdin,
         compile_timeout: argv.ct,
-        run_timeout: argv.rt,
+        run_timeout: argv.rt
     };
 
     let { data: response } = await argv.axios.post('/api/v2/execute', request);
@@ -215,13 +170,13 @@ async function run_non_interactively(files, argv) {
         console.log(chalk.bold(`== ${name} ==`));
 
         if (ctx.stdout) {
-            console.log(chalk.bold(`STDOUT`));
+            console.log(chalk.bold(`STDOUT`))
-            console.log(ctx.stdout.replace(/\n/g, '\n    '));
+            console.log(ctx.stdout.replace(/\n/g,'\n    '))
         }
 
         if (ctx.stderr) {
-            console.log(chalk.bold(`STDERR`));
+            console.log(chalk.bold(`STDERR`))
-            console.log(ctx.stderr.replace(/\n/g, '\n    '));
+            console.log(ctx.stderr.replace(/\n/g,'\n    '))
         }
 
         if (ctx.code) {
@@ -232,9 +187,12 @@ async function run_non_interactively(files, argv) {
         }
 
         if (ctx.signal) {
-            console.log(chalk.bold(`Signal:`), chalk.bold.yellow(ctx.signal));
+            console.log(
+                chalk.bold(`Signal:`),
+                chalk.bold.yellow(ctx.signal)
+            );
+        }
     }
-    };
 
     if (response.compile) {
         step('Compile', response.compile);
@@ -243,14 +201,17 @@ async function run_non_interactively(files, argv) {
     step('Run', response.run);
 }
 
-exports.handler = async argv => {
+exports.handler = async (argv) => {
-    const files = [...(argv.files || []), argv.file].map(file_path => {
+    const files = [...(argv.files || []),argv.file]
+        .map(file_path => {
             return {
                 name: path.basename(file_path),
-            content: fs.readFileSync(file_path).toString(),
+                content: fs.readFileSync(file_path).toString()
             };
         });
 
     if(argv.interactive) await handle_interactive(files, argv);
     else await run_non_interactively(files, argv);
-};
+}
+
+

cli/commands/ppman.js

@@ -2,4 +2,6 @@ exports.command = 'ppman';
 exports.aliases = ['pkg'];
 exports.describe = 'Package Manager';
 
-exports.builder = yargs => yargs.commandDir('ppman_commands').demandCommand();
+exports.builder = yargs => yargs
+    .commandDir('ppman_commands')
+    .demandCommand();

cli/commands/ppman_commands/install.js

@@ -4,31 +4,30 @@ exports.command = ['install <packages...>'];
 exports.aliases = ['i'];
 exports.describe = 'Installs the named package';
 
+
+
 //Splits the package into it's language and version
 function split_package(package) {
-    [language, language_version] = package.split('=');
+    [language, language_version] = package.split("=")
 
     res = {
         language: language,
-        version: language_version || '*',
+        version: language_version || "*"
     };
-    return res;
+    return res
 }
 
 const msg_format = {
-    color: p =>
+    color: p => `${p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')} Installation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-        `${
+    monochrome: p => `Installation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-            p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')
+    json: JSON.stringify
-        } Installation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-    monochrome: p =>
-        `Installation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-    json: JSON.stringify,
 };
 
 exports.handler = async ({ axios, packages }) => {
     const requests = packages.map(package => split_package(package));
     for (request of requests) {
         try {
+
             const install = await axios.post(`/api/v2/packages`, request);
 
             console.log(msg_format.color(install.data));
@@ -36,4 +35,5 @@ exports.handler = async ({ axios, packages }) => {
             console.error(response.data.message);
         }
     }
-};
+
+}

cli/commands/ppman_commands/list.js

@@ -5,21 +5,17 @@ exports.aliases = ['l'];
 exports.describe = 'Lists all available packages';
 
 const msg_format = {
-    color: p =>
+    color: p => `${chalk[p.installed ? 'green':'red']('•')} ${p.language} ${p.language_version}`,
-        `${chalk[p.installed ? 'green' : 'red']('•')} ${p.language} ${
+    monochrome: p => `${p.language} ${p.language_version} ${p.installed ? '(INSTALLED)': ''}`,
-            p.language_version
+    json: JSON.stringify
-        }`,
-    monochrome: p =>
-        `${p.language} ${p.language_version} ${
-            p.installed ? '(INSTALLED)' : ''
-        }`,
-    json: JSON.stringify,
 };
 
 exports.handler = async ({ axios }) => {
     const packages = await axios.get('/api/v2/packages');
 
-    const pkg_msg = packages.data.map(msg_format.color).join('\n');
+    const pkg_msg = packages.data
+        .map(msg_format.color)
+        .join('\n');
 
     console.log(pkg_msg);
-};
+}

cli/commands/ppman_commands/spec.js

@@ -1,26 +1,22 @@
 const chalk = require('chalk');
 const fs = require('fs/promises');
-const minimatch = require('minimatch');
+const minimatch = require("minimatch");
 const semver = require('semver');
 
 exports.command = ['spec <specfile>'];
 exports.aliases = ['s'];
-exports.describe =
+exports.describe = 'Install the packages described in the spec file, uninstalling packages which aren\'t in the list'
-    "Install the packages described in the spec file, uninstalling packages which aren't in the list";
 
 function does_match(package, rule){
     const nameMatch = minimatch(package.language, rule.package_selector);
-    const versionMatch = semver.satisfies(
+    const versionMatch = semver.satisfies(package.language_version, rule.version_selector)
-        package.language_version,
-        rule.version_selector
-    );
 
     return nameMatch && versionMatch;
 }
 
 exports.handler = async ({axios, specfile}) => {
     const spec_contents = await fs.readFile(specfile);
-    const spec_lines = spec_contents.toString().split('\n');
+    const spec_lines = spec_contents.toString().split("\n");
 
     const rules = [];
 
@@ -30,19 +26,19 @@ exports.handler = async ({ axios, specfile }) => {
             comment: false,
             package_selector: null,
             version_selector: null,
-            negate: false,
+            negate: false
         };
 
-        if (line.starts_with('#')) {
+        if(line.starts_with("#")){
             rule.comment = true;
         }else {
             let l = line.trim();
-            if (line.starts_with('!')) {
+            if(line.starts_with("!")){
                 rule.negate = true;
                 l = line.slice(1).trim();
             }
 
-            const [pkg, ver] = l.split(' ', 2);
+            const [pkg, ver] = l.split(" ", 2);
             rule.package_selector = pkg;
             rule.version_selector = ver;
         }
@@ -64,32 +60,30 @@ exports.handler = async ({ axios, specfile }) => {
 
         if(!rule.negate){
             for(const package of packages){
-                if (does_match(package, rule)) matches.push(package);
+                if(does_match(package, rule))
+                    matches.push(package)
             }
 
-            const latest_matches = matches.filter(pkg => {
+            const latest_matches = matches.filter(
+                pkg => {
                     const versions = matches
                         .filter(x=>x.language == pkg.language)
-                    .map(x => x.language_version)
+                        .map(x=>x.language_version).sort(semver.rcompare);
-                    .sort(semver.rcompare);
+                    return versions[0] == pkg.language_version
-                return versions[0] == pkg.language_version;
+                }
-            });
+            );
 
             for(const match of latest_matches){
-                if (
+                if(!ensure_packages.find(pkg => pkg.language == match.language && pkg.language_version == match.language_version))
-                    !ensure_packages.find(
+                    ensure_packages.push(match)
-                        pkg =>
-                            pkg.language == match.language &&
-                            pkg.language_version == match.language_version
-                    )
-                )
-                    ensure_packages.push(match);
             }
         }else{
             ensure_packages = ensure_packages.filter(
                 pkg => !does_match(pkg, rule)
-            );
+            )
         }
+
+        
     }
 
     const operations = [];
@@ -97,87 +91,70 @@ exports.handler = async ({ axios, specfile }) => {
     for(const package of ensure_packages){
         if(!package.installed)
             operations.push({
-                type: 'install',
+                type: "install",
                 package: package.language,
-                version: package.language_version,
+                version: package.language_version
             });
     }
 
     for(const installed_package of installed){
-        if (
+        if(!ensure_packages.find(
-            !ensure_packages.find(
+            pkg => pkg.language == installed_package.language &&
-                pkg =>
-                    pkg.language == installed_package.language &&
             pkg.language_version == installed_package.language_version
-            )
+        ))
-        )
             operations.push({
-                type: 'uninstall',
+                type: "uninstall",
                 package: installed_package.language,
-                version: installed_package.language_version,
+                version: installed_package.language_version
-            });
+            })
     }
 
-    console.log(chalk.bold.yellow('Actions'));
+    console.log(chalk.bold.yellow("Actions"))
     for(const op of operations){
-        console.log(
+        console.log((op.type == "install" ? chalk.green("Install") : chalk.red("Uninstall")) + ` ${op.package} ${op.version}`)
-            (op.type == 'install'
-                ? chalk.green('Install')
-                : chalk.red('Uninstall')) + ` ${op.package} ${op.version}`
-        );
     }
 
     if(operations.length == 0){
-        console.log(chalk.gray('None'));
+        console.log(chalk.gray("None"))
     }
 
     for(const op of operations){
-        if (op.type == 'install') {
+        if(op.type == "install"){
             try{
                 const install = await axios.post(`/api/v2/packages`, {
                     language: op.package,
-                    version: op.version,
+                    version: op.version
                 });
 
                 if(!install.data.language)
                     throw new Error(install.data.message); // Go to exception handler
 
-                console.log(
+                console.log(chalk.bold.green("Installed"), op.package, op.version)
-                    chalk.bold.green('Installed'),
+
-                    op.package,
-                    op.version
-                );
             }catch(e){
-                console.log(
+                console.log(chalk.bold.red("Failed to install") + ` ${op.package} ${op.version}:`, e.message)
-                    chalk.bold.red('Failed to install') +
-                        ` ${op.package} ${op.version}:`,
-                    e.message
-                );
             }
-        } else if (op.type == 'uninstall') {
+        }
+        else if(op.type == "uninstall"){
             try{
                 const install = await axios.delete(`/api/v2/packages`, {
                     data: {
                         language: op.package,
-                        version: op.version,
+                        version: op.version
-                    },
+                    }
                 });
 
                 if(!install.data.language)
                     throw new Error(install.data.message); // Go to exception handler
 
-                console.log(
+                console.log(chalk.bold.green("Uninstalled"), op.package, op.version)
-                    chalk.bold.green('Uninstalled'),
+
-                    op.package,
-                    op.version
-                );
             }catch(e){
-                console.log(
+                console.log(chalk.bold.red("Failed to uninstall") + ` ${op.package} ${op.version}:`, e.message)
-                    chalk.bold.red('Failed to uninstall') +
-                        ` ${op.package} ${op.version}:`,
-                    e.message
-                );
             }
         }
     }
-};
+    
+
+
+}

cli/commands/ppman_commands/uninstall.js

@@ -6,36 +6,31 @@ exports.describe = 'Uninstalls the named package';
 
 //Splits the package into it's language and version
 function split_package(package) {
-    [language, language_version] = package.split('=');
+    [language, language_version] = package.split("=")
 
     res = {
         language: language,
-        version: language_version || '*',
+        version: language_version || "*"
     };
-    return res;
+    return res
 }
 
 const msg_format = {
-    color: p =>
+    color: p => `${p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')} Uninstallation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-        `${
+    monochrome: p => `Uninstallation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-            p.language ? chalk.green.bold('✓') : chalk.red.bold('❌')
+    json: JSON.stringify
-        } Uninstallation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-    monochrome: p =>
-        `Uninstallation ${p.language ? 'succeeded' : 'failed: ' + p.message}`,
-    json: JSON.stringify,
 };
 
 exports.handler = async ({ axios, packages }) => {
     const requests = packages.map(package => split_package(package));
     for (request of requests) {
         try {
-            const uninstall = await axios.delete(`/api/v2/packages`, {
+
-                data: request,
+            const uninstall = await axios.delete(`/api/v2/packages`, { data: request });
-            });
 
             console.log(msg_format.color(uninstall.data));
         } catch ({ response }) {
-            console.error(response.data.message);
+            console.error(response.data.message)
+        }
     }
 }
-};

cli/index.js

@@ -6,8 +6,8 @@ const axios_instance = argv => {
     argv.axios = axios.create({
         baseURL: argv['piston-url'],
         headers: {
-            'Content-Type': 'application/json',
+            'Content-Type': 'application/json'
-        },
+        }
     });
 
     return argv;
@@ -18,11 +18,12 @@ require('yargs')(process.argv.slice(2))
         alias: ['u'],
         default: 'http://127.0.0.1:2000',
         desc: 'Piston API URL',
-        string: true,
+        string: true
     })
     .middleware(axios_instance)
     .scriptName('piston')
     .commandDir('commands')
     .demandCommand()
     .help()
-    .wrap(72).argv;
+    .wrap(72)
+    .argv;

docker-compose.dev.yaml

@@ -1,4 +1,4 @@
-version: '3.2'
+version: "3.2"
 
 services:
   api:
@@ -19,6 +19,6 @@ services:
   repo: # Local testing of packages
     build: repo
     container_name: piston_repo
-        command: ['--no-build'] # Don't build anything
+    command: ["--no-build"] # Don't build anything
     volumes:
       - .:/piston

docs/configuration.md

@@ -124,7 +124,6 @@ Maximum size for a singular file written to disk.
 Resists against large file writes to exhaust disk space.
 
 ## Compile/Run timeouts
-
 ```yaml
 key:
   - PISTON_COMPILE_TIMEOUT
@@ -134,7 +133,6 @@ key:
   - PISTON_RUN_TIMEOUT
 default: 3000
 ```
-
 The maximum time that is allowed to be taken by a stage in milliseconds.
 Use -1 for unlimited time.
 
@@ -179,9 +177,7 @@ default: {}
 
 Per-language overrides/exceptions for the each of `max_process_count`, `max_open_files`, `max_file_size`,
 `compile_memory_limit`, `run_memory_limit`, `compile_timeout`, `run_timeout`, `output_max_size`. Defined as follows:
-
 ```
 PISTON_LIMIT_OVERRIDES={"c++":{"max_process_count":128}}
 ```
-
 This will give `c++` a max_process_count of 128 regardless of the configuration.

package-lock.json

@@ -2,31 +2,5 @@
   "name": "piston",
   "lockfileVersion": 2,
   "requires": true,
-    "packages": {
+  "packages": {}
-        "": {
-            "devDependencies": {
-                "prettier": "2.4.1"
-            }
-        },
-        "node_modules/prettier": {
-            "version": "2.4.1",
-            "resolved": "https://registry.npmjs.org/prettier/-/prettier-2.4.1.tgz",
-            "integrity": "sha512-9fbDAXSBcc6Bs1mZrDYb3XKzDLm4EXXL9sC1LqKP5rZkT6KRr/rf9amVUcODVXgguK/isJz0d0hP72WeaKWsvA==",
-            "dev": true,
-            "bin": {
-                "prettier": "bin-prettier.js"
-            },
-            "engines": {
-                "node": ">=10.13.0"
-            }
-        }
-    },
-    "dependencies": {
-        "prettier": {
-            "version": "2.4.1",
-            "resolved": "https://registry.npmjs.org/prettier/-/prettier-2.4.1.tgz",
-            "integrity": "sha512-9fbDAXSBcc6Bs1mZrDYb3XKzDLm4EXXL9sC1LqKP5rZkT6KRr/rf9amVUcODVXgguK/isJz0d0hP72WeaKWsvA==",
-            "dev": true
-        }
-    }
 }

package.json

@@ -1,5 +1 @@
-{
+{}
-    "devDependencies": {
-        "prettier": "2.4.1"
-    }
-}

packages/deno/1.7.5/test.deno.ts

@@ -1 +1 @@
-console.log('OK');
+console.log("OK")

packages/mono/6.12.0/compile

@@ -1,13 +1,20 @@
 #!/bin/bash
 
+check_errors () {
+    grep -q 'error [A-Z]\+[0-9]\+:' check.txt && cat check.txt 1>&2 || cat check.txt
+    rm check.txt
+}
+
 case "${PISTON_LANGUAGE}" in
     csharp)
         rename 's/$/\.cs/' "$@" # Add .cs extension
-        csc -out:out *.cs
+        csc -out:out *.cs > check.txt
+        check_errors
         ;;
     basic)
         rename 's/$/\.vb/' "$@" # Add .vb extension
-        vbnc -out:out *.vb
+        vbnc -out:out *.vb > check.txt
+        check_errors
         ;;
     *)
         echo "How did you get here? (${PISTON_LANGUAGE})"

packages/mono/6.12.0/metadata.json

@@ -8,13 +8,7 @@
         },
         {
             "language": "basic",
-            "aliases": [
+            "aliases": ["vb", "mono-vb", "mono-basic", "visual-basic", "visual basic"]
-                "vb",
-                "mono-vb",
-                "mono-basic",
-                "visual-basic",
-                "visual basic"
-            ]
         }
     ]
 }

packages/node/15.10.0/test.js

@@ -1 +1 @@
-console.log('OK');
+console.log("OK")

packages/node/16.3.0/test.js

@@ -1 +1 @@
-console.log('OK');
+console.log("OK")

packages/typescript/4.2.3/run

@@ -2,7 +2,7 @@
 
 # Put instructions to run the runtime
 
-CODE=$1.js
+CODE=$(sed 's/ts$/js/' <<<"$1")
 shift
 
 node $CODE "$@"

packages/typescript/4.2.3/test.ts

@@ -1 +1 @@
-console.log('OK');
+console.log("OK")

packages/vyxal/2.4.1/build.sh

@@ -1,18 +0,0 @@
-#!/usr/bin/env bash
-
-# get Python source
-source ../../python/3.9.4/build.sh
-
-# add regex and pwn modules
-bin/pip3 install regex pwn
-
-# make vyxal directory
-mkdir vyxal
-cd vyxal
-
-# Vyxal install
-curl -L "https://github.com/Vyxal/Vyxal/archive/refs/tags/v2.4.1.tar.gz" -o vyxal.tar.xz
-tar xf vyxal.tar.xz --strip-components=1
-rm vyxal.tar.xz
-
-cd ..

packages/vyxal/2.4.1/environment

@@ -1,8 +0,0 @@
-#!/usr/bin/env bash
-
-# Python and Vyxal path
-export PATH=$PWD/bin:$PATH
-export VYXAL_PATH=$PWD/vyxal
-
-# export term to fix curses warning
-export TERM=xterm

packages/vyxal/2.4.1/metadata.json

@@ -1,5 +0,0 @@
-{
-    "language": "vyxal",
-    "version": "2.4.1",
-    "aliases": []
-}

packages/vyxal/2.4.1/run

@@ -1,3 +0,0 @@
-#!/usr/bin/env bash
-
-python3 "$VYXAL_PATH"/Vyxal.py "$1"

packages/vyxal/2.4.1/test.vyxal

@@ -1 +0,0 @@
-`OK

piston

@@ -38,7 +38,6 @@ case $1 in
         echo " clean-repo                     Remove all packages from local repo"
         echo " build-pkg <package> <version>  Build a package"
         echo " rebuild                        Build and restart the docker container"
-        echo " lint                           Lint the codebase using prettier"
 
         else
 
@@ -54,11 +53,7 @@ case $1 in
     logs) docker_compose logs -f ;;
 
     restart) docker_compose restart ;;
-    start)
+    start) docker_compose up -d ;;
-        rm -f .git/hooks/pre-commit
-        ln -s $(realpath $(dirname "$0"))/pre-commit .git/hooks/pre-commit
-        docker_compose up -d
-        ;;
     stop) docker_compose down ;;
     bash) docker_compose exec api /bin/bash ;;
 
@@ -80,11 +75,6 @@ case $1 in
         docker build repo -t piston-repo-builder
         docker run --rm -v "$(realpath $(dirname "$0")):/piston" piston-repo-builder --no-server $PKGSLUG
         ;;
-
-    lint)
-        npm install
-        npx prettier --ignore-unknown --write .
-        ;;
     *)
         cd cli
         npm i > /dev/null

pre-commit

@@ -1,15 +0,0 @@
-#!/bin/sh
-
-echo "Linting staged files..."
-npm install
-
-FILES=$(git diff --cached --name-only --diff-filter=ACMR | sed 's| |\\ |g')
-[ -z "$FILES" ] && exit 0
-
-# Prettify all selected files
-echo "$FILES" | xargs npx prettier --ignore-unknown --write
-
-# Add back the modified/prettified files to staging
-echo "$FILES" | xargs git add
-
-exit 0

readme.md

@@ -47,10 +47,9 @@ While we are accepting pull requests for Hacktoberfest, we will reject any low-q
 If we see PR abuse for Hacktoberfest, we will stop providing Hacktoberfest approval for pull requests.
 
 We are accepting PRs for:
-
+* Packages - updating package versions, adding new packages
--   Packages - updating package versions, adding new packages
+* Documentation updates
--   Documentation updates
+* CLI/API improvements - please discuss these with us in the Discord first
--   CLI/API improvements - please discuss these with us in the Discord first
 
 Any queries or concerns, ping @HexF#0015 in the Discord.
 
@@ -390,7 +389,6 @@ Content-Type: application/json
 `basic`,
 `basic.net`,
 `vlang`,
-`vyxal`,
 `yeethon`,
 `zig`,