(FEAT) Add function parsepson

[david22swan]

This commit adds a new function to parse PSON formatted JSON.
This functionality was formally covered by parsejson which, despite its name, defaulted to PSON.
However this has been changed so that the function now accurately uses JSON, with this function covering the potential need for PSON too specifically be used.

[puppet-community-rangefinder]

parsepson is a function

that may have no external impact to Forge modules.

This module is declared in 318 of 579 indexed public Puppetfiles.

---

These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

[ekohl]

Could this be written as a modern function rather than the legacy one? You also have parsejson in the PR title and commit message, which should be parsepson.

[david22swan]

Had just copied the original function pre changes, but modernizing it would probably be good.
As for the PR and Commit................ yeh
It was a long day

[pmcmaw]

Approving as failing spec tests are currently present on main.
Following ekohls suggestion of using the newer function style this is something on the teams radar and will be addressed as soon as we have bandwidth.

[ekohl]

Following ekohls suggestion of using the newer function style this is something on the teams radar and will be addressed as soon as we have bandwidth.

That's good, but IMHO we should at least only add new style functions (as we did here). Conversion is lower priority.

[alexjfisher]

Following ekohls suggestion of using the newer function style this is something on the teams radar and will be addressed as soon as we have bandwidth.

That's good, but IMHO we should at least only add new style functions (as we did here). Conversion is lower priority.

And possibly also only allow new functions if they are namespaced too?

[ekohl]

I feel stdlib is always a bit different. All other modules should certainly only have namespaced functions but stdlib is in this weird place. Perhaps best to discuss it in its own issue?