fixup! feat(types): add custom types for authorities, plugins and asset

metaplex-foundation · Mar 7, 2024 · 3e741c6 · 3e741c6
1 parent 26460de
commit 3e741c6
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 47 deletions.
diff --git a/clients/js/src/hooked/pluginHelpers.ts b/clients/js/src/hooked/pluginHelpers.ts
@@ -1,9 +1,6 @@
 import {
-  Authority,
   Key,
   PluginHeader,
-  PluginRegistry,
-  PluginType,
   Plugin,
   getPluginSerializer,
   RegistryRecord,
@@ -21,48 +18,19 @@ export function formPluginHeader(
   };
 }
 
-export function formPluginRegistry({
-  pluginType,
-  offset,
-  authorities,
-}: {
-  pluginType: PluginType;
-  offset: bigint;
-  authorities: Authority[];
-}): Omit<PluginRegistry, 'publicKey' | 'header' | 'externalPlugins'> {
-  return {
-    key: Key.PluginRegistry,
-    registry: [
-      {
-        pluginType,
-        offset,
-        authorities,
-      },
-    ],
-  };
-}
-
-export function formPluginWithAuthorities({
-  authorities,
-  plugin,
-}: {
-  authorities: Authority[];
-  plugin: Plugin;
-}) {
-  return {
-    authorities,
-    plugin,
-  };
-}
-
 export function mapPluginFields(fields: Array<Record<string, any>>) {
   return fields.reduce((acc2, field) => ({ ...acc2, ...field }), {});
 }
 
-export function mapPlugin(
-  plugin: Plugin,
-  authorities: BaseAuthorities
-): PluginsList {
+export function mapPlugin({
+  plugin,
+  authorities,
+  offset,
+}: {
+  plugin: Exclude<Plugin, { __kind: 'Reserved' }>;
+  authorities: BaseAuthorities;
+  offset: bigint;
+}): PluginsList {
   const pluginKey = toWords(plugin.__kind)
     .toLowerCase()
     .split(' ')
@@ -71,6 +39,7 @@ export function mapPlugin(
   return {
     [pluginKey]: {
       authorities,
+      offset,
       ...('fields' in plugin ? mapPluginFields(plugin.fields) : {}),
     },
   };
@@ -79,17 +48,23 @@ export function mapPlugin(
 export function registryRecordsToPluginsList(
   registryRecords: RegistryRecord[],
   accountData: Uint8Array
-): PluginsList {
-  return registryRecords.reduce((acc, record) => {
+) {
+  return registryRecords.reduce((acc: PluginsList, record) => {
     const mappedAuthorities = mapAuthorities(record.authorities);
     const deserializedPlugin = getPluginSerializer().deserialize(
       accountData,
       Number(record.offset)
     )[0];
 
+    if (deserializedPlugin.__kind === 'Reserved') return acc;
+
     acc = {
       ...acc,
-      ...mapPlugin(deserializedPlugin, mappedAuthorities),
+      ...mapPlugin({
+        plugin: deserializedPlugin,
+        authorities: mappedAuthorities,
+        offset: record.offset,
+      }),
     };
 
     return acc;

diff --git a/clients/js/src/hooked/types.ts b/clients/js/src/hooked/types.ts
@@ -19,17 +19,16 @@ export type BaseAuthorities = {
 
 export type BasePlugin = {
   authorities: BaseAuthorities;
+  offset: bigint;
 };
 
-export type ReservedPlugin = BasePlugin;
 export type RoyaltiesPlugin = BasePlugin & Royalties;
 export type FreezePlugin = BasePlugin & Freeze;
 export type BurnPlugin = BasePlugin & Burn;
 export type TransferPlugin = BasePlugin & Transfer;
 export type UpdateDelegatePlugin = BasePlugin & UpdateDelegate;
 
 export type PluginsList = {
-  reserved?: ReservedPlugin;
   royalties?: RoyaltiesPlugin;
   freeze?: FreezePlugin;
   burn?: BurnPlugin;

diff --git a/clients/js/test/addAuthorityWithTestFetch.ts b/clients/js/test/addAuthorityWithTestFetch.ts
@@ -1,6 +1,5 @@
 import { generateSigner } from '@metaplex-foundation/umi';
 import test from 'ava';
-// import { base58 } from '@metaplex-foundation/umi/serializers';
 import {
   Asset,
   DataState,
@@ -70,6 +69,7 @@ test('TEST it can add an authority to a plugin TEST', async (t) => {
         pubkey: [delegateAddress.publicKey],
       },
       frozen: false,
+      offset: BigInt(118),
     },
   });
 });