porffor 0.30.1 → 0.30.2

package/compiler/codegen.js

@@ -1800,6 +1800,13 @@ const disposeLeftover = wasm => {
 const generateExp = (scope, decl) => {
   const expression = decl.expression;
 
+  if (expression.type === 'Literal' && typeof expression.value === 'string') {
+    if (expression.value === 'use strict') {
+      scope.strict = true;
+    }
+    return [];
+  }
+
   const out = generate(scope, expression, undefined, undefined, Prefs.optUnused);
   disposeLeftover(out);
 
@@ -3607,10 +3614,8 @@ const generateAssign = (scope, decl, _global, _name, valueUnused = false) => {
   }
 
   if (local === undefined) {
-    // todo: this should be a sloppy mode only thing
-
-    // only allow = for this
-    if (op !== '=') return internalThrow(scope, 'ReferenceError', `${unhackName(name)} is not defined`);
+    // only allow = for this, or if in strict mode always throw
+    if (op !== '=' || scope.strict) return internalThrow(scope, 'ReferenceError', `${unhackName(name)} is not defined`, true);
 
     if (type != 'Identifier') {
       const tmpName = '#rhs' + uniqId();
@@ -3623,6 +3628,8 @@ const generateAssign = (scope, decl, _global, _name, valueUnused = false) => {
     }
 
     if (Object.hasOwn(builtinVars, name)) {
+      if (scope.strict) return internalThrow(scope, 'TypeError', `Cannot assign to non-writable global ${name}`, true);
+
       // just return rhs (eg `NaN = 2`)
       return generate(scope, decl.right);
     }
@@ -3749,7 +3756,6 @@ const generateUnary = (scope, decl) => {
 
         // if ReferenceError (undeclared var), ignore and return true. otherwise false
         if (!out[1]) {
-          // todo: throw in strict mode
           // exists
           toReturn = false;
         } else {
@@ -4022,7 +4028,7 @@ const generateForOf = (scope, decl) => {
   // setup local for left
   let setVar;
   if (decl.left.type === 'Identifier') {
-    // todo: should be sloppy mode only
+    if (scope.strict) return internalThrow(scope, 'ReferenceError', `${decl.left.name} is not defined`);
     setVar = generateVarDstr(scope, 'var', decl.left, { type: 'Identifier', name: tmpName }, undefined, true);
   } else {
     // todo: verify this is correct
@@ -4389,7 +4395,7 @@ const generateForIn = (scope, decl) => {
 
   let setVar;
   if (decl.left.type === 'Identifier') {
-    // todo: should be sloppy mode only
+    if (scope.strict) return internalThrow(scope, 'ReferenceError', `${decl.left.name} is not defined`);
     setVar = generateVarDstr(scope, 'var', decl.left, { type: 'Identifier', name: tmpName }, undefined, true);
   } else {
     // todo: verify this is correct
@@ -5638,7 +5644,8 @@ const generateFunc = (scope, decl) => {
       // not arrow function or main
       (decl.type && decl.type !== 'ArrowFunctionExpression' && decl.type !== 'Program') &&
       // not async or generator
-      !decl.async && !decl.generator
+      !decl.async && !decl.generator,
+    strict: scope.strict
   };
 
   funcIndex[name] = func.index;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "porffor",
   "description": "a basic experimental wip aot optimizing js -> wasm engine/compiler/runtime in js",
-  "version": "0.30.1+616eeaaf5",
+  "version": "0.30.2+2e9f2148f",
   "author": "CanadaHonk",
   "license": "MIT",
   "scripts": {},

package/runner/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import fs from 'node:fs';
-globalThis.version = '0.30.1+616eeaaf5';
+globalThis.version = '0.30.2+2e9f2148f';
 
 // deno compat
 if (typeof process === 'undefined' && typeof Deno !== 'undefined') {
@@ -115,7 +115,10 @@ if (process.argv.length >= 4) {
   if (evalIndex !== -1) {
     source = process.argv[evalIndex + 1];
     if (source) {
-      if (source.startsWith('"') || source.startsWith("'")) source = source.slice(1, -1);
+      // todo: this isn't entirely right, because shells should do the quoting for us but works well enough, see below as well
+      if ((source.startsWith('"') || source.startsWith("'")) && (source.endsWith('"') || source.endsWith("'"))) {
+        source = source.slice(1, -1);
+      }
       process.argv.splice(evalIndex, 2); // remove flag and value
     }
   }
@@ -126,7 +129,9 @@ if (process.argv.length >= 4) {
     process.argv.push('--no-opt-unused');
     source = process.argv[printIndex + 1];
     if (source) {
-      if (source.startsWith('"') || source.startsWith("'")) source = source.slice(1, -1);
+      if ((source.startsWith('"') || source.startsWith("'")) && (source.endsWith('"') || source.endsWith("'"))) {
+        source = source.slice(1, -1);
+      }
       process.argv.splice(printIndex, 2); // remove flag and value
     }