json parser tries to parse nested toml about markdownlint-cli HOT 4 CLOSED

When attempting to parse a configuration file, all three parsers are tried and the result of the first successful is used. In this case, all three are failing as shown in the error message above. The TOML parser is second in the array (index 1), so the following output is relevant:

Parser 1: Expected "=", [ \t] or [A-Za-z0-9_-] but "." found.

No line/column information is provided, but I think it is referring to the following key name as being invalid because it includes a dot: MD007.indent.

from markdownlint-cli.

Hi @DavidAnson thanks for your answer. I tried a bit around with the setup and made some observations:

• I can confirm, that it's complaining about MD007.indent = 4 line
• the toml parser still somehow works. I can confirm that the remaining config is still used. Seems like it just skips over the not understandable lines

That said, the file example stated above seems to me like a valid toml format: https://toml.io/en/v1.0.0#table
While rewriting the config in the minimal example above would be no problem, the pyproject.toml for most users might be much bigger and there are use cases where avoiding this syntax is more annoying.
My other hooks (mypy, ruff) also reads the pyproject.toml and don't have problems with this kind of syntax.

So it seems to me there is either a bug in the used toml parser or the parser doesn't accept the whole toml spec.

from markdownlint-cli.

This is the TOML parser that is used now: https://www.npmjs.com/package/toml

It says it supports version 0.4.0 of the specification which I confirm prohibits dots in key names: https://toml.io/en/v0.4.0#table

If you know of another well-used, dependency-free TOML parser for Node that supports a more recent version, I can have a look at switching to that.

from markdownlint-cli.

Hi @DavidAnson, I'm not a node/js developer so I also have no experience in that regard.

Searching through npm this one seems the most promising:
https://www.npmjs.com/package/smol-toml

• supports toml 1.0.0
• has recently released versions
• no dependencies
• seems to be used ( although not as many downloads as your current parser)

Edit: Seems like you came to the same conclusion a bit earlier 😄

from markdownlint-cli.