Add the --json option to the `fan2go detect` command

[dongfengweixiao]

Describe the solution you'd like

I am working on adding a configuration page for fan2go, drawing from the luci-app-fan2go. Processing the current output of the fan2go detect command poses some challenges. Ideally, I would like to implement a feature that allows the results to be output in JSON format.

[markusressel]

Hi @dongfengweixiao 👋

The fan2go CLI output was not really intended to be consumed programmatically.
I was thinking that the API would be sufficient for third party applications to gather information, but of course this doesn't work if you want to start with an empty config and use fan2go detect to populate a config or a UI. 🤔

Adding a --json flag to the detect command is probably a good first step to find out whether we need more like this. If you want to work on such a thing, feel free to create a (draft) PR and ping me :)

[markusressel]

Update: I had a look at this some time ago, and it turned out to be not as simple as I would have thought.
Providing a custom JSON structure as an "API" to consume for CLI clients requires a lot of "initial specification work" and probably also a bit of refactoring. Its not impossible, but more work than I currently have time for. It stays on the list.

[dongfengweixiao]

Update: I had a look at this some time ago, and it turned out to be not as simple as I would have thought.
Providing a custom JSON structure as an "API" to consume for CLI clients requires a lot of "initial specification work" and probably also a bit of refactoring. Its not impossible, but more work than I currently have time for. It stays on the list.

Thank you for continuing to pay attention to the issue I raised. I am currently migrating from spksrc to fpksrc. While trying to package fan2go as an fpk for FeiNiu OS, I came up with a method to have AI read the output of fan2go detect and convert it to JSON. This method has been tested and works.