Skip to content

Use proxy for token contract#30

Merged
stubbsta merged 14 commits into
mainfrom
use-proxy-for-token-contract
Aug 26, 2025
Merged

Use proxy for token contract#30
stubbsta merged 14 commits into
mainfrom
use-proxy-for-token-contract

Conversation

@stubbsta
Copy link
Copy Markdown
Contributor

@stubbsta stubbsta commented Aug 18, 2025

Description

Add a proxy contract for TestStableToken.
This will facilitate testing of incremental changes to the TST needed for improved token minting and membership registration workflow.

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Make TST contract upgradable
  • Add deployment script for TST Proxy
  • Update tests
  • Verify using waku-simulator

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR converts the TestStableToken contract from a simple contract to an upgradeable contract using OpenZeppelin's proxy pattern. This change enables iterative testing and deployment of token functionality modifications without requiring address changes across dependent systems.

  • Convert TestStableToken to use upgradeable OpenZeppelin contracts (ERC20Upgradeable, OwnableUpgradeable, etc.)
  • Add proxy deployment infrastructure with initialization support
  • Update all test files to deploy TestStableToken through ERC1967Proxy

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test/TestStableToken.sol Convert to upgradeable contract architecture with UUPS proxy pattern
test/TestStableToken.t.sol Update tests to deploy token through proxy and adjust test expectations
test/WakuRlnV2.t.sol Update test setup to use proxy-deployed TestStableToken
script/DeployTokenWithProxy.s.sol Add deployment script for TestStableToken with proxy
test/README.md Add comprehensive documentation for TestStableToken usage and deployment
script/Deploy.s.sol Minor import formatting improvement

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread test/TestStableToken.t.sol Outdated
@jm-clius jm-clius requested a review from 0x-r4bbit August 22, 2025 14:14
@jm-clius
Copy link
Copy Markdown

@0x-r4bbit would appreciate a review from Vac SC :)

Copy link
Copy Markdown
Member

@richard-ramos richard-ramos left a comment

Choose a reason for hiding this comment

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

Pretty cool :)

Comment thread test/README.md
Comment thread test/TestStableToken.t.sol
Comment thread test/TestStableToken.t.sol Outdated
Comment thread script/Deploy.s.sol
Comment thread test/README.md Outdated
Comment thread test/WakuRlnV2.t.sol
@stubbsta stubbsta merged commit b4508dd into main Aug 26, 2025
4 checks passed
@stubbsta stubbsta deleted the use-proxy-for-token-contract branch August 26, 2025 15:34
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.

6 participants