Rebrand wTON as wGRAM#757
Conversation
|
👋 krebernisak, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Pull request overview
This PR rebrands the wrapped-native-coin Jetton wrapper previously referred to as “wTON” to “wGRAM”, updating docs, contract sources, wrapper compile targets, tests, and gas-report tooling to use the new naming.
Changes:
- Renames documentation pages/sidebars and updates internal doc IDs/links from wTON → wGRAM.
- Moves/renames contract and wrapper surfaces to
wgram(including blueprint compile entrypoints and Jest gas-report config/script names). - Updates test suites and adds a new wGRAM gas-report snapshot/config.
Reviewed changes
Copilot reviewed 18 out of 22 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| docs/contracts/overview/wgram/index.md | Rebrands overview page content/ids to wGRAM. |
| docs/contracts/overview/wgram/gas_report.md | Updates gas-report doc references and command to wGRAM. |
| docs/contracts/overview/wgram/design.md | Updates design doc wording/ids to wGRAM. |
| docs/contracts/overview/wgram/category.yaml | Updates sidebar category label/id to wGRAM. |
| docs/contracts/overview/index.md | Updates overview link from wTON to wGRAM. |
| docs/contracts/index.md | Updates contracts docs index link from wTON to wGRAM. |
| contracts/wrappers/wgram/index.ts | Adds barrel exports for wGRAM wrapper constants/errors. |
| contracts/wrappers/wgram/errors.ts | Introduces wGRAM-specific exported error codes. |
| contracts/wrappers/wgram/constants.ts | Introduces wGRAM opcode constants (currently still wTON-named identifiers). |
| contracts/wrappers/wgram.JettonWallet.compile.ts | Points wallet compile config to contracts/wgram/JettonWallet.tolk. |
| contracts/wrappers/wgram.JettonMinter.compile.ts | Points minter compile config to contracts/wgram/JettonMinter.tolk. |
| contracts/wrappers/jetton/JettonWallet.ts | Updates comment to reference wGRAM extension. |
| contracts/wgram-gas-report.json | Adds wGRAM gas-report output snapshot. |
| contracts/wgram-gas-report.config.ts | Renames gas-report test match and report name to wGRAM. |
| contracts/tests/wgram/wgram.spec.ts | Renames test suite and compilation targets to wGRAM. |
| contracts/tests/gas-report/wgram/wgram.spec.ts | Renames gas calibration suite paths/strings to wGRAM. |
| contracts/package.json | Renames yarn script wton-gas-report → wgram-gas-report. |
| contracts/contracts/wton/README.md | Removes old wTON README. |
| contracts/contracts/wgram/README.md | Adds new README for wGRAM (currently contains TON wording inconsistencies). |
| contracts/contracts/wgram/JettonWallet.tolk | Updates contract description/type/version string to wGRAM naming. |
| contracts/contracts/wgram/JettonMinter.tolk | Updates contract description/type/version string to wGRAM naming. |
| contracts/contracts/wgram/fees-management.tolk | Updates gas-report reference comment to wGRAM paths/command. |
Comments suppressed due to low confidence (2)
contracts/tests/wgram/wgram.spec.ts:18
- The test imports
WTON_MINT_OPCODE/WTON_WITHDRAW_EXCESS_OPCODEfromwrappers/wgram, which conflicts with the wGRAM naming and can be confusing. After introducingWGRAM_*exports, update the test to use the wGRAM-named constants.
contracts/tests/gas-report/wgram/wgram.spec.ts:13 - This gas-report test still uses
WTON_MINT_OPCODEeven though it now targets wGRAM. For consistency with the rebrand (and to reduce confusion when reading reports), switch to aWGRAM_MINT_OPCODEconstant once it exists, keepingWTON_*only as a compatibility alias if needed.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
patricios-space
left a comment
patricios-space
left a comment
There was a problem hiding this comment.
Shoul we also rename mentions of TON to Gram? The new name of the coin
There was a problem hiding this comment.
Github won't let me comment on lines not modified.
Some places where "ton" should be replaced by "gram":
TopUpTons
returns TON
a forced TON deposit
forwardTonAmount
hosted TON backing
Accept TONs
There was a problem hiding this comment.
I've left TopUpTons and forwardTonAmount for now as they are part of standard Jetton contracts - we can consider renaming that down the line.
No description provided.