[MC-131636] One additional character at the end is allowed in JSON text components

This command:

/tellraw @s {"text":"a"}@

works, but it shouldn't. It outputs "a". Same with any other character at the end. But not two.

Linked issues

is duplicated by 4

MC-138340 Trailing data in chat component is valid Resolved

MC-222880 Brigadier Json Argument accepts an extra character Resolved

MC-234005 JSON parse error Resolved

MC-267118 Text component argument reads one char more than expected Resolved

Attachments

Comments 14

qwerty23495 2018-06-23T17:04:44Z

Can confirm in 1.13-pre3.

Oval 2018-09-21T03:28:51Z

Confirmed for 1.13.1.

Avoma 2021-01-13T14:38:19Z

Can confirm that this issue is still present in 20w51a.

Avoma 2021-02-27T13:43:37Z

Can confirm in 21w08b. Video attached.

Avoma 2021-03-01T17:14:01Z

Can confirm in 1.16.5.

4 more comments

SolidBlock 2023-12-04T03:15:42Z

The reason is when reading a JSON text component, the position of the end cursor is one more than expected.

SolidBlock 2024-01-24T11:14:12Z

The issue lies on the method with remapped name class_9010.method_55376 (
ParserUtils.getPos), which has an extra +1. In versions 1.20.2 and below, the issue lies on class_2561.class_2562.method_10880 (Component.Serializer.getPos).

SolidBlock 2024-04-08T06:04:26Z

Confirmed not to happen for 24w14a.

SolidBlock 2024-04-12T10:55:03Z

This is fixed since 24w10a

CreeperFriend 2024-04-15T13:50:42Z

Can confirm fixed in 24w10a. The command is not accepted anymore.