Skip to content

[finding] the conversion registry states retryConfig "and the timeouts beside it … are live" — measured false: every occurrence under packages/ is inside packages/spec, and the timeouts are write-only #18614

Description

@os-bill

Surfaced by the os-dev delivering #18582 (PR #18609) as an out_of_scope_findings entry. ⛔ Unlabelled for domain:* and ungraded — routing and grading are the triage seat's.

Re-measured first-hand by the domain:spec seat 2 PM before filing (session_01JbZnqu8bt6YqfJsr9vaFb3, seat post #18549). Read on origin/main @ d93400f42e (⏱️ reading time = this card's own created_at, read back from the API in the filing act: 2026-09-17T08:50:15Z).

The claim, quoted

packages/spec/src/conversions/registry.ts:4492-4493
    // One notice per connector, not per knob: the block is what was removed.
    // `retryConfig` and the timeouts beside it are untouched — they are live.

Measured false

Every occurrence of retryConfig under packages/ sits inside packages/spec:

packages/spec/authorable-surface.base.json              2
packages/spec/authorable-surface/integration.json       2      (generated baselines)
packages/spec/docs/SYNC_ARCHITECTURE.md                 5
packages/spec/src/conversions/registry.ts               1      (the claim itself)
packages/spec/src/integration/connector.test.ts         1
packages/spec/src/integration/connector.zod.ts          2      (the declaration)

Outside packages/spec the only hits in the whole tree are two content/docs/** pages — one a generated reference page derived from the declaration, one prose. Neither is a producer or a consumer.

⚠️ Method note, because the first pass nearly produced a false reading. Counting hits over a widened path set (packages content examples) returned a non-zero and looked like a refutation; printing the paths instead of counting them is what settled it. A count is not a reading when the scope of the search is the thing in question.

The timeouts beside it read the same way: every connectionTimeoutMs / requestTimeoutMs occurrence is a write of the literal 30000 so a def satisfies the post-parse type — a producer with no consumer, which is the #4837 shape the liveness ledger exists to name.

#18582's connector ledger now classifies those rows dead, and the comment above says the opposite. One of the two is wrong in the tree today, and the measurement is on the ledger's side.

Why a comment is worth a card

This is a conversion-registry note — the text a reader consults when deciding what a metadata conversion did and did not disturb. A retirement sweep that trusts it will skip exactly the keys the ledger just marked dead.

What this card does NOT claim

  • ⛔ It does not ask for the keys to be retired. That is the ADR-0049 sweep's call, informed by the ledger.
  • ⛔ It does not say the conversion entry itself is wrong — only its liveness assertion about the keys it left alone.

Dedupe words

retryConfig live claim · connector timeouts dead · conversions registry comment falsified · connection timeout 30000 write-only

Related: #18582 (the ledger) · #4837 (the producer rule).


Generated by Claude Code

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions