Incorrect type for save in Store<T>

[matheus23]

In this example, save is defined as an async function:

iamap/examples/level-backed.js

Lines 74 to 81 in 06d4402

	save: async (value) => {
	// Save some arbitrary object to our store. When IAMap uses this it's saving a plain object
	// representation of an IAMap node. See IAMap#toSerializable() for information on that form.
	store.stats.saves++
	const { cid, bytes } = await store.encode(value)
	await store.backingDb.put(cid.toString(), bytes)
	return cid
	},

But the return type for save is supposed to be T, not Promise<T>:

iamap/interface.ts

Lines 1 to 7 in 06d4402

	// store using a link type `T`
	export interface Store<T> {
	save(node: any): T,
	load(id: T): any,
	isLink(link: T): boolean,
	isEqual(link1: T, link2: T): boolean,
	}

So I think the type should be changed to either Promise<T> or T | Promise<T>.

[rvagg]

good catch, should be fixed in a patch release, thanks

[github-actions]

🎉 This issue has been resolved in version 2.0.6 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀