Update Token Metadata (#508)

Author: lorisleiva

.changeset/dull-jars-listen.md

@@ -0,0 +1,5 @@
+---
+'@metaplex-foundation/js': patch
+---
+
+Update Token Metadata program to v1.11.0

packages/js/package.json

@@ -49,7 +49,7 @@
     "@metaplex-foundation/mpl-candy-guard": "^0.3.0",
     "@metaplex-foundation/mpl-candy-machine": "^5.0.0",
     "@metaplex-foundation/mpl-candy-machine-core": "^0.1.2",
-    "@metaplex-foundation/mpl-token-metadata": "^2.8.6",
+    "@metaplex-foundation/mpl-token-metadata": "^2.11.0",
     "@noble/ed25519": "^1.7.1",
     "@noble/hashes": "^1.1.3",
     "@solana/spl-account-compression": "^0.1.8",

packages/js/src/plugins/nftModule/DelegateType.ts

@@ -14,10 +14,10 @@ export type TokenDelegateType =
   | 'UtilityV1'
   | 'StakingV1';
 export type MetadataDelegateType =
-  // | 'AuthorityV1'
+  // | 'AuthorityItemV1'
   | 'CollectionV1'
   // | 'UseV1'
-  | 'UpdateV1'
+  | 'DataV1'
   | 'ProgrammableConfigV1';
 
 const tokenDelegateRoleMap: Record<TokenDelegateType, TokenDelegateRole> = {
@@ -33,30 +33,33 @@ const metadataDelegateRoleMap: Record<
   MetadataDelegateType,
   MetadataDelegateRole
 > = {
-  // AuthorityV1: MetadataDelegateRole.Authority,
+  // AuthorityItemV1: MetadataDelegateRole.AuthorityItem,
   CollectionV1: MetadataDelegateRole.Collection,
   // UseV1: MetadataDelegateRole.Use,
-  UpdateV1: MetadataDelegateRole.Update,
+  DataV1: MetadataDelegateRole.Data,
   ProgrammableConfigV1: MetadataDelegateRole.ProgrammableConfig,
 };
 
 const metadataDelegateSeedMap: Record<MetadataDelegateRole, string> = {
-  [MetadataDelegateRole.Authority]: 'authority_delegate',
+  [MetadataDelegateRole.AuthorityItem]: 'authority_item_delegate',
   [MetadataDelegateRole.Collection]: 'collection_delegate',
   [MetadataDelegateRole.Use]: 'use_delegate',
-  [MetadataDelegateRole.Update]: 'update_delegate',
+  [MetadataDelegateRole.Data]: 'data_delegate',
   [MetadataDelegateRole.ProgrammableConfig]: 'programmable_config_delegate',
+  [MetadataDelegateRole.DataItem]: 'data_item_delegate',
+  [MetadataDelegateRole.CollectionItem]: 'collection_item_delegate',
+  [MetadataDelegateRole.ProgrammableConfigItem]: 'prog_config_item_delegate',
 };
 
 const delegateCustomDataMap: Record<
   TokenDelegateType | MetadataDelegateType,
   boolean
 > = {
   // Metadata.
-  // AuthorityV1: false,
+  // AuthorityItemV1: false,
   CollectionV1: false,
   // UseV1: false,
-  UpdateV1: false,
+  DataV1: false,
   ProgrammableConfigV1: false,
   // Token
   StandardV1: true,

packages/js/src/plugins/nftModule/operations/updateNft.ts

@@ -431,7 +431,9 @@ export const updateNftBuilder = (
               authorizationRules: auth.accounts.authorizationRules,
               authorizationRulesProgram: TOKEN_AUTH_RULES_ID,
             },
-            { updateArgs: { ...updateInstructionData, ...auth.data } },
+            {
+              updateArgs: { ...updateInstructionData, ...auth.data } as any,
+            },
             tokenMetadataProgram.address
           ),
           signers: [payer, ...auth.signers],
@@ -491,7 +493,7 @@ const toInstructionData = (
           };
         });
 
-  return {
+  return <Omit<UpdateArgs, 'authorityType' | 'authorizationData'>>{
     __kind: 'V1',
     newUpdateAuthority: input.newUpdateAuthority ?? null,
     data: {

packages/js/test/plugins/candyMachineV2Module/createCandyMachineV2.test.ts

@@ -20,7 +20,6 @@ import {
   CandyMachineV2,
   CandyMachineV2Program,
   CreateCandyMachineV2Input,
-  getCandyMachineV2UuidFromAddress,
   sol,
   toBigNumber,
   token,
@@ -61,7 +60,6 @@ test('[candyMachineV2Module] create with minimal input', async (t) => {
     version: 2,
     tokenMintAddress: null,
     collectionMintAddress: null,
-    uuid: getCandyMachineV2UuidFromAddress(candyMachine.address),
     price: spokSameAmount(sol(1.25)),
     symbol: '',
     sellerFeeBasisPoints: 500,
@@ -302,7 +300,6 @@ test('[candyMachineV2Module] create with whitelistMint settings', async (t) => {
 
   // Then a Candy Machine was created with ...
   await tc.assertSuccess(t, response.signature);
-  console.log(candyMachine.whitelistMintSettings?.discountPrice);
   spok(t, candyMachine, {
     $topic: 'Candy Machine',
     model: 'candyMachineV2',

packages/js/test/plugins/nftModule/createNft.test.ts

@@ -1,15 +1,12 @@
 import {
-  createCreateMasterEditionV3Instruction,
-  createCreateMetadataAccountV2Instruction,
   TokenStandard,
   UseMethod,
 } from '@metaplex-foundation/mpl-token-metadata';
+import { AccountState } from '@solana/spl-token';
 import { Keypair } from '@solana/web3.js';
 import spok, { Specifications } from 'spok';
 import test, { Test } from 'tape';
-import { AccountState } from '@solana/spl-token';
 import {
-  amman,
   assertThrows,
   createCollectionNft,
   createWallet,
@@ -23,14 +20,7 @@ import {
   assertCollectionHasSize,
   assertRefreshedCollectionHasSize,
 } from './helpers';
-import {
-  Nft,
-  NftWithToken,
-  toBigNumber,
-  token,
-  toMetaplexFile,
-  TransactionBuilder,
-} from '@/index';
+import { Nft, NftWithToken, toBigNumber, token, toMetaplexFile } from '@/index';
 
 killStuckProcess();
 
@@ -472,85 +462,3 @@ const minimalInput = () => ({
   name: 'My NFT',
   sellerFeeBasisPoints: 200,
 });
-
-/*
- * Regression test.
- * @see https://github.com/metaplex-foundation/metaplex-program-library/issues/383
- */
-test('[nftModule] it works when we give an explicit payer for the create metadata ix only', async (t: Test) => {
-  // Given we have everything we need to create a Metadata account.
-  const mx = await metaplex();
-  const mint = Keypair.generate();
-  const metadata = mx.nfts().pdas().metadata({ mint: mint.publicKey });
-  const edition = mx.nfts().pdas().masterEdition({ mint: mint.publicKey });
-  const { uri } = await mx.nfts().uploadMetadata({ name: 'Metadata Name' });
-
-  const data = {
-    name: 'My NFT',
-    symbol: 'MNFT',
-    sellerFeeBasisPoints: 10,
-    uri,
-    creators: [
-      {
-        address: mx.identity().publicKey,
-        share: 100,
-        verified: false,
-      },
-    ],
-    collection: null,
-    uses: null,
-  };
-
-  // And an explicit payer account that is only used to pay for the Metadata account storage.
-  const explicitPayer = Keypair.generate();
-  await amman.airdrop(mx.connection, explicitPayer.publicKey, 1);
-
-  // When we assemble that transaction.
-  const tx = TransactionBuilder.make()
-    .add(
-      await mx
-        .tokens()
-        .builders()
-        .createTokenWithMint(
-          { initialSupply: token(1), mint },
-          { payer: mx.identity() }
-        )
-    )
-    .add({
-      instruction: createCreateMetadataAccountV2Instruction(
-        {
-          metadata,
-          mint: mint.publicKey,
-          mintAuthority: mx.identity().publicKey,
-          payer: explicitPayer.publicKey,
-          updateAuthority: mx.identity().publicKey,
-        },
-        { createMetadataAccountArgsV2: { data, isMutable: false } }
-      ),
-      signers: [explicitPayer],
-    })
-    .add({
-      instruction: createCreateMasterEditionV3Instruction(
-        {
-          edition,
-          mint: mint.publicKey,
-          updateAuthority: mx.identity().publicKey,
-          mintAuthority: mx.identity().publicKey,
-          payer: explicitPayer.publicKey,
-          metadata,
-        },
-        {
-          createMasterEditionArgs: { maxSupply: 0 },
-        }
-      ),
-      signers: [explicitPayer],
-    });
-
-  // And send it with confirmation.
-  await mx.rpc().sendAndConfirmTransaction(tx);
-
-  // Then the transaction succeeded and the NFT was created.
-  const nft = await mx.nfts().findByMint({ mintAddress: mint.publicKey });
-  t.equal(nft.name, 'My NFT');
-  t.equal(nft.metadataAddress.toBase58(), metadata.toBase58());
-});