feat: better scripts order for follow npm run order

index.js

@@ -3,6 +3,7 @@ const detectIndent = require('detect-indent')
 const detectNewline = require('detect-newline').graceful
 const gitHooks = require('git-hooks-list')
 const isPlainObject = require('is-plain-obj')
+const sortPackageJsonScripts = require('sort-package-json-scripts').default
 
 const hasOwnProperty = (object, property) =>
  Object.prototype.hasOwnProperty.call(object, property)
@@ -131,31 +132,7 @@ const defaultNpmScripts = new Set([
  'version',
 ])
 
-const sortScripts = onObject(scripts => {
- const names = Object.keys(scripts)
- const prefixable = new Set()
-
- const keys = names
- .map(name => {
- const omitted = name.replace(/^(?:pre|post)/, '')
- if (defaultNpmScripts.has(omitted) || names.includes(omitted)) {
- prefixable.add(omitted)
- return omitted
- }
- return name
- })
- .sort()
-
- const order = keys.reduce(
- (order, key) =>
- order.concat(
- prefixable.has(key) ? [`pre${key}`, key, `post${key}`] : [key],
- ),
- [],
- )
-
- return sortObjectKeys(scripts, order)
-})
+const sortScripts = onObject(sortPackageJsonScripts)
 
 // fields marked `vscode` are for `Visual Studio Code extension manifest` only
 // https://code.visualstudio.com/api/references/extension-manifest

package.json

@@ -27,10 +27,10 @@
  ],
  "scripts": {
  "lint": "eslint .",
- "semantic-release": "semantic-release",
- "sort-package-json": "node cli.js package.json --check",
  "test": "ava",
  "test-coverage": "nyc ava",
+ "semantic-release": "semantic-release",
+ "sort-package-json": "node cli.js package.json --check",
  "update-snapshot": "ava -u && node cli.js package.json"
  },
  "husky": {
@@ -76,7 +76,8 @@
  "git-hooks-list": "1.0.3",
  "globby": "11.0.0",
  "is-plain-obj": "2.1.0",
- "sort-object-keys": "^1.1.3"
+ "sort-object-keys": "^1.1.3",
+ "sort-package-json-scripts": "^1.0.3"
  },
  "devDependencies": {
  "@commitlint/cli": "8.3.5",

tests/scripts.js

@@ -18,19 +18,19 @@ const fixture = {
 }
 
 const expect = {
+ start: 'node server.js',
+ pretest: 'xyz',
+ test: 'node test.js',
+ posttest: 'abc',
  postinstall: 'echo "Installed"',
- multiply: '2 * 3',
- 'pre-fetch-info': 'foo',
  prepare: 'npm run build',
+ multiply: '2 * 3',
  preprettier: 'echo "not pretty"',
  prettier: 'prettier -l "**/*.js"',
  postprettier: 'echo "so pretty"',
- start: 'node server.js',
- pretest: 'xyz',
- test: 'node test.js',
- posttest: 'abc',
+ 'pre-fetch-info': 'foo',
  prewatch: 'echo "about to watch"',
- watch: 'watch things',
+ watch: 'watch things'
 }
 
 for (const field of ['scripts', 'betterScripts']) {