Add fixed field to Result object

[audunolsen]

What is the problem you're trying to solve?

Fix option makes me lazy and doesn't help me become a better coder, because I cannot
see what I did wrong. I want stylelint to raise my awareness of good styling practises while simultaneously speeding up my productivity.

What solution would you like to see?

A fixed field in the result object, or alternatively have each warnings-entry have a "fixed" key.

This would allow me to have logs which may look like

_colors.sass
⚠ 4:7 Expected single space before "{" [fixed]
✖︎ 7:9 Expected a trailing semicolon    [fixed]

[audunolsen]

Here's a workaround for people interested, only drawback is that it call's the linter two times on the same file/code.

let [
  {results : [{warnings}]},
  {results : [{warnings: afterFix}]}    
] = await Promise.all([
  lint(options),
  lint(Object.assign(options, {fix: true}))
]);
    
for (entry of warnings) if (!afterFix
  .map(e => JSON.stringify(e))
  .includes(JSON.stringify(entry))
) entry.fixed = true;

[jeddy3]

@audunolsen Thanks for the request, outlining your use case and using the template.

I didn't think is architecturally achievable right now, but that may change when we revisit the way rules report their fixable violations to address #4203 and #2643. That is a sizeable piece of work that no one has found time to tackle yet.

I'll label this issue as an enhancement. It is also linked to those issues so that, hopefully, any solution to them will also factor in this use case.

[github-actions]

This issue is older than one month. Please ask before opening a pull request, as it may no longer be relevant.