Inconsistent use of T[] and Array<T> for arrays

[ckcherry23]

What feature(s) would you like to see in RepoSense

Standardize the style used for arrays to be T[] or Array<T>, based on the complexity of T.

Is the feature request related to a problem?

During the TypeScript migration of the front-end, both of these styles were used interchangeably. Since the 2 styles are functionally equivalent, we can use the same style consistently across the codebase to make it easier for developers to read and understand array types.

If possible, describe the solution

• Add a lint rule to enforce that the same array style is used across the code base. This should be in the global .eslintrc.json.

  We can consider using the array-simple option as this allows us to use the shorter, modern T[] or readonly T[] for simple types (i.e. types that are just primitive names or type references), and Array<T> or ReadonlyArray<T> for all other types (union types, intersection types, object types, function types, etc) where the shorter syntax could get confusing.

{
  "rules": {
    "@typescript-eslint/array-type": ["error", {
        "default": "array-simple",
        "readonly": "array-simple"
      }
    ]
  }
}

• Make edits to the codebase such that the newly introduced lint rule is followed. Most of the changes will be in the props and data() sections of the .vue files.

If applicable, describe alternatives you've considered

We could also consider using the array or generic options for the lint rule if we want to stick to a single style at all times, no matter the complexity of the type.

Additional context

Discussed in #1976