refactor(utils): Replace GenRandomId with getRandomId

[susnux]

☑️ Resolves

• Use Typescript for correct type annotations
• Fix length problem (the old implementation sometimes did not yield the expected length, e.g. if a "short" random number was generated)¹
• Add unit test for the util

¹ Did some benchmarks: Just the method is now ~10% slower. But the old method delivered too short values for ~25% of all cases.

🏁 Checklist

• ⛑️ Tests are included or are not applicable
• 📘 Component documentation has been extended, updated or is not applicable
• 3️⃣ Backport to next requested with a Vue 3 upgrade

[ShGKme]

In the next branch I'd prefer to use useId: https://vuejs.org/api/composition-api-helpers.html#useid

It is Vue "native" way, and it's stable, unlinke Math.random().

The only problem is that it's uniq only in a single Vue package instance. For multiple apps on the page with multiple Vue it requires setting app.config.idPrefix in an app.

[susnux]

In the next branch I'd prefer to use useId: https://vuejs.org/api/composition-api-helpers.html#useid
It is Vue "native" way, and it's stable, unlinke Math.random().

For apps yes but for the library we can not be sure that the user has setup the id prefix correctly.

One alternative would be to provide a custom useId (for internal use!) that outputs:

const sanitizedAppName = String(appName).replaceAll(/[^0-9a-z-_]/ig, '')
return `${sanitizedAppName}-${useId()}`

This could be used in the library while apps of course would just use useId from Vue.

[susnux]

Simplified, what do you think?

[Antreesy]

Since you've merged it, please don't forget to replace GenRandomId, after backport is there: https://github.com/nextcloud-libraries/nextcloud-vue/pull/6454/files#diff-c5595ba3afc4a147c5e8b442fe46ed5f41af898aad982d3b732103fe0fedb9f3R149