Skip to content

Clear signing: track latest registry, provider blocklist, richer processing log - #69

Draft
PrisionMike wants to merge 2 commits into
prisionmike/cs-parser-readablefrom
prisionmike/registry-ingestion
Draft

Clear signing: track latest registry, provider blocklist, richer processing log#69
PrisionMike wants to merge 2 commits into
prisionmike/cs-parser-readablefrom
prisionmike/registry-ingestion

Conversation

@PrisionMike

@PrisionMike PrisionMike commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

Stacked on #68 (readable parser rework), which stacks on #66. Retarget down the chain as each lands.

What

The registry-ingestion half of the floodgates work — the parser half is #68:

  • Track latest registry. do_update.sh no longer pins the clear-signing registry submodule; it follows upstream master like the other submodules (bumped to 5a67786 here).
  • Provider blocklist instead of allowlist. ENABLED_PROVIDERS is replaced by BLOCKED_PROVIDERS (uniswap, safe to start); blocked providers are skipped before parsing, everything else is parsed and emitted. The blocklist is printed as the first line of definitions-latest.log.
  • Richer log. A new Adjustments section reports every accepted-but-modified field (formatter overrides, ABI retypes, materialized constants), grouped by kind and by file, alongside the existing unsupported-features and conflict sections.

Numbers (on registry 5a67786, with #68's parser)

3590 display-format records emitted (was 612 under the allowlist), 709 slice-bearing field defs, 32 calldata and 162 enum fields, provider_name on every record — all serialize to proto. array-slice-path and unrepresentable-params drops: zero; visible: optional fields (28) are hidden per spec and logged.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8ff3ef38-bfcd-48a7-8edb-e8ded442d623

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch prisionmike/registry-ingestion

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@PrisionMike
PrisionMike force-pushed the prisionmike/cs-parser-readable branch from 059b9c1 to e138757 Compare July 26, 2026 22:11
@PrisionMike
PrisionMike force-pushed the prisionmike/registry-ingestion branch from 47cb632 to bb33499 Compare July 26, 2026 22:29
@PrisionMike
PrisionMike marked this pull request as draft July 26, 2026 22:52
@PrisionMike
PrisionMike force-pushed the prisionmike/cs-parser-readable branch from e138757 to f119fe8 Compare July 27, 2026 21:48
@PrisionMike
PrisionMike force-pushed the prisionmike/registry-ingestion branch from bb33499 to 0728378 Compare July 27, 2026 22:17
@PrisionMike
PrisionMike force-pushed the prisionmike/cs-parser-readable branch from f119fe8 to 723a9e3 Compare July 28, 2026 04:49
PrisionMike and others added 2 commits July 28, 2026 06:51
* do_update.sh no longer pins the clear-signing registry submodule; it now
  tracks upstream master like the other submodules (bumped to 5a67786 here).
* The ENABLED_PROVIDERS allowlist is replaced by BLOCKED_PROVIDERS
  (uniswap, safe to start): everything else in the registry is parsed and
  emitted. Blocked providers are skipped before parsing.
* definitions-latest.log gains a "Providers blocked:" header and a new
  Adjustments section (accepted-with-modification fields, grouped by kind
  and by file) alongside the unsupported-features and conflicts sections.

Emitted display formats grow 612 -> 3274; all serialize to proto.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@PrisionMike
PrisionMike force-pushed the prisionmike/registry-ingestion branch from 0728378 to 85d237b Compare July 28, 2026 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant