Skip to content

Commit 70bc076

Browse files
committed
fix(spec): stop teaching the retired ETL layer as a live retry surface in flow.zod.ts (#6630, part 2)
Part 2 of 2 for #6630, closing out the three `automation/flow.zod.ts` prose sites that part 1 (PR #6701) deferred while the file was inside #5593's in-flight surface. #6414 retired the whole L2 ETL layer, so `ETLPipeline.retry` is no longer one of the surfaces that carry the converged `RetryPolicySchema` contract. Three places in this file still said it was: - the `errorHandling` TSDoc enumerated it as a live sibling of `job.retryPolicy` and a `try_catch` node's `retry`; - "a key added to the policy lands on all four surfaces at once" — three; - "the flow surface has a reading the other three do not" — the other two. Text only. No schema key, bound, default, `.describe()` or `retiredKey()` guidance string changes, so nothing regenerates: all 10 of spec's generated artifacts stay green, including `check:docs` and `check:api-surface`. Wording matches part 1's landed choices — enumerations name only surviving surfaces, the retired mention becomes explicitly historical ("did the same until #6414 retired the L2 ETL layer"), counts corrected in place. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018ffcE95NaMJcL9XJ9VDYgk
1 parent 61282f9 commit 70bc076

1 file changed

Lines changed: 10 additions & 9 deletions

File tree

packages/spec/src/automation/flow.zod.ts

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -637,13 +637,14 @@ export const FlowSchema = lazySchema(() => strictObject(
637637
* Error Handling Strategy.
638638
*
639639
* The retry knobs are the converged `RetryPolicySchema` contract, shared with
640-
* `job.retryPolicy`, a `try_catch` node's `retry` and an ETL pipeline's
641-
* `retry` (#4661 + #4964 — see `shared/retry-policy.zod.ts`). Until 17 this
642-
* block spelled the base delay `retryDelayMs` while the converged policy
643-
* spelled it `backoffMs`, so an author who read the newer file and brought
644-
* the word here had it silently stripped (pre-批 11) or rejected (post-批 11)
645-
* — being punished for learning the canonical spelling. `strategy` stays
646-
* here: it selects *whether* the policy runs, it is not part of the policy.
640+
* `job.retryPolicy` and a `try_catch` node's `retry` (#4661 + #4964 — see
641+
* `shared/retry-policy.zod.ts`; `ETLPipeline.retry` did the same until #6414
642+
* retired the L2 ETL layer). Until 17 this block spelled the base delay
643+
* `retryDelayMs` while the converged policy spelled it `backoffMs`, so an
644+
* author who read the newer file and brought the word here had it silently
645+
* stripped (pre-批 11) or rejected (post-批 11) — being punished for learning
646+
* the canonical spelling. `strategy` stays here: it selects *whether* the
647+
* policy runs, it is not part of the policy.
647648
*
648649
* **These defaults are the only defaults** (#4247). The engine reads the
649650
* parsed block field-by-field with no fallback of its own — `retryExecution`
@@ -717,14 +718,14 @@ export const FlowSchema = lazySchema(() => strictObject(
717718
// is what made the divergence so durable: it looked reviewed.
718719
//
719720
// The spread is what keeps that from happening again. A key added to the
720-
// policy lands on all four surfaces at once, instead of on the ones
721+
// policy lands on all three surfaces at once, instead of on the ones
721722
// whoever added it happened to grep for.
722723
...retryPolicyShape(),
723724

724725
// The ONE site-specific override, and it is prose only — same type, same
725726
// bounds, same default, all still single-sourced above. `.describe()`
726727
// lands in `content/docs/references/`, and the flow surface has a reading
727-
// the other three do not: the count is read only under `strategy:
728+
// the other two do not: the count is read only under `strategy:
728729
// 'retry'`, where the `superRefine` below then requires >= 1 (#4247).
729730
// Default 0 = "no retries" is the right reading for the two strategies
730731
// that never retry; under `'retry'` it would mean "retry, zero times",

0 commit comments

Comments
 (0)