Feature request: Node API for 'standard --fix'

[stephenkubovic]

Are there any plans to expose 'standard --fix' in the Node API, something like standard.fixText() and standard.fixFiles to go along with lintText() and lintFiles()?

I'm currently trying to support standard --fix in the Atom plugin for standard formatter. Currently the plugin uses the programatic API from standard-format module, but it's sounding like standard --fix is the way forward so I would like to support this as well (tracking here: standard/atom-standard-formatter#39)

Based on the existing functionality of standard, my approach right now is to --stdin option in combination with --fix in a child process and handle the transformation that way. Obviously a programatic way would be more intuitive.

Let me know your thoughts on this feature (I'd be willing to take a crack at a PR, maybe with a nudge in the right direction and/or agreed API spec).

[feross]

This is already supported! Just pass a {fix: true} option to standard.lintFiles(). I just added this to the documentation.

[stephenkubovic]

I just discovered this 30s ago 👍 Thanks for the prompt reply @feross !

[feross]

Sure thing. 👍

Also, note that {fix: true} is not supported for lintText because ESLint doesn't expose the right functionality. If that's important for your plugin, consider filing an issue with them to see if they think that's a reasonable thing to support.

[stephenkubovic]

Ah, good to know.

I see fix: true is documented as available here https://github.com/Flet/standard-engine#standardenginelinttexttext-opts-callback and in the changelog, but I guess that's just a documentation error.

[feross]

Oh, I guess you're right! Sorry -- I was speaking from memory. The docs are correct.

[feross]

Just updated the standard-engine docs as well: https://github.com/Flet/standard-engine/pull/128/files Thanks for making me take a second look at all this :)