fix: Account names should use natural sort order

[alfiedotwtf]

When you have accounts Account 1 ... all the way to Account 15, the list ends up looking like this:

• Account 1
• Account 10
• Account 11
• ...
• Account 15
• Account 2
• Account 3

etc.

This change makes the accounts list look like this:

• Account 1
• Account 2
• ...
• Account 9
• Account 10
• Account 11
• ...

[helciofranco]

Good catch! Could we apply these two suggestions?

[helciofranco]

can we move this to AccountService.getAccounts?
so we fix the issue at the best place where the incorrect order is being returned

fuels-wallet/packages/app/src/systems/Account/services/account.ts

Lines 66 to 70 in 4fe865a

	static async getAccounts() {
	return db.transaction('r', db.accounts, async () => {
	return db.accounts.toCollection().sortBy('name');
	});
	}

you'll need to use toArray() instead of toCollection() in order to run .sort() on it (and that sortBy('name') can be removed).

I think this approach might work:

static async getAccounts() {
  return db.transaction('r', db.accounts, async () => {
    const accounts = await db.accounts.toArray();
    return accounts.sort((a, b) => a.name.localeCompare(b.name, undefined, { numeric: true, sensitivity: 'base' }));
  });
}

[helciofranco]

also, could you add a short and clear changelog for this fix?

just run pnpm changeset and follow the instruction (it looks like a patch fix)