sandstone

sandstone copied to clipboard

`NBT.stringify` does not escape things correctly when used in JSON (as opposed to commands)

4

`NBT.stringify({ test: '\n' })` should output `"{test:'\n'}"` when used in a JSON file, not `"{test:'\\n'}"`. This input impossible to specify in commands validly though. Minimal reproduction code: ```ts LootTable('test:book', {...

GrantGryczan

`onConflict` config option does not apply to implicitly created functions

1

Practical example:    

GrantGryczan

`TimeArgument` type could be more precise

1

Instead of `number | LiteralUnion`, it could be `` number | `${number}${'t' | 's' | 'd'}` `` (or something equally restrictive that still includes `'1t'`, `'1s'`, and `'1d'` as auto-complete...

GrantGryczan

`BasePathClass.prototype.getResourceName` does not accept substitutions like template tags normally do. It only looks at the template parts of the inputted string.  `name.join()` is the part of the code below...

GrantGryczan

More conventional score method names

1

Many of the current names are very unnatural to type and to read. These should be renamed: * `greaterOrEqualThan` -> `greaterThanOrEqualTo` * `lowerOrEqualThan` -> `lessThanOrEqualTo` * `lowerThan` -> `lessThan` *...

GrantGryczan

Add support for `` basePath`#name` `` as discussed on the Discord server.

GrantGryczan

It should be `scoreboard.objectives.setdisplay`. Nothing else that I know of in the library takes the lowercase of vanilla Minecraft syntax and turns it into camel case where there are officially...

GrantGryczan

The same reasoning that applies to doing this for `execute.if` and `execute.unless` subcommands also applies to these. * `execute.facing.entity` * `execute.positioned.as` * `execute.rotated.as` * `worldborder.damage.amount` * `worldborder.damage.buffer` * `worldborder.warning.distance` *...

GrantGryczan

I expect a function tag to be callable just like a function.

GrantGryczan

It should accept function tags in addition to functions.

GrantGryczan