Creating a 1.13.3 patch for backwards compatibility.

README.md

@@ -16,11 +16,11 @@
 
 ## Getting Started
 
-The packages below can be use to interact with Token Metadata program.
+The packages below can be used to interact with Token Metadata program.
 
 ### TypeScript
 ```sh
-npm install @metaplex-foundation/mpl-token-metadata@alpha
+npm install @metaplex-foundation/mpl-token-metadata
 ```
 
 [See typedoc documentation](https://mpl-token-metadata-js-docs.vercel.app/).
@@ -30,13 +30,13 @@ npm install @metaplex-foundation/mpl-token-metadata@alpha
 cargo add mpl-token-metadata
 ```
 
-[See crate documentation](https://docs.rs/mpl-token-metadata/1.13.1/mpl_token_metadata/).
+[See crate documentation](https://docs.rs/mpl-token-metadata/1.13.3/mpl_token_metadata/).
 
 ## Building
 
 From the root directory of the repository:
 
-- Install the required packges:
+- Install the required packages:
 ```sh
 pnpm install
 ```
@@ -50,7 +50,7 @@ This will create the program binary at `<ROOT>/programs/.bin`
 
 ## Testing
 
-Token Metadata includes two set of tests: BPF and TypeScript.
+Token Metadata includes two sets of tests: BPF and TypeScript.
 
 ### BPF
 
@@ -80,7 +80,7 @@ pnpm build && pnpm test
 
 ## Documentation
 
-Full documentation for Token Metadata can be found [here](https://docs.metaplex.com/programs/token-metadata/).
+Full documentation for Token Metadata can be found here [developers.metaplex.com/token-metadata](https://developers.metaplex.com/token-metadata). 
 
 ## Security
 

idls/token_metadata.json

@@ -7731,7 +7731,7 @@
   "metadata": {
     "origin": "shank",
     "address": "metaqbxxUerdq28cj1RbAWkYQm3ybzjb6a8bt518x1s",
-    "binaryVersion": "0.2.0",
-    "libVersion": "0.2.0"
+    "binaryVersion": "0.4.2",
+    "libVersion": "0.4.2"
   }
-}
+}

programs/token-metadata/program/Cargo.toml

@@ -1,38 +1,44 @@
 [package]
-name = "token_metadata"
-version = "1.13.3"
-description = "Metaplex Metadata"
 authors = ["Metaplex Developers <[email protected]>"]
-repository = "https://github.com/metaplex-foundation/metaplex-program-library"
-license-file = "../../../LICENSE"
+description = "Metaplex Metadata"
 edition = "2021"
+license-file = "../../../LICENSE"
+name = "token_metadata"
 readme = "README.md"
+repository = "https://github.com/metaplex-foundation/metaplex-program-library"
+version = "1.13.3"
 
 [features]
 no-entrypoint = []
-test-bpf = []
 serde-feature = ["serde", "serde_with"]
+test-bpf = []
 
 [dependencies]
 arrayref = "0.3.6"
 borsh = "0.9.3"
-mpl-token-auth-rules = { version = "=1.4.3-beta.1", features = ["no-entrypoint"] }
+mpl-token-auth-rules = { version = "=1.4.3-beta.1", features = [
+  "no-entrypoint",
+] }
 mpl-token-metadata-context-derive = { version = "0.3.0", path = "../macro" }
 mpl-utils = { version = "0.3.1" }
 num-derive = "0.3"
 num-traits = "0.2"
 serde = { version = "1.0.149", optional = true }
 serde_with = { version = "1.14.0", optional = true }
-shank = "0.2.0"
+shank = "0.4.2"
 solana-program = ">= 1.14.13, < 1.17"
+spl-associated-token-account = { version = ">= 1.1.3, < 3.0", features = [
+  "no-entrypoint",
+] }
 spl-token = { version = ">= 3.5.0, < 5.0", features = ["no-entrypoint"] }
-spl-associated-token-account = { version = ">= 1.1.3, < 3.0", features = ["no-entrypoint"] }
 thiserror = "1.0"
 
 [dev-dependencies]
 async-trait = "0.1.64"
 rmp-serde = "1.1.1"
-rooster = { git = "https://github.com/metaplex-foundation/rooster", features = ["no-entrypoint"] }
+rooster = { git = "https://github.com/metaplex-foundation/rooster", features = [
+  "no-entrypoint",
+] }
 serde = { version = "1.0.147", features = ["derive"] }
 solana-program-test = ">= 1.14.13, < 1.17"
 solana-sdk = ">= 1.14.13, < 1.17"

programs/token-metadata/program/src/state/edition.rs

@@ -41,7 +41,15 @@ impl TokenMetadataAccount for Edition {
     }
 
     fn size() -> usize {
-        MAX_EDITION_LEN
+        0
+    }
+
+    fn pad_length(buf: &mut Vec<u8>) -> Result<(), MetadataError> {
+        let padding_length = MAX_EDITION_LEN
+            .checked_sub(buf.len())
+            .ok_or(MetadataError::NumericalOverflowError)?;
+        buf.extend(vec![0; padding_length]);
+        Ok(())
     }
 }
 

programs/token-metadata/program/src/state/master_edition.rs

@@ -88,7 +88,15 @@ impl TokenMetadataAccount for MasterEditionV2 {
     }
 
     fn size() -> usize {
-        MAX_MASTER_EDITION_LEN
+        0
+    }
+
+    fn pad_length(buf: &mut Vec<u8>) -> Result<(), MetadataError> {
+        let padding_length = MAX_MASTER_EDITION_LEN
+            .checked_sub(buf.len())
+            .ok_or(MetadataError::NumericalOverflowError)?;
+        buf.extend(vec![0; padding_length]);
+        Ok(())
     }
 }
 
@@ -148,7 +156,15 @@ impl TokenMetadataAccount for MasterEditionV1 {
     }
 
     fn size() -> usize {
-        MAX_MASTER_EDITION_LEN
+        0
+    }
+
+    fn pad_length(buf: &mut Vec<u8>) -> Result<(), MetadataError> {
+        let padding_length = MAX_MASTER_EDITION_LEN
+            .checked_sub(buf.len())
+            .ok_or(MetadataError::NumericalOverflowError)?;
+        buf.extend(vec![0; padding_length]);
+        Ok(())
     }
 }
 

programs/token-metadata/program/src/state/metadata.rs

@@ -14,7 +14,7 @@ pub const MAX_SYMBOL_LENGTH: usize = 10;
 
 pub const MAX_URI_LENGTH: usize = 200;
 
-pub const MAX_METADATA_LEN: usize = 1 // key 
+pub const MAX_METADATA_LEN: usize = 1 // key
 + 32             // update auth pubkey
 + 32             // mint pubkey
 + MAX_DATA_SIZE
@@ -335,7 +335,15 @@ impl TokenMetadataAccount for Metadata {
     }
 
     fn size() -> usize {
-        MAX_METADATA_LEN
+        0
+    }
+
+    fn pad_length(buf: &mut Vec<u8>) -> Result<(), MetadataError> {
+        let padding_length = MAX_METADATA_LEN
+            .checked_sub(buf.len())
+            .ok_or(MetadataError::NumericalOverflowError)?;
+        buf.extend(vec![0; padding_length]);
+        Ok(())
     }
 }
 
@@ -484,7 +492,7 @@ mod tests {
     }
 
     #[test]
-    fn fail_to_deserialize_metadata_with_wrong_size() {
+    fn successfully_deserialize_metadata_with_different_size() {
         let expected_metadata = expected_pesky_metadata();
 
         let mut buf = Vec::new();
@@ -507,10 +515,9 @@ mod tests {
             1_000_000_000,
         );
 
-        // `from_account_info` should not succeed because this account is not owned
-        // by `token-metadata` program.
-        let error = Metadata::from_account_info(&account_info).unwrap_err();
-        assert_eq!(error, MetadataError::DataTypeMismatch.into());
+        let md = Metadata::from_account_info(&account_info).unwrap();
+        assert_eq!(md.key, Key::MetadataV1);
+        assert_eq!(md, expected_metadata);
     }
 
     #[test]

programs/token-metadata/program/src/utils/fee.rs

@@ -3,7 +3,7 @@ use solana_program::{
     sysvar::Sysvar,
 };
 
-use crate::state::{fee::CREATE_FEE, Metadata, TokenMetadataAccount, METADATA_FEE_FLAG_INDEX};
+use crate::state::{fee::CREATE_FEE, MAX_METADATA_LEN, METADATA_FEE_FLAG_INDEX};
 
 #[repr(C)]
 #[derive(Debug, Clone, Copy)]
@@ -16,7 +16,7 @@ pub(crate) fn levy(args: LevyArgs) -> ProgramResult {
     // Fund metadata account with rent + Metaplex fee.
     let rent = Rent::get()?;
 
-    let fee = CREATE_FEE + rent.minimum_balance(Metadata::size());
+    let fee = CREATE_FEE + rent.minimum_balance(MAX_METADATA_LEN);
 
     invoke(
         &solana_program::system_instruction::transfer(

programs/token-metadata/program/src/utils/mod.rs

@@ -38,7 +38,7 @@ use crate::{
     error::MetadataError,
     state::{
         Edition, Key, MasterEditionV2, Metadata, TokenMetadataAccount, TokenStandard,
-        MAX_NAME_LENGTH, MAX_SYMBOL_LENGTH, MAX_URI_LENGTH,
+        MAX_METADATA_LEN, MAX_NAME_LENGTH, MAX_SYMBOL_LENGTH, MAX_URI_LENGTH,
     },
 };
 
@@ -199,7 +199,7 @@ pub(crate) fn close_program_account<'a>(
     let rent_lamports = match key {
         // Metadata accounts could have fees stored, so we only want to withdraw
         // the actual rent lamport amount.
-        Key::MetadataV1 => rent.minimum_balance(Metadata::size()),
+        Key::MetadataV1 => rent.minimum_balance(MAX_METADATA_LEN),
         // Other accounts the rent is just the current lamport balance.
         _ => account_info.lamports(),
     };
@@ -313,7 +313,7 @@ mod tests {
         let corrupted_data = pesky_data();
 
         let metadata: Metadata =
-            try_from_slice_checked(corrupted_data, Key::MetadataV1, MAX_METADATA_LEN).unwrap();
+            try_from_slice_checked(corrupted_data, Key::MetadataV1, 0).unwrap();
 
         assert_eq!(metadata, expected_metadata);
     }