removing require() calls

[dynst]

Part of #1004

https://typescript-eslint.io/rules/no-require-imports

Ignoring the .js scripts, there are 3 call sites in TypeScript:

• cosmjs/packages/faucet/src/api/webserver.ts

  Line 3 in d708b6a

  import cors = require("@koa/cors");

• cosmjs/packages/crypto/src/random.ts

  Lines 17 to 20 in d708b6a

  	} catch {
  	try {
  	// eslint-disable-next-line @typescript-eslint/no-var-requires
  	const crypto = require("crypto");

• cosmjs/packages/socket/src/reconnectingsocket.spec.ts

  Lines 11 to 13 in d708b6a

  	try {
  	// eslint-disable-next-line @typescript-eslint/no-var-requires
  	exec = require("child_process").exec;

Are these absolutely needed?

[webmaster128]

This code is old. I would not be surprised if this can be rewritten these days. But not sure. Do you want to give it a try?

[dynst]

globalThis.crypto and the full Web Crypto API was added in Node.js 20 so everything there can probably be deleted, right. Nobody should need a Random class wrapper.

Unclear why "@koa/cors" has a completely different import style from the lines just above it and below it though, or what to do in the test.

cosmjs/packages/faucet/src/actions/start/start.ts

Line 1 in 3f8ab89

import cors = require("@koa/cors");

[webmaster128]

Unclear why "@koa/cors" has a completely different import style from the lines just above it and below it though

This is because even the latest readme suggests that import style. import cors from "@koa/cors"; compiles too. I'm trying to write a test for the Webserver to ensure it actually works.