Skip to content

chore: rename c-token/cmint to light-token/light mint in comments#2299

Merged
SwenSchaeferjohann merged 1 commit intomainfrom
chore/rename-light-token
Feb 19, 2026
Merged

chore: rename c-token/cmint to light-token/light mint in comments#2299
SwenSchaeferjohann merged 1 commit intomainfrom
chore/rename-light-token

Conversation

@tilo-14
Copy link
Member

@tilo-14 tilo-14 commented Feb 18, 2026

Summary

  • Rename old terminology (c-token, cmint, compressed mint, CMint, CToken, CompressedMint) to new terminology (light-token, light mint) in JSDoc, inline comments, and error strings
  • 38 files in js/compressed-token/src/v3/, 243 lines changed
  • Code identifiers (CompressedMint, CMint, ctokenConfig, etc.) left unchanged

Test plan

  • Build passes (LIGHT_PROTOCOL_VERSION=V2 pnpm run build)
  • No old terms remain in comments/errors (verified via grep)
  • All code identifiers intact (verified via grep)

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 18, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (45)
  • js/compressed-token/src/v3/actions/create-associated-ctoken.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/create-ata-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/create-mint-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/decompress-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/decompress-mint.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/get-or-create-ata-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/load-ata.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/mint-to-compressed.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/mint-to-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/mint-to.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/transfer-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/unwrap.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/update-metadata.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/update-mint.ts is excluded by none and included by none
  • js/compressed-token/src/v3/actions/wrap.ts is excluded by none and included by none
  • js/compressed-token/src/v3/ata-utils.ts is excluded by none and included by none
  • js/compressed-token/src/v3/derivation.ts is excluded by none and included by none
  • js/compressed-token/src/v3/get-account-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/get-associated-token-address-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/get-mint-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/create-associated-ctoken.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/create-ata-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/create-decompress-interface-instruction.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/create-load-accounts-params.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/create-mint.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/decompress-mint.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/mint-to-compressed.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/mint-to-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/mint-to.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/transfer-interface.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/unwrap.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/update-metadata.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/update-mint.ts is excluded by none and included by none
  • js/compressed-token/src/v3/instructions/wrap.ts is excluded by none and included by none
  • js/compressed-token/src/v3/layout/layout-mint.ts is excluded by none and included by none
  • js/compressed-token/src/v3/layout/layout-token-metadata.ts is excluded by none and included by none
  • js/compressed-token/src/v3/layout/layout-transfer2.ts is excluded by none and included by none
  • js/compressed-token/src/v3/unified/index.ts is excluded by none and included by none
  • js/compressed-token/tests/e2e/create-mint-interface.test.ts is excluded by none and included by none
  • js/compressed-token/tests/e2e/decompress2.test.ts is excluded by none and included by none
  • js/compressed-token/tests/e2e/get-mint-interface.test.ts is excluded by none and included by none
  • js/compressed-token/tests/e2e/load-ata-unified.test.ts is excluded by none and included by none
  • js/compressed-token/tests/unit/layout-mint.test.ts is excluded by none and included by none
  • js/compressed-token/tests/unit/serde.test.ts is excluded by none and included by none
  • js/compressed-token/tests/unit/unified-guards.test.ts is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/rename-light-token

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

const isSplDestination = splInterfaceInfo !== undefined;

// Get compressed token accounts
// Get light-token accounts
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
// Get light-token accounts
// Get compressed light-token accounts (cold balance)

*
* Behavior depends on `wrap` parameter:
* - wrap=false (standard): Decompress light-tokens to the target ATA.
* ATA can be SPL (via pool), T22 (via pool), or light-token (direct).
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
* ATA can be SPL (via pool), T22 (via pool), or light-token (direct).
* ATA can be SPL (via interface PDA), T22 (via interface PDA), or light-token (direct).

* - wrap=false (standard): Decompress compressed tokens to the target ATA type
* (SPL ATA via pool, T22 ATA via pool, or c-token ATA direct)
* - wrap=true (unified): Wrap SPL/T22 + decompress all to c-token ATA
* - wrap=false (standard): Decompress light-tokens to the target ATA type
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
* - wrap=false (standard): Decompress light-tokens to the target ATA type
* - wrap=false (standard): Decompress compressed light-tokens to the target ATA type

Comment on lines +335 to +336
* (SPL ATA via pool, T22 ATA via pool, or light-token ATA direct)
* - wrap=true (unified): Wrap SPL/T22 + decompress all to light-token ATA
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
* (SPL ATA via pool, T22 ATA via pool, or light-token ATA direct)
* - wrap=true (unified): Wrap SPL/T22 + decompress all to light-token ATA
* (SPL ATA via interface PDA, T22 ATA via interface PDA, or light-token ATA direct)
* - wrap=true (unified): Wrap SPL/T22 + decompress all to light-token ATA

}

// 4. Decompress compressed tokens to c-token ATA
// 4. Decompress light-tokens to light-token ATA
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
// 4. Decompress light-tokens to light-token ATA
// 4. Decompress compressed light-tokens (cold balance) to light-token associated token account (hot balance)

@tilo-14 tilo-14 force-pushed the chore/rename-light-token branch 3 times, most recently from c6c51aa to 8fa586a Compare February 19, 2026 00:40
Rename terminology in JSDoc, comments, and error strings:
- c-token → light-token
- cmint/CMint → light mint
- compressed mint → light mint (where referring to the type)
- compressed/decompressed (state) → preserved as-is
- ATA → associated token account (expanded)
- via pool → via interface PDA
- hot/cold terminology for token accounts and mints
@tilo-14 tilo-14 force-pushed the chore/rename-light-token branch from 8fa586a to 709caa8 Compare February 19, 2026 01:04
@SwenSchaeferjohann SwenSchaeferjohann merged commit eb3c45c into main Feb 19, 2026
15 checks passed
tilo-14 pushed a commit that referenced this pull request Feb 25, 2026
… comments

Continuation of #2299. Renames old terminology (ctoken, CToken, cmint,
compressed mint, token pool) to new names (light-token, Light Token,
light-mint, Light Mint, SPL interface PDA) across sdk-libs/ and
sdk-tests/ comments and documentation.

Only comments, doc strings, and markdown files are changed — no code
identifiers, variable names, or module paths were modified.
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.

2 participants