refactor(tempo): consolidate abi generation - #5037
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
commit: |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cebf2c89a7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
jxom
changed the title
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ff1f355f66
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
The public Sepolia fallback RPC only retains state for the most recent blocks, so any pinned fork block ages out of its retention window within hours and anvil fails with 'historical state is not available'. Resolve a fresh block (trailing the head slightly) when the instance starts instead of pinning one. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyZL1bqvK1CTsEB9ca21dj
`pnpm gen:tempo-abis` now regenerates from the source commits pinned in the generated file headers so output is reproducible, and the new `pnpm sync:tempo-abis` resolves the current upstream commits and updates the pins. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyZL1bqvK1CTsEB9ca21dj
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyZL1bqvK1CTsEB9ca21dj
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9a05dba623
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Drops the pinned-commit/sync split: `pnpm gen:tempo-abis` resolves the current upstream commits directly, and the generated headers record which commit produced each artifact (`tempoxyz/<repo>@<commit>`). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyZL1bqvK1CTsEB9ca21dj
Regenerated via `gen:tempo-abis`: tempo and zones ABI content is unchanged at their current heads, so only the recorded source commits move (tempo d656ef5, zones 7e38642). Earn output is unchanged (source unreachable from this environment). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WyZL1bqvK1CTsEB9ca21dj
2 participants
Consolidates Tempo, Earn, and Zone ABI generation into adapters that resolve current upstream commits. Regenerates ABIs, selectors, addresses, and test deploy artifacts while retaining the legacy
nativeMultisigABI until #5029 lands.