Skip to content

Commit 09a73ed

Browse files
committed
Cache alphabetic sort
1 parent 41e284c commit 09a73ed

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

lib/commands/audit.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@ const log = require('../utils/log-shim.js')
1616
const pulseTillDone = require('../utils/pulse-till-done.js')
1717
const reifyFinish = require('../utils/reify-finish.js')
1818

19+
const sortAlphabetically = (a, b) => localeCompare(a.name, b.name)
20+
1921
const verifySignature = async ({ message, signature, publicKey }) => {
2022
const verifier = crypto.createVerify('SHA256')
2123
verifier.write(message)
@@ -63,9 +65,8 @@ class VerifySignatures {
6365
throw new Error('No dependencies found in current install')
6466
}
6567

66-
// Sort alphabetically
67-
const invalid = Array.from(this.invalid).sort((a, b) => localeCompare(a.name, b.name))
68-
const missing = Array.from(this.missing).sort((a, b) => localeCompare(a.name, b.name))
68+
const invalid = Array.from(this.invalid).sort(sortAlphabetically)
69+
const missing = Array.from(this.missing).sort(sortAlphabetically)
6970

7071
const verified = invalid.length === 0 && missing.length === 0
7172

@@ -307,7 +308,7 @@ class VerifySignatures {
307308
return `${nameVersion}${suffix}`
308309
}))
309310

310-
return uniquePackages.join('\n')
311+
return [...uniquePackages].join('\n')
311312
}
312313

313314
makeJSON ({ invalid, missing }) {

0 commit comments

Comments
 (0)