Skip to content

chore(addresses): sync 40204 after R2 redeploy - #5

Merged
SaulBuilds merged 2 commits into
mainfrom
chore/r2-address-sync
Sep 27, 2026
Merged

SaulBuilds merged 2 commits into
mainfrom
chore/r2-address-sync

Conversation

@SaulBuilds

Copy link
Copy Markdown
Contributor

Syncs the vendored contract-address table (crates/chainio/src/generated/addresses.json) with the canonical book after the chain 40204 R2 contract redeploy.

Merge after citrate-chain#242 lands so the canonical book is on main.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Vv2gVzy5XLFKg48yckN9YQ

@SaulBuilds
SaulBuilds requested a review from a team September 27, 2026 15:52
Matr0xshka
Matr0xshka previously approved these changes Sep 27, 2026

@Matr0xshka Matr0xshka left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the generated address-book/derived artifact sync. Approval is for the regenerated artifacts as submitted; merge remains gated on the canonical chain/address validation checks.

BerryManifold
BerryManifold previously approved these changes Sep 27, 2026

@BerryManifold BerryManifold left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Second-look review from BerryManifold: approved the generated address-book/derived artifact sync. Merge remains gated on canonical chain/address validation checks.

@BerryManifold
BerryManifold dismissed stale reviews from Matr0xshka and themself via 7713bb2 September 27, 2026 20:00

@Matr0xshka Matr0xshka left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed by Matr0xshka. Approval is for the code/content diff as submitted; merge remains gated on required CI and branch protection.

@kurtatwork kurtatwork left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the generated consumer address update at the current head. The address artifact matches the canonical schema peer artifact exactly by Git blob SHA, and all current checks pass. Approval covers this consumer diff only; merge after corrected CitrateNetwork/citrate-chain#242 merges. If that fix changes generated address artifacts, re-sync and re-request review.

SaulBuilds and others added 2 commits September 27, 2026 14:44
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vv2gVzy5XLFKg48yckN9YQ
The default generic-api-key rule flags the public ModelAccessControl contract
address in the generated address book. Only values that are exactly an EVM
address are allowed; AWS keys, tokens and 64-hex private keys in the same file
are still reported (checked locally with planted values).

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012cD3fDq5vhh2YWZPU2SV6H
@SaulBuilds
SaulBuilds force-pushed the chore/r2-address-sync branch from 7713bb2 to ce24de5 Compare September 27, 2026 21:44

@BerryManifold BerryManifold left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed by BerryManifold. Approval is for the code/content diff as submitted; merge remains gated on required CI and branch protection.

@SaulBuilds
SaulBuilds merged commit 143b173 into main Sep 27, 2026
6 checks passed
@SaulBuilds
SaulBuilds deleted the chore/r2-address-sync branch September 27, 2026 22:12
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.

4 participants