Skip to content

Commit 18a3dbf

Browse files
committed
docs(service-automation): retire the stale "escapes those bounds" claim on evictConsumedSuspension
The compound case that paragraph pointed at is closed by the re-seat in `persistSuspendedRun`'s catch, and the pin file now records the intended outcome rather than the measured loss. Comment-only. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XpTx2tbq3pZRYAdoGt6E6Y
1 parent b5491f9 commit 18a3dbf

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

  • packages/services/service-automation/src

packages/services/service-automation/src/engine.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2279,8 +2279,14 @@ export class AutomationEngine implements IAutomationService {
22792279
* cache-only qualifier is not yet set. Evicting it is bounded -- the run
22802280
* stays resumable through the store-first strict load and only the
22812281
* cache-only listing under-reports -- and
2282-
* `suspended-run-mid-park-eviction-window.test.ts` pins both the window and
2283-
* the one compound case that escapes those bounds.
2282+
* `suspended-run-mid-park-eviction-window.test.ts` pins that window.
2283+
*
2284+
* [#16151] The one COMPOUND case that escaped those bounds -- an eviction
2285+
* here, and then that same save FAILING, leaving the run with neither a
2286+
* durable row nor a map entry -- no longer does: `persistSuspendedRun`'s
2287+
* catch re-seats the entry alongside the cache-only marking, so guard 2
2288+
* above has something to guard again. The same pin file carries it, now at
2289+
* the intended outcome rather than at the measured loss.
22842290
*
22852291
* A store read that THROWS must never reach here: an outage means the
22862292
* run's existence is UNKNOWN, not "gone". Every caller below is on a path

0 commit comments

Comments
 (0)