Facade codemods: retire the codemod-limit exception rows in the facade gate registry (protocol.ts, engine.ts, sql-driver.ts) #30147

domain:devx#1 · os-support-ai · session_01WGufzgBDcBNVmSmo7VFHpj

Filing gate: ③ a follow-up that two reviews carried. The final contract review of PR #30030 (comment 6628808410) raised a should-fix: about 40 of the 60 exception rows in scripts/facade-delegation-registry.json cite #30020 (state-owning facets), but they are bodies because of codemod limits, not because of facet state. This seat's note on #30020 asked for an owner or a card, and none followed. The #30105 dev reports 25 more rows of the same kind for sql-driver.ts (PR #30139). ⛔ Not a claim. Suggested grade: p2. Route: domain:devx, the lane of the large-file split programme #29465.

Ruling-ref: 6624345766 (item 3: each exception row names the card that will move it)

What is wrong

A facade-gate exception row must name the card that retires it. Rows whose reason is a codemod limit point at #30020, and #30020's design (state ownership) will not remove them. So either nothing retires them, or #30020 grows past its scope.

The rows (reason → count; re-measure on main)

• protocol.ts + engine.ts (landed via refactor(metadata-protocol): generate the protocol.ts section split with a deterministic codemod #29998 / refactor(objectql): generate the engine.ts section split with a deterministic codemod #30012):
  • inferred-return-type 16
  • static-reads-class 9
  • query-options erasure pin 6
  • closed-set 4
  • no-host-read 2
  • name-taken 1
  • host-entry-predicate 1
  • field-call 1
• sql-driver.ts (PR refactor(driver-sql): put SqlDriver into facade form with the sql-driver codemod (sql-driver.ts 18,334 -> 10,979, body-identical) #30139, when it lands):
  • path-reader pin 9
  • accessor 5
  • knex field-call 4
  • inferred-return-type 4
  • computed parameter default 2
  • comment in parameters 1

Wanted

• Re-point every codemod-limit row to this card, leaving state-owning rows on refactor: move shared state out of the protocol.ts and engine.ts facades into state-owning facets, so the hot bodies left on the class can become delegations (ruling 6624345766) #30020.
• Per reason, either extend the codemod so the member becomes a delegation, or record why it stays a body. Examples:
  • write an explicit return type into the generated facet;
  • make the by-path pins read the facet module;
  • handle accessors with a get/set delegation.
• For each change: byte-identity of moved bodies, public .d.ts EQUAL, the facade gate exit 0 with the row removed, plus the touched packages' suites.

Out of scope (⛔)

• State-owning bodies (refactor: move shared state out of the protocol.ts and engine.ts facades into state-owning facets, so the hot bodies left on the class can become delegations (ruling 6624345766) #30020).
• Moving docblocks (refactor: move the private docblocks of the hand-series stubs into their hand modules — about 1,900 lines off engine.ts (86 stubs) and about 4,000 off protocol.ts (157 stubs) #30053).

Dedupe (open and closed): facade exception, codemod limit, facade-delegation-registry. Nearest: #30020 (state-owning facets), #30053 (docblocks), #30019 (the gate, closed). New.