fix: use iarna/toml instead of toml

[ruben-arts]

Using a better toml parser: https://shared.by.re-becca.org/misc/TOML-SPEC-SUPPORT.html

Fixes a known issue were the following table doesn't work but is valid toml:

[tool.pixi]
dependencies.boltons = "*"

[pavelzw]

You need to add this library to the tsup config

[pavelzw]

I think we should also wrap the readfile and parse toml in a try catch and do core.error if we catch something

setup-pixi/src/options.ts

Line 211 in b684872

const fileContent = readFileSync('pyproject.toml', 'utf-8')

The issue was a bit annoying to spot right now.

[ruben-arts]

Is an core.error enough or should we add more info to it?

[pavelzw]

I would add core.error("failed to parse pyproject.toml") and then throw the error again s.t. we see where this error was thrown. The previous error message (https://discord.com/channels/1082332781146800168/1082338253925003385/1245052133053829182) was not really helpful

All errors are catched and displayed in the outer most run action part

[ruben-arts]

Yeah right! will do.

[ruben-arts]

Haha @pavelzw this is why I wanted a pixi.toml, I'm to spoiled by it ;)

https://github.com/prefix-dev/setup-pixi/pull/118/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R354

[ruben-arts]

Or should we updated to pnpm 9?

And should I make a patch release?

[pavelzw]

Let’s use pnpm 9 and a patch release 👍🏻