fix(scripts): declare Runnable entry points with var so modern engines find them

d2bs/kolbot/libs/scripts/Abaddon.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Abaddon = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Abaddon = new Runnable(
   function Abaddon () {
     Pather.useWaypoint(sdk.areas.FrigidHighlands);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/AncientTunnels.js

@@ -5,7 +5,8 @@
 *
 */
 
-const AncientTunnels = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var AncientTunnels = new Runnable(
   function AncientTunnels () {
     Pather.useWaypoint(sdk.areas.LostCity);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Andariel.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Andariel = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Andariel = new Runnable(
   function Andariel () {
     const killAndariel = function () {
       let target = Game.getMonster(sdk.monsters.Andariel);

d2bs/kolbot/libs/scripts/AutoBaal.js

@@ -15,7 +15,8 @@
 *   - should this listen for baal death packet?
 */
 
-const AutoBaal = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var AutoBaal = new Runnable(
   function AutoBaal () {
     // internal variables
     let baalCheck, throneCheck, hotCheck, leader; // internal variables

d2bs/kolbot/libs/scripts/AutoChaos.js

@@ -7,7 +7,8 @@
 */
 
 
-const AutoChaos = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var AutoChaos = new Runnable(
   function AutoChaos() {
     let taxi = "";
     let tpID = 0;

d2bs/kolbot/libs/scripts/Baal.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Baal = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Baal = new Runnable(
   function Baal () {
     const announce = function () {
       let count, string, souls, dolls;

d2bs/kolbot/libs/scripts/BaalAssistant.js

@@ -12,7 +12,8 @@
  * - override Misc.getShrinesInArea to end when we recieve safeCheck message
  */
 
-const BaalAssistant = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var BaalAssistant = new Runnable(
   function BaalAssistant () {
     let Leader = Config.Leader;
     let Helper = Config.BaalAssistant.Helper;

d2bs/kolbot/libs/scripts/BaalHelper.js

@@ -7,7 +7,8 @@
 
 /** @typedef {ScriptContext & { chatEvent: (nick: string, msg: string) => void }} BaalHelperContext */
 
-const BaalHelper = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var BaalHelper = new Runnable(
   /** @param {BaalHelperContext} ctx */
   function BaalHelper (ctx) {
     let quitFlag = false;

d2bs/kolbot/libs/scripts/BattleOrders.js

@@ -7,7 +7,8 @@
 
 // todo - define bo-er name, so bots who are getting bo know who is supposed to give it
 // todo - use profile <-> profile communication so we don't need to set char names, Maybe shout global?
-const BattleOrders = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var BattleOrders = new Runnable(
   function BattleOrders () {
     this.gaveBo = false;
     /** @type {Set<string>} */

d2bs/kolbot/libs/scripts/BattlemaidSarina.js

@@ -5,7 +5,8 @@
 *
 */
 
-const BattlemaidSarina = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var BattlemaidSarina = new Runnable(
   function BattlemaidSarina () {
     Pather.useWaypoint(sdk.areas.KurastBazaar);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Bishibosh.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Bishibosh = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Bishibosh = new Runnable(
   function Bishibosh () {
     Pather.useWaypoint(sdk.areas.ColdPlains);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/BoBarbHelper.js

@@ -6,7 +6,8 @@
 *
 */
 
-const BoBarbHelper = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var BoBarbHelper = new Runnable(
   function BoBarbHelper () {
     if (!me.barbarian && Config.BoBarbHelper.Mode !== 0) return true;
 

d2bs/kolbot/libs/scripts/BoneAsh.js

@@ -5,7 +5,8 @@
 *
 */
 
-const BoneAsh = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var BoneAsh = new Runnable(
   function BoneAsh () {
     Pather.useWaypoint(sdk.areas.InnerCloister);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Bonesaw.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Bonesaw = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Bonesaw = new Runnable(
   function Bonesaw () {
     Pather.useWaypoint(sdk.areas.GlacialTrail);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/ChestMania.js

@@ -7,7 +7,8 @@
 
 // todo - if we have run ghostsbusters before this then some of these areas don't need to be re-run
 
-const ChestMania = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var ChestMania = new Runnable(
   function ChestMania () {
     Config.OpenChests._enabled = Config.OpenChests.Enabled;
     Config.OpenChests.Enabled = true;

d2bs/kolbot/libs/scripts/ClassicChaosAssistant.js

@@ -7,7 +7,8 @@
 
 // redo this, maybe different keys or chat commands instead?
 
-const ClassicChaosAssistant = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var ClassicChaosAssistant = new Runnable(
   function ClassicChaosAssistant () {
     let stargo, infgo, seisgo, vizgo, infseal, seisseal, vizseal, diablopickup, normalpickup = false;
 

d2bs/kolbot/libs/scripts/ClearAnyArea.js

@@ -5,7 +5,8 @@
 *
 */
 
-const ClearAnyArea = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var ClearAnyArea = new Runnable(
   function ClearAnyArea () {
     for (let area of Config.ClearAnyArea.AreaList) {
       try {

d2bs/kolbot/libs/scripts/Coldcrow.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Coldcrow = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Coldcrow = new Runnable(
   function Coldcrow () {
     Pather.useWaypoint(sdk.areas.ColdPlains);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Coldworm.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Coldworm = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Coldworm = new Runnable(
   function Coldworm () {
     Pather.useWaypoint(sdk.areas.FarOasis);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/ControlBot.js

@@ -9,7 +9,8 @@
 
 /** @typedef {ScriptContext & { cleanup: () => void }} ControlBotContext */
 
-const ControlBot = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var ControlBot = new Runnable(
   /** @param {ControlBotContext} ctx */
   function ControlBot (ctx) {
     const thankYouMessages = [

d2bs/kolbot/libs/scripts/Corpsefire.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Corpsefire = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Corpsefire = new Runnable(
   function Corpsefire () {
     Pather.useWaypoint(sdk.areas.ColdPlains);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Countess.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Countess = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Countess = new Runnable(
   function Countess () {
     Pather.useWaypoint(sdk.areas.BlackMarsh);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Cows.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Cows = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Cows = new Runnable(
   function Cows () {
     const getLeg = function () {
       if (me.wirtsleg) return me.wirtsleg;

d2bs/kolbot/libs/scripts/Crafting.js

@@ -8,7 +8,8 @@
 let info;
 let gameRequest = false;
 
-const Crafting = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Crafting = new Runnable(
   function Crafting () {
     info = CraftingSystem.getInfo();
 

d2bs/kolbot/libs/scripts/CreepingFeature.js

@@ -5,7 +5,8 @@
 *
 */
 
-const CreepingFeature = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var CreepingFeature = new Runnable(
   function CreepingFeature () {
     Town.goToTown(2);
 

d2bs/kolbot/libs/scripts/CrushTele.js

@@ -5,7 +5,8 @@
 *
 */
 
-const CrushTele = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var CrushTele = new Runnable(
   function CrushTele () {
     let go = false;
 

d2bs/kolbot/libs/scripts/DeveloperMode.js

@@ -5,7 +5,8 @@
 *
 */
 
-const DeveloperMode = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var DeveloperMode = new Runnable(
   function DeveloperMode () {
     const className = sdk.player.class.nameOf(me.classid);
 

d2bs/kolbot/libs/scripts/Diablo.js

@@ -9,7 +9,8 @@
 *
 */
 
-const Diablo = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Diablo = new Runnable(
   function Diablo () {
     Pather._teleport = Pather.teleport;
     Common.Diablo.clearRadius = Config.Diablo.ClearRadius;

d2bs/kolbot/libs/scripts/DiabloHelper.js

@@ -5,7 +5,8 @@
 *
 */
 
-const DiabloHelper = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var DiabloHelper = new Runnable(
   function DiabloHelper () {
     Common.Diablo.waitForGlow = true;
     Common.Diablo.clearRadius = Config.DiabloHelper.ClearRadius;

d2bs/kolbot/libs/scripts/Duriel.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Duriel = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Duriel = new Runnable(
   function Duriel () {
     const killDuriel = function () {
       let target = Misc.poll(function () {

d2bs/kolbot/libs/scripts/Eldritch.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Eldritch = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Eldritch = new Runnable(
   function Eldritch () {
     Pather.useWaypoint(sdk.areas.FrigidHighlands);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Endugu.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Endugu = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Endugu = new Runnable(
   function Endugu () {
     Pather.useWaypoint(sdk.areas.FlayerJungle);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Eyeback.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Eyeback = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Eyeback = new Runnable(
   function Eyeback () {
     Pather.useWaypoint(sdk.areas.ArreatPlateau);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Fangskin.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Fangskin = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Fangskin = new Runnable(
   function Fangskin () {
     Pather.useWaypoint(sdk.areas.LostCity);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Follower.js

@@ -65,7 +65,8 @@
 *
 */
 
-const Follower = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Follower = new Runnable(
   function Follower () {
     const QuestData = require("../core/GameData/QuestData");
     /** @type {Map<number, Area>} */

d2bs/kolbot/libs/scripts/Frozenstein.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Frozenstein = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Frozenstein = new Runnable(
   function Frozenstein () {
     Pather.useWaypoint(sdk.areas.CrystalizedPassage);
     Precast.doPrecast(true);

d2bs/kolbot/libs/scripts/Gamble.js

@@ -5,7 +5,8 @@
 *
 */
 
-const Gamble = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var Gamble = new Runnable(
   function Gamble () {
     let info = Gambling.getInfo();
     if (!info) throw new Error("Bad Gambling System config.");

d2bs/kolbot/libs/scripts/GemHunter.js

@@ -12,7 +12,8 @@
  * - We should also then keep track of where the shrine was, (I don't remember if gem shrines regen, so check this)
  * - Take into account the next area and sort the shrines to bring us to the exit if its connected
  */
-const GemHunter = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var GemHunter = new Runnable(
   function GemHunter () {
     if (!Town.prepareForGemShrine()) {
       console.log("ÿc4GemHunterÿc0: no gems in inventory - aborting.");

d2bs/kolbot/libs/scripts/GetCube.js

@@ -5,7 +5,8 @@
 *
 */
 
-const GetCube = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var GetCube = new Runnable(
   function GetCube () {
     // Can't get the cube if we can't access the act
     if (!me.accessToAct(2)) return false;

d2bs/kolbot/libs/scripts/GetEssences.js

@@ -6,7 +6,8 @@
 *
 */
 
-const GetEssences = new Runnable(
+// eslint-disable-next-line no-var -- Loader resolves this via global[script]; a top-level const is not a global property on modern JS engines
+var GetEssences = new Runnable(
   function GetEssences () {
     /**
      * @param {number} essence