Skip to content

Fix FactoryStaker verification constructor - #190

Merged
JOY (JOY) merged 1 commit into
mainfrom
codex/mainnet-aa-factory-staker-constructor
Aug 13, 2026
Merged

Fix FactoryStaker verification constructor#190
JOY (JOY) merged 1 commit into
mainfrom
codex/mainnet-aa-factory-staker-constructor

Conversation

@JOY

Copy link
Copy Markdown

Summary

  • pin the deployed FactoryStaker owner constructor argument
  • keep verifier and Mainnet Browser UAT metadata expectations aligned
  • add a regression and update the approved design/plan provenance

Evidence

  • production metadata reports constructor owner 0x9775137314fE595c943712B0b336327dfa80aE8A
  • extractor tests: 5 passed
  • source verifier tests: 10 passed
  • bytecode tests: 9 passed
  • workflow/runtime tests: 17 passed
  • Node syntax, Bash syntax, actionlint, diff-check passed
  • Vietnamese dark HTML rendered with Playwright

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the verification configuration, test assertions, and documentation for the FactoryStaker contract to include its correct ABI-encoded constructor argument (the owner address 0x9775137314fE595c943712B0b336327dfa80aE8A). There are no review comments, and I have no additional feedback to provide.

@JOY
JOY (JOY) merged commit 4ff39cd into main Aug 13, 2026
12 checks passed
@JOY
JOY (JOY) deleted the codex/mainnet-aa-factory-staker-constructor branch August 13, 2026 23:25
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