Use more OVERRIDES_FIELD_NAME and RESOLUTIONS_FIELD_NAME

SocketDev · Dec 23, 2024 · 67657a2 · 67657a2
1 parent 9d99734
commit 67657a2
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/commands/optimize.ts b/src/commands/optimize.ts
@@ -250,7 +250,7 @@ const updateManifestByAgent: Record<Agent, AgentModifyManifestFn> = (() => {
     let insertIndex = -1
     let isPlacingHigher = false
     if (field === OVERRIDES_FIELD_NAME) {
-      insertIndex = getLowestEntryIndex(entries, ['resolutions'])
+      insertIndex = getLowestEntryIndex(entries, [RESOLUTIONS_FIELD_NAME])
       if (insertIndex === -1) {
         isPlacingHigher = true
         insertIndex = getHighestEntryIndex(entries, [...depFields, PNPM])
@@ -259,11 +259,11 @@ const updateManifestByAgent: Record<Agent, AgentModifyManifestFn> = (() => {
       isPlacingHigher = true
       insertIndex = getHighestEntryIndex(entries, [
         ...depFields,
-        'overrides',
+        OVERRIDES_FIELD_NAME,
         PNPM
       ])
     } else if (field === PNPM_FIELD_NAME) {
-      insertIndex = getLowestEntryIndex(entries, ['overrides', 'resolutions'])
+      insertIndex = getLowestEntryIndex(entries, [OVERRIDES_FIELD_NAME, RESOLUTIONS_FIELD_NAME])
       if (insertIndex === -1) {
         isPlacingHigher = true
         insertIndex = getHighestEntryIndex(entries, depFields)