Skip to content

fix: Allow null in the endowment:assets handlers return value #3056

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
GuillaumeRx merged 3 commits into from
Feb 3, 2025
Merged

fix: Allow null in the endowment:assets handlers return value #3056

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
37c6cb7
Allow `null` in assets handlers response
GuillaumeRx Feb 3, 2025
2655fa0
Update the structs
GuillaumeRx Feb 3, 2025
c0bc4d9
Update manifests
GuillaumeRx Feb 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/snaps-controllers/src/snaps/SnapController.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3689,7 +3689,7 @@ export class SnapController extends BaseController<
const { assets: requestedAssets } = requestedParams;

const filteredAssets = Object.keys(assets).reduce<
Record<CaipAssetType, FungibleAssetMetadata>
Record<CaipAssetType, FungibleAssetMetadata | null>
>((accumulator, assetType) => {
const castAssetType = assetType as CaipAssetType;
const isValid =
Expand Down
7 changes: 4 additions & 3 deletions packages/snaps-sdk/src/types/handlers/assets-conversion.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import {
string,
optional,
record,
nullable,
} from '@metamask/superstruct';
import { CaipAssetTypeStruct, type CaipAssetType } from '@metamask/utils';

Expand All @@ -17,7 +18,7 @@ export const AssetConversionStruct = object({
export const OnAssetsConversionResponseStruct = object({
conversionRates: record(
CaipAssetTypeStruct,
record(CaipAssetTypeStruct, AssetConversionStruct),
record(CaipAssetTypeStruct, nullable(AssetConversionStruct)),
),
});

Expand All @@ -40,11 +41,11 @@ export type OnAssetsConversionHandler = (
/**
* The response from the conversion query, containing rates about each requested asset pair.
*
* @property conversionRates - A nested object with two CAIP-19 keys that contains a conversion rate between the two keys.
* @property conversionRates - A nested object with two CAIP-19 keys that contains a conversion rate or null between the two keys.
*/
export type OnAssetsConversionResponse = {
conversionRates: Record<
CaipAssetType,
Record<CaipAssetType, AssetConversion>
Record<CaipAssetType, AssetConversion | null>
>;
};
7 changes: 4 additions & 3 deletions packages/snaps-sdk/src/types/handlers/assets-lookup.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import {
refine,
string,
record,
nullable,
} from '@metamask/superstruct';
import {
assert,
Expand Down Expand Up @@ -46,7 +47,7 @@ export const FungibleAssetMetadataStruct = object({
});

export const OnAssetsLookupResponseStruct = object({
assets: record(CaipAssetTypeStruct, FungibleAssetMetadataStruct),
assets: record(CaipAssetTypeStruct, nullable(FungibleAssetMetadataStruct)),
});

export type FungibleAssetMetadata = Infer<typeof FungibleAssetMetadataStruct>;
Expand All @@ -68,8 +69,8 @@ export type OnAssetsLookupHandler = (
/**
* The response from the query, containing metadata about each requested asset.
*
* @property assets - An object containing a mapping between the CAIP-19 key and a metadata object.
* @property assets - An object containing a mapping between the CAIP-19 key and a metadata object or null.
*/
export type OnAssetsLookupResponse = {
assets: Record<CaipAssetType, FungibleAssetMetadata>;
assets: Record<CaipAssetType, FungibleAssetMetadata | null>;
};
Loading