Merge pull request 'Fix bns_admin command registration' (#24) from fix/bns-admin-int-range into main

pack/overrides/kubejs/server_scripts/bns_admin.js

@@ -56,12 +56,20 @@ ServerEvents.commandRegistry(event => {
                     })))
 
             // /bns admin waystone-set <player> <n>
+            // (Argument-bounds aren't exposed via KubeJS's wrapper, so we
+            // validate the range inside the handler instead.)
             .then(Commands.literal('waystone-set')
                 .then(Commands.argument('target', Arguments.PLAYER.create(event))
-                    .then(Commands.argument('count', Arguments.INTEGER.create(event, 0, 1000))
+                    .then(Commands.argument('count', Arguments.INTEGER.create(event))
                         .executes(ctx => {
                             const target = Arguments.PLAYER.getResult(ctx, 'target');
                             const count = Arguments.INTEGER.getResult(ctx, 'count');
+                            if (count < 0 || count > 1000) {
+                                ctx.source.sendSystemMessage(Text.red(
+                                    `count must be 0..1000 (got ${count})`
+                                ));
+                                return 0;
+                            }
                             target.persistentData.putInt('bnsWaystoneCount', count);
                             ctx.source.sendSystemMessage(Text.gold(
                                 `Set ${target.username}'s bnsWaystoneCount = ${count}`

pack/pack.lock.json

@@ -1,7 +1,7 @@
 {
   "$schema": "Brass-and-Sigil pack.lock.json - generated, do not edit by hand unless you know what you are doing",
   "name": "Brass and Sigil",
-  "version": "0.20.0",
+  "version": "0.20.1",
   "minecraft": "1.21.1",
   "loader": {
     "type": "neoforge",