This makes imports flexible to .tsx file conversion.

Inline the functions and use individual variables for each part of the
two-tuple returned by Object.entries().

Non-exhaustive attempt to add types. Note that with noImplicitAny
disabled, there is still a decent chunk of untyped variables.

- replace generic 'data|state' with 'card|resource|group'
- replace generic 'error' with 'data fetch error'
- replace 'modal' with 'modal resource'

These "worked" in normal JS under expected usage but raise errors by the
TypeScript compiler which generally catches unhandled edge cases. Each
has been addressed with inline reasoning.

The alternative of turning off noUnusedLocals is not desirable since we
still want to catch unused imports which is done by the same rule. This
is the same reason why no-unused-vars is enabled in ESLint with
exceptions in the lines below each change.