ava --init should respect package.json indentation

[OmgImAlexis]

Description

Running ava --init adds ava to my package.json but also reformats the whole file to 2 spaces instead of the 4 I have listed in my .editorconfig.

# top-most EditorConfig file
root = true

# Unix-style newlines with a newline ending every file
# and trailing whitespace removed
[*]
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = true

[*.{js,css,json}]
charset = utf-8
indent_style = space
indent_size = 4

# Matches the exact files either package.json or .travis.yml
[{.travis.yml}]
indent_style = space
indent_size = 2

[sindresorhus]

npm doesn't preserve indentation, so wouldn't help much. Most people use npm install --save which makes npm modify the package.json. I would be willing to add this if npm added supported for preserving the existing indentation, but not worth the overhead when it doesn't.

[OmgImAlexis]

Yarn does preserve indentation so that'd be why I never noticed it. Since this and most of your other repos suggest using yarn over npm would you be willing to add it based on that?

[sindresorhus]

Ah, I didn't know that. I think it makes more sense to just preserve the existing indentation though, than specifically read .editorconfig. Could just get the indentation with detect-indent. Pull request welcome.

[yatharthx]

@sindresorhus I am picking this up. But not really sure if we need to make this improvement in ava-init package or should we do it to write-pkg itself? 😄

[sindresorhus]

@yatharthk Good idea, let's add it to write-pkg.

[sindresorhus]

For reference, npm@5 will also preserve the indentation, so this will be much needed soon :)

[OmgImAlexis]

Would it not be better to add an auto option to write-json-file?

[yatharthx]

@sindresorhus Cool, then I'll should open an issue and work for indent-preservation on write-pkg.

[sindresorhus]

Would it not be better to add an auto option to write-json-file?

👍