Skip to content

tip-1091: specify portal proxy initialization#656

Open
decofe wants to merge 2 commits into
mainfrom
centaur/delegatecall-portal-initializer
Open

tip-1091: specify portal proxy initialization#656
decofe wants to merge 2 commits into
mainfrom
centaur/delegatecall-portal-initializer

Conversation

@decofe

@decofe decofe commented Jul 15, 2026

Copy link
Copy Markdown
Member

Summary

  • define portal proxies versus the shared ZONE_PORTAL_LOGIC_ADDRESS
  • require initialization through a portal proxy DELEGATECALL; direct logic-contract initialization reverts
  • specify the temporary ZonePortal deployment that produces constructor-specialized runtime code, then installs that runtime at the canonical logic address
  • retain factory-only, one-time initialization per portal proxy

Validation

  • git diff --check
  • Forge build remains blocked because GitHub source archives omit the tips/verify submodules

Prompted by: @0xalpharush

@decofe
decofe requested review from 0xKitsune and mattsse as code owners July 15, 2026 01:15
@decofe decofe changed the title fix: require delegatecall portal initialization tip-1091: specify portal proxy initialization Jul 15, 2026
@0xalpharush 0xalpharush mentioned this pull request Jul 15, 2026
@legion2002
legion2002 force-pushed the agent/zone-portal-proxy-storage branch 2 times, most recently from 249cea6 to 3f16f28 Compare July 15, 2026 15:47
Base automatically changed from agent/zone-portal-proxy-storage to main July 15, 2026 16:11
@legion2002
legion2002 force-pushed the centaur/delegatecall-portal-initializer branch from 9718f10 to 8e2b134 Compare July 15, 2026 16:29
@tempo-voight-kampff

Copy link
Copy Markdown

Hi @legion2002 — your review approval was detected by Voight-Kampff, but this repository configures allow-self-reviews: false, so the current head commit author cannot approve their own changes. Your review did not trigger an approval prompt and is not counted toward this PR.

Ask another reviewer to +1 the PR (or submit an "Approve" review) to satisfy the gate.

@tempo-voight-kampff

Copy link
Copy Markdown

Hi @legion2002 — your review approval was detected by Voight-Kampff, but this repository configures allow-self-reviews: false, so the current head commit author cannot approve their own changes. Your review did not trigger an approval prompt and is not counted toward this PR.

Ask another reviewer to +1 the PR (or submit an "Approve" review) to satisfy the gate.

@legion2002
legion2002 enabled auto-merge July 15, 2026 16:36
legion2002
legion2002 previously approved these changes Jul 15, 2026
@tempo-voight-kampff

Copy link
Copy Markdown

Hi @legion2002 — your review approval was detected by Voight-Kampff, but this repository configures allow-self-reviews: false, so the current head commit author cannot approve their own changes. Your review did not trigger an approval prompt and is not counted toward this PR.

Ask another reviewer to +1 the PR (or submit an "Approve" review) to satisfy the gate.

0xalpharush
0xalpharush previously approved these changes Jul 15, 2026
@decofe
decofe dismissed stale reviews from 0xalpharush and legion2002 via f1e48c2 July 15, 2026 16:52
@legion2002
legion2002 added this pull request to the merge queue Jul 15, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Jul 15, 2026
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