Failed to parse JSON file.

[zhenzhenChange]

Describe the bug

parse json fail

Reproduction

https://github.com/zhenzhenChange/vite-parse-json-fail.git

System Info

System:
    OS: Windows 10 10.0.19044
    CPU: (6) x64 Intel(R) Core(TM) i5-9400 CPU @ 2.90GHz
    Memory: 6.55 GB / 15.86 GB
  Binaries:
    Node: 16.13.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.17 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.1.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (102.0.1245.41)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @vitejs/plugin-react: ^1.3.0 => 1.3.2
    vite: ^2.9.9 => 2.9.12

Used Package Manager

pnpm

Logs

Just run the project

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[sapphi-red]

This was because src/city.json starts with a BOM.
For a workaround, you could save it with UTF-8 (without BOM).

[zhenzhenChange]

Wow, yes indeed, I downloaded this file from the internet and at first I thought it was a failure to parse due to the huge amount of data (4w data at first).

I created a new .json file and the problem was solved.

But this BOM tagging issue should be manageable, right? I checked the information, we are able to remove the BOM marker by logical judgment. But I'm not sure if there will be other similar cases?

[zhenzhenChange]

Or force conversion of the file's encoding format before parsing? But that doesn't seem to be a good solution.

[sapphi-red]

Yes I can be handled like #3171.

[zhenzhenChange]

Wow, ok, thank you, I can do this job and launch a pr right after work.

[zhenzhenChange]

By the way, the logic extraction in #3171 is used as a general method.