fix: Replace shortid package with nanoid package

[AkshatJawne]

Resolves #1922

Changes Implemented:

• Replaced shortid with nanoid, given that shortid is deprecated (Reran all tests to ensure that functionality upholds)

Info on Nanoid: https://github.com/ai/nanoid/blob/main/README.md

[AkshatJawne]

Looking into errors here

[mattrunyon]

Remove esmify from golden-layout dev deps and add babelify as a dev dependency.

In karma.conf.cjs remove the plugins line and add this. For some reason esmify isn't transforming node_modules even though it says it does (it might not understand the package.json of nanoid).

transform: [
  ['babelify', { presets: ['@babel/preset-env'], global: true }],
],

This seems to fix the issue by using babel to transpile the ESM before the bundling for the test.

[bmingles]

LGTM. Will wait for @mattrunyon to approve since he suggested the babelify change, and I'm less familiar here

[AkshatJawne]

Sounds good, thanks @bmingles

[mattrunyon]

There's a @types/shortid in the root package.json still. Remove it and run npm install again so it's removed from package-lock too.

[AkshatJawne]

Weird, I remember removing it? Although, I was having some issues when I was switching branches with the differences with my package.json so I clean installed a couple of times on different branches , something may have gotten messed up. Will fix this shortly and repush @mattrunyon