@invarn/cibuild 1.9.3 → 1.9.4

package/dist/src/yaml/steps/cache.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cache.d.ts","sourceRoot":"","sources":["../../../../src/yaml/steps/cache.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,gBAAgB,EAAE,MAAM,WAAW,CAAC;AAC7C,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAkBxD;;;;;;GAMG;AACH,MAAM,WAAW,WAAW;IAC1B,4HAA4H;IAC5H,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,sHAAsH;IACtH,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,yGAAyG;IACzG,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,WAAW,CAUrD,CAAC;AAmFF;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;IACvB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAED;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;IACvB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,qBAAqB,CAAC,EAAE,OAAO,CAAC;CACjC;AAED;;;GAGG;AACH,qBAAa,qBAAsB,SAAQ,gBAAgB;IACnD,OAAO,CAAC,MAAM,EAAE,eAAe,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;YA0GzF,iBAAiB;CA8LhC;AAED;;;GAGG;AACH,qBAAa,qBAAsB,SAAQ,gBAAgB;IACnD,OAAO,CAAC,MAAM,EAAE,eAAe,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;YA4JzF,iBAAiB;CA2IhC"}
+{"version":3,"file":"cache.d.ts","sourceRoot":"","sources":["../../../../src/yaml/steps/cache.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,gBAAgB,EAAE,MAAM,WAAW,CAAC;AAC7C,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAkBxD;;;;;;GAMG;AACH,MAAM,WAAW,WAAW;IAC1B,4HAA4H;IAC5H,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,sHAAsH;IACtH,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,yGAAyG;IACzG,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,WAAW,CAUrD,CAAC;AAmFF;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;IACvB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAED;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,EAAE,CAAC;IACvB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,qBAAqB,CAAC,EAAE,OAAO,CAAC;CACjC;AAED;;;GAGG;AACH,qBAAa,qBAAsB,SAAQ,gBAAgB;IACnD,OAAO,CAAC,MAAM,EAAE,eAAe,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;YA0GzF,iBAAiB;CA8LhC;AAED;;;GAGG;AACH,qBAAa,qBAAsB,SAAQ,gBAAgB;IACnD,OAAO,CAAC,MAAM,EAAE,eAAe,EAAE,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAAC,OAAO,CAAC;YA4JzF,iBAAiB;CAoHhC"}

package/dist/src/yaml/steps/cache.js

@@ -593,34 +593,11 @@ export class CachePushStepExecutor extends BaseStepExecutor {
         if (isDebugMode) {
             commands.push('    echo "Cache size: $(du -h "$CACHE_FILE" | cut -f1)"');
         }
-        // Retention per-scope: (1) keep N newest by mtime — touched on hit in
-        // cache-pull so this is LRU, not strictly creation-order; (2) delete
-        // anything past the age cap regardless of position; (3) enforce a size
-        // budget, oldest-out. Two passes are required: the size pass needs the
-        // post-cleanup set, otherwise its running sum counts tarballs that the
-        // count/age pass is about to delete and over-evicts.
-        commands.push('    __ci_ret_scope="${CACHE_KEY%-*}"');
-        commands.push('    __ci_ret_count_cap=5');
-        commands.push('    __ci_ret_age_cap=$((30 * 86400))');
-        commands.push('    __ci_ret_size_cap_kb="${CIBUILD_CACHE_SCOPE_BUDGET_KB:-10485760}"');
-        commands.push('    __ci_ret_now=$(date +%s)');
-        commands.push('    __ci_ret_idx=0');
-        commands.push('    while IFS= read -r __ci_ret_f; do');
-        commands.push('      __ci_ret_idx=$((__ci_ret_idx + 1))');
-        commands.push('      __ci_ret_age=$(( __ci_ret_now - $(stat -f %m "$__ci_ret_f" 2>/dev/null || echo "$__ci_ret_now") ))');
-        commands.push('      if [ "$__ci_ret_idx" -gt "$__ci_ret_count_cap" ] || [ "$__ci_ret_age" -gt "$__ci_ret_age_cap" ]; then');
-        commands.push('        rm -f "$__ci_ret_f"');
-        commands.push('      fi');
-        commands.push('    done < <(ls -t "$CACHE_DIR"/"$__ci_ret_scope"-*.tar.zst 2>/dev/null)');
-        commands.push('    __ci_ret_running_kb=0');
-        commands.push('    while IFS= read -r __ci_ret_f; do');
-        commands.push('      __ci_ret_size=$(du -k "$__ci_ret_f" 2>/dev/null | cut -f1)');
-        commands.push('      __ci_ret_size=${__ci_ret_size:-0}');
-        commands.push('      __ci_ret_running_kb=$((__ci_ret_running_kb + __ci_ret_size))');
-        commands.push('      if [ "$__ci_ret_running_kb" -gt "$__ci_ret_size_cap_kb" ]; then');
-        commands.push('        rm -f "$__ci_ret_f"');
-        commands.push('      fi');
-        commands.push('    done < <(ls -t "$CACHE_DIR"/"$__ci_ret_scope"-*.tar.zst 2>/dev/null)');
+        // Retention is no longer applied here (ADR 0002 / CBR-6). It moved into the
+        // long-lived peer cache daemon, which owns ~/cache and — unlike the guest —
+        // knows (via the manager) which scopes this runner OWNS and must never evict.
+        // A guest-side prune would have to run blind to ownership and could reap the
+        // fleet's single durable copy, so the daemon owns the whole policy now.
         commands.push('  else');
         commands.push('    echo "Warning: Failed to create cache file"');
         commands.push('  fi');

package/dist/src/yaml/steps/steps.test.js

@@ -424,17 +424,21 @@ describe('Step Implementations', () => {
             expect(result.script).toContain('> "$CACHE_FILE.tmp"');
             expect(result.script).toContain('mv "$CACHE_FILE.tmp" "$CACHE_FILE"');
         });
-        test('should sweep retention (count + age + size budget) after successful preset push', async () => {
+        test('should NOT emit guest-side retention (the peer cache daemon owns it, CBR-6)', async () => {
+            // ADR 0002 / CBR-6: retention moved into the long-lived peer cache daemon,
+            // which knows (via the manager) which scopes this runner owns and must never
+            // evict. The guest cache-push no longer prunes — it could only reap an owner's
+            // durable copy, having no ownership knowledge of its own.
             const executor = new CachePushStepExecutor();
             const result = await executor.execute({ technology: 'kmm' }, {}, testConfig);
-            expect(result.script).toContain('__ci_ret_scope="${CACHE_KEY%-*}"');
-            expect(result.script).toContain('ls -t "$CACHE_DIR"/"$__ci_ret_scope"-*.tar.zst');
-            expect(result.script).toContain('__ci_ret_count_cap=5');
-            expect(result.script).toContain('__ci_ret_age_cap=$((30 * 86400))');
-            expect(result.script).toContain('CIBUILD_CACHE_SCOPE_BUDGET_KB');
-            expect(result.script).toContain('rm -f "$__ci_ret_f"');
-        });
-        test('should not run retention for manual cache_key push (no scope)', async () => {
+            expect(result.script).not.toContain('__ci_ret_scope');
+            expect(result.script).not.toContain('__ci_ret_count_cap');
+            expect(result.script).not.toContain('__ci_ret_age_cap');
+            expect(result.script).not.toContain('CIBUILD_CACHE_SCOPE_BUDGET_KB');
+            // The successful-push path itself is unchanged.
+            expect(result.script).toContain('Cache created successfully');
+        });
+        test('should not emit retention for a manual cache_key push either', async () => {
             const executor = new CachePushStepExecutor();
             const result = await executor.execute({ cache_key: 'my-key', cache_paths: ['build'] }, {}, testConfig);
             expect(result.script).not.toContain('__ci_ret_scope');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@invarn/cibuild",
-  "version": "1.9.3",
+  "version": "1.9.4",
   "description": "CI Build CLI — local pipeline orchestration and validation",
   "type": "module",
   "main": "dist/cli.cjs",