feat: remove used_is_minimum property

Author: hesreallyhim

README.md

@@ -53,9 +53,9 @@ After your job completes, you'll get a nice summary:
 
 ## Outputs
 
-| Name  | Description                                                                                                                       |
-| ----- | --------------------------------------------------------------------------------------------------------------------------------- |
-| usage | JSON string with total, duration_ms, total_is_minimum, and buckets_data (per-bucket used/remaining/crossed_reset/used_is_minimum) |
+| Name  | Description                                                                                                       |
+| ----- | ----------------------------------------------------------------------------------------------------------------- |
+| usage | JSON string with total, duration_ms, total_is_minimum, and buckets_data (per-bucket used/remaining/crossed_reset) |
 
 Example output:
 
@@ -65,9 +65,9 @@ Example output:
   "duration_ms": 12345,
   "total_is_minimum": false,
   "buckets_data": {
-    "core": { "used": 45, "remaining": 4955, "crossed_reset": false, "used_is_minimum": false },
-    "graphql": { "used": 10, "remaining": 4990, "crossed_reset": false, "used_is_minimum": false },
-    "search": { "used": 5, "remaining": 25, "crossed_reset": false, "used_is_minimum": false }
+    "core": { "used": 45, "remaining": 4955, "crossed_reset": false },
+    "graphql": { "used": 10, "remaining": 4990, "crossed_reset": false },
+    "search": { "used": 5, "remaining": 25, "crossed_reset": false }
   }
 }
 ```

dist/post/index.js

@@ -27904,7 +27904,6 @@ function computeBucketUsage(
     used: 0,
     remaining: undefined,
     crossed_reset: false,
-    used_is_minimum: false,
     warnings: []
   };
 
@@ -27936,7 +27935,6 @@ function computeBucketUsage(
       result.warnings.push('limit_changed_across_reset');
     }
     used = endingLimit - endingRemaining;
-    result.used_is_minimum = true;
 
     if (
       checkpointBucket &&
@@ -28255,8 +28253,7 @@ async function run() {
       data[bucket] = {
         used: usage.used,
         remaining: usage.remaining,
-        crossed_reset: usage.crossed_reset,
-        used_is_minimum: usage.used_is_minimum
+        crossed_reset: usage.crossed_reset
       };
       const startingRemaining = Number(startingBucket.remaining);
       const startingLimit = Number(startingBucket.limit);
@@ -28280,7 +28277,7 @@ async function run() {
       if (usage.crossed_reset) {
         crossedBuckets.push(bucket);
       }
-      if (usage.used_is_minimum) {
+      if (usage.crossed_reset) {
         totalIsMinimum = true;
       }
       totalUsed += usage.used;

src/post-utils.js

@@ -66,7 +66,6 @@ function computeBucketUsage(
     used: 0,
     remaining: undefined,
     crossed_reset: false,
-    used_is_minimum: false,
     warnings: []
   };
 
@@ -98,7 +97,6 @@ function computeBucketUsage(
       result.warnings.push('limit_changed_across_reset');
     }
     used = endingLimit - endingRemaining;
-    result.used_is_minimum = true;
 
     if (
       checkpointBucket &&

src/post.js

@@ -181,8 +181,7 @@ async function run() {
       data[bucket] = {
         used: usage.used,
         remaining: usage.remaining,
-        crossed_reset: usage.crossed_reset,
-        used_is_minimum: usage.used_is_minimum
+        crossed_reset: usage.crossed_reset
       };
       const startingRemaining = Number(startingBucket.remaining);
       const startingLimit = Number(startingBucket.limit);
@@ -206,7 +205,7 @@ async function run() {
       if (usage.crossed_reset) {
         crossedBuckets.push(bucket);
       }
-      if (usage.used_is_minimum) {
+      if (usage.crossed_reset) {
         totalIsMinimum = true;
       }
       totalUsed += usage.used;

tests/post-utils.test.mjs

@@ -80,7 +80,6 @@ describe('computeBucketUsage', () => {
       used: 50,
       remaining: 850,
       crossed_reset: false,
-      used_is_minimum: false,
       warnings: []
     });
   });
@@ -97,7 +96,6 @@ describe('computeBucketUsage', () => {
       used: 0,
       remaining: undefined,
       crossed_reset: false,
-      used_is_minimum: false,
       warnings: [],
       reason: 'remaining_increased_without_reset'
     });
@@ -115,7 +113,6 @@ describe('computeBucketUsage', () => {
       used: 100,
       remaining: 900,
       crossed_reset: true,
-      used_is_minimum: true,
       warnings: []
     });
   });
@@ -134,7 +131,6 @@ describe('computeBucketUsage', () => {
       used: 150,
       remaining: 900,
       crossed_reset: true,
-      used_is_minimum: true,
       warnings: []
     });
   });
@@ -151,7 +147,6 @@ describe('computeBucketUsage', () => {
       used: 300,
       remaining: 4700,
       crossed_reset: true,
-      used_is_minimum: true,
       warnings: ['limit_changed_across_reset']
     });
   });
@@ -168,7 +163,6 @@ describe('computeBucketUsage', () => {
       used: 0,
       remaining: undefined,
       crossed_reset: false,
-      used_is_minimum: false,
       warnings: [],
       reason: 'limit_changed_without_reset'
     });