@ball-lang/engine 1.5.2 → 1.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ball-lang/engine",
-  "version": "1.5.2",
+  "version": "1.5.4",
   "description": "Ball language engine — interprets Ball programs in TypeScript/JavaScript. Runs in Node.js and browsers.",
   "type": "module",
   "main": "dist/index.js",
@@ -16,7 +16,7 @@
   "scripts": {
     "build": "tsc",
     "test": "node --experimental-strip-types test/engine_test.ts && node --experimental-strip-types --test test/*.test.ts",
-    "coverage": "c8 --reporter=text --reporter=lcov --include \"src/**\" --exclude \"src/compiled_engine.ts\" --check-coverage --lines 91 --branches 62 --functions 95 --statements 91 npm test",
+    "coverage": "c8 --reporter=text --reporter=lcov --include \"src/**\" --exclude \"src/compiled_engine.ts\" --check-coverage --lines 99 --branches 65 --functions 97 --statements 99 npm test",
     "prepublishOnly": "npm run build",
     "prepack": "npm run build"
   },

package/src/compiled_engine.ts

@@ -3736,6 +3736,7 @@ export class BallEngine {
             return e.key;
           }))];
         }
+        throw new BallRuntimeError(('Cannot access field "keys" on ' + __ball_to_string(((__ball_eq(object, null) ? null : object.runtimeType) ?? 'null'))));
       }
       else if ((__sw === 'values')) {
         if ((typeof object === 'object' && object !== null && !Array.isArray(object) && !(object instanceof BallDouble) && !(object instanceof Set))) {
@@ -3753,6 +3754,7 @@ export class BallEngine {
           }
           return vals;
         }
+        throw new BallRuntimeError(('Cannot access field "values" on ' + __ball_to_string(((__ball_eq(object, null) ? null : object.runtimeType) ?? 'null'))));
       }
       else if ((__sw === 'isNaN')) {
         return _ballNumIsNaN(object);
@@ -7450,19 +7452,27 @@ export class BallEngine {
       }), ['map_keys']: ((i) => {
         const input = i;
         let m = this._stdAsMap(i);
-        if (this._isBallSet(__ball_index(m, 'map'))) {
+        let target = __ball_index(m, 'map');
+        if (this._isBallSet(target)) {
           return [];
         }
-        let result = _ballMapKeysDyn(__ball_index(m, 'map'));
+        if ((!((typeof target === 'object' && target !== null && !Array.isArray(target) && !(target instanceof BallDouble) && !(target instanceof Set))) && !(false /* BallMap is Map in TS */))) {
+          throw new BallRuntimeError('map_keys: expected Map or Set');
+        }
+        let result = _ballMapKeysDyn(target);
         this._trackMemoryAllocation(__ball_mul(result.length, _ballPointerBytes));
         return result;
       }), ['map_values']: ((i) => {
         const input = i;
         let m = this._stdAsMap(i);
-        if (this._isBallSet(__ball_index(m, 'map'))) {
+        let target = __ball_index(m, 'map');
+        if (this._isBallSet(target)) {
           return [];
         }
-        let result = _ballMapValuesDyn(__ball_index(m, 'map'));
+        if ((!((typeof target === 'object' && target !== null && !Array.isArray(target) && !(target instanceof BallDouble) && !(target instanceof Set))) && !(false /* BallMap is Map in TS */))) {
+          throw new BallRuntimeError('map_values: expected Map or Set');
+        }
+        let result = _ballMapValuesDyn(target);
         this._trackMemoryAllocation(__ball_mul(result.length, _ballPointerBytes));
         return result;
       }), ['map_entries']: ((i) => {
@@ -8933,6 +8943,9 @@ export class BallEngine {
         return true;
       }
       else if ((__sw === 'map')) {
+        if (this._isBallSet(value)) {
+          return false;
+        }
         let mapVal = this._stdAsMap(value);
         if ((__ball_eq(mapVal, null) && !((typeof value === 'object' && value !== null && !Array.isArray(value) && !(value instanceof BallDouble) && !(value instanceof Set))))) {
           return false;

package/src/engine_setup.ts

@@ -1709,9 +1709,11 @@ export function createEngineSetup(mod: EngineModule) {
       };
     }
   
-    // Patch __bts (ball-to-string) to unwrap BallFuture/BallGenerator.
-    // The compiled engine sets globalThis.__bts during preamble execution.
-    // We wrap it after the engine runs its preamble.
+    // Patch __bts (ball-to-string) to unwrap BallFuture/BallGenerator, IF the
+    // compiled engine has installed one on globalThis. As of the current
+    // compiled_engine.ts, it never does (grepped: zero assignments to
+    // globalThis.__bts) — this is a defensive no-op guarding against a future
+    // preamble that starts exposing one, not an active code path today.
     const origBts = (globalThis as any).__bts;
     if (typeof origBts === 'function') {
       (globalThis as any).__bts = function(v: any): string {