Consistent type import

[basmasking]

When importing a type, we sometimes do it as import type { DataModel } from './types' and sometimes as import { DataModel } from './types'.

For readability and consistency, we want all the type imports to the first pattern (that includes the type in the import).

[basmasking]

There is an eslint rule we can use for detecting imports of types without the type statement. This rule is

"@typescript-eslint/consistent-type-imports": "error"

and needs to be added to the rules section of the eslint config. After it has been added, running npx eslint --fix will detect and automatically fix all the incorrect imports.