invalid json file validated since v4.40.2

[sda399]

Describe the bug
hi folks
invalid json is parsed and fixed by yq, this breaks file validation tests based on yq

Version of yq: 4.40.2
Operating system: linux
Installed via: github binary

Input Json

{
  "hello": "value"
  "foo": "bar"
}

Command

for V in 4.35.1 4.40.2; do
  curl -L --silent --output /tmp/yq "https://github.com/mikefarah/yq/releases/download/v$V/yq_linux_amd64"
  chmod +x /tmp/yq
  echo "==="
  /tmp/yq --version
  echo '{"hello": "value"\n "foo": "bar"}' | /tmp/yq -e -pj -oj
  rm /tmp/yq
done

Actual behavior

{
  "hello": "value",
  "foo": "bar"
}

Expected behavior
expected an error on parsing

Error: bad file '-': invalid character '"' after object key:value pair

[mikefarah]

Oh interesting! The underlying json tokeniser I'm using (goccy/json) is responsible for that - feel free to raise an issue against that repo

[sda399]

hum,
they seem to have a unit test for it.
https://github.com/goccy/go-json/blame/9872089c316cfe2d0f29b331b75d45bf6d522d96/json_test.go#L257

{`{"X": "foo" "Y": "bar"}`, json.NewSyntaxError("invalid character '\"' after object key:value pair", 13)},

[mikefarah]

Hmm I can see that that error message comes from their indenter code in their encoder...whereas the problem I'm having is in decoding 😕 I'll dig in some more

[mikefarah]

Ok I think I found the culprit, raised issue here: goccy/go-json#548