From aa881bfc650b113dc44e662e4f2e251b6086f093 Mon Sep 17 00:00:00 2001 From: Matt Date: Sat, 23 May 2026 15:41:34 +0000 Subject: [PATCH] bns_admin: drop INTEGER range args (KubeJS wrapper doesn't expose them), validate in handler instead. 0.20.0 -> 0.20.1. --- pack/overrides/kubejs/server_scripts/bns_admin.js | 10 +++++++++- pack/pack.lock.json | 2 +- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/pack/overrides/kubejs/server_scripts/bns_admin.js b/pack/overrides/kubejs/server_scripts/bns_admin.js index 50d9501..0cfe746 100644 --- a/pack/overrides/kubejs/server_scripts/bns_admin.js +++ b/pack/overrides/kubejs/server_scripts/bns_admin.js @@ -56,12 +56,20 @@ ServerEvents.commandRegistry(event => { }))) // /bns admin waystone-set + // (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}` diff --git a/pack/pack.lock.json b/pack/pack.lock.json index 9779f16..c065e7b 100644 --- a/pack/pack.lock.json +++ b/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", -- 2.52.0